diff options
| author |   Vincent S. Cojot <vincent@cojot.name> | 2017-02-23 09:38:24 -0500 |
|---|---|---|
| committer | Vincent S. Cojot <vincent@cojot.name> | 2017-02-28 20:04:36 -0500 |
| commit | 87569bd5711eaf79c1a3f0a4449daef860626daa (patch) | |
| tree | e93105a77830759c38dcfaecdd9b3dc4f3f16d6e /extraconfig/pre_deploy | |
| parent | 1e1a4d415ead0310fc4d6d7ff22140a4a8326887 (diff) | |
Use --disable= in subscription-manager to avoid shell expansion.
In extraconfig/pre_deploy/rhel-registration/scripts/rhel-registration,
there's a line that says:
retry subscription-manager repos --disable '*'
I believe this is broken and will result in shell expansion being made.
The proper line should be:
retry subscription-manager repos --disable='*'
This regression came from commit 2b06ed8adce2bcc18480b71c0f20a0ec2d21de19.
(Also see https://review.openstack.org/#/c/381233 )
This patch fixes the regression while preserving functionality
of the above change.
Closes-Bug: 1667316
Change-Id: I54f0db3f1f596f6356f7445cdc61737f20f14318
Signed-off-by: Vincent S. Cojot <vincent@cojot.name>
Diffstat (limited to 'extraconfig/pre_deploy')
| -rw-r--r-- | extraconfig/pre_deploy/rhel-registration/scripts/rhel-registration | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/extraconfig/pre_deploy/rhel-registration/scripts/rhel-registration b/extraconfig/pre_deploy/rhel-registration/scripts/rhel-registration index 2650a967..156f9314 100644 --- a/extraconfig/pre_deploy/rhel-registration/scripts/rhel-registration +++ b/extraconfig/pre_deploy/rhel-registration/scripts/rhel-registration @@ -133,7 +133,7 @@ case "${REG_METHOD:-}" in if [ -z "${REG_AUTO_ATTACH:-}" -a -z "${REG_ACTIVATION_KEY:-}" ]; then retry subscription-manager attach $attach_opts fi - retry subscription-manager repos --disable '*' + retry subscription-manager repos --disable='*' retry subscription-manager $repos ;; satellite) |