Skip to content

Update Dockerfile to use TARGETOS instead of linux.#289

Merged
nunnatsa merged 1 commit intokubevirt:mainfrom
ashokpariya0:use-targetos
Apr 29, 2025
Merged

Update Dockerfile to use TARGETOS instead of linux.#289
nunnatsa merged 1 commit intokubevirt:mainfrom
ashokpariya0:use-targetos

Conversation

@ashokpariya0
Copy link
Member

This modification ensures that the image can be built for various operating systems, not just Linux, by utilizing the appropriate platform variables during the build process.

What this PR does / why we need it:
This PR updates the Dockerfile to dynamically reference the platform architecture using the ${TARGETOS} and ${TARGETARCH} variables instead of the hardcoded linux platform. This change makes the Dockerfile more flexible and adaptable for building images across different operating systems and architectures

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer:

Checklist

This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

Release note:

None

This modification ensures that the image can be built
for various operating systems, not just Linux, by utilizing
the appropriate platform variables during the build process.

Signed-off-by: Ashok Pariya <ashok.pariya@ibm.com>
@kubevirt-bot kubevirt-bot added the dco-signoff: yes Indicates the PR's author has DCO signed all their commits. label Apr 29, 2025
@nunnatsa nunnatsa merged commit 792c012 into kubevirt:main Apr 29, 2025
2 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 29, 2025
Update Dockerfile to use TARGETOS instead of linux.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dco-signoff: yes Indicates the PR's author has DCO signed all their commits. size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants