=== chihchun_afk is now known as chihchun
dot-tobiasgood morning (not sure if last one went through)06:11
zygaIโ€™m not well today. I had fever yesterday and I seem to have some sort of inflammation that is killing my back06:12
zygaI will work from bed, skip on video calls06:13
zygaUnfortunately I donโ€™t have my laptop this week so Iโ€™ll work from one of the old thinkpad I have at home.06:14
mborzeckizyga: hey06:17
zygaFedora 30 is my host today06:23
mupBug #1822535 opened: Unable to install apps via snapweb <Snappy:New> <https://launchpad.net/bugs/1822535>06:28
zygadarn, no token near bed06:34
zygamborzecki: how are you doing?06:36
zygamborzecki: can you do a few reviews please06:37
mborzeckizyga: heh, caught cold, had a runny nose all weekend :/06:37
zygamborzecki: I have https://github.com/snapcore/snapd/pull/6643 that is now non-embargoed06:37
mupPR #6643: tests: deny ioctl - TIOCSTI with garbage in high bits <Created by zyga> <https://github.com/snapcore/snapd/pull/6643>06:37
zygamborzecki: same here, I was rinding the bike at night and ended up sick06:37
mborzeckizyga: will do, now trying to figure out why google-cloud-sdk breaks the tests06:37
zygaI need to fetch my google sdk token06:38
zygamborzecki: I left my macbook for servicing :/06:38
zygaperfect timing06:38
mborzeckizyga: must be something funny, i removed google-cloud-sdk early in spread.yaml and still prepare failed06:39
mborzeckizyga: saw your tweet, the keyboard right?06:39
zygamborzecki: can I be your garden gnome06:39
zygamborzecki: what is breaking and how do you think it is related to the sdk?06:39
zygamborzecki: yes, the space bar but mostly keys started repeating themselves06:40
zygamborzecki: the spacebar got stuck06:40
* zyga logged into lp.net06:47
mupBug #1822535 changed: Unable to install apps via snapweb <Snappy:Won't Fix> <https://launchpad.net/bugs/1822535>06:49
zygamvo: hello06:56
mvohey zyga06:58
zygamborzecki: so about those failures06:59
zygamborzecki: what do you know?06:59
mborzeckimvo: morning06:59
mvohey mborzecki07:01
mvomore failures?07:01
mborzeckizyga: it breaks right after cache_snaps07:02
zygamborzecki: I know little about this, can you tell me like you would to a garden gnome07:02
mborzeckihaha garden gnome :P07:03
* mvo hugs zyga07:03
zygahaha :)07:04
zygaI'm sick today, sorry07:04
zygaI'll work from bed07:04
zygariding at night was a bit silly in hindsight07:04
zygaI had fever all yesterday07:04
mupPR snapd#6667 opened: tests: enable tests that write /etc/{hostname,timezone} on core18 <Created by mvo5> <https://github.com/snapcore/snapd/pull/6667>07:04
zygatoday I'm better but my back is killing me, I cannot stand up normally07:04
zygamvo: ^ reviewed07:05
mvozyga: thank you07:05
zygamvo: I need some reviews07:07
mborzeckimvo: you did look into this on friday didn't you?07:07
mupPR #6597: cmd/snap-update-ns: refactor of profile application (1/N) <Created by zyga> <https://github.com/snapcore/snapd/pull/6597>07:07
mupPR #6502: dirs,overlord/snapstate: add Soft and Hard refresh checks <Created by zyga> <https://github.com/snapcore/snapd/pull/6502>07:07
zygathose two are most pressing07:07
zygaboth need a 2nd review07:07
mvozyga: looking at 6597 now07:14
mvozyga: whish me luck that I don't get distracted07:14
zygathank you, any reviews you do will help me a lot07:14
mborzeckihmm it breaks earlier than i thought07:15
zygamborzecki: tell me more07:15
zygamborzecki: today I might to classic mount namespace behind a feature flag07:15
zygamborzecki: feels like something I want to do07:15
mborzeckizyga: that'd be great, ping if you want to chat about it07:16
zygamborzecki: I will do the simple stuff for now, let's sync closer to standup to know more07:18
zygamborzecki: darn, fedora with 2.38 is not usable for snapcraft yet07:19
zygamborzecki: 2.39 will be selinux ok?07:19
mborzeckizyga: try with multipass07:19
mborzeckiyou'll probably need setenforce 0 for now too07:19
zygamborzecki: I was trying to build core8107:19
zygait doesn't work with multipass yet07:20
mborzeckizyga: even if you force snapcraft to use multipass?07:20
zyganot sure I know how to07:21
zygathat's fine, I'll work on this now07:21
mborzeckizyga: SNAPCRAFT_BUILD_ENVIRONMENT=multipass07:21
zygamborzecki: maybe 2.38.1 with fedora fixes should be made?07:21
=== pstolowski|afk is now known as pstolowski
mvohey pstolowski07:28
zygapstolowski: hello :)07:29
mvozyga: I reviewed 6597, one of my comments is hidden under something marked as resolved07:33
mvozyga: not sure we it wasn't "unresolved" by the comment but oh well07:33
zygamvo: can you link to it please?07:33
zygaI think I see it07:34
zygamvo: the answer to both questions: I didn't want to reorder patches, just split the big chunk in half07:34
zygaI wanted to limit the possibility of breaking what used to work to to07:35
mvozyga: ok, lets merge and see what the second part of the PR brings07:35
zygamvo:you can see the rest, it's already on github07:35
zygathough it needs more work now that some of the things here changed the interface a little07:35
mvozyga: oh, ok07:35
* mvo nods07:36
zygae.g. this is the locking function https://github.com/zyga/snapd/commit/6525b925e6c3d299e6ce77a1885932c0ef0f2bef07:36
mborzeckihmm, idk, looks like when removing snapd package we don't remove state.json for some reason07:41
mborzeckithough postrm has rm -rf /var/lib/snapd07:41
zygamborzecki: that's on purge07:42
zygamborzecki: perhaps purge vs remove?07:42
mborzeckizyga: apt log shows we're doing --purge07:43
mborzeckizyga: fwiw the mount units are stopped and removed07:43
mborzeckiheh, and out prepare project does: distro_purge_package snapd || true we won't notice if that fails07:45
zygamvo: can we postpone our 1-2-1? I don't have chrome here and I don't believe firefox on fedora even works with google meet07:50
mvozyga: sure, no problem07:51
mborzeckizyga: mvo: https://paste.ubuntu.com/p/Q7f6wtG4mr/ hmm that's project prepare07:53
zygaoutput status 100?07:54
zygawhy is that07:54
mvorm: cannot remove '/var/cache/snapd/aux': Is a directory07:54
mvomborzecki: the same bug from last week07:54
zygamborzecki: reexec creates aux07:54
zygamborzecki: snapd postrm doesn't know about it07:54
zygamborzecki: we discussed this07:54
zygaeither reexec must create better postrm07:54
zygaor postrm must be more rm -rf07:55
mvomborzecki: on what system do you see this? fwiw I just saw two PRs failing on 18.0407:55
zygaeither way07:55
mvozyga: yeah, postrm is fixed iirc07:55
mborzeckimvo: 18.0407:55
mvozyga: just not in the older version07:55
mvomborzecki: looking07:55
mborzeckiok, so maybe extra rm -rf /var/lib/snapd/ right after purge will be fine07:56
mvomborzecki: I am pushing a PR07:56
mborzeckimvo: ok07:57
mupPR snapd#6668 opened: tests: add workaround for missing cache reset on older snapd <Created by mvo5> <https://github.com/snapcore/snapd/pull/6668>08:00
zygamborzecki: classic mount namespace is interesting, somewhat more work than I thought08:01
zygabecause we need to use snap-update-ns there as well08:01
zygaand we need to preserve the mount namespace08:01
zygait's progressing well but just wanted to highlight that it's more than I thought initially08:01
mborzeckimvo: should we push out an update to 2.37 that does rm -rf /var/cache/snapd/* ?08:03
mborzeckiwrong window08:06
* zyga spawned tests and tries to make a coffee08:15
pedroniszyga: hi, sorry you are sick, weren't there bugs to look into?  instead of starting on classic mountspace08:25
zygapedronis: yes, there are bugs to look at08:25
zygapedronis: perhaps I should work on those hmmm08:25
zygaI wasn't thinking much in the morning, just wanted to something08:26
mvomborzecki: the rm -rf was added in feb so I think 2.37 already has it08:26
mvomborzecki: ha! it does not apparently08:27
mvomborzecki: yeah, that is unfortunate08:27
mborzeckimvo: the version that was installed is 2.37.408:27
mvomborzecki: indeed08:27
dot-tobiasogra / zyga or someone else who knows: Am I correct in the assumption that only option to set a NTP time server is by overwriting /etc/systemd/timesyncd.conf (like in ogra's ntpcontrol example snap https://github.com/ogra1/ntpcontrol/blob/be16c59cc24d473f22baa89f534f993d553aa6aa/ntpcontrol#L33), and that the timeserver-control interface allows access to this file (https://github.com/snapcore/snapd/blob/71bdfa33d159509164b614e69d59d0b244db3c6208:27
dot-tobias/interfaces/builtin/timeserver_control.go#L43)? I checked the timedated DBus docs and it doesn't look like there's a method to set a time server: https://www.freedesktop.org/wiki/Software/systemd/timedated/08:27
mvomborzecki: :/ that is annoiny, would have been easy to include in .4 if we were aware08:28
mborzecki#6637 is green, can we land it even before postrm workaround?08:42
mupPR #6637: interfaces: deal with the snapd snap correctly for apparmor 2.13 <Created by mvo5> <https://github.com/snapcore/snapd/pull/6637>08:42
mborzeckithe spread job ran successfuly 4 days ago for that PR08:43
mvois it just me or is running snapd from inside cmd/snapd and ./snapd broken rught now?08:44
mborzeckimvo: it is, due to snap-seccomp08:44
mvomborzecki: yeah, I noticed08:44
mborzeckimvo: can you build snap-seccomp there too?08:44
mvomborzecki: is there a "fix" pending?08:45
mvomborzecki: I can symlink it08:45
mupPR snapd#6637 closed: interfaces: deal with the snapd snap correctly for apparmor 2.13 <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6637>08:45
mvomborzecki: no super strong opinion, its just my workflow, I can change it if we decide it not worth supporting running out of a git checkout08:45
mborzeckimvo: i can look into it, but the last time i tried, the solution was rather fugly08:45
mvomborzecki: *nod*08:45
mvomborzecki: don't worry for now08:46
pedronismvo: mborzecki: wouldn't 6602 make it work  (assuming the system one does have version-info) ?08:46
mvopedronis: yeah, I was wondering the same08:47
mborzeckiyeah, that's the fugly solution as far as i'm concerned08:48
mvomborzecki: heh08:48
mborzeckimvo: fwiw, 6602 could use a 2nd review :)08:48
mvopstolowski: I'm looking at 6660 right now, what can I do to get nested timings? just want to see what the output looks like08:49
mvomborzecki: yeah, its on my list which keeps growing instead of shrinking08:49
mvomborzecki: but I hope to get to it this morning08:49
zygapedronis: I'm adding tests for core -> core18 migration08:52
pstolowskimvo: at the moment we only collect the new timings in ifacemgr around setting up profiles - see the output in the description of the PR; also we filter out those that are fast08:54
mborzeckimvo: #6668 failed, looking at the log08:55
mvopstolowski: thank you, I will play around a bit with the output08:55
mvomborzecki: /o\08:55
mupPR #6668: tests: add workaround for missing cache reset on older snapd <Created by mvo5> <https://github.com/snapcore/snapd/pull/6668>08:55
mborzeckiah, ok, purge may fail if snapd is not installed on some distros08:56
mborzeckimvo: i can take over this if you don't mind08:56
mborzeckifwiw, ubuntu is the only distro that comes with snapd preinstalled08:57
mvomborzecki: go for it09:00
mborzeckimvo: ack09:00
mvomborzecki: I did not add a purge, did I? I mean, my only change was to rm -rf the aux dir, no?09:00
mvomborzecki: so I'm confused how this breaks things - but I will wait for your PR :)09:00
mborzeckimvo: you dropped || true from purge09:01
mvo(or your push to it)09:01
mvomborzecki: silly me09:01
mvomborzecki: well, makes sense to drop it in some way because it masked the earlier error09:01
mvomborzecki: but yeah, that explains things :)09:01
mborzeckithat's fine, we expect it to succeed on system where it 'does' things09:01
mborzeckibtw. #6602 pick up wrong path libexecdir path on amazon for some reason09:04
mupPR #6602: cmd,interfaces: replace local helpers with cmd.InternalToolPath <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6602>09:04
pedronismborzecki: doesn't that come dirs?  is the code always been buggy and we didn't notice?09:06
pedronis*come from dirs09:06
mborzeckipedronis: yes, that's from dirs, and it should be ok, /etc/os-release should check out as 'fedora'09:07
zygapedronis: so, sanity check question09:18
zygapedronis: what should happen when a snap changes base09:18
zygapedronis: should we force all apps to down09:18
zygapedronis: should we discard namespaces?09:18
zygapedronis: should we refuse (like in refresh app awareness work)?09:18
pedroniszyga: when we have implemented app awareness we will have that, no?  so the questio is what we can do quickly to deal with the bug?09:21
zygapedronis: yes, we will09:21
mborzeckimvo: pushed09:21
zygapedronis: perhaps we should discard the namespace when bases change09:21
zygapedronis: this would at least be less broken09:22
zygapedronis: when we have enduring services we should perhaps also discard the namespace09:22
pedroniszyga: that's my thinking as well09:22
zygapedronis: to esure that apps run on top of the base they designated09:22
zygapedronis: I will get to it, thanks!09:22
mborzeckiheh that InternalToolPath is tricky10:03
mupPR snapd#6669 opened: overlord/corecfg: make expiration of automatic snapshots configurable (4/4) <Created by stolowski> <https://github.com/snapcore/snapd/pull/6669>10:04
mborzecki#6668 is green10:05
mupPR #6668: tests: add workaround for missing cache reset on older snapd <Created by mvo5> <https://github.com/snapcore/snapd/pull/6668>10:05
mborzeckimvo: can you take a quick look at that last patch I pushed?10:06
mvomborzecki: sure, was in a meeting10:12
mvomborzecki: but free now10:12
mvomborzecki: its *green*10:13
* mvo hugs mborzecki 10:13
mvomborzecki: merged, I did not even look at it10:13
mvomborzecki: (ok, the last part of not true ;)10:13
mborzeckihaha fine :)10:13
mupPR snapd#6668 closed: tests: add workaround for missing cache reset on older snapd <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6668>10:14
mvomborzecki: thanks for it and your change looks fine10:14
mborzeckimvo: great!10:15
mborzeckineed to push a little fix for InternalToolPath, libexecdir is a mess to handle10:15
pstolowskimvo, Chipaca thanks for the review of snap debug timings; i'm happy to tweak the output, perhaps pedronis wants to comment on this?10:18
zygamborzecki: hey selinux question10:22
mvopstolowski: yeah, if we don't find anything that appeals to everyone we should probably have a HO or something10:22
zygamborzecki: how can I relabel a binary done in make hack10:23
zygamborzecki: shall I restorecon?10:23
mborzeckizyga: yes10:23
mborzeckizyga: try restorecon -V <path>10:23
zygais it recursive?10:23
mborzecki-V should be verbose10:23
mborzeckior -v was verbose10:24
mupPR snapd#6664 closed: cmd/snap,client,daemon,store: layout and sanity tweaks for find/search options <Created by pedronis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6664>10:55
mupPR snapd#6597 closed: cmd/snap-update-ns: refactor of profile application (1/N) <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6597>10:58
zygamvo: thank you, I will prepare the next batch10:59
alan_gogra_, obviously you sometimes use vt.handoff. So does this make sense for mir-kiosk? https://github.com/MirServer/mir-kiosk/pull/2611:00
mupPR MirServer/mir-kiosk#26: Workaround for Mir not starting if the gadget snap sets 'vt.handoff' <Created by AlanGriffiths> <https://github.com/MirServer/mir-kiosk/pull/26>11:00
mvozyga: ta11:05
zygapedronis: I've implemented the discard, it works ok, I will add some missing unit tests for new C code and send the patches11:31
pedroniszyga: thx11:31
mborzeckino suitable *util package to put normalizeYamlValue in11:38
pedronismborzecki: is this because of making gadget a package?11:39
mborzeckipedronis: yes11:40
mborzeckipedronis: we have jsonutil, so yamlutil isn't that bad11:43
pedronismborzecki: yes, though jsontuil is already border line, it has two functions in it11:48
pedronismborzecki: also is the first time I notice netutil,  why was it not called nmutil ?11:48
mborzeckipedronis: in case there are ways to get the metered status from other sources than nm11:49
* Chipaca wanders off in search of sustenance11:51
pedronismborzecki: I see, the problem is that current name make it sounds a companion to the go net package11:53
pedronismborzecki: anyway  I would prefer to call the new package  metautil,  with some description about utilities for working with snap metadata11:54
mborzeckiack, metautil sounds ok11:54
zygamborzecki: https://github.com/snapcore/snapd/pull/667011:59
mupPR #6670: tweak: fix "make hack" on Fedora <Simple ๐Ÿ˜ƒ> <Created by zyga> <https://github.com/snapcore/snapd/pull/6670>11:59
mupPR snapd#6670 opened: tweak: fix "make hack" on Fedora <Simple ๐Ÿ˜ƒ> <Created by zyga> <https://github.com/snapcore/snapd/pull/6670>11:59
* zyga runs one more test before proposing the fixes12:00
mupPR snapd#6671 opened: cmd,tests: forcibly discard mount namespace when bases change <Created by zyga> <https://github.com/snapcore/snapd/pull/6671>12:09
mborzeckioff to pick up the kids12:10
zygaondra, Saviq: ^ one of the core16 -> core18 issues that affects you12:11
ondrazyga yeah! o/12:12
zygaplease have a look at the spread test in the PR for details12:12
=== ricab is now known as ricab|lunch
zygathat's the stale test before the fix :D12:12
* zyga looks at git status12:13
zygamborzecki: ./spread-shellcheck:120: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.12:20
pedronispstolowski: I left a high-level comment in #666012:36
mupPR #6660: [RFC] cmd/debug: integrate new task timings with "snap debug timings" <Created by stolowski> <https://github.com/snapcore/snapd/pull/6660>12:36
pstolowskipedronis: yes i saw it, thanks!12:39
* zyga goes to the standup13:01
zygastandup is in one hour13:02
zygatime changed again13:02
mborzeckican we move that to 2pm or 3pm like it was before?13:04
zygaI'd prefer that13:04
zygamvo: ^13:04
Chipacazyga: mborzecki: person to check with is cachio13:07
Chipacaand cmatsuoka13:08
mvozyga: in a meeting13:08
mvomborzecki: what is missing on 6634 ? can this land?13:09
mborzeckimvo: is it green now?13:09
mvozyga, mborzecki, Chipaca my plan was to discuss meeting time in the standup in 51 min :)13:09
mvomborzecki: I think it is13:09
mborzeckiah, perfect, landing it13:09
mvomborzecki: \o/13:09
mupPR snapd#6634 closed: snap: add validation of gadget.yaml <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6634>13:10
=== cpaelzer__ is now known as cpaelzer
ograalan_g, well, i'm fine doing a kiosk specific gadget for that ... so dont worry about me13:27
Chipacazyga: https://i.imgur.com/98wirsn.jpg13:29
zygaChipaca: little blue men13:29
alan_gogra, I wasn't so much worrying about you, but wanting your opinion of avoiding the potential problem this way. (Before someone out there hits it and get confused.)13:30
sergiusensmvo: hey there, can you comment on https://forum.snapcraft.io/t/snap-try-messaging-and-user-experience/10667 ?13:32
mvosergiusens: sure, looking13:33
mvosergiusens: I like the first part of your comment already :)13:34
mupPR snapd#6670 closed: tweak: fix "make hack" on Fedora <Simple ๐Ÿ˜ƒ> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6670>13:35
=== ricab|lunch is now known as ricab
sergiusensmvo: thanks, will be looking forward to the feedback there :-)13:36
cmatsuokazyga, mborzecki: that's 3pm in what timezone?13:36
zygacmatsuoka: CET13:36
zygaI suspect13:36
zygawhoever created the event "carries" it with the timezone they inhabit13:37
cmatsuokaso is it UTC+2 i believe?13:37
mvopstolowski: thanks for 6660 - <313:37
cmatsuokaI'm in UTC-3, so that would be... 10am here, one hour earlier than the regular 11am13:38
cmatsuokaI would have to change one meeting, but that's feasible13:39
pstolowskimvo: sure, thanks for the ideas13:41
mborzeckicmatsuoka: hm isn't there a time change in your location too within the next 2 weeks or so?13:45
cmatsuokamborzecki: the last one was a few weeks ago with end of daylight saving time13:46
cmatsuokamborzecki: we went from UTC-2 to UTC-313:46
cmatsuokaI believe cachio is in UTC-4 now?13:47
cachiocmatsuoka, I am in UTC-313:48
cachiobut we don't change time13:48
mborzeckiwow, that sounds like uber mess :P13:51
mborzeckiafaik in 2021 we also stop changing tz13:51
mborzeckiwell not tz, just time13:51
cmatsuokathere are some talks about stopping it here as well13:53
ograalan_g, well, the vt_handoff simply hides the text (in case there is no boot splash) by forcing a tty switch to a definitely black tty13:54
ograif you havent seen any scrolling text and if mir did start fine, we're good i think13:54
alan_gogra, what happens today on RPi3 isn't fine: i.e. Mir doesn't start.13:57
alan_gI know I've a PR up that will fix it (once the image updates) but it seems fragile.13:58
ograalan_g, you mean mir doesnt start even if you remove the vt.handoff ?13:58
ograi thought that was your issue13:58
alan_gogra, yes, removing it does avoid the problem (that's the PR).13:59
ograalan_g, right ... and if the splash still works fine and you dont all of a sudden see scrolling text then we're good14:00
alan_gSo you don think I should be concerned about someone adding vt.handoff (or using Mir on the current RPi image)?14:03
ograi was told by vorlon a while ago that vt.handoff= would be deprecated anyway in the main distro so i think its not a bad move to drop it if you pdont see regressions14:04
=== chihchun is now known as chihchun_afk
mupPR snapd#6672 opened: metautil, snap: extract yaml value normalization to a helper package <Simple ๐Ÿ˜ƒ> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6672>14:15
__chip__pstolowski: is #1802339 addressed as part of your timing work, or is that for later?14:28
mupBug #1802339: Tasks should carry separate timestamps for undo paths <snapd:Confirmed> <https://launchpad.net/bugs/1802339>14:28
pstolowski__chip__: for later, it is something else and not directly useful for timings. we're interested in durations and we have them for do and undo already14:29
* zyga iterates on tricks that speed up local spread qemu runs14:50
Chipacadegville: do you have the 'this is work in progress' block for docs somewhere handy?14:55
zygaChipaca: question for you sir, could we move /var/lib/snapd/cache to /var/cache/snapd/15:06
Chipacazyga: what for?15:11
niemeyerHello snappyers15:11
Chipacazyga: I think it'd be a bad idea :-)15:11
niemeyer(see what I did there, Chipaca)15:11
Chipacaniemeyer: hiya15:11
zygaChipaca: why?15:12
Chipacaniemeyer: I did, I did15:12
Chipacaniemeyer: I approve15:12
niemeyerI'm pretty much done with go-yaml v315:12
niemeyerAre there any pet peeves that you'd like me to look into before I put it in the wild and need to hold compatibility15:12
Chipacaniemeyer: lest I refer you to https://www.urbandictionary.com/define.php?term=snapper15:12
Chipacaniemeyer: whitespace control!15:12
niemeyerChipaca: Indentation?  Is in15:13
Chipacaniemeyer: hmm... will that let me align the values in a map output?15:13
niemeyerChipaca: Hmm.. probably not15:13
niemeyerChipaca: No it won't15:13
niemeyerChipaca: I think that'll need some extra fiddling15:13
Chipacaniemeyer: can I tell it to use "\t" for indentation, and then have it write to a tabwriter?15:14
Chipacazyga: in particular I think it's a bad idea because we depend on being able to hardlink things from var/lib/snapd/cache to var/lib/snapd/snaps15:14
niemeyerChipaca: No, as that's the \t makes for invalid yaml.. :(15:14
Chipacazyga: if those are on different volumes we lose15:15
zygaChipaca: mmm15:15
niemeyerChipaca: I think the map alignment would be a nice stock feature15:15
zygaChipaca: it's not that the current directory prevents that from happening15:15
niemeyerChipaca: And *probably* not hard15:15
niemeyerChipaca: There's already good "what column am I in?" control in the encoder15:15
niemeyerChipaca: It's also easy to do in a compatible way, so not a blocker for v315:16
niemeyerChipaca: The indentation piece is in15:16
Chipacaniemeyer: ๐Ÿ‘15:20
* zyga lunch15:20
Chipacazyga: you misspelled "dinner"15:21
roadmruse the common galactic spelling: omnomnom15:23
Chipacaroadmr: is that a pokรฉmon15:24
Chipacaroadmr: i think it's one of these https://deathbulge.com/comics/34315:25
roadmrhehe :) maybe!15:25
zygaChipaca: it's not dinner if I only had breakfast but ... yeah15:31
* cachio lunch15:39
mupPR snapd#6663 closed: cmd: make fmt <Simple ๐Ÿ˜ƒ> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6663>15:56
mupPR snapd#6671 closed: cmd,tests: forcibly discard mount namespace when bases change <Created by zyga> <Closed by zyga> <https://github.com/snapcore/snapd/pull/6671>16:04
zygaI have a file that indent is not stable on16:28
zygayou run indent, you get a change16:28
zygayou run indent16:28
zygayou get ... another change16:28
zygayou run indent16:28
zygayou are back to 1st change16:28
zygaand so on16:28
=== upline is now known as grumble
mupPR snapd#6673 opened: cmd,tests: forcibly discard mount namespace when bases change <Created by zyga> <https://github.com/snapcore/snapd/pull/6673>16:47
=== pstolowski is now known as pstolowski|afk
* zyga is preparing some timing data for before/after speed-up patches for spread17:11
* Chipaca EODs17:11
zygaChipaca: I still didn't have that lunch17:11
zygabut I guess it should be dinner now17:11
* Chipaca EODs even harder17:11
Chipacazyga: I'm going to go make pascualina17:11
Chipacazyga: I could post you some17:11
zygaI will probably look at what I can find in the kitchen17:12
zygaChipaca: about that cache directory, I know it's hard but I think we should rethink that, not urgent17:12
zygaChipaca: I have some ideas on how to make things faster and it is simpler with that17:12
Chipacazyga: tomorrow17:12
zygaChipaca: one last question17:12
Chipacabecause, EOD17:12
Chipacazyga: go on17:12
zygaChipaca: is there a query I can do to tell me snap name for a hash we keep in the cache?17:12
Chipacazyga: snap info17:13
zygasnap info ... what?17:13
zygaon the file?17:13
Chipacazyga: yes17:13
zygaah, super17:13
zygaenjoy your evening :)17:13
Chipacazyga: sudo snap info because 060017:13
Chipacabut yes17:13
popeyjdstrand: hi. i have a snap which tries to access /dev/shm/ring-buffer-foo, which fails in strict confinement, and works in devmode. Is patching the upstream source the only option?18:00
jdstrandpopey: hey, snapcraft-preload might also work18:06
* popey tries snapcraft-preload18:06
mupPR snapcraft#2444 closed: snap: move to core18 <Created by sergiusens> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/2444>18:21
popeyjdstrand: snapcraft-preload breaks more than it fixes.18:21
popey    command: snapcraft-preload  desktop-launch $SNAP/b218:22
popeywonder if I need desktop-launch before snapcraft-preload18:22
mupPR snapcraft#2445 closed: Add core18 support to dotnet plugin <Created by ed10vi> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/2445>18:24
mupPR snapcraft#2504 closed: cli: validate plugin schema before provider <Created by cmatsuoka> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/2504>18:27
jdstrandpopey: I think the ordering is important, yes. I think one of the two parts mentions which should be first18:28
mupPR snapcraft#2463 closed: Pass --work-dir param for out-of-tree builds <Created by abitrolly> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/2463>18:30
mupPR snapcraft#1649 closed: add stage/usr/local/lib to cmake library path <Created by diddledan> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/1649>18:33
mupPR snapcraft#1875 closed: Remove deprecated 'snap' recommendation <Created by ted-gould> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/1875>18:33
mupPR snapcraft#2020 closed: elf: set no-default-lib for all elf files if patching <bug> <enhancement> <Created by sergiusens> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/2020>18:33
mupPR snapcraft#2493 closed: cli: specify default expiration for export-login <Created by felicianotech> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/2493>18:36
popeyjdstrand: yes, changing order fixed the new problem, but doesn't sort the /dev/shm/ring-buffer issue :(18:41
zygamvo: https://github.com/snapcore/snapd/pull/6674 :)18:56
mupPR #6674: tests: use apt via eatmydata <Created by zyga> <https://github.com/snapcore/snapd/pull/6674>18:56
mupPR snapd#6674 opened: tests: use apt via eatmydata <Created by zyga> <https://github.com/snapcore/snapd/pull/6674>18:56
mupPR snapcraft#2135 closed: Add gradle support for task other than just 'jar' <Created by bsutton> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/2135>19:06
popeyjdstrand: worked around the issue by disabling some features in the upstream code19:20
zygajdstrand: ello19:23
mupPR snapd#6675 opened: cmd/snap-confine: allow using tools from snapd snap <Created by zyga> <https://github.com/snapcore/snapd/pull/6675>19:23
zygajdstrand: could you please look quickly at https://github.com/snapcore/snapd/pull/667519:23
mupPR #6675: cmd/snap-confine: allow using tools from snapd snap <Created by zyga> <https://github.com/snapcore/snapd/pull/6675>19:23
zygajdstrand: it's a two line change to s-c apparmor profile19:23
zygajdstrand: I need it for another fix as a prerequisite19:23
jdstrandzyga: hey, done19:26
zygajdstrand: thank you!19:26
zygajdstrand: how are you doing btw? :)19:27
jdstrandzyga: I'm ok. I actually think I'm coming down with something. the daemon user work as progressed a bit. still a bit more to do19:27
jdstrandzyga: how are you?19:28
zygajdstrand: I had a cold and fever yesterday; I kept pushing myself all last week doing biking after dark when it was getting too cold19:29
zygajdstrand: I'm better today though I will take this week slowly19:29
jdstrandzyga: hope you feel all better soon19:30
zygajdstrand: I'm progressing on refresh app awareness19:30
zygajdstrand: though this week is mostly bugfixes fore core18 and mount related things19:30
zygajdstrand: I took a stab at classic confinement mount namespace and I have a better understanding of what is required, I will try to do that after the bugfix weke19:30
zygajdstrand: I'll be EODing soon, ttyl! :)19:32
zygathank you for the quick review!19:32
cachiobut we don't change time20:23
popeyjamesh: hello. should gtk2 themes work as expected in https://forum.snapcraft.io/t/improving-theme-support-for-gtk-2-apps/769320:42
popeyMine still looks like a gtk2 app from the past20:43
popeyhttps://paste.ubuntu.com/p/89rw9srv8H/ is my yaml20:43
popeygtk-common-themes:gtk-2-themes gtk-common-themes:icon-themes gtk2-common-themes:gtk-2-engines are all connected20:44

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