/srv/irclogs.ubuntu.com/2021/12/13/#ubuntu-meeting.txt

=== cpaelzer_ is now known as cpaelzer
sil2100o/16:00
ddstreeto/16:00
parideo/16:00
athoso/16:00
* ddstreet gets coffee before others show up16:00
rbasako/16:00
sil2100\o16:01
sil2100I suppose we'd love to have at least one more member today, right16:02
rafaeldtinocoo/16:04
rafaeldtinocoI'm finishing lunch following on the cell and rushing to the office in 5 min16:05
sil2100Who can chair today?16:05
rbasakI'm in a concurrent meeting, sorry16:06
tewardi'd be here if the log4j crap didnt have me neck deep in phone calls and vuln scanning at FT job16:06
tewardso i am not *really* here right now16:06
ddstreetwe're down the charing list to sil2100 then, if you are able?16:06
ddstreetif not, then it's me16:07
ddstreeti'll take the extended silence as an indication i should chair?16:09
sil2100I think I chaired last time?16:10
sil2100At least that's what I remember16:10
ddstreetah ok you might not have moved yourself to the end of the chairing list16:10
ddstreeti can chair, it's no prob16:10
sil2100Thanks!16:10
sil2100(I'm also in a backlog review meeting for Foundations, so I'm a bit distracted)16:11
ddstreet#startmeeting Developer Membership Board16:11
meetingologyMeeting started at 16:11:18 UTC.  The chair is ddstreet.  Information about MeetBot at https://wiki.ubuntu.com/meetingology16:11
meetingologyAvailable commands: action, commands, idea, info, link, nick16:11
ddstreet#topic Review of previous action items16:11
ddstreeti think some of the action items got mixed into the 'long-term' action item list?16:12
ddstreeti'm going to skip over the ones that i think are long-term items16:12
ddstreet#topic sil2100 to add jawn-smith to core-dev (done)16:12
jawn-smith\o/16:12
ddstreetlooks done, thanks :)16:12
ddstreet#topic sil2100 to announce jawn-smith's successful application (done)16:12
ddstreetalso done, thanks16:12
ddstreet#topic ddstreet to follow up on if we should change the 19:00 UTC meetings as well16:13
sil2100\o/16:13
ddstreeti send a ML email to kick off discussion on this, i don't think we need to keep an action item for it16:13
sil2100+116:13
ddstreet#topic sil2100 to change the 15:00 UTC meeting times to 16:00 UTC (on Agenda and calendar)16:13
ddstreeti think this is done right?16:13
ddstreeti mean, we're having this mtg at 16:00 UTC, so... ;-)16:13
paridethe fridge calendar looks out of date16:14
ddstreeti think sil2100 just recently changed that?16:14
ddstreetlike an hour-ish ago?16:14
parideI mean, the time of the day is still 15:00 UTZ16:14
parideUTC16:14
parideoh!16:14
parideyes I can confirm it's 16:0016:14
ddstreetok good, thanks16:14
sil2100Apologies! hm, maybe the fridge one didn't get synced? I don't really remember how the fridge thing works16:15
sil2100Anyway, yeah, I only did the change some hour or two ago after ddstreet reminded me of that16:15
ddstreet#topic ddstreet to handle "Making it easier for applicants to request special meeting times and/or applying by email" (carried over)16:15
ddstreetI started discussion for this on the ML also, i think that's enough, don't think we need to carry an action item for it16:16
ddstreetok let's move on to applications then16:17
ddstreetwe have 2 applications today, in order of the list on the DMB agenda, the PPU application from Athos Ribeiro is first16:18
ddstreet#topic PPU application by athos-ribeiro16:18
ddstreethello athos, can you introduce yourself?16:19
ddstreet#link https://wiki.ubuntu.com/AthosRibeiro/UbuntuServerDeveloperApplication16:19
rbasakBoth Athos and Paride are colleagues of mine on my team in Canonical, so I'm going to abstain as I normally do in these cases, except if there's a unanimous +1 from all others present and my vote is needed to make quorum.16:19
ddstreetathos you still around?16:20
athosHello everyone! I am Athos, I work for Canonical in the Server team. My primary focus is in the maintainance and production of the OCI images published in both dockerhub and amazon's ECR. Apart from that, I also perform general bug work throughout the team's packages. Today I am applying for upload rights for the server package set16:20
ddstreetjust for clarification, the packageset is 'ubuntu-server', correct? For example for jammy https://people.canonical.com/~ubuntu-archive/packagesets/jammy/ubuntu-server16:22
ddstreetand let's open it up for any questions the DMB members have16:23
athosyes, that is correct16:23
ddstreetthanks!16:23
=== cpaelzer_ is now known as cpaelzer
ddstreetI suspect other members are still reading your application page so let's give things a bit more time16:25
sil2100Yeah, sorry, I didn't have time to do that earlier and now I'm distracted16:25
sil2100But browsing through it now16:25
rafaeldtinocoddstreet: I have already read his application for the past meeting and Im good to vote (as I provided an endorsement already).16:25
sil2100athos: tell me, what are the requirements for a change to be considered for an SRU?16:26
athosWe need to file the SRU paperwork in the LP bug that the change is going to fix16:28
athosThe impact of the change must be well assessed16:28
athosand in general, it should only apply for high impact bugs16:29
athosoh, and the bug must be fixed in the development release16:30
sil2100Ok, thanks, sounds good16:31
sil2100btw. for anyone that needs this, sponsorship miner url: https://udd.debian.org/cgi-bin/ubuntu-sponsorships.cgi?render=html&sponsor=&sponsor_search=name&sponsoree=Athos%20Ribeiro&sponsoree_search=name16:33
ddstreetand further q from anyone?16:33
sil2100I still might have one, one moment16:34
sil2100athos: looking at your recent focal php7.4 SRU - do you think it is good to go? Or is there still some problem something that needs attention?16:35
athossil2100: it LGTM; There was one autopkgtest failure at some point last week, for which I requested a retrigger (it was just a timeout)16:37
sil2100athos: are you sure? Are all the arches good to go..?16:37
* athos chacks16:38
* athos checks16:38
athosthe focal package ftbfs in ppc16:39
sil2100Does this mean the package can be released right now?16:40
athosIt cannot; this needs fixing. I will request a re-build for that so I can verify the logs, to check if any further change will be needed indeed16:41
athosthe hirsute SRU is good to og though16:42
sil2100Final question: can you imagine any situations when an SRU can be released even in such situation?16:42
athosmeaning we have a build failure for one specific arch?16:43
sil2100Yes16:45
* ddstreet coffee refill, back asap16:45
athosI suppose it could be released in case that package was never released to that specific platform before16:46
* ddstreet back16:49
ddstreetsil2100 i have one q if you're done?16:50
ddstreetathos for a patch to either the devel release or stable release, can you explain any restrictions and/or importance on where the patch comes from?16:51
ddstreetthat's poorly worded...basically, where does a devel and/or sru patch need to come from?16:51
athosIt would be nice if a patch comes from the upstream project; however, it may not make much sense sometimes. For instance16:53
athosI submitted a patch to python-debian recently to add support for zstd, which was set as the default compression format in impish16:53
athossince Debian does not support zstd, we included that patch downstream (and I am working with upstream to add that support there as well)16:54
athosIn any case, it is always nice to forward the patch upstream and to debian, to reduce/minimize our deltas.16:55
ddstreetas an example (and i'm nit-picking here, to be clear) in this MR: https://code.launchpad.net/~athos-ribeiro/ubuntu/+source/openvpn/+git/openvpn/+merge/40595716:55
ddstreetis the Origin: listed there the 'best' reference for where the patch came from?16:56
athosI see how this could be enhanced to point to a VCS16:58
ddstreetwhat's the downside of using the ML reference instead of the VCS url?16:58
athosA patch in a mainling list is no guarantee that it was indeed applied upstream. One would need check the project to verify that is the actual origin of the patch17:00
athosfor this case, you can see it is indeed applied: https://github.com/OpenVPN/openvpn/commit/6d8380c78bf77766454b93b49ab2ebf713b0be48; and this should be the actual reference there17:01
ubottuCommit 6d8380c in OpenVPN/openvpn "Increase listen() backlog queue to 32"17:01
ddstreetthanks, yes that's true; please also keep in mind that all patching would should try to make things as easy as possible for the *next* person who has to look at the code; patching packages isn't *only* about fixing bugs, it's also about doing it in a way that's maintainable with the least friction as possible17:01
ddstreetanyone else have more q? rbasak sil2100 rafaeldtinoco teward?17:02
rafaeldtinocono17:02
sil2100No questions here!17:02
rbasakNone from me thanks (see abstaining comment above)17:02
sil2100(sorry, was busy with other meetings, eh)17:03
ddstreet#vote Packageset application for 'ubuntu-server' packageset for Athos Ribeiro17:03
meetingologyPlease vote on: Packageset application for 'ubuntu-server' packageset for Athos Ribeiro17:03
meetingologyPublic votes can be registered by saying +1, -1 or +0 in channel (for private voting, private message me with 'vote +1|-1|+0 #channelname')17:03
ddstreet+1 happy with technical work, good endorsements, and good understanding of process, certainly as far as ubuntu-server packages17:04
meetingology+1 happy with technical work, good endorsements, and good understanding of process, certainly as far as ubuntu-server packages received from ddstreet17:04
ddstreetyou could probably apply for coredev pretty soon, as well17:04
rafaeldtinoco+1 for ubuntu-server, will get him good experience for core-dev application17:05
meetingology+1 for ubuntu-server, will get him good experience for core-dev application received from rafaeldtinoco17:05
sil2100+1 seeing all the positive endorsements and work done so far, I see no big reasons against! I'm sure you'll do the right thing when in doubt17:05
meetingology+1 seeing all the positive endorsements and work done so far, I see no big reasons against! I'm sure you'll do the right thing when in doubt received from sil210017:05
sil2100Just remember about checking ALL the arches for SRUs next time... ;)17:06
sil2100(but also, don't worry too much - there's a lot of core-devs that forget as well!)17:06
ddstreetteward i think we're just waiting on your vote? since rbasak is abstaining17:07
ddstreetif you're still around17:07
ddstreetrbasak do you want to provide your vote, as it looks like teward had to leave?17:08
rbasakIf we can't make an absolute majority even with my vote, then I think I need to abstain for now, sorry.17:09
ddstreetack, let's give teward just another couple minutes then, since we're already over time17:09
=== genii-core is now known as genii
ddstreetok let's call it then17:11
ddstreet#endvote17:11
meetingologyVoting ended on: Packageset application for 'ubuntu-server' packageset for Athos Ribeiro17:11
meetingologyVotes for: 3, Votes against: 0, Abstentions: 017:12
meetingologyMotion carried17:12
ddstreetso technically, it did not pass, as we were short on votes17:12
rbasakOh, wait17:12
rbasakThere were 3 +1s?17:12
ddstreetyes17:12
rbasakSorry I miscounted17:12
ddstreetlol17:12
ddstreetyou want to +1 so we can close it out now?17:13
rbasakIn that case, +1, as everyone else still present is unanimous, and my vote will make quorum. That fits my regular abstention rule.17:13
ddstreetexcellent, in that case the motion passes, congrats athos!17:13
ddstreetas we're over time, i'll take the action to make the changes and announce the results17:13
ddstreetunless anyone else wants to quickly volunteer17:13
ddstreet#action ddstreet change permissions for athos packageset application for ubuntu-server17:14
meetingologyACTION: ddstreet change permissions for athos packageset application for ubuntu-server17:14
ddstreet#action ddstreet announce athos successful application for ubuntu-sever packageset17:14
meetingologyACTION: ddstreet announce athos successful application for ubuntu-sever packageset17:14
ddstreetok, so unfortunately we're now well over time, and we still have paride to consider for coredev17:15
paride:)17:15
ddstreetrbasak sil2100 rafaeldtinoco paride are *all* of you able to continue?17:15
parideI am17:15
rbasakI won't be here for long enough I don't think, unfortunately.17:16
rafaeldtinocoI can continue if others can17:16
rbasakHowever, since I'll be abstaining again, you don't really need me.17:16
rbasak(as an exception to normal)17:16
ddstreetparide so i think we won't be able to pass your application today, however we could proceed with questions and voting from at least rafaeldtinoco and myself, and then move the vote to the ML (and/or the next meeting)?17:17
ddstreetthe alternative is just to reschedule the application to the next meeting17:17
rafaeldtinocolets reschedule the meeting then17:17
rbasakSo I suggest that you continue without me if you can, and if required and everyone else is unanimous, I can provide my +1 if I can. Otherwise if it goes to email, then I can do the same on the email thread later unless someone else is 0 or -1 within a reasonable amount of time.17:17
ddstreetparide if you're ok with that let's do it today?17:17
parideddstreet, I am ok with that17:18
rafaeldtinocoalright17:18
ddstreetok let's do it then :)17:18
ddstreet#topic Paride Legovini application for Ubuntu Core Developer17:18
ddstreetparide can you introduce yourself?17:18
ddstreet#link https://wiki.ubuntu.com/ParideLegovini/UbuntuCoreDeveloperApplication17:18
paridesure17:18
parideI'm Paride Legovini, in the ubuntu-server (and canonical-server) team since 2019.17:19
parideIn the team I do QA and maintain the test infra for the server packages and for the projects developed by the team. But I also do packaging/distro work: merges, bug fixes, SRUs.17:19
parideI also maintain the ISO testing infra for Ubuntu Server, and doing ISO testing I also care about and sometimes touch the installer bits. I am also a Debian Developer.17:19
ddstreetthanks! rafaeldtinoco sil2100, any q?17:20
rafaeldtinoconot from me. I have directly worked with paride and I have confidence in his work. I think he even took too long to apply for core dev. Also, judging from his application page, I think he would be an awesome addition to core-dev team (should even have endorsed him but forgot to).17:22
ddstreetparide i have a couple minor q, both relating to sru work (since 99% of my work is in stable releases, that's almost always my focus)17:24
ddstreetfirst for this MR https://code.launchpad.net/~paride/ubuntu/+source/pmdk/+git/pmdk/+merge/40791017:24
ddstreetwhat's one thing that might make life significantly harder for the next person that looks at your changes?17:25
parideddstreet, well that patch is build by several commits squashed together17:26
parideand that's the reason why there's no direct commit reference in the Origin: DEP3 header17:26
paridethat's a reason for that:17:27
paridethe commits fixing that bugs (and related) tests touched the same files in some occasions17:27
parideand quilt doesn't like different hunks in a single patch file to touch the same file17:28
parideI could have made different patches, one per commit, maybe numerated17:28
ddstreethow does using a single patch file improve maintainability over using separate patch files?17:28
paridewell maybe it doesn't *really* improve it, but it has to be made clear that the separate patch files belong to the same "logical change" (bugfix in this case)17:29
ddstreetfor context, i'm not on the ~ubuntu-sru team, but my day job is 'sustaining' ubuntu so this is mostly my only change to correct people's bad SRU behavior on things that too often make it through SRU review17:29
ddstreetok so does using a single patch file make maintenance significantly worse?17:30
sil2100(no questions from me if anything)17:30
parideddstreet, mmh I may be wrong of course, but I'd say there's no universal answer. I don't think squashing in a single patch files always makes maintenance worse (otherwise I wouldn't have done it)17:32
ddstreetok but also it doesn't make it any better?17:32
ddstreetto ask it another way, when you would want to use one patch file per upstream commit, instead of squashing them all into a single patch file?17:33
parideif I don't expect to drop all of them at the same time17:34
ddstreetdrop all of them?17:34
ddstreeti dont follow what you mean?17:35
paridesorry. Let's say there is a new upstream release, and a related new merge from Debian17:35
paridehmm ... well but you were speaking specifically about SRUs17:36
ddstreetlet's talk generally then17:36
ddstreetis it *ever* good to use one quilt patch file per upstream commit, and also, is it *ever* good to squash multiple upstream commits into a single quilt patch?17:36
parideI'd say that if the upstream commits tackle with different aspects of an issue, and are somehow logically independent, it's good to keep them separated17:39
ddstreeti didn't expect this q to really go this long, so just to skip to the end, the answer is no, please don't ever squash multiple commits into a single patch file17:39
ddstreetthat probably will make it past sru review, but trust me, it makes the life of future maintainers much harder17:39
ddstreet1 quilt patch per upstream commit...PLEASE17:40
parideddstreet, I doubt I'll forget this now! :)17:40
ddstreetlol17:40
ddstreetand re: logical grouping, i personally always name patches with the bug prefix if there's just 1 patch (e.g. lp123456-commit_desc.patch) and use a subdir for multiple patches (e.g. lp1234567/001-first.patch, lp1234567/0002-second.patch, etc)17:41
ddstreetjust a suggestion that makes things much easier for me, that's not required17:41
ddstreetok just one more q, and i know we're WAY over time now17:41
ddstreetin the SRU template, what does the 'Regression Potential' or 'Where Problems Could Occur' section mean? what needs to go in there?17:42
parideddstreet, there we should assess what would happen is the proposed change is itself wrong or broken17:44
parideit basically challenges the assumptions that justify the SRUability of the fix17:44
paridewhich is normally expected not to have regression. That section deals with the unexpected consequences of the SRU.17:45
ddstreetso for example, in this bug https://bugs.launchpad.net/ubuntu/+source/postfix/+bug/190697017:45
ubottuLaunchpad bug 1906970 in postfix (Debian) "[SRU] dpkg hook hostname error" [Unknown, New]17:45
ddstreetwould you change anything about the answer there?17:45
parideI'd remove the generic "very low" bit17:46
ddstreetok thanks17:46
ddstreetand i apologize for even asking, as it's kind of a trick question as almost *everyone* sometimes gets that section wrong...personally i'd like to see the ~ubuntu-sru team just remove it from the SRU template entirely :-/17:47
ddstreetok let's move to voting then, and sorry for delaying the vote even longer17:48
ddstreet#vote Paride Legovini application for Ubuntu Core Developer17:48
meetingologyPlease vote on: Paride Legovini application for Ubuntu Core Developer17:48
meetingologyPublic votes can be registered by saying +1, -1 or +0 in channel (for private voting, private message me with 'vote +1|-1|+0 #channelname')17:48
ddstreet+1 strong yes, very good technical work, good endorsements, strong understanding of process, all my concerns were trivial(ish) nit-picks, certainly ready for coredev17:49
meetingology+1 strong yes, very good technical work, good endorsements, strong understanding of process, all my concerns were trivial(ish) nit-picks, certainly ready for coredev received from ddstreet17:49
sil2100+117:50
meetingology+1 received from sil210017:50
ddstreetrafaeldtinoco still around?17:50
rafaeldtinocoyep17:50
rafaeldtinoco+1 strong yes based in my experience working together17:50
meetingology+1 strong yes based in my experience working together received from rafaeldtinoco17:50
ddstreetrbasak it's three +1, if you're still around for a fourth?17:51
rafaeldtinoco(im not @ canonical any more so I can say that =)17:51
paridethanks Rafael :-)17:51
ddstreetok so i suspect that's all the votes we will get today, which isn't enough to pass now, but i'm fairly sure rbasak will give his +1 by email17:52
ddstreethe did say he would provide a +1 but just for the sake of process, let's get it officially from him in email17:53
ddstreet#endvote17:53
meetingologyVoting ended on: Paride Legovini application for Ubuntu Core Developer17:53
meetingologyVotes for: 3, Votes against: 0, Abstentions: 017:53
meetingologyMotion carried17:53
paridethanks to you all for staying so long!17:53
ddstreetagain not yet carried, but i expect it will pass in the next day or so once we get one more +117:53
ddstreetsorry for taking so long!17:54
ddstreet#action carry vote for paride to ML list for final +117:54
meetingologyACTION: carry vote for paride to ML list for final +117:54
ddstreet#undo17:54
meetingologyRemoving item from minutes: ACTION17:54
rafaeldtinocoparide: "almost" welcome =) I hope you get another +117:54
ddstreet#action ddstreet carry vote for paride to ML list for final +117:54
meetingologyACTION: ddstreet carry vote for paride to ML list for final +117:54
pariderafaeldtinoco, thanks! :)17:54
rafaeldtinocoddstreet: thanks for chairing AND for managing this action17:54
ddstreeti am sure you will paride so, pre-emptive congratulations :)17:54
paride\o/17:55
ddstreetok let's move on i think we are almost done17:55
ddstreet#topic change to DMB meeting day/time (ddstreet)17:55
ddstreetthis is done; i also started ML discussion around changing the 19:00 UTC mtgs17:55
ddstreet#topic next chair17:55
ddstreetgeez who knows on this...not sure if the chairing list really is useful anymore, at least currently17:56
ddstreet#topic AOB17:56
rafaeldtinocohahah17:56
ddstreet#topic officially cancel Dec 27 meeting?17:56
rafaeldtinocoYES17:56
rafaeldtinocoill be drinking margheritas :P17:57
rafaeldtinocoon the beach with kids17:57
ddstreetrafaeldtinoco damn you, southern-hemisphere! it's cold here! ;-)17:57
rafaeldtinoco=o)17:57
ddstreetok so i think that mtg is *officially* canceled.17:57
rafaeldtinocoi'd say its safe to consider that17:58
ddstreet#action ddstreet add note in agenda that the dec 27 mtg officially canceled17:58
meetingologyACTION: ddstreet add note in agenda that the dec 27 mtg officially canceled17:58
ddstreeti'll also note that i would like to hold just a pro-forma session that day, to run out the 6-meeting absence counter that we previously established...i'd like to start recruiting new members asap in the new year17:59
ddstreetbut none of our 'normally attending' members need to show up17:59
ddstreetnothing else from me, anything else before we close?17:59
ddstreet#endmeeting17:59
meetingologyMeeting ended at 17:59:55 UTC.  Minutes at https://ubottu.com/meetingology/logs/ubuntu-meeting/2021/ubuntu-meeting.2021-12-13-16.11.moin.txt17:59
ddstreetthanks all! o/17:59
ddstreetwow 2 hours18:00
rafaeldtinocoddstreet: thanks for chairing the longest meeting from DMB18:03
ddstreetrafaeldtinoco lol i get paid double for that meeting, right? xD18:04
rafaeldtinocoyep, in debcoins18:04
ddstreetoh sweet, i'm gonna be a debcoins millionaire...where is that debcoin ATM? ;-)18:05
ddstreetathos btw, i've already added you to ~ubuntu-server-dev so you should have upload rights now :)18:07
tewardddstreet: yeah i'm a bit insanely busy :P18:49
tewardlog2j has us in a tizzy at FT job18:49

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!