/srv/irclogs.ubuntu.com/2022/09/06/#ubuntu-meeting.txt

cpaelzerslowly prepping the MIR meeting14:27
cpaelzer#startmeeting Weekly Main Inclusion Requests status14:31
meetingologyMeeting started at 14:31:53 UTC.  The chair is cpaelzer.  Information about MeetBot at https://wiki.ubuntu.com/meetingology14:31
meetingologyAvailable commands: action, commands, idea, info, link, nick14:31
cpaelzerPing for MIR meeting - didrocks joalif slyon sarnold cpaelzer jamespage14:32
joalifo/14:32
cpaelzerhello everyone, I know a few are on PTO, others are sick and I wasn't around for a few weeks so I might have lost all context14:32
cpaelzernevertheless les us get going14:32
cpaelzerhi joalif14:32
sarnoldgood morning14:32
didrockshey14:32
cpaelzerhi sarnold14:32
cpaelzerhi didrocks14:32
cpaelzerslyon is off today14:32
cpaelzerand jamespage always has a hard time to attend, btw @jamespage if you want to send anyone else representin the openstack team let us know about it14:33
cpaelzer#topic current component mismatches14:33
cpaelzerMission: Identify required actions and spread the load among the teams14:33
cpaelzer#link https://people.canonical.com/~ubuntu-archive/component-mismatches-proposed.svg14:33
cpaelzer#link https://people.canonical.com/~ubuntu-archive/component-mismatches.svg14:33
cpaelzerdo I see 6 approved MIRs14:33
cpaelzer- all known false positives14:34
cpaelzerthat might be 414:34
jamespagecpaelzer: an idea14:34
cpaelzerlibhtml-tokeparser-simple-perl and libfreezethaw-perl seem to be ready for promotion AFICS14:35
didrocksyeah, they are14:35
cpaelzerI'm taking a todo to double check and do that tomorrow morning14:35
cpaelzertoday is meeting overload, hence tomorrow :-)14:36
cpaelzerok, nothing else in there14:36
cpaelzerfeature freeze helps :-)14:36
cpaelzer#topic New MIRs14:36
cpaelzerMission: ensure to assign all incoming reviews for fast processing14:36
cpaelzer#link https://bugs.launchpad.net/ubuntu/?field.searchtext=&orderby=-date_last_updated&field.status%3Alist=NEW&field.status%3Alist=CONFIRMED&assignee_option=none&field.assignee=&field.subscriber=ubuntu-mir14:36
cpaelzertund is now up14:36
cpaelzertuna already got assigned to joalif last week IIRC14:36
didrocksI can have a look at tuned as nobody volunteered in between14:37
cpaelzersarnold: is your question of the relationship between those sufficiently answered?14:37
joalifI'm still reviewing it tuna has a few problems that i need to discuss14:37
cpaelzeryes, didrocks that would be great14:37
sarnoldcpaelzer: not really, it'd be nice to hear something specific that it does that the others cannot do14:37
cpaelzerjoalif: do you want/need to discuss it here (and now)? Or with the bug reporter on the bug?14:37
joalifwell tbh there's no template filed for tuna, should I ask for it ? (i've almost done the review)14:38
joalifalso to run it needs root shall it go through security ?14:38
joalifthere are some other problems but those canbe discussed with the reporter in the bug14:39
cpaelzerI updated the case in regard to seths question14:39
cpaelzernow reading the issue here14:39
cpaelzeryes ask for a template for tuna14:40
cpaelzerjsalibury just isn't away of the process details, so let him know and I'm sure he will add it14:40
joalifack thanks14:40
cpaelzerjoalif: you mean the daemon runs as root right?14:40
joalifit is not a deamon, it is the application itself14:41
joalifit messes with cpu affinity and irqs14:41
didrocksclearly needs a security review IMHO :)14:41
joalifso it needs to be run as root14:41
joalifyeah that's my feeling too14:41
joalifit needs security check14:41
cpaelzerI guess we are all clear why that rule exists, running as root ives it more power which makes it more prone to mess up things when exploited14:42
cpaelzerhence yes, it usually means that we want a security check14:42
joalifhowever this is for later, there are other need to be done before security review14:42
cpaelzereven though, not being a daemon there will not be a port or api that can be accessed and exploited14:42
cpaelzerthere might be e.g. users dropping conffiles somewhere for root to pick it up via the tool - and boom14:43
sarnoldsomewhere in the service there's a d-bus interface14:43
cpaelzeris it, then eeven more -> yes security review14:43
sarnoldthat might be more the tuned thing than a tuna thing14:43
joalifsarnold: i think dbus is for tuned14:43
cpaelzerand joalif, if you have more for them to answer or implement reflect it back to them and set it to incomplete14:43
cpaelzersince we also wait for the answers to Seths question14:43
joalifyup I'll do14:43
cpaelzerand to whatever didrocks finds on tuna14:43
cpaelzerI guess for now this case is ok14:43
didrocksyep14:43
cpaelzerit won't be 22.10 material anyway14:43
cpaelzernothin else in the list14:44
cpaelzer#topic Incomplete bugs / questions14:44
cpaelzerMission: Identify required actions and spread the load among the teams14:44
cpaelzer#link https://bugs.launchpad.net/ubuntu/?field.searchtext=&orderby=-date_last_updated&field.status%3Alist=INCOMPLETE_WITH_RESPONSE&field.status%3Alist=INCOMPLETE_WITHOUT_RESPONSE&field.subscriber=ubuntu-mir14:44
cpaelzeropenconnect is stil getting updates, but also still incomplewte14:44
cpaelzerno need to act on it14:44
cpaelzerqtr I got contacted by seb14:45
sarnoldah, good14:45
cpaelzerhe mentioned that he discussed it her ein the past, we entered a mail series that tries to come up with "if we do, what would we need to ack it as special case"14:45
cpaelzerafter some iterations of that we might come back here for a group ack14:45
cpaelzerbut it isn't ready enough yet14:46
cpaelzerso I can#t discuss more details yet, but will come back14:46
didrocksteasing :)14:46
sarnoldlol14:46
cpaelzerTL;DR in some cases e.g. vendoring and some other cases e.g. multipath/kernel not having all HW - we already make excuses. But we then in turn rquire a clear "yes I really know what I'm committing to here" + "This is how I try to make this situation better in the long run"14:47
cpaelzersomething along these lines it might end up to make me consider it a "well ok, special case ack"14:47
cpaelzerI understand the case and want to help, but OTOH I do not want to make it too easy tough, or it will just become the pattern everone uses14:48
cpaelzerok enough of that, going on ...14:48
cpaelzer#topic MIR related Security Review Queue14:48
cpaelzerMission: Check on progress, do deadlines seem doable?14:48
cpaelzer#link https://bugs.launchpad.net/~ubuntu-security/+bugs?field.searchtext=%5BMIR%5D&assignee_option=choose&field.assignee=ubuntu-security&field.bug_reporter=&field.bug_commenter=&field.subscriber=ubuntu-mir14:48
cpaelzerInternal link14:48
cpaelzer- ensure your teams items are prioritized among each other as you'd expect14:48
cpaelzer- ensure community requests do not get stomped by teams calling for favors too much14:48
cpaelzer#link https://warthogs.atlassian.net/jira/software/c/projects/SEC/boards/59414:48
sarnoldunfortunately I stillh aven't caught up after covid :/ mdevctl is in progress, mark is working on editorconfig-core -- I'm worried that it might allow untrusted inputs straight to pcre, which is historically a horrible idea14:50
cpaelzerbut pcre itself it mean to be "safe" as it is in main- isn't it?14:50
sarnoldthe team has other, higher, priorities at the moment, so it'll be just me again for a while14:50
sarnoldnot really :(14:50
sarnoldthe expression compiler is unsafe14:51
cpaelzerit was so great seing the recent progress, sad to hear you are along again14:51
cpaelzerbut I understand that there are many things pulling, and for now we are way into FF anyway14:51
sarnoldafaik only go and rust's regex engines are intended to be safe for untrusted inputs14:51
cpaelzersarnold: if there is anything not "yes ok" on mdevctl please let myself and athos know immediately about it14:51
sarnold(they're not perfect but at least they try)14:51
sarnoldcpaelzer: will do, athos has been very responsive so far :) 100% would recommend, hehe14:52
cpaelzergreat14:52
cpaelzersarnold: one more thing - all the ccid/opensc/smart-card which is marked "in progress"14:52
cpaelzersarnold: I was told this might get a bump as you lost some related resources, does this go back to square #1 or will this stay in this state?14:53
sarnoldcpaelzer: good question. I think it'd be wise for security to talk again with desktop and make sure it's still a desired feature14:53
jbichasarnold: btw, gnome-text-editor 42 embeds editorconfig-core; 43 is switching to use the system library 🫤14:53
cpaelzerI think it is for all the enterprise desktop people asking for it14:53
sarnoldcpaelzer: maybe there's sufficient interest to keep trying on it once we've made more hires14:53
sarnoldjbicha: ugh :)14:54
cpaelzerbut yeah having that talk is the right next step sarnold14:54
sarnoldjbicha: thanks for fighting the devendoring fight :)14:54
cpaelzerjbicha: so we had it all the time, just now it becomes visible?14:54
jbichacpaelzer: gnome-text-editor is new to main for 22.1014:54
cpaelzerI see14:54
sarnoldwell, gnome-text-editor is pretty new, replacing whatever the old gnome text editor was, right?14:54
jbichaI didn't devendor it; upstream did to get us to pcre2 I guess14:54
cpaelzerok, but mark is ont hat csae14:55
cpaelzerso we can expect some progress14:55
cpaelzergoo14:55
* athos feels recommended!14:55
cpaelzerI think we are fine with this section14:55
sarnold:)14:55
cpaelzerathos: - you are14:55
cpaelzer#topic Any other business?14:55
sarnoldyes14:55
cpaelzernothing from me, the one I had was that libqtr thing I mentioned above14:56
cpaelzerdidrocks: joalif: anything from you14:56
joalifnothing from me14:56
cpaelzersarnold: what is it then?14:56
didrocksnothing for me14:56
sarnoldthe openconnect, stoken, fstrm, etc MIRs were all filed by  Luís -- and he has been uninvited from ubuntu for another year14:56
sarnolds/another//14:56
cpaelzer"uninvited" ?!14:56
sarnoldyeah, the community council heard enough complaints about his working style that they weren't able to address to their satisfaction14:57
sarnoldI hope when the year is up he's more open to workflow suggestions..14:57
cpaelzerI see14:57
cpaelzerso you are saying there will be a social/community aspect to the review7approval of these once they are no more "incomplete"14:58
sarnoldanyway, we've got a half-dozen or so half-filed MIRs in various states; we could either continue on without his input, or set them all WONTFIX, or leave them as is and let them expire on their own organically14:58
sarnoldhe signed up teams for support of the things without actually having conversations with the affected teams, so I don't think we can just take the bugs at face value14:59
cpaelzerI would not want to stop someone trying to improve Ubuntu. But one of the main things all these need is a team that will own it, we might check on that and only that first when the cases are no more incomplete.14:59
jbichadesktop hasn't reviewed the openconnect request. I was concerned that Security was hesitant about it in their initial review14:59
cpaelzerwithout finding such, we can't go on - no matter how ok or not any other aspect is14:59
cpaelzersarnold: we might just point at the "owning team rule" in FYI updates to the cases15:00
cpaelzerjust to manage expectations15:00
cpaelzerfells better (to me) than immediately going to Won't Fix15:00
cpaelzerWDYT?15:00
didrocksyeah, basically the fact that the team owner should agree first is the deal breaker15:00
sarnoldthese bugs weren't a significant hindrence to our meeting today so status quo might be perfectly fine15:00
cpaelzerok15:01
cpaelzerthanks for raising this for awareness15:01
cpaelzerwill help to not spend too much effort before those details are clarified15:02
cpaelzerthat seems it was all we had15:02
cpaelzerready to close this for today?15:02
sarnoldfine by me15:02
didrockssounds good!15:02
joalifthanks cpaelzer, all :)15:02
sarnoldthanks cpaelzer, all :)15:02
cpaelzerok, thank you all!15:02
didrocksthanks cpaelzer, all15:02
cpaelzer#endmeeting15:02
meetingologyMeeting ended at 15:02:52 UTC.  Minutes at https://ubottu.com/meetingology/logs/ubuntu-meeting/2022/ubuntu-meeting.2022-09-06-14.31.moin.txt15:02
=== genii_ is now known as genii
=== arraybolt3_ is now known as arraybolt3

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