fixes #4570 - display AIX version differently

[joshuabaird]

Previously, AIX versions were displayed as "AIX 7100.0301" With this commit, they are now displayed like "AIX 7.1" where major=7 and minor=1. Previously, the other values included service pack and technology levels which doesn't really belong in the main operating system version.

[domcleal]

[test]

[GregSutcliffe]

@joshuabaird you forgot to update the AIX test :)

<"6100"> expected but was
<"6">. (test_0016_should set os.major and minor for from AIX facts)

[joshuabaird]

Ok, lets try this again :)

[domcleal]

[test]

[dLobatog]

Looks good to me and it parses the facts as expected, I'd say ready to merge if nobody has any issues with the PR.

[domcleal]

@elobato please hold off, there was a discussion on the ticket

[isratrade]

@elobato, @domcleal, @joshuabaird, what's the result of the discussion on this ticket?

[domcleal]

A bit undecided, but either leaving it as is, or a slightly different variant. Best yet would probably be to allow the OS model to have different versioning schemes based on the family or configuration, as we have minor issues in other OS family types.
http://projects.theforeman.org/issues/4570

[dLobatog]

Did we reach any consensus on this? From what I see on the issue not really, I believe allowing alphanumeric characters in major minor would be more flexible. Allowing different versioning schemes defined by us via Family sounds good too.

@joshuabaird what do you think? Do you plan to work on this in the near future? If not I suggest we close this off and reopened later if needed.

[dLobatog]

@joshuabaird @domcleal ping?

[ohadlevy]

closing due to lack of activity, please re-open if something changes