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

3.31.1 patch request for permissions sync commits #24513

Closed
Tracked by #24515
bobheadxi opened this issue Sep 1, 2021 · 2 comments
Closed
Tracked by #24515

3.31.1 patch request for permissions sync commits #24513

bobheadxi opened this issue Sep 1, 2021 · 2 comments
Labels
patch-release-request Patch release request: https://about.sourcegraph.com/handbook/engineering/releases

Comments

@bobheadxi
Copy link
Member

@sourcegraph/distribution I am requesting the following commits be included in a patch release. They are already merged into main:

The intent of the questions below is to ensure we keep Sourcegraph high quality and only create patch releases based on a strict criteria. If you can answer yes to many or most of these questions, we will be happy to create the patch release.

I have read when and why we perform patch releases and answer the questions as follows:

Are users/customers actively asking us for these changes and cannot wait until the next full release?

Yes

Are the changes extremely minimal, well-tested, and low risk such that not testing as we do in a full release is OK?

They are not minimal, but they are reasonably unit/integration tested, and are also feature-flagged (authorization.groupsCacheTTL). The changes have been deployed to both cloud and dogfood where the default behaviour continues to work as expected.

Is there some functionality completely broken that warrants redacting the prior release of Sourcegraph and advising users wait for the patch release?

No

This will interrupt our regular planned work and release cycle, taking one full working day of our time, and will take up all of our site admin's valuable time by asking them to upgrade or producing noise for them if they don't need to upgrade.

Do you believe the changes are important enough to warrant this?

Yes - this is relevant to a number of high-priority customers in trial

Patch releases are a signal we can do something better to improve the quality of Sourcegraph. Have you already scheduled a call (or created a google doc) to perform a retrospective and identify ways we can improve?

There are ongoing discussions about balancing customer and product requirements, such as RFC 457, to avoid having functionality like this be urgently requested on short time frames.


For the release captain - after reviewing and approving this request:

Comment and close this issue once the relevant commit(s) have been cherry-picked into the release branch.

@bobheadxi bobheadxi added team/distribution patch-release-request Patch release request: https://about.sourcegraph.com/handbook/engineering/releases labels Sep 1, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Sep 1, 2021

Heads up @davejrt @ggilmore @daxmc99 @dan-mckean - the "team/distribution" label was applied to this issue.

@davejrt davejrt changed the title 3.30.1 patch request for permissions sync commits 3.31.1 patch request for permissions sync commits Sep 1, 2021
@bobheadxi
Copy link
Member Author

Comment and close this issue once the relevant commit(s) have been cherry-picked into the release branch.

Closing this 👍

@bobheadxi bobheadxi reopened this Sep 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch-release-request Patch release request: https://about.sourcegraph.com/handbook/engineering/releases
Projects
None yet
Development

No branches or pull requests

1 participant