bad test for snappy systems
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
cloud-init |
Fix Released
|
Medium
|
Unassigned | ||
cloud-init (Ubuntu) |
Fix Released
|
Medium
|
Unassigned |
Bug Description
Reviewing the latest SRU for cloud-init, I noticed the following:
def system_is_snappy():
# channel.ini is configparser loadable.
# snappy will move to using /etc/system-
# this is certainly not a perfect test, but good enough for now.
content = load_file(
if 'ubuntu-core' in content.lower():
return True
if os.path.
return True
return False
This isn't a good test for whether a system is an ubuntu-core system. 'system-image' is historical baggage, and not likely to be present at all in future versions.
I'm afraid I don't know a good alternative test offhand, but wanted to log the bug so someone could look into it rather than being caught by surprise when ubuntu-core image contents later change.
Changed in cloud-init (Ubuntu): | |
status: | New → Confirmed |
importance: | Undecided → Medium |
Changed in cloud-init: | |
status: | New → Confirmed |
importance: | Undecided → Medium |
This issue was fixed a while back in:
https:/ /github. com/canonical/ cloud-init/ commit/ 4bcc94730
cloud-init retains the legacy bits, but prefers the use of os-release file or kernel commandline to for checking for UbuntuCore.