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

Node.js Foundation Technical Steering Committee (TSC) Meeting 2018-02-21 #494

Closed
mhdawson opened this issue Feb 19, 2018 · 28 comments
Closed
Assignees

Comments

@mhdawson
Copy link
Member

mhdawson commented Feb 19, 2018

Time

UTC Wed 21-Feb-2018 17:00 (05:00 PM):

Timezone Date/Time
US / Pacific Wed 21-Feb-2018 09:00 (09:00 AM)
US / Mountain Wed 21-Feb-2018 10:00 (10:00 AM)
US / Central Wed 21-Feb-2018 11:00 (11:00 AM)
US / Eastern Wed 21-Feb-2018 12:00 (12:00 PM)
London Wed 21-Feb-2018 17:00 (05:00 PM)
Amsterdam Wed 21-Feb-2018 18:00 (06:00 PM)
Moscow Wed 21-Feb-2018 20:00 (08:00 PM)
Chennai Wed 21-Feb-2018 22:30 (10:30 PM)
Hangzhou Thu 22-Feb-2018 01:00 (01:00 AM)
Tokyo Thu 22-Feb-2018 02:00 (02:00 AM)
Sydney Thu 22-Feb-2018 04:00 (04:00 AM)

Or in your local time:

Links

Agenda

Extracted from tsc-agenda labelled issues and pull requests from the nodejs org prior to the meeting.

nodejs/node

  • src: delete proccess.env values set to undefined #18158

