Bug #4570
open
AIX Operating System version is shown in a strange way.
Added by Ruediger Mueck about 11 years ago.
Updated over 5 years ago.
Description
In /usr/share/foreman/app/services/facts_parser.rb
The AIX Version is constructed in a strange way. I think it would be better
to follow the general construction scheme for os versions.
31 elsif os_name[/AIX/i]
32 majoraix, tlaix, spaix, yearaix = orel.split("-")
33 orel = majoraix + "." + tlaix + spaix
Result:
AIX 7100.0301
I think a better way would be:
34 major = majoraix0
35 minor = majoraix1
36 orel = major + "." + minor
Result:
AIX 7.1
- Related to Bug #1489: AIX not showing up on Hosts page added
- Related to Bug #105: Operating Major version too short for AIX added
- Assignee set to Josh Baird
- Status changed from New to Ready For Testing
- Target version set to 1.9.0
Testing Josh fix ... so far it does the job!
There has been some discussion in #theforeman-dev regarding this topic. It has been suggested that it may make more sense to make "7.1" the "major," and the TL (technology level) as the "minor." Example:
6100-02-06-0943
MAJOR = 6.1
MINOR = 2 (TL)
Should we also consider adding the service pack somehow (06)?
I agree the TL of AIX is very important at all (like the TL of AIX 7.1 will be responsible for supporting Power8).
And in my daily doing the TL is also important. The SP in not that important but i will discuss that tomorrow ;)
so you think it should be reported like: 7.1.2 or more the AIX way 7100-02 ?
I was thinking of making "major" = 7.1, and "minor" = TL2. Thoughts?
That's also my preferred choice. If Josh wants to include SP level it is also good for me.
Unfortunately, to use non-alpha numeric characters in major/minor, the OS model needs to be re-vamped/re-written so this will require more effort than we originally anticipated. The options with the OS model as-is are to leave the version as is (7100.0301) or change major=7 and minor=1.
I would then vote for something like:
major 7.1
minor 02 or just 2 (TL)
7.1.02
Sorry Ruediger Mueck but '7.1' contains non numeric chars.
In that case, I'd also prefer the option to leave the version as is (7100.0301).
imho:
then it would be the best to keep it majoraix + "." + tlaix + "." + spaix
@Benjamin : I think we should separate the TL from the SP.
- Target version changed from 1.9.0 to 1.8.4
- Target version changed from 1.8.4 to 1.8.3
- Target version changed from 1.8.3 to 1.8.4
- Target version changed from 1.8.4 to 1.8.3
- Target version changed from 1.8.3 to 1.8.2
- Target version changed from 1.8.2 to 1.8.1
- Target version changed from 1.8.1 to 1.8.0
- Target version changed from 1.8.0 to 1.7.5
- Target version changed from 1.7.5 to 1.7.4
- Target version changed from 1.7.4 to 1.7.3
- Target version changed from 1.7.3 to 1.7.2
- Target version deleted (
1.7.2)
Also available in: Atom
PDF