Skip to content

Conversation

@tkatila
Copy link
Contributor

@tkatila tkatila commented Jan 9, 2026

Fixes: #1693

@tkatila tkatila requested review from bart0sh, kad and mythi as code owners January 9, 2026 14:54
@mythi
Copy link
Contributor

mythi commented Jan 12, 2026

Do we (still) have reasons to keep two somewhat different flavors in one Dockerfile. For the normal dockerhub publishing flows in our repo, we can get the tag from from any source Dockerfile quite easiliy

@tkatila
Copy link
Contributor Author

tkatila commented Jan 12, 2026

Discussed with @mythi offline. I split the idxd and levelzero Dockerfiles into two (one for deb, one for rpm). This has the benefit of removing the if statements from the Dockerfiles. Building image works the same way: make image UBI=1 BUILDER=podman. Build-image script will change the target Dockerfile for idxd/levelzero.

Copy link
Contributor

@eero-t eero-t left a comment

Choose a reason for hiding this comment

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

Seems fine now, except for use of unstable repo, that should be explained.

@tkatila tkatila force-pushed the ubi-base-images branch 3 times, most recently from 94edab5 to c493daf Compare January 19, 2026 09:50
To build UBI based images, set UBI=1 for the make command.
For idxd or levelzero images, the build host needs to
be registered to RH (subscription-manager) so packages
can be installed from repositories.

Signed-off-by: Tuomas Katila <[email protected]>
idxd-config-initcontainer and gpu-levelzero are not built as
they require repositories which are not available in GH.

Signed-off-by: Tuomas Katila <[email protected]>
Signed-off-by: Tuomas Katila <[email protected]>
Copy link
Contributor

@mythi mythi left a comment

Choose a reason for hiding this comment

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

LGTM

@mythi mythi merged commit f8a6986 into intel:main Jan 19, 2026
64 checks passed
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.

intel-idxd-config-initcontainer cannot use UBI base images.

3 participants