Merge ~bbaude/cloud-init:rh_sub_rm_first into cloud-init:master

Proposed by Brent Baude
Status: Merged
Merged at revision: edd94ebff613fd99768d9287b12d72a9409ae6bc
Proposed branch: ~bbaude/cloud-init:rh_sub_rm_first
Merge into: cloud-init:master
Diff against target: 34 lines (+5/-4)
2 files modified
cloudinit/config/cc_rh_subscription.py (+3/-2)
tests/unittests/test_rh_subscription.py (+2/-2)
Reviewer Review Type Date Requested Status
cloud-init Commiters Pending
Review via email: mp+309478@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/cloudinit/config/cc_rh_subscription.py b/cloudinit/config/cc_rh_subscription.py
index d858f65..bd6b5f8 100644
--- a/cloudinit/config/cc_rh_subscription.py
+++ b/cloudinit/config/cc_rh_subscription.py
@@ -421,11 +421,12 @@ class SubscriptionManager(object):
421 "because it is not enabled".format(fail))421 "because it is not enabled".format(fail))
422422
423 cmd = ['repos']423 cmd = ['repos']
424 if len(enable_list) > 0:
425 cmd.extend(enable_list)
426 if len(disable_list) > 0:424 if len(disable_list) > 0:
427 cmd.extend(disable_list)425 cmd.extend(disable_list)
428426
427 if len(enable_list) > 0:
428 cmd.extend(enable_list)
429
429 try:430 try:
430 self._sub_man_cli(cmd)431 self._sub_man_cli(cmd)
431 except util.ProcessExecutionError as e:432 except util.ProcessExecutionError as e:
diff --git a/tests/unittests/test_rh_subscription.py b/tests/unittests/test_rh_subscription.py
index 891dbe7..7962387 100644
--- a/tests/unittests/test_rh_subscription.py
+++ b/tests/unittests/test_rh_subscription.py
@@ -83,8 +83,8 @@ class GoodTests(TestCase):
83 '''83 '''
84 call_lists = []84 call_lists = []
85 call_lists.append(['attach', '--pool=pool1', '--pool=pool3'])85 call_lists.append(['attach', '--pool=pool1', '--pool=pool3'])
86 call_lists.append(['repos', '--enable=repo2', '--enable=repo3',86 call_lists.append(['repos', '--disable=repo5', '--enable=repo2',
87 '--disable=repo5'])87 '--enable=repo3'])
88 call_lists.append(['attach', '--auto', '--servicelevel=self-support'])88 call_lists.append(['attach', '--auto', '--servicelevel=self-support'])
89 self.SM.log_success = mock.MagicMock()89 self.SM.log_success = mock.MagicMock()
90 reg = "The system has been registered with ID:" \90 reg = "The system has been registered with ID:" \

Subscribers

People subscribed via source and target branches