[03:02] <mup> PR snapd#10051 opened: cmd/snap-bootstrap/initramfs-mounts: move time forward using assertion times <Run nested> <Squash-merge> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10051>
[07:48] <mborzecki> morning
[07:53] <zyga-mbp> long time no see, I was not feeling very well lately
[07:54] <mborzecki> zyga-mbp: hey
[07:54] <zyga-mbp> hey :)
[08:04] <pstolowski> morning
[08:05] <zyga-mbp> hey pstolowski :)
[10:28] <mup> PR snapd#10052 opened: features: add refresh-control feature flag <Refresh control> <Simple 😃> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10052>
[10:48] <mup> PR snapd#10053 opened: o/snapstate: helper for getting snaps affected by refresh, define new hook <Needs Samuele review> <Refresh control> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10053>
[11:13] <mup> PR snapd#10054 opened: snapdtool, wrappers: add dependency on usr-lib-snapd.mount for services on core with snapd snap <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10054>
[12:49] <mup> PR snapd#10055 opened: wrappers: launch apps through /usr/lib/snapd/snap on core with snapd snap <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10055>
[13:49] <mup> PR snapd#9959 closed: o/snapstate: update validation sets assertions with auto-refresh <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/9959>
[15:11] <mup> PR snapcraft#3474 opened: Nimble sread <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3474>
[15:16] <mup> PR snapcraft#3474 closed: Nimble sread <Created by sergiusens> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/3474>
[15:16] <mup> PR snapcraft#3475 opened: godeps: set default channel to 1.15/stable <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3475>
[15:16] <mup> PR snapcraft#3476 opened: ci: reduce amount of artifacts to upload for spread <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3476>
[15:30] <pedronis> ijohnson: I reviewed #100051
[15:30] <mup> Bug #100051: ubuntu distro updater crashed <update-manager (Ubuntu):Confirmed> <https://launchpad.net/bugs/100051>
[15:30] <pedronis> heh
[15:30] <pedronis> ijohnson: I reviewed #10051
[15:30] <mup> Bug #10051: gdm: new changes from Debian require merging <gdm (Ubuntu):Fix Released by seb128> <https://launchpad.net/bugs/10051>
[15:30] <mup> PR #10051: cmd/snap-bootstrap/initramfs-mounts: move time forward using assertion times <Run nested> <Squash-merge> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10051>
[15:33] <pedronis> mborzecki: question in https://github.com/snapcore/snapd/pull/10054
[15:33] <mup> PR #10054: snapdtool, wrappers: add dependency on usr-lib-snapd.mount for services on core with snapd snap <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10054>
[15:37] <ijohnson> pedronis: thanks, I'll have a look in a minute, snapcraft clinic right now
[15:44]  * cachio_ lunch
[15:45] <zyga> ijohnson really nice branch with that model assertion and time
[15:46] <ijohnson> thanks zyga
[15:57] <pedronis> cachio_: we are getting failure like  /home/gopath/src/github.com/snapcore/snapd/tests/lib/prepare-restore.sh: line 493: go: command not found
[15:58] <pedronis> cachio_: in https://github.com/snapcore/snapd/pull/9307
[15:58] <mup> PR #9307: interfaces/tee: add TEE/OPTEE interface <Squash-merge> <Created by ogra1> <https://github.com/snapcore/snapd/pull/9307>
[16:24] <mup> PR snapd#10056 opened: cmd/snap: packaging: move snap binary to /usr/lib/snapd/snap, fix invocation of internal tools <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10056>
[16:26] <mup> PR core20#100 opened: Add a wrapper for invoking the snap binary <Created by bboozzoo> <https://github.com/snapcore/core20/pull/100>
[16:33] <ijohnson> pedronis: cachio_: I wonder if the hirsute image changed, also I see that there is a failure above that line too:
[16:33] <ijohnson> This package is uninstallable
[16:33] <ijohnson> Dependency is not satisfiable: dh-systemd
[16:33] <ijohnson> pedronis: I bet merging master would help that pr
[16:41] <cachio_> ijohnson, the image change every day
[16:41] <mup> PR snapcraft#3477 opened: spread tests: pin go for v1 plugin snaps <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3477>
[16:41] <cachio_> is it imported from cloud project
[16:41] <ijohnson> cachio_: right so I wonder if the go package was removed from the image
[16:42] <cachio_> I can check that
[16:48] <cachio_> ijohnson, https://paste.ubuntu.com/p/G2dhMZ7sJN/
[16:48] <cachio_> no go in the image
[16:48] <ijohnson> cachio_: so we probably need to install go then
[16:48] <ijohnson> cachio_: do the other ubuntu images have go in them?
[16:49] <cachio_> ijohnson, yes but because we process the image
[16:49] <ijohnson> but we don't process the hirsute image?
[16:49] <cachio_> this image is imported from ubuntu-os-cloud-devel project
[16:49] <cachio_> no
[16:49] <ijohnson> right
[16:49] <cachio_> once the image is released
[16:49] <cachio_> I start preinstalling
[16:49] <cachio_> all the deps
[16:49] <cachio_> because this image changes so frequently
[16:49] <ijohnson> makes sense
[16:50] <cachio_> we need to add that dep in the test which needs it
[16:50] <ijohnson> cachio_: the whole suite is failing to be prepared, it needs to always be in the image
[16:50] <cachio_> or better in pkgdb helper
[16:50] <ijohnson> it's not a specific test that is failing due to missing the go command, it's every test on hirsute right now
[16:51] <cachio_> ijohnson, is it in you pr or in master?
[16:51] <ijohnson> cachio_: it's in this pr: https://github.com/snapcore/snapd/pull/9307
[16:51] <mup> PR #9307: interfaces/tee: add TEE/OPTEE interface <Squash-merge> <Created by ogra1> <https://github.com/snapcore/snapd/pull/9307>
[16:51] <ijohnson> cachio_: but maybe that pr needs a master merge
[16:52] <cachio_> ijohnson, you can try merging master
[16:52] <cachio_> or add the dep here https://github.com/snapcore/snapd/blob/master/tests/lib/pkgdb.sh#L597
[16:53] <cachio_> ijohnson, ah, the pr is not yours
[16:53] <cachio_> I can try in that case
[16:54] <ijohnson> right
[16:57] <pedronis> that PR can be probably landed anyway, now is down to 2-3 failures
[16:57] <pedronis> but the missing go confused me (it's that kind of day)
[16:57] <cachio_> pedronis, ijohnson so the go problem is for all the prs
[16:57] <cachio_> I'll create a new pr to fix that
[16:58] <ijohnson> cachio_: right we should fix that asap, hirsute is close to being stable so it would be good to catch regressions there before hirsute is released
[16:58] <ijohnson> cachio_: it's still on my queue to work on the security uevent test fix for hirsute, I should have something today or early next week
[16:58] <cachio_> ijohnson, nice
[16:59] <cachio_> that test is the only one in red
[16:59] <ijohnson> great
[17:01] <cachio_> ijohnson, which packages do we need for hirsuite?
[17:01] <ijohnson> cachio_: just go I think
[17:04] <cachio_> ok, testing it
[17:13] <cachio_> ijohnson, still failing
[17:14] <cachio_> it is trying to install go 1.10
[17:14] <mup> PR snapd#9307 closed: interfaces/tee: add TEE/OPTEE interface <Squash-merge> <Created by ogra1> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9307>
[17:15] <cachio_> but it doesn't exist
[17:15] <cachio_> for hirsute
[17:16] <cachio_> ijohnson, distro_install_package --no-install-recommends autoconf automake autotools-dev build-essential clang curl devscripts expect gdb gdebi-core git indent jq apparmor-utils libapparmor-dev libglib2.0-dev libseccomp-dev libudev-dev man netcat-openbsd pkg-config python3-docutils udev udisks2 upower uuid-runtime avahi-daemon cups dbus-x11 fontconfig gnome-keyring jq man nfs-kernel-server printer-driver-cups-pdf python3-dbus python3-gi python3-yaml
[17:16] <cachio_> upower weston xdg-user-dirs xdg-utils zsh dbus-user-session golang qemu-utils
[17:16] <cachio_> Reading package lists...
[17:16] <cachio_> Building dependency tree...
[17:16] <cachio_> Reading state information...
[17:16] <cachio_> systemd is already the newest version (247.3-1ubuntu4).
[17:16] <cachio_> The following package was automatically installed and is no longer required:
[17:16] <cachio_>   squashfs-tools
[17:16] <cachio_> Use 'sudo apt autoremove' to remove it.
[17:16] <cachio_> 0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
[17:16] <cachio_> + '[' '' = maybe ']'
[17:16] <cachio_> + loginctl enable-linger test
[17:16] <cachio_> + loginctl disable-linger test
[17:16] <cachio_> + '[' '' = yes ']'
[17:16] <cachio_> + case "$SPREAD_SYSTEM" in
[17:17] <cachio_> + [[ ubuntu-21.04-64 == debian-9-* ]]
[17:17] <cachio_> + best_golang=golang-1.10
[17:17] <cachio_> + quiet xargs -r eatmydata apt-get install -y
[17:17] <cachio_> + gdebi --quiet --apt-line ./debian/control
[17:17] <cachio_> This package is uninstallable
[17:17] <cachio_> Dependency is not satisfiable: dh-systemd
[17:17] <cachio_> ++ command -v go
[17:17] <cachio_> + '[' -z /usr/bin/go ']'
[17:17] <cachio_> ++ command -v govendor
[17:17] <cachio_> + '[' -z '' ']'
[17:17] <cachio_> + rm -rf /home/gopath/src/github.com/kardianos/govendor
[17:17] <cachio_> + go get -u github.com/kardianos/govendor
[17:17] <cachio_> ++ seq 10
[17:17] <cachio_> + for _ in $(seq 10)
[17:17] <cachio_> + quiet govendor sync
[17:17] <cachio_> + break
[17:17] <cachio_> + chown test.test -R /home/gopath/src/github.com/snapcore/snapd
[17:17] <cachio_> + '[' -z '' ']'
[17:17] <cachio_> + case "$SPREAD_SYSTEM" in
[17:17] <cachio_> + build_deb
[17:17] <cachio_> ++ dpkg-parsechangelog --show-field Version
[17:17] <cachio_> + dch --newversion 1337.2.49.1 'testing build'
[17:17] <cachio_> dch warning: neither DEBEMAIL nor EMAIL environment variable is set
[17:17] <cachio_> dch warning: building email address from username and FQDN
[17:17] <cachio_> dch: Did you see those 2 warnings?  Press RETURN to continue...
[17:17] <cachio_> + os.query is-debian-sid
[17:17] <cachio_> + su -l -c 'cd /home/gopath/src/github.com/snapcore/snapd && DEB_BUILD_OPTIONS='\''nocheck testkeys'\'' dpkg-buildpackage -tc -b -Zgzip' test
[17:18] <cachio_> dpkg-buildpackage: info: source package snapd
[17:18] <cachio_> dpkg-buildpackage: info: source version 1337.2.49.1
[17:18] <cachio_> dpkg-buildpackage: info: source distribution UNRELEASED
[17:18] <cachio_> dpkg-buildpackage: info: source changed by root <root@mar191704-731217.c.computeengine.internal>
[17:18] <cachio_>  dpkg-source -Zgzip --before-build .
[17:18] <cachio_> dpkg-buildpackage: info: host architecture amd64
[17:18] <cachio_> dpkg-source: info: using options from snapd/debian/source/options: --tar-ignore=tests/lib/cache/*
[17:18] <cachio_> dpkg-checkbuilddeps: error: Unmet build dependencies: debhelper (>= 9) dh-apparmor dh-autoreconf dh-golang (>= 1.7) dh-systemd gcc-multilib gettext gnupg2 libcap-dev libfuse-dev liblzma-dev liblzo2-dev python3-markdown xfslibs-dev
[17:18] <cachio_> dpkg-buildpackage: warning: build dependencies/conflicts unsatisfied; aborting
[17:18] <cachio_> dpkg-buildpackage: warning: (Use -d flag to override.)
[17:18] <cachio_> ijohnson, sorry
[17:18] <cachio_> https://paste.ubuntu.com/p/sYYXhJvy6r/
[17:22] <ijohnson|lunch> cachio_: hmm what's the problem?
[17:22] <ijohnson|lunch> Also /me is at lunch, bbiab
[17:36] <cachio_> ijohnson|lunch, created the pr $10057
[17:37] <cachio_> but the problem not is what I showed https://paste.ubuntu.com/p/sYYXhJvy6r/
[17:40] <mup> PR snapd#10057 opened: tests: adding golang packages as dependency for hirsuite <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10057>
[17:47] <cachio_> ijohnson|lunch, I am comparing with the previous image to determine what changed in terms of dependencies
[17:56] <mup> PR snapcraft#3477 closed: spread tests: pin go for v1 plugin snaps <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3477>
[18:11] <mup> PR snapcraft#3475 closed: godeps: set default channel to 1.15/stable <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3475>
[18:11] <mup> PR snapcraft#3476 closed: ci: reduce amount of artifacts to upload for spread <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3476>
[18:17] <cachio_> ijohnson|lunch, I used an old image and the errror is the same
[18:17] <cachio_> then I used old code and the error is the same
[18:17] <cachio_> it meand the repo has changed
[19:02] <ijohnson> cachio_: I'm looking at your pr now
[19:04] <ijohnson> cachio_: yeah now the issue is different from that other pr
[19:04] <ijohnson> dpkg-checkbuilddeps: error: Unmet build dependencies: debhelper (>= 9) dh-apparmor dh-autoreconf dh-golang (>= 1.7) dh-systemd gcc-multilib gettext gnupg2 libcap-dev libfuse-dev liblzma-dev liblzo2-dev python3-markdown xfslibs-dev
[19:05] <ijohnson> up above from there a bit is this same error though
[19:05] <ijohnson> + quiet xargs -r eatmydata apt-get install -y
[19:05] <ijohnson> + gdebi --quiet --apt-line ./debian/control
[19:05] <ijohnson> This package is uninstallable
[19:05] <ijohnson> Dependency is not satisfiable: dh-systemd
[19:06] <ijohnson> hmmm debian packaging is not my strong suit so I'm a bit lost, but I think there's some issue snapd depending on dh-systemd in hirsute
[19:06] <cjwatson> Needs to be something like "debhelper (>= 9.20160709~) | dh-systemd".  dh-systemd is gone in hirsute.
[19:06] <cjwatson> If you use something like that instead then it should work on both old and new systems.
[19:07] <cjwatson> (dh-systemd was folded into debhelper)
[19:07] <ijohnson> I could have sworn I saw mvo file a pr doing that
[19:07] <cjwatson> I certainly suggested that
[19:07] <cjwatson> I guess it must not have landed yet
[19:07] <ijohnson> ahhh I even approved it, but it didn't land yet
[19:07] <ijohnson> https://github.com/snapcore/snapd/pull/10044
[19:07] <mup> PR #10044: packaging: drop dh-systemd from build-depends on ubuntu-16.04+ <Created by mvo5> <https://github.com/snapcore/snapd/pull/10044>
[19:08] <ijohnson> well "approved"
[19:08] <cjwatson> Oh
[19:08] <cjwatson> It's also wrong
[19:08] <cjwatson> Needs to replace the dh-systemd line, not supplement it
[19:08] <ijohnson> ah good to know
[19:08] <ijohnson> I can push to mvo's branch
[19:09]  * cjwatson comments on the PR to that effect
[19:10] <ijohnson> thanks cjwatson
[19:10] <ijohnson> cachio_: so we need both your pr adding the go package to the spread image _and_ a fixed version of mvo's branch too
[19:11] <ijohnson> cachio_: do you mind if I just cherry-pick your branch on top of the fix I will push to mvo's pr ?
[19:11] <cachio_> ijohnson, nice
[19:11] <cachio_> go ahead
[19:11] <cachio_> no problem
[19:13] <ijohnson> ack
[19:15] <mup> PR snapd#10057 closed: tests: adding golang packages as dependency for hirsuite <Created by sergiocazzolato> <Closed by anonymouse64> <https://github.com/snapcore/snapd/pull/10057>
[19:39]  * cachio_ afk
[23:11] <mup> PR snapcraft#3478 opened: ci: add requirements for snapcraft legacy in spread <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3478>