Skip to content
This repository was archived by the owner on Mar 27, 2024. It is now read-only.

Add a cloudbuild.yaml trigger for container-diff releases. #125

Merged
merged 1 commit into from
Oct 23, 2017

Conversation

dlorenc
Copy link
Contributor

@dlorenc dlorenc commented Oct 20, 2017

No description provided.

args: ['make', 'cross']
# Upload to GCS.
- name: 'gcr.io/cloud-builders/gsutil'
args: ['cp', '-r', 'out/*', 'gs://container-diff/builds/$COMMIT_SHA/']
Copy link
Contributor

Choose a reason for hiding this comment

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

Might be worthwhile to have a tag-watcher cloudbuild too?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, what's a tag-watcher?

Copy link
Contributor

Choose a reason for hiding this comment

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

Lol sorry - "trigger" was the word I was looking for. Not sure where I got "tag-watcher"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Still not following... :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My plan was to submit this then setup a trigger in the UI to run this for every commit to master. Is that what you're asking for?

Copy link
Contributor

Choose a reason for hiding this comment

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

I was just suggesting that you add another cloudbuild.yaml using the same dockerfile, but triggered only when new tags are pushed (instead of every commit), with the step being slightly different:

args: ['cp', '-r', 'out/*', 'gs://container-diff/releases/$TAG_NAME/']

Copy link
Contributor

Choose a reason for hiding this comment

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

I think thats what @nkubala was planning to do eventually?

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah I think we had originally talked about adding that. that way we can have the actual releases automated for us as well, as soon as the Github release gets published we get the binaries uploaded to the correct directories. we might also consider adding a

args: ['cp', '-r', 'out/*', 'gs://container-diff/releases/latest/']

to that cloudbuild (to release the newest tag as latest), though we might want to have that in a different job so we can do rollbacks easily.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, yeah. I was thinking we'd have a separate release job to tag "latest" here.

@dlorenc dlorenc merged commit b726215 into GoogleContainerTools:master Oct 23, 2017
@dlorenc dlorenc deleted the release branch October 23, 2017 15:53
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.

3 participants