/srv/irclogs.ubuntu.com/2017/07/13/#snappy.txt

liveuserHello. Is this the IRC channel for snapcraft?02:03
liveuserI want to edit a snap, but it appears to be read-only.02:03
liveuserThis is my first snap, how do I get around this? I need to edit a bash script included in the snap.02:03
liveuserThe internet says run it in "Classic" containment02:04
liveuserbut I have no idea where the snapcraft.yaml is located02:04
liveuserAlso, what happens to the files when I'm examining the hard drive from a live instance?02:04
liveuserI went into my /snap directory, found the program, and the directory is empty02:05
liveuserThis is a lot of effort just to edit a bash script. :c02:05
mz_I have installed snapd, snapcraft. Then installed snap package hello-word.But when run "hello-world.env",output "command not find",why?02:56
=== elfgoh_ is now known as elfgoh
mwhudsonmz_: you might need to log out and in again to get /snap/bin on $PATH?04:00
zygagood morning06:44
* zyga slowly tunes into Polish weather and timezone06:48
mupPR snapd#3578 closed: store: talk to api.snapcraft.io for purchases <Created by cjwatson> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3578>07:00
=== JanC_ is now known as JanC
davidcalleelopio: for you to have a look https://github.com/canonical-websites/tutorials.ubuntu.com/pull/28309:25
* zyga -> breakfast09:29
=== chihchun_afk is now known as chihchun
* Chipaca needs more coffee today09:58
mvopstolowski: hey, we do generate the snapctl cookies on startup now in master, right? I was doing the following: "snap run --shell hello && snapctl get foo" but I get an "invalid snap cookie requested" error. is this expected?10:00
pstolowskimvo, hey, yes, this change landed in master. snapd generates them if missing. so no, that's not expected10:04
mvopstolowski: ok, I dig a bit to see what is going on10:05
pstolowskimvo, i'm looking at this too, may have a few questions in a sec10:05
mvopstolowski: but the usage above should work, right? i.e. if I run inside the snap environment I should be able to use snapctl get?10:05
mvopstolowski: fwiw, I see snap-cookies in the state10:06
mvopstolowski: including one for "hello"10:06
=== matteo` is now known as matteo
mvopstolowski: meh, it maybe that I have not everything updated to master, looks my snap-confine is too old10:11
pstolowskimvo, hmm wait, the snapctl.. part is not run the context of the hello snap, is it?10:11
mvopstolowski: it should be, snap run --shell hello should give me the exact environment that "hello" would run in. but like I said, I tink my snap-confine is out-of-date10:12
pstolowskimvo, one other thing to check is if /var/lib/snapd/cookie/ has a cookie file for hello snap (both snap-cookies and /var/lib/snapd/cookie should be in sync)10:13
mupPR snapd#3549 closed: many: expose services commands for snap services <Created by chipaca> <Closed by chipaca> <https://github.com/snapcore/snapd/pull/3549>10:22
mupPR snapd#3552 closed: daemon: implement service commands <Created by chipaca> <Closed by chipaca> <https://github.com/snapcore/snapd/pull/3552>10:22
mupPR snapd#3554 closed: client: wrap services calls <Created by chipaca> <Closed by chipaca> <https://github.com/snapcore/snapd/pull/3554>10:22
mvopstolowski: yep, sorry for the noise, all is fine, I my system was not using master for everything10:24
pstolowskimvo, np, good to hear it's fine, thanks10:25
davidcalleSaviq: for you https://github.com/canonical-websites/developer.ubuntu.com/pull/30510:27
pstolowskidavidcalle, hi David!10:32
davidcallepstolowski: hey o/10:33
davidcalle(on my way to lunch, but I'm around)10:33
pstolowskidavidcalle, fyi, we have just landed support for install & remove hooks in snapd master. and some more hooks are in the pipeline and will probably land in next few weeks10:34
pstolowskidavidcalle, so we may need to update the docs10:34
davidcallebrilliant!10:36
Saviqdavidcalle, ack!10:37
mupPR snapd#3586 opened: client, daemon: rest api to reconfigure snapd with a custom store <Created by atomatt> <https://github.com/snapcore/snapd/pull/3586>11:59
* zyga -> break12:00
Son_Gokuwhat wait12:01
Son_Gokuniemeyer: what is snapd#3586?12:03
mupPR snapd#3586: client, daemon: rest api to reconfigure snapd with a custom store <Created by atomatt> <https://github.com/snapcore/snapd/pull/3586>12:03
Son_Gokudare I hope that this means arbitrary stores could be configured without patching and recompiling snapd?12:03
niemeyerSon_Goku: This is about hosting of local stores in environments that can't talk to the open net12:07
niemeyerIt's a sort of proxy pretty much, and changes the system altogether, so won't be very good at handling what you describe.. at least not yet12:08
Son_Gokuniemeyer: I see12:10
davidcallepstolowski: what are the other hooks?12:22
zygare12:22
davidcalle(in the pipeline, I mean)12:23
pstolowskidavidcalle, in the pipeline: refresh hook and interface hooks (executed when you 'snap connect ..' 2 snaps)12:24
davidcallety12:24
davidcallepstolowski: I'll send you a PR to look over the doc update (probably not before tuesday, though)12:25
pstolowskidavidcalle, sure. keep in mind this stuff just started landing and it will become available in next release(s)12:26
davidcallepstolowski: +112:27
pstolowskidavidcalle, so a few weeks away i suppose. nb, i'll be off for next 2 weeks12:27
davidcallepstolowski: I'll be off for three weeks in 10 days, but we'll manage something ;)12:29
ogra_three weeks!12:29
ogra_vive la france ? :)12:30
davidcalleAhaha12:30
Chipacaogra_: we get twenty something days too (but the custom here is to not take them all together)12:30
ogra_yeah12:31
* davidcalle raises his 2y and 3y old kids card12:32
zyga2017-07-13T14:46:32+02:00 ERROR cannot create alias symlink: symlink etcd.etcdctl /snap/bin/etcdctl: file exists12:54
zygahmmm12:54
* zyga happily switches to the interfaces branch13:27
zygaat least something I can bash and smash _and_ see a result13:28
cachiomvo, some change introducing in 2.26 is making tests fail when we do "systemctl stop snapd.service snapd.socket"13:29
cachiomvo, I'll research it13:29
zygamvo, niemeyer: question about the idea to use and release "RC"s rather than releases, what will the core snap version look like then?13:30
ogra_same as the final one, no ?13:30
ogra_after all the version doesnt get modified when core migrates from beta->candidate->stable13:31
mvocachio: thank you - where do you see that?13:31
niemeyerI don't have a strong opinion.. if we intend to release the same build assuming the candidate "tastes good" we'll need to preserve the final version on it though13:31
niemeyerWhat ogra_ said, basically13:31
cachiomvo, two builds failed on master13:31
ogra_i guess it would simply just be ahead one version bump13:31
zygayes13:31
zygathough that looks "less nice"13:32
cachiomvo, this is the last one https://travis-ci.org/snapcore/snapd/builds/253090762#L379613:32
ogra_while edge is jumping between git and release versions as needed13:32
zygamaybe we could alter the version in a snap revision assertion?13:32
zyga(purely presentation-wise)13:32
ogra_do you really think people look at it that much that it is worth it ?13:33
mvocachio: hm, "Job for snaps.service canceled" is a bit dry as an error message :/13:33
zygaogra_: not on core13:33
zygaogra_: but maybe, not sure13:33
ogra_i'd say also not on classic ... except when they run snap --version to copy/paste that into bugs :)13:33
cachiomvo, yes, it started yesterday and it happened on different tests13:33
cachiomvo, trying to reproduce it now13:34
* zyga tried to write "spans" and kept typing "snaps"14:46
Chipacazyga: snaps span pan-sass pans14:50
ogra_schnaps !14:52
Chipacaogra_: a bit early for me14:53
ogra_but helps to get things back in order ;)14:53
ogra_(or totally out of order ... a matter of volumes ... )14:54
Pharaoh_Atemso... snapd-glib 1.15 for fedora, thanks to robert-ancell:15:05
Pharaoh_Atem* F24: https://bodhi.fedoraproject.org/updates/FEDORA-2017-5d326072db15:05
Pharaoh_Atem* F25: https://bodhi.fedoraproject.org/updates/FEDORA-2017-f3da5ab6fc15:05
Pharaoh_Atem* F26: https://bodhi.fedoraproject.org/updates/FEDORA-2017-8ba37dcc1c15:05
=== chihchun is now known as chihchun_afk
* zyga dinner15:19
mvofwiw, I branched 2.27 now and will update the forum topic15:21
mvoif anyone absolutely needs something in 2.27 shout!15:21
Pharaoh_Atemmvo: do we have an upstream changelog for 2.27 yet?15:23
Pharaoh_AtemI figured I'd experiment with shipping an upstream-ish changelog entry in the package changelog15:23
mvoPharaoh_Atem: I will write a short (highlights) changelog into the forum soon, the auto-generated one is part of debian/changelog, I will push the branch in a little bit15:24
Pharaoh_Atemmvo: I was wondering if maybe we should also push a variant of that into the fedora spec15:24
mvoits based on current master, so the new install/remove hooks are part of it (thanks to pstolowski)15:24
mvoPharaoh_Atem: sure, I can do that, its mostly automatic, I will need to look at how changelogs work in spec files :)15:25
Pharaoh_Atemthey're pretty simple :)15:26
Pharaoh_AtemI'm guessing though that you probably don't have rpmdevtools on your computer, right?15:26
Pharaoh_Atemhttps://build.opensuse.org/package/show/home:Pharaoh_Atem:debrpm/rpmdevtools :)15:27
mvoPharaoh_Atem: apt list rpmdevtools gives me an empty list15:27
mvoPharaoh_Atem: how does http://paste.ubuntu.com/25082523/ look?15:29
mvoPharaoh_Atem: hm, not good at all I suppose, missing the date and revision, right?15:29
mvoeh version15:29
Pharaoh_Atemyeah15:29
Pharaoh_Atemyou're missing the header15:29
Pharaoh_Atemnote the entry below yours15:29
mvoPharaoh_Atem: yeah15:29
Pharaoh_Atemyou can also indent multiple levels15:30
Pharaoh_Atemchangelogs are mostly freeform15:30
mvoPharaoh_Atem: how do you guys represent "rc" versions. 2.27~rc1 ?15:30
Pharaoh_Atemofficially, Fedora uses 0.rc1.1 in the Release field15:30
Pharaoh_Atembut you can use ~rc115:30
Pharaoh_Atemthat's totally acceptable15:30
Pharaoh_Atemand in fact, I'm okay with that15:31
Pharaoh_Atemso just use 2.27~rc1 and leave the Release value at 0%{?dist}15:31
mvoPharaoh_Atem: so http://paste.ubuntu.com/25082538/ ?15:31
Pharaoh_Atemso the changelog entry version would be 2.27~rc1-015:31
Pharaoh_Atemlooks good15:31
Pharaoh_Atemthough I think you're the author :)15:31
mvoPharaoh_Atem: aha, so this is the author, not the uploader? sure, I can fix the field15:32
Pharaoh_Atemit can be either15:32
Pharaoh_Atembut I prefer author of change :)15:32
Pharaoh_Atembesides, I'll just have a 2.27-1 entry when I release into Fedora :)15:32
mvoPharaoh_Atem: sounds good15:32
zygare15:32
mvoPharaoh_Atem: thanks for your help! I15:32
Pharaoh_Atemnp15:33
Pharaoh_Atemthis is my particular experiment with having upstream changelogs along with maintainer ones15:33
* zyga hugs both Pharaoh_Atem and mvo 15:33
Pharaoh_Atemhistorically speaking, I've never been a fan of them, but enough people ask for them now that I'll see how it plays out15:33
mvoPharaoh_Atem: sure, I will keep this in mind (or try) - just let me know if the experiment does not pan out15:34
mvo(and you want the short maintainer ones back)15:34
Pharaoh_Atemmvo: My preference would be that you'd do the following structure for the entry15:35
Pharaoh_Atem* @DATE@ @AUTHOR@ - @VERSION@15:35
Pharaoh_Atem- New release15:35
Pharaoh_Atem  - Entry details15:35
Pharaoh_Atemyou can indent freeform just like you do with debian changelogs15:35
Pharaoh_Atemthe idea is to just make it clear15:35
Pharaoh_Atemand adjust the Version: tag to be the same as @VERSION@ :)15:35
Pharaoh_AtemI still plan on using the highlights for the updateinfo data used for bodhi updates15:36
mupPR snapd#3587 opened: Using udev tagging for snap interfaces <Created by adglkh> <https://github.com/snapcore/snapd/pull/3587>15:36
Pharaoh_Atemmvo: is that alright with you?15:37
mvoPharaoh_Atem: sure!15:38
Pharaoh_Atemwe can tweak as needed going forward15:38
Pharaoh_AtemI think that as long as the entries are prepared alongside, it'll go pretty well15:38
zygawa15:39
zyga:w15:39
mvoPharaoh_Atem: https://github.com/snapcore/snapd/blob/release/2.27/packaging/fedora/snapd.spec#L638 I pressume?15:39
mvozyga: :!15:39
Pharaoh_Atem:)15:39
Pharaoh_Atemperfect15:39
Pharaoh_Atemmvo: you can also adjust the version field: https://github.com/snapcore/snapd/blob/release/2.27/packaging/fedora/snapd.spec#L5115:39
Pharaoh_Atemunless you don't plan on tagging 2.27~rc315:40
Pharaoh_Atemmvo: could you also adjust the Version field?15:44
Pharaoh_Atemsomething I'm thinking of trying is having COPR track branches to offer pre-release builds from Git directly15:45
Pharaoh_Atemno biggie if you don't want to, though15:46
Pharaoh_AtemI'm can just track myself15:46
mvoPharaoh_Atem: aha, the version is not taken automatically from the changelog?15:52
Pharaoh_Atemmvo: nope15:52
Pharaoh_Atemin fact, you can leave the version off entirely and put it in the changelog text15:53
mvoPharaoh_Atem: ok, updating this15:53
Pharaoh_Atemsome people prefer that15:53
Pharaoh_Atemagain, I debate about it myself15:53
Pharaoh_Atemtechnically, RPM is actually only aware of @DATE@ @AUTHOR15:53
Pharaoh_Atemerr @AUTHOR@15:53
mvoPharaoh_Atem: hm, ok. no idea :) if I can leave it off, thats nice because it means less things to remember15:53
Pharaoh_Atemmvo: if you do that, please put it in the changelog text :)15:54
Pharaoh_Atemit helps so people know what it's associated with15:54
Pharaoh_Atemso you can do "New upstream release 2.27~rc3"15:54
Pharaoh_Atemwhich is probably okay for both Debian and Fedora changelogs15:54
mvoPharaoh_Atem: sounds good. updated. I think its ok to now upload this just yet to fedora I hope we have a real 2.27 early next week15:57
Pharaoh_Atemawesome15:57
Pharaoh_Atemmvo: sorry if I seem like I'm being picky, but I'm excited that with Fedora actually plugged into the CI (admittedly not doing a lot), it'd be nice if it was also bumped in a similar manner for releases15:58
Pharaoh_Atemmvo: hold on15:59
Pharaoh_Atemthis is wrong15:59
Pharaoh_Atemhttps://github.com/snapcore/snapd/commit/9610a9260e80cd059394d9ef5c729f5698064ef115:59
Pharaoh_Atemthe Version: does need to be filled in15:59
Pharaoh_AtemI meant that15:59
Pharaoh_Atem* Thu Jul 13 2017 Michael Vogt <mvo@ubuntu.com> [ - 2.27~rc3 ]<= not mandatory15:59
Pharaoh_AtemI misunderstood you16:00
Pharaoh_AtemVersion: is mandatory16:00
Pharaoh_Atemthe version in the changelog entry header is not16:00
mvoPharaoh_Atem: aha, ok16:00
Pharaoh_Atemrpm doesn't use that at all for anything16:00
mvoPharaoh_Atem: let me fix that16:00
Pharaoh_Atemmvo: sorry for that :)16:00
* Pharaoh_Atem sighs in relief16:02
Pharaoh_Atemmvo: awesome, now it's right16:02
mupPR snapcraft#1404 closed: cleanbuild: ensure we enter a shell on failures on --debug <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1404>16:03
=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
mupPR snapd#3588 opened: tests: fix how package lists are updated for opensuse and fedora <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/3588>16:35
zygaPharaoh_Atem: can you have a look at ^16:38
zygaPharaoh_Atem: it smells like that metadata-update you mentioned16:38
mupPR snapcraft#1357 closed: tests: do not break a systems `bzr whoami` <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1357>17:33
noise][FYI, we are having an issue with snap downloads, investigating17:51
pstolowskiniemeyer, wrt #3569 when writing the tests I found out that some of the unmarshall -> decoder+usenumber change were not actually needed, so I reverted them. moreover, I found that the changes to state.go are most likely not needed either - the end-to-end spread test execute without state.go changes proves that, however one unit test disagrees and I need to understand why, but probably it's a problem with how this unit test is implemented17:52
pstolowskiniemeyer, long story short.. the change may become 2x shorter and a single problematic unit test led me to some dead ends17:53
niemeyerpstolowski: Nice, that's good news I guess17:53
niemeyerpstolowski: Why are those cases not needed?17:54
niemeyerpstolowski: Without digging in much, it felt like a good thing that we were preserving the numbers as originally fed17:54
pstolowskiniemeyer, they apparently don't affect the data format (thanks to json.RawMessage I guess?). the critical part is snap-get/snapctl-get and config/transaction17:56
pstolowskiit's getting late here.. ttyt o/18:02
=== rumble is now known as grumble
Pharaoh_Atemmvo: what's this "profilese" you speak of? :P https://github.com/snapcore/snapd/commit/7cb1b27c546bce219114010fe9c8f4d0e6a4807119:11
davidcalleelopio: https://tutorials.ubuntu.com/tutorial/continuous-snap-delivery-from-travis-ci19:45
mwhudsonis chipaca on leave currently?22:14

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