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

Add setting to force install from MIRROR_HTTP on PXE boot #20657

Merged

Conversation

alvarocarvajald
Copy link
Contributor

@alvarocarvajald alvarocarvajald commented Nov 18, 2024

This commit introduces the setting FORCE_INSTALL_FROM_MIRROR_HTTP, which when set to true forces boot/boot_from_pxe to install the SUT from the URL contained in MIRROR_HTTP (grub boot install= argument) even when running in workers with a closer mirror.

Idea behind this is to allow workers to specify a combination of PXE_PRODUCT_NAME and MIRROR_HTTP settings which will lead to a proper installation of a product/build under test.

Copy link
Contributor

@mpagot mpagot left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@BillAnastasiadis BillAnastasiadis left a comment

Choose a reason for hiding this comment

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

LGTM

This commit introduces the setting *FORCE_INSTALL_FROM_MIRROR_HTTP*,
which when set to true forces `boot/boot_from_pxe` to install the SUT
from the URL contained in *MIRROR_HTTP* (grub boot `install=` argument)
even when running in workers with a closer mirror.

Idea behind this is to allow workers to specify a combination of
*PXE_PRODUCT_NAME* and *MIRROR_HTTP* settings which will lead to a
proper installation of a product/build under test.
@alvarocarvajald alvarocarvajald merged commit ad9e83a into os-autoinst:master Nov 21, 2024
10 checks passed
@alvarocarvajald alvarocarvajald deleted the pxe-prg-from-http branch November 21, 2024 12:10
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.

3 participants