-
Notifications
You must be signed in to change notification settings - Fork 188
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
chore: Adjusts release process to run tests against QA before releasing #1812
Conversation
- name: Create release tag | ||
uses: rickstaa/[email protected] # will fail if existing tag is present | ||
with: | ||
tag: ${{ inputs.version_number }} |
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.
as of now tag is not being signed, which is the behaviour we currently have when creating tags for our releases. Will reach out to security team to verify if we can use the gpg signing feature of this action.
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.
Could you confirm that the action is creating an annotated tag?
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.
yes verified that annotated tags were created in the forked repo used for testing, can also be seen in the code https://github.com/rickstaa/action-create-tag/blob/main/entrypoint.sh#L79
92b179a
to
4eb71d0
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.
LGTM. Thanks for all the follow ups
Description
Link to any related issue(s): CLOUDP-220682
Used a forked repository to test logic of when acceptances test and release is run.
Scenarios that were tested:
true
-> tests are not run and release is madeWhat has not been tested and will be tested on release?
Type of change:
Required Checklist:
Further comments