/srv/irclogs.ubuntu.com/2019/11/21/#snappy.txt

mborzeckimorning06:31
=== pstolowski|afk is now known as pstolowski
pstolowskimornings08:01
mborzeckipstolowski: hey08:02
pstolowskipedronis: hey, i just noticed you created a 20.4 card for snapctl but there is already one in Doing, and more detailed08:03
pedronispstolowski: ?08:03
pedronispstolowski: I just moved the theme card08:03
pedroniswhich doesn't go into doing08:03
pstolowskipedronis: ah, sorry, got confused08:03
mborzeckipedronis: morning08:04
pedronismvo: hi, what's the status of https://trello.com/c/AGmzhqA7/288-make-netplan-apply-work ? only one item is unchecked08:23
mvopedronis: looking08:24
mvopedronis: I think this is done now, let me double check08:24
mvopedronis: I double check and move to done if all good?08:24
pedronissounds good08:24
mupPR snapd#7757 closed: devicestate: rename ensureSeedYaml -> ensureSeeded <Simple 😃> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7757>08:38
mborzeckibtw. standup is at 4pm today?08:46
mvomborzecki: on my calendar it's at the regular time08:47
mborzeckihm maybe evolution is just confused, it's showing 4pm08:48
mborzeckioh and gnome-calendar is showing 3pm08:48
mvoxnox: 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 774609:53
mupPR #7746: snap-bootstrap: implement "snap-bootstrap initramfs-mounts" <Needs Samuele review> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7746>09:53
xnoxmvo:  sure. and the matching core20 build i guess.09:53
xnoxmvo:  do you want to chat in a hangout, or simply for me to review the snap-bootstrap tool?09:54
xnoximho i'm not sure how to "include" it into the initrd right now =)09:54
mvoxnox: 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:55
xnoxlet's hangout09:56
mupPR snapd#7763 opened: [RFC] boot,bootloader: make MakeBootable() uc20 aware <Created by mvo5> <https://github.com/snapcore/snapd/pull/7763>09:56
xnoxyour git repo has too many branches!09:56
mvoxnox: yes :(09:57
mvoxnox: but all my stuff is proposed for merging so shouldn't be super terrible09:57
pedronismvo: something is strange about needing a recovery grub09:58
pedronisthey are both grub09:58
Chipacasuper interesting use case: https://forum.snapcraft.io/t/offline-update-of-multiple-snaps/1425809:59
mvopedronis: yes, I can try to simply make it a config option09:59
mvopedronis: happy to explore this10:00
pedronisalso we have options that we pass to Find now, that might help as well10:00
mvopedronis: I will explore using the same a bit and push that as well10:06
pedronismvo: as I suspected adding snapd type in asserts is not enough, there are bugs, trying to write tests in seed/ and fix them10:20
pstolowskimvo: can you take a look at https://github.com/snapcore/snapd/pull/7727 when you have a moment?10:50
mupPR #7727: tests: improve TestDoPrereqRetryWhenBaseInFlight to fix occasional flakiness <Test Robustness> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7727>10:50
zygagood morning :/10:59
zyga5AM managed to sleep10:59
zyga11AM wake-up10:59
zygamvo: do you remember when the improved snapfuse binary was shipped?11:00
zygahey mborzecki11:00
mborzeckizyga: hey11:01
mupPR snapd#7764 opened: many: test various kinds of overriding for the snapd snap in Core 20 <Created by pedronis> <https://github.com/snapcore/snapd/pull/7764>11:01
pedronismvo: ^11:01
mupBug #1560942 changed: Support swap <snapd:Invalid> <Snappy:Fix Released> <ubuntu-core-config (Ubuntu):Fix Released> <https://launchpad.net/bugs/1560942>11:13
Chipacasil2100: hiya. Are you the person to ask about details of the rpi gadget?11:15
sdhd-saschasil2100: what is it about?11:20
Chipacasdhd-sascha: ?11:24
sdhd-saschachipaca: sorry, i want you to ask about what question it is?11:28
Chipacasdhd-sascha: about the pi3-gadget, https://github.com/snapcore/pi3-gadget11:28
Chipacasdhd-sascha: why?11:28
Chipacabecause I _think_ #1500164 is fix-released, but I'm not the expert11:29
mupBug #1500164: console= cmdline options on the RPi2 should be moved from uboot.env to cmdline.txt <Snappy:In Progress> <https://launchpad.net/bugs/1500164>11:29
ograChipaca, it is11:29
Chipacaogra: you said you thought so, but that we should check with foundations :)11:29
Chipacaogra: so here i am trying to do that11:29
ograwell, i cheked the code but yeah, a re-assurance from foundations would be bad ... though you can always re-open the bug anyway :P11:30
ogra*woudlnt11:30
ograbah11:30
* ogra subscribes to a typing course 11:31
Chipacaogra: in my experience subtle bugs like this very rarely get re-opened11:31
Chipacawe're lucky they get filed in the first place :D11:31
ograi'm pretty sure it is fix-released (90%+)11:31
Chipacaogra: OTOH the reporter of this bug in particular is a crotchety ol' guy that'll open it again without a doubt11:31
ograhaha11:31
Chipacaand, that's all the triage i'll be doing today, methinks11:32
Chipacanow to attempt a run11:32
* Chipaca goes11:32
zygamvo: do you have a moment11:33
zygamvo: can we chat about the thing from yesterday11:33
mvopstolowski: hey, sure thing!11:33
zygamvo: we've discussed the technical side with maciek just now and wanted to give you some options11:33
zygamvo: we're in https://meet.google.com/nst-hqvw-bhb?authuser=111:33
mvozyga: uh, nasty jetlag this time :(11:34
mupBug #1500164 changed: console= cmdline options on the RPi2 should be moved from uboot.env to cmdline.txt <Snappy:Fix Released> <https://launchpad.net/bugs/1500164>11:34
zygamvo: tell me about it ;D11:34
mvozyga: the improved snapdfuse is in 2.41+ but I think you need to install the deb to get the benefits. I *might* be wrong though11:34
mvozyga: improved deb is in -proposed11:34
cmatsuokamvo: thanks for the review. I'm addressing the issues and will push again soon11:34
mvocmatsuoka: \o/11:35
=== ricab is now known as ricab|lunch
zygatests running12:58
mupPR snapd#7765 opened: boot,bootloader: make MakeBootable() uc20 aware <Created by mvo5> <https://github.com/snapcore/snapd/pull/7765>13:34
mupPR snapd#7763 closed: [RFC] boot,bootloader: make MakeBootable() uc20 aware <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/7763>13:39
ogramvo, what happened to http proxy settings in core18 ? (/etc/environment is not writable anymore ?!?)13:49
ograogra@pi4:~$ snap set system proxy.http="http://my.proxy.com:8080"13:49
ograogra@pi4:~$ grep my.proxy /etc/environment13:49
ograogra@pi4:~$ sudo touch /etc/environment13:49
ogratouch: cannot touch '/etc/environment': Read-only file system13:49
ograogra@pi4:~$13:49
ograor zyga ^^^13:49
zygaogra: I'm sorry, I'm not very reactive today13:51
ograwell, you cant set a http/https proxy in core18 at all it seems ... dont we have tests for that ? or did the implementation change13:52
ogralooking 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 set13:52
ograogra@pi4:~$ snap change 138413:53
ograStatus  Spawn               Ready               Summary13:53
ograDone    today at 13:43 UTC  today at 13:43 UTC  Run configure hook of "core" snap13:53
ogranot really telling me about the fact that it didnt set the proxy13:54
zygaI don't know13:55
ogranothing in the journal either13:55
ograogra@pi4:~$ sudo journalctl|grep proxy13:55
ograogra@pi4:~$13:55
zygasorry13:56
ograwell, thats pretty gross we have a few customers relying on that working as documented13:56
zygais /etc/environment a squash13:56
zygasorry, I'm not feeling very well13:56
ograit wasnt in core16 it seems to be in core1813:56
zygathen it's a bug13:56
ograyes13:56
zyganeeds to be addressed, regression tests etc13:56
ograjust wondering how that can happen13:56
zygaperhaps existing test can be reused if there is onew13:56
zygawe don't run many tests on core1813:56
mvoogra: let me have a look (but we have a meeting in 3min so I maybe a bit slow)13:56
ograk13:56
mvozyga: I beg to differ, we run almost all tests on core1813:57
ograzyga, just leave it to michael :)13:57
zygamvo: if that's the case I'm sorry, I wasn't aware of that13:57
zygamvo: what I meant is that may of our tests don't use base: core1813:57
zygathough for this problem it should not matter13:57
mvozyga: aha, I see13:58
mvozyga: yeah, that's true. but I think ogra issues is on a core18 based core system, yes?13:58
ogramvo, i think so ... i'll ask the customer that complained, they only provided snap list which indeed shows core and core18 installed13:59
mvoogra: I found the relevant test, running this now on core1814:00
ograbut i think they run on core18 (it also works fine on core16, i just checked)14:00
ograseems the internal snapd setting of the proxy to contact the store is fine ... but not the global change to /etc/environment14:00
ogra(simply because it got dropped from writable-paths i guess)14:01
ograogra@stream:~$ snap set system proxy.http="http://my.proxy:8080"14:03
ograogra@stream:~$ sudo grep proxy /etc/environment14:03
ograhttp_proxy=http://my.proxy:808014:03
ogracore16 works fine14:03
mvoogra: yeah, it's anyoing, we have a spread test that tests the internal setting on core18 but I think not the other one14:09
ogramvo, well, if the config hook would at least spill an error it would have been easier to catch :)14:09
ograbut even that goes quietly through14:10
mupPR snapd#7756 closed: tests: update mount-ns test for bionic image changes <Created by stolowski> <Closed by stolowski> <https://github.com/snapcore/snapd/pull/7756>14:10
mvoogra: yeah, no discussion that this is bad(tm)14:10
ogra(unless i'm seriously missing something and we dont set systemwide proxy settings in /etc/environment anymore)14:10
mvoogra: running tests now (and in a meeting) - I extended the proxy test and then I should be able to say more14:13
=== ricab|lunch is now known as ricab
Chipacameet kicked me out and refuses to connect, now14:32
roadmrit does that a lot :/14:33
* Chipaca curses the google goblins14:34
pedronisChipaca: oopsy, we wanted to stay a bit and sort out confusion with mvo, anyway we can chat on Monday14:40
ijohnsonChipaca: to answer your question from the HO chat, no I have not tested any of this on spinning disk systems14:40
Chipacapedronis: i can't access any bit of google right now14:40
ijohnsonChipaca: do you think I should?14:40
Chipacaijohnson: I think it's a lot more painful there, and might be painful in different ways14:41
pedronisChipaca: oops, that seems bad/strange14:41
Chipacaijohnson: is that something that's easy for you to do?14:41
Chipacapedronis: now it's back, fwiw14:41
ijohnsonChipaca: 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 there14:41
Chipacaijohnson: I've got 19.10 on a spinning-disk laptop, if that's any use14:42
Chipacapedronis: https://forum.snapcraft.io/t/offline-update-of-multiple-snaps/14258 is the topic i mentioned in the standup14:43
ijohnsonhmm, 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 things14:43
Chipacaijohnson: 1904, or 1804?14:43
ijohnsonChipaca: don't worry about it, I can handle it, you should prepare for your conference :-) (and have fun too)14:43
ijohnson190414:43
Chipacaijohnson: you typoed "panic"14:44
ijohnson:-)14:44
* zyga breaks for coffee14:44
zygabrb14:44
Chipacaijohnson: thanks!14:44
Chipacaanyway, i'm off14:44
Chipacattfn!14:44
mvoogra: https://github.com/snapcore/core18/pull/14314:44
mupPR core18#143: static: add /etc/environment to writable-paths <Created by mvo5> <https://github.com/snapcore/core18/pull/143>14:44
mupPR core18#143 opened: static: add /etc/environment to writable-paths <Created by mvo5> <https://github.com/snapcore/core18/pull/143>14:45
mvoogra: also test missing on our side, working on this too14:45
mborzeckihmm snapd calling systemctl stop snapd.failure.service when snapd.failure.service actually stated snapd in recovery mode :/14:53
* ogra hugs mvo14:55
mvoogra: 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
mvoogra: with the fix the configure hook fails and says "/etc/environment: read-only fs"14:58
mvoogra: but yeah, there should be a PR (or two) for this by my eod14:59
ogramvo, thanks a lot !15:00
ograijohnson, was it you who complained about slow forum ? i see it now too ... i have seen more of the spiner than of the content today15:30
ogra*spinner15:30
ijohnsonogra: yes and others were discussing the issue in our standup today too16:07
ijohnsonseems a global problem16:07
zygaDinner16:10
ograyeah16:12
ograi 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:13
mvoogra: 7766 and 7767 address your issue, thanks for reporting16:33
mupPR snapd#7767 opened: tests: run snap-set-core-config on all core devices <Created by mvo5> <https://github.com/snapcore/snapd/pull/7767>16:33
* cachio lunch16:34
* ogra hugs mvo once more16:35
=== pstolowski is now known as pstolowski|afk
zygamvo: hey17:38
zygaremember your recent PRs?17:38
zygaerror: cannot perform the following tasks:17:38
zyga- Run configure hook of "core" snap (run hook "configure": open /etc/environment: read-only file system)17:38
zygamvo: this requires the new core18 release, right?17:39
zygasil2100: https://github.com/snapcore/core18/pull/143 an you please do a 2nd review?17:40
mupPR core18#143: static: add /etc/environment to writable-paths <Created by mvo5> <https://github.com/snapcore/core18/pull/143>17:40
sil2100zyga: o/17:49
zygahey17:50
zygagood evening17:50
zygasorry to poke you so late17:50
zyganot urgent but would help to unblock this for tomorrow17:50
zygathanks a lot :)17:50
zygaenjoy your evening17:51
mupPR core18#143 closed: static: add /etc/environment to writable-paths <Created by mvo5> <Merged by sil2100> <https://github.com/snapcore/core18/pull/143>17:51
zygasil2100: I cannot find the build recipe for core18, can you please trigger a build?17:56
sil2100zyga: sure o/ Will need to sync the mirror first18:06
sil2100I mean18:06
sil2100The builds auto-trigger on new code18:06
zygasure, anything it takes to release new core18 to edge18:06
sil2100(and other stuff)18:06
=== ijohnson is now known as ijohnson|lunch
mvozyga: yes, that requires the latest core1818:20
mvozyga: I thought I did mention it in the PR but maybe I forgot18:20
mupPR snapcraft#2816 opened: assorted: address errors from mypy uprev <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2816>18:21
zygamvo: it's all right :)18:27
zygamvo: I made solution B work, I think, it's not terrible actually18:27
zygamvo: I'll grab a coffee (heh) and sit tight18:28
zygamvo: I'll prepare a few PRs for tomorrow for maciek18:28
zygabrb18:28
zygamvo: lol18:30
zygamvo: check out this snippet18:30
zyga(backstory: iOS 13 was more buggy than older releases)18:30
zygaSoftware 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
zygavia 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
zygafeature flags FTW :)18:30
mvozyga: thanks! let's see if we can get something into beta tomorrow18:37
mvozyga: haha, yes18:37
* ogra is impressed by kyrofa's accent free russian !18:54
kyrofaogra, right?!18:54
kyrofaYou don't KNOW my skills18:54
ogra:D18:54
kyrofaI miss you buddy. You gonna be in Frankfurt I hope?18:55
ograyeah, i actually will !!!!18:55
kyrofaAwesome18:56
ogralooking forward to it18:56
zygameh, there are still 11 unit tests to adjust20:28
zygaand god knows how many comments20:28
zygabut it passes all regression tests20:28
zygathat's good20:28
mupPR snapd#7766 closed: configcore: fix missing error propagation <Created by mvo5> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/7766>20:33
* cachio afk20:57
=== arnatious_ is now known as arnatious
mupPR snapd#7768 opened: interfaces: add raw-volume interface for access to partitions <Created by jdstrand> <https://github.com/snapcore/snapd/pull/7768>22:13
jdstrandondra: fyi, ^22:17
setuidDoes 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:23
ograyou 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)22:30
* zyga does midnight supper23:00
=== ijohnson|lunch is now known as ijohnson

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!