-
Notifications
You must be signed in to change notification settings - Fork 44
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
GitHub Release Action Fixes #202
GitHub Release Action Fixes #202
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.
The workflow kind of runs fine (didn't test out the publish and push part). Just curious where did you use the is-stable
?
27b30c9
to
964a0e7
Compare
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.
Thanks. LGTM. I did not test out the pypi publish and quay. The rest of the steps ran fine.
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.
Thank you @anishasthana for the improvements. I have one comment.
This fixes a few issues that were discovered in the CodeFlare 0.0.5 release 1. Tag image as stable, not latest. We have switched over to using podman as well 2. Update github release step to include release notes 3. Move github tag + release step to end Signed-off-by: Anish Asthana <[email protected]>
Co-authored-by: ted chang <[email protected]>
Signed-off-by: Anish Asthana <[email protected]>
b01878c
to
5729708
Compare
@MichaelClifford @Maxusmusti can I get eyes on this PR? |
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
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
This fixes a few issues that were discovered in the CodeFlare 0.0.5 release
v
as part of the tag for consistency