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

manifest: Turn on mutate-os-release #68

Merged
merged 1 commit into from
Mar 19, 2019
Merged

Conversation

jlebon
Copy link
Member

@jlebon jlebon commented Mar 19, 2019

This allows RPM-OSTree to inject some information in the os-release
about the compose. Notably the VERSION and PRETTY_NAME keys are
updated to reflect the OSTree commit version. A new OSTREE_VERSION key
is also added. This also matches what we had in Fedora Atomic Host.

This allows RPM-OSTree to inject some information in the os-release
about the compose. Notably the `VERSION` and `PRETTY_NAME` keys are
updated to reflect the OSTree commit version. A new `OSTREE_VERSION` key
is also added. This also matches what we had in Fedora Atomic Host.
@lucab
Copy link
Contributor

lucab commented Mar 19, 2019

coreos/fedora-coreos-tracker#154 (comment) has context on some more future usage we want out of this.

@cgwalters cgwalters merged commit 74e3eb0 into coreos:master Mar 19, 2019
@dustymabe
Copy link
Member

Before:

NAME=Fedora
VERSION="29 (CoreOS preview)"
ID=fedora
VERSION_ID=29
VERSION_CODENAME=""
PLATFORM_ID="platform:f29"
PRETTY_NAME="Fedora 29 (CoreOS preview)"
ANSI_COLOR="0;34"
LOGO=fedora-logo-icon
CPE_NAME="cpe:/o:fedoraproject:fedora:29"
HOME_URL="https://fedoraproject.org/"
DOCUMENTATION_URL="https://docs.fedoraproject.org/en-US/fedora/f29/system-administrators-guide/"
SUPPORT_URL="https://fedoraproject.org/wiki/Communicating_and_getting_help"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_BUGZILLA_PRODUCT="Fedora"
REDHAT_BUGZILLA_PRODUCT_VERSION=29
REDHAT_SUPPORT_PRODUCT="Fedora"
REDHAT_SUPPORT_PRODUCT_VERSION=29
PRIVACY_POLICY_URL="https://fedoraproject.org/wiki/Legal:PrivacyPolicy"
VARIANT="CoreOS preview"
VARIANT_ID=coreos

After:

NAME=Fedora
VERSION="29.10 (CoreOS preview)"
ID=fedora
VERSION_ID=29
VERSION_CODENAME=""
PLATFORM_ID="platform:f29"
PRETTY_NAME="Fedora 29.10 (CoreOS preview)"
ANSI_COLOR="0;34"
LOGO=fedora-logo-icon
CPE_NAME="cpe:/o:fedoraproject:fedora:29"
HOME_URL="https://fedoraproject.org/"
DOCUMENTATION_URL="https://docs.fedoraproject.org/en-US/fedora/f29/system-administrators-guide/"
SUPPORT_URL="https://fedoraproject.org/wiki/Communicating_and_getting_help"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_BUGZILLA_PRODUCT="Fedora"
REDHAT_BUGZILLA_PRODUCT_VERSION=29
REDHAT_SUPPORT_PRODUCT="Fedora"
REDHAT_SUPPORT_PRODUCT_VERSION=29
PRIVACY_POLICY_URL="https://fedoraproject.org/wiki/Legal:PrivacyPolicy"
VARIANT="CoreOS preview"
VARIANT_ID=coreos
OSTREE_VERSION=29.10

Does this have any effect on what ansible picks up. There was a recent discussion on this cc @samdoran

@jlebon
Copy link
Member Author

jlebon commented Mar 19, 2019

Apps ideally shouldn't be using human-targeted strings like PRETTY_NAME and VERSION (unless of course, it's meant to just passthrough for display to humans). Codified keys like VERSION_ID and VARIANT_ID are stable, and those mutate-os-release doesn't touch.

@samdoran
Copy link

I don't think this will greatly affect what Ansible gets from this file and ultimately displays to the end user in the form of facts. Thank you for the heads up, though.

@jlebon jlebon deleted the pr/mutate branch July 4, 2019 21:55
c4rt0 pushed a commit to c4rt0/fedora-coreos-config that referenced this pull request Mar 27, 2023
oscontainer: Fix Jenkinsfile syntax
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants