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

[COST-5214] Move TARGETARCH declaration to the top of the Dockerfile #5195

Merged
merged 2 commits into from
Jun 28, 2024

Conversation

samdoran
Copy link
Contributor

Jira Ticket

COST-5214

Description

There is a bug in podman where this is only used correctly for the multi-stage build if it is defined as the first line.

Testing

podman build --build-arg GIT_COMMIT=1f0842d71 -t koku:1f0842d -f Dockerfile .

Release Notes

  • proposed release note
* [COST-5214](https://issues.redhat.com/browse/COST-5214) Update containerfile to build with `podman`

There is a bug in podman where this is only used correctly for the
multi-stage build if it is defined as the first line.
@samdoran samdoran requested review from a team as code owners June 28, 2024 16:05
@samdoran samdoran added the hot-fix-smoke-tests pr_check label to run minimal smoke tests for fast moving bug-fix label Jun 28, 2024
@samdoran
Copy link
Contributor Author

/retest

@samdoran samdoran merged commit 4eddf8e into main Jun 28, 2024
11 checks passed
@samdoran samdoran deleted the COST-5078/COST-5214-multiarch-containerfile-fix branch June 28, 2024 18:43
djnakabaale pushed a commit that referenced this pull request Jul 9, 2024
…5195)

There is a bug in podman where this is only used correctly for the
multi-stage build if it is defined as the first line.

Update Jenkinsfile to use RHEL 8

Unfortunately this breaks the image build for Docker. I'll fix that in a followup PR.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hot-fix-smoke-tests pr_check label to run minimal smoke tests for fast moving bug-fix smokes-required
Projects
None yet
2 participants