[05:02] morning [05:16] hmm found an interesting error: `error: store.RevisionNotAvailable with 2 snaps` [05:21] well, nvidia seems to work just fine [05:37] apparmor denials are not logged in dmesg, so it's easy to miss those [05:37] type=AVC msg=audit(1529645643.115:406): apparmor="DENIED" operation="open" profile="snap.ohmygiraffe.ohmygiraffe" name="/etc/pulse/client.conf.d/" pid=10207 comm="love" requested_mask="r" denied_mask="r" fsuid=1000 ouid=0 [05:50] hey [05:51] mborzecki-suse: hey [05:51] zyga: hey [05:52] I was up till 2AM iterating on the packaging [05:52] zyga: so far so good, no issues with package from your repo [05:52] found a few annoying bugs and fixed them [05:52] and learned a few things that we need to change in our build system [05:52] (not having a central build system sucks) [05:56] but man, textual yast installer is awful, didn't manage to install it the way i'd like, gave up and went with the defaults at some point, ended up with gpt partitioned disk, no efi, no luks, no lvm, btrfs on rootfs, xfs on home and non-blacklisted nouveau (what made xorg unusable) [05:57] hey [05:57] so how do I go about backing up a snap's data [05:57] Like nextcloud. [05:58] snaphelp: fantastic question [05:58] you can do it manually today [05:58] but we are also working on a feature that makes it easy [05:58] zyga: Snaps aren't auto updating are they? [05:58] don't know why people even bother building such installers, what i really need is creating partitions myself, mounting them in the layout i want, bootstrapping the system, update fstab, crypttab, rebuild initramfs, install some bs pacakges, install bootloader (the way i want) [05:58] all snap data is held in /var/snap/$SNAP_NAME and $HOME/snap/$SNAP_NAME [05:59] mborzecki-suse: hmm? [05:59] snaphelp: the upcoming feature will allow you to snapshot this state easily from the snap CLI [05:59] zyga: is the snap directory in my home directory just a symlink to /var/snap? [05:59] snaphelp: no [05:59] snaphelp: those are separate data sets, one is per user and the other is global for the system [06:00] hmm [06:00] zyga: just being grumpy about distro installers :| [06:02] hey mvo [06:02] mvo: morning [06:02] hey zyga and mborzecki [06:04] zyga: so the global data is in /var [06:04] and there's no auto updating correct? [06:05] snaphelp: per-snap global data is in /var/snap/$SNAP_NAME [06:05] snaphelp: I don't understand the question about auto-updating [06:05] Do snaps automagically update themselves [06:05] or is it snap update [06:06] snaphelp: it is snap refresh but you don't have to do it explicitly, snapd automatically refreshes everything periodically [06:07] Can I disable automatic refreshes? [06:07] you can set a schedule but you cannot turn it off [06:10] mvo: I'm a bit tired, I spent half of the night on this packaging [06:11] mvo: I will be sending patches when I'm consistently awake [06:11] zyga: any plans to allow disabling? [06:11] snaphelp: no, there are no such plans [06:11] hm [06:13] zyga: I'm torture testing a snap [06:14] heh wow it's handling being beat with 40gb of data [06:18] mvo: we need to have a makefile in data/ [06:18] and stop installing files by hand in packaging [06:18] it's easy to miss files [06:18] and it's silly to do this [06:21] need to go to kids school for a while, bbl [06:22] PR snapd#5352 closed: many: expose full publisher info over the snapd API [06:23] zyga: ok [06:53] PR snapd#5375 opened: packaging: put snapctl into /usr/lib/snapd and symlink in usr/bin [06:53] oh, interesting [06:57] PR core18#34 opened: hooks: make /usr/bin/snapctl available in the base [06:57] zyga: one more commit to 5375 (apparmor default update) [07:07] PR core18#34 closed: hooks: make /usr/bin/snapctl available in the base === pstolowski|afk is now known as pstolowski [07:13] morning [07:17] * zyga is super sleepy [07:18] sorry folks, this day will be hit-and-miss [07:18] hey pstolowski, good morning [07:28] PR snapd#5376 opened: tests: skip security-udev-input-subsystem without /dev/input/by-path [07:30] zyga: just get some rest :) it will be better i'm sure [07:35] PR snapd#5373 closed: release: 2.33.1 [07:36] mvo: thank you [07:36] I'm chopping final bits of packaging updates [07:36] thank you for 2.33.1 [07:43] re https://forum.snapcraft.io/t/gnome-calculator-failed-to-create-symbolic-link/5742/10 , the after-install seeding doesn't look good in cosmic, gets stuck on autoconnect and in the meantime an update of snapd arrives through deb and gets stuck on deb configure step [07:43] PR snapd#5377 opened: packaging/opensuse: remaining packaging updates for 2.33.1 [07:44] mborzecki: ^ [07:44] mborzecki: which distributions did you test? [07:44] popey: do you have a suse box for some testing? [07:45] popey: I'd like to update snapd in openSUSE [07:45] mborzecki: I also have one issue I don't know how to solve [07:45] mborzecki: the new snapd.apparmor.service needs to be enabled manually [07:45] mborzecki: I don't know what to do about it [08:01] mborzecki: could you please have a look at https://build.opensuse.org/request/show/618441 [08:02] Pharaoh_Atem: ^ if you have the time [08:02] Pharaoh_Atem: this syncs the package in system:snappy with the package in snapd master [08:03] remaning diff is: patches, small change in path used to get packaging overrides (we don't have a symlink for packaging/opensuse in 2.33 _yet_). [08:03] in 2.34 the delta should be empty === chihchun_afk is now known as chihchun [08:33] * zyga -> walk === chihchun is now known as chihchun_afk [08:45] mvo, pedronis it appears that seeding+prerequisites explode in cosmic on autoconnect, also with 2.33.1 (i've built the deb manually) [08:46] pstolowski: explode in which sense? [08:46] we have no support atm if the seeding is not sorted right [08:46] pedronis: autoconnect is in state Doing, never completes [08:47] pstolowski: is this a spread test or something else? [08:47] pedronis: there are 6 gnome packages + core in seed/ [08:47] pedronis: no, a fresh install of cosmic daily [08:47] if they are sorted wrong [08:48] no good [08:48] given that seeding is serial [08:49] we should teach seeding to do some sorting [08:49] but that hasn't really changed much === chihchun_afk is now known as chihchun [08:50] pstolowski: I'm not even sure what autoconnect code is exactly in 2.33.1 [08:50] I suppose is not like master [08:51] pedronis: correction, i actually build and tested with current master [08:51] pstolowski: can you print seed.yaml there [08:51] pstolowski: we might have made something worse [08:51] but it was probably working by chance [08:53] or I'm missing something, which could be [08:53] pedronis: http://pastebin.ubuntu.com/p/pPZkdHhp9B/ [08:54] themes is wrong [08:54] there was even a bug [08:54] we told them to move [08:54] not sure why is still like that [08:54] pedronis: i don't think we made it worse, i started with fresh cosmic image (it has 2.28.x), the problem was already visible [08:54] pedronis: oh ah [08:55] zyga: i have a suse vm, but not sure how much time i will have today to help test [08:55] morning all [08:55] * Chipaca having a weird day (but a good'un) [08:58] pstolowski: is still open, though there was a fix: https://bugs.launchpad.net/ubuntu/+source/ubuntu-meta/+bug/1772844 [08:58] bit confused [08:58] Bug #1772844: snapd didn't initialize all the seeded snaps [09:01] pedronis: right, clearly not landed [09:02] pstolowski: we can teach either seeding or autoconnect to do something in that case, I want to have a clearer big picture related to conflicts before going there [09:02] though [09:03] the manual fix seems easy enough, not sure why is not landed [09:03] or is just different teams building different images [09:07] mvo: do you understand what is happening with that bug ? #1772844 [09:07] Bug #1772844: snapd didn't initialize all the seeded snaps [09:09] re [09:09] took longer than i expected [09:57] Hi. I just installed snapd on Antergos from the AUR, and I set it up following the instructions for Arch. Installed the hello snap, and tried to run it. I get this error "cannot locate the core or legacy core snap (current symlink missing?): No such file or directory". snapd version 2.33, kernel 4.17.2-1-ARCH. Any ideas to what may be the issue? [09:58] searedvandal: does Antergos use systemd? [09:58] Chipaca, yes [09:59] phew [09:59] searedvandal: 'snap list' lists nothing? [09:59] searedvandal: can you pastebin the output of 'snap tasks --last=install' [10:00] searedvandal: and pacman -Qi snapd [10:00] Chipaca, https://ptpb.pw/_0LP [10:00] mborzecki, https://ptpb.pw/iJb7 [10:01] hmmmm [10:01] Chipaca, and snap list shows core and hello [10:02] searedvandal: yep. I'm guessing something is expecting /snap instead of /var/lib/snapd/snap or viceversa [10:02] mvo: there's somethign wrong with the 2.33.1 release on github [10:02] searedvandal: what's in /etc/os-release? [10:02] mvo: it doesn't have a tag (?) [10:03] Chipaca, my /etc/os-release https://ptpb.pw/xFrR [10:03] ID_LIKE=archlinux chceks out [10:03] mborzecki, the AUR has the 2.33 build, not the 2.33.1 [10:03] searedvandal: yeah, i'm updating to 2.33.1 now ;) [10:03] Mornings [10:04] mborzecki, ah :) [10:04] when I do 'snap version' the antergos line has this - , is that correct? https://ptpb.pw/rOFw [10:05] searedvandal: probably [10:05] alright [10:05] searedvandal: because it's a rollin' release [10:05] so no 'version' [10:05] ah, yeah [10:07] Chipaca, and I have the /snap directory and the /var/lib/snapd/snap directory [10:07] searedvandal: yes, it's the same on arch [10:07] searedvandal: /snap is just a symlink in your system right? [10:08] searedvandal: can you pastebin find /{var/lib/snapd/,}snap/ -maxdepth 2 -ls [10:09] mborzecki, Chipaca here is the paste https://ptpb.pw/Qp_i [10:10] searedvandal: ls -l / [10:10] mborzecki: that looks like /snap is not a symlink but the real thing [10:10] mborzecki: and /var/lib/snapd/snap has nothing [10:10] mborzecki, https://ptpb.pw/Pj_e [10:10] oh, you're right [10:11] that is very strange [10:11] something's askew (gesundheit) [10:11] searedvandal: yeah, the package ships with /var/lib/snapd/... only [10:11] strange. I have not created any directories or changed anything [10:11] it used to ship /snap -> /var/lib/snapd/snap symlink a while ago but we got rid of that [10:12] searedvandal: can you pacman -Qo /snap ? [10:12] mborzecki: I suspect distro detection being confuse [10:12] mborzecki, no package owns /snap [10:14] in hindsight, we should print a summary of directory info on startup, at least in debug [10:14] searedvandal: ok, if there's any snap mounted, unmount it and remove the package, remove /snap and install the package again [10:14] Chipaca: +1 [10:15] snap debug distro :) [10:16] and now i need to stop myself from doing that and getting back to figuring out why warnings has my snapd all panicky [10:19] mborzecki: let me check, it should have a tag [10:19] alright, you're right, it gets confused on os-detection. I removed a pacman hook that sets the antergos info and updated the filesystem package to revert to identifying as regular arch. now everything seems to work just fine [10:20] snap run hello gives me a nice hello world output [10:20] mvo: https://github.com/snapcore/snapd/releases/tag/untagged-ec50ee5bfb45daefc236 <-- untagged-... [10:22] searedvandal: yay [10:22] mborzecki: better now? sorry, no idea what happened there, github glitch [10:23] so snap doesn't like it when the system is identified as antergos, but when it's identified as arch it's all good :) [10:23] hm otoh we do release.DistroLike(..., "arch", ..) this should probably be "archlinux" to catch the derivatives [10:23] ah, probably :) [10:23] mvo, zyga, I just realized that next week will be the two year anniversary of me working on snappy on Fedora [10:23] mvo: yay! [10:23] :) [10:23] Time flies! [10:24] Son_Goku: we should get you some cake [10:24] two years ago next week, I agreed on a lark to go to the first snappy sprint [10:24] \o/ [10:24] I had received the email while (ironically) being at the Red Hat Summit [10:24] Son_Goku: woah, congrats \o/ [10:24] great to still have you here !! [10:24] * mvo hugs Son_Goku [10:25] and Josh Bressers and Thomas Cameron both told me I should go ahead and do it [10:25] so I did [10:25] (Josh: https://twitter.com/joshbressers; Thomas: https://twitter.com/thomasdcameron) [10:26] at the time, I was really unsure if I was the right person to go to this [10:27] of course, technically, I started two years ago next month, but... [10:27] searedvandal: could revert back to identifying as antegros and rebuild the package with this little fix https://ptpb.pw/Oo6u ? [10:27] because the Heidelberg sprint was July 22, 2016 [10:33] Interesting.. we're importing from the snap package inside client [10:34] This is something we should avoid as otherwise importing the client will depend on importing snapd entirely [10:34] Son_Goku: That reminds me that we need a dev sprint soon [10:35] niemeyer, I was wondering if we were going to go without one this year :) [10:36] PR snapd#5378 opened: dirs: improve identification of Arch Linux like systems [10:38] Son_Goku: We just need to think about what would be the best time and place.. we have non-dev sprints in July and September.. So either August or later in the year [10:39] so I'm going to be at DevConf.us Aug 17-19, so please don't schedule it then [10:39] searedvandal: just opened a PR with this change, if it works in our test suite I'll include it as a patch in 2.33.1 package, so your next rebuild should work fine [10:40] niemeyer, Flock is actually going on in Dresden, Germany on Aug 8-11: https://flocktofedora.org/ [10:40] if zyga wants to talk about snaps at Flock, now would be a good time ;) [10:41] Son_Goku: Ah, interesting [10:41] Very likely [10:41] Son_Goku: Are you going? [10:41] I can't afford to, so sadly no [10:42] niemeyer: we've been depending on snap from client for a long long time [10:42] zyga: almost anything goes for CfP: https://flocktofedora.org/#cfp [10:42] niemeyer: snap is where Revision and Epoch live [10:43] Chipaca: Yeah.. my memory is weak by now, but IIRC the agreement we got about this kind of thing was to revert the dependency, having such types in client itself [10:43] Chipaca: So that importing the client becomes isolated [10:44] Chipaca: The snapd/snap package by now spreads its legs through quite a few bits [10:44] As in, importing deps [10:44] niemeyer, but it'd be cool if Snapcraft was listed as a sponsor for Flock :P [10:44] Which is not a problem for it, to be clear.. it's just that the transitive dependency from client becomes weird [10:45] niemeyer: I thought it just used dirs and the different *util [10:45] Son_Goku: We can check that.. I haven't been involved in sponsorings for a while, so I'm not sure about what the current rules are [10:48] Son_Goku: this Flock is distinct from the flock that is a startup and an app? [10:48] yes... [10:48] Flock is the name for the Fedora contributor conference [10:48] which used to be called FUDCon [10:48] I can see how a contention mechanism is a better name than FUDCon [10:49] but it's not a wide margin :) [11:09] PR snapd#5378 closed: dirs: improve identification of Arch Linux like systems [11:11] mvo: i think the tag can be removed: [11:11] * [new tag] untagged-ec50ee5bfb45daefc236 -> untagged-ec50ee5bfb45daefc236 [11:12] suprising github actually allowed creating this [11:13] re [11:13] * zyga picked up the coffee he made at 7 [11:13] mborzecki, Pharaoh_Atem: can you please eyeball https://github.com/snapcore/snapd/pull/5377 [11:13] it's the last of the bunch [11:13] PR #5377: packaging/opensuse: remaining packaging updates for 2.33.1 [11:21] mvo: hey [11:21] did you have a chance to look at https://github.com/snapcore/snapd/pull/5369 [11:21] PR #5369: overlord,interfaces,cmd: WIP early support for interfaces on core18 [11:21] hey zyga [11:22] zyga: looking at it currently [11:22] :-D ok, let me know once you've read it [11:22] I'll write something on the forum now [11:22] oh my how smooth TrackMania runs, I'm impressed [11:23] zyga: ok [11:27] snap downloads capped at 100kBps again? [11:27] mborzecki: hmm, not for me [11:28] 7MB/s [11:28] hmm 2 times hit 100kBps, another restart and 1.2MB now [11:28] over my LTE link [11:29] I looked at stats and I still cannot stop being impressed by how I can effortlessly live on LTE with 100s of GB of data used monthly [11:29] for the price of a meal in spain [11:29] (one meal, for one person) === pstolowski is now known as pstolowski|lunch [11:36] PR snapd#5379 opened: overlord/snapstate: disallow installing snapd on baseless models [11:36] zyga: I split the snapd prevention bits (and added a trivial test) - I think we deefinitely want this [11:36] prevention bits? [11:36] the snapd.snap removal/installation? [11:36] zyga: disallow installing the snapd snap [11:37] note that it breaks the tests today :/ [11:45] zyga: it does? which one? [11:46] just look at the test failures, all the core 18 tests exploded [11:46] I asked if this was expected (perhaps the way we install snapd is not right) but we didn't talk about it since [11:46] zyga: aha, I will debug it :) [11:46] zyga: "all the core18 tests" - all two of them ;) ? [11:47] zyga: aha, I see what is going on [11:47] * mvo scratches head [11:49] zyga: testing a fix now, and then I will go over the rest. thanks in any case! [11:52] searedvandal: changes are in AUR now [11:52] mborzecki, nice, thanks! [11:55] mvo, wait, does that mean it'd be possible to make snapd snap either-or if a base snap provided snapd? [11:55] I'm confused what PR#5379 actually does [11:58] Son_Goku: we're transitioning from having snapd in core to having snapd in snapd [11:58] :/ [11:58] Son_Goku: we're special casing the special case so we can special case while we special case [11:58] Son_Goku: why :/? [11:58] because I don't understand what's going on here [11:59] I'm confused [11:59] Son_Goku: core 18 won't have snapd [11:59] rather [11:59] Son_Goku: because of core18 and core16 the idea of having snapd in _a_ core is gone [11:59] Son_Goku: core18 does not have snapd [11:59] Son_Goku: neither will core16 [11:59] so snapd moves to its own proper package [11:59] eventually the only snap that ships snapd will be snapd.snap [11:59] Son_Goku: today, core does; core will become core16 and snapd [11:59] hmm [11:59] this will simplify and unify various special cases [12:00] there were special cases? [12:00] and will make it natural to have bases [12:00] Son_Goku: but as core and core16 will co-exist, we want to put little yellow "don't step on the bubbling lava" signs [12:00] yeah, plenty of them [12:00] still are but this will help to remove it [12:00] core wil transition to core16 [12:00] Son_Goku: the end game is _less_ special cases, not more, fwiw [12:00] okay, I think I get it [12:00] Son_Goku: where there must be a snapd snap, but you can have any base (and that base evolves separately from snapd itself) [12:00] Son_Goku: snap-confine will hopefully shrink a little thanks to this [12:01] most of this doesn't really affect me though, since we don't allow re-exec on Fedora [12:01] and snap-run/snap-exec/snapd code will drop some if name == "core" code [12:01] since your snapd breaks things [12:01] * Chipaca hopes Son_Goku doesn't ask what base snapd will use [12:01] I think I have a feeling that snapd will use base snapd [12:01] or worse, base ubuntu-core16 [12:01] * Chipaca votes it uses base 'bo' [12:02] Son_Goku: snapd won't use any base [12:02] or maybe base 'politically-incorrect-inanimate-object' [12:02] it's not a snap that ships apps or services in the traditional sense [12:02] more like a vehicle for runtime bits for any base [12:03] (snap-confine/exec/snapctl chain) [12:03] so it pretty much doesn't affect me [12:03] since we don't use those bits [12:03] no, you are slightly mistaken [12:03] it'd only become a problem once we have a fedora bootable base :/ [12:03] you do use snap-exec and snapctl from core [12:03] that's always been the case [12:04] you don't use snap-confine from core when running snaps from the outside [12:04] Son_Goku: bootable fedora base is happening though, right? [12:04] that's something zyga and I are working toward, yes [12:04] I already know that we're in for hell because I can't swap the snapd :/ [12:04] Chipaca: yes, eventually, I think this cycle we will get fedora29 base for apps and future development though [12:05] * Son_Goku really wishes core16 / core18 would be ubuntu-core16 / ubuntu-core18 [12:05] that's what they actually are, after all [12:08] Son_Goku: I think it's that until there are other bases realistically out there being used, we don't want to have people think they're downloading all of ubuntu just to run tmnationsforever [12:08] well there's already other bases in use [12:08] at least ikey has the one for steam [12:08] * Chipaca misses ikey [12:09] and _a_ fedora base snap will be available by Fedora 29 GA [12:09] I can almost nearly guarantee it [12:10] primarily because I can make one whenever I want [12:10] Son_Goku: one good thing is that "core" will go away and become core{16,18} and those will shrink [12:10] the work zyga and I will be doing this development cycle is integrating this into the infra tooling so it's literally part of the compose [12:10] Son_Goku: I'll raise it in the standup (unless zyga does it first) [12:10] that is, part of the process to make all the release artifacts [12:10] we also considered (though not scheduled to do it) a bootless core which would be even smaller by a large amount [12:11] I think the bootless split was kicked to 20 [12:11] yeah [12:11] too many moving parts [12:11] yep [12:11] Chipaca: raise what exactly? [12:11] zyga: ubuntu-core18 instead of just core18 [12:11] zyga, what does snapd.seeded.service do? [12:11] Son_Goku: wait until snapd is seeded [12:12] Son_Goku: tells you that the system has installed all of the snaps that are in the seed (from the model) [12:12] Son_Goku: it basically tells you that it is ready after first boot [12:12] ah [12:12] so is this something we need to have enabled in the fedora preset? [12:12] before that snapd is busy doing stuff and not really responding [12:12] because right now, it ai't [12:12] *ain't [12:12] Son_Goku: it's useful as target of "after snapd is there" [12:12] Son_Goku: shouldn't hurt, and would let you play with seeds [12:12] we added it because someone needed this kind of thing for another piece of software [12:12] I added it to suse [12:13] it's not core specific, I think it could be in fedora as well [12:16] though maybe not enabled by default right? [12:16] why not? [12:16] because it will wake up snapd [12:16] mmmm [12:16] interesting [12:17] seeded -> snapd.socket -> snapd.service [12:17] can we write the unit in a way that would make it do stuff only if snapd was enabled? [12:17] no [12:17] because you added Requires [12:17] I see [12:17] you forced it on [12:17] then I agree with mborzecki [12:17] yeah, so probably should stay off by default [12:17] i mean, once you know you need it you'll enable it :) [12:18] it's only needed for firstboot if we had them preloaded, right? [12:18] maybe the unit can be adjusted to have firstboot condition [12:19] mmm, interesting [12:19] how could we do that? [12:19] I think that's a pretty good idea [12:19] ConditionFirstBoot= takes a boolean argument. This condition may be used to conditionalize units on whether the system is booting up with an unpopulated /etc directory (specifically: an /etc with no /etc/machine-id). This may be used to populate /etc on the first boot after factory reset, or when a new system instance boots up for the first time. [12:19] zyga: that's a very particular "first boot" condition [12:20] from systemd.unit(5): https://www.mankier.com/5/systemd.unit#ConditionFirstBoot= [12:20] econnreset again, 2018/06/22 11:12:14.534944 store.go:1555: DEBUG: Download succeeded in 5.257s (102MB/s). [12:20] alternatively, you could do ConditionPathExists= and do some custom logic here in the seeding code: https://www.mankier.com/5/systemd.unit#ConditionPathExists= [12:21] like the presence of /var/lib/snapd/{seed} or model or something [12:21] not sure but something like that [12:22] yeah [12:23] something that would make it far less penalizing to have it [12:23] zyga, ConditionPathExists also takes a negation operator [12:23] so !/var/lib/snapd/seeded would also work [12:27] pedronis: hey [12:27] pedronis: I wrote the post that I was supposed to, about the refresh + restart issue [12:27] https://forum.snapcraft.io/t/issue-with-using-snapstate-active-for-interface-repository/6065 [12:27] could you please read it and ensure I didn't miss anything critical or didn't skew the facts [12:31] mborzecki, zyga , what happens if xdg-desktop-portal < 0.11 is installed on fedora for snapd? [12:31] Son_Goku: I don't know the version specifically [12:31] is that the old portal that was not aware of snapd? [12:32] yes [12:32] xdg-desktop-portal 0.9 is in Fedora 27 [12:32] 0.11 is in Fedora 28+ [12:32] portals would not know they are used by a confined app [12:32] zyga: pushed an update to 5370 [12:32] so they would not trigger the portal-specific file open dialogs [12:32] (in cooperating apps) [12:33] the UX would suck [12:33] :/ [12:33] (like it does now, not a regression) [12:33] so it's no worse than today? [12:33] I believe it would be exactly like today [12:33] so I don't need to add a "Conflicts: xdg-desktop-portal < 0.11"? [12:33] one sec [12:33] no no [12:34] https://github.com/snapcore/snapd/pull/5271 [12:34] PR #5271: cmd/snap: attempt to start the document portal if running with a session bus [12:34] scroll to the bottom [12:34] there's some talk about this exact issue === pstolowski|lunch is now known as pstolowski [12:36] so the answer is, openSUSE is fucked but not Fedora [12:37] tumbleweed to? [12:39] yes [12:39] openSUSE has been shipping Flatpak by default for a while now [12:40] it's also shipping in SLE 15, too [12:41] Son_Goku: tumbleweed will probably get the latest version soon [12:41] it has 0.11.7 [12:41] Son_Goku: note that I enabled apparmor on tumbleweed [12:42] Son_Goku: so there's way more confinement than before :) [12:42] eh [12:42] I'm slightly annoyed you didn't merge my changes entry into the OBS package [12:42] I can do that in the final sync [12:42] it is not out yet [12:46] I'm building snapd 2.33.1 and snapd-glib 1.41 now [12:46] niemeyer, mvo: I've got a meeting that I need to be in instead of the standup today [12:47] Chipaca: Ack, thanks for the note [12:47] niemeyer, mvo: it should be short, but if not --- i'm working on warnings, fixing all the tests in daemon/ that i've broken :-) [12:47] figured out the source of the panic, also [12:47] so huzzah [12:48] anyone wants to look at #5363 and #5366? [12:48] PR #5363: snap: introduce the instance key field [12:48] PR #5366: snap: helper for validating snap instance names [12:49] zyga, also, xdg-desktop-portal will now supplement either flatpak or snapd [12:49] err, the gtk and kde portals will [12:50] https://src.fedoraproject.org/rpms/xdg-desktop-portal-gtk/blob/master/f/xdg-desktop-portal-gtk.spec#_22 [12:50] https://src.fedoraproject.org/rpms/xdg-desktop-portal-kde/blob/master/f/xdg-desktop-portal-kde.spec#_40 [12:57] Thank you pedronis ! [12:59] zyga: your description was not too wrong, but seemed to me a bit still too high level [13:00] yes, you got the specific details in place, thank you [13:26] mvo: https://launchpad.net/bugs/1772844 [13:26] Bug #1772844: snapd didn't initialize all the seeded snaps [13:30] pedronis: thank you! [13:31] sil2100: does foundations have https://bugs.launchpad.net/ubuntu/+source/ubuntu-meta/+bug/1772844 on the radar? if not, who should I ask about this? the bug is most likely that the seed.yaml needs to be tweeaks and things needs to be around a bit. happy to talk about the details once I know with whom :) [13:31] Bug #1772844: snapd didn't initialize all the seeded snaps [13:33] mvo: hmm, first time I hear about this! Let me read it up [13:36] niemeyer: experimental.parallel-instances then? [13:37] pstolowski: what's the status of disconnect hooks? [13:37] mborzecki: +1 [13:38] ack [13:38] pstolowski: this is a strange error: Disconnect bluez:client from bluez:service (snapd changed, please retry the operation: cannot disconnect bluez:client from bluez:service, it is not connected) [13:39] pedronis: i think i know why that is, i forgot to disable discard-conns, at the very least [13:40] we do need to keep it though [13:40] pedronis: i'll ping you when it's ready [13:40] pedronis: i know [13:40] but making it idempotent [13:40] pedronis: handler needs to stay, but task shouldn't be created for new changes [13:40] ok, I'm still a bit confused [13:41] I would have expected it to come before, not after [13:41] pstolowski: also notice that these strange error seems about self connections [13:41] pstolowski: are we trying to disconnect them twice? [13:42] pedronis: ah, i didn't think of it, might be the same issue we had with autoconnect [13:42] I doubt because the code would be different [13:42] but maybe the connect does strange things [13:43] anyway an interesting datapoint [13:43] I would expect the discard-conns to do nothing, assuming it happens last [13:43] I'm going offline for a while, will check in tonight again (with moar warnings) [13:43] * Chipaca -> school stuff and such [13:47] pstolowski: ah, mayb repo connections in this case return the same connection twice ? [13:48] pstolowski: yes, the issue is repo.Connections, self connections are reported twice [13:59] pedronis: heh, indeed! i wonder if there is any undesired effect if i fix this centrally (in the repo.Connections) [14:00] it shouldn't afaict, don't think it has a use case [14:18] pstolowski: it shouldn't, it's kind of weird/unexpect to get the same conn twice [14:29] should i install snapcraft from apt or from snap? [14:32] niemeyer: rename to parallel-instances pushed to #5370 [14:32] PR #5370: overlord/{config,snap}state: introduce parallel-installs feature flag [14:32] mborzecki: Thanks! [14:34] ali1234: there is no reason not to use the one from snap; go for the snap; it also makes it easy to switch to a beta version (and back) in case of an important fix coming [14:36] ali1234: i use the snap, personally. [14:36] pedronis: pushed the repo fix but discard-conns still needs work (undo for single disconnect) [14:53] * zyga fixed all the entries in the opensuse snapd.changes changelog to have non-fake copy/pasted/tweaked dates [14:54] Pharaoh_Atem: ^ that's for you :) [14:54] :) [14:55] apparently copy/pasting and tweaking seconds is ... popular [14:55] :/ [14:56] zyga: my bad - I need an emacs mode or something of this. or obs-dch or whatnot [14:56] mvo: "ons vc" [14:56] er [14:56] "osc vc" [14:57] (sorry, spellchecker doesn't love acronyms) [14:57] that edits the changelog [14:58] well, to be fair it'd be nice if changelog included package version/release, otherwise rpm -q --changelog is not really that useful [14:58] nice, thank you [14:58] I hope I remember [14:58] the changelog is still wrong [14:58] sigh [14:58] someone wrote a tool that returns false [14:58] rather than "on line 123, ..." === chihchun is now known as chihchun_afk [15:02] zyga: on line 123, .. false :) [15:03] gah, the bugzilla email apocalypse [15:03] :D [15:04] btw emacs' rpm-mode can add changelog entries, but it's not aware of opensuse's *.changes file split [15:04] which is actually annoying [15:04] and also I'm working on fixing `osc vc` to properly generate entries with full author identities [15:08] zyga, mborzecki, niemeyer, mvo: https://forum.snapcraft.io/t/snapd-updates-in-fedora/4342/5?u=conan_kudo [15:08] zyga: I found the issue with 5369 - ERROR installation not allowed by "daemon-notify" slot rule of interface "daemon-notify" [15:08] zyga: this is when it tries to mount the core snap [15:25] mvo: zyga's RFC branch has code for that [15:25] yeah [15:25] you need to take that patch (that really pedronis wrote :) [15:28] pedronis: yeah, thank you, I figured it out by now, I added a fix that should make these tests work [15:29] mvo: shall i land #5374 or do you plan to push more patches? [15:29] PR #5374: tests: disable core tests on all core systems (16 and 18) [15:35] when i run snapcraft: E:Failed to fetch copy:/home/al/.cache/snapcraft/stage-packages/apt/.../var/lib/apt/lists/partial/security.ubuntu.com_ubuntu_dists_bionic-security_main_dep11_icons-48x48.tar.gz Hash Sum mismatch [15:36] "..." is a very long hex string [15:36] PR snapd#5374 closed: tests: disable core tests on all core systems (16 and 18) [15:37] mvo: https://github.com/snapcore/snapd/pull/5379 can land as well [15:37] shall I? [15:37] PR #5379: overlord/snapstate: disallow installing snapd on baseless models [15:51] mborzecki: 5374 should be good [15:52] zyga: same for 5379 [15:52] there is a comment on 5374 [15:53] PR snapd#5379 closed: overlord/snapstate: disallow installing snapd on baseless models [15:56] Pharaoh_Atem: I wrote a tool for checking changelog entries [15:56] that's much nicer than the original [16:00] hi [16:03] niemeyer: where is the meeting taking place? [16:03] zyga: We're in the standup [16:03] i'm having this bug on snapcraft.io website [16:03] https://github.com/canonical-websites/snapcraft.io/pull/687 [16:04] but it seems resolved by this PR [16:04] i've tried to upload a 48px icon to a snap => 502 [16:05] 5309 is hopefully ready now if someone wants to do a second review [16:13] commented the PR [16:14] anyway, i'm happy/proud, my first snap is released on stable :) https://snapcraft.io/timeline [16:15] with a ridiculus small icon, but the dev team can't find the source file :p [16:16] see u ++ [16:43] why am I being denied access to files in core? Log: apparmor="DENIED" operation="open" profile="snap.mycroft.bus" name="/snap/core/4830/lib/x86_64-linux-gnu/liblzma.so.5.0.0" pid=16522 comm="python3" requested_mask="r" denied_mask="r" fsuid=1000 ouid=0 [16:53] zyga: oh? [16:53] yeh [16:53] one moment [16:53] I'm in a call [16:57] zyga: can you take another look at #5326 (doesn't have to be today, Monday is fine)? [16:57] PR #5326: api/snapctl: allow -h and --help for regular users [16:57] PR snapd#5380 opened: tests: blacklist more main tests for core18 [17:10] Pharaoh_Atem: wanna see it? [17:12] niemeyer: have a good weekend too :) [17:13] Pharaoh_Atem: halp [17:13] %{!?systemdgeneratordir: %global systemdgeneratordir %{_prefix}/lib/systemd/system-generators} [17:13] this doesn't produce a path that is absolute [17:13] zyga: running setup-profiles twice is not a problem right? [17:13] no, not a problem [17:14] Pharaoh_Atem: undo help, I see the typo now [17:18] zyga: the answer is yes [17:19] cool, just polishing a bit [17:25] Pharaoh_Atem: https://gitlab.com/zygoon/rpmtools/blob/master/changes-checker [17:26] Pharaoh_Atem: I need to take a break from work and ... buy some DVD-RWs [17:26] (don't ask why) [17:26] I should actually hop on my bike and go to a mall nearby [17:28] zyga: now a remember though there is wrinkle which that setup-profiles and the active managing *link* tasks are not symmetrics at the moment [17:29] zyga: on the Do path we have unlink-current-snap and link-snap but only one setup-profiles doing both [17:29] hmm hmm [17:30] pedronis: I will think about this over weekend [17:30] we really to split the tasks over those two somehow [17:30] or we get again a misaligned state [17:30] zyga: what I mean for example on undo the undo of unlin-current-snap should resetup the profile of the old revision [17:31] that's not how it works now though [17:31] is done much earlier in the undo of setup-profiles [17:33] zyga: conceptuall the sequence should have been (but isn't also because of optimisation): unlink-current-snap -> remove-profiles -> ... -> setup-profiles -> link-snap [17:34] mmm [17:34] that's really the sequence we want to collapse into the *link* tasks [17:34] to get correct state behavior [17:34] otherwise we still the bug on undo [17:34] setup profiles has correct undo handling _now_ but I see how the collapse needs to change that [17:34] yes, it doesn't work [17:34] because again we would on undo setup profiles [17:35] while the snap is inactive [17:35] it needs to be done in the undo of unlink-current-snap [17:35] right, I agree; when I said correct I meant undo (if it has the right state) does the setup for the correct revision [17:35] yep [17:36] mborzecki, Pharaoh_Atem: can you ack https://build.opensuse.org/request/show/618540 [17:36] (if you agree :) [17:36] this is up-to-date with master and https://github.com/snapcore/snapd/pull/5377 [17:36] PR #5377: packaging/opensuse: remaining packaging updates for 2.33.1 [17:36] so please review 5377 first [17:36] and if that is ok, ack the sync request [18:35] PR snapd#5329 closed: DON'T REVIEW: tests: Adding debug information to know why econnreset is failing [19:00] * cachio akf [19:01] re! [19:01] Pharaoh_Atem: how do you like it? [19:02] * zyga feels much better today, having spent a few hours outside [19:02] I should plan an active outdoor weekend [19:03] Son_Goku: how do you like my rpmtools script? [19:03] zyga, hadn't seen it, sorry [19:04] wasn't at my desk [19:04] no worries [19:04] I will play with it to be more full-featured [19:06] pstolowski: Sorry, the meeting has overrun [19:06] pstolowski: I'm available to do it now if you still want to [19:07] zyga, it looks nice! [19:07] I'll make it read the rest of the changelog and model it [19:07] stick a license on top [19:07] and then look at spec files [19:07] niemeyer: yep, let's [19:07] to read embedded changelogs from fedora-like systems [19:07] pstolowski: Cool, heading to the standup [19:08] zyga: nice [19:08] pstolowski: what's the topic? [19:08] note that RPM now supports two time styles [19:08] hotplug? [19:08] zyga: yes [19:08] PR snapd#5381 opened: interfaces: make findSnapdPath smarter [19:08] can I listen? [19:08] the one in changes files, and the one you've seen in spec files [19:08] zyga: sure [19:08] Son_Goku: oh? they are different? [19:08] I didn't notice [19:08] well [19:08] some testing required :) [19:08] I'll write a spec file _for_ this tool [19:09] cool, I'll keep you in the loop [19:09] Son_Goku: I also plan to see how gitlab CI works [19:10] to see if I can do a pipeline with mypy/tests, package builds and maybe even publishing [19:10] zyga: niemeyer: there is another thing to consider with the plan we discussed, which is what to do about things guaranteed by ifacemanager Runner blocked predicate, now we would have potentially tasks across two managers/runners [19:11] ah, because setup-profiles would block in the old world but not in the new one (naively) without extra help, correct? [19:14] yes [19:14] and blocking is messy because two managers/runners [19:15] unless we set them up with one runner [19:15] which is theoritecally possible [19:15] but the predicate would be more involved than the current one [19:20] zyga: I wrote something in the topic about what I understand of the plan and also describing this issue [19:21] Thank you, I will look at it after the call === pstolowski is now known as pstolowski|afk [19:48] Pharaoh_Atem: do you want to review the changes to opensuse packaging? [19:48] or shall I pull the trigger and just merge it? [19:52] * zyga merges it [19:53] PR snapd#5377 closed: packaging/opensuse: remaining packaging updates for 2.33.1 [20:44] kyrofa ping [20:48] Hey there ondra [20:48] kyrofa hey [20:48] You're up late! [20:48] kyrofa just quick question :) [20:49] Hit me [20:49] kyrofa I know :( [20:49] kyrofa is there plan to support layouts definitions natively or we will rely on passthrough? [20:50] ondra, oh certainly, just using passthrough while layouts are unstable [20:50] kyrofa I have situation where I need to overlay ust/lib//something [20:51] ondra, feel free to experiment using passthrough, it won't suddenly break once we introduce official support [20:51] kyrofa and struggling to come up with syntax to make it combined with "- to armhf:" [20:51] Oh, that'll be problematic indeed [20:52] kyrofa so I use passthrough for this in other projects [20:52] ondra, your concern is the I'm assuming? [20:52] kyrofa but here I'd need in top level support for "to : [20:52] Yeah, no such thing I'm afraid [20:52] This is just to support the in there, right? [20:53] kyrofa I guess I cannot use "to in top level right? [20:53] kyrofa well there could be two ways [20:53] Correct. But try replacing with $SNAPCRAFT_ARCH_TRIPLET [20:53] kyrofa ah that is there? [20:53] kyrofa let me try [20:54] this could solve it [20:54] ondra, only in snapcraft, but it'll should turn into a text replacement for the resulting snap.yaml [20:54] PR snapd#5382 opened: tests: add halt-timeout to google backend [20:56] kyrofa damn close [20:56] kyrofa so it replaces only one [20:56] I'm not sure I understand [20:57] kyrofa https://paste.ubuntu.com/p/QWpVVsk6MC/ [20:58] ondra, nooo, that's terrible [20:58] :P [20:59] Let me look into that real quick [20:59] kyrofa how's that possible? :D [20:59] kyrofa thanks bunch! [20:59] I suspect a naive search/replace [21:02] kyrofa no I think I know why [21:02] kyrofa it's deliberate [21:03] kyrofa because when you for example use there $SNAP_COMMON [21:03] kyrofa this cannot be replaces there it has only meaning on actual install [21:03] kyrofa so I guess there is some rule [21:04] kyrofa but you do not expect this to be used in actual target, as there is no point overlay your own files, since you have control over those anyway [21:05] kyrofa but may be we should treat snapcraft's own environment variables differently as those on opposite have no meaning on actual install [21:06] Indeed, and we do. The replacement isn't very smart, it seems [21:07] We pass everything through except these specific types of things [21:15] kyrofa is there easy fix for this? [21:16] ondra, yes, but it's a snapcraft fix, working on it now [21:16] ondra, I can give you a branch once I'm done, if that's helpful [21:16] kyrofa ah, thanks buch! [21:16] kyrofa yeah [21:16] Okay give me a few [21:16] kyrofa sure :) [21:20] kyrofa: note, layout spec is stable [21:20] it's the implementation that varies [21:20] I think it'd be nice if we could get layouts yaml into snapcraft proper now [21:21] and hey :) [21:21] yes, it's late [21:21] * zyga experiments with opensuse [21:25] zyga :) [21:25] zyga so need for snap set core has been removed? [21:26] ondra, any chance you could log a bug for me? Got it licked [21:26] ondra: not yet [21:26] but that's the implementation [21:26] not the yaml [21:26] kyrofa: licked? [21:26] kyrofa sure, I can do that [21:26] * zyga burns some recovery DVDs [21:27] zyga, snapcraft can't break backward compat. As long as you have it behind a feature flag, you can. Until you can't, seems imprudent to bake it into snapcraft [21:27] mm [21:27] k [21:27] I'll do my best to get it off feature flags [21:27] If we didn't have passthrough, it would be different [21:28] zyga, you've never heard someone say "licked" before? [21:28] no, that's why I'm curious [21:28] I recall some cartoon [21:28] zyga I'm all for that feature, I have even git snap pending for release :D [21:28] when it was clear that this has a second meaning [21:29] zyga, it's a colloquialism, just meaning to beat a problem [21:30] I see [21:30] neat [21:30] lol [21:32] * zyga mutters something about midnight petroleum [21:32] kyrofa https://bugs.launchpad.net/snapcraft/+bug/1778287 [21:32] Bug #1778287: snapcraft not replacing all occurrences of $SNAPCRAFT_ARCH_TRIPLET [21:33] kyrofa I had even evil hack idea to fix it on LP, unfortunately not working [21:33] kyrofa I called from one override-build: sed -i 's/\/usr\/lib\/.*\/ondra/\/usr\/lib\/'"$SNAPCRAFT_ARCH_TRIPLET"'\/ondra/g' ../../../snap/snapcraft.yaml [21:34] kyrofa but one can't cheat snapcraft :) [21:38] * cachio afk [21:39] Death, taxes, and snapcraft [21:42] PR snapcraft#2166 opened: project_loader: replace dict keys as well as values [21:42] ondra, that's ^ the one you want. I'm building a snap of it now [21:43] kyrofa how can I test with snapcraft-pr? [21:43] snapcraft-pr.init 2166 [21:43] Then use `snapcraft-pr 2166` instead of `snapcraft` in any command [21:43] PR #2166: snap: stop using ubuntu-core-launcher, use snap-confine [21:50] kyrofa running clean build now [21:51] ondra, snapcraft-pr will get angry if you try to cleanbuild. It's the same as running from source: you can't cleanbuild, it'll just pull in the deb [21:51] ondra, if you want to cleanbuild, wait for the snap, or install snapcraft-pr inside a clean container and run it normally in there [21:52] LP claims the amd64 snap is 8 minutes away [21:52] kyrofa worked :D [21:52] kyrofa this is simple snap, only stage packages + dump plugin [21:52] Awesome! Mind mentioning that on the PR? [21:53] ondra, would releasing the snap in a branch be helpful to you, or should I cancel that? [21:54] kyrofa you can cancel that [21:55] kyrofa I'd need it in LP more [21:55] kyrofa but not super urgent [21:55] kyrofa I can manually repack [21:55] ondra, yeah, sorry, not a log I can do about that, but it'll be in the next release [21:56] kyrofa yeah I know for that we need LP start using snapcraft as snap [21:56] kyrofa but apparently in works [21:57] kyrofa but nothing urgent for me now, so I can wait for next release :) [21:57] kyrofa thanks bunch for quick fix though :) [22:02] ondra: you can select it manually [22:02] cjwatson how can I do that? [22:03] ondra: currently API-only - see auto_build_channels on https://launchpad.net/+apidoc/devel.html#snap, or channels on https://launchpad.net/+apidoc/devel.html#snap-requestBuild [22:05] cjwatson this is awesome, I can see one can choose edge of core :D [22:05] (I have a patch in the review queue to add web UI for it, but you can use it without that) [22:05] yeah, that's mostly intended for QA [22:06] cjwatson pretty cool [22:06] thanks [22:06] ondra, of course!