/srv/irclogs.ubuntu.com/2018/11/06/#snappy.txt

=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
zygao/06:07
zygagood morning06:08
mvohey zyga06:08
zygahey :)06:08
mborzeckizyga: mvo: morning guys06:09
zygagood morning06:09
mvohey mborzecki06:09
zygamborzecki: please wait on the opens use review, I found some things to change last night that I didn't address yet06:10
mborzeckizyga: ok06:10
mvozyga: your opensuse build update shows "+ echo 'Package build incorrect, '\''snap --version'\'' mentions '\''unknown'\'''"06:16
mvozyga: in the spread tests06:16
zygayeah, I saw that06:16
zygathat's what I meant :)06:17
mvook06:17
mvozyga: oh, sorry, missed that I guess06:17
zygait works fine in casual testing but I wanted to see spread run before releasing this06:17
zygano worries, thank you for looking06:17
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
mborzeckihttps://github.com/snapcore/snapd/pull/6094 could use another review07:35
mupPR #6094: wrappers: fix generating of service units with multiple `before` dependencies <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6094>07:36
mborzeckithen we can land it and pick it for 2.3607:36
zygayep07:49
zygamborzecki: interesting, in my local build I see the correct version on "snap version"07:51
pstolowskigood morning!08:07
=== chihchun is now known as chihchun_afk
zyga--debug session shows unknown08:09
zygaso odd, let's see why08:09
zygaah08:10
zygabecause of how we build in spread08:10
zyganormal builds are fine08:11
mborzeckihm i'm debugging why #5896 spread test for device keys seeminglly doesn't work08:14
mupPR #5896: snapcraft.yaml: set grade to stable <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5896>08:14
mborzeckithe test snap finds a device /dev/input/by-path/platform-i8042-serio-0-event-kbd which as it turns out has all the attributes that make it match with device-buttons interface :/08:15
mborzeckihm no it doesn't match, but the snap can still access it08:23
mborzeckizyga: any clues, there's a device /dev/input/event2 maj:min is 13:66, it's not listed in devices.list for that snap, but when running under --shell i can still open it08:24
zygamborzecki: _hmm_08:36
zygamborzecki: when you open shell can you see the process listed in the control group?08:36
mborzeckizyga: heh, no cgroup.procs is empty08:38
zygawhat system is that?08:39
zygais that devmode system?08:39
mborzeckizyga: 18.0408:39
zygathat's interesting08:40
zygaah08:40
zygawell08:40
zygathere's this one old odd behavior08:40
zygaif you have no device tags for a given security tag08:40
zygathen we don't use cgroups08:40
zygano tagging == no device cgroup08:40
mborzeckizyga: sounds like a problem, with this interface you could get access to all /dev/input/*08:42
zygayes, it feels like we should make it unconditional08:43
zygacan you check if that's still the case08:43
zygathe code is in snap-confine/ in udev-support AFAIR08:43
mborzeckizyga: feels like this is going to be fun, i need some coffee first08:45
zygauh, I forgot about dentist appointment08:47
zyganeed to go08:47
mvopstolowski: quick question: we only run the post-refresh hook on refreshes, right? not on a fresh install? so if I want something that run on a fresh install and a refresh I need to install two hook hanlders?08:50
pstolowskimvo: correct, pre- and post-refresh hooks run only on refreshes. and install hook is run only on fresh install08:53
dokomvo, zyga: the snapd autopkg tests are not happy with python 3.7 in disco. please fix08:54
mvopstolowski: ta08:54
mvodoko: sure, looking08:55
zygaThank you doko09:10
* zyga is at the dentist09:10
dokois snapd such a pain?09:13
mwhudsonzyga: thanks for https://bugs.launchpad.net/ubuntu/+source/plainbox/+bug/180168709:15
mupBug #1801687: Please remove the plainbox package <plainbox (Ubuntu):New> <https://launchpad.net/bugs/1801687>09:15
Chipacamorning from the summit09:19
mborzeckiChipaca: hey, what's the turnout?09:28
Chipacamborzecki: ~100,000 people09:29
mborzeckiChipaca: over 9000 ;)09:29
Chipacamborzecki: :-)09:29
mvoChipaca: good morning! how are things there? anything exciting happening already?09:33
diddledanmvo: popey is playing with his nipple already09:46
popey!09:47
ograin public ?!?!09:47
diddledanINORITE09:47
mvodiddledan: *cough*09:48
=== iliv_ is now known as ivan_CMD
mupPR snapd#6096 opened: spread.yaml: add more systems to the autopkgtest and qemu backends <Created by mvo5> <https://github.com/snapcore/snapd/pull/6096>09:59
mupPR snapd#6097 opened: interfaces/tests: MockHotplugSlot test helper <Hotplug 🔌> <Simple 😃> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6097>10:00
mvodoko: I uploaded a new snapd into disco that should now run the adt tests, will be interessting to see if they fail or pass so maybe there is more work needed but this should at least unblock things10:06
zygaDentist over, heading home10:06
mvozyga: welcome back10:06
zygaWhat was the issue with python 3.7?10:06
mvozyga: pr is 609610:07
mvozyga: its an internal issue might be worth reworking how we drive adt10:07
dokoI love it when you have to do things like that for every release ... maybe just do the reverse, where it's *not* supported?10:08
mvodoko: sure, let me look what we can do to make this simpler10:08
mvodoko: thanks for approving it though10:09
* mvo looks into reworking that code10:09
popeydegville: https://snapcraft.io/docs/reference/env is referenced in a few places and redirects to a 410 page https://docs.snapcraft.io/reference/env10:15
degvilleooh, thanks popey.10:21
pedronismvo: what's the status of 2.36?  is there anything immediately blocked on me?10:23
mvopedronis: we need confirmation about the 2.36 API of "system" vs "core"10:24
mvopedronis: I sent a mail about this but no reply yet10:25
mupPR snapd#6098 opened: interfaces/builtin: support hotplug for camera interface <Hotplug 🔌> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6098>10:25
mborzeckizyga: about that cgroup & apparmor, i think now with the process ending up in devices cgroup always, accessing /dev/input/event2 is blocked before lsm gets to have a say on this10:27
zygaand back home now10:28
zygamborzecki: that's correct10:28
mborzeckizyga: ok, i'll push the change for s-c as separate PR, some tests may need to be updated, especially the ones where we check between EPERM and EACCESS10:29
zygaaha, that makes sense10:30
zygathank you10:30
ogramvo, i'm recently working a lot with x86 images ... was there any actual reason that we show the grub menu there ?10:30
ograi mean ... you normally dont select anything there anyway ...10:31
mvoogra: no real reason, it may become more interessting once we provide recovery via this mechanism but even then its not a huge reason10:33
mvoogra: also kind of nice to show people what system they use but *shrug* really no strong reason :)10:33
ograit feel like wasted boot time10:34
ogra*feels10:34
zygaI built the package locally out of a release tarball and the version is correct10:51
zygathe issue must be related to how we build the tarball inside spread10:51
mupPR snapd#6099 opened: cmd/snap-confine: always put the snap process under a device cgroup <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6099>11:02
zygamborzecki: interesting, let's see what we get11:06
zyga(from tests)11:06
mborzeckizyga: yeah, much fun11:07
Chipacahmm11:08
Chipacamborzecki: https://forum.snapcraft.io/t/parallel-installs/767911:08
Chipacamborzecki: says 1. parallel installs are experimental and you need to 'snap set hose-my-system=very-yes'11:09
Chipacamborzecki: but also, 2. you need 2.3611:09
mborzeckiChipaca: yes11:09
mborzeckiChipaca: 2*yes11:09
Chipacamborzecki: I thought 2.36 marked them non-experimental?11:09
mborzeckiChipaca: no, we delayed that to 2.3711:09
Chipacaah, missed that11:09
Chipacaok11:09
Chipacamborzecki: are the store-side limitations still true?11:10
mborzeckiChipaca: no, there's a note from wgrant, i should probably update the first message11:10
Chipacamborzecki: yes please :-)11:11
mborzeckiChipaca: aand done11:12
mborzeckiChipaca: are you guys actively trying to break it? :)11:12
Chipacamborzecki: 👍11:12
Chipacamborzecki: people are being told it works /o\11:12
mborzeckiChipaca: good, good, ping me if there are any questions11:13
Chipacawill do :-)11:13
mborzeckizyga: that device cgroup change, i think it's more of a 2.37 material, wdyt?11:15
zygamborzecki: not for 2.36 for sure11:16
zygawe need to discuss with jdstrand and pedronis11:16
mupPR snapd#5946 closed: cmd/snap: unhide --name parameter to snap install, tweak help message <Parallel installs ⛓> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/5946>11:18
pedroniszyga: I'm going a bit through my email etc backlog and some travel admin stuff, but I should be available later in the week to discuss things11:24
Chipacamvo: https://pastebin.ubuntu.com/p/4hpHK8MRHq/11:24
Chipacaniemeyer: ^ i really wish we could do smarter formatting of tables on the terminal :-(11:26
mvoChipaca: woah11:26
Chipaca(granted that message is longer than it needs to be :) )11:26
zygapedronis: thank you11:29
zygaChipaca: use case for those ultra crazy wide monitors ;)11:29
ogra80 chars are enough for everyone !11:39
mupPR snapd#5916 closed: data: run snapd.autoimport.service only after seeding <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5916>11:49
mborzeckizyga: a bunch of tests failed in 609911:49
Chipacazyga: I see your wide monitor use case, and I raise you a use case for in-snap sub-mount namespaces11:50
zygaChipaca: whaaat?11:50
Chipacazyga: also in-snap network namespaces11:50
Chipacazyga: you started! this is all your fault11:50
zygathat I actually want11:50
zygaapp firewall11:50
zygalooking at logs there maciej11:51
zygaeh11:51
zygalog too long11:51
zygamborzecki: Ensure we can run a statically linked binary from an empty base <- this failed, interesting11:51
mborzeckizyga: yeah, the log hit travis limits11:51
zygamborzecki: I think at this point you need to see what happens until you can run the statically linked echo test11:52
mborzeckizyga: anyways, Operation not permitted popping up quite frequently, must have been blocked by device cgroups then11:52
zygayeah11:53
Chipacapedronis: o/11:57
mborzeckiChipaca: error: invalid argument for flag `--id' (expected main.snapshotID): strconv.ParseUint: parsing "": invalid syntax, is this something you were looking for?12:07
Chipacamborzecki: you should have logs12:09
Chipacamborzecki: if it's amazon telling you about id -2, take its vodka away12:10
Chipacamborzecki: uid -2 that is12:10
mborzeckiChipaca: yes, it is https://paste.ubuntu.com/p/Ms9jXqvH6V/12:10
Chipacabah12:10
Chipacabah²12:10
Chipacabah³12:10
Chipacabah⁹⁹12:10
Chipacamborzecki: i'm not sure i can handle it better than just failing12:11
Chipacamborzecki: but also i don't know why i'm getting that -212:11
Chipacamborzecki: I do know why it's a -2 :-)12:11
Chipacamborzecki: amazon linux 2, yes?12:11
mborzeckiChipaca: yes12:11
mborzeckihm we could dump /etc/passwd in debug12:12
Chipacamborzecki: AIUI it's a manifestation of https://github.com/golang/go/issues/2273912:13
Chipacamborzecki: the -2, i mean12:13
zygaChipaca: it is an off-by-one issue :)12:14
Chipacanope, it's golang's syscall return value munging12:14
Chipacaso12:14
Chipacathat -2 comes from os.Getuid() returning something too big12:14
Chipacafor go12:14
pedronisChipaca: hi12:15
Chipacawhich doesn't make sense on itself12:15
Chipacabut ¯\_(ツ)_/¯12:15
mborzeckiChipaca: although it's globbed, the path i mean12:15
Chipacapedronis: hi! welcome back. Random question about aliases  (from a user at the summit): are multiple snaps having the same alias supported ok store-side?12:15
mborzeckiChipaca: some stat maybe? uid/gid handled incorrectly12:15
Chipacapedronis: i assume yes but dunno how tested it is :-)12:15
Chipacamborzecki: i literally just remembered where the -2 comes from today, so i need to comb it12:16
pedronisChipaca: yes, you'll need to use snap prefer  and --unaliased in case you want them both on the same system12:16
Chipacapedronis: yep12:18
niemeyerChipaca: Yeah, that looks pretty bad12:19
niemeyerChipaca: We need something significantly more polished12:19
Chipacaniemeyer: I can make it shorter, but I can't make it short enough without giving the user context about why they're being warned12:19
Chipacai mean, short,  or with context12:20
Chipacajust saying "this app is devmode" isn't enough imo12:20
niemeyerChipaca: The date formating is also breaking our old rule about having no spaces before the last item12:20
cachioniemeyer, hey, when you have some time, could you please take a look to this PR? https://github.com/snapcore/spread/pull/7012:20
mupPR spread#70: Reboot on backgraound to avoid spread wait for long time <Created by sergiocazzolato> <https://github.com/snapcore/spread/pull/70>12:20
niemeyerMaking it even harder to read12:20
niemeyercachio: Sure12:20
niemeyerI have some time now, but it'll be hard to review it while in line at the border :)12:21
niemeyerChipaca: We might have a short description and a link12:22
Chipacaniemeyer: i was thinking of "snap help devmode"12:22
Chipacaie having topics12:22
Chipacaas well as commands12:23
niemeyerThat sounds nice!12:23
niemeyerChipaca: Perhaps just some twist on it so it doesn't look like a command?12:23
niemeyerDo we support that today?12:24
niemeyersnap help <cmd>?12:24
Chipacaniemeyer: yes12:24
niemeyerYeah, so we need to disambiguate12:24
Chipacaniemeyer: and snap help --all  is how you get the long list of everything12:24
Chipacaso 'snap' on its own tells you to 'snap help --all' for ex12:24
cachioniemeyer, great, thanks12:25
niemeyersnap about devmode12:25
Chipacaniemeyer: wrt the timestamp, i'll change it to use the shorter format we discussed for 'snap saved'12:25
niemeyerChipaca: Sounds good12:25
Chipacathat one doesn't have spaces12:26
niemeyerChipaca: Or we could really use the link for the detailing of an issue12:26
niemeyerChipaca: Or alternatively, transform the output of warnings into yaml12:27
niemeyerSo we can actually read it12:27
mupPR snapd#6100 opened: overlord/ifacestate: hotplug-remove-slot task handler <Hotplug 🔌> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6100>12:33
mupPR snapd#6101 opened: switch travis unit tests to xenial <Created by chipaca> <https://github.com/snapcore/snapd/pull/6101>12:34
zygasigh12:35
* zyga tries again12:35
pedronisChipaca: I remember we do line wrapping for some errors, either way the super informative warning is good if it's are, but will get annoying if the warning is very common12:35
mupPR snapd#5963 closed: tests/hotplug: spread test for hotplug based on dummy interface <Hotplug 🔌> <⛔ Blocked> <Created by stolowski> <Closed by stolowski> <https://github.com/snapcore/snapd/pull/5963>12:35
pedroniss/it's are/it's rare/12:35
Chipacapedronis: agreed about that12:41
zygahttps://www.irccloud.com/pastebin/mEgjLBWg/12:42
zygamborzecki: ^ weird, right?12:42
zygainfo file is correct too12:43
mborzeckizyga: 1337 version is better than none :)12:43
mborzecki3133712:43
mborzeckizyga: the one i have here locally is ok12:44
zygayeah12:44
mborzeckizyga: from what i see we change it in prepare when bulding rpms12:45
zygaI think I know what's wrong now12:45
zygacopy12:46
mborzeckizyga: are you calling go generate as part of the build?12:46
zygaI call mkversion.sh in prepare12:46
zygagoogle:opensuse-42.3-64 /# find -name version_generated.go12:46
zyga./usr/src/packages/BUILD/snapd-1337.2.36/gopath/src/github.com/snapcore/snapd/cmd/version_generated.go12:46
zygaso this file is good12:46
Chipacaniemeyer: some cool people from travis-ci are here12:47
Chipacaniemeyer: it turns out we can throw money at them and get more concurrent runs12:47
mborzeckizyga: then why it's unknown in snap version output?12:47
Chipacaniemeyer: it's just a bit messy and requires a human, because we're using .org12:47
zyga./usr/src/packages/BUILD/snapd-1337.2.36/gopath/src/github.com/snapcore/snapd/cmd/VERSION12:47
Chipacaniemeyer: let's chat12:47
zygathere's also this which is equally correct12:47
zygamborzecki: now I'm confused :/12:48
* zyga looks12:48
Chipacagrr, unit tests are 8 minutes again :-(12:49
mborzeckizyga: in arch i'm calling ./mkversion.sh $pkgver-$pkgrel12:49
mborzeckizyga: so it's not guessing the version12:49
pedronismborzecki: hi, was https://forum.snapcraft.io/t/cross-snap-service-ordering/8319 discussed with niemeyer, it doesn't seem to match the brief discussion we had in SLC12:49
zygathe version is fine in VERSION and in go so it's not that12:50
mborzeckipedronis: hi, no hence the forum topic, could you leave a comment with what you discussed in slc there?12:50
zygamust be something I'm missing :/12:50
zygaI was thinking it must be a copy12:50
zygaC and Go are built differently12:50
zygaC picks up the version okay12:50
zygago uses import paths to build stuff12:50
zygaso I was thinking there must be a bit of copy that was not changed with mkversion somewhere12:51
pedronismborzecki: I'll try to put something there today or tomorrow, we just sketched some ideas12:51
pedronisit will need further discussion12:51
zygagoogle:opensuse-42.3-64 /root# GOPATH=/usr/src/packages/BUILD/snapd-1337.2.36/gopath go build -buildmode=pie github.com/snapcore/snapd/cmd/snap12:51
zygahttps://www.irccloud.com/pastebin/szuxkg0r/12:51
zygaso this worked okay12:51
mborzeckipedronis: fair enough, i'm out to conference on wed and thu so no rush12:51
niemeyerChipaca: Ack12:51
zygaI'll add a sanity check in %build to see what version we se12:51
zygamborzecki: ah, there _is_ a copy12:53
zygawtf?12:53
zygawe have /home/gopath12:53
zygaand /usr/src/12:53
* zyga looks12:53
zygabut this is why this happened12:53
mborzeckisrc.rpm got installed?12:53
zygano, I think that's our spread hacking magic12:54
zygaaka mess12:54
zygahmm12:55
zygamborzecki: what is GOHOME?12:55
mborzeckiGOHOME?12:56
zygait's set in spread.yaml12:56
zygagit grep GOHOME12:56
zygalooks like our invention12:56
mborzeckizyga: right, but we set GOPATH using that12:56
zygathat's fine though12:56
zygaI wonder where /usr/src came from12:56
zygaspecifically usr/src/packages/BUILD12:57
zygadoesn't feel like something out of srcrpm12:57
* zyga looks12:58
zygaquick coffee while spread starts up12:58
Chipacazyga: /usr/src is GOROOT12:59
ChipacaI'm guessing12:59
zygaChipaca: nope, suse sets GOROOT in the environment to something like /usr/lib64/go/1.1113:02
zygathere is no match for /usr/src in our tree so I'm puzzled13:02
pedronismvo: I pinged you from a couple of unanswered forum topics13:03
mborzeckioff to pick up the kids13:05
zygaI see what's going on now13:13
zygajust not sure how /usr/share/packages is defined13:14
zygabut that's fine13:14
zygagot it!13:14
zyga# rpm -E '%{_topdir}'13:14
zyga/usr/src/packages13:14
zygaso13:14
zygawe have a copy there in BUILD13:14
zygaand we have a copy in /home/gopath13:15
zyganow just to untangle that13:15
Son_Gokuzyga, it's /usr/lib/rpm/macros13:18
Son_GokuSUSE patches rpm to force that13:18
zygayeah, I know now :)13:18
Son_Gokuwhat it does is that if ~/rpmbuild doesn't exist, it will use that directory instead13:19
Son_Goku(IMO, that's broken, but whatever...)13:19
Son_Gokualso... zyga: https://twitter.com/fedora/status/105972834266698956813:19
Son_Gokulooky, you're in the picture :D13:19
zygaNiiiice :)13:20
Son_Gokuas am I! weee! :D13:20
ograseems he surrenders13:20
ogra:)13:20
Son_Gokuyep, he's now a Fedora man13:21
Son_Gokuno more Ubuntu for him :D13:21
ogrago IBM !13:21
ogra:P13:21
Son_Goku:S13:22
jdstrandmborzecki: fyi, the fuse-support test in your cgroup PR is failing cause you're checking for Permission Denied (as you said above)13:32
Son_Gokuzyga, mborzecki, you guys will want to look into this: https://bugzilla.redhat.com/show_bug.cgi?id=1438079#c413:34
Son_GokuI have a feeling we're switching the unified hierarchy for cgroups (aka cgroups v2) very soon13:35
mvopedronis: thanks for the pings, I'm replying now13:35
zygaSon_Goku: I know about it, I talked to zbyszek13:35
zygaI think that v2 is not ready yet13:35
zygait doesn't have the controllers we need13:35
Son_Gokuok13:36
mborzeckijdstrand: a couple of tests fail the same way, and i have some issues with devmode, we do not generate udev rules in that case but with the PR s-c will still put the snap under a cgroup13:38
mborzeckiSon_Goku: thanks for the heads up13:38
mborzeckizyga: iirc we're only missing freezer at this point, aren't we?13:39
zygamborzecki: haha, not sure if only but the freezer is the first one I noticed13:39
Son_Gokuzyga, need karma for this: https://bodhi.fedoraproject.org/updates/FEDORA-2018-48cc10ba1d13:40
zygaack, I'll test it today13:40
mborzeckizyga: maybe it'd be a good idea to check what's in there already and what we're missing13:40
mborzeckizyga: switching will be proobably held for as long as docker dones not support v213:41
zygamborzecki: based on my discussion with zbyszek there is really no rush for this, it will likely take years13:41
ograppisati, did you take a look at the pi3 b+ NIC-LED isues described in https://forum.snapcraft.io/t/support-for-raspberry-pi-3-model-b/4509 ?13:42
ogra(i think the patchset i pointed to initially had fixes for that too)13:42
jdstrandmborzecki: I understand the PR and the motivation. Please see my comments in both PR 5897 and PR 6099. 5897 can land if we force the use of the cgroup, like we did with joystick (see comment)13:44
mupPR #5897: interfaces/builtin: add device-buttons interface for accessing events <⛔ Blocked> <Created by bergotorino> <https://github.com/snapcore/snapd/pull/5897>13:44
mupPR #6099: cmd/snap-confine: always put the snap process under a device cgroup <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6099>13:44
mborzeckijdstrand: thanks! will do13:51
zygamborzecki: fixed, I think13:51
zygajust quick local spread run before pushing13:52
mborzeckizyga: what was it?13:52
zygamborzecki: GOPATH takes priority over what we want to build13:52
mupPR snapd#6094 closed: wrappers: fix generating of service units with multiple `before` dependencies <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6094>13:52
zygaI swapped GOPATH with indigo_gopath13:52
zygahttps://www.irccloud.com/pastebin/wYHe3zeU/13:52
zygamborzecki: ^ my notes from this session13:52
ppisatiogra: i'm looking into it - the led patches are already there, but one of the function (lan78xx_otp_read()) is buggy, or to better - if i apply the correct fix, then leds are working but the entire chip stops working (we don't get any phy interrupt when packate arrives at the interface)13:55
ppisati*packets13:55
ograok ...13:56
* ogra comments in the forum thread13:56
ppisatiogra: IOW, the lan78xx driver in 4.4 relies on the buggy behaviour of that function13:56
zygamborzecki: in distro builds this doesn't happen because osc uses a chroot and controls GOPATH13:57
Chipacazyga: pstolowski: using a serial port on classic, does it need hotplug or is it alredy there?13:59
zygaChipaca: you need hotplug13:59
pedronismvo: thanks for answering those topics14:00
mborzeckimvo: can you cherry pic https://github.com/snapcore/snapd/pull/6094 to 2.36?14:01
jdstrandmborzecki: ok, and now you have my code review for 6099. thanks again for picking that up14:01
mupPR #6094: wrappers: fix generating of service units with multiple `before` dependencies <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6094>14:01
zyga(new old standup time)14:01
mborzeckijdstrand: thanks, it was fun to look into this :)14:01
zygaChipaca mvo pstolowski pedronis  ^14:01
ogramvo, couldnt you just remove the state.json to get back into console-conf (effectively doing a fake "factory reset") ?14:03
zygamborzecki: can you look at https://github.com/snapcore/snapd/pull/6095 after the call please14:09
zygaI think it should pass now14:09
mupPR #6095: packaging/opensuse: stop using golang-packaging <Created by zyga> <https://github.com/snapcore/snapd/pull/6095>14:09
Chipacapedronis: another question14:10
pedronisChipaca: in the standup, but do ask, I will come back later14:11
mvoogra: yeah, that should work as well, I didn't suggest it in the forum because I wasn't sure if there were any "precious" things on the system, maybe he has some snaps he does not want to lose, but yeah, if not its a solution too14:11
Chipacapedronis: ok14:12
mvomborzecki: 6094 is cherry-picked14:12
mborzeckimvo: thanks!14:12
Chipacapedronis: if somebody wanted to use snaps in a cloudy context where they want the image to boot and have a number of snaps already installed14:13
Chipacapedronis: they feel they could just install them, snapshot the disk, and then just run with it14:13
Chipacapedronis: seeding is too slow, installing from zero is too slow14:14
mvoI wonder if we could put things on disk in the prepare image state and just run the hooks at "seeding" time. probably needs careful thinking but my gut feeling is that everyone wants faster seeding14:16
Chipacaye14:18
Chipacamborzecki: also also14:18
Chipacamborzecki: you lied to me: the " Due to the current limitations in the store, multiple instances of a snap need to be installed from the same single command, as shown above." thing is still on https://forum.snapcraft.io/t/parallel-installs/767914:18
cachioChipaca, https://paste.ubuntu.com/p/XjP3hcFr4P/14:19
cachioI saw this one today14:19
Chipacacachio: yes14:19
Chipacacachio: thank you14:19
Chipacacachio: it's the -2 user id, i need to look into it14:19
Chipacanot today14:19
cachiosure14:19
Chipacacachio: restart, but thank you for the paste (is it set to expire?)14:19
cachiono14:20
cachioChipaca,14:20
Chipacacachio: tks14:21
Chipacamborzecki: i edited14:23
mborzeckiChipaca: thanks14:31
mborzeckiChipaca: does it work so far for you guys? :)14:31
Chipacamborzecki: i have not been lynched yet14:32
Chipacamborzecki: and i get to keep _most_ of my fingers!14:32
mborzeckiChipaca: maybe they're still building the pyre14:32
Chipacamborzecki: they're still drunk from guy fawkes last night14:32
zygaPharaoh_Atem: what's the difference between %{!?... and %{?!...} ?14:45
Pharaoh_Atempreference, mainly14:46
Pharaoh_Atemsome broken spec parsers handle them differently14:46
Pharaoh_Atemboth otherwise they should behave the same14:46
zygawhy do we use both?14:46
zygashould we use the same syntax or is there a good reason to keep the difference14:46
zygaI just noticed because of a bug that affected leap14:47
Pharaoh_AtemI just switched everything in the fedora spec to %{!?...}14:48
zygathanks14:48
zygaI'll do the same14:48
Pharaoh_Atemit makes more sense in my head anyway14:49
Pharaoh_Atem"does not exist" vs "exists, not?"14:49
mupPR snapcraft#2392 opened: ci: update travis.yaml to use xenial <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2392>14:49
Chipacachesty: https://bugs.libssh.org/T11814:55
pedronisChipaca: if they use a prebooted image the serial number would be wrong,  we don't have an answer at the moment except you do need to do seeding, but is an open question, we discussed about it a bit with mvo etc in SLC14:58
Chipacapedronis: yeah, seeding is going to be too slow15:00
Chipacapedronis: we're talking about a 14 seconds boot being omg slow15:00
Chipacapedronis: not a blocker but we really need to address this soonish15:01
chestyChipaca, nice. why struct passwd *pwdbuf = NULL; ? is extra precaution in case  of a bug in getpwuid_r ?15:07
Chipacachesty: i guess so yes15:08
jdstrandroadmr: fyi, finally got all the revisions before r79 of pivx and now I was able to run the automated tools again on 7915:22
roadmr\o/15:22
roadmrthanks for checking.15:22
Chipacahmmmmmmmmmmmmₘₘₘₘₘₘₘₘₘₘ15:24
mvoChipaca: well, I think we should thing about putting things on disk in prepare image and just running the hooks but its a bigger change and needs careful investigation15:25
Chipacamvo: ₘₘₘₘₘₘ15:25
Chipacamvo: :-)15:25
Chipacamvo: why run the hooks?15:26
mvoChipaca: running the hooks in prepare image is going to be a challenge - we may create an arm image with a amd64 host15:27
Chipacamvo: we could also not support doing the whole fast boot set up on an m×n arch grid15:33
Chipacaanyway i figured out where the snapshot -2 uid error comes from, probably™15:34
pedronisChipaca: we can add some constraints, but we cannot be fully arbitrary or hackish either15:34
pedronisit needs some toughts15:34
Chipacapedronis: yes15:34
=== JanC_ is now known as JanC
ograzyga, are layouts on by default in 2.36 ?15:42
zygayes15:44
ograso i can drop the passthrough in my snapcraft.yaml ?15:45
roadmrogra: try dropping it; if snapcraft complains, then you'll still need it :)15:45
pedronisthe passthrough is about snapcraft, not snapd15:45
roadmr^^ this15:45
ograhahaha15:45
zygaogra: that's a question for kyrofa and sergiusens15:46
zygaroadmr: processing...15:46
roadmrzyga: yay!15:46
zygait worked!15:46
roadmrof course ;) thanks to the work jdstrand did yesterday15:46
zygaand released!15:48
zyga(to beta)15:48
kyrofazyga, ogra layouts are supported in 3.0 as long as you specify a base15:52
zygawoot15:52
kyrofaOtherwise passthrough is still required15:52
ograurgh15:52
roadmrzyga: yay \o/15:52
ograkyrofa, how does that work for all my exiting snaps ... is a default base picked when i do a rebuild ?15:52
ogra*existing (even though some are exciting :P )15:53
zygaogra: you need to pick base: core15:54
zygaogra: or core1815:54
ograzyga, so i need to update all exiting snapcraft.yamls ?!?!15:55
zygayes15:55
zygathat's intentional15:55
ograsigh ...15:55
zygato opt into the new build semantics15:55
kyrofazyga, no, "core" is not a valid base, "core16" is. But that isn't really a thing yet15:55
ograyou mean build.s.io will fall back to snapcraft 2.x if i dont ?15:55
kyrofaBingo15:56
ographew15:56
ograk15:56
zygakyrofa: why core is not a valid base?15:56
kyrofaogra, we really didn't want to break folks15:56
zyga(core is a valid base in snapd)15:56
kyrofazyga, isn't core going to only contain snapd?15:56
zygano15:56
kyrofaEh?15:57
zygakyrofa: you are talking about the snapd snap15:57
ograkyrofa, yeah, i would have been surprised if you did ... i trust you and sergio blindly normally ;)15:57
zygakyrofa: core will contain snapd and core1615:57
zyga(and does so today)15:57
zygakyrofa: core16 is just core sans snapd15:57
kyrofazyga, core isn't being renamed to core16, then?15:57
zygakyrofa: no15:57
zygakyrofa: core16, core and core18 are separate snaps15:57
ograthey dropped that renaming idea15:57
zygakyrofa: core16 and core18 don't have snapd anymore15:57
kyrofaYikes. All three will need to be maintained?15:57
zygakyrofa: snapd snap is a separate required snap in that case15:57
zygakyrofa: for now yes15:57
zygakyrofa: we will transition people from core to core1615:58
zyga(that is core to core16 + snapd)15:58
kyrofaOkay, I wasn't clued into the dropping of the rename. Still though, I think "core" shouldn't be used as a base15:58
zygapedronis: ^ in case I'm mistaken15:58
kyrofaWe should steer people toward core1615:58
zygapedronis: core, core16 and core18 will be maintained until we complete the transition15:58
zygakyrofa: sync with pedronis and mvo on timing please15:59
zygakyrofa: core16 can be used as a base AFAIK (correctly)15:59
zygakyrofa: as can core1815:59
kyrofaWhich means the only supported bases for now (at least in snapcraft, given that it's a new feature) should be core16 and core1815:59
kyrofazyga, it's not stable yet though, no?15:59
zygathe transition is moving people away from core as the holder of snapd15:59
kyrofa(core16, I mean)15:59
zygakyrofa: I think it is equally stable15:59
zygakyrofa: we just don't have the migration code in place15:59
zygabut operationally it is just like core16:00
kyrofazyga, I mean it literally isn't in the stable channel16:00
kyrofaIt's edge only16:00
pedroniskyrofa: I don't think core can be used as a base explicitly16:00
zygakyrofa: note that core16 and core18 are AFAIK maintained by foundations16:00
zygakyrofa: I see16:00
kyrofapedronis, agreed16:00
kyrofaAs it should be16:00
kyrofaAny ETA for when core16 will be stable?16:01
zygakyrofa: I honestly don't know16:03
zygait probably should be now16:03
zygabut it depends on the transition process16:03
zyga(perhaps)16:03
ograso if i want to use layouts in my snaps that are built specifically for Ubuntu Core 16, what would i do (and note in no casse ever i want core18 to end up on a core16 install)16:03
pedronisogra: the decision to connect layouts to 3.0 and base is a bit strange, not sure what was the rationale there16:06
ograyeah16:07
ograwe'll definitely want to use it for existing customer ... and many of them will never go to 18 ... (and wont want an additional core18 base nasp installed either)16:08
ogra*cusstomerss16:08
ogra(GRRR ... broken kbd)16:08
sparkiegeeksergiusens: https://paste.ubuntu.com/p/ZFKJXbVMSj/16:13
sparkiegeeksergiusens: 95% sure that's not related to my change16:14
sergiusenspedronis: we did not have anyone request it, it can be done16:14
sergiusenspedronis: that said, folks can still use passthrough and should be fine for the time being16:15
pedronissergiusens: I understand but given it might take a bit fore core16 to go stable, first class support might make sense16:22
* cachio lunch16:23
zygaogra: for ubuntu core 16 - can you use core18 base?16:23
ograzyga, i personally might ... 90% of our customers wont want to yet16:24
mupPR snapd#6102 opened: overlord/snapshots: survive an unknown user <Created by chipaca> <https://github.com/snapcore/snapd/pull/6102>16:24
zygaogra: note that it doesn't imply booting core1816:24
Chipaca^ that should fix the weird amazon issue16:24
zygaogra: but sure, I'm just curious16:24
ChipacaI still don't know _why_ amazon would trigger it, nor why with -2, but, there ya go16:24
zygaChipaca: do you know why the uid is unknown?16:24
ograzyga, it talke disk space on a already size limited device16:25
Chipacazyga: so there's a /home/foo/snap directory owned by uid -216:25
Chipacazyga: that bit, i have no idea why16:25
zygaaha...16:25
zygahmm16:25
Chipacazyga: espeially because the uid comes from stat16:25
zygathat's indeed curious16:25
Chipacazyga: so it's not the syscall.Getuid bug16:25
zygaI mean, if you stat it in a shell you see -2?16:25
zyga!16:25
Chipacazyga: I haven't been able to reproduce it wiht -debug,  but if I could, I'd see the big uid16:26
bashfulrobotniemeyer:16:31
Chipacabashfulrobot: ssh16:31
Chipaca:-)16:31
ograChipaca, he should ssh into gustavo ?!16:32
ograis there a pub-key available for that ?16:32
Chipacabashfulrobot: g.ustavo is presenting and has irc on16:32
Chipacabashfulrobot: what could go wrong16:32
bashfulrobotChipaca: sorry accidental. Apologies.16:33
* zyga dinner 16:33
mupPR snapcraft#2393 opened: lifecycle: make snapcraft init template use > not | <Created by sparkiegeek> <https://github.com/snapcore/snapcraft/pull/2393>16:58
mupPR snapcraft#2388 closed: project: early snapcraft.yaml validation <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2388>17:31
mupPR snapcraft#2391 closed: runtests: run black with --diff <Created by abitrolly> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2391>17:31
zygare17:32
pedronismvo: https://github.com/snapcore/snapd/pull/6093 , is this really for 2.36 ?18:19
mupPR #6093: daemon: spool sideloaded snap into blob dir <Created by chipaca> <https://github.com/snapcore/snapd/pull/6093>18:19
mvopedronis: yeah, we got a message from field (CE) that it affected a customer18:20
pedronisah, ok18:20
mvopedronis: if you feel uneasy about this please let me know18:20
mvopedronis: it seems relatively harmless (famous last words)18:21
pedronismvo: I haven't actually looked at it yet, just wondered to see something still under 2.3618:21
mvopedronis: ok18:22
pedronismvo: I can look at it in the morning18:55
mvopedronis: as you wish - I will leave it until then18:56
zygaurgh19:07
zygarpm -q --whatprovides /usr/share/pkgconfig/systemd.pc19:07
=== kali_ is now known as kaliy
zygaand now all three suse releases have working snapd19:36
zygacachio: hey, do we have opensuse leap 15?19:49
zygacachio: and opensuse tumbleweed19:49
zygaleap 42.3 is a bit dated now19:50
sunesitohi all20:01
sunesitoi have installed snapd on ubuntu and when i try to run an app i have that error:Gtk-Message: Failed to load module "gail" Gtk-Message: Failed to load module "atk-bridge" Gtk-Message: Failed to load module "unity-gtk-module"  (acestreamplayer:5130): GLib-GIO-ERROR **: No GSettings schemas are installed on the system20:02
sunesitowhat am i missing?20:02
zygasunesito: hey20:11
zygaI assume the app doesn't start at all20:11
zygawhat is the name of that app?20:11
sunesitono20:11
sunesitoacestreamplayer20:12
ograyou should be using a desktop launcher part for snapcraft20:14
zygaogra: I'm not sure if sunesito is the developer of that app20:14
ograoh20:14
ograyeah, ignore me then (giving developer tips :) )20:14
sunesito¿? ill try it20:15
ograwell, are you the developer of that snap ?20:15
ogra(i was refrerring to code changes in the snap package)20:16
sunesitoahhh...no...im not the developer...only a user20:16
ograhttps://snapcraft.io/acestreamplayer ...20:17
ograLast updatedFeb 21 201720:17
ograhmm20:17
sunesitosorry...is this a developers chanel?20:17
ograhasnt been touched in nearly two years20:17
ograsunesito, no, not at all20:17
ogra(well ... too ... but as well for user questions/support)20:17
sunesitothanks...i will investigate under ubuntu problem...because i have running it on debian os20:18
ograthe snap ?20:19
sunesitono....the error20:20
ogrado you have the snap running on debian ?20:20
sunesito@ogra sorry....yes, the snap with acestreamplayer20:20
ograaha20:20
ograthats an interesting datapoint20:20
ograwhats the output of: snap version20:21
ograon your ubuntu20:21
sunesito2.35.520:21
ograthe full output please20:21
sunesitook20:21
sunesitoubuntu_1604:~$ snap version snap    2.35.5 snapd   2.35.5 series  16 ubuntu  16.04 kernel  4.2.820:21
ograwhere does that kernel come from ?20:22
ograthats not an ubuntu kernel20:22
ogra(and likely the reason for your issue)20:22
ograzyga, wasnt there a fix in snapd that warns you if you use an unsupported kernel ? or is that not in stable yet ?? (see above)20:23
sunesitothis is an qnap20:23
sunesitoNAS20:23
zygaogra: not exactly, there's only one specific version we warn about20:23
ograah20:23
zygaon 14.04 that is still on 3.1420:23
sunesitobut i have running snap correctly in this system20:23
zygasunesito: interesting, did you install snapd yourself or did it come with it?20:23
zygaand wait, didn't you say this is a desktop app?20:24
sunesitoi install via apt-get20:24
zygais this a NAS with a screen?20:24
sunesitoyes...20:24
sunesitohdmi port20:24
zyga:D20:24
zyganice, so a nas / desktop :)20:24
sunesitoyes20:24
sunesitoi need to reinstall the ubuntu....and now i have this problem...but i think the problem is in the ubuntu packages20:25
mupPR snapd#6103 opened: tests: split nested vm suite on core and classic and new snapshots test <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6103>21:15

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