[02:33] PR snapd#9830 opened: Clarify error message [06:18] morning [08:16] morning [08:19] good morning pstolowski [08:33] pstolowski: mvo : hey [08:33] good morning mborzecki [08:34] pstolowski: can you take a look at https://github.com/snapcore/snapd/pull/9828 ? [08:34] PR #9828: sandbox/cgroup, tess/main: fix unit tests on v2 system, disable broken tests on sid [08:36] hey guys [08:37] I've sent some more patches for spread and I will continue until the CI systems fixed [08:37] good morning zyga [08:37] hey mvo, how are things? [08:37] +1 [08:37] zyga: good, good, how are you? [08:37] hey zyga [08:38] * zyga starts first scrum-like cycle today, so some extra planning and non-coding work [08:39] heh, we keep on getting 429 Too Many Requests when tryting to pull the commands list from the store [08:40] and tests/main/snap-advise-command and apt-hooks keep failing because of that [08:43] hmm https://github.com/snapcore/snapd/pull/9830 looks off, or am i missing something? [08:43] PR #9830: Clarify error message [09:20] mvo: can we land https://github.com/snapcore/snapd/pull/9828 ? [09:20] PR #9828: sandbox/cgroup, tess/main: fix unit tests on v2 system, disable broken tests on sid [09:20] the failures are unrelated [09:26] mborzecki: sure [09:26] mvo: thanks! [09:27] mborzecki, it's really happening :) [09:27] zyga: hm? [09:27] what is happening? [09:27] v2 support [09:27] once debian gets it [09:28] ubuntu will have it sooner or later [09:29] zyga: rbalint wanted to enable it in 21.x, but we're still not ready in snapd [09:30] zyga: see https://bugs.launchpad.net/ubuntu/+source/lxd/+bug/1850667/ [09:30] Bug #1850667: cgroup v2 is not fully supported yet, proceeding with partial confinement [09:30] PR snapd#9828 closed: sandbox/cgroup, tess/main: fix unit tests on v2 system, disable broken tests on sid [09:55] PR snapd#9803 closed: many: clarify gadget role-usage consistency checks for UC16/18 vs UC20 [10:03] niemeyer, hello [10:03] niemeyer, I've sent a small PR to unbreak CI in spread: https://github.com/snapcore/spread/pull/112 [10:03] PR spread#112: Bump base version of go to 1.13 [10:03] niemeyer, and a small fix for one of the issues picked up by go vet: https://github.com/snapcore/spread/pull/111 [10:03] PR spread#111: Do not copy log.Logger and the contained sync.Mutex [10:04] zyga: Thanks! [10:04] niemeyer, I have one more for more go vet issues in humbox but I need to spend some more time on it [11:01] mborzecki: do you think you could take a look at #9732? [11:01] PR #9732: asserts: snapasserts method to validate installed snaps against validation sets [11:02] pstolowski: sure [11:02] ty! [11:19] should we disable snap-advise-command test temporarily? [11:48] pstolowski: idk, pedronis do you know more about store side of things? is this temporary? [11:48] pedronis: we're seeing 429 accessing /api/v1/snaps/names [11:51] PR snapd#9457 closed: [RFC] o/snapshotstate: test for roundtrip with corrupted zip file inside exported snapshot [11:53] mborzecki, pedronis: i can also add an extra check in the test not to fail on 429 [11:53] (just warn and exit) [11:56] PR snapd#9831 opened: interfaces/builtin: fix unit test expecting snap-device-helper at /usr/lib/snapd [11:57] i asked in the store channel [12:26] PR snapd#9833 opened: tests: skip main part of snap-advise test if 429 error is encountered [13:06] * pstolowski lunch [13:41] PR snapd#9834 opened: tests: normalize test tools - part 1 [14:19] PR snapcraft#3413 opened: elf: add a class to check for dependency problems in a group of files [14:43] pedronis: we ought to talk about validations sets, i wasn't pushing for it this morning because i still had stuff to do, but maybe later/tomorrow morning? [14:44] pstolowski: yes, sorry [15:09] cachio: updated #9833 [15:09] PR #9833: tests: skip main part of snap-advise test if 429 error is encountered [15:10] pstolowski, thanks [15:16] pstolowski, +1 [15:17] ty [15:17] pstolowski, apt-hooks test is havin same problem [15:18] I can replicate the solution for that as well [15:19] cachio: ah [15:19] cachio: ok, please do, thanks [15:19] pstolowski, sure [15:20] cachio: although - fingers crossed - it shouldn't be a problem for now after they changed store config [15:20] pstolowski, yes, but I just re run tests and failed with that error [15:20] re-ran [15:21] ogra: hey if you didn't already could you file a bug about snapctl needing CAP_NET_ADMIN ? [15:21] cachio: oh, so still an issue, not good [15:22] lets wait a bit [15:23] ijohnson, eventually ... 🙂 [15:23] k, thanks [15:23] * cachio lunch [15:57] * ogra curses ... [15:57] so wh the heck does the python snapcraft plugin break in massive ways with each new base i need to port to [16:03] :-( what's the problem ogra [16:03] cjp256, https://github.com/ogra1/nodered-snap/blob/master/snap/snapcraft.yaml#L130 [16:03] i'm trying to move that somehow to core20 [16:04] i'm at a point where, while getting a lot bdist_wheel errors at the start, the pip install seems to at least finish on the second run [16:05] (if i take oout the Adafruit DHT stuff that needs special treatment before building) [16:05] python2 ? [16:05] yeah [16:06] well i see a potential bug in python plugin for python2.. [16:06] as far as wheel, you can install that python package (it's no longer automatically installed) [16:07] yeah, i gathered that from the forum ... but i still get the bdist_wheel errors on first try ... it re-trie then and succeeds [16:07] *re-tries [16:13] ogra: i'm not sure what you are seeing in your log, but the wheel errors generally can be ignored, python should still chug along without the wheels [16:13] okay ... [16:14] hmm there is no pip anymore in 20.04 it seems [16:14] (for py2) [16:18] * ogra doesnt see how he can ever get that to work [16:21] cjp256, so what i'm seeing when using py3 is https://paste.ubuntu.com/p/nHHsT5Kq4s/ ... but i doubt the modules will work since they are all written for py2 (and are unlikely to get ported) [16:22] when setting "SNAPCRAFT_PYTHON_INTERPRETER: "python2"" it just falls flat on its face telling me it doesnt have venv [16:24] yeah, the core20 plugin really expects python3 [16:25] PR snapd#9835 opened: tests: apply workaround done for snap-advise-command to apt-hooks test [16:32] mvo: can you merge #9833 manually, there are a couple of unrelated failures [16:32] PR #9833: tests: skip main part of snap-advise test if 429 error is encountered [16:32] ogra: i think a custom override is in order for this, want me to help you create one? might be worth creating a python2 plugin [16:33] cjp256, well, there is no venv in the archive at all ... how would i get that ? [16:33] orgra: virtualenv [16:33] pstolowski: sure [16:33] ogra: `python2 -m pip install virtual` if you used pip [16:33] cjp256, oh, that works for py2 still ? [16:33] err virtualenv^ [16:34] yeah seems to for me [16:34] mvo: ty! [16:34] cachio: you may want to update your PR with master now [16:34] ogra: you use the get-pip.py to install pip? [16:35] PR snapd#9833 closed: tests: skip main part of snap-advise test if 429 error is encountered [16:35] not yet, wheer do i find that ? [16:35] *where [16:36] `curl https://bootstrap.pypa.io/get-pip.py --output get-pip.py ; python2 get-pip.py` [16:38] that gets me pip, yeah ... [16:38] but then ... [16:38] + python2 -m venv /root/nodered-snap/parts/sensor-libs/install [16:38] Failed to build 'sensor-libs'. [16:38] grrr [16:38] /usr/bin/python2: No module named venv [16:41] ogra: use nil plugin for now [16:43] i'll ping sergio about adding an equivalent python2 plugin [16:43] heh ... symlinking venv to virtualenv in dist-packages helps [16:43] hah yeah, that'd do it :P [16:44] so where are the python packages stored now ... i need to modify one of them before building [16:45] (there used to be a "python-packages" dir) [16:46] should be in $SNAPCRAFT_PART_INSTALL if that's where you installed the virtualenv [16:46] something like $SNAPCRAFT_PART_INSTALL/lib/python2.7/site-packages/... [16:48] well, i need the tarball ... before building anything ... in core18 this was downloaded during the pull step [16:55] oh my ... [16:55] it goes to /root/.cache ?!? [16:55] ah, no, thats the result ... [17:05] PR snapd#9836 opened: cmd/snap-repair: save base snap and mode in device info; other misc cleanups [17:20] PR snapcraft#3414 opened: project loader: export SNAPCRAFT_TARGET_ARCH in build environment [17:25] PR snapcraft#3415 opened: cli: enable SNAPCRAFT_TARGET_ARCH envvar matching --target-arch [17:50] PR snapcraft#3416 opened: repo: default to target arch for stage package cache [18:45] PR snapcraft#3409 closed: Fix a few licenses in ros-related test files [18:55] PR snapcraft#3412 closed: grammar: ensure all dictionary primitives are captured [19:00] PR snapcraft#3343 closed: build providers: set snapd proxy settings prior to _setup_snapcraft() [19:40] PR snapcraft#3414 closed: project loader: export SNAPCRAFT_TARGET_ARCH in build environment [19:55] PR snapcraft#3371 closed: snap: get legacy branch from local [20:00] PR snapcraft#3198 closed: snap: drop legacy, switch to core20 for bootstrap [20:00] PR snapcraft#3407 closed: Add new gnome-3-38 extension which uses gnome-3-38-2004 and core20 [20:35] PR snapcraft#3417 opened: Allow validation assertions to be revoked [21:31] PR snapd#9837 opened: cmd/snap-repair: filter repair assertions based on bases + modes [21:55] PR snapcraft#3418 opened: project: always set target arch even if not cross compiling [22:29] Bug #1911480 opened: cgroupv2 support [22:59] Bug #1911480 changed: cgroupv2 support