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

Add clarity wrt SIGs that need to be projects #348

Merged
merged 4 commits into from
Jun 28, 2024

Conversation

mihaimaruseac
Copy link
Contributor

When SIGs produce code or specification, they need to be projects, but this was not fully clear before. Aiming to clarify this.

When SIGs produce code or specification, they need to be projects, but
this was not fully clear before. Aiming to clarify this.

Signed-off-by: Mihai Maruseac <mihaimaruseac@google.com>
@mihaimaruseac mihaimaruseac requested a review from a team as a code owner June 20, 2024 20:53
@marcelamelara
Copy link
Contributor

marcelamelara commented Jun 21, 2024

@mihaimaruseac Thanks for your PR. While I agree that clarifications in the process are needed, the distinction between SIG and Project isn't strictly about code vs. no code. It's about scope, rather.

SIGs are meant to be more independently managed within the hosting TI, considerably narrower in scope, and have a shorter lifetime. But this doesn't preclude the SIG from producing code that is experimental or a PoC for a spec. On the other hand, if the SIG eventually intends to create production-quality software beyond PoC, that's when I think it makes sense to migrate to a Project designation a Project designation since the Project lifecycle includes more software-related requirements.

Again, I think our process docs would benefit from clarity around these details, but I don't think we should discourage SIGs from producing any code at all. These are my thoughts at least, others on the TAC may have a different view.

@lehors
Copy link
Contributor

lehors commented Jun 22, 2024

I agree with @marcelamelara. I think the proposed wording is a bit too strong. Our [governance structure]9https://github.com/ossf/tac/blob/main/organizational-structure-overview.md) states that the "primary output" of a SIG is "not software". This implies that it could be a secondary output and clearly does not prohibit any software development.
We discussed cases where a SIG might develop some small tool to support its activities and that would be ok. So the proposed wording should be revised to align with that.
Thanks.

process/Technical_Initiative_Lifecycle.md Outdated Show resolved Hide resolved
process/sig-lifecycle.md Outdated Show resolved Hide resolved
Signed-off-by: Mihai Maruseac <mihai.maruseac@gmail.com>
@mihaimaruseac
Copy link
Contributor Author

Thank you for reviewing. I've made some amendments to the wording to include the feedback.

@marcelamelara marcelamelara added the Content Updates/additions to TAC content/process. Must include a changelog entry. Needs 3 approvals. label Jun 26, 2024
Copy link
Contributor

@SecurityCRob SecurityCRob left a comment

Choose a reason for hiding this comment

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

+1 on revised changes

Copy link
Contributor

@marcelamelara marcelamelara left a comment

Choose a reason for hiding this comment

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

Thanks for the revisions @mihaimaruseac ! LGTM.

process/Technical_Initiative_Lifecycle.md Outdated Show resolved Hide resolved
@marcelamelara
Copy link
Contributor

I also think we need a changelog entry for this one? Or should we consider this an editorial change?

Co-authored-by: Marcela Melara <marcela.melara@intel.com>
Signed-off-by: Mihai Maruseac <mihai.maruseac@gmail.com>
@mihaimaruseac
Copy link
Contributor Author

I also think we need a changelog entry for this one? Or should we consider this an editorial change?

I can add one if needed.

Signed-off-by: Mihai Maruseac <mihai.maruseac@gmail.com>
@mihaimaruseac mihaimaruseac force-pushed the update-documentation branch from b9d5446 to 8220606 Compare June 27, 2024 20:57
@marcelamelara
Copy link
Contributor

Thanks so much @mihaimaruseac !

Copy link
Contributor

@lehors lehors left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

Copy link
Contributor

@mlieberman85 mlieberman85 left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@torgo torgo left a comment

Choose a reason for hiding this comment

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

lgtm

@lehors lehors merged commit d102336 into ossf:main Jun 28, 2024
4 checks passed
@mihaimaruseac mihaimaruseac deleted the update-documentation branch June 29, 2024 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content Updates/additions to TAC content/process. Must include a changelog entry. Needs 3 approvals.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants