[02:28] <mup> PR snapd#5448 opened: tests: start using the new opensuse image with test dependencies <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/5448>
[05:13] <mborzecki> morning
[07:02] <pstolowski> mornings
[07:04] <mborzecki> pstolowski: hey
[07:05] <zyga> Hey ho
[07:05] <zyga> A bit late today,
[07:06] <zyga> mvo: I went on a small detour to simplify some code and add tests
[07:06] <zyga> I have some misc branches and then more meat
[07:07] <zyga> (Rice, soy, ...)
[07:07] <mvo> zyga: hey, thanks
[07:08] <zyga> I was working towards testing all of the patches properly
[07:08] <zyga> Out of the interface WIP one
[07:09] <mvo> zyga: neat
[07:10] <mborzecki> mvo: zyga: hey guys
[07:12] <mvo> hey mborzecki
[07:13] <mborzecki> mvo: i'm looking at sysctl manpage, have you checked that sysctl -p <file> with <file> as separate arg, not -p<file> works?
[07:14] <mvo> mborzecki: the test will check, I have not, let me try that now
[07:14] <mborzecki> i have a vague recollection of something being iffy there, but that's from a few years back and with busybox
[07:17] <mvo> mborzecki: I did a quick test and it seems to be fine
[07:17] <mborzecki> mvo: ok :)
[07:17] <mvo> mborzecki: but the test will also fail if it is not
[07:26] <Gargoyle> Can you check the confinement of a snap before you install it? I can't see anything in "snap info x" or another obvious snap command.
[07:26] <mborzecki> Gargoyle: try snap info --verbose
[07:27] <Gargoyle> Cool. Ta. :-)
[07:28] <mup> PR snapd#5447 closed: snap,interfaces: move interface name validation to snap <Core18> <Simple> <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5447>
[07:36] <mborzecki> mvo: could you take a look at #5434?
[07:36] <mup> PR #5434: overlord: introduce InstanceKey to SnapState and SnapSetup, renames <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5434>
[07:52] <mvo> mborzecki: sure
[07:52] <mborzecki> mvo: ta
[08:02] <zyga> mvo: thanks for that!
[08:06] <mup> PR snapd#5449 opened: snap: add helper for renaming slots <Created by zyga> <https://github.com/snapcore/snapd/pull/5449>
[08:07] <zyga> mvo: that's one of the helpers ^
[08:07] <zyga> pedronis: could you please look at https://github.com/snapcore/snapd/pull/5443 - it has two reviews already but I wanted to ensure you ack it
[08:07] <mup> PR #5443: interfaces: treat "snapd" snap as type:os <Core18> <Created by zyga> <https://github.com/snapcore/snapd/pull/5443>
[08:11] <zyga> mvo: quick review on https://github.com/snapcore/snapd/pull/5446
[08:11] <mup> PR #5446: coreconfig: add support for `snap set core network.disable-ipv6` <Created by mvo5> <https://github.com/snapcore/snapd/pull/5446>
[08:15] <mup> PR snapd#5445 closed: tests: add artful for sru validation on google backend <Created by sergiocazzolato> <Merged by zyga> <https://github.com/snapcore/snapd/pull/5445>
[08:16] <mvo> zyga: aha, nice. yeah, initially I wanted to do ipv4/ipv6 but then a) YAGNI b) ipv4 is not as straightforward
[08:19]  * Chipaca waves
[08:20] <mborzecki> Chipaca: hey, now that you're here, would you mind taking a look at #5426? :)
[08:20] <mup> PR #5426: client, cmd/snap: pass snap instance name when installing from file <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5426>
[08:20] <Chipaca> mborzecki: no
[08:21] <zyga> mvo: it's fine not to do ipv4
[08:21] <Gargoyle> Discord have both beta and "bleeding edge" versions available, is it possible to get these wired up into the snap?
[08:22] <Chipaca> Gargoyle: depending on who's doing the snapping, those can be tracks or just the pre-existing risks
[08:25] <Gargoyle> Not sure what that means! :/
[08:25] <zyga> Gargoyle: if the people making the snap want to, they can do that with snaps
[08:26] <zyga> Gargoyle: they can offer a beta channel/track
[08:26] <zyga> Gargoyle: and an "edge" channel/track
[08:26] <zyga> Gargoyle: alongside stable, that is
[08:26] <zyga> and people installing the snap can choose which one to get
[08:33] <Chipaca> Gargoyle: you can look at the 'go' snap for (a rather extreme) example
[08:34] <Chipaca> Gargoyle: you can get anything from 1.6 to devel, in a snap
[08:36] <zyga> Chipaca: and by look we mean "snap info go"
[08:36] <Chipaca> zyga: or gnome software, or https://snapcraft.io/go (and click on 'all versions')
[08:37] <Gargoyle> That's a lot of options! :D
[08:37] <zyga> Chipaca: I wonder if we will ever need to do paging on "snap info" ;-)
[08:38] <zyga> Chipaca: hum
[08:38] <zyga> Chipaca: why is 1.10 above 1.11?
[08:38] <pedronis> mborzecki: I did a pass over #5434
[08:38] <mup> PR #5434: overlord: introduce InstanceKey to SnapState and SnapSetup, renames <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5434>
[08:38] <zyga> Chipaca: do we need number sorting there?
[08:38] <mborzecki> pedronis: thanks!
[08:38] <zyga> (actually the ordering of tracks is a bit weird in general)
[08:38] <Chipaca> zyga: there is no order to tracks yet
[08:39] <pedronis> zyga: we don't know how to sort them,  last time we had a discussion with Chipaca and other store people, we need a sort order defined in the store
[08:39] <Chipaca> zyga: there probably will be at some point, but that point is still in the future :-) AFAIK at least
[08:39] <zyga> Chipaca: should we version sort them "just in case"
[08:39] <pedronis> the result we get from the store are sorted
[08:39] <zyga> aha
[08:39] <zyga> I see
[08:39] <Chipaca> well
[08:39] <pedronis> except they are lexically sorted atm
[08:39] <Chipaca> they are _ordered_, which is more important
[08:39] <zyga> re-sorted by us or by the store?
[08:39] <Chipaca> :)
[08:39] <pedronis> no, that's the point
[08:40] <Chipaca> if snapd is sorting them, it shouldn't be
[08:40] <zyga> ok
[08:40] <Chipaca> snapd gets them with a defined order and should leave it alone
[08:40] <pedronis> there's a store order (is just not a good one)
[08:40] <pedronis> but we should't try to fix that
[08:40] <pedronis> it's up to the store
[08:40] <Chipaca> (to answer whether snapd sorts them i'd need to look at the code)
[08:40] <zyga> agreed
[08:40] <Chipaca> OTOH, snapcraft.io and 'snap info' show a different order
[08:40] <Chipaca> so that's a separate issue i'm chasing down
[08:41] <Chipaca> (it just so happens to be a nicer order! grrr)
[08:41] <Chipaca> (for go, on snapcraft.io)
[08:42] <mborzecki> pedronis: yeah, the non-test code setting InstanceKey in either snapsup or snapst lives here atm https://github.com/bboozzoo/snapd/commits/bboozzoo/parallel-install-overlord
[08:50] <Chipaca> pedronis: ETOOMANYCHANNELS
[08:50] <Chipaca> er
[08:50] <Chipaca> mborzecki: ^
[08:50] <Chipaca> :)
[08:51] <mborzecki> heh
[08:54] <zyga> pretty please https://github.com/snapcore/snapd/pull/5449 :)
[08:54] <mup> PR #5449: snap: add helper for renaming slots <Created by zyga> <https://github.com/snapcore/snapd/pull/5449>
[08:54] <zyga> it's short and test code
[08:59] <zyga> mvo: thank you for the update, two more comments on 5446
[08:59]  * zyga coffee
[09:11] <mup> PR snapd#5450 opened: store: keep all files with link-count > 1 in the cache <Created by mvo5> <https://github.com/snapcore/snapd/pull/5450>
[09:18] <mborzecki> pedronis: pushed a fix to #5434, i've tweaked the error message a little, added some tests
[09:18] <mup> PR #5434: overlord: introduce InstanceKey to SnapState and SnapSetup, renames <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5434>
[09:27] <zyga> mvo: one last comment on 5446,
[09:29] <pstolowski> dog emergency, need to go to vet
[09:30] <mvo> zyga: thanks for your comment. this is a bit tricky, if things get removed sysctl -p won't work. on removal the system default is used. we can make this explicit but it has the side-effect that there will always be a sysctl file created even with an empty config
[09:32] <mup> PR snapd#5451 opened: interfaces: honor static attributes when reloading conns <Created by stolowski> <https://github.com/snapcore/snapd/pull/5451>
[09:37] <zyga> pstolowski|vet: interesting, reading
[09:40] <mvo> zyga: I commented, I can rework the code accordingly
[09:40] <mvo> zyga: I think it makes sense, will do so in a wee bit
[09:40] <zyga> thank you, looking
[09:43] <zyga> mvo: I commented as well, let me know where you want to take this
[09:45] <zyga> mborzecki: replied on your comment on RenameSlot
[09:47] <ogra_> mvo, not sure you have noticed, ppc64el and s390x daily core snap builds failed (looks like a gpg error)
[09:51] <mvo> zyga: thanks, I updated the code now, I think we are fine with EnsureDirState and a bit of extra smarts, I pushed a new revision, please let me know if I overlooked anything but I think we should be fine now
[09:52] <mvo> ogra_: you mean the builds in https://launchpad.net/~snappy-dev/+archive/ubuntu/edge/+packages?field.name_filter=snapd&field.status_filter=&field.series_filter= ?
[09:53] <ogra_> mvo, https://launchpad.net/~snappy-dev/+snap/core/+build/267041 and https://launchpad.net/~snappy-dev/+snap/core/+build/267042
[09:53] <mvo> ogra_: they are flaky on these arches but not consistently  failing? or am I looking at the wrong place maybe?
[09:53] <mvo> ogra_: aha, the core snap, let me check
[09:53] <ogra_> it is the first build error i have seen in a while
[09:53]  * mvo looks
[09:53] <ogra_> noit flaky usually ...
[09:53] <cjwatson> ogra_,mvo: Those are transient failures due to known keyserver network chaos
[09:53] <ogra_> *not
[09:53] <cjwatson> just retry
[09:54] <ogra_> ah, great
[09:54] <mvo> thanks cjwatson
[09:54] <ogra_> yeah, i noticed a ton of livefs mails too
[09:54] <cjwatson> AIUI somebody discovered that you could upload a gigabyte key to the keyserver network and have it kill stuff
[09:54] <cjwatson> or something along those lines
[09:54] <ogra_> lol
[09:54] <mvo> *cough*
[09:54] <cjwatson> the livefs stuff is different and I'm investigating
[09:54] <ogra_> ah, k
[09:56] <mup> PR snapd#5428 closed: devicestate: fix panic in firstboot code when no snaps are seeded <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5428>
[10:02] <zyga> mvo: looking
[10:06] <zyga> mvo: it is good
[10:06] <zyga> mvo: I suggested one more tweak to make it more clear why this happens
[10:06] <zyga> but logic wise it is correct
[10:06] <zyga> smart way of doing that btw :)
[10:08] <zyga> cjwatson: so gpg network is the new bittorrent?
[10:08] <cjwatson> hopefully not ...
[10:19] <mvo> zyga: sure thing, thank you
[10:20] <popey> niemeyer: any chance we can bump the largest post size from 32000 to ~40000 on discourse. I'm trying to migrate a large doc to the site, but it's ~35K
[10:22] <niemeyer> Heya
[10:22] <niemeyer> Which doc is that?
[10:23] <popey> https://github.com/snapcore/snapd/wiki/REST-API
[10:23] <popey> it's the last one still on the wiki
[10:24] <popey> well, last one linked on the front page of the wiki
[10:24] <niemeyer> popey: I think we should break it down a bit on the way in, even so we can read it reasonably
[10:25] <niemeyer> popey: IIRC I provided an outline of how to break it down in the thread where we discuss the doc site
[10:25] <popey> that seems reasonable.
[10:26] <niemeyer> popey: We don't need to break it down every single section, but rather by area of interest or something similar
[10:26] <popey> Robert seems to "own" that doc. I'd rather he broke it up and kept owning it
[10:26] <niemeyer> popey: Thanks for the work on that, btw
[10:27] <popey> I'll drop Robert a mail as he's in another TZ
[10:27] <popey> thanks
[10:31] <niemeyer> popey: np, and if you want I'd be happy to work with you on that migration.. it's been pending for too long
[10:50] <zyga> mborzecki, mvo: can you please look at https://github.com/snapcore/snapd/pull/5449
[10:50] <mup> PR #5449: snap: add helper for renaming slots <Created by zyga> <https://github.com/snapcore/snapd/pull/5449>
[10:50] <zyga> I have a pair of follow ups on top
[10:52] <zyga> hmm
[10:52] <zyga> journalctl fails to restart https://www.irccloud.com/pastebin/w7dISItk/
[10:52] <zyga> have you guys seen that before ^
[10:53] <zyga> journalctl failing to _restart_ ?
[10:57]  * zyga -> walk
[10:57] <mborzecki> zyga: yes, seen it before
[11:20] <niemeyer> mvo: Heya
[11:20] <niemeyer> mvo: One question on spread#61
[11:20] <mup> PR spread#61: client: fix dialOnReboot() if the remote does not reply <Created by mvo5> <https://github.com/snapcore/spread/pull/61>
[11:22] <mborzecki> parallel installs will be fun to merge/resolve conflicts :/
[11:46] <mup> PR snapd#5452 opened: store, overlord/snapstate: introduce instance name in store APIs <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5452>
[11:46] <mborzecki> pedronis: Chipaca: probably something for you guys ^^
[11:47] <mborzecki> it's very likely this will conflict with changes i wanted to open for review after landing #5434
[11:47] <mup> PR #5434: overlord: introduce InstanceKey to SnapState and SnapSetup, renames <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5434>
[11:50] <mborzecki> pedronis: you've seen part of the store changes 2-3 weeks ago, most of it is the same patches just updated to account for downloads + some extra adjustments
[11:50] <zyga> re
[11:51] <zyga> MATCH mystery again https://www.irccloud.com/pastebin/p8HCgyXK/
[11:53] <pedronis> mborzecki: well, I should probably land my other PR and then it will likely conflict
[11:54] <mborzecki> pedronis: no worries, i'll keep resolving the conflicts as they come
[12:24] <zyga> mvo: can you please review https://github.com/snapcore/snapd/pull/5449 quickly
[12:24] <mup> PR #5449: snap: add helper for renaming slots <Created by zyga> <https://github.com/snapcore/snapd/pull/5449>
[12:26] <mborzecki> Chipaca: did you manage to come up with anything regading 'the mystery of MATCH' yesterday?
[12:26] <Chipaca> mborzecki: no, i didn't carry on digging
[12:27] <Chipaca> falling too far behind with warnings
[12:27] <mborzecki> ack
[12:30] <zyga> thank you mborzecki  :)
[12:30]  * zyga looks at the diff from the upcoming branch and is very happy
[12:33] <mup> PR snapcraft#2175 opened: ci: disable osx tests until a new pyyaml is released <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2175>
[12:34] <mvo> zyga: sure, I have a look
[12:35] <mvo> zyga: match mystery> I think your idea to hexdump the output we got is interessting, maybe there is some whitespace mangling going on
[12:36] <mborzecki> mvo: travis hides certain characters in the output log, though i'm not sure if the same applies to the raw log
[12:41] <Chipaca> mborzecki: mvo: I did 'od -a' the file fwiw
[12:41] <Chipaca> and it was fine
[12:41] <Chipaca> (i thought i'd told you this yesterday0
[12:41] <Chipaca> )
[12:43] <Chipaca> why haven't i had lunch yet?
[12:44] <Chipaca> also, where has my morning gone
[12:44]  * Chipaca scrambles to get lunch
[12:45] <zyga> mvo: I honestly don't believe my theory anymore
[12:45] <zyga> mvo: I bet I will be surprised by the _actual_ issue
[12:57] <mup> PR snapd#5449 closed: snap: add helper for renaming slots <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5449>
[13:01] <zyga> joining
[13:03] <mup> PR snapd#5453 opened: interfaces: tweak tests to have less repetition of "core" and "ubuntu… <Created by zyga> <https://github.com/snapcore/snapd/pull/5453>
[13:06] <zyga> mvo: ^ the small refactor for the disconnect/connect resolve with snapd awareness
[13:06] <zyga> the patch on top of this one is the main part of the WIP branch that was still pending
[13:06] <zyga> so very close with getting all of that out
[13:06] <zyga> I will prepare a branch that is derived from your integration branch where I have the new patch set (from master) and see how things are going
[13:07] <zyga> I didn't manage that last night, sorry
[13:12] <Chipaca> cachio: the Software crash, is that in 18.04?
[13:12] <mvo> Chipaca: artful iirc
[13:25] <mborzecki> zyga: hmm https://paste.ubuntu.com/p/CNqt2p6gj8/
[13:26] <zyga> oh
[13:26] <zyga> weird
[13:26] <zyga> ah, sorry
[13:26] <zyga> I didn't read this righ
[13:26] <zyga> *right
[13:26] <zyga> this is a bug
[13:27] <zyga> caused by >1 instance of "change to this directory and then back"
[13:27] <zyga> without handling the error by going to the void
[13:27] <mborzecki> heh lucky me
[13:27] <zyga> is this new?
[13:27] <Chipaca> zyga: why is this a bug?
[13:28] <Chipaca> it's in /tmp
[13:28] <mborzecki> zyga: dk, tried to figure out why shellcheck in tests/unit/go does not work
[13:28] <zyga> Chipaca: because the correct behavior was (and it worked before) to go to /var/lib/snapd/void
[13:28] <Chipaca> zyga: ahhh
[13:28] <Chipaca> zyga: maybe /var/lib/snapd/void doesn't exist?
[13:28] <zyga> Chipaca: yes but the /tmp inside the snap is different and cannot contain what is on the host (it's a private tmp)
[13:28] <zyga> Chipaca: oh, that's interesting
[13:28] <zyga> does it mborzecki /
[13:29] <mborzecki> zyga: it does drwxr-xr-x 2 root root  4096 Jul  3 12:36 void
[13:29] <mborzecki> try it locally
[13:29] <zyga> the permissions are wrong
[13:29] <zyga> anyway, that's something to add to the queue
[13:29] <zyga> probably a fallout of something that happened recently
[13:30] <mborzecki> running it outside of tmp at least once makes the problem go away
[13:30] <zyga> this implies the problem is in the first part of the setup code
[13:32] <zyga> mvo: https://github.com/snapcore/snapd/pull/5446#issuecomment-402155147
[13:32] <mup> PR #5446: coreconfig: add support for `snap set core network.disable-ipv6` <Created by mvo5> <https://github.com/snapcore/snapd/pull/5446>
[13:34] <mvo> zyga: yeah, just noticed and looking
[13:36] <mvo> zyga: fixed, it was just that the output of sysctl has some extra whitespace
[13:37] <mvo> zyga: did you tag all PRs that help with the iface renames as core18?
[13:37] <mvo> zyga: if so, I will merge and run them now against core18
[13:38] <zyga> no, not yet, hold on
[13:39] <zyga> I didn't open all of them due to stacking
[13:40] <mborzecki> zyga: https://paste.fedoraproject.org/paste/4hQSsupdzQNcvvbwpZlhUA makes it go away
[13:40] <cachio> Chipaca, it is on 17.10
[13:40] <Chipaca> just finished installing it
[13:40] <Chipaca> (because why not)
[13:42] <mborzecki> zyga: or https://paste.ubuntu.com/p/yWyRj3Pqqj/ no clue why fpaste is so slow today
[13:42] <Chipaca> cachio: and you start 'ubuntu software', and search for 'snapd', and the UI crashes?
[13:43] <mborzecki> zyga: why does it even bother chdir()ing in setup_private_mount() while it done so in sc_populate_mount_ns() already
[13:46] <mvo> zyga: no worries, I think I found the relevant ones
[13:46] <zyga> I haven't opened all of them
[13:46] <zyga> I actually wanted to ask you
[13:46] <zyga> well, tell you
[13:46] <mvo> zyga: aha ok
[13:47] <zyga> I'll create a branch from your working one but replace the older versions of the patches as I said in the standup
[13:47] <zyga> it's fine to test what you made, let's where it gets
[13:47] <zyga> but I want to have a clean picture of the patches that are not in master
[13:47] <zyga> (a list that shrinks rather than grows0
[13:50] <mvo> zyga: ok, it might be easier to use core18-all-fixes-all-tests-2
[13:50] <mvo> zyga: which is a clean master with just all the core18 PRs
[13:50] <zyga> sure, I'll use that
[13:50] <pstolowski> cachio: can #5391 land?
[13:50] <zyga> nice
[13:50] <mvo> zyga: and it also enables all core18 tests
[13:50] <mup> PR #5391: tests: simplify econnreset test <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/5391>
[13:53] <cachio> pstolowski, yes
[13:53] <pstolowski> thanks
[13:54] <mup> PR snapd#5391 closed: tests: simplify econnreset test <Created by sergiocazzolato> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/5391>
[13:57] <zyga> pedronis: thank you for the comments! I'm on that now
[13:58] <zyga> mborzecki: we need to rework more code I'm afraid, to get the right "way" of handling that error
[14:00] <mborzecki> zyga: i can open a PR with http://paste.ubuntu.com/p/XvBhhFSMk8/ which resolves the issue, the question is why it tries to chdir into the old cwd while doing that, since it's already being restored at the end
[14:01] <mvo> pedronis: zyga: core18 meeting
[14:01] <zyga> mborzecki: later
[14:01] <zyga> ack
[14:07] <mup> PR snapd#5369 closed: overlord,interfaces,cmd: WIP early support for interfaces on core18 <Core18> <Created by zyga> <Closed by zyga> <https://github.com/snapcore/snapd/pull/5369>
[14:26] <mup> PR snapd#5453 closed: interfaces: tweak tests to have less repetition of "core" and "ubuntu… <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/5453>
[14:26] <zyga> mvo: thank you for the review, one more for implicit operations this time ^
[14:26] <zyga> well v
[14:27] <zyga> 5454
[14:27] <mup> PR snapd#5454 opened: interfaces: prefer "snapd" when resolving implicit connections <Core18> <Created by zyga> <https://github.com/snapcore/snapd/pull/5454>
[14:30] <Chipaca> cachio: how do I reproduce the crash?
[14:31] <cachio> just open ubuntu software
[14:31] <Chipaca> cachio: done
[14:31] <cachio> and search "snapd"
[14:31] <Chipaca> cachio: done
[14:31] <Chipaca> cachio: … and the crash?
[14:32] <mvo> zyga: thanks, looking
[14:33] <cachio> Chipaca, did it return results?
[14:33] <cachio> for me, it closed the app
[14:33] <mvo> zyga: also, please share the run results if you run the full tests with your PRs against core18, maybe there is some stuff that I need to tweak
[14:33] <Chipaca> cachio: yes
[14:33] <Chipaca> cachio: with what version of snapd?
[14:33] <cachio> try again
[14:33] <cachio> I try with the 2.38.*
[14:33] <cachio> and with 2.33.1
[14:33] <cachio> 2.28 sorry
[14:34] <Chipaca> cachio: this is snapd from the package, yes?
[14:34] <cachio> yes
[14:34] <Chipaca> cachio: I get 2.32.9+17.10 from the archive
[14:34] <Chipaca> cachio: where do i get 2.28 from?
[14:34] <zyga> mvo: yes, I will share the results, I need to spend some time to prepare the branch
[14:35] <cachio> 1 sec
[14:35] <Chipaca> cachio: ah, i can get 2.28.5 from artful/main (as opposed to artful-updates/main)
[14:35] <cachio> I have 2.28.5+17.10
[14:36] <Chipaca> cachio: yes, taht's the one in artful/main (you've not updated)
[14:36] <ogra_> mvo, zyga i recently see this (not sure if it was there before) when starting a core 16 edge image on my pi's https://paste.ubuntu.com/p/gGKBBSG4nB/ ... the system seems to behave normally but it prints in red during boot so it is pretty noticeable
[14:36] <Chipaca> cachio: apt update && apt upgrade should get you 2.32.9
[14:37] <Chipaca> cachio: in any case, it worked ok with both
[14:37] <Chipaca> cachio: also with core from edge
[14:37] <Chipaca> so I don't know what I'm looking for
[14:38] <cachio> Chipaca, it is failing here
[14:38] <cachio> perheps it is a different version of ubuntu software
[14:39] <cachio> did you search for something else and then "snapd"
[14:39] <cachio> ?
[14:39] <cachio> Chipaca, I just got distro_install_package google-compute-engine
[14:39] <cachio> sorry, seg fault
[14:40] <zyga> ogra_: that's very interesting
[14:40] <zyga> can you provide journal data from that service?
[14:40] <cachio> Chipaca, [  173.948719] pool[1815]: segfault at 0 ip 00007fdea83ec1f0 sp 00007fde6effc7e8 error 4 in libglib-2.0.so.0.5400.1[7fdea83b2000+111000]
[14:41] <cachio> Chipaca, gnome-software 3.26.1
[14:41] <zyga> ogra_: thank you for reporting this
[14:42] <mup> PR snapd#5455 opened: many: assorted shellcheck fixes <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5455>
[14:42] <Chipaca> cachio: how much memory in your vm?
[14:43] <cachio> 2GB
[14:43] <Chipaca> cachio: can you try with 4?
[14:43] <cachio> sure
[14:43] <Chipaca> cachio: gnome-software is that same version here
[14:45] <Chipaca> cachio: it doesn't crash here with 2 either though
[14:46] <cachio> Chipaca, weird, I'll update snapd and retry
[14:48] <cachio> Chipaca, perhaps we are using different artfull images
[14:50] <Chipaca> cachio: I'm just running it with kvm after installing it from the artful desktop iso, so yeah
[14:50] <Chipaca> cachio: how are you running yours?
[14:51] <cachio> same, but using virtual machine manager
[14:53] <zyga> "options matteR"
[14:53] <Chipaca> zyga: -march=broken
[14:54] <cachio> Chipaca, after upgrade all the packages the ubuntu software does not crash anymore
[14:54] <Chipaca> cachio: i'm not sure if that's a win or a lose
[14:55] <cachio> did you upgrade before trying to reproduce it?
[14:56] <zyga> Chipaca: virtio/accel all kinds of visual magic nobody knows how to run by hand
[14:56] <mup> PR snapd#5411 closed: many: remove core-support interface <Core18> <Created by zyga> <Closed by zyga> <https://github.com/snapcore/snapd/pull/5411>
[14:57] <Chipaca> cachio: I think I checked the "download ugprades while installing" checkbox
[14:57] <Chipaca> cachio: should I try again without that?
[14:57] <Chipaca> (i then also upgraded, and then tried with snapd from edge)
[14:58] <Chipaca> (i didn't understand exactly what you're trying to do so i covered most things)
[14:58] <Chipaca> i can easily re-install it without the upgrades checkbox
[14:58] <cachio> Chipaca, I don't know which is the error, if we need to fix something og it is already fixed :(
[14:59] <Chipaca> cachio: when you installed, did you check the 'download upgrades' button?
[14:59] <cachio> no
[14:59] <Chipaca> i'll try without that
[14:59] <cachio> basically, I installed as clean as possible
[15:00] <cachio> then I reproduced the error
[15:00] <Chipaca> cachio: and what is the error blocking us from doing?
[15:00] <cachio> Chipaca, well, for sru we have a test which searches "snapd"
[15:01] <cachio> and then install test-snapd-tools
[15:01] <cachio> and it was failing
[15:01] <Chipaca> cachio: but for the sru test you've upgraded snapd to the sru version, right?
[15:01] <cachio> yes
[15:01] <cachio> it is also happening for 2.33.1
[15:01] <cachio> I just upgraded snapd
[15:02] <cachio> not all the packages
[15:05] <Chipaca> cachio: so it's not happening for you with 2.28.5?
[15:06] <ogra_> pedronis, hmm, should the connections: keyword in gadget.yaml work with the snapd in beta ? ubuntu-image gets me:
[15:06] <ogra_> gadget.yaml parse error: Invalid gadget.yaml @ connections
[15:06] <cachio> Chipaca, yes
[15:06] <Chipaca> cachio: yes it isn't?
[15:06] <cachio> it happes
[15:06] <cachio> happens
[15:06] <Chipaca> …
[15:06] <Chipaca> cachio: so again, i ask, what are we testing
[15:06] <cachio> with 28.5 and 33.1
[15:07] <cachio> Chipaca, we test the integration between ubuntu software and snapd
[15:07] <Chipaca> cachio: but you're not testing the SRU package
[15:07] <cachio> yes
[15:07] <cachio> Chipaca, let me explain
[15:07] <Chipaca> please because I'm losing it
[15:07] <pedronis> ogra_: yes,   should work, are you sure ubuntu-image is using the right snapd?
[15:07] <Chipaca> (fresh reinstall, this time no updates selected, no crashes here)
[15:08] <cachio> what I did is: I installed the snapd from proposed in a vm
[15:08] <cachio> I searched snapd in ubuntu software and I saw the crash
[15:08] <pedronis> ogra_: or   ubuntu-image does its own strict parsing? in which case it needs fixing
[15:08] <ogra_> pedronis, no idea, which one should it use ? the image itself is built with --channel edge, the machine executing ubuntu-image runs 16-2.34~pre1 from beta
[15:08] <cachio> then in another vm which was totally clean
[15:09] <ogra_> ah, i didnt think u-image does parsing at all and relies on snap prepare-image
[15:09] <cachio> Chipaca, I did the same with without the sru package to check if it was a regression
[15:09] <pedronis> ogra_: well it needs to parse volumes
[15:09] <ogra_> oh, indeed
[15:09] <cachio> Chipaca, and I reproduced the error with 28.5
[15:09] <pedronis> but indeed I didn't think it would do strict parsing
[15:09] <ogra_> yeah, that m,ight be it then
[15:09] <cachio> Chipaca, so, it wasn't a regressions
[15:09] <cachio> Chipaca, that's it
[15:09] <Chipaca> cachio: ok
[15:09] <pedronis> ogra_: in which case, can you open a u-i bug?
[15:10] <Chipaca> cachio: so, I have again not been able to reproduce the issue
[15:10] <Chipaca> just in case, I'll grab the snapd from proposed
[15:10] <ogra_> pedronis, yeah, i'll look into it and file a bug
[15:10] <cachio> Chipaca, it is ok, it is the same I did
[15:11] <cachio> Chipaca, I suppose there is something else causing this
[15:11] <cachio> another package
[15:11] <cachio> and perhaps you are using an new iso
[15:12] <cachio> and I am using an old one
[15:12] <Chipaca> cachio: 17.10 doesn't have more than one iso
[15:12] <Chipaca> afaik
[15:13] <Chipaca> cachio: just tried with snapd from proposed, also no crash
[15:14] <cachio> Chipaca, in the clean image, with no upgrades?
[15:15] <Chipaca> cachio: yes
[15:15] <Chipaca> cachio: can you strace the process?
[15:16] <Chipaca> cachio: that is: start it, then check its pid and use strace -f -p <that pid>
[15:16] <Chipaca> (just stracing ubuntu-software doesn't work)
[15:23] <cachio> Chipaca, sure
[15:31] <mup> PR core#92 opened: Remove core-support plug <Created by zyga> <https://github.com/snapcore/core/pull/92>
[15:32] <zyga> mvo: I created https://github.com/snapcore/core/pull/92 and I'm testing a corresponding change to snapd master
[15:32] <mup> PR core#92: Remove core-support plug <Created by zyga> <https://github.com/snapcore/core/pull/92>
[15:32] <zyga> I will just drop the plug from the core snap
[15:32] <zyga> CC jdstrand, pedronis  ^
[15:34] <cachio> Chipaca, https://paste.ubuntu.com/p/HzzcTp3cWz/
[15:34] <cachio> 404
[15:35] <Chipaca> [pid  1931] sendto(20, "GET /v2/snaps/(null) HTTP/1.1\r\nH"..., 177, MSG_NOSIGNAL, NULL, 0) = 177
[15:35] <Chipaca> that looks a lot like a bug in gnome software or libsnapd-glib1
[15:35] <cachio> yes
[15:35] <cachio> because if I search another string it works
[15:36] <Chipaca> cachio: but if you upgrade everything (not to proposed, just regular artful-updates) the bug goes away?
[15:36] <cachio> Chipaca, yes
[15:37] <cachio> Chipaca, I just realized that when you could not reproduce the error
[15:37] <Chipaca> cachio: so, I'd drop robert ancell an email about this, but it's probably known and fixed already
[15:38] <Chipaca> cachio: I suggest we update the SRU testing to say we upgrade to whatever's in -updates before trying snapd from -proposed
[15:38] <Chipaca> but I don't know if that's kosher
[15:38] <Chipaca> mvo: would that be ok?
[15:40] <mvo> Chipaca: yes, absolutely fine
[15:40] <mvo> Chipaca: and sensible
[15:41] <mvo> zyga: I'm dying of curiosity, how is the core18 main test run going?
[15:41] <zyga> mvo: I haven't started everything-at-once yet, I'm still running partial checks for the things I'm changing
[15:41] <Chipaca> zyga: running like the little lambs in https://www.youtube.com/watch?v=WQO-aOdJLiw
[15:41] <zyga> mvo: ~3 hours from now realistically (I want to break now to take a walk before it's dark)
[15:42] <mvo> zyga: ok, thanks for the update .)
[15:42]  * mvo hugs zyga
[15:42] <mvo> zyga: sorry for being so impatient
[15:42] <zyga> no worries, it's good to check :)
[15:49] <zyga> mvo: I'm running a full main run with snapd modified to kill core support
[15:49] <zyga> clarification: with snapd modified to remove the core-support-plug from the core snap
[15:50] <mup> PR snapd#5456 opened: snapstate: refuse to remove bases or core if snaps need them <Created by mvo5> <https://github.com/snapcore/snapd/pull/5456>
[15:51]  * cachio lunch
[15:55] <zyga> mvo: https://github.com/snapcore/snapd/pull/5456/files#r199860433
[15:55] <mup> PR #5456: snapstate: refuse to remove bases or core if snaps need them <Created by mvo5> <https://github.com/snapcore/snapd/pull/5456>
[15:55] <zyga> mvo: mainly to put the logic about "I need this" in one clear spot
[15:55] <zyga> and not all over the place
[15:55] <zyga> the first one is giving you simple hints (all the snaps with those names are related to myself)
[15:55] <zyga> this would return the base snap
[15:56] <zyga> but also, perhaps, default content provider or even the one that is connected right now
[15:56] <zyga> the second would look at a real snap info and say if this is ok to remove
[15:56] <zyga> I suspect they may need snap state at that time
[15:56] <zyga> (I didn't include that in my comment on the PR)
[15:56] <zyga> so that we can check connections and what not
[15:57] <zyga> just a though, I'll go for a walk now because it will be getting dark soon and woods are not like a park
[16:30] <zyga> mvo: still testing, got one error that spewed megabytes of journal that didn't say much, re-testing that one to see if it is something real
[16:30] <zyga> mvo: please review https://github.com/snapcore/core/pull/92 if you can
[16:30] <mup> PR core#92: Remove core-support plug <Created by zyga> <https://github.com/snapcore/core/pull/92>
[16:31] <zyga> mvo: I will really really go in ~30 minutes because $life tasks
[16:33] <mvo> zyga: thanks for the upate
[16:35] <zyga> mvo: I added a patch that does the removal snapd-side, if it works I will open another PR
[16:37] <zyga> mvo: can we land https://github.com/snapcore/snapd/pull/5443
[16:37] <zyga> I think we can now
[16:37] <mup> PR #5443: interfaces: treat "snapd" snap as type:os <Core18> <Created by zyga> <https://github.com/snapcore/snapd/pull/5443>
[16:37] <zyga> mvo: and we need a review on https://github.com/snapcore/snapd/pull/5454
[16:37] <mup> PR #5454: interfaces: prefer "snapd" when resolving implicit connections <Core18> <Created by zyga> <https://github.com/snapcore/snapd/pull/5454>
[16:38] <mvo> zyga: ok
[16:39] <Son_Goku> :/
[16:40] <Son_Goku> this bug is really bothering me
[16:40] <Son_Goku> I really don't know what causes this
[16:40] <Son_Goku> and the worst part is, it's pretty easily reproducible
[16:41] <zyga> Son_Goku: I have no idea, I have a F28 system to play with but I'm deep in core18 work that actually  a prerequisite for fedora snap that works without pulling in ubuntu
[16:41] <Son_Goku> yeah
[16:41] <zyga> Son_Goku: the work is also time boxed as we are running towards a commercial deadline
[16:42] <Son_Goku> oh?
[16:42] <zyga> Son_Goku: I will devote way more time to fedora after this
[16:42] <zyga> Son_Goku: just making core18 available
[16:42] <mvo> Son_Goku: how is it reproducible?
[16:42] <zyga> for customers that want that
[16:43] <zyga> Son_Goku: is it _any_ snap that triggers this?
[16:44] <Son_Goku> zyga, the launching of a snap seems to do it
[16:45] <Son_Goku> I triggered it with ohmygiraffe
[16:46] <zyga> Son_Goku: thanks
[16:46] <zyga> well
[16:46] <zyga> can you check if hello-world does that too?
[16:46] <zyga> it's shorter to strace to eventually find out what the problem is
[16:47] <Son_Goku> I'll check later in the day
[16:47] <Son_Goku> I'm unfortunately busy at the moment
[16:48] <zyga> no worries, thank you for the information
[16:54] <cachio> zyga, the documentation for spread images is described on this PR
[16:54] <cachio> https://github.com/snapcore/spread-images/pull/13
[16:54] <mup> PR spread-images#13: Adding base images and test dependencies including new documentation <Created by sergiocazzolato> <https://github.com/snapcore/spread-images/pull/13>
[16:56]  * zyga hugs cachio :-)
[16:56] <cachio> zyga, it is first version
[16:57] <cachio> zyga, open to rewrite it if needed :)
[16:57] <zyga> I will read it
[16:58] <cachio> thanks
[17:00] <ahoneybun> I'm seeing an issue with snapd.service that it has issues starting. So it is holding the boot process.
[17:00] <ahoneybun> I can provide log files.
[17:00] <zyga> ahoneybun: please report all you know in a forum thread on forum.snapcraft.io
[17:01] <zyga> and coordinate here for more feedback as we review
[17:03] <zyga> mvo: I tested master with this extra patch https://github.com/zyga/snapd/commit/366ca43e3149db942207e6e81a0b17fe61f80f0c
[17:03] <zyga> mvo: and this test change https://github.com/snapcore/snapd/compare/master...zyga:feature/lessen-use-of-core-support?expand=1
[17:03] <zyga> I will propose the test change in a moment,
[17:04] <zyga> this will let us land the core change with confidence (dropping core-support-plug from core{,16}
[17:04] <zyga> mvo: I haven't done that yet but we can also drop it from core18 if it is there, I haven't checked yet
[17:05] <zyga> I'm running one more test on this upcoming branch in isolation (without the suppor patch that removes core-support-plug programmatically)
[17:10] <zyga> jdstrand: briefly, my mind thought about having compiler.apparmor.net that exposes ultra-fast, cached apparmor_parser
[17:10] <zyga> especially since it would have a small (<<~1M) set of profiles
[17:10] <zyga> that could be cached efficiently
[17:11] <zyga> and it would probably scream from RAM on a modest server
[17:11] <zyga> it could reply with some status code if overloaded (fallback to local)
[17:11] <zyga> but on slow machines it would meaningfully speed things up
[17:49] <mup> PR snapcraft#2175 closed: ci: disable osx tests until a new pyyaml is released <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2175>
[18:50] <mvo> zyga: core18 does not have this plug
[18:54] <zyga> mvo: that's great
[18:54] <zyga> tests passed, opening PR
[18:56] <zyga> mvo: one more, I hope it passes
[18:57] <mup> PR snapd#5457 opened: many: lessen the use of core-support <Core18> <Created by zyga> <https://github.com/snapcore/snapd/pull/5457>
[19:04]  * cachio afk
[20:58] <twobitsprite> I'm trying to boot up the KVM/Qemu image and it stalls booting right after "Started network service" and ssh is just reseting connection... it's been sitting there like that for about 10 minutes...
[21:01] <twobitsprite> Err... sorry, I thought this channel was dedicated to Ubuntu Snappy Core, not the whole snappy project :P
[21:02] <twobitsprite> Is there a better place to ask about Ubuntu Snappy?
[21:16] <mup> PR snapcraft#2174 closed: build_providers: inject snaps when running from a snap <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2174>
[22:03] <mup> PR snapd#5458 opened: tests: check catalog refresh before and after restart snapd <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/5458>