Review sprint #4

It’s that time again!

We have quite a few pending PRs open ranging from major features all the way to trivial tweaks. To ensure these get addressed timely, we’ll have a more focused effort over the next few days to work through them. If you are an author or a reviewer, let’s please collaborate towards that.

Below we have the PRs which were open when the sprint started today, and perhaps a few important ones that were opened later and then integrated into the sprint. The notations used are described after the list.

For any required updates unrelated to status changes, please comment below. Status changes will be updated regularly and automatically.


:negative_squared_cross_mark::negative_squared_cross_mark: #3734: packaging: add debian-unstable {+:nz+} :white_check_mark::white_check_mark: #3916: snap,wrappers: add support for socket activation {+:nzcj+} :grey_question::grey_question: #3963: cmd/snap-confine: add support for per-user mounts {+:z+} :white_check_mark::negative_squared_cross_mark: #3994: tests: fix revert test when a new core image is pushed {s:zmw} :white_check_mark::exclamation: #3998: snap-confine, snap-seccomp: utilize new seccomp logging features {+:zj+} :jack_o_lantern: :no_entry::grey_question: #4049: debian,vendor: import and use {m:} :jack_o_lantern: :white_check_mark::white_check_mark: #4063: tests: add new kernel refresh/revert test for spread-cron {m:wz} :jack_o_lantern: :no_entry::grey_question: #4068: interfaces/builtin: add support for content “source” section {z:wm} :exclamation::grey_question: #4073: snap: add io.snapcraft.Settings to snap userd {m:jz} :white_check_mark::white_check_mark: #4078: tests: new test to check interfaces after reboot the system {s:zmw} :jack_o_lantern: :white_check_mark::exclamation: #4100: add ssh-keys, ssh-public-keys, gpg-keys and gpg-public keys interfaces {j:zn} :exclamation::exclamation: #4103: snapstate: auto install default-providers for content snaps {m:zw} (2.30) :white_check_mark::white_check_mark: #4108: repo: ConnectedPlug and ConnectedSlot types {w:pnz} :white_check_mark::white_check_mark: #4109: cmd/libsnap: fix parsing of empty mountinfo fields {z:bwj} :white_check_mark::white_check_mark: #4120: repo: use PlugInfo and SlotInfo for permanent plugs/slots {w:zm} :white_check_mark::white_check_mark: #4122: configstate: add support for configure-snapd for snapstate.IgnoreHookError {m:pw} (2.30) :white_check_mark::white_check_mark: #4129: wrappers: do not error on incorrect Exec= lines {m:wzpc} :white_check_mark::grey_question: #4140: interfaces: add an interface for gnome-online-accounts D-Bus service {+:zj} :white_check_mark::white_check_mark: #4144: interfaces: fix udev tagging for hooks {z:wjmp} :white_check_mark::white_check_mark: #4149: tests: new tests for network setup control and observe interfaces {s:bm} :white_check_mark::white_check_mark: #4152: snapd: fix snap cookie bugs {w:pmz} :white_check_mark::white_check_mark: #4154: overlord/snapstate: support completion for command aliases {c:pw} :white_check_mark::white_check_mark: #4157: add spread test for connecting all interfaces (excepting gadget slots) {j:bm} :white_check_mark::white_check_mark: #4158: snapctl: don’t error out on start/stop/restart from configure hook during install or refresh {w:pzm} (2.30) :negative_squared_cross_mark::negative_squared_cross_mark: #4159: cmd/snap-confine: add slave PTYs and let devpts newinstance perform mediation {j:z} :white_check_mark::white_check_mark: #4161: snapstate: add support for refresh.schedule=managed {m:pnzb} (2.30) :white_check_mark::white_check_mark: #4163: cmd/snap-update-ns: re-factor secureMkdirAll into secureMk{Prefix,Dir} {z:wbjp} :white_check_mark::white_check_mark: #4166: cmd/snap-update-ns: detect and report read-only filesystems {z:bw} :white_check_mark::white_check_mark: #4169: cmd/snap-update-ns: add secureMkfileAll {z:bj} :white_check_mark::white_check_mark: #4170: cmd/snap-update-ns: add planWritableMimic {z:bjpm} :white_check_mark::white_check_mark: #4171: tests: adding test to test physical memory observe interface {s:mbzw} :white_check_mark::white_check_mark: #4173: corecfg: validate refresh.schedule when it is applied {m:zp} :white_check_mark::white_check_mark: #4174: packaging/arch: packaging update {b:z+} :white_check_mark::white_check_mark: #4177: state: add change.LaneTasks helper {w:mpb} :white_check_mark::white_check_mark: #4178: asserts/assertstest: fix use of hardcoded value when the passed or default keys should be used {p:mz} :white_check_mark::white_check_mark: #4179: tests: add a spread test for setting together with store assertion {p:mws} :white_check_mark::white_check_mark: #4180: interfaces/many: misc policy updates for browser-support, cups-control and network-status {j:mz} :white_check_mark::white_check_mark: #4184: tests: adding new test for uhid interface {s:mz+} :white_check_mark::white_check_mark: #4185: interfaces/builtin/account_control: use gid owning /etc/shadow to setup seccomp rules {b:zj} :white_check_mark::white_check_mark: #4187: tests: fix xdg-open-compat {m:wb} :negative_squared_cross_mark::negative_squared_cross_mark: #4188: osutil: replace cgo bits with non-cgo, vendored os/user {b:c} :white_check_mark::white_check_mark: #4189: interfaces/builtin/lxd_support: allow discovering of host’s os-release {b:wzm} :white_check_mark::white_check_mark: #4190: cmd/snap-seccomp: do not use group ‘shadow’ in tests {b:wm} :white_check_mark::white_check_mark: #4191: cmd/snap-update-ns: do not assume ‘nogroup’ exists {b:wz} :white_check_mark::white_check_mark: #4192: osutil: add helper for obtaining group ID of given file path {b:wm} :white_check_mark::white_check_mark: #4197: cmd/snap-confine: Support bash as base runtime entry {+:mbj} :white_check_mark::white_check_mark: #4198: release: 2.29.3 {m:z} :negative_squared_cross_mark::negative_squared_cross_mark: #4199: cmd/snap-confine: Add support for 32-bit NVIDIA on biarch {+:b+} :white_check_mark::white_check_mark: #4200: daemon,overlord/snapstate: return snap-not-installed error in more cases. {+:mcz} :white_check_mark::white_check_mark: #4201: tests/lib: handle distro specific grub-editenv naming {b:zm}


50 total → ( 1 waiting → 7 reviewing → 3 reviewed → 34 merged ) → 39 closed

Status of each PR:

  • :grey_question: Waiting for review
  • :exclamation: Open review
  • :white_check_mark: Reviewed and approved
  • :white_check_mark: Merged
  • :negative_squared_cross_mark: Closed without merging
  • :no_entry: Blocked
  • :scream: Critical
  • :skull: Decaying
  • :jack_o_lantern: Broken spread tests (click to see)
  • (2.NN) Targets given milestone
  • {.:…} Reviews from regulars (see below)

Regular reviewers

As an aid during the review process, developers participating in reviews regularly are marked with:

  • {1:2345}

Where the regular reviewer:

  1. Submitted the PR
  2. Approved the PR
  3. Was asked to review the PR
  4. Requested changes in PR
  5. Commented on the PR

The following regular reviewers are currently being tracked:

Char Reviewer
b @mborzecki
c @chipaca
j @jdstrand
m @mvo
n @niemeyer
p @pedronis
s @cachio
w @pstolowski
z @zyga-snapd
+ Exceptional

Note that those are the regular reviewers. We have many more contributors than this.

1 Like

This review sprint round is now finished!

Thanks to everybody who helped getting those bits in (or out :).