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

x/build/cmd/releasebot: check that security branch is merged into release branch #34505

Closed
bradfitz opened this issue Sep 24, 2019 · 2 comments
Closed
Labels
Builders x/build issues (builders, bots, dashboards) FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Milestone

Comments

@bradfitz
Copy link
Contributor

We recently changed how we handle point release branch management for bug fixes & security releases. We used to have one linear history in release branches and we had to land things in the correct order (security things only before a security release, and then land all bug fixes for non-security releases)

We've changed to land bug fixes on release branches immediately, and instead cutting security releases from the last release on its own branch.

There's the very real possibility now that humans'll be human and forget a step here and not merge the security branch into the release branch, resulting in a future bug fix release that removes the security fixes.

We should add a check for this in cmd/releasebot so we don't do this by accident.

/cc @andybons @dmitshur

@gopherbot gopherbot added this to the Unreleased milestone Sep 24, 2019
@gopherbot gopherbot added the Builders x/build issues (builders, bots, dashboards) label Sep 24, 2019
@bradfitz bradfitz added the NeedsFix The path to resolution is known, but the work has not been done. label Sep 24, 2019
@dmitshur
Copy link
Contributor

/cc @cagedmantis Here's an issue related to releasebot for you to consider.

@cagedmantis cagedmantis self-assigned this Oct 31, 2019
@gopherbot
Copy link

Change https://golang.org/cl/206437 mentions this issue: cmd/releasebot: check that the security branch is rebased into the release branch

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Builders x/build issues (builders, bots, dashboards) FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Development

No branches or pull requests

4 participants