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
pkgrepo: fix enabling/disabling repositories on Debian #51720
Conversation
@marmarek Since this is a bug, can you add a regression test? |
On plain Debian, enabled/disabled argument was ignored. Fixes saltstack#27067
For now cover only cases related to saltstack#27067
Added. |
When forward merging, |
@marmarek We are no longer accepting PRs against |
Have anything changed regarding a policy for bugfix pull requests? Last time they should be made against oldest supported branch, not |
What does this PR do?
Fix enabling/disabling repositories on Debian.
On plain Debian, enabled/disabled argument was ignored.
What issues does this PR fix or reference?
Fixes #27067
Previous Behavior
Adding/enabling Debian repository with
pkgrepo.managed
state did nothing if the repository was already present but commented out.New Behavior
Repository correctly enabled.
Tests written?
Yes
Commits signed with GPG?
Yes