governance: expand use of CTC issue tracker by Trott · Pull Request #8945 · nodejs/node · GitHub

Checklist

documentation is changed or added
commit message follows commit guidelines

Affected core subsystem(s)

Description of change

As the CTC grows and has representation from more time zones, we need to
embrace asynchronous decision making and rely less on the actual
meeting. This change is a proposal for that which, ironically, probably
has to be approved at a meeting.

/cc @nodejs/ctc

proposal

Is there a whitespace at the end of the line?

SGTM!

LGTM modulo what @thefourtheye pointed out.

Tiny nit but isn't it spelled 'commit access', no dash?

LGTM subject to comments identified by others.

LGTM. Again, with the nits addressed.

Seems a bit tight for ensuring consensus imo

Seems a bit tight for ensuring consensus imo

What would seem a better duration in your opinion? I chose 72 hours because that seemed like a duration for an issue on the tracker that was more likely to actually map to real consensus than a conversation at a meeting attended by maybe 10 of the CTC's 18 members. Arguably, that's not a high bar, but it's what we have now, so that's what I used as an approximate metric. Totally open to other ideas. What did you have in mind?

This LGTM with nits addressed.
I have been wondering if a separate ctc-review label would be useful for distinguishing between things that just require CTC review vs. things that need to be escalated to a meeting.

As the CTC grows and has representation from more time zones, we need to embrace asynchronous decision making and rely less on the actual meeting. This change is a proposal for that which, ironically, probably has to be approved at a meeting.

Nits addressed, rebased, force pushed.

Does this still need to be on the ctc-agenda?

Does this still need to be on the ctc-agenda?

Yes.

It would be great if CTC folks could read the new paragraph and the bullet points and weighed in before the meeting. I'd be happy to not have to bring this to the meeting.

CTC folks who have not offered an opinion yet:

ctc-review might be a good idea, however it's really at the discretion of the meeting chair whether something goes on the agenda or not and some things can be left off as required.

This lgtm for now. I'm hesitant with the 72-hours two-only requirement for passing major things, but we can only see how it goes and adjust if enough of us are unhappy with the process. My main concern is simply with the workload of GitHub notifications and I'm constantly behind these days. I'll just have to make sure I tune in to nodejs/ctc tagged items!

lgtm. I'd be +1 to a ctc-review label. I think it would make it a lot easier to see what issues need attention.

+1 in general, but with splitting a «CTC should take a look at this» with «this escalated to voting».

The moment when it has been escalated to voting should be clearly visible. That is probably a separate thing from ctc-review (review needed by the CTC) and ctc-agenda (consensus-seeking). ctc-vote, perhaps?

Currently ctc-review and ctc-agenda labels are present or proposed. Could we codify both of those?

LGTM

I want to echo @rvagg's hesitation on the 72 hour + 2 ctc minimum for moving things forward. Obviously this will be a case by case basis... but if something is controversial I do thing we need to have a bit of a wider consensus on it. I think a hard rule here may miss the point, but I feel we may need more time + opinions on it. I'm +1 on everything but that nit.

LGTM w/ the same hesitation as @thealphanerd and @rvagg.

ctc-review might be a good idea, however it's really at the discretion of the meeting chair whether something goes on the agenda or not and some things can be left off as required.

That may be true in practice, but it is in contradiction of our GOVERNANCE.md doc which says of the CTC agenda:

The moderator and the CTC cannot veto or remove items.

I've created the ctc-review label. Documenting it in GOVERNANCE.md will be in a subsequent pull request.

I count at least 11 (and possibly as many as 14) approvals from current CTC members.

The approvals as I see it come from:

addaleax
bnoordhuis
cjihrig
evanlucas
jasnell
mhdawson
misterdjules
rvagg
shigeki
trott

There are also maybe-approvals-I'm-not-sure from:

chalker
ofrobots
thealphanerd

So I think this can land.

I'll create a ctc-agenda issue for discussion of how this is working or not working. We can probably leave that on the meeting agenda for two or three weeks. If nothing else, people can give feedback on whether 72 hours is working or not.

As the CTC grows and has representation from more time zones, we need to embrace asynchronous decision making and rely less on the actual meeting. This change is a proposal for that which, ironically, probably has to be approved at a meeting. PR-URL: nodejs#8945 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Julien Gilli <jgilli@nodejs.org> Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Shigeki Ohtsu <ohtsu@ohtsu.org>

Landed in b899140. Thanks, everyone.

