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

Clean up ownership governance #20

Merged
merged 5 commits into from
Aug 4, 2023

Conversation

jacobtomlinson
Copy link
Member

@jacobtomlinson jacobtomlinson commented Jul 25, 2023

This PR attempts to update the Dask governance to better represent the current state of Dask Ownership.

I've tried to incorporate everything from all of the discussions I've had with folks about this lately, but feedback is very welcome. I want to be sure I have accurately reflected things.

Key changes:

  • Make it clear that "Steering Council" and "Owners" are the same thing.
  • Reduce duplication around how folks join the Steering Council.
  • Add explanation that inactive Council Members lose ownership rights and voting power and become Emeritus Council Members.
  • Emeritus Council Members can restore their permissions by simply being active again, but this can be vetoed by the active council.
  • Currently, the Council only has employees from two institutions so updating voting rules to reflect that. It would be nice to restore this in the future though.

@mrocklin @jrbourbeau @quasiben @fjetter

@quasiben
Copy link
Member

+1

Copy link
Member

@jcrist jcrist left a comment

Choose a reason for hiding this comment

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

Some small typos, otherwise LGTM!

governance.md Outdated Show resolved Hide resolved
membership.md Outdated Show resolved Hide resolved
@jacobtomlinson
Copy link
Member Author

Thanks for the review @jcrist. Agree we should use American spelling here.

Copy link
Member

@fjetter fjetter left a comment

Choose a reason for hiding this comment

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

Thanks @jacobtomlinson for taking care of this

@mrocklin
Copy link
Member

mrocklin commented Aug 2, 2023

Currently, the Council only has employees from two institutions so updating voting rules to reflect that. It would be nice to restore this in the future though.

I'm still seeing "three" often in the changes.

Also, FWIW, I'd still consider @jcrist and @jsignell still part of the steering council for the time being. I'd expect that we probably send them a note in 6-12 months if their current activity level stays as low as it is today. They were active ~6 months ago and I think about a year is a good timeline for promoting to emeritus.

@mrocklin
Copy link
Member

mrocklin commented Aug 2, 2023

To be clear, I do like the idea of changing three to two. In practice most decision-making seems to be happening between Coiled and NVIDIA today (others welcome, of course)

@jacobtomlinson
Copy link
Member Author

Good catch thanks @mrocklin. I think updating to two here reflects reality, but it would be nice to update reality back to three at some point.

I'll leave this open a little longer to give @jsignell and @jrbourbeau time to weigh in, but I'd like to merge this sooner rather than later.

@jsignell
Copy link
Member

jsignell commented Aug 2, 2023

This works for me! Thanks for taking it on @jacobtomlinson. Also just to say that I am happy to be moved to emeritus anytime.

@jacobtomlinson
Copy link
Member Author

jacobtomlinson commented Aug 3, 2023

Ok great, everyone had weighed in now and there haven't been any -1 style comments. So I intend to merge this tomorrow unless anyone wants to comment further.

@jacobtomlinson jacobtomlinson merged commit 66aa89c into dask:main Aug 4, 2023
@jacobtomlinson jacobtomlinson deleted the emeritus-owners branch August 4, 2023 08:55
@jacobtomlinson
Copy link
Member Author

Ok this is merged. I've created two teams in GitHub for "Steering Council" and "Emeritus Steering Council" and updated the ownership permissions accordingly. @jcrist I've put you in Steering Council as @mrocklin suggested but @jsignell I've put you in Emeritus Steering Council as you requested.

I'm going to send an email to everyone who has moved to emeritus as part of this change to update them.

@mrocklin could you update the owners@dask.org mailing list to reflect the Steering Council group? I don't seem to have permission to change it.

@mrocklin
Copy link
Member

mrocklin commented Aug 4, 2023 via email

@mrocklin
Copy link
Member

mrocklin commented Aug 4, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants