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

Update the contributor-strategy.md information page for enduring validity. #1126

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

riaankleinhans
Copy link
Contributor

The update aims to provide stable, timeless information that avoids references to individuals, working groups, and objectives subject to change, reducing maintenance efforts while ensuring up-to-date content.

Fix issue: cncf/tag-contributor-strategy#464

Signed-off-by: Riaan Kleinhans <[email protected]>
@aliok
Copy link
Member

aliok commented Jul 20, 2023

timeless information

+1 on this!

When I check the charter documents of other TAGs in this repository, I see they all have content and not just links to the charter documents in their own repository. Ex: network, app-delivery

IMO, it would be much better to have a single charter document. ...and, ideally all TAG charter docs in this repository should also go with the same approach.

tags/contributor-strategy.md Outdated Show resolved Hide resolved
tags/contributor-strategy.md Outdated Show resolved Hide resolved
tags/contributor-strategy.md Outdated Show resolved Hide resolved
riaankleinhans and others added 3 commits July 21, 2023 06:00
Co-authored-by: Ali Ok <[email protected]>
Signed-off-by: Riaan Kleinhans <[email protected]>
Co-authored-by: Ali Ok <[email protected]>
Signed-off-by: Riaan Kleinhans <[email protected]>
Co-authored-by: Ali Ok <[email protected]>
Signed-off-by: Riaan Kleinhans <[email protected]>
@jberkus
Copy link
Contributor

jberkus commented Jul 20, 2023

@TheFoxAtWork @amye

This raises a general question for the TOC.

Currently, most TAGs have duplicate (and inconsistent) information in the toc/tags folder, and in their individual repo READMEs. Where should canonical TAG information live? It needs to be one or the other, because clearly maintaining the same text in both places doesn't work.

Copy link
Contributor

@TheFoxAtWork TheFoxAtWork left a comment

Choose a reason for hiding this comment

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

I am generally in support of this so long as each TAG has governance in place to ensure any changes to their Charter received concurrence from the TOC prior to merge.
@jeefy given Amye is out, do you know if there is any reason TAG Charters need to exist within the TOC repo if we have processes in place to ensure updates are approved by the TOC?

@amye
Copy link
Contributor

amye commented Jul 31, 2023

I am generally in support of this so long as each TAG has governance in place to ensure any changes to their Charter received concurrence from the TOC prior to merge. @jeefy given Amye is out, do you know if there is any reason TAG Charters need to exist within the TOC repo if we have processes in place to ensure updates are approved by the TOC?

This was sort of organically grown. We wanted one place to track all of the charters when they were approved, but given as the charters have shifted, that may not be as relevant anymore.

@amye
Copy link
Contributor

amye commented Oct 10, 2023

We did some backlog grooming and I took this one -I've had a chance to review and this isn't quite the right place for this.
All of the other TAGs have their charters listed here, (and we'd end up undoing all of those too) so we have two options:
(1) Rename this one to be more of 'about TAG Contributor Strategy'
(2) Put this in the TAG CS repo.

@Riaankl - does that make sense?

Signed-off-by: Riaan Kleinhans <[email protected]>
@riaankleinhans
Copy link
Contributor Author

We did some backlog grooming and I took this one -I've had a chance to review and this isn't quite the right place for this. All of the other TAGs have their charters listed here, (and we'd end up undoing all of those too) so we have two options: (1) Rename this one to be more of 'about TAG Contributor Strategy' (2) Put this in the TAG CS repo.

@Riaankl - does that make sense?

@amye yes, renaming it as "About CNCF TAG Contributor Strategy" is a good idea.

Our roadmapping and clean-up that we are working on in the TAG CS repo leaked out into this document.
The Charter here was a very dated copy of what is in the repo, so we went for less content that would hopefully have a longer shelf life and points to our repo. Then we only need to keep the TAG's own repo a tidy and as fresh as possible over time.

TIA

@amye
Copy link
Contributor

amye commented Oct 11, 2023

Ok, so yes - but we're still overwriting the charter. Let's do this:
(1) Add an updated version of the TAG CS charter so that there's still continuity
(2) Rename this file

@riaankleinhans
Copy link
Contributor Author

riaankleinhans commented Oct 11, 2023

The charter live in the TAG CS repo.They got badly out of sync because there are two copies.
https://github.com/cncf/tag-contributor-strategy/blob/main/CHARTER.md

My thought about this was under "TOC/TAG" directory we only have a intro to each TAG that would not get dated, and a link to their repo that include their charter. (Kind of a restructuring proposal for all, but leading by example)

@riaankleinhans riaankleinhans added the process-documentation Doc changes for process and procedures label Apr 29, 2024
@TheFoxAtWork
Copy link
Contributor

@riaankleinhans can you confirm if this PR is up to date with the recent recommendations and changes regarding the TAGs? If so, could you update the PR to resolve the existing conflicts?

@riaankleinhans
Copy link
Contributor Author

No, this need to be updated.
Working with TAG CS to update their charter.

@amye amye removed their assignment May 29, 2024
@angellk
Copy link
Contributor

angellk commented Oct 11, 2024

@riaankleinhans is there a status on this PR? Has TAG CS updated their charter?

Copy link
Contributor

@TheFoxAtWork TheFoxAtWork left a comment

Choose a reason for hiding this comment

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

LGTM

@TheFoxAtWork
Copy link
Contributor

even with the charter out of date, it is outside the scope of this PR as i understand it. @riaankleinhans @mrbobbytables could you confirm so we can get this merged?

@linsun
Copy link
Contributor

linsun commented Dec 17, 2024

cc @linsun to review

@linsun
Copy link
Contributor

linsun commented Dec 17, 2024

@riaankleinhans could you resolve the merge conflicts pls?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
process-documentation Doc changes for process and procedures
Projects
Status: Ready for assignment
Development

Successfully merging this pull request may close these issues.

7 participants