-
Notifications
You must be signed in to change notification settings - Fork 56
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
pipeline: start creating ibmcloud artifacts #280
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.
LGTM
Thanks. Will merge this once a new COSA has been built with coreos/coreos-assembler#1740 |
I think this was an oversight in the past and no one noticed because the cosa buildextend-aws --upload that runs in the "Upload AWS" stage will build the image if it wasn't already built.
bc02ef5
to
2233c6b
Compare
OK, I reworked this a bit too because I noticed AWS was missing when I was updating the CI job in the COSA PR. So now I made them be in alphabetical order and I also added AWS to the list. |
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.
Nice!
No description provided.