Skip to content
This repository has been archived by the owner on Dec 7, 2020. It is now read-only.

Extend Dockerfile to use plain multi stage build #438

Closed
wants to merge 1 commit into from
Closed

Extend Dockerfile to use plain multi stage build #438

wants to merge 1 commit into from

Conversation

aklinkert
Copy link

During the development of #436 I set up a docker hub repo with automatic building of the docker image to test the setup in place. To achieve that I changed the Dockerfile to use multi stage builds to be build-system agnostic and compile the go binary in place. I'm not sure whether that's useful for you so I opened this PR to check back. 👋

@abstractj
Copy link

abstractj commented Nov 30, 2018

@apinnecke could you please file a Jira (https://issues.jboss.org/browse/KEYCLOAK), add the affected version and steps to reproduce the issue? It helps us to plan/prioritize the next releases.

@aklinkert
Copy link
Author

@abstractj this is a PR, not an issue. And you obviously didn‘t even read the PR?!

@abstractj
Copy link

@apinnecke JIRA is used for feature requests too. Please, read https://github.com/keycloak/keycloak-gatekeeper/blob/master/README.md#help-and-documentation

@aklinkert
Copy link
Author

This PR is not a feature request 😂

@abstractj
Copy link

@apinnecke no problem. If it's not a feature request, an enhancement or a bug. I appreciate the time you took submitting it, but I believe it does not fit into this repository. For Docker images, please send PRs to https://github.com/jboss-dockerfiles/keycloak/tree/master/gatekeeper. Also make sure to file a Jira ;)

@abstractj abstractj closed this Nov 30, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants