Merge lp://staging/~adam-collard/charm-helpers/openstack-upgrade-available-swift-stable into lp://staging/~openstack-charmers/charm-helpers/stable-1604
Proposed by
Adam Collard
| Status: | Merged |
|---|---|
| Merged at revision: | 369 |
| Proposed branch: | lp://staging/~adam-collard/charm-helpers/openstack-upgrade-available-swift-stable |
| Merge into: | lp://staging/~openstack-charmers/charm-helpers/stable-1604 |
| Diff against target: |
58 lines (+13/-5) 2 files modified
charmhelpers/contrib/openstack/utils.py (+7/-5) tests/contrib/openstack/test_openstack_utils.py (+6/-0) |
| To merge this branch: | bzr merge lp://staging/~adam-collard/charm-helpers/openstack-upgrade-available-swift-stable |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Liam Young (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
Backport r436 to stable.
See https:/
"Special case swift packages for version comparison purposes in openstack_
Without this change, the swift charms will think that OpenStack upgrades are constantly available because the code ended up comparing numbers from two different version schemes and it turns out that '2014.1' > '1.13.1'"
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
Approve