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

Create sigs.yml and add GC liaisons to README.md #1987

Merged
merged 31 commits into from
Apr 17, 2024

Conversation

svrnm
Copy link
Member

@svrnm svrnm commented Mar 5, 2024

Follow up to #1955, this PR adds a GC liaison column such that one can be appointed per SIG.

@open-telemetry/governance-committee let's go through that list in sigs.yml and appoint one of us per SIG. I added a few names already, but they are in no way prescriptive, I just needed some examples for testing.


This PR also introduces another big change: since the SIG tables are growing quickly in the number of rows and columns (like in this PR), I propose that we extract the SIGs into a YAML file and generate the table from that. I copied the overall logic from how we generate tables in the semantic conventions.

The big advantage of this change is that it is now much easier to grow that list in both directions, the other one is that we can now apply validations, e.g. make sure that each SIG has a GC liaison, that each SIG has a slack channel.

Signed-off-by: svrnm <neumanns@cisco.com>
sigs.yml Outdated Show resolved Hide resolved
sigs.yml Outdated Show resolved Hide resolved
sigs.yml Outdated Show resolved Hide resolved
Signed-off-by: svrnm <neumanns@cisco.com>
sigs.yml Outdated Show resolved Hide resolved
Copy link
Member Author

@svrnm svrnm left a comment

Choose a reason for hiding this comment

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

@open-telemetry/governance-committee please take a look and self-assign yourself to the different SIGs as it makes sense, based on existing relationships and other criteria where you think you are a good liaison person for an individual SIG

sigs.yml Show resolved Hide resolved
sigs.yml Outdated Show resolved Hide resolved
sigs.yml Outdated Show resolved Hide resolved
sigs.yml Outdated Show resolved Hide resolved
sigs.yml Outdated Show resolved Hide resolved
sigs.yml Outdated Show resolved Hide resolved
sigs.yml Show resolved Hide resolved
sigs.yml Outdated Show resolved Hide resolved
sigs.yml Outdated Show resolved Hide resolved
sigs.yml Outdated Show resolved Hide resolved
trask and others added 2 commits March 7, 2024 10:25
Co-authored-by: Severin Neumann <neumanns@cisco.com>
Co-authored-by: Severin Neumann <neumanns@cisco.com>
@trask
Copy link
Member

trask commented Mar 7, 2024

I didn't list the database semconv stability SIG isn't listed on the community page since it's expected to be short-lived, but I probably should have. I can be GC liaison for it.

@svrnm
Copy link
Member Author

svrnm commented Mar 7, 2024

I didn't list the database semconv stability SIG isn't listed on the community page since it's expected to be short-lived, but I probably should have. I can be GC liaison for it.

Makes it harder to track of course, but I think it can work this way as well. This here is mostly useful for long- to medium living SIGs

sigs.yml Show resolved Hide resolved
sigs.yml Show resolved Hide resolved
sigs.yml Outdated Show resolved Hide resolved
scripts/validate-sigs.py Outdated Show resolved Hide resolved
Co-authored-by: Morgan McLean <morgan_mclean@hotmail.com>
sigs.yml Outdated Show resolved Hide resolved
sigs.yml Outdated Show resolved Hide resolved
Copy link
Member

@jpkrohling jpkrohling left a comment

Choose a reason for hiding this comment

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

I'm approving this, meaning that I'm OK in being the liaison for:

  • Collector
  • Go Instrumentation
  • Operator
  • Security

sigs.yml Outdated Show resolved Hide resolved
sigs.yml Outdated Show resolved Hide resolved
Signed-off-by: svrnm <neumanns@cisco.com>
README.md Outdated Show resolved Hide resolved
Signed-off-by: svrnm <neumanns@cisco.com>
Signed-off-by: svrnm <neumanns@cisco.com>
Signed-off-by: svrnm <neumanns@cisco.com>
@svrnm svrnm merged commit f111f34 into open-telemetry:main Apr 17, 2024
3 checks passed
@svrnm svrnm deleted the create-sigs-yml-add-gc-liaisons branch April 17, 2024 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants