-
Notifications
You must be signed in to change notification settings - Fork 5.3k
add Kubernetes AI Conformance working group #8515
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 Kubernetes AI Conformance working group #8515
Conversation
/hold |
cc @mfahlandt |
efficiently run AI/ML workloads. \n" | ||
charter_link: charter.md | ||
stakeholder_sigs: | ||
- Architecture |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe this should list the same SIGs that are listed as stakeholders in your charter.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, so does everybody agree to list only: SIG Architecture and SIG Testing for now? Please let me know so I can make the change and get this merged asap to start meeting
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 as SIG Testing TL.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 as SIG Arch Chair
cc SIG Arch chairs @derekwaynecarr @dims @johnbelamaric for approval |
@janetkuo let me paste @derekwaynecarr 's question from doc here
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd like to advocate for an explicit nomination process for the leads of AI Conformance.
wg-ai-conformance/charter.md
Outdated
|
||
#### Code, Binaries and Services | ||
|
||
- The primary artifact will be the Kubernetes AI Conformance specification and a suite of tests to demonstrate conformance. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In which repository will this be maintained? Who will be the maintainers of the conformance specification?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is an open question from the SIG arch meeting.
IIRC there was loose agreement to follow up in the WG to discuss options for how best to implement and host this and present them to the relevant groups (SIG Arch, SIG Testing, CNCF, ....).
In think this should be clarified, it is undetermined if there will be binary artifacts in the kubernetes organization. They may be in the CNCF, or the workgroup might determine that there's an alternate approach.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this @fedebongio!
Co-authored-by: Benjamin Elder <bentheelder@google.com>
Co-authored-by: Janet Kuo <chiachenk@google.com>
Co-authored-by: Janet Kuo <chiachenk@google.com>
Fixing 80 char line correctly
/retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With all the bits in this
/lgtm (from steering)
/label tide/merge-method-squash
Looks like one more verify failure 😉 |
👍 (steering and future liaison) |
This file will be auto generated
@fedebongio can you rerun the generator, i can't pr into your branch GitHub does not find it ^^ |
Apologies for delay, I have been traveling. /lgtm +1 SIG Arch Chair |
+1 (steering) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 (Steering)
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: BenTheElder, fedebongio, franciscojavierarceo, janetkuo, justaugustus, mfahlandt, saschagrunert, soltysh The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
we have sufficient steering 👍 -ing |
Which issue(s) this PR fixes:
This PR is a follow up on the email sent 2 weeks ago on WG creation, see https://groups.google.com/a/kubernetes.io/g/dev/c/u6I_mCRC4lE
cc @janetkuo