Project

General

Profile

Actions

Bug #4570

open

AIX Operating System version is shown in a strange way.

Added by Ruediger Mueck about 10 years ago. Updated over 4 years ago.

Status:
Ready For Testing
Priority:
Normal
Assignee:
Category:
Facts
Target version:
-
Difficulty:
easy
Triaged:
No
Fixed in Releases:
Found in Releases:

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 issues 2 (0 open2 closed)

Related to Foreman - Bug #1489: AIX not showing up on Hosts pageClosed02/08/2012Actions
Related to Foreman - Bug #105: Operating Major version too short for AIXResolved12/01/2009Actions
Actions #1

Updated by Dominic Cleal about 10 years ago

  • Related to Bug #1489: AIX not showing up on Hosts page added
Actions #2

Updated by Dominic Cleal about 10 years ago

  • Related to Bug #105: Operating Major version too short for AIX added
Actions #3

Updated by Josh Baird about 10 years ago

  • Assignee set to Josh Baird
Actions #4

Updated by Dominic Cleal about 10 years ago

  • Status changed from New to Ready For Testing
  • Target version set to 1.9.0
Actions #5

Updated by Ruediger Mueck about 10 years ago

Testing Josh fix ... so far it does the job!

Actions #6

Updated by Josh Baird about 10 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)?

Actions #7

Updated by Ruediger Mueck about 10 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 ?

Actions #8

Updated by Josh Baird about 10 years ago

I was thinking of making "major" = 7.1, and "minor" = TL2. Thoughts?

Actions #9

Updated by Benjamin Papillon about 10 years ago

That's also my preferred choice. If Josh wants to include SP level it is also good for me.

Actions #10

Updated by Josh Baird about 10 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.

Actions #11

Updated by Ruediger Mueck about 10 years ago

I would then vote for something like:

major 7.1
minor 02 or just 2 (TL)

7.1.02

Actions #12

Updated by Benjamin Papillon about 10 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).

Actions #13

Updated by Ruediger Mueck about 10 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.

Actions #14

Updated by Anonymous about 10 years ago

  • Target version changed from 1.9.0 to 1.8.4
Actions #15

Updated by Anonymous almost 10 years ago

  • Target version changed from 1.8.4 to 1.8.3
Actions #16

Updated by Anonymous almost 10 years ago

  • Target version changed from 1.8.3 to 1.8.4
Actions #17

Updated by Anonymous almost 10 years ago

  • Target version changed from 1.8.4 to 1.8.3
Actions #18

Updated by Anonymous almost 10 years ago

  • Target version changed from 1.8.3 to 1.8.2
Actions #19

Updated by Anonymous almost 10 years ago

  • Target version changed from 1.8.2 to 1.8.1
Actions #20

Updated by Anonymous over 9 years ago

  • Target version changed from 1.8.1 to 1.8.0
Actions #21

Updated by Anonymous over 9 years ago

  • Target version changed from 1.8.0 to 1.7.5
Actions #22

Updated by Anonymous over 9 years ago

  • Target version changed from 1.7.5 to 1.7.4
Actions #23

Updated by Anonymous over 9 years ago

  • Target version changed from 1.7.4 to 1.7.3
Actions #24

Updated by Dominic Cleal over 9 years ago

  • Target version changed from 1.7.3 to 1.7.2
Actions #25

Updated by Tomer Brisker over 5 years ago

  • Target version deleted (1.7.2)
Actions

Also available in: Atom PDF