[EDIT: https://github.com/nodejs/node/pull/17062 removed by @Trott]
[EDIT: https://github.com/nodejs/node/pull/18131 removed by @Trott per @jasnell]

nodejs/TSC

  • Tracking issue for updating TSC on Board Meetings #476
  • Strategic Initiatives - Tracking Issue #423

Invited

[EDIT: @indutny and @mscdex removed by @Trott]

Observers

Notes

The agenda comes from issues labelled with tsc-agenda across all of the repositories in the nodejs org. Please label any additional issues that should be on the agenda before the meeting starts.

Joining the meeting

Uberconference; participants should have the link & numbers, contact me if you don't.

Public participation

We stream our conference call straight to YouTube so anyone can listen to it live, it should start playing at https://www.youtube.com/c/nodejs+foundation/live when we turn it on. There's usually a short cat-herding time at the start of the meeting and then occasionally we have some quick private business to attend to before we can start recording & streaming. So be patient and it should show up.

Many of us will be on IRC in #node-dev on Freenode if you'd like to interact, we have a Q/A session scheduled at the end of the meeting if you'd like us to discuss anything in particular. @nodejs/collaborators in particular if there's anything you need from the TSC that's not worth putting on as a separate agenda item, this is a good place for that

@mhdawson mhdawson self-assigned this Feb 19, 2018
@Trott
Copy link
Member

Trott commented Feb 19, 2018

Removing nodejs/node#17062. It was referred to the Release WG. Looks like it's only on the agenda because nobody removed the tsc-agenda label.

@Trott
Copy link
Member

Trott commented Feb 19, 2018

(Also removing @mscdex and @indutny. Can you update your template, @mhdawson?)

@Trott
Copy link
Member

Trott commented Feb 19, 2018

I'd like to find a different way to handle regular updates that doesn't involve tracking issues that are permanently on the agenda. Having permanent agenda items perpetuates the idea that the meetings are where things happen and where things should happen. The meeting should be the last resort for anything. We should strive for needing fewer meetings, not latch onto the meeting as an easy way to distribute information.

@Trott
Copy link
Member

Trott commented Feb 19, 2018

Last time, we deferred acting on nodejs/node#18158 until we could have @TimothyGu at a meeting. But I don't think this time slot works for him. In fact, it's possible none of the current time slots work for him, although he indicated that his schedule will be changing several weeks from now.

So that brings up a few things:

  • Do we want to schedule something at a different time with @TimothyGu? (We'd want @targos there as well.) Honestly, not sure what we expect to get out of having him attend that we can't get out of the thread in the issue tracker, but scheduling an ad hoc meeting is an option.

  • A separate issue: If Timothy is unlikely to be able to attend any meetings until (say) March or April, does it make sense to put his TSC nomination on hold until then and work to onboard someone else? Alternately, does it make sense to get his schedule information and see if meeting-picker comes up with something for the group that could work for him? (And of course there's always the option to do nothing about it and wait for it to resolve itself as it's not an urgent issue.)

For reference, here's Timothy's objection to that PR: nodejs/node#18158 (review)

There are two other TSC members who have registered opposition to the PR:

Additionally, @BridgeAR is opposed (nodejs/node#18158 (comment)).

There are multiple TSC and Collaborator approvals as well, and the PR's author is another TSC member (@targos). It was added to the agenda by @mcollina but he didn't seem to register an opinion on the PR itself. This raises another question:

  • Is there anybody who endorses the PR strongly enough that they would want it escalated to the TSC so it can land over the objections? (I guess this question is primarily directed at @targos, but it could be anyone, including @mcollina, although if it's @mcollina, then I'd ask that he register his approval in the pull request thread.)

If no one feels strongly enough about it that they want it over the objections raised, then the TSC need not take it up.

@Trott
Copy link
Member

Trott commented Feb 19, 2018

TL;DR on that last comment I left:

I left a comment at nodejs/node#18158 (comment) asking for there to be some GitHub dialogue before the TSC takes this up.

@Trott
Copy link
Member

Trott commented Feb 19, 2018

I acknowledge that this comment is an example of the irksome practice of raising a problem without offering a solution. I apologize for that. Uh...solutions...maybe we can have a cron job open an issue in the main issue tracker periodically that @-mentions all the people who should be supplying updates etc.? For example, it could be weekly for the strategic initiatives and monthly for the board thing.

@benjamingr
Copy link
Member

@Trott I appreciate you doing this and honestly I don't think that that particular PR is a good case study or that people have strong feelings about it. There weren't any strong feelings (at least that I saw), the facts are pretty well known and it has mostly stagnated. Taking things up to the TSC (for a vote) is a way to deal with stagnation - another way is to default to not landing things.

I don't think there is a very compelling case to raise either way - it's arguably nicer to delete values set to undefined, it's arguably more correct and backwards compatible to set it to undefined - I think the parties involved (feel free to correct me) understand what's at stake and the arguments and people don't really care about that particular PR enough to utilize the consensus seeking process to its fullest.

I think the TSC plays a much more important role when making decisions where there are stronger feelings (or stakes) involved. The other issue discussed for example - "esm: provide named exports for all builtin libraries" is about how Node.js APIs will be export in the future and has a lot of discussion, it's also huge (although that should probably be in nodejs/modules domain at some point).

@mcollina
Copy link
Member

I’m traveling this week and I might not be able to attend.

@Trott
Copy link
Member

Trott commented Feb 19, 2018

nodejs/node#18131 was put on the agenda by @jasnell (nodejs/node#18131 (comment)).

It's not clear to me if he meant a "possible vote" on the issue as a whole, or on the issue of filtering vs. not filtering. (@jasnell, can you clarify?)

Seems like discussion there is still pretty robust so I wonder if it should be taken off the agenda for now?

Not a single Collaborator has weighed in with a green checkmark approval or a red X request for changes. But since there has been robust conversation, I'd recommend that we don't take this up at the TSC level yet. I think the active conversation is enough evidence to leave it alone for now, but if we want more eyes on it or something, let's instead ping the nodejs/modules team and/or the nodejs/collaborators team to get some definite opinions.

@jasnell
Copy link
Member

jasnell commented Feb 19, 2018

It should be ok to remove that one from the agenda

@mhdawson
Copy link
Member Author

Should we remove nodejs/node#18158 (comment) from the agenda. I agree that there is no urgent reason for the TSC to make a decision as there does not seem to be anybody strongly pushing for the change.

@mhdawson
Copy link
Member Author

In terms of standing issues, I think we don't have to cause ourselves a lot of work by coming up with a new way to generate them. Instead we should just treat them as being on the agenda unless there are no other issues (ie we can cancel them if they are the only issues on the agenda). Having said that if somebody else has time to put in the effort to automate a better way of handling them then I'm good with that as well.

@Trott
Copy link
Member

Trott commented Feb 20, 2018

Should we remove nodejs/node#18158 (comment) from the agenda.

We are prohibited from doing that. From https://github.com/nodejs/node/blob/master/GOVERNANCE.md#tsc-meetings:

The meeting chair and the TSC cannot veto or remove [agenda] items.

So unless @mcollina removes it (since he's the one who put it on the agenda in the first place), it must be on the agenda this week.

What we can do is generate consensus ahead of time that we're declining to act on it at this time (so it can be mentioned briefly in the meeting and everyone agrees that there's nothing to say or do). Or generate consensus for tabling the topic until @mcollina can attend. (But still, it must be on the agenda and be raised.) Or generate consensus that we don't need to have a meeting this week if it's the only agenda item.

@mcollina
Copy link
Member

nodejs/node#18158 is well described. IMHO It does not need a champion, it just need a vote called on. I don’t think there is much to discuss. If we don’t call for a vote, that is going to sit there for a few months, and there cannot be consensus in this case (either we do it or we do not). May I open a voting issue?

@mcollina
Copy link
Member

Consider my above comment the reason not to hold this meeting if it’s the only item in the agenda. Let’s open a vote for a couple of weeks.

@Trott
Copy link
Member

Trott commented Feb 20, 2018

Sounds like as of now the meeting is cancellable. I suggest we cancel it now rather than try to accommodate last-minute items on the agenda. Any items added can be on next week's agenda or else handled in GitHub if possible.

@MylesBorins
Copy link
Member

Would people be open to doing a shorter meeting where we simply do an update on strategic initiatives? Seems like a good piece of content to share with community

@targos
Copy link
Member

targos commented Feb 21, 2018

I don't mind doing that

@mhdawson
Copy link
Member Author

mhdawson commented Feb 21, 2018

I'm good doing a short meeting or cancelling.

Link to zoom conf for attendees. Will send password to TSC email.

https://zoom.us/j/315209925?pwd=aeCmmUz1mQCwIumekTziMw

YouTube link will be added as broadcast starts as I don't think I can do that in advance with zoom.

@Fishrock123
Copy link
Member

Short meeting sounds fine to me.

@jasnell
Copy link
Member

jasnell commented Feb 21, 2018

If there's a meeting, I (and likely @mcollina) won't be making it

@mhdawson
Copy link
Member Author

@Trott
Copy link
Member

Trott commented Feb 21, 2018

Report for @nodejs/moderation: No significant activity this month thus far. I'll amend this if there's anything else to add but I don't believe there is. /cc @nodejs/community-committee

@williamkapke
Copy link
Member

williamkapke commented Feb 23, 2018

DELETED COMMENT: oops! the CommComm YouTube video linked to this issue by mistake :(

@richardlau
Copy link
Member

^^ cc @nodejs/collaborators

@MylesBorins
Copy link
Member

Closing as the meeting has happened

@Trott
Copy link
Member

Trott commented Feb 28, 2018

Re-opening because we don't close the meeting issues until the minutes have been landed. (Don't think that's documented anywhere, but it's been our de facto process for as long as I'm aware.)

@Trott Trott reopened this Feb 28, 2018
@mhdawson
Copy link
Member Author

PR for minutes: #498

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

No branches or pull requests