/srv/irclogs.ubuntu.com/2020/01/13/#snappy.txt

mborzeckimorning06:17
sdhd-saschamorning :-)06:28
zygagood morning07:29
* zyga applies updates07:30
mborzeckizyga: hey07:48
zyga:-)07:49
zygahey, how are you doing?07:49
mborzeckizyga: caught a cold :/07:49
zygagiven where we live I think it is inevitable07:50
mborzeckizyga: but managed to build centos cloud image, since there's still no official ones yet07:50
zygaI spent most of the weekend doing homework with kids07:51
zygaI spent some hours on manual pages for API docs07:52
mborzeckizyga: bet is was much fun ;)07:52
zygaI'll try to publish my toy library next weekend07:52
mvohey zyga and mborzecki07:53
mvozyga: what is you library about?07:53
zygahey mvo, how are you?07:53
mborzeckimvo: hey07:53
zygamvo: testing! :)07:53
zygamvo: libzt, robust and simple testing07:53
zygafor C07:53
zygatested, documented, building in all major OSes07:53
zyga(that's the goal)07:53
zygaI'm maybe 60% there07:53
mborzeckizyga: zt? libzygatest?07:54
zygamborzecki: over time my private libraries evolved in their naming scheme but I guess z is just z, not an acronym07:55
zygat is indeed for "testing"07:55
zygaI ended up writing it after another private toy project needed more testing than I could do by just playing with it07:55
zygaand I didn't want to depend on glib07:55
zygaor anything that's a mess to build outside of linux07:55
zygaanyway07:56
mvozyga: nice! I'm good, thanks, didn't sleep that well though :/07:56
zygatoday is snap run --explain day :)07:56
zygajamie is off this week so I will focus on things that don't touch C07:56
pstolowskimorning08:00
zygahey pawel08:00
pstolowskio/08:00
zygagosh bugzilla is such a dinosaur08:02
zygatrying to find snapd bugs08:02
mborzeckipstolowski: hey08:03
zygais search implemented by grepping the database?08:03
mborzeckizyga: rhbz? or suse one?08:03
zygasuse08:03
zygaI clicked "search" like two minutes ago08:04
zyga...08:04
zygaand got 40008:04
zygahttps://bugzilla.novell.com/buglist.cgi?query_format=specific&order=Importance&no_redirect=1&bug_status=__all__&product=&content=snapd08:05
mvohey pstolowski !08:05
mborzeckizyga: hmm maybe they structured the project differently than rhbz has08:05
mborzeckizyga: in rhbs it's product - pick fedora or EPEL, component - pick snapd, status all open and that's it08:06
zygaI'll search in my mail instead08:06
mborzeckiat least it's not mantis, had trouble running a trivial search in centos bug tracker08:06
zygabrb, tea refill08:14
zygarainy morning08:14
mborzeckioh, and it's possible to install snapd on centos8, but one has to enable `cr` repository as well, sice the selinux packages are part of 8.1 that's not in base os repo yet08:14
zygacr?08:14
mborzeckizyga: continuous release, iow upcoming updates08:15
zygaaha08:15
zygaoh well :)08:15
mupPR snapd#7979 closed: many: drop NameAndRevision, use snap.PlaceInfo instead <UC20> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7979>08:25
mupPR snapd#7978 closed: data/selinux, test/main/selinux-clean: update the test to cover more scenarios <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7978>08:31
mborzeckidegville: hi, i've added a note about CentOS CR repo under https://forum.snapcraft.io/t/installing-snap-on-centos/10020 can you take a look? :)08:33
mborzeckiheh, suprised one took the time to write this https://bugs.launchpad.net/ubuntu/+source/snapd/+bug/1859185/comments/308:36
mupBug #1859185: No (obvious) way to turn autorefresh off <amd64> <apport-bug> <bionic> <snapd (Ubuntu):New> <https://launchpad.net/bugs/1859185>08:36
degvillemborzecki: will do - thanks for letting me know.08:37
pstolowskizyga: got your new mbp yet?08:44
zygapstolowski: no, it's still in netherlands09:39
zygapstolowski: and only with a shipping label :/09:40
zygapstolowski: so sigh09:40
zygapstolowski: I don't doubt it will arrive on time but they said 14-15th09:40
zygaprobably waiting just to arrive "on time"09:40
pstolowskimhm09:41
pedronisChipaca: hi, are there open problems with #7984 (just skimmed it) or are you ok with it?  if not I'm happy to discuss09:48
mupPR #7984: store, overlord/snapstate, etc: SnapAction now returns a []…Result <Created by chipaca> <https://github.com/snapcore/snapd/pull/7984>09:48
Chipacapedronis: I'm ok with it09:49
Chipacapedronis: it's shorter than my first attempt so that's a plus :-)09:49
pedronisChipaca: ok, let me know if we land in trouble further down, but let's go this way for now09:49
Chipacapedronis: ok, next pr should be up before noon09:50
Chipacastacked on this one i mean09:50
pedronisunderstood09:50
pedronismborzecki: I did a quick pass about the changes in #797210:02
mupPR #7972: overlord/snapstate, wrappers: undo of snapd on core <Remodel πŸš‹> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7972>10:02
mborzeckipedronis: cool, thanks10:03
pstolowskipedronis: hey, do you have a moment to talk quickly about some aspect of snap disconnect --forget?10:03
pedronispstolowski: can do in ~20 mins ?10:06
pstolowskipedronis: sure10:06
zygabrb10:24
pedronispstolowski: I'm available now10:27
pstolowskipedronis: ok, coming to standup ho10:28
mupPR snapd#7985 opened: tests: use snap remove with --purge flag in most of the spread tests <Test Robustness> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7985>11:04
pstolowski^ pretty mechanical11:05
mvolooks like core18 is failing right now in master, has anyone already looked? if not, I'm doing so now11:07
zygahaven't looked, can look if you are busy11:07
pstolowskiChipaca: ty!11:09
Chipacapstolowski: i was about to ask about just setting the option, but this way it's explicit so +111:10
mvozyga: thanks, should be fine11:12
pstolowskiChipaca: yeah i was thinking about some magic, but preferred explicit11:12
mvopstolowski: nice PR, thanks for this11:17
pstolowskimvo: i should have thought about it earlier!11:17
mvopstolowski: do we have (at least) one test left that run snap remove without purge? I assume we do, just want to double check :)11:19
pstolowskimvo: yes, a few tests that test remove itself, snapshots test, and selinux tests (i mentioned this in the commit and PR desc)11:20
mvopstolowski: \o/11:20
cachiomvo, hey11:45
cachiomvo, about beta release11:46
cachioare you planning a 2.43.1 ?11:46
cachioor it needs to be discussed?11:46
mvocachio: I'm not aware of 2.43 issues right now, so hopefully we don't need 2.43.1. or am I missing something?11:52
cachiomvo, I fwd an email on friday11:52
mupPR snapd#7985 closed: tests: use snap remove --purge flag in most of the spread tests <Simple πŸ˜ƒ> <Test Robustness> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7985>11:55
mvocachio: oh, missed that, yes, this sounds very much like we need 2.43.111:56
mvocachio: let me read the full bug11:56
cachiomvo, nice, thanks11:56
mvocachio: thank you!11:57
Chipacapedronis: https://github.com/snapcore/snapd/pull/7986/commits/479e46211c3ec18c95b6634d1b33fb54acc5ff4c12:08
mupPR snapd#7986 opened: store, o/snapstate: send default-tracks header, use RedirectChannel <Created by chipaca> <https://github.com/snapcore/snapd/pull/7986>12:08
mupPR #7986: store, o/snapstate: send default-tracks header, use RedirectChannel <Created by chipaca> <https://github.com/snapcore/snapd/pull/7986>12:08
* Chipaca will start abbreviating overlord to 😎12:08
Chipacaalas there is no panopticon emoji12:09
Chipacaemojus?12:09
Chipacaemoja?12:09
* Chipaca hides12:09
mupPR snapd#7987 opened: snap: make `snap version` output host without extra whitespace <⚠ Critical> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7987>12:10
Chipacamvo: is the host-and-virt-in-version only just getting released?12:11
mvoChipaca: yes, part of 2.43 which is in beta right now, never made it beyond beta afaict12:12
Chipacamvo: use U+2620 instead of / :-p12:15
mvoChipaca: heh12:15
Chipacamvo: did you mean to target it to master?12:16
pedronisChipaca: mvo: we need to discuss that bit12:17
mvoChipaca: yes, I will backport it from there once we have a agreement12:17
mvopedronis: sure, should we do it right after the standup?12:17
pedronismvo: yes12:18
mvook12:18
=== ricab is now known as ricab|lunch
=== msalvatore_ is now known as msalvatore
=== ricab|lunch is now known as ricab
mupPR snapd#7987 closed: snap: make `snap version` output host without extra whitespace <⚠ Critical> <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/7987>14:32
zygare14:32
mupPR snapcraft#2868 opened: cli: implement progressive releases <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2868>14:40
mupPR snapd#7988 opened: snap: remove "host" output from `snap version` <⚠ Critical> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7988>14:42
mupPR snapcraft#2869 opened: build providers: use multipass from stable <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2869>14:49
Chipacamborzecki: wrt 596,523 hours, it's weird because nowhere do we format numbers with a comma like that14:52
Chipacamborzecki: that's not snapd :)14:52
Chipacamborzecki: in particular snapd's progress bar would've printed "68y" if it had to render that number14:54
zygaChipaca: libc may do that14:54
zygaChipaca: , is a common 1k separator14:54
Chipacawhich, sure, weird, but that's why i'm asking where that's coming from14:54
Chipacazyga: yes i know it may14:54
Chipacazyga: but we don't use libc to format how long is left for a download in snapd14:54
Chipacaso that's why i'm asking :)14:55
mborzeckiChipaca: hmm right14:56
pedronisthere's not much info there, wouldn't do anything unless they give more info14:57
mborzeckiChipaca: otoh, i should probably open a PR to tweak quantity formatting in the progress bar, sub second values are the funny ones i mentioned15:00
Chipacamborzecki: :)15:00
Chipacamborzecki: there are probably bugs (tm)15:00
mborzeckis/bugs/features/15:00
Chipacamborzecki: rather, we have yet to adequately sample the probability cloud of bugs15:01
Chipacacertainly bug(xβƒ—) > 0 βˆ€ xβƒ—15:02
roadmr🧐15:02
Chipacamborzecki: what kind of weirdness, btw?15:05
mborzeckiChipaca: when the download is close to finishing sometimes the left time reported is in us/ns range15:06
mborzeckiChipaca: as in right before it's finished15:07
Chipacamborzecki: ah, yes15:07
Chipacamborzecki: I've wondered what it goes on to do after finishing :)15:07
ijohnsonpedronis: mvo: is it safe to say that all the tests that are uc20 specific are given by function name `Test*20*` ? Using a similar regex I found these tests: https://pastebin.ubuntu.com/p/NHq7sB68dj/15:07
ijohnsondo y'all think that's exhaustive or are there other packages/test suites that I need to look at as well ?15:07
pedronisijohnson: yes, it's reasonable, but some one have the 20 in the suite15:08
pedronisnot each test15:08
ijohnsonah good point15:08
ijohnsonpedronis: so that adds these 3: https://pastebin.ubuntu.com/p/FNdtRsttBc/15:09
ijohnsonis that probably all of the uc20 specific tests ?15:09
pedronisijohnson: seems right to me15:10
ijohnsonack, thanks15:10
pedronis(I'm in a meeting)15:11
cachiomvo, this is the model I am usign https://github.com/sergiocazzolato/snapd/blob/tests-enable-nested-on-core20/tests/lib/assertions/nested-20-amd64.model.json15:24
zygabrb15:24
mvocachio: thank you! I think I have an idea what might need tweaking. I am in a meeting right now but let me try to help15:24
cachiomvo, great, thanks15:25
mvocachio: I think this needs "grade: dangerous" (or something) and a snaps: subsection, here is an example https://github.com/snapcore/snapd/blob/master/tests/lib/assertions/ubuntu-core-20-amd64.model15:26
cachiomvo, nice, thanks15:26
ijohnsonmvo: pedronis: cmatsuoka: thoughts on having modeenv.Write() also return the filepath where the modeenv was written out? this will make mocking easier in tests15:34
pedronisijohnson: still in meeting15:34
ijohnsonoh ok15:34
Chipacagrr jq in 14.04 doesn't like dashes in object keys15:41
pstolowskiChipaca: use jq snap15:51
Chipacayeah15:56
mvoijohnson: sounds fine to me15:59
ijohnsonmvo: ack, I will file a quick PR for that this morning then15:59
ijohnsonthanks15:59
pedronisijohnson: I have another meeting, but I'm a bit unclear why you need that? it writes thing in a predictable place16:01
pedronisno?16:01
ijohnsonpedronis: yes it's predictable, but in all these tests I have to call modeenv.Write(), then immediately do a filepath.Join(dirs.Root /*or c.tmpdir*/,dirs.SnapModeenvFile) to remove it after the tests16:02
ijohnsonpedronis: so it would be nice to just have `f, err := m.Write(); defer os.Remove(f)`16:02
* cachio lunch16:03
pedronisijohnson: ok, I would prefer not to do it, because usually we don't have that style16:09
pedronisijohnson: you can have a test helper if needed, you can use AddCleanup in it16:10
ijohnsonpedronis: ok16:10
pedronisijohnson: not a lot of the tests you listed have a devicemgr around16:10
pedronisijohnson: are you adding write modeenv to a lof of places?16:11
pedronisI fear I'm missing something here16:11
ijohnsonpedronis: well it's probably fine to just have the same modeenv in most tests, but other tests are not consistent about the name of the base/kernel for uc20 tests iiuc16:11
ijohnsonit's fine I guess to have a helper16:12
pedronisijohnson: ok, notice that we might need a higher level helper anyway at some point16:15
* zyga-laptop had dinner and need a moment to rest16:15
zyga-laptopcode review time!16:15
ijohnsonyes perhaps16:15
roadmrzyga's laptop had dinner? πŸ˜‹16:15
zyga-laptoproadmr sadly the laptop recharges while I'm not looking ;)16:15
pedronisijohnson: we bootloadertest.MockBootloader but it doesn't fit quite the fact that Core20 has boot state in more than one location16:15
pedronis*we have16:16
pedronisthat one has things like SetBootBase and SetBootKernel etc16:16
pedronisbut there's mismatch with Core2016:16
ijohnsonpedronis: yes I added a number of functions to the MockBootloader for the ExtractedRunKernelImage interface16:19
Zylophello, i have a lot of time for your kind reply, i get this error when installing youtube music cannot self-bind mount /run/snapd/ns: Cannot allocate memory16:29
Zylop5.5.0-1-MANJARO16:29
=== heather is now known as hellsworth
cachiomvo, hey16:33
Zylopi will check if there's a new kernel beta soon16:34
Zylopthank you16:34
cachiomvo, using snaps as you did in the other model16:34
cachioI get error: cannot decode model assertion "/home/gopath/src/github.com/snapcore/snapd/tests/lib/assertions/nested-20-amd64.model": assertion model: type of snap "snapd" must be one of app|base|gadget|kernel|core16:34
zygaZylop: hello16:34
cachiomvo, which type we should use for snapd?16:34
zygaZylop: that's interesting, I've never seen an error like that before16:34
ijohnsonzyga: Zylop: I seem to remember seeing that error on the forum recently16:36
zygaZylop: perhaps it's some kind of new hardening or bug16:36
Zylopor just downgrade to 5.316:36
Zylopits a bug in 5.4 and 5.516:36
Zylopkernel16:36
Zyloppatched in 5.5rc516:36
ijohnsonZylop: zyga: see https://forum.snapcraft.io/t/on-ubuntu-18-04-3-with-5-4-5-5-kernels-snaps-are-not-launching/14662/1816:36
Zylopwitch atm its not officially launched so im downgrading to 5.3 for while16:36
Zylopi founf this16:36
Zylopah ok16:36
Zylopty16:36
Zylopi see in arch forum16:37
* zyga l boks 16:37
zygathanks ijohnson16:37
mupPR snapcraft#2869 closed: build providers: use multipass from stable <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2869>16:37
mupPR snapcraft#2870 opened: snap: set PYLXD_WARNINGS to inhibit unknown LXD attribute warnings <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2870>16:37
mupPR snapd#7989 opened: devicestate: do not allow remodel between core20 models <Created by mvo5> <https://github.com/snapcore/snapd/pull/7989>16:47
pedroniscachio: that means the snapd is too old17:02
pedroniswe allow type: snapd now17:03
cachiopedronis, ah, ok, that makes sense17:03
cachiothanks!!17:03
=== alan_g is now known as alan_g_
Zylopworking17:18
Zylopthanks17:18
ijohnson:-)17:18
mupPR snapcraft#2871 opened: snaps: account for forwarded (effective) installation channels <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2871>17:19
ijohnsonpedronis: oh also I was going to ask you if we expect classic devices with the new uc20 models to have modeenv? currently my HasModeenv method just checks if the model is not ModelGradeUnset17:24
=== Spacehog is now known as Zylop
ograis the forum down again ?17:35
ijohnsonseems like it17:36
ograit should really be added to status.snapcraft.io17:37
ijohnson+1 for that17:37
cachiopedronis, which version of snapd is needed?17:37
ijohnsonsnapcraft.io/docs should be on status.snapcraft.io too17:37
cachioi just tried with the 2.42.1 and did't work17:38
cachiopedronis, and core 2.42.817:46
cachio2.42.517:46
cachiopedronis, with 2.43 works17:47
mupPR snapcraft#2861 closed: meta: remove Application's `prepend_command_chain` <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2861>18:05
pedronisijohnson: we don't let you set classic and grade at the same time atm, we will need to think through what a core20 model means for classic18:22
pedroniscachio: yes, it was in the 2.43 dev cycle18:25
ijohnsonpedronis: ack that's what I thought18:25
cachiopedronis, yes, the image is created now18:25
cachiopedronis, thanks for the help+18:25
grewtinnoice18:38
mupPR snapcraft#2866 closed: spread tests: limit adapter test to amd64 <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2866>19:08
mupPR snapcraft#2870 closed: snap: set PYLXD_WARNINGS to inhibit unknown LXD attribute warnings <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2870>19:08
cmatsuokazyga: could you have a quick look at https://bugs.launchpad.net/ubuntu/+source/snapd/+bug/1859160 to see if it's something you recognize?19:15
mupBug #1859160: Snapd fails to delete obsolete dev-loops, thus causing delay at lockscreen <snapd (Ubuntu):New> <https://launchpad.net/bugs/1859160>19:15
mupPR snapcraft#2858 closed: add support for system-usernames <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2858>19:23
mupPR snapcraft#2872 opened: Release/3.9.5/cherry picks <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2872>19:29
mupPR snapcraft#2867 closed: elf: remove return parameters for ElfFile's _extract() <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2867>20:23
mupPR snapd#7990 opened: many: misc tweaks <Simple πŸ˜ƒ> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7990>20:40
mupPR snapd#7947 closed: boot,bootloader: support new UC20 style kernel extraction <UC20> <β›” Blocked> <Created by anonymouse64> <Closed by anonymouse64> <https://github.com/snapcore/snapd/pull/7947>20:48
mupPR snapd#7991 opened: boot: add HasModeenv to Device <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7991>20:56
mupPR snapcraft#2872 closed: Release/3.9.5/cherry picks <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2872>21:11
mupPR snapcraft#2868 closed: cli: implement progressive releases <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2868>21:53
mupPR snapd#7992 opened: bootloader: add ExtractedRunKernelImageBootloader interface, implement in grub <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7992>22:13

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