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

mantle/ore: glcoud: add ability to specify image description for gcp uploads #1328

Merged

Conversation

dustymabe
Copy link
Member

GCP expects a certain image description to be attached to images in the
format of DISTRO, DISTRO RELEASE, VERSION, INFO. We need to be able to
specify a description for our images when they are created.

…e in upload.go

This should have been done as part of 5cab86f.
@dustymabe
Copy link
Member Author

there is a 2nd commit in there to fix a problem introduced in 5cab86f - so please review that too

…uploads

GCP expects a certain image description to be attached to images in the
format of `DISTRO, DISTRO RELEASE, VERSION, INFO`. We need to be able to
specify a description for our images when they are created.
Copy link
Contributor

@bgilbert bgilbert left a comment

Choose a reason for hiding this comment

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

LGTM

@bgilbert
Copy link
Contributor

bgilbert commented Apr 7, 2020

/lgtm

@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bgilbert, dustymabe

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-robot openshift-merge-robot merged commit e6bf36b into coreos:master Apr 7, 2020
@dustymabe dustymabe deleted the dusty-ore-add-description branch May 8, 2020 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants