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

Fix docker container multi arch build issue by passing GOARCH details to make build #1445

Merged
merged 3 commits into from
Nov 12, 2021

Conversation

jkandasa
Copy link
Contributor

@jkandasa jkandasa commented Nov 11, 2021

Signed-off-by: Jeeva Kandasamy [email protected]

Description

Resolves: #1442

How Has This Been Tested?

Tested it locally and the dockers images are available in https://quay.io/repository/jkandasa/oauth2-proxy?tab=tags

Checklist:

  • My change requires a change to the documentation or CHANGELOG.
  • I have updated the documentation/CHANGELOG accordingly.
  • I have created a feature (non-master) branch for my PR.

@jkandasa jkandasa requested a review from a team as a code owner November 11, 2021 13:38
@JoelSpeed JoelSpeed added the bug label Nov 12, 2021
@JoelSpeed
Copy link
Member

Would you please add a changelog entry and then I can merge this? Thanks for the fast fix!

@jkandasa jkandasa changed the title Pass GOARCH details to make build Fix docker container multi arch build issue by passing GOARCH details to make build Nov 12, 2021
Signed-off-by: Jeeva Kandasamy <[email protected]>
@jkandasa
Copy link
Contributor Author

@JoelSpeed I have updated the changelog. Kindly review the PR

Copy link
Member

@JoelSpeed JoelSpeed left a comment

Choose a reason for hiding this comment

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

Great, lgtm, thanks!

@JoelSpeed JoelSpeed merged commit 7ed4e3c into oauth2-proxy:master Nov 12, 2021
@jkandasa jkandasa deleted the fix_multi_arch_dockerx branch November 12, 2021 21:45
k-jell pushed a commit to liquidinvestigations/oauth2-proxy that referenced this pull request Apr 6, 2022
… to make build (oauth2-proxy#1445)

* pass GOARCH details to make process

Signed-off-by: Jeeva Kandasamy <[email protected]>

* update changelog

Signed-off-by: Jeeva Kandasamy <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Multi-arch container image binary is built for the wrong architecture
2 participants