-
Notifications
You must be signed in to change notification settings - Fork 107
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
worker: fix OSBUILD_SOURCES_CURL_SSL_CA_CERT variable #4423
Conversation
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.
Why is this PR introducing a second environment variable that isn't used elsewhere? It's not mentioned in the title, description, or commit.
Ah that was a leftover, I am working on this code in a different branch I was splitting git hunk. Fixed. |
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.
Ok, thank you.
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.
Good catch. How long has that been there?
Thanks!
Some testing would be nice for this. Could you add one in a follow-up? |
How exactly I should be testing this? I just verified that this variable is properly used in This code only passes the variable into the worker, as long as there is no typo it will work. I believe the cert is enrolled at the OS level in If you mean some kind of end to end test, this could be tricky. |
This looks like a copy and paste error.
The variable name was not correct:
https://github.com/osbuild/osbuild/blob/4b4b5cea950910ed9e1bc9d2fc1c618e0596b2b7/sources/org.osbuild.curl#L14