Patch Pilot Hand-off 24.04

Community
46

Did a very short session - reviewed the latest patch for Bug #2045816 “project-x fails to build from source in noble” : Bugs : project-x package : Ubuntu, made a slight modification to it to make a lintian-override more specific, then uploaded it. This is my first upload as an MOTU :tada:

2 Likes
47
48
49
50

No IRC pings to report.

I’d like to draw some attention on the xorg-server MP: the user clearly put some effort in preparing it, and it would be a pity to keep commenting on the nits (like I did) without being able to give actual feedback on the actual contents on the MP. I hope a future PP somehow familiar with xorg will be able to have a look at that diff.

51
  • LP: #2049515 - sponsored merge for iwd
  • LP: #2049311 - per above, submitter agreeded with suggestion to drop the delta on ldapscripts and sync the package. I did the sync.
  • no IRC pings
52
  • ldapscripts LP: #2049311 / MP 458569 a sync was already sponsored, so unsubscribed ubuntu-sponsors from the bug and marked the MP ‘Disapprove’
  • rsyslog LP: #2045033 / MP 458725 the merge already sponsored, so set the bug status to ‘Fix Committed’ and marked the MP ‘Approve’
  • cunit LP: #2049554 reviewed and sync’d
  • freeradius LP: #2042824 / MR 458744 reviewed and sponsored SRU
  • iptables LP: #2049318 reviewed and sponsored SRU
  • There was a request on irc to help diaspora-installer migrate, it had been removed from the archive and had previous autopkgtest results, but migration-reference/0 tests did not help. Added a hint to allow it to migrate
53
  • #2046871: unsubscribed sponsors given there are two unanswered path pilot comments already
  • xorg tearfree debdiff: out of my depth there, just added a simple comment, asked to rebase to noble-proposed, and subscribed @tjaalton
  • #2039873: reviewed the noble debdiff, asking for changes to the DEP8 test.
  • apparmor userns for systemd-coredump: this looks OK now, changing just systemd-coredump's apparmor profile, but the security team should have the final say. Left my non-security +1 in the MP.
  • #2049188: just flipped the bug status to “fix committed”. The sponsorship report page I was following still had this in the queue, but it was already uploaded a few hours ago.
  • #2049715: was already uploaded by the time I got to it in my shift, but I noticed that the linked MP had a bug in that it still had UNRELEASED in d/changelog, but LP accepted and built the package even so.
  • lektor MP: requested forwarding to debian, and DEP3 headers in the patches
54
  • LP: #1971547 – cyradm fails on jammy; sponsored
  • LP: #2044264 – mmv aborts with realloc; sponsored
  • LP: #1875642 – FSVS - not able to commit changes to http SVN repo – reviewed and suggested minor change, but if that can be addressed this should be good to sponsor
  • LP: #2039873 – liblxc-dev built with LXC_DEVEL=1 – reviewed current status; agreed quick-fix for LXC_DEVEL=1 is preferable way forward, suggested minor fix to DEP-8 test to check runtime files, not build-time, and addressed a minor build-failure on noble. Will sponsor for noble if someone can do a quick review of the changes
55
  • MP #2050390: Left a comment asking the submitter to mention the bug being fixed in the changelog.
  • MP #458934: Left comments requesting improvements to the MP.
  • MP #459015: Left a comment asking for DEP-3 headers + some clarification re. the MP description.
56
  • ltrace: already uploaded; marked MP as Merged
  • xorg-server “tearfree”: this has been in the queue for a while and commented on by various people. The issue is that it is (good) original work that really belongs upstream rather than in a distro patch with a maintenance burden. Previous pilots have tried to help in this direction, but ultimately it doesn’t really belong in the queue since there is no action to take in Ubuntu. I marked the MP as “Work in progress” with an explanation and instructions on how to put it back in the queue again if the submitter needs.
  • liblxc-dev LXC_DEVEL=1: again this has been commented on by various pilots. @waveform kindly said he’d take another look, rather than me adding yet another cook to spoil this broth.
  • lektor update for werkzeug: again this has been commented on by previous pilots. The issue I have with sponsoring this is that it doesn’t seem to be possible to reconcile the patch against the upstream it is claimed to came from. I asked for details, and queried why we aren’t taking the upstream maintenance release that includes the necessary fixes.
  • onionshare update for werkzeug: I asked about appropriate testing on this. Sure, it might make dep8 tests pass, but if we’re pushing ahead with selecting backports from upstream, does it actually fix the issue such that the app works again, and what’s the plan to verify this? I left a comment.

Some observations

The werkzeug work is important, but I wonder if it stretches a little beyond the capabilities of patch piloting. Same for liblxc-dev where the bug seems to have gone into a tangent about making more significant packaging updates than a simple patch.

I also noticed that the ~ubuntu-sponsors MP slot is being taken and thus MPs are disappearing from the queue. I’ve commented about this before but there was no further discussion on this point. I think I will probably just change the bot and queue to use ~ubuntu-sponsors-reporter when I get round it to work around this issue as I suggested. In the meantime, I think MPs are still being lost from the queue.

57

Sponsored fsvs, dotnet8 and lcov. Also investigated gettext which sadly had already been sponsored without git-ubuntu.

Looked at openssh, cinder and mutter but didn’t feel up to reviewing those more complex merges today.

58

Short flight today:

  • Sponsored google-guest-agent for @kajiya on Friday(?), looks like @ahasenack sponsored a followup with some minor tweaks. I’ve reviewed those tweaks and now understand what I was missing on my end; thanks for the followup, I’ll be happy to sponsor the SRUs too if needed.
  • I’ll volunteer to take ruby3.1, assuming @adrien can come up with a followup diff. :smile:
  • Sponsored linux-media-player for sudip, that’s in the Jammy queue.

I’d recommend python-uefivars as a starting point for the next pilot.

Thanks!

1 Like
59

Many things from the sponsoring report are currently in-flight and I was mostly side-tracked helping out with the time_t transition NMUs in Debian, so didn’t get a lot of stuff done, but at least I could clean up a little bit:

  • MP#458934: Clear onionshare vs python-werkzeug 3.0 MP, it has been resolved via Debian experimental
  • MP#458931: Clear lektor vs python-werkzeug 3.0 MP, it has been resolved via Debian unstable
60
  • MP: #458533: Reviewed, will sponsor once it’s ready to go
61

Minor housekeeping on open bugs and MPs. Open context:

IRC activity: handover comments between pilots. No users’ questions/requests.

62
63
1 Like
64
65
  • Looked through the walinuxagent merge, but it seems Lucas has things well in hand there, and given the complexity of the patch I’m inclined to wait for his review of Calvin’s recent revision before taking any further action.
  • LP: #2039037 – Reviewed optee-test-s32 and left some suggestions and requests
  • LP: #2034650 – Reviewed and sponsored uboot-s32 with some minor adjustments to the changelog
  • Skipped the no-esm merges as those are pending reviews from specific interested parties
  • LP: #2051908 – Reviewed; pending concerns from prior reviewers, unsubscribed ubuntu-sponsors until these are addressed
  • LP: #2051270 – Reviewed and sponsored sysdig SRU
  • LP: #2040059 – Reviewed and intending to sponsor sagemath pending a local build; would be a no-change rebuild but for a horrid need to bump the arbitrary “tolerated build failures” number (but given that’s already 200 (!) by default, and the application is in a completely-broken-won’t-even-launch state, what’s 30 more … urgh!)