CentOS fails to bootstrap using juju 1.24
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
juju-core |
Fix Released
|
High
|
Gabriel Samfira | ||
1.24 |
Fix Released
|
High
|
Gabriel Samfira |
Bug Description
Starting with commit #2436, centOS support has been broken.
http://
This commit was meant to fix bug #1459250. So I will try to put as much context as possible here:
1) If by any chance we will decide to support another OS which is also at version 7, as long as it has a different codename like Jessie, it will simply be another entry under the same version. This is confusing for humans, I agree, but machines will not care. See bellow for alternative to avoid this confusion.
2) The way version detection is done now, is through reading /etc/os-release on modern Linux based systems. On CentOS 7, the contents of this file are as follows:
http://
So VERSION_ID in that particular file is just "7".
3) We need to trust that the test has been written with valid sample data :). Changing the test so that it passes, but breaks the actual functionality is not a good thing.
Now, like I said, I agree that just a simple digit is confusing, but its technically correct. That is the actual "version" of the operating system, in the same way that 14.04 is the version nr for trusty. If we do not want to create confusion, we can determine the OS, then the version, create an internal map, look up that version inside that particular map. This can be done with minimal changes to what we already have. It just needs to be changed in the right place, while taking into account that we may support other operating systems that are redhat based in the future (Fedora for example).
tags: | added: bootstrap centos |
Changed in juju-core: | |
assignee: | nobody → Gabriel Samfira (gabriel-samfira) |
status: | Triaged → In Progress |
Changed in juju-core: | |
status: | In Progress → Fix Committed |
Changed in juju-core: | |
status: | Fix Committed → Fix Released |
We cannot accept a juju version of just "7" is must be a string to differentiate itself from other series. So the code need to change to make centos7