Merge ~dylan.perry/cloud-init:fix-yum-config-array-quoting into cloud-init:master
Proposed by
Dylan Perry
Status: | Merged |
---|---|
Merged at revision: | d16632ad8dfd1844d265d93ab00b54d419626019 |
Proposed branch: | ~dylan.perry/cloud-init:fix-yum-config-array-quoting |
Merge into: | cloud-init:master |
Diff against target: |
134 lines (+64/-15) 2 files modified
cloudinit/config/cc_yum_add_repo.py (+14/-9) tests/unittests/test_handler/test_handler_yum_add_repo.py (+50/-6) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Server Team CI bot | continuous-integration | Approve | |
cloud-init Commiters | Pending | ||
Review via email: mp+322268@code.launchpad.net |
Commit message
Fix yum repo config where keys contain array values
ConfigObj produces configuration files that are incompatible with yum if
multiple values are listed for a configuration key. Switch to the builtin
configparser, and ConfigParser (Python 2) which correctly handles this
case.
Remove broken bitly link
Add additional test case for array values in yum_repos definition
Description of the change
This is an attempt to fix https:/
To post a comment you must log in.
PASSED: Continuous integration, rev:6540f21fd16 a88739b78106414 bc67ee161bfdb5 /jenkins. ubuntu. com/server/ job/cloud- init-ci/ 220/ /jenkins. ubuntu. com/server/ job/cloud- init-ci/ nodes=metal- amd64/220 /jenkins. ubuntu. com/server/ job/cloud- init-ci/ nodes=metal- arm64/220 /jenkins. ubuntu. com/server/ job/cloud- init-ci/ nodes=metal- ppc64el/ 220 /jenkins. ubuntu. com/server/ job/cloud- init-ci/ nodes=metal- s390x/220 /jenkins. ubuntu. com/server/ job/cloud- init-ci/ nodes=vm- i386/220
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
Click here to trigger a rebuild: /jenkins. ubuntu. com/server/ job/cloud- init-ci/ 220/rebuild
https:/