Skip to content

build_image: Allow specifying which OEM sysexts to build#3983

Open
krnowak wants to merge 1 commit intomainfrom
krnowak/select-oem-sysext
Open

build_image: Allow specifying which OEM sysexts to build#3983
krnowak wants to merge 1 commit intomainfrom
krnowak/select-oem-sysext

Conversation

@krnowak
Copy link
Copy Markdown
Member

@krnowak krnowak commented Apr 28, 2026

Useful if we want to locally generate one kind of image for testing.

Tested locally for building qemu OEM sysext only.

@krnowak krnowak requested a review from a team as a code owner April 28, 2026 15:28
Copy link
Copy Markdown
Contributor

@chewi chewi left a comment

Choose a reason for hiding this comment

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

Handy, but I'm about to kill common-oem-files. This will still work, but it will check for the presence of coreos-base/oem-*/*.ebuild instead.

@krnowak
Copy link
Copy Markdown
Member Author

krnowak commented Apr 28, 2026

Handy, but I'm about to kill common-oem-files. This will still work, but it will check for the presence of coreos-base/oem-*/*.ebuild instead.

Sure, whatever you do, it shouldn't affect my additions as long as get_oem_sysext_matrix works the same. I'll reword the help string to be less specific, though.

@krnowak krnowak force-pushed the krnowak/select-oem-sysext branch from c0ba6fa to 11fc14f Compare April 28, 2026 15:43
Comment thread build_library/prod_image_util.sh Outdated
if [[ ${#invalid_oems[@]} -gt 0 ]]; then
die "Requested OEMs to build sysexts for are invalid: ${invalid_oems[*]}, valid OEMs are ${all_oems[*]}"
fi
mapfile -t oem_sysexts < <(printf '%s\n' "${oem_sysexts[@]}" | grep '|\('"${requested_oem_sysexts_csv//,/'\|'}"'\)$')
Copy link
Copy Markdown
Contributor

@chewi chewi Apr 28, 2026

Choose a reason for hiding this comment

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

I think this is matching the last field? That's the USE flag field rather than the name field, which is first. I'm also killing OEM sysext USE flags, so this field will be empty.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yeah, I used the field that had the OEM name. I'll use the first field then.

Useful if we want to locally generate one kind of image for testing.

Signed-off-by: Krzesimir Nowak <knowak@microsoft.com>
@krnowak krnowak force-pushed the krnowak/select-oem-sysext branch from 11fc14f to ecb2e3c Compare April 29, 2026 11:03
@krnowak
Copy link
Copy Markdown
Member Author

krnowak commented Apr 29, 2026

Updated, tested locally.

@krnowak krnowak requested a review from chewi April 29, 2026 11:03
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.

2 participants