-
-
Notifications
You must be signed in to change notification settings - Fork 650
Settings: Add proxies key #2989
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
Conversation
b607b9f
to
b6875a8
Compare
Codecov Report
@@ Coverage Diff @@
## main #2989 +/- ##
==========================================
- Coverage 62.03% 62.02% -0.01%
==========================================
Files 102 102
Lines 14428 14427 -1
==========================================
- Hits 8950 8949 -1
Misses 5478 5478
Continue to review full report at Codecov.
|
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, Thanks!
I will wait with merging this until it was tested by @nodeg and me. Maybe I have forgotten something the CI didn't catch. |
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. Tested it locally in the special 3.3.2 sumaform test environment from @meaksh. I left some small suggestions.
This commit introduces first-level support for Uyuni Proxies. The only thing missing for this was to allow a new settings key called "proxies". The name is not perfect for the moment, but changing the name to a more fitting one will be done in a future version to make the changes minimal for both Uyuni and Cobbler users alike. More explanation for this feature of Uyuni can be found the linked document in "cobbler-conf.rst". Co-authored-by: Dominik Gedon <dgedon@suse.de>
9c0b143
to
932ae76
Compare
This commit introduces first-level support for Uyuni Proxies. The only thing
missing for this was to allow a new settings key called "proxies". The name is
not perfect for the moment, but changing the name to a more fitting one will be
done in a future version to make the changes minimal for both Uyuni and Cobbler
users alike.
More explanation for this feature of Uyuni can be found the linked document in
"cobbler-conf.rst".
Fixes #2988