[06:31] morning === pstolowski|afk is now known as pstolowski [08:01] mornings [08:02] pstolowski: hey [08:03] pedronis: hey, i just noticed you created a 20.4 card for snapctl but there is already one in Doing, and more detailed [08:03] pstolowski: ? [08:03] pstolowski: I just moved the theme card [08:03] which doesn't go into doing [08:03] pedronis: ah, sorry, got confused [08:04] pedronis: morning [08:23] mvo: hi, what's the status of https://trello.com/c/AGmzhqA7/288-make-netplan-apply-work ? only one item is unchecked [08:24] pedronis: looking [08:24] pedronis: I think this is done now, let me double check [08:24] pedronis: I double check and move to done if all good? [08:24] sounds good [08:38] PR snapd#7757 closed: devicestate: rename ensureSeedYaml -> ensureSeeded [08:46] btw. standup is at 4pm today? [08:47] mborzecki: on my calendar it's at the regular time [08:48] hm maybe evolution is just confused, it's showing 4pm [08:48] oh and gnome-calendar is showing 3pm [09:53] xnox: please holler when you are around, would love to move forward with the snap-bootstrap intramfs-mounts and wanted to double check if you are happy with the PR 7746 [09:53] PR #7746: snap-bootstrap: implement "snap-bootstrap initramfs-mounts" [09:53] mvo: sure. and the matching core20 build i guess. [09:54] mvo: do you want to chat in a hangout, or simply for me to review the snap-bootstrap tool? [09:54] imho i'm not sure how to "include" it into the initrd right now =) [09:55] xnox: either way is fine, HO, irc or just looking at it. HO might be easiest, give me 2min (we may want to pull in pedronis is he wants to join) [09:56] let's hangout [09:56] PR snapd#7763 opened: [RFC] boot,bootloader: make MakeBootable() uc20 aware [09:56] your git repo has too many branches! [09:57] xnox: yes :( [09:57] xnox: but all my stuff is proposed for merging so shouldn't be super terrible [09:58] mvo: something is strange about needing a recovery grub [09:58] they are both grub [09:59] super interesting use case: https://forum.snapcraft.io/t/offline-update-of-multiple-snaps/14258 [09:59] pedronis: yes, I can try to simply make it a config option [10:00] pedronis: happy to explore this [10:00] also we have options that we pass to Find now, that might help as well [10:06] pedronis: I will explore using the same a bit and push that as well [10:20] mvo: as I suspected adding snapd type in asserts is not enough, there are bugs, trying to write tests in seed/ and fix them [10:50] mvo: can you take a look at https://github.com/snapcore/snapd/pull/7727 when you have a moment? [10:50] PR #7727: tests: improve TestDoPrereqRetryWhenBaseInFlight to fix occasional flakiness [10:59] good morning :/ [10:59] 5AM managed to sleep [10:59] 11AM wake-up [11:00] mvo: do you remember when the improved snapfuse binary was shipped? [11:00] hey mborzecki [11:01] zyga: hey [11:01] PR snapd#7764 opened: many: test various kinds of overriding for the snapd snap in Core 20 [11:01] mvo: ^ [11:13] Bug #1560942 changed: Support swap [11:15] sil2100: hiya. Are you the person to ask about details of the rpi gadget? [11:20] sil2100: what is it about? [11:24] sdhd-sascha: ? [11:28] chipaca: sorry, i want you to ask about what question it is? [11:28] sdhd-sascha: about the pi3-gadget, https://github.com/snapcore/pi3-gadget [11:28] sdhd-sascha: why? [11:29] because I _think_ #1500164 is fix-released, but I'm not the expert [11:29] Bug #1500164: console= cmdline options on the RPi2 should be moved from uboot.env to cmdline.txt [11:29] Chipaca, it is [11:29] ogra: you said you thought so, but that we should check with foundations :) [11:29] ogra: so here i am trying to do that [11:30] well, i cheked the code but yeah, a re-assurance from foundations would be bad ... though you can always re-open the bug anyway :P [11:30] *woudlnt [11:30] bah [11:31] * ogra subscribes to a typing course [11:31] ogra: in my experience subtle bugs like this very rarely get re-opened [11:31] we're lucky they get filed in the first place :D [11:31] i'm pretty sure it is fix-released (90%+) [11:31] ogra: OTOH the reporter of this bug in particular is a crotchety ol' guy that'll open it again without a doubt [11:31] haha [11:32] and, that's all the triage i'll be doing today, methinks [11:32] now to attempt a run [11:32] * Chipaca goes [11:33] mvo: do you have a moment [11:33] mvo: can we chat about the thing from yesterday [11:33] pstolowski: hey, sure thing! [11:33] mvo: we've discussed the technical side with maciek just now and wanted to give you some options [11:33] mvo: we're in https://meet.google.com/nst-hqvw-bhb?authuser=1 [11:34] zyga: uh, nasty jetlag this time :( [11:34] Bug #1500164 changed: console= cmdline options on the RPi2 should be moved from uboot.env to cmdline.txt [11:34] mvo: tell me about it ;D [11:34] zyga: the improved snapdfuse is in 2.41+ but I think you need to install the deb to get the benefits. I *might* be wrong though [11:34] zyga: improved deb is in -proposed [11:34] mvo: thanks for the review. I'm addressing the issues and will push again soon [11:35] cmatsuoka: \o/ === ricab is now known as ricab|lunch [12:58] tests running [13:34] PR snapd#7765 opened: boot,bootloader: make MakeBootable() uc20 aware [13:39] PR snapd#7763 closed: [RFC] boot,bootloader: make MakeBootable() uc20 aware [13:49] mvo, what happened to http proxy settings in core18 ? (/etc/environment is not writable anymore ?!?) [13:49] ogra@pi4:~$ snap set system proxy.http="http://my.proxy.com:8080" [13:49] ogra@pi4:~$ grep my.proxy /etc/environment [13:49] ogra@pi4:~$ sudo touch /etc/environment [13:49] touch: cannot touch '/etc/environment': Read-only file system [13:49] ogra@pi4:~$ [13:49] or zyga ^^^ [13:51] ogra: I'm sorry, I'm not very reactive today [13:52] well, you cant set a http/https proxy in core18 at all it seems ... dont we have tests for that ? or did the implementation change [13:52] looking at https://github.com/snapcore/snapd/blob/master/overlord/configstate/configcore/proxy.go it still writes /etc/environment ... but i dont get any error when running snap set [13:53] ogra@pi4:~$ snap change 1384 [13:53] Status Spawn Ready Summary [13:53] Done today at 13:43 UTC today at 13:43 UTC Run configure hook of "core" snap [13:54] not really telling me about the fact that it didnt set the proxy [13:55] I don't know [13:55] nothing in the journal either [13:55] ogra@pi4:~$ sudo journalctl|grep proxy [13:55] ogra@pi4:~$ [13:56] sorry [13:56] well, thats pretty gross we have a few customers relying on that working as documented [13:56] is /etc/environment a squash [13:56] sorry, I'm not feeling very well [13:56] it wasnt in core16 it seems to be in core18 [13:56] then it's a bug [13:56] yes [13:56] needs to be addressed, regression tests etc [13:56] just wondering how that can happen [13:56] perhaps existing test can be reused if there is onew [13:56] we don't run many tests on core18 [13:56] ogra: let me have a look (but we have a meeting in 3min so I maybe a bit slow) [13:56] k [13:57] zyga: I beg to differ, we run almost all tests on core18 [13:57] zyga, just leave it to michael :) [13:57] mvo: if that's the case I'm sorry, I wasn't aware of that [13:57] mvo: what I meant is that may of our tests don't use base: core18 [13:57] though for this problem it should not matter [13:58] zyga: aha, I see [13:58] zyga: yeah, that's true. but I think ogra issues is on a core18 based core system, yes? [13:59] mvo, i think so ... i'll ask the customer that complained, they only provided snap list which indeed shows core and core18 installed [14:00] ogra: I found the relevant test, running this now on core18 [14:00] but i think they run on core18 (it also works fine on core16, i just checked) [14:00] seems the internal snapd setting of the proxy to contact the store is fine ... but not the global change to /etc/environment [14:01] (simply because it got dropped from writable-paths i guess) [14:03] ogra@stream:~$ snap set system proxy.http="http://my.proxy:8080" [14:03] ogra@stream:~$ sudo grep proxy /etc/environment [14:03] http_proxy=http://my.proxy:8080 [14:03] core16 works fine [14:09] ogra: yeah, it's anyoing, we have a spread test that tests the internal setting on core18 but I think not the other one [14:09] mvo, well, if the config hook would at least spill an error it would have been easier to catch :) [14:10] but even that goes quietly through [14:10] PR snapd#7756 closed: tests: update mount-ns test for bionic image changes [14:10] ogra: yeah, no discussion that this is bad(tm) [14:10] (unless i'm seriously missing something and we dont set systemwide proxy settings in /etc/environment anymore) [14:13] ogra: running tests now (and in a meeting) - I extended the proxy test and then I should be able to say more === ricab|lunch is now known as ricab [14:32] meet kicked me out and refuses to connect, now [14:33] it does that a lot :/ [14:34] * Chipaca curses the google goblins [14:40] Chipaca: oopsy, we wanted to stay a bit and sort out confusion with mvo, anyway we can chat on Monday [14:40] Chipaca: to answer your question from the HO chat, no I have not tested any of this on spinning disk systems [14:40] pedronis: i can't access any bit of google right now [14:40] Chipaca: do you think I should? [14:41] ijohnson: I think it's a lot more painful there, and might be painful in different ways [14:41] Chipaca: oops, that seems bad/strange [14:41] ijohnson: is that something that's easy for you to do? [14:41] pedronis: now it's back, fwiw [14:41] Chipaca: ok I can try to, I have empty spinning disks I could use but I don't have an OS installed on them, so I would have to install a system there [14:42] ijohnson: I've got 19.10 on a spinning-disk laptop, if that's any use [14:43] pedronis: https://forum.snapcraft.io/t/offline-update-of-multiple-snaps/14258 is the topic i mentioned in the standup [14:43] hmm, not sure if 19.10 vs 19.04 is important, I'm still on disco and when I first started all of this mvo, pedronis and I decided I shouldn't upgrade to 19.10 to run more tests because it could be radically different for graphics and things [14:43] ijohnson: 1904, or 1804? [14:43] Chipaca: don't worry about it, I can handle it, you should prepare for your conference :-) (and have fun too) [14:43] 1904 [14:44] ijohnson: you typoed "panic" [14:44] :-) [14:44] * zyga breaks for coffee [14:44] brb [14:44] ijohnson: thanks! [14:44] anyway, i'm off [14:44] ttfn! [14:44] ogra: https://github.com/snapcore/core18/pull/143 [14:44] PR core18#143: static: add /etc/environment to writable-paths [14:45] PR core18#143 opened: static: add /etc/environment to writable-paths [14:45] ogra: also test missing on our side, working on this too [14:53] hmm snapd calling systemctl stop snapd.failure.service when snapd.failure.service actually stated snapd in recovery mode :/ [14:55] * ogra hugs mvo [14:58] ogra: was working on this and then another fire came - but I certainly will come back to it. also a silly bug in our code that did not report this as an error as it should have :( [14:58] ogra: with the fix the configure hook fails and says "/etc/environment: read-only fs" [14:59] ogra: but yeah, there should be a PR (or two) for this by my eod [15:00] mvo, thanks a lot ! [15:30] ijohnson, was it you who complained about slow forum ? i see it now too ... i have seen more of the spiner than of the content today [15:30] *spinner [16:07] ogra: yes and others were discussing the issue in our standup today too [16:07] seems a global problem [16:10] Dinner [16:12] yeah [16:13] i have some feeling it is related to the avatars ... i often see the page but the avatars only load a lot later ... same goes for quotes, that often shows "Loading ..." (i actually didnt know it could do that !!) [16:33] ogra: 7766 and 7767 address your issue, thanks for reporting [16:33] PR snapd#7767 opened: tests: run snap-set-core-config on all core devices [16:34] * cachio lunch [16:35] * ogra hugs mvo once more === pstolowski is now known as pstolowski|afk [17:38] mvo: hey [17:38] remember your recent PRs? [17:38] error: cannot perform the following tasks: [17:38] - Run configure hook of "core" snap (run hook "configure": open /etc/environment: read-only file system) [17:39] mvo: this requires the new core18 release, right? [17:40] sil2100: https://github.com/snapcore/core18/pull/143 an you please do a 2nd review? [17:40] PR core18#143: static: add /etc/environment to writable-paths [17:49] zyga: o/ [17:50] hey [17:50] good evening [17:50] sorry to poke you so late [17:50] not urgent but would help to unblock this for tomorrow [17:50] thanks a lot :) [17:51] enjoy your evening [17:51] PR core18#143 closed: static: add /etc/environment to writable-paths [17:56] sil2100: I cannot find the build recipe for core18, can you please trigger a build? [18:06] zyga: sure o/ Will need to sync the mirror first [18:06] I mean [18:06] The builds auto-trigger on new code [18:06] sure, anything it takes to release new core18 to edge [18:06] (and other stuff) === ijohnson is now known as ijohnson|lunch [18:20] zyga: yes, that requires the latest core18 [18:20] zyga: I thought I did mention it in the PR but maybe I forgot [18:21] PR snapcraft#2816 opened: assorted: address errors from mypy uprev [18:27] mvo: it's all right :) [18:27] mvo: I made solution B work, I think, it's not terrible actually [18:28] mvo: I'll grab a coffee (heh) and sit tight [18:28] mvo: I'll prepare a few PRs for tomorrow for maciek [18:28] brb [18:30] mvo: lol [18:30] mvo: check out this snippet [18:30] (backstory: iOS 13 was more buggy than older releases) [18:30] Software chief Craig Federighi and lieutenants including Stacey Lysik announced the changes at a recent internal "kickoff" meeting with the company's software developers. The new approach calls for Apple's development teams to ensure that test versions, known as "daily builds," of future software updates disable unfinished or buggy features by default. Testers will then have the option to selectively enable those features, [18:30] via a new internal process and settings menu dubbed Flags, allowing them to isolate the impact of each individual addition on the system. When the company's iOS 13 was released alongside the iPhone 11 in September, iPhone owners and app developers were confronted with a litany of software glitches. [18:30] feature flags FTW :) [18:37] zyga: thanks! let's see if we can get something into beta tomorrow [18:37] zyga: haha, yes [18:54] * ogra is impressed by kyrofa's accent free russian ! [18:54] ogra, right?! [18:54] You don't KNOW my skills [18:54] :D [18:55] I miss you buddy. You gonna be in Frankfurt I hope? [18:55] yeah, i actually will !!!! [18:56] Awesome [18:56] looking forward to it [20:28] meh, there are still 11 unit tests to adjust [20:28] and god knows how many comments [20:28] but it passes all regression tests [20:28] that's good [20:33] PR snapd#7766 closed: configcore: fix missing error propagation [20:57] * cachio afk === arnatious_ is now known as arnatious [22:13] PR snapd#7768 opened: interfaces: add raw-volume interface for access to partitions [22:17] ondra: fyi, ^ [22:23] Does anyone know the snap analogue to something like 'virsh -c qemu:///system version --daemon'? Basically to query the libs and versions used inside snap core overlay itself? [22:30] you mean the libs in the core snap ? just take a look in /snap/core/current (or respectively in the core18 dir for snaps using a core18 base) [23:00] * zyga does midnight supper === ijohnson|lunch is now known as ijohnson