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

Push image to docker hub #651 #709

Merged
merged 5 commits into from
Nov 5, 2021

Conversation

AugustasV
Copy link
Contributor

@AugustasV AugustasV commented Oct 30, 2021

Edited github CI yaml file to upload artifact into dockerhub during build stage.

Issue #651

Copy link
Member

@ytsarev ytsarev left a 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

.github/workflows/release.yaml Outdated Show resolved Hide resolved
Copy link
Member

@ytsarev ytsarev left a 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.

.github/workflows/curldemo.yaml Outdated Show resolved Hide resolved
.github/workflows/curldemo.yaml Outdated Show resolved Hide resolved
.github/workflows/curldemo.yaml Outdated Show resolved Hide resolved
.github/workflows/release.yaml Outdated Show resolved Hide resolved
.github/workflows/curldemo.yaml Outdated Show resolved Hide resolved
.github/workflows/curldemo.yaml Outdated Show resolved Hide resolved
Signed-off-by: AugustasV <reg1nt1z@gmail.com>
Signed-off-by: AugustasV <reg1nt1z@gmail.com>
Signed-off-by: AugustasV <reg1nt1z@gmail.com>
@AugustasV AugustasV force-pushed the push_image_to_dockerhub branch from 388d5a2 to 1eca721 Compare November 1, 2021 22:53
@AugustasV AugustasV requested review from jkremser and ytsarev November 2, 2021 06:35
.github/workflows/release.yaml Outdated Show resolved Hide resolved
.github/workflows/curldemo.yaml Outdated Show resolved Hide resolved
.github/workflows/curldemo.yaml Outdated Show resolved Hide resolved
Signed-off-by: AugustasV <reg1nt1z@gmail.com>
@AugustasV AugustasV force-pushed the push_image_to_dockerhub branch from dace4f5 to 6e02ed3 Compare November 2, 2021 20:46
Signed-off-by: AugustasV <reg1nt1z@gmail.com>
@AugustasV AugustasV force-pushed the push_image_to_dockerhub branch from f47a78f to 4d1b763 Compare November 3, 2021 09:06
@AugustasV AugustasV requested review from ytsarev and jkremser November 3, 2021 09:07
Copy link
Collaborator

@k0da k0da left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@ytsarev ytsarev left a 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 :)

@jkremser jkremser merged commit 9d68057 into k8gb-io:master Nov 5, 2021
@AugustasV AugustasV deleted the push_image_to_dockerhub branch November 5, 2021 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants