Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix version part of osfinger on Pop!_OS #61619

Merged
merged 2 commits into from Sep 30, 2022

Conversation

bdrung
Copy link
Contributor

@bdrung bdrung commented Feb 10, 2022

Pop!_OS uses the same version scheme than Ubuntu (YY.MM) and releases every six month in April and October. Using only the year from the version in the osfinger does not allow differentiating between 20.04 and 20.10. Both has the osfinger set to Pop-20.

So include the full version in the osfinger grain.

This merge request is the third in the series after merge request #61597 and #61589.

Update the OS grains test cases for Pop!_OS to still supported versions.

Signed-off-by: Benjamin Drung <benjamin.drung@ionos.com>
Pop!_OS uses the same version scheme than Ubuntu (YY.MM) and releases
every six month in April and October. Using only the year from the
version in the `osfinger` does not allow differentiating between 20.04
and 20.10. Both has the `osfinger` set to `Pop-20`.

So include the full version in the `osfinger` grain.

Signed-off-by: Benjamin Drung <benjamin.drung@ionos.com>
@garethgreenaway garethgreenaway added the Sulfur v3006.0 release code name and version label Sep 30, 2022
@garethgreenaway garethgreenaway added this to the Sulphur v3006.0 milestone Sep 30, 2022
@dmurphy18 dmurphy18 self-requested a review September 30, 2022 20:00
@garethgreenaway garethgreenaway merged commit 8d243d5 into saltstack:master Sep 30, 2022
@bdrung bdrung deleted the fix-pop-osfinger branch September 30, 2022 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Sulfur v3006.0 release code name and version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants