Patch Pilot Hand-off 25.10

Hello fellow Patch Pilots and Ubuntu community members,

This post serves as the official hand-off for the Patch Pilot Program for the 25.10 cycle. The purpose of this thread is to ensure continuity in our program, keeping operations smooth and contributors supported. Whether you’ve just wrapped up a piloting session or are gearing up for your session, this space ensures that no detail gets lost in the shuffle. If you are interested in what is going on, you can subscribe to this post.

:warning: Please reserve responses in this thread ONLY to hand-off reports. If you have a question as a contributor please check with the current patch pilot. If you need a clarification from the last pilot please reach out to them separately. If you have a more general question please create a new topic in this category.

To the incoming pilot: please take a moment to review the updates from the previous session. It provides the context you’ll need to pick up where they left off. When writing your own report, please include the following headlines:

  • Notable Achievements
  • Important Context
  • Notable Contributors (tag them if you know their discourse username)
  • Program Improvements (anything that you think needs documenting or otherwise improving)

Links: Latest hand-off post | Previous cycle hand-off

1 Like
  • Skimmed Julian’s rust-sequoia merge; suspect there’s nothing for patch-pilots to do here
  • LP: #2099283 – landscape-client – Reviewed current status, and left a comment requesting a questing upload
  • LP: #2059246 – rclone-browser – Reviewed current status; left comment: nothing to do until sync from Debian completes
  • LP: #1980991 – powermgmt-base – Reviewed current status; left review on merge

Review:

Added comment:

Unsubscribed sponsors:

Submitter needs to raise the SRU:

Should be reviewed by the maintainer:

Sponsored:

  • LP: #1980991 – powermgmt-base – Reviewed current status; nothing to do as it’s already been sponsored (and it’s a merge so won’t disappear from the list until the merge lands)
  • LP: #2099283 – landscape-client – Reviewed current status; sponsoring for … well, pretty much everything (reviewing and sponsoring all these took up most of the shift)
  • LP: #2059246 – rclone-browser – Reviewed status; prepared and uploaded patches for plucky, oracular, noble, and filled out SRU template
  • MP: #486095 - reviewed, tested, and added comments, this one took most of my time to go over
  • LP: #2105849 - reviewed for questing

Sponsorship:

Review:

Commented on:

  • ubuntu-x1e-settings SRU LP: #2107692 already uploaded, unsubscribed ubuntu-sponsors
  • mdadm LP: #2111802 / MP 486139 reviewed and commented, reviewed changes and sponsored
  • gcc-15 LP: #2108968 delayed until gcc-15 is default, unsubscribed ubuntu-sponsors
  • broadcom-sta SRU LP: #2110927 already uploaded, unsubscribed ubuntu-sponsors
  • linux-apfs-rw SRU LP: #2110915 already uploaded, unsubscribed ubuntu-sponsors
  • openrazer SRU LP: #2111339 already uploaded, unsubscribed ubuntu-sponsors
  • fontconfig LP: #2111850 / MP 486158 reviewed and commented

I’m skipping sponsorships or reviews of the SRU type, since that would mean I should abstain from doing the SRU review later on.

  • LP: #2109890 – lxc – Reviewed, fixed up questing patch, targetted and sponsored for questing, plucky, oracular, and noble
  • Spent some time figuring out why Firefox won’t load the sponsoring report … something something HSTS, clear all history for ubuntu.com, and it loads … bah!
  • LP: #2100570 – euslisp, jskeus – Reviewed and sponsored euslisp for noble / oracular, couldn’t find source for jskeus

I started with 57 items and went down to 40 minus I think 3 or so pending so we should come out at 37. Given the sheer number of items I was unable to effectively take notes. The important bits are:

To maintain efficiency I skipped over non-merges of new upstream versions as trying to find the tarball and validating those can be time consuming.

LP: #2110421 - Reviewed and sponsored
MP: #487210 - Reviewed and sponsored
LP: #2112053 - Ran PPA autopkgtests and started looking over it. As far as I’ve seen it looks good but I ran out of time to review manpage and other diff updates

Many of the items in the sponsorship queue are related to SRUs. I skip those, because otherwise I won’t be able to do the SRU processing (conflict of interest). The remaining ones are:

  • resource agents merge: preemptively approved, pending autopkgtests results
  • azure-proxy-agent PR: marked as merged, as it was sponsored already, and is even in the archive by now.
  • LP: #2112632: efibootmgr: looks like a valid bug, based on my testing. I marked the affected ubuntu releases, found an upstream patch, asked if the user would like to prepare a PR with the upstream patch instead and that I would guide them if needed. I subscribed to the bug so I can get notified of a response.
  • PAM merge from debian: I just started looking when I noticed that there was a reviewer assigned already. In any case, just left a comment saying that it needs to be rebased on top of a security update.
  • LP: #2114945: interesting discussion about automounting different types of filesystems and the security implications. I learned that we already don’t automount certain filesystems via a set of udev rules. Left a command there asking for next steps, since the original diff (for kmod) is probably being abandoned.
  • LP: #2115041: hard to understand SRU template. I left a question, and also commented that the status of the bug for oracular, plucky, and questing, needs to be clear and set before the SRU for noble can continue.
  • LP: #2114778: I have no idea what “Patched AdwSpinner to Match yaru.dart’s YaruCircularProgressIndicator Design and Animation” means, sorry. So I just did a syntax-based review, and added a review slot for ubuntu-desktop.
  • LP: #2109673: sssd and apparmor profile. I did some tests, and I’m wondering if we shouldn’t take the opportunity to create a better profile for sssd overall, one that uses child profiles as well. This is a complex piece of software, and won’t be easy, but could be a good goal for the next LTS, and we have one interim release to stabilize it. Also, given that this profile is not installed in enforce mode by default, I question the need to SRU this. I think we should first get it right in questing and later, so that it can be used in enforce mode.
  • debmirror merge from debian: reviewed, left a comment and a +1.
  • LP: #2106320 – libapache2-mod-auth-openidc – Security issue; fix already released, marked bionic “won’t fix”
  • LP: #2109673 – sssd – Reviewed status; looks like ahasenack and security are handling this one; added brief note on the associated MR about including changelog bug-ref
  • LP: #2107773 – dovecot – Reviewed status; targetted according to Bryce’s recommendation in the noble MP (given reporter may not have rights to target); left comments summarising remaining actions for reporter
  • LP: #2106757 – wsl-pro-service – Reviewed status; left comments about debdiff and full-version bump (are SRU team okay with this package being bumped without an exception?)
  • LP: #2106791 – libvirt – Reviewed status; nothing to do currently (pending changes requested by prior reviewer)
  • LP: #2084169 – filelight – Ready for sponsorship but only targets oracular which is near EOL; leaving as time is likely better spent on other issues
  • LP: #2108997 – gdb – Reviewed status; sponsored for plucky and noble, leaving oracular (near EOL), and jammy (out of scope, perhaps?); unsubscribed sponsors