Add repository enable/disable feature to repo_manager domain - #5176
Open
richardhenwood-dell wants to merge 1 commit into
Open
Add repository enable/disable feature to repo_manager domain#5176richardhenwood-dell wants to merge 1 commit into
richardhenwood-dell wants to merge 1 commit into
Conversation
richardhenwood-dell
marked this pull request as draft
September 8, 2026 04:03
Collaborator
Author
|
sorry - need to fix the lint issues before requesting a review. |
richardhenwood-dell
force-pushed
the
repo-manager-rocky
branch
2 times, most recently
from
September 8, 2026 10:42
9500502 to
6e42548
Compare
Collaborator
Author
|
It turns out this code is not suitable. I'm re-working it. |
This feature allows administrators to selectively enable or disable individual OS repositories in repo_manager_config.yml, providing storage optimization, bandwidth savings, and security control. Changes: - Add 'enabled' boolean field to repository configuration schema (default: true) - Update repository processing logic to skip disabled repositories - Enhance catalog resolution to respect repository enablement status - Add comprehensive documentation and examples - Implement test coverage for enablement functionality Files modified: - src/repo_manager/input/repo_manager_config.yml - src/repo_manager/plugins/module_utils/input_validation/schema/repo_manager_config.json - src/repo_manager/plugins/module_utils/input_validation/validators/repo_manager_config.py - src/repo_manager/plugins/module_utils/repo_manager/repo_settings.py - src/repo_manager/plugins/module_utils/repo_manager/config.py - src/repo_manager/plugins/module_utils/repo_manager/catalog_resolver.py - src/repo_manager/docs/content-configuration-guide.md - test/repo_manager/library/messages/repo_manager_msgs.py - test/repo_manager/fvt/policy/test_repo_enablement.py (new) Generated with [Devin](https://devin.ai) Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
richardhenwood-dell
force-pushed
the
repo-manager-rocky
branch
from
September 8, 2026 13:35
6e42548 to
5474645
Compare
richardhenwood-dell
marked this pull request as ready for review
September 9, 2026 03:22
Collaborator
Author
|
@Nagachandan-P @snarthan - do you have feedback on this PR? |
Nagachandan-P
approved these changes
Sep 9, 2026
Collaborator
Author
|
there is an alternative here that might be better: |
Collaborator
|
As discussed, having the enable/disable capability at the individual repository level is a good enhancement. However, since we are currently using DNF caching in the code to download specific packages, there could be some impact on that workflow. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
RE: #4993
This feature allows administrators to selectively enable or disable individual OS repositories in repo_manager_config.yml, providing storage optimization, bandwidth savings, and security control.
Changes:
Files modified:
Generated with Devin
Issues addressed by this Pull Request
Addresses #4993
Description of the Solution
Add a new field in
repo_manager_config.ymlthat enables or disables inclusion of a given respositorySuggested Reviewers
@snarthan