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

psc/releng: Update references to the security release process #900

Merged
merged 5 commits into from
Dec 9, 2019

Conversation

justaugustus
Copy link
Member

@justaugustus justaugustus commented Dec 7, 2019

  • psc/releng: Update references to the security release process

    • Include security-release-team@ as a contact in release-managers.md

      The Product Security Committee and Release Managers will now
      coordinate security releases using this list.

    • Ensure Branch Manager membership on release-managers-private is
      documented and included in the Release Manager onboarding template

    • Mention the Security Release Process in the Branch Manager handbook

    • Update table of contents in Patch Release Team handbook

  • lint: Fix markdown warnings in Release Team Lead handbook

  • release-team-lead: Remove reference to the private Release Managers list

    Security release coordination now happens between the Product Security
    Committee, Patch Release Team, Branch Managers, and SIG Chairs.

    Information regarding security releases will be disseminated to Release
    Team Leads by Branch Managers on a need-to-know basis.

  • release-team-lead: Add onboarding template

    Here we add an onboarding template for Release Team Leads and Lead
    Shadows and move information about requisite access from the handbook
    and into the issue template.

Signed-off-by: Stephen Augustus [email protected]

/hold (there are few things I want to tweak on the RT Lead handbook before merge)
cc: @kubernetes/product-security-committee @kubernetes/release-engineering @kubernetes/release-team
ref: #896
/area release-eng release-team
/committee product-security
/milestone v1.17
/kind documentation cleanup
/priority important-soon

@k8s-ci-robot k8s-ci-robot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. area/release-eng Issues or PRs related to the Release Engineering subproject labels Dec 7, 2019
@k8s-ci-robot k8s-ci-robot added this to the v1.17 milestone Dec 7, 2019
@k8s-ci-robot k8s-ci-robot added area/release-team Issues or PRs related to the release-team subproject committee/security-response Denotes an issue or PR intended to be handled by the product security committee. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/documentation Categorizes issue or PR as related to documentation. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. labels Dec 7, 2019
@k8s-ci-robot k8s-ci-robot requested review from jberkus and listx December 7, 2019 21:09
@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. sig/release Categorizes an issue or PR as relevant to SIG Release. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Dec 7, 2019
Copy link
Member

@saschagrunert saschagrunert left a comment

Choose a reason for hiding this comment

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

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 7, 2019
@saschagrunert
Copy link
Member

/hold

For more eyes.

@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed lgtm "Looks good to me", indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Dec 7, 2019
@justaugustus
Copy link
Member Author

Ready for review...
For Release Engineering subproject approval:
/assign @tpepper @calebamiles

For PSC approval:
/assign @tallclair @liggitt @cjcullen @lukehinds

For RT Leads visibility:
cc: @kubernetes/release-team-leads

@Bubblemelon Bubblemelon mentioned this pull request Dec 8, 2019
18 tasks
@justaugustus
Copy link
Member Author

To look over RT Lead changes:
/assign @guineveresaenger @alejandrox1

@@ -381,6 +382,17 @@ However, there is an embargo policy which requires the nomock release publicatio

To better prepare and see what to expect, this is a sequence of events that took place on past [official release days](https://docs.google.com/document/d/1K0B91lgeEiJTbT602VloA5arb6AkaTif-MLryaHmlYc/edit?usp=sharing).

#### Security fixes

The Product Security Committee (PSC) may contact you if there are security fixes needed on the release branch.
Copy link
Member

Choose a reason for hiding this comment

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

suggest adding the method of communication to expect?

Suggested change
The Product Security Committee (PSC) may contact you if there are security fixes needed on the release branch.
The Product Security Committee (PSC) may contact you via $mailing-list if there are security fixes needed on the release branch.

- Include security-release-team@ as a contact in release-managers.md

  The Product Security Committee and Release Managers will now
  coordinate security releases using this list.

- Ensure Branch Manager membership on release-managers-private is
  documented and included in the Release Manager onboarding template
- Mention the Security Release Process in the Branch Manager handbook
- Update table of contents in Patch Release Team handbook

Signed-off-by: Stephen Augustus <[email protected]>
Security release coordination now happens between the Product Security
Committee, Patch Release Team, Branch Managers, and SIG Chairs.

Information regarding security releases will be disseminated to Release
Team Leads by Branch Managers on a need-to-know basis.

Signed-off-by: Stephen Augustus <[email protected]>
Here we add an onboarding template for Release Team Leads and Lead
Shadows and move information about requisite access from the handbook
and into the issue template.

Signed-off-by: Stephen Augustus <[email protected]>
@liggitt
Copy link
Member

liggitt commented Dec 9, 2019

PSC bits lgtm

Copy link
Member

@saschagrunert saschagrunert left a comment

Choose a reason for hiding this comment

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

/lgtm
:)

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 9, 2019
Copy link
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

/lgtm

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cpanato, justaugustus, saschagrunert

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@justaugustus
Copy link
Member Author

Releasing the hold as we have PSC and Releng approval.
If the RT Lead changes need to be addressed, let's do so in a follow-up.

/hold cancel

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 9, 2019
@k8s-ci-robot k8s-ci-robot merged commit 77b60f4 into kubernetes:master Dec 9, 2019
@k8s-ci-robot k8s-ci-robot modified the milestones: v1.17, v1.18 Dec 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/release-eng Issues or PRs related to the Release Engineering subproject area/release-team Issues or PRs related to the release-team subproject cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. committee/security-response Denotes an issue or PR intended to be handled by the product security committee. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. kind/documentation Categorizes issue or PR as related to documentation. lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. sig/release Categorizes an issue or PR as relevant to SIG Release. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.