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

SIG-Scheduling 2020 Annual Report #5557

Merged

Conversation

Huang-Wei
Copy link
Member

Adding initial template for SIG-Scheduling 2020 Annual Report.

This is due by March 8th and will be shown to the SIG and Slack.

Which issue(s) this PR fixes:

Fixes #5510.

@k8s-ci-robot
Copy link
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 24, 2021
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Huang-Wei

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added sig/scheduling Categorizes an issue or PR as relevant to SIG Scheduling. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Feb 24, 2021
@Huang-Wei
Copy link
Member Author

Opened this as a placeholder, @ahg-g and I will fill out unanswered parts by this week.

/assign @liggitt

@nikhita
Copy link
Member

nikhita commented Feb 25, 2021

/committee steering

@k8s-ci-robot k8s-ci-robot added the committee/steering Denotes an issue or PR intended to be handled by the steering committee. label Feb 25, 2021
Copy link
Member

@ahg-g ahg-g left a comment

Choose a reason for hiding this comment

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

some suggestions

@ahg-g
Copy link
Member

ahg-g commented Feb 25, 2021

/sig scheduling

@Huang-Wei Huang-Wei force-pushed the sig-scheduling-2020-annual-report branch from 466fb62 to 04216b0 Compare February 26, 2021 02:48
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 26, 2021
@Huang-Wei Huang-Wei marked this pull request as ready for review February 26, 2021 02:49
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 26, 2021
@Huang-Wei
Copy link
Member Author

@liggitt the first version is ready for review. Take your time.

@Huang-Wei Huang-Wei force-pushed the sig-scheduling-2020-annual-report branch from 04216b0 to 1f25fd6 Compare March 2, 2021 20:50

Owners from (active) subprojects introduce the latest development, and sometime demonstrate
cool features.
OWNER files in k/k are not that up to date. We may need a cleanup.
Copy link
Contributor

Choose a reason for hiding this comment

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

is this something contribex can help with?

Copy link
Member

Choose a reason for hiding this comment

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

I think we are fine, we can send a PR to remove inactive members and tag them on that PR.

Copy link
Member Author

Choose a reason for hiding this comment

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

I suppose it's a common issue across all sigs. It'd be neat if we can come up with a script auto-identifying the inactive maintainers/reviewers:

  • for a particular SIG, not participated in any PR reviews in the last 12 (or 18) months
  • a single bare "/lgtm" or "/approve" doesn't count as a legal review

Copy link

Choose a reason for hiding this comment

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

ok


**Are all listed SIG leaders (chairs, tech leads, and subproject owners) active?**

Yes, except for inactive subprojects.
Copy link
Contributor

Choose a reason for hiding this comment

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

which ones are inactive?

Copy link
Member Author

Choose a reason for hiding this comment

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

  • kube-batch. The original plan is to develop batch-related features in a separate project, and eventually converge with default scheduler. But kube-batch is now incubated into a cncf sandobx project called volcano. So kube-batch is not active now.
  • poseidon. I don't see much interested in this project, and it's also no longer actively developed.

Copy link
Member

Choose a reason for hiding this comment

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

are those subprojects inactive to the point that their readme's should be updated to note that?

Copy link
Member Author

Choose a reason for hiding this comment

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

I will create issues in those subprojects to discuss what to do next: whether specify the inactive status in README, or just archive them.

Copy link
Member Author

Choose a reason for hiding this comment

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

**How does the group measure reviewer and approver bandwidth? Do you need help in any area now?
What are you doing about it?**

PRs are usually directed to the reviewer most familiar with the code base the PR is modifying.
Copy link
Contributor

Choose a reason for hiding this comment

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

are you growing more reviewers and approvers?

Copy link
Member

Choose a reason for hiding this comment

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

I think we have have enough approvals for the volume of the PRs we are getting, reviewers are growing.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, we're fine by now.

activities that the group does to encourage this? What programs are you participating in to
grow contributors throughout the contributor ladder?**

There is no official onboarding process. One thing we try to do frequently is breaking up
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you use help-wanted or good-first issue labels?

Copy link
Member

Choose a reason for hiding this comment

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

both depending on the complexity of the issue.

**Does the group have contributors from multiple companies/affiliations? Can end users/companies
contribute in some way that they currently are not?**

Yes.
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you want more companies to contribute? If so, to what and what extent? This is a microphone to end users

Copy link
Member

Choose a reason for hiding this comment

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

I don't think we are lacking diversity, there are 5+ companies involved in the sig on/off.

**What are initiatives that should be highlighted, lauded, shout out, that your group is proud of?
Currently underway? What are some of the longer tail projects that your group is working on?**

- Initiatives:
Copy link
Contributor

Choose a reason for hiding this comment

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

ty for sectioning them out this way


**What areas and/or subprojects does the group need the most help with?**

- **Docs improvement**:
Copy link
Contributor

Choose a reason for hiding this comment

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

ty for calling this out

Copy link
Member

Choose a reason for hiding this comment

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

agreed, thanks. I noticed https://docs.google.com/document/d/1t2QoASmg_BssFczl8n7UNTDWKgncMSrLTX802fQ0lK8/edit# and #5674 recently, which seemed to address some of the developer-facing bits. Would it make sense to open help-wanted issues for remaining dev doc gaps and for specific user-oriented gaps?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, @ingvagabund 's recent contributions really make the docs greatly improved. With those docs in place, I think we're fine on dev docs. In terms of user-oriented doc gaps, we will open help-wanted issues for tracking.

@dims
Copy link
Member

dims commented Apr 9, 2021

/committee steering

Copy link
Member

@liggitt liggitt left a comment

Choose a reason for hiding this comment

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

lgtm, I just had one question about the two inactive subprojects mentioned


**What's the average open days of a PR and Issue in your group? What metrics does your group care about and/or measure?**

We haven't stated leveraging devstat data or Github board to get a high-level picture of PR/Issue.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
We haven't stated leveraging devstat data or Github board to get a high-level picture of PR/Issue.
We haven't started leveraging devstat data or Github board to get a high-level picture of PR/Issue.


**What areas and/or subprojects does the group need the most help with?**

- **Docs improvement**:
Copy link
Member

Choose a reason for hiding this comment

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

agreed, thanks. I noticed https://docs.google.com/document/d/1t2QoASmg_BssFczl8n7UNTDWKgncMSrLTX802fQ0lK8/edit# and #5674 recently, which seemed to address some of the developer-facing bits. Would it make sense to open help-wanted issues for remaining dev doc gaps and for specific user-oriented gaps?

@Huang-Wei Huang-Wei force-pushed the sig-scheduling-2020-annual-report branch from 1f25fd6 to 0b47ff0 Compare April 13, 2021 17:35
@liggitt
Copy link
Member

liggitt commented Apr 13, 2021

/lgtm
with steering hat

/hold
for lgtm by sig-scheduling co-lead @ahg-g

@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 Apr 13, 2021
@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 13, 2021
@ahg-g
Copy link
Member

ahg-g commented Apr 13, 2021

/lgtm

@liggitt
Copy link
Member

liggitt commented Apr 13, 2021

/hold cancel

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 13, 2021
@k8s-ci-robot k8s-ci-robot merged commit af45c14 into kubernetes:master Apr 13, 2021
@Huang-Wei Huang-Wei deleted the sig-scheduling-2020-annual-report branch April 13, 2021 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. 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. lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/scheduling Categorizes an issue or PR as relevant to SIG Scheduling. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2021 Annual Report: SIG Scheduling
9 participants