Merge lp://staging/~bbaude/cloud-init/rh_sub_rm_first into lp://staging/~cloud-init-dev/cloud-init/trunk

Proposed by Brent Baude
Status: Merged
Merge reported by: Scott Moser
Merged at revision: not available
Proposed branch: lp://staging/~bbaude/cloud-init/rh_sub_rm_first
Merge into: lp://staging/~cloud-init-dev/cloud-init/trunk
Diff against target: 12 lines (+1/-1)
1 file modified
cloudinit/config/cc_rh_subscription.py (+1/-1)
To merge this branch: bzr merge lp://staging/~bbaude/cloud-init/rh_sub_rm_first
Reviewer Review Type Date Requested Status
cloud-init Commiters Pending
Review via email: mp+298689@code.staging.launchpad.net

Description of the change

cc_rh_subscription - Remove repos before adding repos

Several users have pointed out that when using the rh_subscription portion of cloud-init, that the code always added Yum repositories before removing any. The preferred use case is that rh_subscription would remove repositories and then add them. This helps users who like to remove all yum repositories and then only add in select ones. This is fairly common.

To post a comment you must log in.
Revision history for this message
Scott Moser (smoser) wrote :

Hello,
Thank you for taking the time to contribute to cloud-init. Cloud-init has moved its revision control system to git. As a result, we are marking all bzr merge proposals as 'rejected'. If you would like to re-submit this proposal for review, please do so by following the current HACKING documentation at http://cloudinit.readthedocs.io/en/latest/topics/hacking.html .

Note, I think this is actually merged.

https://git.launchpad.net/cloud-init/commit/?id=edd94ebff613f

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.