Skip to content

wg-node-identity: add working group charter and registration - #9028

Open
randomvariable wants to merge 1 commit into
kubernetes:mainfrom
randomvariable:topic/randomvariable/node-identity-wg
Open

wg-node-identity: add working group charter and registration#9028
randomvariable wants to merge 1 commit into
kubernetes:mainfrom
randomvariable:topic/randomvariable/node-identity-wg

Conversation

@randomvariable

@randomvariable randomvariable commented Jun 15, 2026

Copy link
Copy Markdown
Member

What this does

Registers a new Working Group: WG Node Identity.

Files changed

  • sigs.yaml — new WG entry under workinggroups
  • OWNERS_ALIASESwg-node-identity-leads alias (chairs to be populated post-approval)
  • sig-cluster-lifecycle/README.md — generated: WG listed under sponsoring SIG
  • sig-list.md — generated: WG row added
  • wg-node-identity/README.md — generated from sigs.yaml
  • wg-node-identity/charter.md — new charter

Known TODOs (to be resolved before merge)

  • Slack channel name (#tbd placeholder)
  • Mailing list URL (not yet created)
  • Meeting schedule (to be decided by initial WG meeting)
  • wg-node-identity-leads OWNERS alias populated once chairs confirm

Checklist

  • sigs.yaml edited directly
  • Docs regenerated with make generate
  • Charter follows wg-governance

@k8s-ci-robot

Copy link
Copy Markdown
Contributor

There are empty aliases in OWNER_ALIASES, cleanup is advised.

@linux-foundation-easycla

linux-foundation-easycla Bot commented Jun 15, 2026

Copy link
Copy Markdown

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: ahmedtd / name: Taahir Ahmed (e99d345)
  • ✅ login: elmiko / name: Michael McCune (e99d345)
  • ✅ login: hakman / name: Ciprian Hacman (e99d345)
  • ✅ login: pfeifferj / name: Josephine Pfeiffer (e99d345)
  • ✅ login: randomvariable / name: Naadir Jeewa (e99d345)
  • ✅ login: rata / name: Rodrigo Campos Catelin (e99d345)

@k8s-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: randomvariable
Once this PR has been reviewed and has the lgtm label, please assign cblecker for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. label Jun 15, 2026
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. committee/steering Denotes an issue or PR intended to be handled by the steering committee. sig/cluster-lifecycle Categorizes an issue or PR as relevant to SIG Cluster Lifecycle. labels Jun 15, 2026
@randomvariable

Copy link
Copy Markdown
Member Author

/hold

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jun 15, 2026
@randomvariable
randomvariable force-pushed the topic/randomvariable/node-identity-wg branch from 526b0cb to 1daa9ab Compare June 15, 2026 23:02
@k8s-ci-robot

Copy link
Copy Markdown
Contributor

There are empty aliases in OWNER_ALIASES, cleanup is advised.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Jun 15, 2026
@randomvariable
randomvariable force-pushed the topic/randomvariable/node-identity-wg branch from 1daa9ab to cec3d57 Compare June 15, 2026 23:04
@k8s-ci-robot

Copy link
Copy Markdown
Contributor

There are empty aliases in OWNER_ALIASES, cleanup is advised.

@randomvariable
randomvariable marked this pull request as draft June 15, 2026 23:05
@k8s-ci-robot k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jun 15, 2026
@hakman

hakman commented Jun 29, 2026

Copy link
Copy Markdown
Member

/cc @neolit123

@kubernetes-prow
kubernetes-prow Bot requested a review from neolit123 June 29, 2026 16:53
@hakman

hakman commented Jun 29, 2026

Copy link
Copy Markdown
Member

/cc @justinsb

@kubernetes-prow
kubernetes-prow Bot requested a review from justinsb June 29, 2026 16:54

@neolit123 neolit123 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/approve
in SIGL CL we had consensus this WG to be sponsored.
personally, i mentioned i don't clearly understand what will be product of this WG but let's see.

  • there could definitely other participating SIGs here, did you get +1 from any others?
  • has the steering committee +1-ed?

@kubernetes-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: neolit123, randomvariable
Once this PR has been reviewed and has the lgtm label, please assign cblecker for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@randomvariable
randomvariable force-pushed the topic/randomvariable/node-identity-wg branch from cec3d57 to 616806d Compare June 30, 2026 00:03
@randomvariable
randomvariable marked this pull request as ready for review June 30, 2026 00:03
@kubernetes-prow kubernetes-prow Bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jun 30, 2026
## Working Groups

The following [working groups][working-group-definition] are sponsored by sig-cluster-lifecycle:
* [WG Node Identity](/wg-node-identity)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

sig autoscaling and other sponsors needs to be updated too.
but note they must have agreed to it during a recorded zoom meeting.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

is sig auth participating?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

In some form we need confirmation that the relevant SIGs are in principle +1 and ideally have reviewed teh proposal for any issues they see.

Comment thread wg-node-identity/charter.md Outdated
@kubernetes-prow kubernetes-prow Bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 1, 2026
@hakman

hakman commented Jul 16, 2026

Copy link
Copy Markdown
Member

/test pull-community-verify

@elmiko

elmiko commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

although not a sponsor, i am +1 from the sig cloud provider side.

@BenTheElder

Copy link
Copy Markdown
Member

FYI @kubernetes/steering-committee

I think we mainly need evidence of sponsoring SIGs reviewing / agreeing, if you can help collect that then we can move to reviewing the charter in more depth after their feedback. Overall I personally think this looks reasonable at a high level.

@jackfrancis

Copy link
Copy Markdown
Contributor

+1 from SIG Autoscaling

cc @towca @omerap12 @adrianmoisey

@elmiko

elmiko commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

@ahmedtd given this comment should we add you to the charter as a chair as well?

@ahmedtd

ahmedtd commented Jul 28, 2026

Copy link
Copy Markdown

@ahmedtd given this comment should we add you to the charter as a chair as well?

I would be happy to, but I don't know how many chairs a WG is supposed to have.

@BenTheElder

Copy link
Copy Markdown
Member

I would be happy to, but I don't know how many chairs a WG is supposed to have.

There isn't any hard limit, it's recommended to have at least 2, and not recommended to have a LOT, but we have workgroups with as many as 5 chairs currently. 2-4 is typical.

@ahmedtd

ahmedtd commented Jul 28, 2026

Copy link
Copy Markdown

I would be happy to, but I don't know how many chairs a WG is supposed to have.

There isn't any hard limit, it's recommended to have at least 2, and not recommended to have a LOT, but we have workgroups with as many as 5 chairs currently. 2-4 is typical.

Then I am fine to just participate.

@elmiko

elmiko commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

thanks @ahmedtd , your input is definitely welcome =)

