/srv/irclogs.ubuntu.com/2019/08/30/#snappy.txt

mupPR snapcraft#2694 opened: multipass: fix unexpected FileNotFoundError exception in setup_multipass <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2694>00:20
mupPR snapd#7376 closed: image,o/devicestate,seed: oops, make sure to clear seedtest helpers <Created by pedronis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7376>05:49
mupPR snapd#7368 closed: cmd/snap,image,seed:  move image.ValidateSeed to seed.ValidateFromYaml <Created by pedronis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7368>05:51
mupPR snapd#7373 closed: overlord/snapstate: revert track-risk behavior change and validation on install <Needs Samuele review> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7373>05:53
mupPR snapd#7377 opened: patch: add an overlord patch that normalizes channel in state <Created by mvo5> <https://github.com/snapcore/snapd/pull/7377>06:22
mupPR snapd#7372 closed: overlord/snapstate: revert track-risk behavior and change validation on install for 2.41 <Needs Samuele review> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7372>06:32
mvohey pedronis ! good morning. I'm releasing 2.41 now (just fyi)07:04
mvopedronis: also - 7067, should this go in or do you want to work on the feedback first?07:10
pedronismvo: I'll work on feedback first, it's not urgent07:11
=== pstolowski|afk is now known as pstolowski
pstolowskimornings07:11
mvopedronis: ok07:15
mvopstolowski: good morning. thanks for your reverts07:15
mvopstolowski: its all in and ready now, 2.41 should be in beta soon (just fyi)07:16
pstolowskimvo: sure, np!07:16
zygagood morning, sorry for being gone yesterday07:23
* zyga jumps into coding07:24
mupPR snapd#7378 opened: release: 2.41 <Created by mvo5> <https://github.com/snapcore/snapd/pull/7378>07:33
* pstolowski is doing some bug triaging08:23
pstolowskilove this one https://bugs.launchpad.net/snapd/+bug/182922908:46
mupBug #1829229: league of legends snap crashes on entering in the game <snapd:New> <https://launchpad.net/bugs/1829229>08:46
Chipacazyga: https://forum.snapcraft.io/t/hard-coded-absolute-paths/12987?u=chipaca08:54
zygalooking08:54
zygathanks08:56
zygabrb09:07
zygamvo: I fixed https://github.com/snapcore/snapd/pull/7370 - I'll push the fix after a run of local tests09:08
mupPR #7370: tests: fix ephemeral mount table in left over by prepare <Test Robustness> <Created by zyga> <https://github.com/snapcore/snapd/pull/7370>09:08
mvozyga: thanks09:08
mupPR snapd#7379 opened: tests: add version-tool for comparing versions <Created by zyga> <https://github.com/snapcore/snapd/pull/7379>09:38
* zyga needs coffee09:39
mupPR snapd#7313 closed: many:  add the start of Core 20 extensions support to the model assertion <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/7313>09:40
pedronismvo: got John +1 ^09:40
mupPR snapd#7380 opened: debian: set GOCACHE dir during build to fix FTBFS on eoan <Created by mvo5> <https://github.com/snapcore/snapd/pull/7380>09:41
mvopedronis: \o/09:41
* zyga looks at https://github.com/snapcore/snapd/pull/721809:51
mupPR #7218: tests: measure behavior of the device cgroup <Created by zyga> <https://github.com/snapcore/snapd/pull/7218>09:51
=== Raboo_ is now known as Raboo
* pstolowski lunch break10:42
zygauh, soo hot10:52
ChipacaI like how go's testing package avoids 'testing.Testing' by just calling it 'testing.T'11:09
Chipacawhat can I call mockbootloader.MockBootloader? :-)11:09
zygaChipaca: could ve testing.It11:10
zygaChipaca: mockbootloader.Bootloader seems more appropriate as a type though11:10
Chipacamockbootloader.Er11:11
zygaoh oh11:11
zygamockbootload.Er11:11
zyga;D11:11
* zyga hides11:11
Chipacamockbootloader.WhoYaMockin11:11
Chipacamockbootloader.YouTalkingToME11:11
* Chipaca stops going down that one11:11
mupPR snapd#7381 opened: seed,image,o/devicestate: extract seed loading to seed/seed16.go <Created by pedronis> <https://github.com/snapcore/snapd/pull/7381>11:12
* Chipaca commits the boring one and goes on to the next chunk11:12
pedronisthat is a big PR11:13
* Chipaca looks11:14
Chipacapedronis: is there nowhere non-toplevel for 'seed'?11:14
pedronisChipaca: is not new to that PR11:15
pedronisalso it's fairly central11:15
Chipacaah i hadnt noticed11:15
pedronisif we start worrying about top level packages11:15
pedronisI would do something about some other ones11:15
Chipacai do :) but it's not a new worry11:15
Chipacathat's part of the reason for this refactor chunk11:15
pedronisanyway what I said11:16
pedronisChipaca: we can have a chat about your worry (in general) at some point11:16
Chipacak11:16
zygahmmmm11:38
zygamborzecki will be happy next week11:38
zygawoah11:41
zygaI'm so happy11:41
zygamvo: I think by and large our cgroup woes are over11:42
zygamvo: some actual work needed but conceptually we're going to have a good and clean way to handle our needs11:43
ijohnsonmorning folks11:54
pedronisijohnson: hi, I'm pushing some small simplications to #714911:59
mupPR #7149: cmd: add snap model command; daemon: add /v2/model, /v2/model/serial REST APIs <Remodel :train:> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7149>11:59
ijohnsonhi pedronis11:59
ijohnsonsounds good11:59
pedronisChipaca: ijohnson: can you give a quick look at my last change to snap model: https://github.com/snapcore/snapd/pull/7149/commits/1a679f945673dd6a864a146f00908cffb1a4ddc812:10
pedroniswhen you have time12:10
mupPR #7149: cmd: add snap model command; daemon: add /v2/model, /v2/model/serial REST APIs <Remodel :train:> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7149>12:10
ijohnsonpedronis: sure I can look now12:11
pedronisijohnson: I also tweaked separely api_model_test.go, please look what I did there12:11
pedronisfyi12:12
ijohnsonack12:12
* Chipaca needs to go see about lunch12:14
ijohnsonpedronis: reviewed, I think a couple comments around the test behavior would be helpful12:21
ijohnsonI need to finish breakfast actually, bbiab12:21
mvozyga: nice, great to hear12:26
=== ricab is now known as ricab|lunch
=== pleia2_ is now known as pleia2
=== ricab|lunch is now known as ricab
diddledanwhy do I always read "ack" as "bah"-like, aka a moan or sign of annoyance, instead of "acknowledged"?!13:41
roadmrdiddledan: it sounds to me like a bird skwawking or something13:42
diddledanack13:42
diddledan:-p13:42
roadmr🐦13:45
diddledan🐧13:46
roadmrohai tux13:46
diddledan:-)13:46
* roadmr checks calendar... yup, it's Friday13:46
diddledanyey!13:46
diddledantwo new streaming TV series that I've been anticipating have landed today! (Carnival Row on Amazon Prime in the UK at least, and The Dark Crystal on Netflix)13:47
roadmrdark crystal? nice!13:48
diddledanI guess my evenings are tied up for the foreseeable :-p13:48
zygaChipaca: so13:48
diddledanThe Dark Crystal Trailer: https://youtu.be/WwcvJTVnUgc13:49
Chipacazyga: so13:50
roadmrthanks diddledan ! /me gets out his skeksis tunic and starts making chamberlain skwawks13:50
zygaChipaca: I'll make something cold to drink13:53
Chipacazyga: my main issue is I can't tell whether you're +1'ing with nits, or what13:54
Chipacazyga: your review was wishy-washy :-)13:54
diddledanthinking about Netflix, makes me wonder what we're missing in the Chromium snap to support it. Amazon also complains.14:16
zygaChipaca: +114:17
zygaChipaca: +2 even14:17
zygaChipaca: we can  iterate, that's always better unless there's a clear bug in the  code14:17
Chipacadiddledan: does chromium work outside of the snap?14:17
Chipacadiddledan: i thought it was widevine14:17
diddledanI've not tried that14:17
Chipacai.e. secret google sauce14:17
diddledanlet me give it a go14:17
* diddledan apt installs and gets confused at having two chromiums :-p14:18
* Chipaca wins14:18
zygaChipaca: I'm going for lunch now14:18
Chipacazyga: buen provecho14:18
diddledanok yeah, it fails outside, too14:20
diddledanhmm :-(14:20
diddledan@Wimpress get googlie onboard to publish the real chrome :-p14:21
Chipacawooo14:21
* Chipaca merges14:21
mupPR snapd#7185 closed: boot, etc.: refactor boot to have a lookup with different imps <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/7185>14:21
Chipacazyga: I'll address some of your concerns in the followup that i'm trying to wrap up14:21
diddledaninteresting: https://packages.debian.org/stretch/chromium-widevine14:24
pedronispstolowski: I did a high-level pass on #735514:28
mupPR #7355: [RFC] interfaces/apparmor: load multiple profiles in a single batch <Complex> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7355>14:28
=== vorlon` is now known as vorlon
mupPR snapd#7378 closed: release: 2.41 <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7378>14:41
mupPR snapd#7380 closed: debian: set GOCACHE dir during build to fix FTBFS on eoan <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7380>14:41
zygaChipaca: sounds great14:49
* Wimpress adds a task to my list for diddledan 15:02
pstolowskipedronis: ty15:02
diddledan\o/15:02
pstolowskizyga: was this fixed / understood? https://bugs.launchpad.net/snapd/+bug/183272515:02
mupBug #1832725: snap refresh failure <snapd:New for zyga> <https://launchpad.net/bugs/1832725>15:02
diddledanWimpress: my hero 😻15:03
zygapstolowski: ah, yes15:03
zygapstolowski: this is a dupe, let me find the other one15:03
zygapstolowski: dedupped15:04
zygathank you15:04
pstolowskity15:06
zygahmmm15:17
mupPR snapcraft#2690 closed: remote-build: error when --user is required <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2690>15:19
* cachio afk15:26
cachiofor lunch15:26
zygaquick update and reboot15:34
popeyjdstrand: seeing conversations about printing popping up again. Can we please re-visit https://forum.snapcraft.io/t/inkscape-autoconnect-cups-control/873916:02
popey(inkscape)16:02
popey6 months later, no reply, and still complaints from users (and developers) that they can design something and then not print it.16:03
ijohnsonpedronis: could you add your thoughts you shared this morning from SU to the forum topic on `snap model`: https://forum.snapcraft.io/t/remodeling-to-a-new-model/10477 ?16:04
ijohnsonpedronis: I'm willing to keep sculpting at the output until it's what we want too16:05
=== pstolowski is now known as pstolowski|afk
ijohnsonthough I am now starting to think about this code as less of something that's built like a house and more something that's carved away like a marble michaelangelo sculpture :-)16:06
roadmrijohnson: the sculpture is already there, you just have to chisel away the leftover chunks16:09
ijohnsonexactly16:09
ijohnsonI should start all PR's with 100000 lines of random chars and just whittle away the diff until I have a masterpiece16:09
roadmrijohnson: meh just train an AI - these days, the artist would be replaced by some machiine learning on michelangel.io16:11
* ijohnson considers buying michelangel.io and not michelangelo.io16:13
roadmrijohnson: only because there's no .lo TLD16:16
ijohnsonwell also michelangelo.io is $3500/year and michelangel.io is only $4516:18
roadmrouch16:21
zygait's kind of crazy but the temperature is just right, now, to work :)16:23
pedronisijohnson: I'll write something my monday morning there16:26
ijohnsonpedronis: ack thanks16:26
ijohnsonso am I good to merge PR #7149?16:27
mupPR #7149: cmd: add snap model command; daemon: add /v2/model, /v2/model/serial REST APIs <Remodel :train:> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7149>16:27
Chipacaok peeps, i'm out for a while. Will probably be back later to wrap up. TTFN and have an awesome weekend if I don't see you!16:38
zygaijohnson: thanks for the review, you found a bug indeed :-)16:55
ijohnsonhappy to help zyga :-)17:32
zygaijohnson: updated https://github.com/snapcore/snapd/pull/737917:34
mupPR #7379: tests: add version-tool for comparing versions <Created by zyga> <https://github.com/snapcore/snapd/pull/7379>17:34
zygaif you +1 this it would be my highlight of the day :)17:34
ijohnsonlooking now17:41
ijohnsonzyga: +1'd17:43
zygawoot, thank you!17:43
zygaback to cgroups :)17:43
* ijohnson takes a break17:44
mupPR snapd#7149 closed: cmd: add snap model command; daemon: add /v2/model, /v2/model/serial REST APIs <Remodel :train:> <Created by anonymouse64> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/7149>18:02
pedronisijohnson: ^ I merged it18:02
mupPR snapd#7067 closed: overlord/devicestate:  support the device service returning a stream of assertions <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/7067>18:04
ijohnsonthanks pedronis18:54
B|ack0phi20:01
B|ack0peverytime i boot into ubuntu 18.04 i face system error report popup20:02
B|ack0perror report doesnt give any detail but when i check the logs20:02
B|ack0pat the same time i received error report popup these are the possible errors/fails: https://termbin.com/xlo920:03
B|ack0pTime is 22:5420:03
B|ack0pmany errors like this:20:03
B|ack0pAug 30 22:54:17 uthink-x61 gnome-software[3571]: Failed to load snap icon: local snap has no icon20:03
B|ack0pAug 30 22:54:19 uthink-x61 gnome-software[3571]: Failed to load snap icon: local snap has no icon20:04
B|ack0poh it is same sorry20:04
B|ack0pthere are 7 same error log as above20:04
mupPR snapcraft#2695 opened: spread tests: install package marker into ament index <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/2695>20:40
jdstrandpopey: please note that there was nothing for me to reply to. the vote is cast. it was contentious. I asked for others to rule on it21:45
jdstrandpopey: I did respond to today's posts21:46
jdstrands/rule/comment/21:46
=== Aavar is now known as Aavar_
=== Aavar_ is now known as Aavar
=== lborda is now known as lborda_afk

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