-
Notifications
You must be signed in to change notification settings - Fork 98
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
Push image to docker hub #651 #709
Conversation
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.
Thanks a lot for the contribution 👍 Curldemo is not connected to main k8gb release so it's better to create a separate flow for it. Also env.GITHUB_JOB
will probably create unrelated tag name for the image
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.
Please address suggested changes.
Please also use --signoff to satisfy DCO https://github.com/k8gb-io/k8gb/pull/709/checks?check_run_id=4060681862 .
We are CNCF project and it is a strong requirement.
Signed-off-by: AugustasV <reg1nt1z@gmail.com>
Signed-off-by: AugustasV <reg1nt1z@gmail.com>
Signed-off-by: AugustasV <reg1nt1z@gmail.com>
388d5a2
to
1eca721
Compare
Signed-off-by: AugustasV <reg1nt1z@gmail.com>
dace4f5
to
6e02ed3
Compare
Signed-off-by: AugustasV <reg1nt1z@gmail.com>
f47a78f
to
4d1b763
Compare
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
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.
@AugustasV looks great, thanks for all the fixes :)
Edited github CI yaml file to upload artifact into dockerhub during build stage.
Issue #651