Skip to content

Comments

TRITON-2536 Move iPXE build to 24.4.1#31

Open
danmcd wants to merge 1 commit intomasterfrom
24.4.1
Open

TRITON-2536 Move iPXE build to 24.4.1#31
danmcd wants to merge 1 commit intomasterfrom
24.4.1

Conversation

@danmcd
Copy link
Contributor

@danmcd danmcd commented Feb 11, 2026

Will need to assign this a real bugid and slap it on Kebecloud to test all three boot varieties.

@danmcd danmcd changed the title TRITON-xxxx Move iPXE build to 24.4.1 TRITON-2536 Move iPXE build to 24.4.1 Feb 16, 2026
@danmcd
Copy link
Contributor Author

danmcd commented Feb 16, 2026

Real bugid now present. Moving out of draft, and testing on the two netboot cases for starters.

Will ALSO need PR for sdc-booter.

@danmcd danmcd marked this pull request as ready for review February 16, 2026 20:40
@danmcd danmcd requested a review from a team February 16, 2026 20:40

agent {
label joyCommonLabels(image_ver: '19.4.0')
label joyCommonLabels(image_ver: '24.4.1', pi: '20210826T002459Z')
Copy link
Contributor Author

Choose a reason for hiding this comment

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

NOTE: It is possible that for ipxe a joyent-retro image will not be needed. If I can remove the pi spec from here it would be nice.


# our base image is triton-origin-x86_64-24.4.1
BASE_IMAGE_UUID = 41bd4100-eb86-409a-85b0-e649aadf6f62
BUILD_PLATFORM = 20210826T002459Z
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Same here with BUILD_PLATFORM.

@danmcd
Copy link
Contributor Author

danmcd commented Feb 16, 2026

Test notes updated.

nshalman
nshalman previously approved these changes Feb 18, 2026
Copy link

@nshalman nshalman left a comment

Choose a reason for hiding this comment

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

Assuming the testing notes are referring to a build based on these changes, looks good to me.

tsoome
tsoome previously approved these changes Feb 18, 2026
@danmcd
Copy link
Contributor Author

danmcd commented Feb 18, 2026

So the build with a joyent-retro PI did work and is on Kebecloud now. I'd like to remove the PI requirement to see if it builds on a native 24.4.1 zone. It'll be easier to diff binaries.

@danmcd danmcd dismissed stale reviews from tsoome and nshalman via 650974b February 18, 2026 20:11
@danmcd
Copy link
Contributor Author

danmcd commented Feb 18, 2026

I'm still upset I have to bring out the bigger stick of ENGBLD_SKIP_VALIDATE_BUILDENV, but it appears the suggested ENGBLD_SKIP_VALIDATE_BUILD_PLATFORM. Hmmm... do shell functions need exported environments? If so then maybe I needed to export ENGBLD_SKIP_VALIDATE_BUILD_PLATFORM=true ?

@danmcd
Copy link
Contributor Author

danmcd commented Feb 18, 2026

Okay, it just isn't going to work. Reverting to previously-approved change (and that we'll need to continue with smartos-retro... 🙃 ).

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