os_release should detect if upgrade has occurred and reset cache
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Ceilometer Agent Charm |
Fix Released
|
Medium
|
Corey Bryant | ||
OpenStack Ceilometer Charm |
Fix Released
|
Medium
|
James Page | ||
OpenStack Cinder Charm |
Fix Released
|
Medium
|
Corey Bryant | ||
OpenStack Dashboard Charm |
Fix Released
|
Medium
|
James Page | ||
OpenStack Glance Charm |
Fix Released
|
Medium
|
Corey Bryant | ||
OpenStack Heat Charm |
Fix Released
|
Medium
|
James Page | ||
OpenStack Keystone Charm |
Fix Released
|
Medium
|
James Page | ||
OpenStack Nova Cloud Controller Charm |
Fix Released
|
Medium
|
David Ames | ||
OpenStack Nova Compute Charm |
Fix Released
|
Medium
|
Corey Bryant |
Bug Description
In upgrade testing for Pike we tripped over at least 2 scenarios so far where os_release() returns the old cached release after upgrade. For late release fixes, we've patched these issues at the calls to os_release, like this for example:
diff --git a/hooks/
index 837a167..f3dcecb 100644
--- a/hooks/
+++ b/hooks/
@@ -2012,7 +2012,7 @@ def enable_
if release:
_release = release
else:
- _release = os_release(package, base='icehouse')
+ _release = os_release(package, base='icehouse', reset_cache=True)
if not _release:
_release = get_os_
and this for example:
https:/
We should really fix this in os_release() though so we never have to deal with this issue again in another call to os_release. os_release should be updated to detect if an upgrade has occurred and reset it's cache after upgrade on it's own.
description: | updated |
description: | updated |
description: | updated |
description: | updated |
description: | updated |
no longer affects: | ceilometer |
Changed in charm-ceilometer: | |
status: | New → Triaged |
Changed in charm-heat: | |
status: | New → Triaged |
Changed in charm-ceilometer: | |
importance: | Undecided → High |
importance: | High → Medium |
Changed in charm-heat: | |
importance: | Undecided → Medium |
Changed in charm-ceilometer: | |
milestone: | none → 17.08 |
Changed in charm-heat: | |
milestone: | none → 17.08 |
Changed in charm-nova-cloud-controller: | |
milestone: | none → 17.08 |
Changed in charm-nova-compute: | |
milestone: | none → 17.08 |
Changed in charm-keystone: | |
status: | New → Triaged |
Changed in charm-openstack-dashboard: | |
status: | New → Triaged |
importance: | Undecided → Medium |
Changed in charm-keystone: | |
importance: | Undecided → Medium |
milestone: | none → 17.08 |
Changed in charm-openstack-dashboard: | |
milestone: | none → 17.08 |
Changed in charm-nova-cloud-controller: | |
assignee: | Corey Bryant (corey.bryant) → David Ames (thedac) |
Changed in charm-ceilometer-agent: | |
status: | New → Triaged |
importance: | Undecided → Medium |
assignee: | nobody → Corey Bryant (corey.bryant) |
Changed in charm-openstack-dashboard: | |
assignee: | nobody → James Page (james-page) |
Changed in charm-keystone: | |
assignee: | nobody → James Page (james-page) |
Changed in charm-heat: | |
assignee: | nobody → James Page (james-page) |
Changed in charm-ceilometer: | |
assignee: | nobody → James Page (james-page) |
Changed in charm-ceilometer-agent: | |
milestone: | none → 17.08 |
Changed in charm-keystone: | |
status: | Triaged → In Progress |
Changed in charm-nova-cloud-controller: | |
status: | Triaged → In Progress |
Changed in charm-cinder: | |
status: | New → Triaged |
Changed in charm-glance: | |
status: | New → Triaged |
Changed in charm-cinder: | |
importance: | Undecided → Medium |
Changed in charm-glance: | |
importance: | Undecided → Medium |
Changed in charm-cinder: | |
assignee: | nobody → Corey Bryant (corey.bryant) |
Changed in charm-glance: | |
assignee: | nobody → Corey Bryant (corey.bryant) |
Changed in charm-openstack-dashboard: | |
status: | Fix Committed → Fix Released |
Changed in charm-keystone: | |
status: | Fix Committed → Fix Released |
Changed in charm-nova-cloud-controller: | |
status: | Fix Committed → Fix Released |
Changed in charm-ceilometer: | |
status: | Fix Committed → Fix Released |
Changed in charm-heat: | |
status: | Fix Committed → Fix Released |
Changed in charm-nova-compute: | |
milestone: | 17.08 → 17.11 |
Changed in charm-ceilometer-agent: | |
milestone: | 17.08 → 17.11 |
Changed in charm-nova-compute: | |
milestone: | 17.11 → 18.02 |
Changed in charm-ceilometer-agent: | |
milestone: | 17.11 → 18.02 |
Changed in charm-nova-compute: | |
milestone: | 18.02 → 18.05 |
Changed in charm-ceilometer-agent: | |
milestone: | 18.02 → 18.05 |
Changed in charm-nova-compute: | |
milestone: | 18.05 → 18.08 |
Changed in charm-ceilometer-agent: | |
milestone: | 18.05 → 18.08 |
Changed in charm-nova-compute: | |
milestone: | 18.08 → 18.11 |
Changed in charm-ceilometer-agent: | |
milestone: | 18.08 → 18.11 |
Changed in charm-nova-compute: | |
milestone: | 18.11 → 19.04 |
Changed in charm-ceilometer-agent: | |
milestone: | 18.11 → 19.04 |
Changed in charm-nova-compute: | |
milestone: | 19.04 → 19.07 |
Changed in charm-ceilometer-agent: | |
milestone: | 19.04 → 19.07 |
Changed in charm-nova-compute: | |
milestone: | 19.07 → 19.10 |
Changed in charm-ceilometer-agent: | |
milestone: | 19.07 → 19.10 |
Changed in charm-nova-compute: | |
milestone: | 19.10 → 20.01 |
Changed in charm-ceilometer-agent: | |
milestone: | 19.10 → 20.01 |
tags: | added: openstack-upgrade |
Changed in charm-nova-compute: | |
milestone: | 20.01 → 20.05 |
Changed in charm-ceilometer-agent: | |
milestone: | 20.01 → 20.05 |
Changed in charm-nova-compute: | |
milestone: | 20.05 → 20.08 |
Changed in charm-ceilometer-agent: | |
milestone: | 20.05 → 20.08 |
Changed in charm-nova-compute: | |
milestone: | 20.08 → none |
Changed in charm-ceilometer-agent: | |
milestone: | 20.08 → none |
Changed in charm-ceilometer-agent: | |
status: | In Progress → Fix Released |
Changed in charm-cinder: | |
status: | In Progress → Fix Released |
Changed in charm-glance: | |
status: | In Progress → Fix Released |
Changed in charm-nova-compute: | |
status: | In Progress → Fix Released |
Changed in charm-ceilometer-agent: | |
milestone: | none → 17.08 |
Changed in charm-cinder: | |
milestone: | none → 17.08 |
Changed in charm-glance: | |
milestone: | none → 17.08 |
Changed in charm-nova-compute: | |
milestone: | none → 17.08 |
another option would be to update all charms to call os_release(... , reset_cache=True) after they upgrade.