given where we are in the process at this point, i'm curious if sig auth would be willing to give us a +1 towards forming the WG?

@elmiko

elmiko commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

cc @aramase @micahhausler @ritazh for a follow up comment on approval from sig auth.

@randomvariable
randomvariable force-pushed the topic/randomvariable/node-identity-wg branch from ef4b3cb to 2feb71c Compare July 31, 2026 20:46
@kubernetes-prow kubernetes-prow Bot added the sig/auth Categorizes an issue or PR as relevant to SIG Auth. label Jul 31, 2026
@enj enj moved this to Needs Triage in SIG Auth Aug 1, 2026
@enj enj added this to SIG Auth Aug 1, 2026
@neolit123

neolit123 commented Aug 5, 2026

Copy link
Copy Markdown
Member

@randomvariable

please include SIG Auth as participating SIG. this is just a formality. we had the conversation here:
https://kubernetes.slack.com/archives/C0EN96KUY/p1785953599305489

TL;DR

  • it seems odd to me that a security / authn feature has no expert SIG listed, that should be either Auth or Security. Since @ahmedtd is a SIG Auth regular member, let's include them and he can represent the SIG.
  • note that if any changes in core k8s like kubelet are required, we need KEPs, and that means core owners. In fact we likely need KEPs for any type of change, unless we align on small docs and blog posts.
  • sponsoring SIG means a group that wants work done. participating SIG are just advisory. SIG Auth collectively agreed that they do not see the need for work to be done in core ATM.

/hold
until that's resolved.

@elmiko

elmiko commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

thanks @neolit123

Comment thread wg-node-identity/charter.md
@rata
rata force-pushed the topic/randomvariable/node-identity-wg branch from 0b81c2e to 1d49c13 Compare August 6, 2026 10:11
@randomvariable
randomvariable force-pushed the topic/randomvariable/node-identity-wg branch 2 times, most recently from e1285ac to f4933cd Compare August 6, 2026 10:21
@kubernetes-prow kubernetes-prow Bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 6, 2026
@randomvariable
randomvariable force-pushed the topic/randomvariable/node-identity-wg branch from f4933cd to 9f96bb2 Compare August 6, 2026 10:25
@kubernetes-prow kubernetes-prow Bot added cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. and removed cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Aug 6, 2026
Registers WG Node Identity across sigs.yaml, OWNERS_ALIASES, sig-list.md,
and sig-cluster-lifecycle/README.md, and adds the charter and autogenerated
README under wg-node-identity/.

Co-authored-by: Taahir Ahmed <taahm@google.com>
Co-authored-by: Rodrigo Campos Catelin <rodrigo@sdfg.com.ar>
Co-authored-by: Ciprian Hacman <chacman@microsoft.com>
Co-authored-by: Naadir Jeewa <naadir.jeewa@broadcom.com>
Co-authored-by: Michael McCune <msm@opbstudios.com>
Co-authored-by: Josephine Pfeiffer <hi@josie.lol>
@randomvariable
randomvariable force-pushed the topic/randomvariable/node-identity-wg branch from 9f96bb2 to e99d345 Compare August 6, 2026 10:28
@kubernetes-prow kubernetes-prow Bot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Aug 6, 2026
@randomvariable

Copy link
Copy Markdown
Member Author

@randomvariable

please include SIG Auth as participating SIG. this is just a formality. we had the conversation here: https://kubernetes.slack.com/archives/C0EN96KUY/p1785953599305489

TL;DR

* it seems odd to me that a security / authn feature has no expert SIG listed, that should be either Auth or Security. Since @ahmedtd is a SIG Auth regular member, let's include them and he can represent the SIG.

* note that if any changes in core k8s like kubelet are required, we need KEPs, and that means core owners. In fact we likely need KEPs for any type of change, unless we align on small docs and blog posts.

* sponsoring SIG means a group that wants work done. participating SIG are just advisory. SIG Auth collectively agreed that they do not see the need for work to be done in core ATM.

/hold until that's resolved.

Thx, we've updated as per conversation.


* SIG Node
* SIG Cloud Provider
* SIG Autoscaling

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
* SIG Autoscaling
* SIG Autoscaling
* SIG Auth

still missing?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. committee/steering Denotes an issue or PR intended to be handled by the steering committee. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. sig/auth Categorizes an issue or PR as relevant to SIG Auth. sig/cluster-lifecycle Categorizes an issue or PR as relevant to SIG Cluster Lifecycle. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

Status: Needs Triage

Development

Successfully merging this pull request may close these issues.