zygaI will start later today. My son is going for a weekly school trip. Need to see him off05:09
zygaHey hey05:57
zyga07:09 <zyga> I will start later today. My son is going for a weekly school trip. Need to see him off05:57
mborzeckizyga: hey06:26
zygaStill waiting for the coach to pick the kids up06:41
zygaLots of traffic in Waw this morning, could be a while06:42
=== pstolowski|afk is now known as pstolowski
mupPR snapd#6875 opened: store,daemon: add client-user-agent support to store.SnapInfo <Created by mvo5> <https://github.com/snapcore/snapd/pull/6875>07:10
zygaHej PaweĊ‚07:25
zygaHey pedronis, how are you doing?07:32
zygaI will be starting in about an hour07:33
zygaBus is packed and ready to go (school trip)07:33
mupPR snapd#6876 opened: Optical-drive Interface: Add scsi-generic support <Created by diddledan> <https://github.com/snapcore/snapd/pull/6876>07:36
diddledanmorning :-)07:37
mborzeckipstolowski: hey07:41
mborzeckimvo taking a day off today?07:49
mvomborzecki: I'm here07:51
mborzeckimvo: hi, didn't notice07:51
mvomborzecki: doing silly admin stuff right now :)07:51
mvomborzecki: yeah, mostly silent07:51
mborzeckimvo: #6872 needs a glimpse from you07:51
mupPR #6872: many: backport fixes to 2.39 <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6872>07:51
mvomborzecki: sure, will do !07:52
pedronisI'm around too, will swap Fri07:54
* Chipaca thinks things are too quiet and goes to check his email08:05
zygaChipaca: hey :)08:08
zygahow was last week08:08
zygaman, it's good to be home with all the peace and quiet here!08:08
Chipacazyga: :)08:09
Chipacalast week was a'ight08:09
zygaand *plenty* of updates after last week's CPU bugs08:09
* zyga reboots 08:09
mvohey Chipaca and pstolowski !08:11
pstolowskihey o/08:12
Chipacamvo: o/08:13
Chipacamborzecki: with parallel installs, hooks don't run with remapped names, right? ie a hook can't do "snapctl restart foo.bar", they need to do "snapctl restart $SNAP.bar"?08:16
mborzeckiChipaca: the forum topic?08:16
Chipacamborzecki: yes08:16
Chipacamborzecki: just checking before telling them they're holding it wrong08:17
mborzeckiChipaca: wanted to sync with pedronis/mvo first, but I believe, all the commands should use $SNAP_INSTANCE_NAME & $SNAP_NAME instead of hardcoding nextcloud08:17
mvoyeah, hardcoding sounds bad(tm)08:17
mborzeckiChipaca: in that case, SNAP_INSTANCE_NAME == nextcloud_108:17
* Chipaca always hardcodes nextcloud even if the snap is called something else08:17
Chipacamborzecki: good :)08:17
Chipacabah, i can see the dev's perspecive on this, seems weird, but yeh08:18
pedronisyes, I'm not sure we can reasonably ask them that08:18
mborzeckidoing magic could work, but i sense possible problems when we have cross snap ordering and whatnot08:18
pedronisby definition we will not let snap refer to other snaps by name08:19
zygaI think snapctl driven requests could easily be remapped08:19
mborzeckihmm if there's no chance they'd attempt to refer to other snap by name, then we can do it08:20
pedronismborzecki: we really don't want snap to talk about other snap by name,  default-provider are the only exception, for sure not at runtime08:21
pedronisanyway I fear that the all set of service commands were a bit under thought out, as we know08:21
pedronisanyway it's fine to suggest a workaround, I don't see us jumping on changing this now either08:21
mborzeckii can put it my todo queue and take a look after gadget updates08:22
pedronismborzecki: ok, maybe,  let's discuss when we get there08:23
mborzeckipedronis: sgtm :)08:23
mborzeckizyga: can you take a look at #6874 later on?08:25
pedronispstolowski: hi, do you have something blocked on me?  what's the status of auto snapshot docs?08:25
mupPR #6874: cmd/snap-confine: do not mount over non files/directories <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6874>08:25
zygamborzecki: will do08:25
mborzeckizyga: pinged jdstrand and pedronis for reviews on this one too08:26
pstolowskipedronis: hi, not really, you can take a look at the snap remove --purge PR, but it's not blocking antyhing. i've send the auto snapshot doc change to degville a few days ago08:28
degvillepstolowski / pedronis: morning! I updated the docs with the auto snapshot details pstolowski sent over - on the getting started page, system options page and on the snapshots page.08:29
pstolowskidegville: i think i forgot to clarify the time duration format ("d" for "days"), or did you sort that out yourself?08:30
degvillepstolowski: I removed the day example (just used the hours example) until we had some clarification, so it still makes sense.08:31
pstolowskidegville: k, thanks08:32
zygamborzecki: reviewed08:34
Chipacaxnox: i'd like to know more about #182951008:41
mupBug #1829510: snapd connectivity check did not change fastly cdn <subiquity:New> <snapd (Ubuntu):New> <https://launchpad.net/bugs/1829510>08:41
Chipacaxnox: istm there's more going on and you might've jumped to conclusions somewhere08:41
mvopstolowski: was "Infrastructure to measure subtimings for operations" done for 2.39 or for 2.40 (iirc the later but not quite sure)08:43
mvoChipaca: "snap download and related changes for snapcraft" is done, right? iirc it was 2.39 or is there something missing08:45
Chipacamvo: no not done yet08:45
mvoChipaca: oh, ok08:45
Chipacamvo: it has two parts, one part is done, the other isn't08:45
mvoChipaca: aha, ok - the --output part is missing?08:45
mupPR snapd#6877 opened: overlord/configstate: don't panic on invalid configuration <Created by stolowski> <https://github.com/snapcore/snapd/pull/6877>08:46
pstolowskimvo: let me check that08:46
Chipacamvo: yes08:46
mvoChipaca: ta!08:46
pstolowskimvo: yes, timings are 2.40; 2.39 is missing some important commits. and i've one more PR and a bug to fix there too08:55
mvopstolowski: sure thing, thank you!08:56
mvopstolowski: I will update trello08:56
mvoChipaca: do you have a few min for a quick HO on the Snap-Client-User-Agent feature? just need someone to bounce ideas off :)09:16
Chipacamvo: in 5?09:17
Chipacagot a bit of context right now09:17
pedronismvo: I suspect  you maybe should leave that to Chipaca at this point, I fear I'll have opinions on details and will drag a bit09:20
mvoChipaca: sure, or in 10 if you prefer09:20
mvopedronis: ok, I hope this won't change to radically from what we have right now up for review09:21
pedronisprobably not, but we are adding context everywhere, we should try to do it right, given now is in the new cycle work09:21
mvopedronis: do you already have a sense of how to transmit the client-user-agent from a "snap install"? I mean, have you looked at this yet? it seems we need to put it on the download task09:21
pedronishave we agreed to need it on download?09:22
ChipacaI thought it was only needed in the synchronous part09:23
Chipacai.e. in what we call the info and refresh endpoints09:23
mvopedronis, Chipaca we can do it either way, I think doing it in the task is more realistic but either way will work09:23
Chipaca(refresh being what we hit for install/download/refresh)09:23
mvo(this is mostly why I wanted to talk :)09:23
Chipacai'll just finish wrapping up this (more PRs for reviewage) and then will be free for y'awl09:24
pedroniswe can ask the store what they prefer, not needing it in download would slightly better for us09:24
mvopedronis: ok, I can kick this off and then we can decide based on their reply09:24
mvopedronis: out of curiosity, is your current thinking to add a context to "snapstate.Install" ?09:25
pedronisin theory anything that we would like to interrupt should take it09:26
pedronisunrelatedly of this precise task09:26
mvopedronis: all right! I think thats all I wanted to know, so no need for further discussion with Chipaca  I think. and its easy to extend to download if we have to09:26
mvopedronis: thank you!09:26
mupPR snapd#6878 opened: cmd/snap, etc: add health to 'snap list' and 'snap info' <Created by chipaca> <https://github.com/snapcore/snapd/pull/6878>09:26
Chipacamvo: pedronis: ok i'm ready, am i needed?09:27
pedronisChipaca: I don't know09:27
* Chipaca weeps09:27
mupPR snapd#6653 closed: tests: try to be a little bit invariant <Created by chipaca> <Closed by chipaca> <https://github.com/snapcore/snapd/pull/6653>09:30
mvoChipaca: I think pedronis already answerd all my questions :)09:31
mvoChipaca: sorry, we can still chat if you want09:32
Chipacamvo: nah :)09:32
mvoheh :)09:32
mvothanks degville for updating trello :)09:57
degvilleoh, no problem mvo - sorry I left the card there so long.09:58
mvodegville: no worries09:58
mupPR snapd#6879 opened: [RFC] overlord: add context.Context to snapstate.Install() <Created by mvo5> <https://github.com/snapcore/snapd/pull/6879>10:12
* Chipaca takes a break10:21
mupPR snapd#6877 closed: overlord/configstate: don't panic on invalid configuration <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6877>10:28
mborzeckizyga: about nsswitch, it doesn't happen in our spread setup bc it's modified in our image10:44
zygado you know why it was modified?10:44
zygamborzecki: https://github.com/zyga/mimic-bug offtopic10:44
zygamborzecki: this illustrates the issue I was fighting last week10:45
mborzeckizyga: no clue, i don't see anything in our test setup or spread10:45
zygamborzecki: probably the image was "cured"10:45
zygamborzecki: if you have a moment please have a look at bug.sh10:45
zygaI would love to have a call and discuss options10:45
zygaI feel a bit stuck now :/10:45
zygajust need to grab a coffee10:46
zygastill overdosed from last week :)10:46
xnoxChipaca:  re connectivity check => so snapd found new subiquity snap (visibile with like $ snap info SNAPNAME) but $ snap download SNAPNAME was stalling, ie. zero progress because connectivity to fastly requires proxy.10:56
xnoxChipaca:  what logs / details do you need from me? i can try to recreate that network setup and gather things for you.10:56
zygamborzecki: ok10:56
Chipacaxnox: but "snap download" and "snap refresh" are very different paths wrt proxies10:56
Chipacaxnox: which one was it? because the bug mentions refresh10:57
xnoxChipaca:  it was both.10:57
Chipacaxnox: and the connectivity check reflects what happens on refresh, not download10:57
xnoxChipaca:  both refresh, and download stalling, despite info showing that new stuff is available.10:58
xnoxChipaca:  i believe subiqutiy queries snapd's connectivity check before showing the "new installer is available screen"10:58
zygamborzecki: do you have some time for interactive debugging?10:59
Chipacaxnox: if you could turn set SNAPD_DEBUG=1 and SNAPD_DEBUG_HTTP=7 in /etc/environment that might give us an insight into what's going on10:59
mborzeckizyga: need ~15 minutes10:59
zygathank you!10:59
xnoxChipaca:  ack.10:59
Chipacaxnox: (snapd would have to restart to pick those up)10:59
Chipacaxnox: if the connectivity check is not picking up on the problem, something's wrong and needs fixing11:00
Chipacaxnox: otoh i don't know where these things run, maybe cloud-init needs tweaking?11:00
Chipacaits configuration i mean11:01
Chipacai honestly don't know11:01
Chipacahere's hoping for logs :)11:01
xnoxwell, i care about this in the context of subiquity server live image.... and I know how to hack that to do what I want =) so we are all good.11:02
xnoxmvo:  vorlon: i still think i want to pre-built correct initrds for all arches, and publish them into the snapstore. Only for the snapcraft kernel plugin to consume them as staging those snaps..... Not for them to be installed on target devices directly.11:04
xnoxKind of like "shipping a snapcraft build-dependency as a snap in the store"11:05
xnoxcause i am being asked for a FDE embedable initrd, for uc16 and uc18 as well as uc2011:05
zygamborzecki: https://meet.google.com/bpg-rxba-igg?ijlm=1558350555676&hs=125&adhoc=1&authuser=0 when ready11:09
mupPR pc-amd64-gadget#14 opened: gadget.yaml: add system-recovery partition <Created by mvo5> <https://github.com/snapcore/pc-amd64-gadget/pull/14>11:13
pedronisChipaca: connectivity-check tries a download but yes snap download will use different configs all together atm11:18
Chipacapedronis: yes as i said11:18
Chipacapedronis: but the bug talked about 'refresh', not download, otherwise i would've brought that up earlier11:19
Chipacapedronis: but it turns out connectivity-check works, but neither refresh nor download do11:19
Chipacapedronis: hence, gimme logs11:19
pedronisyea, that's unexpected11:19
Chipacalunch is calling11:31
* Chipaca obeys11:38
=== ricab is now known as ricab|lunch
cachiopstolowski, hey11:48
pstolowskicachio: hey! i've been debugging the test issue11:48
cachiopstolowski, nice11:48
cachiopstolowski, is it a test issue?11:48
cachioor is it a problem in the code?11:49
pstolowskicachio: so, first of, the test is incorrect11:49
pstolowskicachio: after fixing it passed on 16.0411:49
cachiopstolowski, and on core18?11:50
pstolowskicachio: it sill fails on core 18 and i'm trying to find out why. fwtw i've flashed my pi3 with core18 from edge this morning and hotplug worked with real device, so i suspect it is still something with the est11:50
cachiopstolowski, yes, probably, it is the first time we run this test on core11:50
pstolowskicachio: also, not sure if you noticed, it's probably not affecting this test directly, but in both cases the "initialize device" change fails repeatedely and is retried11:51
pstolowskicachio: https://paste.ubuntu.com/p/YRWth9FdjM/11:52
pstolowskicachio: it's device serial to be exact11:52
cachiopstolowski, yes11:52
cachioI knew that11:53
pstolowskiah ok11:53
cachiopstolowski, but it shouldn't affect11:53
pstolowskicachio: indeed, seems to be irrelevant12:00
pstolowskicachio: re core18 test failure i think it's not using the latest versions?12:25
pstolowskicachio: https://paste.ubuntu.com/p/67fJ4k8Mvw/12:25
cachiopstolowski, snanpd is still on stable12:26
cachiocould you push the change you did to fix core16?12:27
pstolowskicachio: right, that's the issue12:27
cachioso then I'll push the fix fot that with other stuff that I also fixed today12:27
pstolowskicachio: ok, will do in a while, trying to run everything from this test now12:29
cachiopstolowski, nice12:30
pstolowskicachio: here is the diff https://pastebin.ubuntu.com/p/c3kMDWhzzM/12:59
pstolowskicachio: it passed for me on core1612:59
cachiopstolowski, nice, I'll add it to the next commit13:00
zygamborzecki: dziala!13:01
=== ricab|lunch is now known as ricab
* zyga goes for lunch13:53
cachiopstolowski, so, could you please paste the whole test13:54
pstolowskicachio: https://paste.ubuntu.com/p/mKryYZNtHK/13:55
cachiopstolowski, with this change in ubuntu-image I'll create a new PR after this one13:55
cachioso, the idea is to pre-generate the image witt the desired core/snapd and avoid the refresh which takes time13:55
pstolowskicachio: it's basically the original test from classic, minus the -writing bits for serial port that required dialout to work13:55
cachiopstolowski, nice, thanks13:56
mupPR snapd#6869 closed: daemon: only allow `users`/`create-users` when not on classic* <Created by chipaca> <Closed by chipaca> <https://github.com/snapcore/snapd/pull/6869>13:56
pedronismborzecki: mvo: I gave my +1 to #687214:19
mupPR #6872: many: backport fixes to 2.39 <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6872>14:19
mvopedronis: thank you14:19
mupPR snapd#6872 closed: many: backport fixes to 2.39 <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6872>14:20
pedronispstolowski: degville: auto snapshots docs don't seem to mention that atm they are off by default on core14:31
mborzeckioff taking the kids to the troop meeting14:34
degvillepedronis: thanks for checking - that's important - I'll add it now.14:34
mupPR snapd#6880 opened: daemon: refactor user ops to api_users <Created by chipaca> <https://github.com/snapcore/snapd/pull/6880>14:36
Chipacapedronis: I addressed the concerned you'd raised in #6564 fwiw (other than two that're better left for a later pr)14:50
mupPR #6564: cmd/snap, tests: refactor info to unify handling of 'direct' snaps <Created by chipaca> <https://github.com/snapcore/snapd/pull/6564>14:50
pstolowskipedronis, degville to clarify - no snapshots for core snaps are created, only for app snaps; yes14:52
pedronispstolowski: ?14:52
degvillepstolowski: thank you!14:52
pedronisdegville: pstolowski: I think we are talking about different things14:53
pedronisdegville: pstolowski: I'm talking about this: https://github.com/snapcore/snapd/blob/master/overlord/snapshotstate/snapshotstate.go#L10314:54
degvillepedronis/pstolowski: I'll mention both cases.14:55
pedronison core we default to no14:55
pstolowskipedronis: ah, this landed while i was away beginning of May. i see, makes sense, thanks14:58
pedronispstolowski: likely, we did this to be more conservative with disk space on core15:15
* cachio lunch15:30
pedronispstolowski: Chipaca: I did a pass on #687015:51
mupPR #6870: cmd/snap, api, snapstate: implement "snap remove --purge" <Created by stolowski> <https://github.com/snapcore/snapd/pull/6870>15:51
pstolowskipedronis: ty15:52
mupPR core#38 closed: Add another pi-config option <Created by sergey-borovkov> <https://github.com/snapcore/core/pull/38>16:06
mupPR core#83 closed: move most of the ubuntu-core config deb into the snap snap build <Created by mvo5> <https://github.com/snapcore/core/pull/83>16:06
mupPR core#104 closed: snapcraft.yaml: use remote fc-cache-builder <Created by mvo5> <https://github.com/snapcore/core/pull/104>16:06
mupPR core#38 opened: Add another pi-config option <Created by sergey-borovkov> <https://github.com/snapcore/core/pull/38>16:07
mupPR core#83 opened: move most of the ubuntu-core config deb into the snap snap build <Created by mvo5> <https://github.com/snapcore/core/pull/83>16:07
mupPR core#104 opened: snapcraft.yaml: use remote fc-cache-builder <Created by mvo5> <https://github.com/snapcore/core/pull/104>16:07
=== pstolowski is now known as pstolowski|afk
cachiopstolowski|afk, hey, I pushed the hotplug test working on both cores16:58
cachiopstolowski|afk, in a following PR I'll add the new improvemente to avoid test code duplication16:58
cachiopstolowski|afk, but I prefer keep it as it is to simplify the change and start running this new change as part of the nightly suite16:59
mupPR snapd#6618 closed: tests: validates snapd from ppa <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6618>17:59
* cachio afk18:11
mupPR snapd#6874 closed: cmd/snap-confine: do not mount over non files/directories <Created by bboozzoo> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6874>20:49

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