Bug #4570
openAIX Operating System version is shown in a strange way.
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
Updated by Dominic Cleal almost 11 years ago
- Related to Bug #1489: AIX not showing up on Hosts page added
Updated by Dominic Cleal almost 11 years ago
- Related to Bug #105: Operating Major version too short for AIX added
Updated by Dominic Cleal almost 11 years ago
- Status changed from New to Ready For Testing
- Target version set to 1.9.0
Updated by Ruediger Mueck almost 11 years ago
Testing Josh fix ... so far it does the job!
Updated by Josh Baird almost 11 years ago
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)?
Updated by Ruediger Mueck almost 11 years ago
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 ?
Updated by Josh Baird almost 11 years ago
I was thinking of making "major" = 7.1, and "minor" = TL2. Thoughts?
Updated by Benjamin Papillon almost 11 years ago
That's also my preferred choice. If Josh wants to include SP level it is also good for me.
Updated by Josh Baird almost 11 years ago
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.
Updated by Ruediger Mueck almost 11 years ago
I would then vote for something like:
major 7.1
minor 02 or just 2 (TL)
7.1.02
Updated by Benjamin Papillon almost 11 years ago
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).
Updated by Ruediger Mueck almost 11 years ago
imho:
then it would be the best to keep it majoraix + "." + tlaix + "." + spaix
@Benjamin : I think we should separate the TL from the SP.
Updated by Anonymous almost 11 years ago
- Target version changed from 1.9.0 to 1.8.4
Updated by Anonymous almost 11 years ago
- Target version changed from 1.8.4 to 1.8.3
Updated by Anonymous almost 11 years ago
- Target version changed from 1.8.3 to 1.8.4
Updated by Anonymous almost 11 years ago
- Target version changed from 1.8.4 to 1.8.3
Updated by Anonymous almost 11 years ago
- Target version changed from 1.8.3 to 1.8.2
Updated by Anonymous over 10 years ago
- Target version changed from 1.8.2 to 1.8.1
Updated by Anonymous over 10 years ago
- Target version changed from 1.8.1 to 1.8.0
Updated by Anonymous over 10 years ago
- Target version changed from 1.8.0 to 1.7.5
Updated by Anonymous over 10 years ago
- Target version changed from 1.7.5 to 1.7.4
Updated by Anonymous over 10 years ago
- Target version changed from 1.7.4 to 1.7.3
Updated by Dominic Cleal over 10 years ago
- Target version changed from 1.7.3 to 1.7.2