Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
sig-scheduling annual report #7215
sig-scheduling annual report #7215
Changes from all commits
242a0d5
18993ca
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
While we are doing well in the nationality/ethnicity front, we are not doing well in gender diversity. Not really sure how to improve this.
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.
Seems we have no CONTRIBUTING.md to sig-scheduling specifically. It's empty.
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 think we can draft one, I recalled we got a developer from the community who asked for the same question about how to learn kube-scheduler in the community 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.
I can't find the thread, but I remember the guidance was that if we don't have any special training or requirements beyond https://git.k8s.io/community/community-membership.md, we don't need this page.
This is not the same as developer documentations, which can be found in https://github.com/kubernetes/community/tree/master/contributors/devel/sig-scheduling
One thing I can think of, but it's not 100% enforced: each PR needs first an LGTM from a reviewer before getting approval.
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.
A developer document could also be part of the CONTRIBUTING.md, I found some sigs have self-owned ones, like the sig-apps, sig-node, but it's not strict.
Anyway, I can draft one later.
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.
#7228
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.
em.. one thing I think related here is the APAC meeting, usually hosted by @Huang-Wei ❤️
but we have very little attendees each time. 🤷♂️ but I think we can still move on if we have agenda. FYI.
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.
The APAC meeting doesn't run smoothly atm. We don't have concrete agenda items and less attendees than expected. TBH I'm considering closing it sometime in the future.
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.
Yes, this grows the burden for you can't just ignore it, but another problem is the primary sig scheduling meeting is usually too late for APAC, 12 am or 1 am in the midnight, I admit I have selfishness 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.
Negotiating a meeting time across the globe is always challenging :(
@alculquicondor @ahg-g what's your take?
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.
Maybe we should go back to alternated times, preserving meetings every 2 weeks.
We have 3 TLs, two in PDT and one in EDT. Ideally each meeting should fall within business hours of at least two TLs. The current one falls within business hours of all of us. I think we can keep this one and move the other one, but I think it will still have to be slightly out of business hours.
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.
Not at all, we should accommodate for as many contributors as possible.
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.
It would be nice to have this. 😄
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 guess nothing to add to the report with regards to this thread, please let me know if not
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.
@kerthcet can you create an issue for this in this repo?
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.
See #7254