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

Update anaconda variables source #6129

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

elkoniu
Copy link
Contributor

@elkoniu elkoniu commented Jan 31, 2025

Anaconda IsFinal property use to be processed out of ANACONDA_ISFINAL environmental variable. This commit changes this behaviour to use RELEASE_TYPE variable out of /etc/os-release instead.

Resolves: INSTALLER-4068

@github-actions github-actions bot added the f42 Fedora 42 label Jan 31, 2025
@elkoniu elkoniu force-pushed the IsFInal-property-change branch from da1b81c to ea36cd9 Compare January 31, 2025 16:55
@elkoniu elkoniu force-pushed the IsFInal-property-change branch from ea36cd9 to 3aac39d Compare January 31, 2025 18:00
@elkoniu elkoniu force-pushed the IsFInal-property-change branch from 3aac39d to a1822df Compare January 31, 2025 18:04
@elkoniu elkoniu force-pushed the IsFInal-property-change branch from a1822df to d4d7148 Compare January 31, 2025 18:12
@elkoniu elkoniu force-pushed the IsFInal-property-change branch from d4d7148 to 8bb1c48 Compare February 1, 2025 23:14
@elkoniu elkoniu force-pushed the IsFInal-property-change branch from 8bb1c48 to 72a2c20 Compare February 1, 2025 23:16
@elkoniu elkoniu force-pushed the IsFInal-property-change branch from 72a2c20 to e1ea1cc Compare February 2, 2025 01:00
Anaconda IsFinal property use to be processed out of ANACONDA_ISFINAL
environmental variable. This commit changes this behaviour to use
RELEASE_TYPE variable out of /etc/os-release instead.

Related change in Fedora spec file:
- https://src.fedoraproject.org/rpms/fedora-release/pull-request/347

Resolves: INSTALLER-4068
@elkoniu elkoniu force-pushed the IsFInal-property-change branch from e1ea1cc to c7ceeed Compare February 2, 2025 01:11
Copy link
Member

@jkonecny12 jkonecny12 left a comment

Choose a reason for hiding this comment

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

You should probably also change:
https://github.com/rhinstaller/anaconda/blob/main/data/liveinst/liveinst#L83

also comment a few lines above

      3) Environment variable ANACONDA_ISFINAL

@jkonecny12
Copy link
Member

Also @bcl, isn't the ANACONDA_ISFINAL environment variable used somewhere Lorax / Image Builder?

@bcl
Copy link
Contributor

bcl commented Feb 3, 2025

Also @bcl, isn't the ANACONDA_ISFINAL environment variable used somewhere Lorax / Image Builder?

In lorax the --isfinal cmdline switch is used, and it sets IsFinal in the .buildstamp file. It doesn't check the environmental variable or set it. Neither does livemedia-creator.

osbuild-composer operates the same way, setting IsFinal in .buildstamp and has no references to the variable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
f42 Fedora 42
Development

Successfully merging this pull request may close these issues.

3 participants