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

Initial commit for CNCF incubating due diligence #111

Merged
merged 9 commits into from
Sep 7, 2021
Merged

Initial commit for CNCF incubating due diligence #111

merged 9 commits into from
Sep 7, 2021

Conversation

mazzystr
Copy link
Contributor

@mazzystr mazzystr commented Jun 1, 2021 •
edited
Loading

Initial commit for CNCF incubating due diligence

We will submit this document to toc/pulls

Fixes #96

doc built using accepted due diligence forms from Falco and Thanos

@mazzystr
Initial commit for CNCF incubating due diligence
247e1e7 
Signed-off-by: Chris Callegari <mazzystr@gmail.com>
@kubevirt-bot kubevirt-bot added the dco-signoff: yes Indicates the PR's author has DCO signed all their commits. label Jun 1, 2021
@mazzystr mazzystr removed the request for review from codificat June 1, 2021 22:06
@mazzystr mazzystr self-assigned this Jun 1, 2021
@mazzystr mazzystr added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. enhancement and removed do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. labels Jun 1, 2021
@kubevirt-bot kubevirt-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jun 1, 2021
@mazzystr
Add what is kubevirt
81e89c2 
Signed-off-by: Chris Callegari <mazzystr@gmail.com>
@kubevirt-bot kubevirt-bot added size/M and removed size/S labels Jun 1, 2021
mazzystr added 3 commits June 1, 2021 17:14
@mazzystr
Added community section
a832076 
Signed-off-by: Chris Callegari <mazzystr@gmail.com>
@mazzystr
Added metrics section
1050cf6 
Signed-off-by: Chris Callegari <mazzystr@gmail.com>
@mazzystr
Add items to Roadmap
3e0ebc1 
Signed-off-by: Chris Callegari <mazzystr@gmail.com>
@mazzystr
Copy link
Contributor Author

mazzystr commented Jun 4, 2021 •
edited
Loading

@davidvossel I added some items to the roadmap regarding ver 1.0. Can you think of other items to add? Feel free to comment, review or add a commit. Thanks.

davidvossel
davidvossel reviewed Jun 21, 2021
View reviewed changes
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
proposals/governance/incubation_dd.md Show resolved Hide resolved
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
proposals/governance/incubation_dd.md Show resolved Hide resolved
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
@mazzystr
Add more items to Roadmap
eb00ebf 
Signed-off-by: Chris Callegari <mazzystr@gmail.com>
@fabiand
Copy link
Member

fabiand commented Jul 13, 2021 •
edited
Loading

/hold

I'd appreciate if at least 3 active maintainers can approve this PR before we are merging it

i.e. (but not limited to) @rmohr @vladikr @mhenriks @AlonaKaplan @dhiller …

@kubevirt-bot kubevirt-bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 13, 2021
@jberkus
Copy link
Contributor

jberkus commented Jul 13, 2021

Hey, so we queried two members of the TOC on Friday. Here's what they'd like to see us fill in on the DD:

  1. Our current community contributions information
  2. Any programs/ongoing efforts we have to recruit/promote contributors
  3. Our participation in CNCF committees and interlock with other CNCF projects

fabiand
fabiand reviewed Jul 21, 2021
View reviewed changes
Copy link
Member

@fabiand fabiand left a comment

Choose a reason for hiding this comment

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

Submitting my comments that have been pending for a while

proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
proposals/governance/incubation_dd.md Show resolved Hide resolved
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
dhiller
dhiller reviewed Jul 22, 2021
View reviewed changes
Copy link
Contributor

@dhiller dhiller left a comment

Choose a reason for hiding this comment

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

Looks good to me in general, I have only nits to comment on.

/approve

proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
proposals/governance/incubation_dd.md Outdated Show resolved Hide resolved
@jberkus
Copy link
Contributor

jberkus commented Aug 6, 2021

Hey, to make it easier to collaborate on this, I've created a hackmd document: https://hackmd.io/OMiyZrpJREypqUUPXlyhGw

@mazzystr
Adjustments per David Vossel
2262bcd 
Signed-off-by: Chris Callegari <mazzystr@gmail.com>
@mazzystr
Updates per fabiand and dhiller
4e00f43 
Signed-off-by: Chris Callegari <mazzystr@gmail.com>
@mazzystr
Adding edits from Josh B.
5a163a6 
Signed-off-by: Chris Callegari <mazzystr@gmail.com>
@mazzystr
Copy link
Contributor Author

mazzystr commented Sep 7, 2021

@jberkus I think I have your edits from hack integrated. It's hard to tell since there is no diff without a lot of copy pasta.

@mazzystr
Copy link
Contributor Author

mazzystr commented Sep 7, 2021

Karena wants us to merge this so we can continue on with merging to the CNCF ToC

@mazzystr
Copy link
Contributor Author

mazzystr commented Sep 7, 2021

/lgtm
/approve

@kubevirt-bot
Copy link

@mazzystr: you cannot LGTM your own PR.

In response to this:

/lgtm
/approve

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.

@kubevirt-bot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dhiller, mazzystr

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

@kubevirt-bot kubevirt-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 7, 2021
@mazzystr mazzystr removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 7, 2021
@kwiesmueller
Copy link

/lgtm

@kubevirt-bot kubevirt-bot added the lgtm Indicates that a PR is ready to be merged. label Sep 7, 2021
@kubevirt-bot kubevirt-bot merged commit d0ef43c into kubevirt:master Sep 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fabiand fabiand fabiand left review comments

@jberkus jberkus jberkus left review comments

@davidvossel davidvossel davidvossel left review comments

@dhiller dhiller dhiller left review comments

@peterclauterbach peterclauterbach peterclauterbach left review comments

@dankenigsberg dankenigsberg dankenigsberg left review comments

@rmohr rmohr rmohr left review comments

Assignees

@mazzystr mazzystr

@kwiesmueller kwiesmueller

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. enhancement lgtm Indicates that a PR is ready to be merged. size/L
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Promote KubeVirt to a CNCF incubating project
10 participants
@mazzystr @fabiand @jberkus @kubevirt-bot @kwiesmueller @davidvossel @dhiller @peterclauterbach @dankenigsberg @rmohr