mupPR snapd#8902 closed: tests: fix assertion disk handling for nested UC systems <Squash-merge> <Test Robustness> <Created by anonymouse64> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/8902>00:05
=== jamesh_ is now known as jamesh
zygaGood morning05:39
zygaIโ€™ll start at 905:40
zygaJust checking mail in the morning05:40
mborzeckizyga: hey05:41
zygaMy keyboard is already in Warsaw05:43
zygaMaybe it will be delivered a day earlier05:43
zygaOh 2.45 has regressed Snapcraft and Ubuntu image05:44
mborzeckizyga: hmm? bugs?06:52
mborzeckizyga: do you know what broke?06:52
zygaI will look now06:56
zygaI just got mail that autopkgtest regressed06:57
zygathe log is rather useless07:00
zygait says07:00
zygadeb2snap             FAIL non-zero exit status 107:00
zygaearlier there's this warning07:01
zyga[1mWarning:[0m /snap/bin was not found in your $PATH. If you've not restarted your07:01
zygahey pawel07:01
zyga(along with ansi escapes, yuck)07:01
zygamaybe something did regress on path?07:02
zygalet's look at the other regressions07:02
zygathere's a rather silly regression in snapd07:04
zyga+ snap install --edge test-snapd-hello-multi-arch07:04
zygaerror: snap "test-snapd-hello-multi-arch" is not available on edge for this architecture (i386) but exists on other architectures (amd64).07:04
mvozyga: wuut? how can that happen07:05
zygagood morning mvo!07:05
zygaI was about to ask that :)07:05
mvozyga: oh, is that on 20.04 where i386 got removed?07:05
mvozyga: good morning :)07:06
zygano, this is on 19.1007:06
zygamaybe something go removed wrongly07:06
zygaoh oh07:06
zygamy keyboard is coming now07:06
zygamvo: this snap is multi-arch in the sense that it contains i386 binary inside amd64 contianer07:08
zygaso it should work on 20.04+07:09
zygathere's only one revision07:10
zygaso whatever broke didn't involve the snap changing07:10
mvozyga: woah07:14
mvozyga: very strange07:14
zygamvo: does snapcraft error make any sense to you07:14
zygaI'm trying to make some sense out of the log07:14
mvozyga: you mean [1mWarning:[0m /snap/bin was not found in your $PATH. If you've not restarted your ?07:15
zygathat seems like a warning that's earlier up the log07:15
zygathe log ends with:07:15
zygadeb2snap             FAIL non-zero exit status 107:15
zygawhat is deb2snap?!07:15
zygathe whole part reads:07:16
mvozyga: it's some sort of helper iirc to transition debs to snaps, iirc it started with chromium07:16
zygamaybe version changed :D07:16
* zyga checks07:16
mvozyga: I bet07:16
mvozyga: the test failure does not make much sense - also autopkgtest07:16
mvozyga: ha!07:17
zygacurrent version is 4.0507:17
zygaso where is the test that needs fixing07:17
* zyga looks07:17
mvozyga: this is a silly test then - will you tell sergiusens about "autopkgtest [00:57:26]: test deb2snap: [-----------------------" failing because the version number changes :) ?07:17
zygasure :)07:17
pedronismvo: it seems we broke the snap-auto-mount test on master for core2007:25
mborzeckipedronis: do you have logs?07:26
mborzeckipedronis: nvm, it failed in my pr too07:26
pedronismborzecki: yes, as I said seems to be really broken, failing consistently07:26
mborzeckiwas this modified recently?07:27
mborzeckii did some changes to coldplug auto-import, but it was back in may iirc07:28
mborzeckiand debugging is broken too, there's no /var/log/syslog :/07:30
pedronismborzecki: not sure, but maybe some of the test assertions were changed07:30
pedronisfor other tests07:30
mvoyeah, wonder why it's happening now. maybe something else changed too?07:30
mvonew udev or systemd in core20 maybe?07:30
zyganew systemd!07:30
=== jamesh_ is now known as jamesh
mupPR snapd#8913 opened: tests/core/snap-auto-mount: fix debug section <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8913>07:37
mborzeckilet's see if there's any useful output07:37
* mvo is in a meeting but will follow from the sidelines07:37
pedronis#8889 needs a 2nd review07:55
mupPR #8889: snapstate: fix autorefresh from classic->strict <Bug> <Needs Samuele review> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8889>07:55
mborzeckiheh, the test runs succesfuly in isolation08:04
zygamborzecki: updated https://github.com/snapcore/snapd/pull/891008:05
mupPR #8910: usersession: support additional zoom URL schemes <Bug> <Needs security review> <Simple ๐Ÿ˜ƒ> <Created by zyga> <https://github.com/snapcore/snapd/pull/8910>08:06
zygamvo: https://github.com/snapcore/snapd/pull/8889#pullrequestreview-43642013208:11
mupPR #8889: snapstate: fix autorefresh from classic->strict <Bug> <Needs Samuele review> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8889>08:11
mborzeckizyga: lucky us that zoom has no more url schemas ;P08:14
zygamborzecki: yet :D08:16
zygakeyboard is here08:16
mborzeckizyga: maybe you could reference https://github.com/snapcore/snapd/pull/8761#pullrequestreview-425508154 for the msteams://08:17
mupPR #8761: usersession/userd: add msteams url support <Needs security review> <Created by call-a3> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8761>08:17
zygamborzecki: sure, I was looking for it08:25
zyganeed a moment to adjust08:26
zygaso weird08:30
zygaso weird and comfortable08:31
zygagetting used to the layout will take a momentt08:32
zygathere are four layers to lean08:32
mborzeckizyga: it's 2 separate pieces right?08:33
mborzeckiwireless in between or wired?08:33
zygawired with a detachable cord08:33
mborzeckizyga: and keyboard -> pc wired too?08:33
zygaRJ11 phone connector08:34
zygait has a custom app08:34
zygaFOSS, they provide a appimage08:34
mborzeckiwe need a snap then :)08:35
mborzeckibut wow, rj11, haven't seen those in a while08:35
zygait's actually practical08:35
zyganot custom08:36
zyganot USB08:36
zygamouse layer!08:38
zygaok, somewhat adjusted now08:57
zygaapart from annoyance when my muscle memory doesn't match08:57
zygathere is no strain now08:57
mvozyga: thank you, lookin gnow09:05
zygafeel free to push this to me09:05
mupPR snapd#8702 closed: overlord/configstate: add system.kernel.printk.console-loglevel option <Created by EthanHsieh> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/8702>09:22
mupPR snapd#8914 opened: o/ifacestate: fix connect undo handler <Bug> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8914>09:22
zygamborzecki: I've added the reference to msteams pull request now09:38
mborzeckizyga:  thanks!09:38
* zyga small break to move around a little09:38
zygaI need to focus on https://github.com/snapcore/snapd/pull/8881 now09:44
mupPR #8881: interfaces: optimize rules of multiple connected iio/i2c/spi plugs <Bug> <Needs security review> <Created by zyga> <https://github.com/snapcore/snapd/pull/8881>09:45
zygawhy do my PRs get 50+ comments ;-)09:45
mborzeckipedronis: zyga: got some debug logs in snap-auto-mount PR https://github.com/snapcore/snapd/pull/8913/checks?check_run_id=80267363810:10
mupPR #8913: tests/core/snap-auto-mount: fix debug section <Simple ๐Ÿ˜ƒ> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8913>10:10
mborzeckimeh: 2020-06-24T09:38:28.0114940Z + is_core_nested_system10:12
mborzecki2020-06-24T09:38:28.0115148Z /home/gopath/src/github.com/snapcore/snapd/tests/lib/nested.sh: line 158: NESTED_TYPE: unbound variable10:12
zygagood old shell10:12
mborzeckiduh, how come some of the code runs under -u and other parts do not10:13
mborzeckiset -u10:14
mborzeckihm spread does not set -u for the test scripts, so how did it end up being set there?10:15
zygaIIRC there was something similar10:16
zygawhen we learned that set -e is easily lost10:16
zygawith stuff as simple as ( )10:16
mborzeckizyga: there was some subshell weirdness combined with () and {}10:16
mborzecki2020-06-24T09:38:27.4134740Z Jun 24 09:38:27 ubuntu snapd[2470]: daemon.go:313: DEBUG: pid=2735;uid=0;socket=/run/snapd.socket; POST /v2/assertions 10.868987ms 20010:16
mborzecki2020-06-24T09:38:27.4135673Z Jun 24 09:38:27 ubuntu snapd[2470]: daemon.go:313: DEBUG: pid=2735;uid=0;socket=/run/snapd.socket; POST /v2/users 446.144ยตs 20010:16
mborzeckiso importing assertions was succesful?10:17
zygamborzecki: don't you love our state-sharing shell functions10:17
pedronismborzecki: some our .sh set set -u10:17
zygathat's why I don't want to see more shared state shell code10:17
mborzeckipedronis: oh, right sourcing those can mutate the state :/10:17
pedronisisn't this prepare.sh anyway?10:17
pedronisit has a set -eux10:18
zygaonly executed programs are possible to reason about10:18
mborzeckipedronis: the unbound error popped up in debug section10:18
mvozyga: I replied to #888910:19
mupPR #8889: snapstate: fix autorefresh from classic->strict <Bug> <Needs Samuele review> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8889>10:19
mborzeckipedronis: iirc those were separate shell instances10:19
zygathanks mvo10:19
mvothank you!10:19
pedronismborzecki: not sure, the log is a bit confusing10:19
* pstolowski lunch & errand10:20
ijohnsonmorning folks10:20
ijohnsonsorry mborzecki10:20
ijohnsonthat NESTED_TYPE was my fault :-(10:20
zygamvo: replied10:21
zygagood morning ijohnson!10:21
mborzeckiijohnson: no worries and good morning10:21
clmsyhi, quick question, there are some packages that their snaps dont exist, such as dracut, clevis etc (things i need to implement full disk encryption on ubuntu core 16 based image).  I decided to do stage-packages into the gadget snap and when the machine boots i could reorg files under /etc /usr/lib /usr/bin etc but i ofc forgot for a moment that its read-only for security10:22
clmsyam i completely bound to the fact that i need these packages as snaps so they can be used by the system, i cant just prime them into the gadget than try to move them under main paths like /usr/..10:23
ijohnsonclmsy: iiuc trying to use dracut on an ubuntu-core system will not work, as the initramfs is in the kernel snap and that is read-only10:23
ijohnsonclmsy: yes you cannot add things to the root filesystem that is not already in the base snap (which for ubuntu core 16 is the core snap)10:23
ijohnsonclmsy: that being said some directories are writable and can be modified10:23
ijohnsonclmsy: see writable-paths man page on ubuntu10:23
zygaclmsy: squashfs is pretty much read only and it is unrelated to security10:23
mborzeckiwas dracut even used for uc16?10:24
ijohnsonmborzecki: I don't think so10:24
* zyga small break10:24
clmsydoes this mean guys that i am bound to wait for ubuntu core 20 for full disk encryption that i just cannot implement it for 1610:24
ijohnsonclmsy: it is implementable for uc16, we have had commercial customers for whom Canonical Field Engineering team implemented full disk encryption, but I don't recall more details than that10:25
zygaclmsy: on which architecture?10:25
ijohnsonclmsy: if you are interested in talking to our Field engineering team, I would refer you to ogra10:25
clmsywe are using this https://fit-iot.com/web/products/fitlet2/10:25
clmsyintel atom 64bit10:25
zygaif it has TPM I would really go after core2010:26
clmsyit has tpm yes10:26
zygait's probably easier than retrofitting this into uc1610:26
clmsyi should also email ogra maybe to take his opinion10:26
zygait's a lot of work10:26
ijohnsonclmsy: does it have tpm 2.0 ?10:26
zygareally nice box btw!10:27
ijohnsonclmsy: also yes what zyga said, we designed uc20 with full disk encryption in mind so that it's easier than it used to be for ubuntu core 16 and 18 to implement full disk encryption10:27
ijohnsonclmsy: also if ogra isn't around, you could also reach out to lool10:27
zygaand uc20 based system can install and use core16 and core18 apps10:27
zygaso you don't lose anything10:28
zygaanyway brb10:28
clmsyi really appreciate your feedback guys, i ve been fiddling and fighting to make it work for the last week10:28
clmsyis there a generic email addr for field engineering team at canonical i can send an email to10:28
mborzeckiclmsy: if you have the box around, you can try uc20 beta (just saying)10:28
ijohnsonclmsy: yes, let me get email that for you, one minute10:29
ijohnsonclmsy: also it seems that device has tpm 2.0 through the intel firmware10:29
ijohnsonso it should just work with uc20 beta: https://docs.ubuntu.com/core/en/releases/uc2010:30
clmsyi actually implemented uc20 beta10:30
clmsyhaving an issue there10:30
clmsyi used the baseline from snapcore10:30
clmsyin the gadget.yaml file under the repo snapcore10:31
clmsya volume is defined as ubuntu-boot, but when i bot it says no such device etc10:31
clmsyill try keep working on it10:31
clmsylet me get an email addr i would really like that10:31
ijohnsonclmsy: the ubuntu-boot missing on first boot is expected, it should continue to boot past that though10:32
ijohnsonclmsy: do you have uefi enabled on the device ? it seems relatively new so I assume so10:32
clmsyuefi is enabled, secure boot is also enabled, it does continue to recovery boot but than you know what happens, the-tool service fails with exit code (1) and says the following: model does not specify all the requestted essential snaps: [base kernel snapd]10:33
clmsyand the model actually does specify all of them10:33
clmsyi actually send an email to a canonical contact of ours10:33
clmsyOndrej Kubik10:33
clmsywith all the details and screenshots, and even our model json file10:33
clmsyhe is the only contact i know10:34
ijohnsonclmsy: can you show your model assertion snaps header ? (and sorry I'm still waiting trying to figure out an email to give you)10:34
ijohnsoni.e. the bit that has `snaps: [...]`10:35
clmsyyes 1 sec10:36
clmsythis is the snaps map: https://pastebin.com/Z49qbDVr10:37
clmsyon top of the model base is core20 and grade is secured10:37
ijohnsonclmsy: ah you are hitting this bug: https://bugs.launchpad.net/snapd/+bug/188397310:39
mupBug #1883973: cannot boot uc20 model with multiple base snaps <snapd:Fix Committed by pedronis> <https://launchpad.net/bugs/1883973>10:39
ijohnsonclmsy: unfortunately the fix that bug has not yet been released, but you can kinda fix it locally until it gets released to the pc-kernel10:39
ijohnsonclmsy: also I would suggest using latest/beta for all your snaps for now, rather than latest/stable, as right now all the stable snaps for uc20 are not quite stable, I mean they should be good, but we are much more actively testing the beta snaps right now rather than the stable ones10:40
clmsyoooh that seems to be the bug yes10:41
clmsythank you for letting me know! i really appreciate it... i was trying a lot of different things to see if i can do a workaround10:41
clmsyim glad this irc channel exists ^_^10:41
ijohnsonclmsy: give me a minute to get you a script that you can use to fix the kernel snap10:41
ijohnsonclmsy: ok so download this script from this gist: https://gist.github.com/bboozzoo/010ed5e94ee0f695d1aeece43513a01810:47
mupPR snapd#8889 closed: snapstate: fix autorefresh from classic->strict <Bug> <Needs Samuele review> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8889>10:47
ijohnsonthen use this script to rebuild the kernel snap (which uses that script): https://pastebin.ubuntu.com/p/J7TXCMrdYZ/10:48
ijohnsonclmsy: ^10:48
clmsythank you very much for the help!!!!10:50
ijohnsonclmsy: also to contact field engineering please reach out to sales@canonical.com and you can mention that you were referred there by snapd developers on #snappy IRC channel10:50
clmsyperfect!! most helpful10:50
clmsyi'll get to working on this issue10:50
ijohnsonyes please reach out again if you are having more trouble with uc20, we are happy to help and welcome any feedback/bugs you have about it :-)10:52
zygaback to fixes10:59
mborzeckiEighth_Doctor: hey, i see you've started pushing out updates, thank you!10:59
mupPR snapd#8915 opened: gadget: do only one mount point lookup in mounted fs updater <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8915>11:27
mborzeckiijohnson: pushed a fix for NESTED_TYPE to #891311:31
mupPR #8913: tests/core/snap-auto-mount: fix debug section <Simple ๐Ÿ˜ƒ> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8913>11:31
ijohnsonthanks mborzecki, approved11:33
* zyga is slow today, sorry, learning curve 11:48
mborzeckihm maybe snap-auto-mount is racy, but afaict (pedronis please correct me), by the time the POST /api/v2/assert request returns all processing is done12:07
mborzeckior it's another another udev settle/async thing that pops up randomly12:07
pedronismborzecki: yes, should be done12:15
NerdSnappero/ need help with youtube-dl snap, seems to be an outdated version not working anymore. can i manually update the binary in the snap packet?12:23
mupPR snapd#8901 closed:  tests/lib/tools: apply linger workaround when needed  <Precious Logs> <Simple ๐Ÿ˜ƒ> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8901>12:58
mupPR snapd#8908 closed: overlord/snapstate: graceful handling of denied "managed" refresh schedule <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8908>12:58
NerdSnappersorry if this is the wrong place to get help with a outdated snap package... where can I talk to?13:00
mborzeckiNerdSnapper: https://github.com/joedborg/youtube-dl is listed in contact-url, i suggest you try opening an issue there13:01
mupPR snapcraft#3187 opened: cli: use maxval of UnknownLength for pack progress <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3187>13:02
mupPR snapd#8887 closed: bootloader: pull recovery grub config from internal assets <UC20> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/8887>13:03
cjp256I  created a content snap mirroring gnome 3.x platform ones.  it's supposed to share "$SNAP" or "/", like the gnome snap, but when it surfaces inside of the snap, I get the content's snap revision number as the first level directory?  Is that supposed to happen?13:06
cjp256(e.g. x4)13:07
zygacjp256: hi, can you show me the plug and slot definition please13:17
cjp256zyga: ^^ thanks for looking :D13:21
zyganote that in all the cases, / maps to $SNAP, right13:23
cjp256I would expect so?  the gnome-platform uses "/"13:23
zygasure, just stating that so it's not unclear13:24
NerdSnappermborzecki: if a snap package (e.g. youtube-dl) is outdated, who provides an update for snap? the original software seems to be actual and working fine, only the snap version is old...?13:24
mborzeckiNerdSnapper: whoever published the snap13:24
zygacjp256: I think I know what's going on13:25
zygacjp256: in a call, I'll get back to you13:26
cjp256zyga: k thanks13:26
zygaso, the content interface has two modes of operation13:28
zyga1:1 and 1:n13:28
NerdSnappermborzecki: OK I try to find out...13:30
mborzeckiNerdSnapper: if you run `snap info <name>` it lists a contact-url for the snap publisher13:30
zygathe former uses the target directory and replaces it with the connected content; the latter uses the target directory to enunerate the connected slots13:31
zygathe latter mode uses the directory name of the source - if you share the whole snap you will get the revision number13:33
zygaif you want to use the 1:1 mode instead please drop the "source" component from the syntax13:33
zygai think this is documented13:35
zygalet me know if this aspect is unclear please13:36
zygasergiusens: hi13:36
NerdSnappermborzecki: great thank you, you helped me understand it, that was the contact-url you mentoined... ;)13:36
zygasergiusens: we got a SRU regression on a test that checks snapcraft version to be 3.*13:36
mupPR snapd#8891 closed: o/servicestate: add updateSnapstateServices helper (5/9) <Needs Samuele review> <Services โš™๏ธ> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/8891>13:38
NerdSnappermborzecki: I see I am not the only one with this issue :)  with youtube-dl13:39
joedborgNerdSnapper: hi there, i've just checked the snap.  due to a merge conflict from upstream, the automatic builds had stopped and gone unnoticed.  it's now back up and there'll be a new build very soon.13:45
NerdSnapperjoedborg: thumbs-up great! thank you very much13:48
cjp256zyga: makes perfect sense as you explain it.  and it works now! I don't think that's an obvious in the documentation, and it feels like an error case maybe worthy of a snap warning? I'm not sure how the consumer snap would account for a unknown revision tag unless wildcarding?  Docs say source is `presents one or more sub-directories` which is true, but doesn't elaborate on this case as to a potential difference in13:48
cjp256behavior for a single shared directory.  Thanks for helping me out!13:48
sergiusenszyga: hi, Snapcraft is on version 4 now14:00
sergiusensbut I guess you know that14:00
sergiusensSRU regression where?14:01
mupPR snapd#8916 opened: tests: adding ubuntu-20.04 to google-sru backend <Simple ๐Ÿ˜ƒ> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8916>14:03
zygasergiusens: Iโ€™m afk for a moment but I will share the SRU link when I am back14:06
=== Son_Goku is now known as Conan_Kudo
=== Conan_Kudo is now known as Son_Goku
NerdSnappero/ wishing you all a nice evening...14:24
=== benfrancis2 is now known as benfrancis
=== Son_Goku is now known as Eleventh_Doctor
=== Eleventh_Doctor is now known as Son_Goku
jdstrandogra: hi! I still have a bbb kicking but lately I've been seeing: snapd[1122]: taskrunner.go:271: [change 1688 "Request device serial" task]14:29
jdstrandfailed: cannot deliver device serial request: Cannot process serial request for device with brand14:29
jdstrand"QfOqF7d2M1Pk2O0SbEKqTdB9Ry2aI0BP" and model "bbb"14:29
jdstrandogra: have you seen this?14:29
mborzeckierrands, bbl14:30
ijohnsonjdstrand: that's expected unless you have `serial-authority: [generic]` in your model assertion14:30
ograjdstrand, well, this device doesnt use a canonical model14:30
ijohnsonjdstrand: or unless you have a brand store for your bbb :-)14:30
ograbeing core16 i dont think there are ways to make it obtain one14:30
ogra(one = serial)14:30
ogracore20 allows the new "generic" serial14:31
ijohnsonogra: you could try rebuilding that image with a model that uses serial-authority14:31
ograprobably time that i dust off my bbb and create a core20 img14:31
ijohnsonogra: also serial-authority: [generic] is allowed on any uc model, not just uc2014:31
ograoh, i didnt know that !14:31
ijohnsonyes it's a generic feature of model assertions now14:31
jdstrandijohnson: ok, well, this is a new, somewhat noisy message14:32
ograi'll play with it ..14:32
ijohnsonthe appliance images which are all uc18 use this14:32
ijohnsonjdstrand: it shouldn't be new though ...14:32
jdstrandnewish? I feel like I've only started seeing it in my logs. maybe the message changed and my filter isn't catching it...14:32
ograwell, i think snapd used to be quiet about it ... apart from an error in snap changes14:32
jdstrandok, yes, the message changed14:35
jdstrandit used to say: cannot deliver device serial request: Cannot process serial request for device with brand "QfOqF7d2M1Pk2O0SbEKqTdB9Ry2aI0BP", store can sign serial only for brand "canonical"14:36
* jdstrand updates logcheck14:36
=== Son_Goku is now known as Conan_Kudo
=== Conan_Kudo is now known as Son_Goku
=== Son_Goku is now known as Conan_Kudo
=== Conan_Kudo is now known as Son_Goku
pedronismaybe the store error status changed a bit and the effect is different15:13
pedronisI would need to check15:13
pedronisif the http status code is the same the behavior shouldn't be much different15:14
ijohnsoncachio: can you explain the purpose or intent of this line in prepare.sh: https://github.com/snapcore/snapd/blob/master/tests/lib/prepare.sh#L28815:15
ijohnsoncachio: I keep seeing google-nested:...:tests/nested/classic/ fail to prepare from this line15:16
ijohnsonand it's unclear to me why we need that line, since we install the core snap below, why does it matter that we didn't have the core snap before we go to install it?15:16
pedronisijohnson: I think mvo added that because we had this images that sometimes have core and sometimes don't15:17
pedronisand if core is there you get race errors15:17
ijohnsonpedronis: no that line is much older15:18
cachioijohnson, it is because we need to make sure that the env is clean15:18
ijohnsonor rather https://github.com/snapcore/snapd/commit/4ba6181f9b4ea724752f48114a4a263897d4197a actually15:18
pedronisijohnson: ah, it's prepare classic, not core15:18
ijohnsoncachio: so why don't we instead try to just purge snapd from the system instead of just failing the test if the core snap is there?15:19
ijohnsonI don't know why the core snap is there but it feels like every run I will get the core snap there15:19
cachioijohnson, so, you face that issue when reuse your system right?15:19
cachioor when using a clean env?15:19
ijohnsoncachio: yes it happens when the system is reused afaict15:20
ijohnsonerr rather after the system was used for a test, restore runs, then the next test prepare runs and fails like this because the core snap is there15:20
cachioijohnson, in this case we need to make something to clean up better when we restore15:20
cachiobecause the idea is to make sure the env is clean15:21
ijohnsoncachio: ok, I will try something locally to see if it fixes the problem in restore and propose today15:21
cachioijohnson, perhaps just removing snapd is enough15:22
cachioremove --purge15:23
ijohnsoncachio: that is what I was going to try15:23
cachioijohnson, nice, thakns!!15:23
cachioijohnson, just ping me, I'll review it15:23
cachioafter lunch15:24
ijohnsoncachio: sounds good15:24
* cachio lunch15:46
mborzeckiheh 2020-06-24T13:49:05.9507944Z 2020-06-24 13:49:05 Cannot allocate google:ubuntu-14.04-64: cannot perform Google request: Get https://www.googleapis.com/compute/v1/projects/computeengine/zones: oauth2: cannot fetch token: 500 Internal Server Error15:54
zygagoogle ran out of ram15:58
mupPR snapcraft#3187 closed: cli: use maxval of UnknownLength for pack progress <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3187>16:02
mupPR snapd#8916 closed: tests: adding ubuntu-20.04 to google-sru backend <Simple ๐Ÿ˜ƒ> <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8916>16:29
mupPR snapd#8913 closed: tests/core/snap-auto-mount: try to make the test more robust <Simple ๐Ÿ˜ƒ> <Test Robustness> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/8913>16:39
* zyga takes a break,17:01
zygatomorrow will be a more productive day17:01
mupPR snapd#8835 closed: POC:  skip binary to skip tests in an easy way <Proof of concept> <Skip spread> <โ›” Blocked> <Created by sergiocazzolato> <Closed by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8835>17:24
* cachio -> kinesiologist17:43
ograLukewh, did our metrics stop working or did my zoom-client snap simply hit a threshold ? the numbers havent changed i a week now17:56
ogra*in a week17:56
=== benfrancis0 is now known as benfrancis
mupPR snapd#8917 opened: osutil/disks: add mock disk and tests for happy path of mock disks <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8917>19:39
mupPR snapcraft#3188 opened: snap: allow for different compressions for pack <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3188>20:18
mupPR snapd#8918 opened: many: make nested spread tests more reliable <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8918>20:40
mupPR snapd#8919 opened: gadget/install,secboot: test if the tpm can be provisioned <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/8919>21:00
mupPR snapcraft#3118 closed: plugins: add support for local v2 plugins (core20) <enhancement> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3118>21:38
mupPR snapd#8920 opened: interfaces: update cups-control and add cups for providing snaps <โ›” Blocked> <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8920>21:55
secretfaderIs there a currently recommended method for self-hosting a snap store?23:12
secretfader(I tried searching the forum, but only came back with posts that seem outdated.)23:12
oerheksone can publish private snaps on snapcraft.io23:17
oerheksbut your own  or 'offline' snapstore, no.23:18
oerheksone can download snaps and store them, ofcourse23:18
oerheks(correct me if i am wrong)23:19
secretfaderoerheks: Thanks for the insight. That's what I understand as well, but hoped I was wrong. (I'd like to investigate p2p distribution methods, in addition to official server/client techniques.)23:26

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