Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
More explicit documentation for release process. #2185
More explicit documentation for release process. #2185
Changes from 19 commits
14803d2
ebcab8d
b58dc70
36e8a9b
956f3cf
ae08672
6a8a671
27f102c
55953c6
e0ae122
b13fb02
967805a
1464c7a
5103be3
7507c0e
98a0919
08c90ae
beed1e4
c271bf6
726794c
e3f92a4
bde2d7b
8593c56
15a2689
8cae818
2bea192
f16cc93
fcd1da9
6a37a82
d806277
bb77636
3c799eb
47469c0
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Should we link to https://github.com/sourcegraph/deploy-sourcegraph/blob/master/README.dev.md and/or modify it to de-duplicate information?
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.
This checklist should be the canonical checklist for releasing Sourcegraph. It is ok if it links to other docs for certain steps (e.g. how to run Sourcegraph on a clean Kubernetes cluster), but we should avoid duplication anywhere.
Parts of https://github.com/sourcegraph/deploy-sourcegraph/blob/master/README.dev.md are duplicated and should be removed. There is one part (an update to deploy-sourcegraph that doesn't involve bumping image tags) that probably needs to stay documented there and not here.
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.
I was going to add this during the meeting but it's more of a nuetral/non-argument and I didn't want to interrupt the discussion:
Before this meeting, I was on the side of releasing on the 3rd tuesday every month because of the prospect of us having to do work on weekends. But if the work of the release captain is to do all of the steps "on or before" the release date, and that means it would be fine to do all the final work for the release on the 18th if the 20th is a monday AND a holiday, then there's not real difference between whether we release on the 20th or the 3rd tuesday. The fact that our end goal is for the release is to be a "non-event" and ideally automated away in the future also supports this opinion. Since it should be a "non-event" and the work can be done before the release date in the mean time, I think we should go with the option that takes less thought, which is IMO releasing on a fixed date (though the difference is pretty small).
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 for sharing your thoughts!
After re-reading this, I think the "no exception is made for weekends or holidays" is somewhat redundant so I am going to remove it.