8000 Settings: Add proxies key by SchoolGuy · Pull Request #2989 · cobbler/cobbler · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Merged
merged 1 commit into from
Mar 23, 2022
Merged

Settings: Add proxies key #2989

merged 1 commit into from
Mar 23, 2022

Conversation

SchoolGuy
Copy link
Member

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

@SchoolGuy SchoolGuy requested a review from a team March 23, 2022 08:12
@SchoolGuy SchoolGuy added this to the v3.4.0 milestone Mar 23, 2022
@SchoolGuy SchoolGuy force-pushed the feature/uyuni-support branch from b607b9f to b6875a8 Compare March 23, 2022 08:38
@codecov
Copy link
codecov bot commented Mar 23, 2022

Codecov Report

Merging #2989 (932ae76) into main (873cb14) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            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              
Impacted Files Coverage Δ
cobbler/settings/__init__.py 89.71% <100.00%> (ø)
cobbler/settings/migrations/V3_4_0.py 92.85% <100.00%> (-0.48%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 873cb14...932ae76. Read the comment docs.

Copy link
Member
@meaksh meaksh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thanks!

@SchoolGuy
Copy link
Member Author

I will wait with merging this until it was tested by @nodeg and me. Maybe I have forgotten something the CI didn't catch.

Copy link
Member
@nodeg nodeg left a 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>
@SchoolGuy SchoolGuy force-pushed the feature/uyuni-support branch from 9c0b143 to 932ae76 Compare March 23, 2022 12:58
@SchoolGuy SchoolGuy merged commit 9bebebd into main Mar 23, 2022
@SchoolGuy SchoolGuy deleted the feature/uyuni-support branch March 23, 2022 14:20
SchoolGuy added a commit that referenced this pull request Apr 19, 2022
This backports:

- #2972
- #2973
- #2974
- #2976
- #2985
- #2986
- #2989
- #2990
- #2991
- #2996
- #3008
- #3009
- #3017

The exact content of the patches can please be taken from the Pull Requests on
GitHub.
SchoolGuy added a commit that referenced this pull request Apr 20, 2022
This backports:

- #2973
- #2974
- #2976
- #2985
- #2986
- #2989
- #2990
- #2991
- #2996
- #3008
- #3009
- #3017

The exact content of the patches can please be taken from the Pull Requests on
GitHub.
SchoolGuy added a commit that referenced this pull request Apr 20, 2022
This backports:

- #2973
- #2974
- #2976
- #2985
- #2986
- #2989
- #2990
- #2991
- #2996
- #3008
- #3009
- #3017

The exact content of the patches can please be taken from the Pull Requests on
GitHub.
SchoolGuy added a commit that referenced this pull request Apr 20, 2022
This backports:

- #2973
- #2974
- #2976
- #2985
- #2986
- #2989
- #2990
- #2991
- #2996
- #3008
- #3009
- #3017

The exact content of the patches can please be taken from the Pull Requests on
GitHub.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Uyuni Proxy Support
3 participants
0