-
Notifications
You must be signed in to change notification settings - Fork 277
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
Create ReleaseOwner.md file #1459
Create ReleaseOwner.md file #1459
Conversation
Signed-off-by: Abhinav Gupta <abhng@amazon.com>
Codecov Report
@@ Coverage Diff @@
## main #1459 +/- ##
=========================================
Coverage 94.35% 94.35%
Complexity 12 12
=========================================
Files 149 149
Lines 3169 3170 +1
Branches 9 9
=========================================
+ Hits 2990 2991 +1
Misses 172 172
Partials 7 7
Continue to review full report at Codecov.
|
This is a draft PR with initial steps and their descriptions. I will keep adding to this. Please feel free to commit any changes in document. Inputs are welcome. |
This seems to mostly duplicate the content in https://github.com/opensearch-project/opensearch-build/blob/main/.github/ISSUE_TEMPLATE/release_template.md. I would extend that template as needed and not create a Another option is that this content goes into Finally, a third option is |
Signed-off-by: Abhinav Gupta <abhng@amazon.com>
We could add it as a part of the template, but it would be difficult to copy paste from it to create an issue in future. How would that work? Completely unrelated, we'll need to update |
Signed-off-by: Abhinav Gupta <abhng@amazon.com>
Signed-off-by: Abhinav Gupta <abhng@amazon.com>
Signed-off-by: Abhinav Gupta <abhng@amazon.com>
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 majority of this content is duplicate of https://github.com/opensearch-project/opensearch-build/blob/main/.github/ISSUE_TEMPLATE/release_template.md and is bound to get out of date.
What's the reason not to modify that template with the missing content? Maybe a new section under the instructions div?
Upon further looking into the issue - The instructions are specific to the release owner only who performs these tasks. Adding all the details here explaining each task may make the issue very large with a lot details that maybe irrelevant to all the stakeholders. Alternatively, do you think we can add the this doc and link these instructions to the Issue_template? |
#1612 has replaced most of this, no? @peternied @abhinavGupta16 |
I agree. There are items from #1258 that are in this PR such as the release candidate selection process that I would recommend get spun off into a new PR on the issue template |
We are looking in to creating a new PR directly on the issue template with the suggested changes. |
Signed-off-by: Abhinav Gupta abhng@amazon.com
Description
This file lists down the duties to be performed by the release manager.
Issues Resolved
#1257
Check List
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.