As the CTC grows and has representation from more time zones, we need to embrace asynchronous decision making and rely less on the actual meeting. This change is a proposal for that which, ironically, probably has to be approved at a meeting. PR-URL: #8945 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Julien Gilli <jgilli@nodejs.org> Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Shigeki Ohtsu <ohtsu@ohtsu.org>

Trott added doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project. ctc-agenda labels Oct 6, 2016

nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Oct 6, 2016

thefourtheye reviewed Oct 6, 2016

View reviewed changes

addaleax approved these changes Oct 6, 2016

View reviewed changes

bnoordhuis approved these changes Oct 6, 2016

View reviewed changes

cjihrig approved these changes Oct 6, 2016

View reviewed changes

Fishrock123 reviewed Oct 6, 2016

View reviewed changes

Trott added 2 commits October 6, 2016 19:31

governance: expand use of CTC issue tracker

b247618

As the CTC grows and has representation from more time zones, we need to embrace asynchronous decision making and rely less on the actual meeting. This change is a proposal for that which, ironically, probably has to be approved at a meeting.

squash: nits

a7485d1

Trott force-pushed the ctc-agenda branch from c0ad262 to a7485d1 Compare October 7, 2016 02:32

Trott mentioned this pull request Oct 11, 2016

Node.js Foundation Core Technical Committee (CTC) Meeting 2016-10-12 #9020

Closed

thefourtheye approved these changes Oct 12, 2016

View reviewed changes

shigeki approved these changes Oct 12, 2016

View reviewed changes

misterdjules approved these changes Oct 12, 2016

View reviewed changes

Trott closed this Oct 13, 2016

Trott mentioned this pull request Oct 13, 2016

Review how ctc-review process is working nodejs/CTC#20

Closed

ChALkeR mentioned this pull request Oct 13, 2016

Consider changing TSC membership structure. nodejs/TSC#146

Closed

Trott removed the ctc-agenda label Oct 20, 2016

MylesBorins added land-on-v4.x labels Nov 11, 2016

This was referenced Nov 22, 2016

v6.9.2 proposal #9735

Merged

v4.7.0 proposal #9736

Merged

Trott deleted the ctc-agenda branch January 13, 2022 22:44

governance: expand use of CTC issue tracker by Trott · Pull Request #8945 · nodejs/node · GitHub

Uh oh!

Uh oh!

governance: expand use of CTC issue tracker - #8945

governance: expand use of CTC issue tracker#8945Trott wants to merge 2 commits intonodejs:masternodejs/node:masterfrom Trott:ctc-agendaTrott/io.js:ctc-agendaCopy head branch name to clipboard

Conversation

Trott commented Oct 6, 2016

Checklist

Affected core subsystem(s)

Description of change

Uh oh!

thefourtheye Oct 6, 2016

Choose a reason for hiding this comment

Uh oh!

thefourtheye Oct 6, 2016

Choose a reason for hiding this comment

Uh oh!

addaleax left a comment

Choose a reason for hiding this comment

Uh oh!

bnoordhuis left a comment

Choose a reason for hiding this comment

Uh oh!

bnoordhuis Oct 6, 2016

Choose a reason for hiding this comment

Uh oh!

mhdawson commented Oct 6, 2016

Uh oh!

cjihrig left a comment

Choose a reason for hiding this comment

Uh oh!

Fishrock123 Oct 6, 2016

Choose a reason for hiding this comment

Uh oh!

Trott Oct 7, 2016

Choose a reason for hiding this comment

Uh oh!

jasnell commented Oct 6, 2016

Uh oh!

Trott commented Oct 7, 2016

Uh oh!

jasnell commented Oct 10, 2016

Uh oh!

Trott commented Oct 10, 2016

Uh oh!

Trott commented Oct 12, 2016

Uh oh!

rvagg commented Oct 12, 2016

Uh oh!

evanlucas commented Oct 12, 2016

Uh oh!

ChALkeR commented Oct 12, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

misterdjules left a comment

Choose a reason for hiding this comment

Uh oh!

MylesBorins commented Oct 12, 2016

Uh oh!

ofrobots commented Oct 12, 2016

Uh oh!

Trott commented Oct 13, 2016

Uh oh!

Trott commented Oct 13, 2016

Uh oh!

Trott commented Oct 13, 2016

Uh oh!

Trott commented Oct 13, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

16 participants

governance: expand use of CTC issue tracker#8945
Trott wants to merge 2 commits into
nodejs:masterfrom
Trott:ctc-agenda

ChALkeR commented Oct 12, 2016 •
edited
Loading