-
Notifications
You must be signed in to change notification settings - Fork 20
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
Introduce RHEL based image #34
Conversation
since RHEL parent image is in private repo we should login before docker build
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think REGISTRY should be push.registry.devshift.net
Makefile
Outdated
REGISTRY := push.registry.devshift.net/osio-prod | ||
else | ||
DOCKERFILE := Dockerfile | ||
REGISTRY := registry.devshift.net |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
REGISTRY sould be push.registry.devshift.net
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice, catch! thanks
@riuvshin Your image is available in the registry: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Enable build on RHEL if TARGET=rhel.
The cico environment will continue to build the CentOS based container and in a second step it will build and push a RHEL based container.
cc: @jmelis @msrb @miteshvp