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

Cortex incubation review #315

Merged
merged 1 commit into from Aug 20, 2020
Merged

Conversation

gouthamve
Copy link
Contributor

@gouthamve gouthamve commented Nov 3, 2019

Cortex is currently a CNCF Sandbox project which joined more than an year ago. Over the last year we've grown as a project and community and we would like to be considered to move into incubating stage.

/cc @amye @caniszczyk

Signed-off-by: Goutham Veeramachaneni <gouthamve@gmail.com>
@amye
Copy link
Contributor

amye commented Nov 3, 2019

We are currently scheduling for December and beyond, is there a preferred time that you'd like to present an incubating review?

@gouthamve
Copy link
Contributor Author

gouthamve commented Nov 4, 2019

@amye Anytime after December 10th should be fine, thanks!

@caniszczyk caniszczyk added this to In progress (due diligence/presentation) in Initial Project Triage & Sandbox Projects Backlog via automation Nov 4, 2019
@caniszczyk
Copy link
Contributor

caniszczyk commented Nov 4, 2019

RFC @cncf/toc for any due diligence volunteers

This may have to wait until after kubecon / early next year to be honest

@gouthamve
Copy link
Contributor Author

gouthamve commented Dec 5, 2019

@amye Just a gentle reminder now that Kubecon has now died down :) Could you let us know tentative dates when this could be scheduled?

@amye
Copy link
Contributor

amye commented Dec 5, 2019

Currently working with TOC to outline priorities for next year, suggest getting a member of the TOC for due diligence as next steps.

# Cortex proposal for CNCF Incubation

## Background
[Request for Incubation](https://github.com/cncf/toc/pull/125)
Copy link
Contributor

@bwplotka bwplotka Jan 23, 2020

Choose a reason for hiding this comment

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

Suggested change
[Request for Incubation](https://github.com/cncf/toc/pull/125)
[Request for Sandbox](https://github.com/cncf/toc/pull/125)

@lizrice
Copy link
Contributor

lizrice commented Feb 24, 2020

Next step for this will be a recommendation by SIG Observability once that SIG is up and running cc @halcyondude

@amye amye added this to Needs TOC Triage & Public Comment Kickoff in Graduating Projects Backlog Mar 13, 2020
@amye amye removed this from Needs TOC Triage & Public Comment Kickoff in Graduating Projects Backlog Mar 13, 2020
@michelleN michelleN removed this from In progress (due diligence/presentation) in Initial Project Triage & Sandbox Projects Backlog Mar 13, 2020
@michelleN michelleN added this to Needs SIG Recommendation in Incubation Projects Backlog Mar 13, 2020
@caniszczyk caniszczyk added the tag-observability label Mar 19, 2020
@RichiH
Copy link
Contributor

RichiH commented May 26, 2020

SIG o11y reviewed the due dilligence document linked at https://docs.google.com/document/d/1mCS5C9wQxEyFQrknaY4nFvgWR34ZKUgFH7I2osMhbIY/edit and reached consensus as follows: "SIG o11y accepts the Cortex due diligence document, will submit it to CNCF TOC for further discussion, and suggest moving Cortex to incubation stage" as documented in https://docs.google.com/document/d/1_QoF-njScSuGFI3Ge5zu-G8SbL6scQ8AzT1hq57bRoQ/edit# .

@RichiH RichiH mentioned this pull request May 26, 2020
@amye amye moved this from Needs SIG Review & Recommendation to Needs TOC Review & Sponsor in Incubation Projects Backlog Jun 11, 2020
@kgamanji
Copy link

kgamanji commented Jun 16, 2020

I will conduct the DD for Cortex. Will reach out with more details once I have reviewed the DD

@kgamanji
Copy link

kgamanji commented Jun 29, 2020

I have completed the DD for Cortex. Here are my takeaways:

  • the project is a great addition to the CNCF ecosystem and has a healthy governance and contributor base
  • overall the project meets the incubation criteria

A few points I would like more clarity on:

  • a well-defined roadmap
  • seems like some of the details presented for the project are outdated (e.g. introduction of single-binary mode and indexed documentation). It might be the case that the details were filled for the sandbox integration, but would be good to have updated information
  • a clear understanding on the dependency introduced around issue reporting with systems from Weaveworks and Grafana Labs

I have outlined these points on the DD doc as well. Once I have more details on the above, I am happy to proceed to the next step.

@gouthamve
Copy link
Contributor Author

gouthamve commented Jul 7, 2020

Hi Katie, thanks for the feedback! We're currently working on your feedback: removing dependency on 2 companies and adding a public roadmap.

I will update here once both the changes are made.

@gouthamve
Copy link
Contributor Author

gouthamve commented Jul 9, 2020

Hi Katie, we've made the changes requested. They are good suggestions! I've also started reaching out to potential end-users to encourage them to participate in the release process.

Please let us know if you have any more questions or suggestions!

@kgamanji
Copy link

kgamanji commented Jul 10, 2020

Amazing! Thank you for introducing the changes so promptly.

At this stage I am happy to move forward with the process and open the public comment period for Cortex: https://lists.cncf.io/g/cncf-toc/message/4961

@amye amye moved this from Needs TOC Review & Sponsor to In Due Diligence in Incubation Projects Backlog Jul 23, 2020
@amye amye moved this from In Due Diligence to In Public Comment Period in Incubation Projects Backlog Jul 23, 2020
@amye amye moved this from In Public Comment Period to In TOC Vote in Incubation Projects Backlog Jul 28, 2020
@caniszczyk
Copy link
Contributor

caniszczyk commented Aug 20, 2020

+1 approved by the TOC, welcome to incubation!

8/11
Katie Gamanji: https://lists.cncf.io/g/cncf-toc/message/5035
Xiang Li: https://lists.cncf.io/g/cncf-toc/message/5052
Sheng Liang: https://lists.cncf.io/g/cncf-toc/message/5054
Justin Cormack: https://lists.cncf.io/g/cncf-toc/message/5057
Alena Prokharchyk: https://lists.cncf.io/g/cncf-toc/message/5069
Brendan Burns https://lists.cncf.io/g/cncf-toc/message/5167
Liz Rice: https://lists.cncf.io/g/cncf-toc/message/5171
Michelle Noorali: https://lists.cncf.io/g/cncf-toc/message/5173

@caniszczyk caniszczyk merged commit 6bc10c7 into cncf:master Aug 20, 2020
@amye amye moved this from In TOC Vote to Done in Incubation Projects Backlog Sep 16, 2020
@amye amye removed this from Done in Incubation Projects Backlog Sep 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
incubation tag-observability
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

7 participants