Skip to content

[chore] Replace sigs.yml by workstreams.yml#3366

Merged
mx-psi merged 7 commits intoopen-telemetry:mainfrom
mx-psi:mx-psi/workstream-data-model
Apr 22, 2026
Merged

[chore] Replace sigs.yml by workstreams.yml#3366
mx-psi merged 7 commits intoopen-telemetry:mainfrom
mx-psi:mx-psi/workstream-data-model

Conversation

@mx-psi
Copy link
Copy Markdown
Member

@mx-psi mx-psi commented Apr 14, 2026

This PR is a subset of #3337 in the hopes of finding one sub-part that can be merged.

It tries to move from sigs.yml to workstreams.yml without making any modifications on the TC sponsors or add non-SIG projects. The intent is to tackle all of these on separate PRs.

@mx-psi mx-psi force-pushed the mx-psi/workstream-data-model branch 2 times, most recently from f7459ac to e74d209 Compare April 14, 2026 14:16
Co-Authored-By: Jack Berg <34418638+jack-berg@users.noreply.github.com>
Assisted-By: Claude Opus 4.6
@mx-psi mx-psi force-pushed the mx-psi/workstream-data-model branch from e74d209 to e69d47b Compare April 14, 2026 16:32
@mx-psi mx-psi marked this pull request as ready for review April 14, 2026 16:37
Comment thread README.md
Copy link
Copy Markdown
Member

@jack-berg jack-berg 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. I checked out locally, did a diff with my branch, and determined that this is the same as my branch, with the following changes:

  • Strip out notion of working group from workstreams data model
  • Strip out TC sponsorship assignments not already encoded in sigs.yml (Note, I didn't pull that out of thin air, the assignments came from this private TC sponsorship tracking doc. Still, makes sense to decouple.)
  • Strip out make report-generation and resulting workstreams.md
  • Other little things like keeping make validate-sigs instead of make validate-workstreams

Thanks for picking this up @mx-psi!

@mx-psi
Copy link
Copy Markdown
Member Author

mx-psi commented Apr 15, 2026

Looks good to me. I checked out locally, did a diff with my branch, and determined that this is the same as my branch, with the following changes:

Thanks for taking a look, sorry, I should have added a list like this to the README.

I intend to add all other changes you mention in future PRs, on this one I wanted to focus on changing the data model

Comment thread scripts/schema/workstreams.schema.yml Outdated
Comment thread scripts/schema/workstreams.schema.yml Outdated
Comment thread scripts/update-sig-tables.py Outdated
mx-psi and others added 2 commits April 15, 2026 18:30
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
@mx-psi mx-psi requested a review from trask April 15, 2026 16:33
Comment thread project-template.md Outdated
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
@mx-psi
Copy link
Copy Markdown
Member Author

mx-psi commented Apr 20, 2026

I will merge this on Wednesday morning unless there are blocking comments before then

Copy link
Copy Markdown
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.

LGTM. We could perhaps keep a sigs.yml pointing to the new one, but that's definitely not a blocker.

I accidentally added it back after the last merge
@mx-psi
Copy link
Copy Markdown
Member Author

mx-psi commented Apr 21, 2026

LGTM. We could perhaps keep a sigs.yml pointing to the new one, but that's definitely not a blocker.

I removed sigs.yml, sorry about that, I botched the merge. I think we can always revert back to sigs.yml if needed

@mx-psi mx-psi added this pull request to the merge queue Apr 22, 2026
Merged via the queue into open-telemetry:main with commit f537af7 Apr 22, 2026
8 checks passed
@mx-psi mx-psi deleted the mx-psi/workstream-data-model branch April 22, 2026 09:56
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.

6 participants