Skip to content

grains: extend os_family mappings#68715

Open
st-man wants to merge 7 commits intosaltstack:masterfrom
st-man:st-man-patch-1
Open

grains: extend os_family mappings#68715
st-man wants to merge 7 commits intosaltstack:masterfrom
st-man:st-man-patch-1

Conversation

@st-man
Copy link

@st-man st-man commented Feb 10, 2026

Extend os_family mapping for additional Linux distributions.

What does this PR do?

Adds missing os_family mappings for additional distro identifiers to ensure they are classified under existing families (e.g. Suse, RedHat) instead of falling back to the raw os value.

What issues does this PR fix or reference?

N/A

Previous Behavior

On affected systems, os_family could fall back to the os string because the identifier was not present in the mapping.

New Behavior

os_family is mapped to the appropriate existing family (Suse / RedHat) for the added identifiers.

Merge requirements satisfied?

  • Docs (not needed)
  • Changelog (not needed)
  • Tests written/updated (not added)

Commits signed with GPG?

No

Extend os_family mapping for additional Linux distributions.

This aligns os_family detection with existing SUSE and RedHat
families without affecting default behavior.
@st-man st-man requested a review from a team as a code owner February 10, 2026 19:58
@st-man st-man changed the title grains: add missing os_family mappings grains: extend os_family mappings Feb 11, 2026
twangboy
twangboy previously approved these changes Feb 11, 2026
twangboy
twangboy previously approved these changes Feb 13, 2026
"osmajorrelease": 1,
}

_run_os_grains_tests(_os_release_data, {}, expectation)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably needs a new line here as well

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My mistake, added again.

@st-man st-man requested a review from twangboy February 14, 2026 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:full Run the full test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants