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.

Sign up for GitHub

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

Jump to bottom

SIG Security 2021 annual report #6454

Merged
merged 3 commits into from
Apr 12, 2022
Merged

SIG Security 2021 annual report #6454

merged 3 commits into from
Apr 12, 2022

Conversation

IanColdwater
Copy link
Contributor

Which issue(s) this PR fixes:

Fixes #6380

First draft of SIG Security 2021 annual report. WILLFIX

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 14, 2022
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: IanColdwater

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

@k8s-ci-robot k8s-ci-robot added the sig/security Categorizes an issue or PR as relevant to SIG Security. label Feb 14, 2022
@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 14, 2022
@IanColdwater
Copy link
Contributor Author

/hold

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 14, 2022
@tabbysable
Copy link
Member

/label tide/merge-strategy-squash

@k8s-ci-robot
Copy link
Contributor

@tabbysable: The label(s) /label tide/merge-strategy-squash cannot be applied. These labels are supported: api-review, tide/merge-method-merge, tide/merge-method-rebase, tide/merge-method-squash, team/katacoda, refactor

In response to this:

/label tide/merge-strategy-squash

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@tabbysable
Copy link
Member

/label tide/merge-method-squash

@k8s-ci-robot k8s-ci-robot added tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Mar 1, 2022
@IanColdwater IanColdwater changed the title initial commit for SIG Security 2021 annual report SIG Security 2021 annual report Mar 1, 2022
@tabbysable
Copy link
Member

I read through the rendered preview and picked a bunch of Markdown nits using GitHub suggestion mode.

@tabbysable
Copy link
Member

cc @parispittman

@parispittman
Copy link
Contributor

/assign

parispittman
parispittman suggested changes Mar 3, 2022
View reviewed changes
Copy link
Contributor

@parispittman parispittman left a comment

Choose a reason for hiding this comment

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

this looks GREAT!! thank you so much for your thoughtful responses. You all have really grown since the last time we did this. :) I have small nits and in the meantime, feel free to email this PR to your SIG mailing list/slack with whatever your usual lazy consensus period (72 hours?) is for last calls for comments before merging.

sig-security/annual-report-2021.md Show resolved Hide resolved
sig-security/annual-report-2021.md Show resolved Hide resolved
sig-security/annual-report-2021.md Outdated
Any areas with 2 or fewer OWNERs? (link to more details)
**1. What areas and/or subprojects does your group need the most help with?**

- Our Docs subproject is always looking for security-minded contributors of all experience levels to share their learning and knowledge with the community! This subproject has consistently been a place where people [merge](https://github.com/kubernetes/website/pull/28248) their [first](https://github.com/kubernetes/website/pull/31518) Kubernetes PRs.There’s always room for continuous improvement in our documentation, and contributing to this provides an opportunity to learn more about Kubernetes security while helping everyone run their clusters more safely. We’re really proud of the way Docs encourages and welcomes new contributors, and we’d love to encourage you to become a part of it!
Copy link
Contributor

Choose a reason for hiding this comment

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

at the end of this, how can someone be apart of it - show up at a meeting? where? slack channel? same q

@parispittman parispittman mentioned this pull request Apr 4, 2022
Closed
31 tasks
@IanColdwater @tabbysable @christophetd
Apply suggestions from report review
cb95812 
Co-authored-by: Tabitha Sable <51767484+tabbysable@users.noreply.github.com>
Co-authored-by: Christophe Tafani-Dereeper <christophe@tafani-dereeper.me>
@parispittman
Copy link
Contributor

/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 Apr 12, 2022
@parispittman
Copy link
Contributor

/lgtm

thanks team!

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 12, 2022
@k8s-ci-robot k8s-ci-robot merged commit 3b21eda into kubernetes:master Apr 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@christophetd christophetd christophetd left review comments

@tabbysable tabbysable tabbysable left review comments

@parispittman parispittman parispittman requested changes

Assignees

@parispittman parispittman

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/security Categorizes an issue or PR as relevant to SIG Security. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2021 Annual Report: SIG Security
5 participants
@IanColdwater @k8s-ci-robot @tabbysable @parispittman @christophetd