Skip to content

Fix version part of osfinger on Pop!_OS#61619

Merged
garethgreenaway merged 2 commits intosaltstack:masterfrom
bdrung:fix-pop-osfinger
Sep 30, 2022
Merged

Fix version part of osfinger on Pop!_OS#61619
garethgreenaway merged 2 commits intosaltstack:masterfrom
bdrung:fix-pop-osfinger

Conversation

@bdrung
Copy link
Copy Markdown
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.

@bdrung bdrung requested a review from a team as a code owner February 10, 2022 14:05
@bdrung bdrung requested review from twangboy and removed request for a team February 10, 2022 14:05
@bdrung bdrung force-pushed the fix-pop-osfinger branch 3 times, most recently from c141d16 to 7454f25 Compare February 11, 2022 00:28
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.

3 participants