/srv/irclogs.ubuntu.com/2017/06/12/#snappy.txt

mupPR snapd#3416 closed: tests: fix for test interfaces-openvswitch <Created by sergiocazzolato> <Closed by sergiocazzolato> <https://github.com/snapcore/snapd/pull/3416>01:54
=== chihchun_afk is now known as chihchun
=== JanC_ is now known as JanC
=== chihchun is now known as chihchun_afk
mupPR snapd#3468 opened: debian: add missing  Type=notify in 14.04 packaging <Created by mvo5> <https://github.com/snapcore/snapd/pull/3468>07:02
mupPR snapd#3458 closed: tests: check that locale-control is not present on core <Created by fgimenez> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3458>07:09
=== koza|away is now known as koza
=== chihchun_afk is now known as chihchun
mupPR snapd#3454 closed: spread: add fedora snap bin dir to global PATH <Created by morphis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3454>07:24
morphismvo: thanks!07:25
mupPR snapd#3452 closed: tests/main: check for confinement in a few more interface tests <Created by morphis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3452>07:25
mupPR snapd#3451 closed: tests/main: use dir abstraction in a few more test cases <Created by morphis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3451>07:26
zygagood morning07:27
mupPR snapd#3443 closed: Unity7 interface grows gtk2/3 settings and user's specific ini <Created by didrocks> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3443>07:28
mvofgimenez: if you de-conflict 3391 I am happy to merge it07:30
mvomorphis: my pleasure, thanks for working on this!07:30
fgimenezmvo: sure thx! on it07:31
morphismvo: np, more to come :-)07:31
mupPR snapd#3405 closed: tests: fix for upgrade test when it is repeated <Created by sergiocazzolato> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3405>07:31
mvomorphis: 3222 has a conflict and test failures (probably unrelated) - could you merge master and de-conflict, hopefully this can go in then :)07:31
mvofgimenez: no rush :)07:31
morphismvo: yeah will start working on these PRs in a bit07:34
mvomorphis: ta07:37
fgimenezmvo: snapd#3391 is ready thx :) btw when you have some time (not urgent) it would be great to create these test snaps under the shared account in prod: https://code.launchpad.net/~snappy-dev/+snap/test-snapd-system-observe-consumer https://code.launchpad.net/~snappy-dev/+snap/test-snapd-autopilot-consumer https://code.launchpad.net/~snappy-dev/+snap/test-snapd-upower-observe-provider07:39
mupPR snapd#3391: tests: reboot after upgrading to snapd on the -proposed pocket <Created by fgimenez> <https://github.com/snapcore/snapd/pull/3391>07:39
pachuloI'm working on a MuseScore snap: https://github.com/pachulo/musescore-snap and trying to get the MuseScore devs to make it "official": any tips on doing it?08:54
ograpachulo, dont they have some "how to contribute" doc on their website ?  i'd try to follow it (make a PR on github etc)09:00
pachuloI've already did something like this: https://github.com/musescore/MuseScore/pull/320409:02
mupPR musescore/MuseScore#3204: Add snap package <Created by pachulo> <https://github.com/musescore/MuseScore/pull/3204>09:02
ograwell, then wait for answer :)09:03
pachuloshould I recommend them to use https://build.snapcraft.io/ to create the snaps?09:07
pachuloogra:09:07
ograwell, thats up to them but yes, show them the possible ways of making use of your code :)09:08
pachulodoes it makes sense to have different snapcraft.yaml for the different channels all in the master branch of the project?09:14
zygapachulo: I would kepe them in separate branches, not sure what the actual difference between those is though09:15
* Chipaca ~> coffee09:16
mupPR snapd#3465 closed: hooks: re-org of some hooks types <Created by stolowski> <Closed by stolowski> <https://github.com/snapcore/snapd/pull/3465>09:26
morphisPharaoh_Atem: `zypper si -d` doesn't help for local srpm's, it only works with packages listed in the archive09:45
abeatoogra, hi, do you know how can I force generation of core file on program crash, in UC16?09:51
ograabeato, you should be able to do the same stuff apport does i think09:52
ogra(iirc it pokes around in sysfs to make file dumps happen)09:52
abeatoogra, ok, will check, thanks09:52
abeatomvo, is 2.27 already in candidate? or latest changes (including androidboot) are only in edge?10:00
mvoabeato: only in edge, we have no pushed 2.26 out, some small issue in specific revert cases still10:04
abeatomvo, got it, thanks10:05
pedronismvo: fgimenez: hi, have we found out more about those revert problems? is still related to profiles or something else?10:23
fgimenezhi pedronis, the core-revert test now shows the problem with the changes at snapd#3466, this is syslog of the testbed after the failure http://paste.ubuntu.com/24815993/ looks like the problem is related to "hsearch_r failed for NETLINK_KOBJECT_UEVENT: No such process" after the revert10:26
mupPR snapd#3466: tests: extend core-revert test to cover bluez issues <Created by fgimenez> <https://github.com/snapcore/snapd/pull/3466>10:26
pedronisok, profiles, thanks10:26
pachulois it possible to upload a snap for different archs in the webui of the store?10:50
dakerpachulo: i think yes, the store will automatically detect the arch10:53
pachulobut if I want to upload the snap for x86 & for amd64?10:54
ograthen the store simply generates a revision per arch ... it check the meta/snap.yaml inside your snap10:58
pachuloah, ok, different revisions per arch11:00
pachulothat makes sense11:00
pachuloand how do you make the snap availabe in the stable channel? In the webui I can only pusblish to "beta" and/or "edge"11:01
ograid your snap confinement: strict and grade: stable ?11:04
ogras/id/is/11:05
ograonly stable snaps with strict confinement can go into stable11:05
pachulook11:06
pachulothanks for the info ogra !11:07
ogranp :)11:07
niemeyerMornings11:20
mupPR snapd#3469 opened: many: add "release.BuildStamp" to identify the current build <Created by mvo5> <https://github.com/snapcore/snapd/pull/3469>11:24
Chipacaoops, i done goofed11:36
* Chipaca fixes11:36
fgimenezmvo: ogra we are still getting errors with the new kernel snaps published to beta due to missing /dev/ram* https://travis-ci.org/snapcore/spread-cron/builds/241982348 should we change the tests to stop using them? the related bug recently expired bug #167762212:02
mupBug #1677622: missing ramdisks in latest amd64 kernel snap <kernel-da-key> <linux (Ubuntu):Expired> <https://launchpad.net/bugs/1677622>12:02
ografgimenez, well, that seems to be a userspace tool issue if i read ppisati's comment correctly12:06
ograyou could just run mknod before mkfs12:07
fgimenezthanks ogra, i'll try that with beta's kernel12:09
ografgimenez, i also think it might be realted to https://github.com/snapcore/snapd/pull/301012:10
mupPR snapd#3010: snap: skip /dev/ram from auto-import assertions to make it less noisy <Created by mvo5> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/3010>12:10
ogra(i suspect the mount attemmpt makes the kernel actually create the device (just y theory though))12:11
ogras/y/a/12:11
fgimenezogra: not sure, we noticed the problem before snapd#3010 was proposed, the first errors are from 2017-03-15 https://travis-ci.org/snapcore/spread-cron/builds/211518618#L66812:14
mupPR snapd#3010: snap: skip /dev/ram from auto-import assertions to make it less noisy <Created by mvo5> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/3010>12:14
ograhmm, k12:14
ograhttps://launchpad.net/ubuntu/+source/snapd/2.23.1 landed on 2017-03-1412:18
ograquite some changes in there12:18
ograand https://launchpad.net/ubuntu/+source/linux/4.4.0-67.88 went into beta on the 15th12:19
ograalso not a small changeset12:19
mupPR snapd#3470 opened: interfaces/builtin: sync connected slot and permanent slot snippet <Created by morphis> <https://github.com/snapcore/snapd/pull/3470>12:59
mupPR snapd#3471 opened: snap: make `snap run` look at the system-key for security profiles <Created by mvo5> <https://github.com/snapcore/snapd/pull/3471>13:30
fgimenezmvo: i didn't receive yet the test snaps emails, should i have them already?13:54
mvofgimenez: you should, let me double check13:56
fgimenezmvo: thx!13:56
mvofgimenez: probably primary email vs not primary sso email again, sorry for that13:57
fgimenezmvo: np :) already received, thank you!13:58
=== chihchun is now known as chihchun_afk
jdstrandnoise][: hey, fyi, https://dashboard.snapcraft.io/dev/snaps/7807/rev/1/ says 'manifest not available'14:03
noise][jdstrand: you are getting an err on the page?14:04
jdstrandnoise][: https://dashboard.snapcraft.io/dev/snaps/7809/ too14:05
jdstrandnoise][: sorry, if I go to Overview, then review capabilities14:05
jdstrandManifest (snap.yaml)14:05
jdstrandmanifest not available14:05
noise][ah, i see14:05
noise][weird - can you file a bug on https://bugs.launchpad.net/snapstore and i'll get someone to take a look ASAP14:06
jdstrandsure14:06
jdstrandnoise][: https://bugs.launchpad.net/snapstore/+bug/169745914:09
mupBug #1697459: Manifest (snap.yaml): manifest not available <Snap Store:New> <https://launchpad.net/bugs/1697459>14:09
* zyga lunch and small break14:10
zygare14:26
morphiszyga, mvo, Pharaoh_Atem: https://github.com/snapcore/snapd/pull/3449 is ready for another review and merge :-)14:56
mupPR snapd#3449: tests/lib: generalize RPM build support <Created by morphis> <https://github.com/snapcore/snapd/pull/3449>14:56
Pharaoh_Atemmorphis: why are you not using `dnf builddep` or `zypper si`?14:58
morphisPharaoh_Atem: as zypper si doesn't work15:06
morphisit works only for packages coming from the archive15:06
morphisand with that I couldn't find a good abstraction and left the comming thing in I had before15:06
Pharaoh_Atemyou gave it the full path of the RPM, not just the rpm name?15:06
morphiscorrect15:06
morphisit complains about not being able to find that package15:07
Pharaoh_Atemwow, that's... dumb15:07
morphisgiving it just a name works fine15:07
morphisif you find a better way on suse I am all ears but want to get this in as I have a lot more on my plate at the moment15:08
Pharaoh_Atemcan you pull all the deps into an array, ignore the ones that are rpmlib() ones, and then just pass that as an arg to zypper install / dnf install?15:08
Pharaoh_Atembecause the way you're doing it now is really slow15:09
mvojdstrand: it looks like we need to revert http://paste.ubuntu.com/24841683/ this is using new symbols and things break on revert- this is 17389627 - it looks dangerous though. what is your opinion here? instead of reverting we could hold a stable release back until the seccomp-bpf branch is finished and lands.15:10
mupPR snapd#3463 closed: client, daemon: expose service commands (start, stop, etc) <Created by chipaca> <Closed by chipaca> <https://github.com/snapcore/snapd/pull/3463>15:15
morphisPharaoh_Atem: that is what I am doing right now15:17
Pharaoh_Atemah, I see where you're passing the array15:17
Pharaoh_AtemI missed that15:18
morphis:-)15:18
Pharaoh_Atemthen the only piece left is that spurious --nocheck on rpmbuild -bs15:19
morphislet me drop that15:20
morphisactually why doesn't rpm complain if it is there and not used?15:20
morphisPharaoh_Atem: done15:21
jdstrandmvo: which stable would we hold back?15:26
mvojdstrand: 2.25 is currently in candidate but never progressed to stable because when 2.24.1 -> 2.25 -> 2.24.1 reverts happen some snaps (like network-manager) fail to start because of seccomp symbols15:28
mvojdstrand: and we found during testing that the above commit is also problematic15:29
jdstrandthat whole PR has a bunch of new symbols15:29
jdstrandI've started looking at the bpf PR15:30
jdstrandI don't know what kind of timeframe you are looking for at this point15:30
jdstrandI'd be inclined to wait for the bpf at this point15:31
jdstrandmvo: ^15:34
mvojdstrand: ok, that is something to discuss I think, I can't make this decision alone, but I have the same feeling, it feels like whack-a-mole currently and the risk is that we break things. seccomp-bpf I can pick-up again and its pretty safe15:37
mvojdstrand: but I definitely want your input if all things I do there are sound :)15:37
jdstrandmvo: I will be looking at the PR in depth15:38
jdstrandmvo: note I already commented on it regarding missing tests15:39
mvojdstrand: tests about the >= etc syntax? those should be ported in a different way, let me look for the link15:39
jdstrandmvo: so all the reverts are making me uneasy because there is risk in getting the revert wrong and getting the unrevert wrong in the future15:40
mvojdstrand: https://github.com/snapcore/snapd/pull/3431/files#diff-be7cfe1e5aff69d70d80b1c2cabcaaccR148 is the testing, it uses a bpf.VM and gives it the same inputs as the kernel would give it. is your concern that you want sometimg more integration-ish? or am I missing someting in those tests that I overlooked?15:41
mupPR snapd#3431: interfaces: simplify snap-confine by just loading pre-generated bpf code <Created by mvo5> <https://github.com/snapcore/snapd/pull/3431>15:41
jdstrandmvo: the tests I commented on were cmd/snap-confine/tests. the will definitely need to be ported in a different way, but the PR doesn't show them15:41
mvojdstrand: yeah, I agree, I don't feel good about those reverts either.15:41
jdstrandmvo: I suspect that the porting will be done in two ways: syntax checks in TestCompile and functional tests to spread15:43
mvojdstrand: aha, I see, something more integration test-ish then, I make that a priority in my morning. it should be unit tested now but you are right, integration type tests in this style are missing15:43
jdstrandmvo: where the spread tests will consist of rewriting the seccomp filter15:44
* mvo nods15:44
jdstrandmvo: it's more than just integration though. there are a ton of syntax tests. eg test_restrictions_working_args_socket15:44
jdstrandthey won't be hard to port or anything, just saying more is needed in TestCompile15:45
jdstrandin addition to integration/spread tests15:45
mvojdstrand: thank you, indeed, I will work on it in my morning. it looks straightfoward to port to TestCompile fortunately15:47
jdstrandmvo: yes. there are relatively few things that need to go to spread otoh15:48
jdstrandmvo: thanks for taking care of that15:49
mvojdstrand: yeah, thanks a lot for your feedback15:49
jdstrandnp15:49
jdstrandmore will be coming :)15:49
mvojdstrand: thank you. I updated the forum thread on the 2.25 issue, feel free to jump in and add your opinion15:59
jdstrandok16:00
mupBug #1697492 opened: systemd fails to run /lib/systemd/system-sleep/wpasupplicant script when wpa-supplicant is installed <Snappy:New> <https://launchpad.net/bugs/1697492>16:12
ograabeato, urgh ... we have a wpa-supplicant snap ?16:13
abeatoogra, we do... was needed for some wowlan stuff in caracalla16:14
ograabeato, i doubt that can work, given that we have a wpa-supplicant in the core snap too16:14
ograunless you block that or make it unexecutable somehow16:15
abeatoogra, it does, mostly, see https://github.com/snapcore/core-build/blob/master/config/lib/systemd/system/wpa_supplicant.service.d/snap.conf , that is how the one in core is blocked16:16
ograah, i remember landing that, yeah16:16
ograbut that only blocks execution, you still have all the files and scripts in the rootfs16:17
ograthats a really tricky bug16:17
ograwe cant just hack the default script16:17
abeatoyeah, it is quite ugly thing16:17
abeatoI agree...16:18
abeatoI'm not sure how this could be solved, maybe the path should have been to add patches to wpa-s in xenial instead16:18
ograwell, if you think that coulld solve the need for a snap ... xenial is there, SRUs are always possible :)16:20
abeatosure... I'll think about that and leave the bug there open for the moment16:22
ograabeato, hmm, i wonder what happens if you ship a /writable/system-data/lib/systemd/system-sleep/wpasupplicant in your gadget snap16:23
ogra(if that gets copied into place etc)16:23
ogramight be possible to override the default file from the gadget that way16:23
abeatoogra, good idea, will give that a try16:23
ograi know we allow copying a bunch of things ... mot sure if /lib is included in that though ... (the ubuntu-image source might know :) )16:24
mupPR snapd#3472 opened: interfaces, tests: add mising dbus abstraction to system-observe and extend spread test <Created by fgimenez> <https://github.com/snapcore/snapd/pull/3472>16:33
cachio_Chipaca, are you working in a store related change? is it something that could be addressed https://travis-ci.org/snapcore/snapd/builds/241920749#L2050 ?17:22
mupPR snapd#3473 opened: tests: fix create-key by generating entropy in case the current it is not enough <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/3473>17:29
=== cachio_ is now known as cachio_afk
niemeyermvo: Still here?18:13
bdmurrayIs this an issue with the kubelet snap or snappy itself? https://errors.ubuntu.com/problem/19272ebc18709d4407dba0438a536d56bb14306918:25
ograbdmurray, https://forum.snapcraft.io/t/test-failures-with-cannot-create-lock-directory-run-snapd-lock/39018:26
niemeyercachio_afk: Some feedback on 343718:26
mupPR snapd#3437 closed: tests: apt autoclean <Created by sergiocazzolato> <Closed by niemeyer> <https://github.com/snapcore/snapd/pull/3437>18:26
bdmurrayogra: ah, thanks18:30
ograogra@bbb:~$ sudo classic18:33
ogracannot open cookie file /var/lib/snapd/cookie/snap.classic18:33
ograhmm, whats that ? that wasnt there yesterday, since todays core refresh i seem to get it everywherre18:34
ogra(i end up just fine in the classic shell after the message)18:34
niemeyermorphis: snapd#3222 is good to go assuming one trivial tweak mentioned there18:54
mupPR snapd#3222: many: fix test cases to work with different DistroLibExecDir <Created by morphis> <https://github.com/snapcore/snapd/pull/3222>18:54
mupPR snapd#3461 closed: debian: add missing "make -C data/systemd clean" <Created by mvo5> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/3461>18:58
morphisniemeyer: thanks, will fix that tomorrow!19:37
niemeyermorphis: Thank you!  Glad to see that one in19:37
morphisniemeyer: lets see how friendly travis is tomorrow to me :-)19:42
niemeyermorphis: I've heard things have been improving much there19:43
morphisniemeyer: yeah they did :-)19:53
=== elfgoh_ is now known as elfgoh
=== blr_ is now known as blr
=== cachio_afk is now known as cachio
ssbashhi everyone!21:38
cachioniemeyer, please when you have a minute could you take a look to the PR 3473?21:43
niemeyercachio: Will do21:50
cachiotc21:50
cachiotx21:50
ssbashDoes anyone know how to fix python path issues? I'm experiencing the same problems from this post https://askubuntu.com/questions/906199/how-to-setup-pythonpath-for-a-snap-package21:59
ssbashBasically my python package is in the site packages directory, but I am getting the error that my command cannot find my package22:00
ssbashcachio do you have any experience dealing with python path issues?22:44
naccssbash: so the wrapper script doesn't set PYTHONPATH at all (should be viewable from the shell session, i think)22:48
ssbashi checked the shell session, this is what it returns https://paste.ubuntu.com/24844778/22:50
cachiossbash, let me see22:51
ssbashhere is the error message http://paste.ubuntu.com/24844781/22:52
ssbashalso here is the snapcraft.yaml http://paste.ubuntu.com/24844790/22:53
naccssbash: so i think the issue is site-packages vs. dist-packages, but i'm not sure why22:54
ssbashenv PYTHONPATH variable seems to be correct as I have specificed in the .yaml file. However my python script is still not picking up on the package.22:54
ssbashok, how could I move the install location of my package to a dist-package?22:55
naccssbash: it's strange, i think the python plugin should be handling that for you22:55
ssbashIt's automatically moved to site-packages, since I use setup.py to build22:55
naccssbash: i wonder if the python2.7 site.py needs to be udpated22:56
cachiossbash, what do you have in the requirements.txt?22:56
naccssbash: again, not something you should have to do, either way22:56
ssbashI'm not sure what you mean by the python 2.7 site.py? I've specified the python plugin that this is a pyhton 3.5 project. Also here is my requirements.txt http://paste.ubuntu.com/24844809/22:58
cachiossbash, did you try creating a wrapper and setting up the pythonpath on there?23:02
ssbashlike a virtualenv? I'm confused what you mean by wrapper23:03
cachiossbash, I mean a bash script able to call the python module23:06
cachiossbash, let me take a look to the lib to see why it is givving that error23:06
naccor try to do that form the shell itself (spin up the interpreter from your snap shell)23:07
naccssbash: i just was thinking through how site-packages works normally23:07
ssbashcachio: this is the command wrapper that snapcraft created. #!/bin/sh export PATH="$SNAP/usr/sbin:$SNAP/usr/bin:$SNAP/sbin:$SNAP/bin:$PATH" export LD_LIBRARY_PATH="$LD_LIBRARY_PATH:$SNAP/lib:$SNAP/usr/lib:$SNAP/lib/x86_64-linux-gnu:$SNAP/usr/lib/x86_64-linux-gnu" export LD_LIBRARY_PATH="$SNAP/usr/lib/x86_64-linux-gnu:$LD_LIBRARY_PATH" export LD_LIBRARY_PATH=$SNAP_LIBRARY_PATH:$LD_LIBRARY_PATH exec "$SNAP/usr/bin/python3" "$SNAP/bin/wr23:10
ssbashwhoops http://paste.ubuntu.com/24844868/23:11
ssbashI dont see any mention of the python path in  the wrapper23:11
cachiossbash, no23:13
cachiossbash, did you check the project was correctly downloaded¡?23:13
cachiossbash, I am talking about wraticus23:14
ssbashoh, yes I've check that the entire package and sub packages have been downloaded in site-packages.23:14
cachiossbash, what I have done with good results is to create a wrapper script where I setup all the veriables that I need and also I make the call the to command23:17
cachiolet me check for an example23:17
cachiossbash, there is some doc on https://snapcraft.io/docs/build-snaps/metadata23:19
cachiosection Using wrappers23:19
cachiossbash, does it work for you?23:20
ssbashI'm still a bit confused. how could I specific the python path in the wrapper without hard coding a path?23:21
cachioyou can do, export PYTHONPATH=PYTHONPATH:blablabbla23:23
ssbashPYTHONPATH: $PYTHONPATH:$SNAP/lib/python3.5/site-packages would that be work?\23:23
cachioand then exec "$SNAP/usr/bin/python3" "$SNAP/bin/wraticus.py" "$@"23:23
ssbashok and then point the command to this new wrapper executable23:24
cachioyes23:24
ssbashok ill give it a try23:24
ssbashcachio I added the wrapper command. python still isnt able to find wraticus.23:47
cachiossbash, can you share the wrapper?23:59

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