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

Add Build Images to CI results #599

Merged
merged 2 commits into from
Sep 22, 2022

Conversation

markmandel
Copy link
Contributor

@markmandel markmandel commented Sep 21, 2022

What type of PR is this?

Uncomment only one /kind <> line, press enter to put that in a new line, and remove leading whitespace from that line:

/kind breaking
/kind bug
/kind cleanup
/kind documentation

/kind feature

/kind hotfix

What this PR does / Why we need it:

Added the image artifacts to successful builds.

Since Cloud Build doesn't track pushed images on failure (and the image may/may not exist depending on where it failed) I can only provide directions on how to search for the image after the fact.

Which issue(s) this PR fixes:

Closes #593

Special notes for your reviewer:

Also updated the Go version, and now the CI container registry is public, so one can search for development images.

If you can think of any other improvements, please suggest them!

Added the image artifacts to successful builds.

Since Cloud Build doesn't track pushed images on failure (and the image
may/may not exist depending on where it failed) I can only provide
directions on how to search for the image after the fact.

Closes googleforgames#593
@markmandel markmandel force-pushed the ci/image-notification branch from 89a62db to 7f5e379 Compare September 21, 2022 23:37
@quilkin-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: b5f40bc2-3ac0-49fa-a799-d2637616e49a

The following development images have been built, and will exist for the next 30 days:

To build this version:

git fetch git@github.com:googleforgames/quilkin.git pull/599/head:pr_599 && git checkout pr_599
cargo build

@markmandel markmandel marked this pull request as ready for review September 22, 2022 02:13
@XAMPPRocky XAMPPRocky merged commit de03e95 into googleforgames:main Sep 22, 2022
@markmandel markmandel deleted the ci/image-notification branch September 22, 2022 17:43
XAMPPRocky pushed a commit that referenced this pull request Oct 10, 2022
XAMPPRocky pushed a commit that referenced this pull request Oct 10, 2022
@markmandel markmandel added kind/feature New feature or request area/build-tools Development tooling. labels Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build-tools Development tooling. kind/feature New feature or request size/s
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Include image built in PR on CI Bot Result
3 participants