=== chihchun_afk is now known as chihchun
[06:09] <zyga> Hello
[06:10] <mborzecki> morning
[06:13] <zyga> hey maciek
[06:13] <zyga> I must ask you, are your kids already in school?
[06:13] <zyga> how are you doing it so that you start so early :)
[06:30] <mborzecki> zyga: my wife drives them to school ~7am
[06:42] <zyga> mmmm
[06:54] <mborzecki> zyga: btw. desktop-portal-activation strikes again on fedora 29
[07:16] <mup> PR snapd#6254 closed: tests: improve how the log is checked to see if the system is waiting for a reboot <Created by sergiocazzolato> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6254>
[07:18] <zyga> mborzecki: I saw it fail a bunch of times yesterday
[07:18] <zyga> any ideas?
[07:21] <mborzecki> zyga: running the suite now, i suppose xdg-desktop-portal is installed for some reason and gets activated instead of the helper used by test
[07:22] <zyga> mmm
[07:22] <zyga> ok
[07:22] <mborzecki> zyga: btw. we've had weak dependencies in dnf disabled earlier for installation from fedora repos
[07:23] <zyga> yeah, I remember
[07:30] <mborzecki> zyga: have you repened snapd.mk pr?
[07:30] <zyga> mborzecki: no, I bailed from most work yesterday
[07:31] <zyga> my daughter is going to school today
[07:31] <zyga> I feel like going to bed instead
[07:31] <mborzecki> zyga: ah, how are you feeling today?
[07:31] <zyga> I can reopen it quickly
[07:31] <zyga> same as yesterday, let's see how the day unfolds
[07:34] <mup> PR snapd#6111 opened: packaging/opensuse: move most logic to snapd.mk <Created by zyga> <https://github.com/snapcore/snapd/pull/6111>
[07:34] <zyga> mborzecki: ^ reopened
[07:34] <zyga> mborzecki: since tests are hit and miss
[07:34] <zyga> I would land as-is
[07:34] <zyga> and try to iterate on top
[07:58] <mup> PR core18#94 opened: hooks: add cloud-init <Created by mvo5> <https://github.com/snapcore/core18/pull/94>
[08:02] <zyga> mborzecki: I've prepared the extra changes on top of the vanilla snapd.mk
[08:02] <mborzecki> ack
[08:02] <zyga> I'll verify that nothing is missing and push them as well
=== pstolowski|afk is now known as pstolowski
[08:07] <pstolowski> morning
[08:08] <zyga> o/
[08:08] <mborzecki> pstolowski: hey
[08:12] <ackk> hi, are there plans to SRU snapcraft 3 to bionic, or is snap the way to go?
[08:18] <pstolowski> ackk: sergiusens may be able to answer this, but he will probably show up in a few hours
[08:19] <ackk> pstolowski, thanks
[08:20] <zyga> mborzecki: much nicer now, I also added the with_... variables so the spec is shorter and nicer as well
[08:23] <zyga> mborzecki: pushed now
[08:23] <zyga> please look :)
[08:26] <zyga> messed up some history, pushed again
[08:26] <zyga> the new thing is https://github.com/snapcore/snapd/pull/6111/commits/e3f89e4e9ca3b9081375291daf9e6ee45ab9b1cc
[08:26] <mup> PR #6111: packaging/opensuse: move most logic to snapd.mk <Created by zyga> <https://github.com/snapcore/snapd/pull/6111>
[08:26] <zyga> but this is still broken, man
[08:26] <zyga> my built tree :)
[08:27] <zyga> sorry
[08:29] <zyga> and pushed again
[08:29] <zyga> https://github.com/snapcore/snapd/pull/6111/commits/da6ddecb72b9b0a8abf775b96938c1cc80456a94
[08:29] <mup> PR #6111: packaging/opensuse: move most logic to snapd.mk <Created by zyga> <https://github.com/snapcore/snapd/pull/6111>
[08:32] <mborzecki> zyga: https://www.youtube.com/watch?v=d2E_iiTd9hQ
[08:34] <zyga> mborzecki: evil in me wants to call spread with a fixed seed so that tests are less insanely annoying
[08:38] <zyga> mborzecki: I see and I raise https://www.youtube.com/watch?v=Pmd3UiNfNkA ;D
[08:38] <mborzecki> hahah
[08:53] <zyga> this doesn't look like a happy day
[08:53] <zyga> mborzecki: shall we disable that test until it is sorted out?
[08:54] <mborzecki> zyga: yeah, on fedora at least
[09:06] <zyga> mborzecki: can you please?
[09:08] <mborzecki> there's something fishy about the fedora image we use under spread
[09:14] <mborzecki> zyga: take a look at this https://paste.ubuntu.com/p/W3yxdGYTxW/
[09:14] <zyga> mmm
[09:14] <zyga> so
[09:14] <zyga> those are different images,
[09:14] <zyga> the /etc/os-release will confirm as much
[09:15] <zyga> I bet those pull in different defaults
[09:15] <zyga> IMO our spread image looks like a workstation image
[09:15] <mborzecki> dnf version is different, spread has 4.0.4, cloud has 4.0.9
[09:15] <zyga> and the cloud image is a genuine cloud image
[09:15] <mborzecki> yup
[09:15] <zyga> what does /etc/os-release say?
[09:15] <mborzecki> sec, it's blocked installing upgrades
[09:31] <mborzecki> zyga: package lists diff https://paste.ubuntu.com/p/MGRxmnZ9cB/ pkgs-there is the spread image, pkgs-here is the cloud image i'm using
[09:31] <mborzecki> idk why there's a bunch of desktop related package
[09:32] <zyga> Looking
[09:32] <zyga> Haskell?
[09:33] <mborzecki> shellcheck probably or sth
[09:33] <mborzecki> gnome-online-accounts?
[09:34] <mborzecki> aahhh
[09:34] <mborzecki> damn, those are the images that have dependencies installed
[09:34] <mborzecki> zyga: iirc that was something about reducing how long it takes to run a test
[09:35] <mborzecki> s/a test/test suite/
[09:36] <zyga> Yeah, probably exactly that
[09:36] <mborzecki> damn, those are the images that have dependencies installed ents xdg-desktop-portal-gtk gives
[09:36] <mborzecki> (gtk3 and (flatpak or snapd))
[09:36] <mborzecki> so yeah, gtk3 is installed, and it's installing snapd
[09:36] <mborzecki> though pulling in flatpak feels like a bug
[09:37] <mborzecki> but hey, xdg-desktop-portal has flatpak >= 0.11.1 in requires
[09:38] <mborzecki> so this feels like a bug in xdg-desktop-portal packaging
[09:42] <mborzecki> zyga: wow, it really does require flatpak to build even
[09:44] <zyga> For real?
[09:44] <zyga> It partially makes sense since it was split out only recently
[09:47] <mborzecki> yeah
[09:50] <mborzecki> zyga: maybe we should just disable weak deps on fedora
[09:51] <mborzecki> zyga: i'm writing an elaborate workaround, but the simplest solution to jsut disable it feels most appealing
[09:55] <zyga> I’m eager to see both
[09:55] <zyga> now
[09:55] <zyga> I’m afk for some time no
[09:56] <mup> PR snapd#6271 opened: tests/lib/pkgdb: disable weak deps on Fedora <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6271>
[09:56] <mborzecki> zyga: ^^
[09:57] <mborzecki> zyga: elaborate was (if fedora && installing snapd) -> disable weak deps
[10:24] <zyga> Brb
[10:31] <mup> PR core18#95 opened: hooks: add foreign libc6 on amd64/arm64 to enable bi-arch support <Created by mvo5> <https://github.com/snapcore/core18/pull/95>
[10:35] <mup> PR snapd#6272 opened: overlord/snapstate: run 'remove' hook before 'auto-disconnect' <Created by stolowski> <https://github.com/snapcore/snapd/pull/6272>
[10:38] <zyga> Approved the
[10:38] <zyga> Approved the pr mborzecki
[10:38] <mborzecki> thx, we'll see if it works in practice or not
[10:39] <mborzecki> zyga: damn, that upgrade to f29 was tricky, lots of little things falling apart
[10:40] <mborzecki> zyga: left some comments under snapd.mk pr
[10:53] <zyga> mborzecki: wow, https://github.com/snapcore/snapd/pull/6111 got green :)
[10:53] <mup> PR #6111: packaging/opensuse: move most logic to snapd.mk <Created by zyga> <https://github.com/snapcore/snapd/pull/6111>
[10:53] <zyga> thanks, looking now
[10:55] <mup> PR snapd#6273 opened: centos: enable SELinux support on CentOS 7 <SELinux> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6273>
[10:57] <mup> PR core18#95 closed: hooks: add foreign libc6 on amd64/arm64 to enable bi-arch support <Created by mvo5> <Merged by sil2100> <https://github.com/snapcore/core18/pull/95>
[11:03] <zyga> mborzecki: pushed to https://github.com/snapcore/snapd/pull/6111
[11:03] <mup> PR #6111: packaging/opensuse: move most logic to snapd.mk <Created by zyga> <https://github.com/snapcore/snapd/pull/6111>
[11:03] <zyga> mborzecki: I'd like to land that now and tweak various with settings, unless you disagree
[11:05] <mborzecki> zyga: sounds good to me, but maybe ping Son_Goku about that %bcond_with thing
[11:05] <zyga> sure
[11:05] <zyga> Son_Goku: hey
[11:06] <zyga> Son_Goku: I've resurrected https://github.com/snapcore/snapd/pull/6111
[11:06] <mup> PR #6111: packaging/opensuse: move most logic to snapd.mk <Created by zyga> <https://github.com/snapcore/snapd/pull/6111>
[11:06] <zyga> it now implements the split makefile with definitions idea
[11:06] <zyga> I think you understand various with/bcond semantics in rpm better than I do
[11:06] <zyga> I'd like to merge that branch as soon as it is green (a luxury lately)
[11:06] <zyga> but I'm happy to improve things on top
[11:07] <zyga> namely the set of variables used
[11:07] <zyga> for easier fedora comparison
[11:07] <zyga> as well as the bcond/with use
[11:07] <zyga> for that second one I'm happy to take suggestions as I struggled with it and got nowhere
[11:24] <zyga> I’m afk again
[11:34]  * pstolowski lunch
[11:36] <mborzecki> pstolowski: can you take a look at https://github.com/snapcore/snapd/pull/6271 ?
[11:36] <mup> PR #6271: tests/lib/pkgdb: disable weak deps on Fedora <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6271>
[11:42] <zyga> It should also be critical
[11:42] <zyga> Anything that fixes master to some degree
[11:43] <mborzecki> zyga: let's start with green :)
[11:44] <mborzecki> anyways, it's green
[11:49] <cachio> mborzecki, hey
[11:49] <cachio> we had some issues to update fedora 29
[11:50] <cachio> mborzecki, have you seen this before https://paste.ubuntu.com/p/nMhRYJPwRc/ ?
[11:51] <mborzecki> cachio: nope
[11:51] <cachio> ok, mborzecki I am gonna debug it
[11:51] <cachio> I need to create a test image first
[11:52] <mup> PR snapd#6244 closed: release: detect too old apparmor_parser <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6244>
[11:55] <cachio> zyga, mborzecki I have the end of year event in my daughter's school
[11:55] <cachio> zyga, mborzecki not sure if I'll be back for the standup
[11:56] <cachio> currently I am working on making the snapd test suite work on rigado
[11:56] <cachio> and 2 issues
[11:56] <cachio> 1 fedora 29 is failing to update
[11:56] <cachio> and 2 failover test still fails to install the dangerous snap
[11:59] <mborzecki> cachio: fedora 29 as in dnf upgrade -y ?
[11:59] <zyga> Ack
[12:01] <cachio> mborzecki, yes
[12:01] <mborzecki> cachio: what's the problem then? i've run it today in spread shell, worked just fine
[12:02] <cachio> mborzecki, not sure yet, it failed during the nightly execution
[12:02] <cachio> I am creating a test image right now to debug it
[12:07] <cachio> mborzecki, I have to go now, I'll continue with this when I am back
[12:07] <mborzecki> ok
[12:08] <cachio> mborzecki, the problem is with autoremove
[12:08] <cachio> + dnf -q -y autoremove
[12:08] <cachio> Error:
[12:08] <cachio>  Problem: The operation would result in removing the following protected packages: kernel-core
[12:09] <cachio> mborzecki, that's what I saw
[12:09]  * cachio afk
[12:23] <mup> PR snapd#6274 opened: cmd, dirs, interfaces/apparmor: update distro identification to support ID="archlinux" (2.36) <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6274>
[12:24] <mborzecki> zyga: pstolowski: ^^ simple cherry pick
[12:27] <pstolowski> k
[12:33] <zyga> Mhmm
[12:33] <popey> no chipaca or mvo today?
[12:33] <zyga> popey: that’s correct
[12:33] <zyga> All week in fact
[12:34] <popey> ah okay, thanks!
[12:34] <zyga> Do you need something?
[12:34] <popey> we have a regular catch up with them, but if they're not around that's okay.
[12:34] <popey> We onlyt had one thing to discuss :)
[12:34] <popey> You may know the answer.
[12:34] <popey> What's the support status for 14.04?
[12:34] <zyga> Shoot :-)
[12:35] <zyga> Still supported, Extended support status unclear
[12:35] <zyga> We need clear indication if snapd is part of the extended support package
[12:35] <popey> https://forum.snapcraft.io/t/cannot-perform-operation-no-such-file-or-directory/8870
[12:36] <popey> ^ for example.
[12:36] <popey> I think Wimpress also had an issue with a core18 snap on 14.04
[12:36] <zyga> I’ll check after lunch
[12:36] <zyga> Thanks for bringing this up
[12:36] <Wimpress> zyga: Do you know the status of the font cache fixes?
[12:37] <Wimpress> I know the binaries we're in the core image in edge. But what about the PR for snapd?
[12:37] <zyga> Yes, they are in 2.36.2
[12:37] <Wimpress> Everything? All done?
[12:37] <zyga> As soon as that hits stable we are good
[12:37] <zyga> I believe so
[12:37] <zyga> However
[12:37] <zyga> We know more work is needed
[12:37] <Wimpress> Yeh, I know this is a first pass.
[12:37] <zyga> mborzecki found issues on Fedora
[12:38] <Wimpress> But, my tests are very positive
[12:38] <Wimpress> And this could unblock the release of a popular editor.
[12:38] <zyga> Yes
[12:38] <zyga> Wink wink :-)
[12:38] <Wimpress> ;-)
[12:39] <Wimpress> OK, thanks. That is enough to be going on with.
[12:39] <Wimpress> The 14.04 thing keeps coming up.
[12:39] <Wimpress> We need clear guidance on that.
[12:39] <zyga> Agreed
[12:39] <zyga> Who can decide?
[12:41] <mborzecki> emacs as a snap? :)
[12:43] <Wimpress> zyga: Perhaps JamieBennett can?
[12:44] <jdstrand> diddledan: hi! glad to hear you're picking pushing the sem_open to snapcraft-preload. note there is a TODO in there I'd like to implement. I'll ping you when I do it (I'll do it this morning)
[12:45] <diddledan> ok, thanks :-)
[12:46] <jdstrand> diddledan: were you able to test it with your application?
[12:47]  * jdstrand was thinking about diddledan while doing the investigation
[12:47] <diddledan> I'm just finishing the compile now and will test soon
[12:47] <jdstrand> neat :)
[12:49] <diddledan> I've actually got two apps I want to try it with - makemkv and mycroft
[12:49] <diddledan> although I think mycroft is actually shm that's wonky, not semaphores
[12:50] <diddledan> err. I mean makemkv with the shm
[12:50] <diddledan> mycroft is python multiprocessing, so that should work now, hopefully
[12:52] <mborzecki> zyga: pushed more tweaks to #6273
[12:52] <mup> PR #6273: centos: enable SELinux support on CentOS 7 <SELinux> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6273>
[12:58] <mborzecki> off to pick up the kids
[13:03] <Son_Goku> zyga, I wonder if we're going to get to a point where the openSUSE conditionals are absorbed into the Fedora spec as well
[13:04] <Son_Goku> at this point, there isn't that much else left to make that happen
[13:22] <zyga> Looking
[13:24] <zyga> Son_Goku: maaaaybe
[13:29] <mup> PR snapd#6271 closed: tests/lib/pkgdb: disable weak deps on Fedora <Simple 😃> <Created by bboozzoo> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6271>
[13:56] <zyga> I’ll skip standup today
[14:01] <pstolowski> zyga: ack
[14:01] <pstolowski> cachio: standup?
[14:02] <pstolowski> cachio: ah, you skipping as well, sorry, missed that
[14:12]  * diddledan sits down
[15:19] <mup> PR snapcraft#2420 opened: static: update to the latest flake8 <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2420>
[15:30]  * cachio lunch
[15:34] <mup> PR snapcraft#2421 opened: tests: remove obsolete snap and external tests <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2421>
=== chihchun is now known as chihchun_afk
[16:15] <jdstrand> diddledan: fyi, https://forum.snapcraft.io/t/python-multiprocessing-sem-open-blocked-in-strict-mode/962/15
[16:16] <diddledan> thanks jdstrand
[16:17] <jdstrand> now if github would actually show me the branch I pushed, I could submit the PR
[16:17] <jdstrand> (to snapd)
[16:18] <diddledan> :-)
[16:22] <jdstrand> is github behaving weird for anyone else?
[16:23] <jdstrand> I go to me https://github.com/jdstrand/snapd/ and it just spins on 'Fetching last commit'
[16:23] <jdstrand> and I can look at my branchs
[16:24] <jdstrand> huh, hmaybe it is firefox. chromium is acting differently
[16:28] <mup> PR snapd#6275 opened: apparmor: allow hard link to snap-specific semaphore files <Simple 😃> <Created by jdstrand> <https://github.com/snapcore/snapd/pull/6275>
[16:28] <sergiusens> jdstrand: browser restart to pick up updates maybe?
[16:30] <jdstrand> sergiusens: I did that. it seems that github added something that noscript was blocking. allowed that and it works again
[16:34] <mup> PR snapd#6276 opened: apparmor: allow hard link to snap-specific semaphore files - 2.36 <Simple 😃> <Created by jdstrand> <https://github.com/snapcore/snapd/pull/6276>
=== pstolowski is now known as pstolowski|afk
[17:44] <mup> PR snapd#6275 closed: apparmor: allow hard link to snap-specific semaphore files <Simple 😃> <Created by jdstrand> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6275>
[17:44] <mup> PR snapd#6276 closed: apparmor: allow hard link to snap-specific semaphore files - 2.36 <Simple 😃> <Created by jdstrand> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6276>
[18:31] <mup> PR snapd#6273 closed: centos: enable SELinux support on CentOS 7 <SELinux> <Simple 😃> <Created by bboozzoo> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6273>
[18:53] <mup> PR snapcraft#2420 closed: static: update to the latest flake8 <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2420>
[18:59] <zyga> Pharaoh_Atem: hello
[18:59] <zyga> do you have a moment?
[19:02] <Pharaoh_Atem> zyga: hm?
[19:02] <zyga> hey!
[19:02] <zyga> I'd like to ask about https://github.com/snapcore/snapd/pull/6111
[19:02] <mup> PR #6111: packaging/opensuse: move most logic to snapd.mk <Created by zyga> <https://github.com/snapcore/snapd/pull/6111>
[19:02] <zyga> I would like to merge it but I'd rather not do it without your approval
[19:02] <zyga> could we come up with some plan to move forward
[19:03] <Pharaoh_Atem> let me look over it this weekend, and let me see if I can give you actionable feedback to move forward
[19:03] <zyga> thank you
[19:47] <mup> PR snapcraft#2422 opened: clean: user friendly message when cleaning <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2422>