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

Update release guide with more details #651

Merged
merged 2 commits into from
Sep 14, 2024

Conversation

IanHoang
Copy link
Collaborator

Description

After encountering a few issues in the release process, we updated the release guide.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Ian Hoang <[email protected]>
Copy link
Collaborator

@gkamat gkamat left a comment

Choose a reason for hiding this comment

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

Might be good to add some material about the steps in the release process -- the github action and the webhook that are distinct, the latter triggering jenkins, and how this can be aborted by not approving the issue, in case someone needs to deal with that.

The editing of the release issue should also be added to "Error Handling" (or add a pointer back to the previous section.)

RELEASE_GUIDE.md Outdated
@@ -55,7 +55,7 @@ NOTE: The version number below is in semantic format, for instance, `1.2.0`.

1. Create a tag: `git tag <VERSION> main`
1. Ensure that this is done in the main official opensearch-benchmark repository
2. This should be the new version that matches the version in version.txt.
2. This should be the `<VERSION>` tag matches the version in version.txt.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should be the <VERSION> tag that matches the entry in version.txt.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good catch

Signed-off-by: Ian Hoang <[email protected]>
@gkamat gkamat merged commit 620a669 into opensearch-project:main Sep 14, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants