| === chihchun_afk is now known as chihchun | ||
| zyga | Hello | 06:09 |
|---|---|---|
| mborzecki | morning | 06:10 |
| 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:13 |
| mborzecki | zyga: my wife drives them to school ~7am | 06:30 |
| zyga | mmmm | 06:42 |
| mborzecki | zyga: btw. desktop-portal-activation strikes again on fedora 29 | 06:54 |
| 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:16 |
| zyga | mborzecki: I saw it fail a bunch of times yesterday | 07:18 |
| zyga | any ideas? | 07:18 |
| 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:21 |
| 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:22 |
| zyga | yeah, I remember | 07:23 |
| mborzecki | zyga: have you repened snapd.mk pr? | 07:30 |
| zyga | mborzecki: no, I bailed from most work yesterday | 07:30 |
| 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:31 |
| 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:34 |
| mup | PR core18#94 opened: hooks: add cloud-init <Created by mvo5> <https://github.com/snapcore/core18/pull/94> | 07:58 |
| 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 | 08:02 |
| === pstolowski|afk is now known as pstolowski | ||
| pstolowski | morning | 08:07 |
| zyga | o/ | 08:08 |
| mborzecki | pstolowski: hey | 08:08 |
| ackk | hi, are there plans to SRU snapcraft 3 to bionic, or is snap the way to go? | 08:12 |
| pstolowski | ackk: sergiusens may be able to answer this, but he will probably show up in a few hours | 08:18 |
| ackk | pstolowski, thanks | 08:19 |
| zyga | mborzecki: much nicer now, I also added the with_... variables so the spec is shorter and nicer as well | 08:20 |
| zyga | mborzecki: pushed now | 08:23 |
| zyga | please look :) | 08:23 |
| 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:26 |
| zyga | sorry | 08:27 |
| 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:29 |
| mborzecki | zyga: https://www.youtube.com/watch?v=d2E_iiTd9hQ | 08:32 |
| zyga | mborzecki: evil in me wants to call spread with a fixed seed so that tests are less insanely annoying | 08:34 |
| zyga | mborzecki: I see and I raise https://www.youtube.com/watch?v=Pmd3UiNfNkA ;D | 08:38 |
| mborzecki | hahah | 08:38 |
| zyga | this doesn't look like a happy day | 08:53 |
| zyga | mborzecki: shall we disable that test until it is sorted out? | 08:53 |
| mborzecki | zyga: yeah, on fedora at least | 08:54 |
| zyga | mborzecki: can you please? | 09:06 |
| mborzecki | there's something fishy about the fedora image we use under spread | 09:08 |
| 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:14 |
| 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:15 |
| 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:31 |
| zyga | Looking | 09:32 |
| zyga | Haskell? | 09:32 |
| mborzecki | shellcheck probably or sth | 09:33 |
| mborzecki | gnome-online-accounts? | 09:33 |
| 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:34 |
| mborzecki | s/a test/test suite/ | 09:35 |
| 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:36 |
| mborzecki | but hey, xdg-desktop-portal has flatpak >= 0.11.1 in requires | 09:37 |
| mborzecki | so this feels like a bug in xdg-desktop-portal packaging | 09:38 |
| mborzecki | zyga: wow, it really does require flatpak to build even | 09:42 |
| zyga | For real? | 09:44 |
| zyga | It partially makes sense since it was split out only recently | 09:44 |
| mborzecki | yeah | 09:47 |
| mborzecki | zyga: maybe we should just disable weak deps on fedora | 09:50 |
| mborzecki | zyga: i'm writing an elaborate workaround, but the simplest solution to jsut disable it feels most appealing | 09:51 |
| zyga | I’m eager to see both | 09:55 |
| zyga | now | 09:55 |
| zyga | I’m afk for some time no | 09:55 |
| 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:56 |
| mborzecki | zyga: elaborate was (if fedora && installing snapd) -> disable weak deps | 09:57 |
| zyga | Brb | 10:24 |
| 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:31 |
| mup | PR snapd#6272 opened: overlord/snapstate: run 'remove' hook before 'auto-disconnect' <Created by stolowski> <https://github.com/snapcore/snapd/pull/6272> | 10:35 |
| 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:38 |
| mborzecki | zyga: damn, that upgrade to f29 was tricky, lots of little things falling apart | 10:39 |
| mborzecki | zyga: left some comments under snapd.mk pr | 10:40 |
| 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:53 |
| 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:55 |
| 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> | 10:57 |
| 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:03 |
| 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:05 |
| 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:06 |
| 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:07 |
| zyga | I’m afk again | 11:24 |
| * pstolowski lunch | 11:34 | |
| 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:36 |
| zyga | It should also be critical | 11:42 |
| zyga | Anything that fixes master to some degree | 11:42 |
| mborzecki | zyga: let's start with green :) | 11:43 |
| mborzecki | anyways, it's green | 11:44 |
| cachio | mborzecki, hey | 11:49 |
| cachio | we had some issues to update fedora 29 | 11:49 |
| cachio | mborzecki, have you seen this before https://paste.ubuntu.com/p/nMhRYJPwRc/ ? | 11:50 |
| 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:51 |
| 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:52 |
| 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:55 |
| 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:56 |
| mborzecki | cachio: fedora 29 as in dnf upgrade -y ? | 11:59 |
| zyga | Ack | 11:59 |
| cachio | mborzecki, yes | 12:01 |
| mborzecki | cachio: what's the problem then? i've run it today in spread shell, worked just fine | 12:01 |
| 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:02 |
| cachio | mborzecki, I have to go now, I'll continue with this when I am back | 12:07 |
| mborzecki | ok | 12:07 |
| 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:08 |
| cachio | mborzecki, that's what I saw | 12:09 |
| * cachio afk | 12:09 | |
| 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:23 |
| mborzecki | zyga: pstolowski: ^^ simple cherry pick | 12:24 |
| pstolowski | k | 12:27 |
| 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:33 |
| 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:34 |
| 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:35 |
| 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:36 |
| 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:37 |
| 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:38 |
| 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:39 |
| mborzecki | emacs as a snap? :) | 12:41 |
| Wimpress | zyga: Perhaps JamieBennett can? | 12:43 |
| 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:44 |
| diddledan | ok, thanks :-) | 12:45 |
| jdstrand | diddledan: were you able to test it with your application? | 12:46 |
| * 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:47 |
| 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:49 |
| diddledan | err. I mean makemkv with the shm | 12:50 |
| diddledan | mycroft is python multiprocessing, so that should work now, hopefully | 12:50 |
| 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:52 |
| mborzecki | off to pick up the kids | 12:58 |
| 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:03 |
| Son_Goku | at this point, there isn't that much else left to make that happen | 13:04 |
| zyga | Looking | 13:22 |
| zyga | Son_Goku: maaaaybe | 13:24 |
| 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:29 |
| zyga | I’ll skip standup today | 13:56 |
| pstolowski | zyga: ack | 14:01 |
| pstolowski | cachio: standup? | 14:01 |
| pstolowski | cachio: ah, you skipping as well, sorry, missed that | 14:02 |
| * diddledan sits down | 14:12 | |
| mup | PR snapcraft#2420 opened: static: update to the latest flake8 <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2420> | 15:19 |
| * cachio lunch | 15:30 | |
| mup | PR snapcraft#2421 opened: tests: remove obsolete snap and external tests <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2421> | 15:34 |
| === chihchun is now known as chihchun_afk | ||
| jdstrand | diddledan: fyi, https://forum.snapcraft.io/t/python-multiprocessing-sem-open-blocked-in-strict-mode/962/15 | 16:15 |
| diddledan | thanks jdstrand | 16:16 |
| jdstrand | now if github would actually show me the branch I pushed, I could submit the PR | 16:17 |
| jdstrand | (to snapd) | 16:17 |
| diddledan | :-) | 16:18 |
| jdstrand | is github behaving weird for anyone else? | 16:22 |
| 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:23 |
| jdstrand | huh, hmaybe it is firefox. chromium is acting differently | 16:24 |
| 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:28 |
| jdstrand | sergiusens: I did that. it seems that github added something that noscript was blocking. allowed that and it works again | 16:30 |
| 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> | 16:34 |
| === pstolowski is now known as pstolowski|afk | ||
| 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> | 17:44 |
| 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:31 |
| 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:53 |
| zyga | Pharaoh_Atem: hello | 18:59 |
| zyga | do you have a moment? | 18:59 |
| 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:02 |
| 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:03 |
| mup | PR snapcraft#2422 opened: clean: user friendly message when cleaning <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2422> | 19:47 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!