-
-
Notifications
You must be signed in to change notification settings - Fork 249
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Extending keyserver timeout to make alpine build more forgiving #3520
Extending keyserver timeout to make alpine build more forgiving #3520
Conversation
Right now, about 10% of alpine linux x64 builds fail due to timeouts when requesting keys from the keyserver. The hope is that these timeouts are just temporary slowness due to server load, and that an extended timeout is the correct fix. Signed-off-by: Adam Farley <adfarley@redhat.com>
This PR may fix this issue: #3518 |
And I'm asking the original committer for a review, to make sure this isn't a deliberate duplication. Signed-off-by: Adam Farley <adfarley@redhat.com>
I noticed that the relevant code is duplicated. I don't see why this should be needed, so I'm removing it and tagging the original comitter to make sure it's not duplicated for a reason. Also, I added a temporary echo cmd "Proof of timeout change." to make absolutely sure I'm modifying the right code here. I think I am, because it's preceded by the only instance of .gpg-temp creation that's followed by a pid ($$), which the output indicates is the thing that gets created prior to failure. |
Ok, we're definitely modifying the right code. Removing the echo cmd now. |
Signed-off-by: Adam Farley <adfarley@redhat.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For clarification and historic reference we're talking about an issue with the GPG signature check process on the ALSA download which is giving this error, right?
18:05:42 GNUPGHOME=/tmp/.gpg-temp.194
18:05:42 gpg: keybox '/tmp/.gpg-temp.194/pubring.kbx' created
18:06:51 gpg: keyserver receive failed: Operation timed out
I'm somewhat surprised that that's timing out and I worry it's part of a more severe problem - if it hasn't git something after 50 seconds I'm not sure it'll get it after a longer time. How frequently has this occurred?
EDIT: Hmmm https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-alpine-linux-x64-temurin/278/consoleFull seems to show it taking about 50 seconds - I wonder what's making it so slow ...
18:05:51 gpg: keybox '/tmp/.gpg-temp.192/pubring.kbx' created
18:06:40 gpg: /tmp/.gpg-temp.192/trustdb.gpg: trustdb created
Based on #3518 (comment) could you add an |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I noticed that the relevant code is duplicated. I don't see why this should be needed, so I'm removing it and tagging the original comitter to make sure it's not duplicated for a reason.
Since you asked me directly elsewhere I conform that yeah the de-duplication of this SGTM 👍🏻
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Yup, that's the one.
Around 10% of the time. No apparent pattern.
This is slightly unclear, as the line where I had the "echo" is after the gpg recieve (gpg ... --recv-keys ...). I'm putting the uptime command into the script before the recv command. Do let me know if I've read that wrong. |
Yeah that's fine with me :-) |
Right now, about 10% of alpine linux x64 builds fail due to timeouts when requesting keys from the keyserver.
Example: https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-alpine-linux-x64-temurin/277
The hope is that these timeouts are just temporary slowness due to server load, and that an extended timeout is the correct fix.
Test run: https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-alpine-linux-x64-temurin/279/