/srv/irclogs.ubuntu.com/2017/08/25/#snappy.txt

=== JoshStrobl is now known as JoshStrobl|AFK
willdeberryany around around by chance?01:31
willdeberryanyone*01:31
=== JanC is now known as Guest55020
=== JanC_ is now known as JanC
mupPR snapd#3806 opened: packaging/fedora: Merge changes from Fedora Dist-Git <Created by Conan-Kudo> <https://github.com/snapcore/snapd/pull/3806>04:22
=== Justin` is now known as Justin`afk
mupPR snapd#3693 closed: snapstate: improve the error message when classic confinement is not supported <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3693>05:53
mupPR snapd#3806 closed: packaging/fedora: Merge changes from Fedora Dist-Git <Created by Conan-Kudo> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/3806>07:04
pedronis++ dnf -q -y --refresh install 'pkgconfig(systemd)'07:11
pedronisError: Failed to synchronize cache for repo 'updates'07:11
zyga-susekyrofa: re07:33
zyga-susekyrofa: not sure about bug tracking, I think keeping it all in snapd is easier though07:33
zyga-susegood morning everyone :-)07:33
mvozyga-suse_: hey, I think I addressed your points in 3260, let me know. I will ponder a bit more over the cmd_userd_test.go in the meantime07:39
zyga-suse_hey07:39
zyga-suse_thank you, let me look07:40
zyga-suse_I'm working on a blanket in the grass07:40
zyga-suse_I wonder if people will wonder what I do here ^_^07:40
zyga-suse_mvo: poor network connection, are you reading this?07:49
mvozyga-suse_: yes07:50
zyga-suse_odd, I can chat on IRC but I cannot load any website08:01
zyga-suse_loaded08:04
zyga-suse_mvo: replied on the Ping comment08:09
zyga-suse_mvo: that's the only part I really cared about08:09
zyga-suse_let me read the diff08:09
zyga-suse_mvo: reviewed08:13
zyga-suse_mvo: I'm sorry about being strict about that ping method there, let me know if that works in the end08:15
mvozyga-suse_: we are not implementing Ping() currently08:18
mvozyga-suse_: we are not implementing the Dbus.Peer interface at all08:18
zyga-suse_mvo: go dbus doesn't do it for free?08:18
zyga-suse_it's typically implemented by libraries08:18
mvozyga-suse_: unfortunately not08:18
zyga-suse_aww, bummer08:18
zyga-suse_that's not good08:18
mvozyga-suse_: at least not as far as I can tell08:18
mvozyga-suse_: d-feet is what i use08:18
zyga-suse_does it show up in introspection?08:18
zyga-suse_aha08:18
mvozyga-suse_: yes, introspection shows up08:18
zyga-suse_I mean, does Peer show up?08:19
mvozyga-suse_: no, peer does not show up at all, it does show up for the old safe launcher though (because dbus-gio)08:19
zyga-suse_right08:19
Chipacagood morning all! happy "all versions of go shipped in ubuntu are obsolete" day!08:20
Chipaca(1.9 is out)08:20
zyga-suse_oh, nice08:20
zyga-suse_anything shiny?08:20
zyga-suse_are generics in yet?08:20
Chipacazyga-suse_: type aliases are a thing08:21
Chipacazyga-suse_: and monotonic time08:21
zyga-suse_whee08:21
zyga-suse_so 2001 of us08:21
mvozyga-suse_: I look into it, maybe we are just holding^Wusing it wrong08:21
zyga-suse_mvo: maybe we could just implement Peer08:22
zyga-suse_it's no-op Ping08:22
zyga-suse_and get machine id08:22
Chipacazyga-suse_: the nicest change that would impact my work is that ./... no longer looks in vendor08:22
zyga-suse_yes, I saw that one08:23
zyga-suse_that's a nice change08:23
Chipacaso go test ./... would finally dtrt08:23
mvozyga-suse_: yeah, otoh, if we can call OpenURL and get a reply like "cannot use ping-ping-ping:" we know the interface is up as well. but yeah, ping is nicer, I have a look08:23
zyga-suse_Chipaca: I'm still holding out for generics08:24
zyga-suse_mvo: agreed08:24
zyga-suse_mvo: if you want +1, just move to libexecdir08:24
zyga-suse_mvo: I may look at Ping over weekend08:24
mvozyga-suse_: we are actually using the normal "snap" command for the userd subcommand, so it will be the regular bindir (unless I miss something)08:25
zyga-suse_mvo: ahh, sorry08:25
zyga-suse_I must have misread the diff08:25
mvozyga-suse_: no worries08:25
zyga-suse_I assumed it would be a new binary08:25
zyga-suse_ok, I have two tests to fix08:25
zyga-suse_and a lot of goodies will be ready08:26
zyga-suse_but first08:26
zyga-suse_coffee!!!08:26
mvozyga-suse_: heh, fun! if I export the ping interface things work08:27
mvozyga-suse_: oh well, I will update things08:27
Chipacazyga-suse_: did you see the guy that had generics implemented via unicode and generators?08:30
pedronismvo: there was no joy with the fedora tests btw, still failed in prepare in strange ways, are we tuning ip6 on fedora as well?08:34
zyga-suse_mvo: things work?08:35
zyga-suse_Chipaca: I think I don't actually want to know anymore :D08:35
zyga-suse_pedronis: probably not, it's behind ubuntu spread test flag08:35
pedronisanyway something is off and makes those even more unstable than usual08:37
Chipacazyga-suse_: bottom line: it looked like “type ImmutableTreeListᐸElementTᐳ struct {”08:37
pedronisor the fedora infra itself is flakier, no clue08:37
mvopedronis: I have not looked at the details for fedora unfortunately08:37
mvozyga-suse_: yeah, ping works once the interface is exported via xml, no code needed, I will update in a bit (working on tests right now)08:37
zyga-suse_woot!08:38
zyga-suse_nice :)08:38
zyga-suse_ah08:38
zyga-suse_right08:38
zyga-suse_because introspection is how dfeet knows about it08:38
zyga-suse_though odd that this is not done by gdbus itself08:38
mupPR snapd#2837 closed: interfaces/apparmor: allow reading from ecryptfs <Blocked> <Decaying> <Created by zyga> <Closed by chipaca> <https://github.com/snapcore/snapd/pull/2837>08:40
zyga-suse_thank you :'-(08:44
zyga-suse_though we _may_ have a way to solve that with what I'm working on08:44
mupPR snapd#3346 closed: [WIP] allow access to nm-dispatcher scripts <Decaying> <Created by felicianotech> <Closed by chipaca> <https://github.com/snapcore/snapd/pull/3346>08:45
Chipacahmm, i've got to test stuff that involves rebooting. I'll be pestering you with in/out messages soon :-)08:50
zyga-suse_Chipaca: VMs/08:51
Chipacazyga-suse_: i'm too close to my memory limit :-(08:52
Chipaca(this is a test that needs a full desktop)08:52
Chipacabrb08:52
zyga-suse_aww08:53
* zyga-suse_ has apparmor on opensuse :)08:53
zyga-suse_and we can now probably also enable reexec08:53
zyga-suse_there's still a few things missing but I think it's very very closenow08:53
zyga-suse_mvo: is the PR ban still in effect?08:54
Chipacamorphis_: zyga-suse_: so it looks like moving /etc/X11/Xsession.d/65snappy to e.g. /etc/profile.d/snappy.sh will DTRT in both X and Wayland (might need some tweaks, but basically it'll work)08:59
Chipacamorphis_: zyga-suse_: I know RH/fedora are slightly different wrt profile.d, but afaik it boils down to it getting loaded more, not less; can you confirm?09:00
zyga-suse_Chipaca: if you test it, +109:00
zyga-suse_Chipaca: I think we want to dedupe those files09:00
Chipacathis is me looking at snapd#339809:00
mupPR snapd#3398: env: set XDG_DATA_DIRS for wayland et.al <Created by sergiusens> <https://github.com/snapcore/snapd/pull/3398>09:00
zyga-suse_Chipaca: there's one in every distro packaging09:00
zyga-suse_Chipaca: and then we want to call it sanely, say snapd.sh09:00
zyga-suse_Chipaca: and keep it in data/09:01
Chipacazyga-suse_: i tested it on ubuntu, but as i said i don't have the space to test it on other things09:01
zyga-suse_Chipaca: I see09:01
Chipacabah. i might be able to get this fedora running09:01
Chipacashould work with 1G, right?09:01
zyga-suse_Chipaca: I can test on all of them but I'd prefer to do that in the afternoon with less sharp mind09:01
Chipacayeah no probs09:01
zyga-suse_Chipaca: nah09:01
zyga-suse_try 2G09:01
Chipacasigh, ok09:01
zyga-suse_how much ram do you have?09:01
Chipacalots of swapping ahead for me :-)09:01
zyga-suse_I have a _physical_ fedora box with 2GB and it's a pain09:01
zyga-suse_I added 1GB stick to make it work09:02
Chipacazyga-suse_: free right now? 1G09:02
zyga-suse_OMG09:02
zyga-suse_and in general?09:02
Chipacazyga-suse_: 4G total09:02
zyga-suse_darn09:02
zyga-suse_no way to expand?09:02
Chipacai had 8 but one of the sticks died09:02
Chipacaand the whole box is starting to die09:02
Chipacaso i'm just letting it09:02
zyga-suse_aha09:02
zyga-suse_thinkpad?09:02
Chipacawhile saving up for its replacement09:02
zyga-suse_man, buy t430 for the price of a good meal09:02
Chipacazyga-suse_: latitude e series09:02
zyga-suse_they are indestructible09:02
zyga-suse_and cost peanuts09:03
Chipacait's ~8 years old09:03
Chipacaif i were to buy the same thing, it'd cost peanuts too09:03
zyga-suse_what are you saving for?09:03
Chipacabut these old cpus suck 45W when angry, i'll gladly pay to worry about that less09:03
zyga-suse_so what if you can get a battery with 11hrs of life?09:03
Chipacazyga-suse_: probably a latitude e series :-)09:04
zyga-suse_and still buy three for the price of one new box09:04
Chipacazyga-suse_: 1509:04
Chipacazyga-suse_: this box had 15 hours battery life when it and the batteries were new09:04
* zyga-suse_ chooses not to buy new laptops again09:04
Chipacazyga-suse_: ah, it'd not be new, it'd be refurb09:04
zyga-suse_ah09:04
zyga-suse_+100 on that09:04
zyga-suse_it's worse than with used cars09:04
Chipacai can't in good conscience justify the price nor environmental cost of new09:05
zyga-suse_you open the door and 30% is gone09:05
zyga-suse_yep09:05
zyga-suse_especially when new CPUs suck so badly lately09:05
zyga-suse_with less and less reliablity and features09:05
zyga-suse_I wonder when amd will show their mobile ryzen/threadripper units09:06
* zyga-suse_ sees ants crawling over his keys09:08
zyga-suse_not ogod09:08
ogra_they probably just want to help09:09
* zyga-suse_ plays bug squashing09:10
zyga-suse_Chipaca: buy this and run !ubuntu distro on it http://www.ebay.es/itm/Lenovo-ThinkPad-T530-i5-3210M-2x2-5GHz-8GB-128GB-SSD-UMTS-NVS5400-Webcam-WIN10-/253112700548?hash=item3aeeb14e84:g:KsAAAOSw01dZnrBX09:11
zyga-suse_Chipaca: and work on it 2/5 days a week09:11
zyga-suse_snapd must feel great everywhere09:11
ogra_well, if you want low-end, get a beaglebone :P09:12
zyga-suse_ogra_: ahem09:12
zyga-suse_ogra_: that's way more than sufficient for work09:12
zyga-suse_ogra_: and it has twice the RAM that Chipaca has now09:12
zyga-suse_it would be good if we had someone running centos09:13
zyga-suse_or if feeling less bold, just run opensuse or fedora where there's very good support already09:13
zyga-suse_ogra_: what are you using daily?09:13
zyga-suse_(h/w wise)09:13
ogra_well, depends where i sit09:14
ogra_in my office i have a "Intel(R) Core(TM) i5-2500K CPU @ 3.30GHz" with 32G, 3TB raid, 500G SSD, triple head nvidia 97009:15
ogra_(+3x 22" LG HD monitors )09:15
ogra_in my living room my first gen XPS1309:15
ogra_(some early i7, i forgot which and 8GB)09:16
Chipacazyga-suse_: http://www.morgancomputers.co.uk/c/512/IBM-Lenovo/09:16
zyga-suse_nice desktop setup!09:16
ChipacaOTOH i coould set up my bbb09:17
Chipacai'd need bluetooth and wifi to be working on it though09:17
zyga-suse_punch-card speed IO09:17
ogra_hmm, then a pi3 rather ...09:17
Chipacai thought io on the pi3 was slower than the bbb09:17
ogra_(though thats pretty powerful for an arm board)09:17
zyga-suse_no, nothing is slower than bbb09:18
ogra_well, pi3 is quad core 1.2GHz ...09:18
zyga-suse_Chipaca: pi3 when using HDD is very much usable09:18
ogra_bbb is single 800MHz09:18
zyga-suse_Chipaca: just get that fancy Pi drive and boot away09:18
zyga-suse_no SD cards needed09:18
ogra_as long as you dont use any other USB devices on it, yeah09:18
zyga-suse_(or any HDD with a usb cable)09:18
zyga-suse_it's good enough for coding IMO09:19
ogra_wired anetwork and HDD at the same time can already get nasty09:19
ogra_the USB hub gets saturated easily09:19
zyga-suse_it's limited but for the price it's really good09:20
ogra_well ... for the price, yeah09:21
zyga-suse_mvo: trivia PR pre-reviewed by jdstrand https://github.com/snapcore/snapd/pull/380709:32
mupPR snapd#3807: cmd/snap-confine,packaging: import snapd-generated policy <Created by zyga> <https://github.com/snapcore/snapd/pull/3807>09:32
zyga-suse_I stated doing internal reviews with jamie so that we can start with a +1 and land things faster09:32
mupPR snapd#3807 opened: cmd/snap-confine,packaging: import snapd-generated policy <Created by zyga> <https://github.com/snapcore/snapd/pull/3807>09:33
ogra_hmm ...09:33
mupPR snapd#3791 closed: cmd/snap-confine: allow using additional libraries required by openSUSE <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3791>09:34
mupPR snapd#3792 closed: cmd/snap-confine: allow running snap-exec without confinement <Created by zyga> <Closed by zyga> <https://github.com/snapcore/snapd/pull/3792>09:34
ogra_why is my daily edge image installing core 2466 on first boot ?09:34
zyga-suse_2466 feels 1000 builds too old09:34
ogra_yes09:34
ogra_especially for an edge build09:35
ogra_2739 is the current core in armhf edge09:35
ogra_this smells like firstboot didnt set up the system properly09:36
ogra_:/09:36
zyga-suse_what's in /var/lib/snapd/snaps ?09:36
ogra_ogra@pi2:~$ snap list09:38
ogra_No snaps are installed yet. Try "snap install hello-world".09:38
ogra_ogra@pi2:~$09:38
ogra_grmpf09:38
ogra_ogra@pi2:~$ ls /var/lib/snapd/snaps/09:38
ogra_core_2466.snap  core_2739.snap  pi2-kernel_39.snap09:38
ogra_ogra@pi2:~$09:38
ogra_no gadget09:38
* ogra_ starts over with a fresh flash to capture the log of the first boto09:39
ogra_*boot09:39
zyga-suse_Chipaca: did you see the new "dep" tool?09:41
zyga-suse_ogra_: super odd09:41
Chipacazyga-suse_: in go?09:41
zyga-suse_ogra_: how did youget 2466? from ubuntu-image?09:41
zyga-suse_Chipaca: yes09:41
ogra_zyga-suse_, i didnt ..09:41
ogra_zyga-suse_, core_2739.snap09:42
zyga-suse_sure but 2466 is present there as well09:42
ogra_u-image did the right thing ... but the initial setup failed ... i usually install avahi as first thing after setting the hostname when testing ... that pulled core from stable09:42
zyga-suse_aha09:43
ogra_and this time it takes actually longer to get to "please press enter" ... last round this casme up immediately which indicates the key generation didnt take place09:46
ogra_ogra@localhost:~$ snap list09:47
ogra_Name        Version                   Rev   Developer  Notes09:47
ogra_core        16-2.27.4+git331.b672daf  2739  canonical  core09:47
ogra_pi2         16.04-0.18                39    canonical  gadget09:47
ogra_pi2-kernel  4.4.0.1070.70             39    canonical  kernel09:47
ogra_ogra@localhost:~$09:47
ogra_hrm ... so there was a race in the first run ... which is gone now :/09:47
ogra_i'm always surprised how different the memory usage between the different arm boards is09:51
ogra_on the nanopi-air htop shows 48.7MB used ... the pi2 with the same set of snaps installed and running uses 70.3MB09:52
mupPR snapd#3808 opened: apparmor,release: add better apparmor detection/mocking code <Created by zyga> <https://github.com/snapcore/snapd/pull/3808>09:53
zyga-suse_mvo: another small branch pre-approved with jdstrand09:53
ogra_(both armhf, both using the same core )09:53
mvozyga-suse_: ok, looking09:53
zyga-suse_mvo: I have a tiny branch on top that will let debian and suse to enable apparmor09:53
zyga-suse_mvo: and that will let us enable reexec on suse :)09:53
zyga-suse_mvo: the last holdout of reexec will be distro layout (fixed with layouts) and /snap location (not covered by layouts yet)09:54
ogra_fgimenez, ppisati, any idea why all our stable kernels are like 6 revisions behind ?09:56
* zyga-suse_ -> small break10:02
zyga-suse_everyone: please request my reviews on things you want me to look at10:02
fgimenezogra_: nope, i don't know10:03
pedronismvo: so the problem test seems a quirk of task without undo handlers, their change is ready "sooner", don't think it should block your branch, but I would like to explore not needed Overlord.Loop there10:26
mvopedronis: aha, I see! I can add an undo handler for now as a workaround until we have time to explore further. thanks a lot for looking into this10:37
Chipacamvo: if what's now /etc/profile.d/apps-bin-path.sh suddenly becomes /etc/profile.d/snapd.sh, does apt/dpkg/something freak out?10:49
mupPR snapd#3809 opened: tests: copy files with less verbosity <Created by zyga> <https://github.com/snapcore/snapd/pull/3809>10:53
zyga-suse_mvo: trivial for my modem woes ^10:53
zyga-suse_Chipaca: yes10:53
zyga-suse_you need to rm conffile10:53
zyga-suse_Chipaca: or in other words, you want to ship your computer to the arctic, move north and herd sheep10:58
Chipacazyga-suse_: the name just strikes me as odd, especially now that it'll have XDG stuff in it :-)10:59
Chipacabut they are technically both paths10:59
Chipacaso ¯\_(ツ)_/¯10:59
zyga-suse_Chipaca: yeah, the name is bad10:59
zyga-suse_Chipaca: I'd like to have just "snapd.sh", period10:59
Chipacathat's what i was proposing above10:59
zyga-suse_Chipaca: new systemd (we cannot assume it though) has nicer way to do this10:59
Chipacazyga-suse_: it does?11:00
zyga-suse_one sec11:00
Chipacaanyway, time for me to go run11:00
* Chipaca waits11:00
zyga-suse_let me pull the discussion with systemd devs11:00
mupPR snapd#3810 opened: interfaces/hooks: PlugData and SlotData wrappers <Created by stolowski> <https://github.com/snapcore/snapd/pull/3810>11:00
zyga-suse_from my blog ...11:00
zyga-suse_https://new.zygoon.pl/post/case-study-snapd-on-centos/11:01
zyga-suse_Chipaca: have a look there11:01
zyga-suse_there are links to a PR that has since landed in systemd: https://github.com/systemd/systemd/pull/513111:02
mupPR systemd/systemd#5131: Environment generators <pid1> <Created by keszybz> <Merged by poettering> <https://github.com/systemd/systemd/pull/5131>11:02
zyga-suse_and while looking there, enjoy 102 open PRs for systemd11:03
zyga-suse_(we are not doing bad)11:03
ogra_well11:03
zyga-suse_though 806 contributors is x10 more than we have11:03
ogra_from our downstram POV thats not acgtually much different11:03
ogra_(environment.d vs profile.d ... )11:04
ogra_you still need to ship the snippet that assembles the variable11:04
ogra_just in another place11:04
Son_Gokuenvironment.d happens earlier than profile.d11:04
ogra_sure sure11:04
Son_Gokuthough I hate putting things in either, really11:04
ogra_just saying that iit doesnt matter much regarding the naming of the fil ;)11:05
ogra_*fiile11:05
ogra_sigh ...11:05
Son_Gokuthat's okay, one day you'll get it :)11:05
ogra_my laptop keyboard slowly gives up11:05
Son_Gokuzyga-suse_, we'll probably not have anywhere close to that number of contributors11:06
zyga-suse_Son_Goku: as I said, x10 less11:06
zyga-suse_Son_Goku: good to see you11:06
zyga-suse_Son_Goku: did you see my reply on the forum there?11:06
Son_Gokuthat would require rejiggering the project in a way that currently isn't possible :(11:06
Son_Gokuthe forum makes me angry, so I haven't looked since I posted yesterday11:07
* zyga-suse_ hugs Son_Goku again11:07
* Son_Goku sighs11:08
mcphailkyrofa: thanks for the 11.0.4 nextcloud update. Is 12.x on the roadmap? (I'm not complaing, btw. I don't even know what version 12 brings. Just curious!)11:20
zyga-suse_ogra_: what does it take to add /var/lib/snapd/apparmor/snap-confine.d to the core snap?11:33
ogra_two lines ?11:33
ogra_(assuming you want it writable, else one line)11:33
zyga-suse_yes, two lines11:33
zyga-suse_er11:33
zyga-suse_yes writable11:33
zyga-suse_where?11:33
ogra_heh11:33
zyga-suse_note that it's under /var/lib/snapd/apparmor which was writable before11:34
zyga-suse_does it need any work?11:34
zyga-suse_or do I just need to fix CI prep to mkdir that too11:34
ogra_zyga-suse_, https://github.com/snapcore/core/blob/master/live-build/hooks/20-extra-files.chroot for the mkdir -p ...11:34
zyga-suse_why there's no /var/lib/snapd/apparmor there/11:35
ogra_zyga-suse_, https://github.com/snapcore/core-build/blob/master/config/etc/system-image/writable-paths to make it writable11:35
zyga-suse_-> /var/lib/snapd auto persistent transition none11:35
zyga-suse_that's already present, so I think I'm good11:35
ogra_well, you want a mkdir somewhere ... if snap-confine creates it on startup or package install that should be sufficient11:36
ogra_else you need it in some of the build scripts like above11:36
zyga-suse_ogra_: the snapd package has that directory11:36
ogra_ok11:37
zyga-suse_ogra_: it's made and packaged in the deb11:37
mupPR snapd#3809 closed: tests: copy files with less verbosity <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3809>11:39
zyga-suse_mvo: thank you11:40
zyga-suse_mvo: please help me out with one thing11:40
zyga-suse_mvo: when I want to add a new directory under /var/lib/snapd/ (specifically /var/lib/snapd/apparmor/snap-confine.d) should I do anything more than just update packaging?11:40
zyga-suse_mvo: tests are failing and I assume it is because of this11:41
zyga-suse_mvo: I have a patch that looks like :11:41
zyga-suse_https://paste.gnome.org/pkhbjpoju11:41
zyga-suse_mvo: but I'm unsure if that will really hide the problem until release11:42
ogra_hmm11:43
mvozyga-suse_: lets get to the bottom of this, the dir in the package should be enough, let me try to reproduce via spread11:43
ogra_you might have a problem if you need it on non-new systems ...11:44
ogra_since /var/lib/snapd is "transition" not synced11:44
zyga-suse_can you explain?11:44
ogra_transition only copies the dirs on the first run ...11:45
ogra_from core to writable11:45
zyga-suse_"transition"?11:45
zyga-suse_aha11:45
zyga-suse_I see11:45
ogra_so an upgraded ubuntu-image install wont simply add it11:45
zyga-suse_well11:45
zyga-suse_suggestions?11:45
ogra_systemd job ... or switch to "synced" but i'm not 100% sure about the latter11:46
ogra_(have to check the code)11:46
willdeberrywhen making a change to an interface, is building snapd cmd sufficient to pick up the changes? `go build -o /tmp/snapd github.com/snapcore/snapd/cmd/snapd`11:46
zyga-suse_ugh11:46
zyga-suse_willdeberry: yes, you need to re-start it though11:46
zyga-suse_willcooke: which backends are you changing11:46
zyga-suse_willdeberry: some things are not refreshed on startup11:47
zyga-suse_willcooke: (sorry, bad tab completion)11:47
ogra_zyga-suse_, for refrence http://manpages.ubuntu.com/manpages/xenial/man5/writable-paths.5.html11:47
zyga-suse_ogra_: I just need an empty directory11:48
zyga-suse_but it _must_ be there11:48
zyga-suse_ogra_: and it may need to be there very early11:48
zyga-suse_ogra_: before apparmor profiles are loaded11:48
zyga-suse_and I think those are done before regular systemd units start11:49
ogra_well, given apparmor itself relies on writable-paths, that should be a given11:49
ogra_the issue is transition vs synced11:49
ogra_     WARNING:  This is a one-off operation which requires that the11:49
ogra_                 source  directory  on  the  writable  partition   not   exist11:49
ogra_                 initially: if this condition is satisfied, the directory will11:49
ogra_                 then be created and the data moved on  first  boot.  Although11:49
ogra_                 the  mountpoint  will be writable, note that subsequent boots11:49
ogra_                 will ignore any new files appearing or  disappearing  in  the11:49
ogra_                 original  read-only  rootfs  location  unless  you  perform a11:49
ogra_                 factory reset.11:49
ogra_(thats "transition" )11:49
ogra_note that subsequent boots11:50
ogra_                 will ignore any new files appearing11:50
ogra_that one specifically11:50
* zyga-suse_ feels like not wanting to fight this fight on friday11:51
willdeberryzyga-suse_: still fighting on adding bluez interface to classic OS11:51
zyga-suse_but this is something he needs :/11:51
zyga-suse_willdeberry: which backends are you using in that interface?11:51
ogra_we might need to switch /var/lib/snapd to be "synced" to make your dir appear ... i'm just not sure what happens with existing files in the target dir11:51
ogra_alternatively a systemd unit that runs mkdir ...11:52
zyga-suse_yeah, I don't like that really :/11:52
ogra_which is surely the uglier option11:52
willdeberryzyga-suse_: i am not sure i am following. I am just needing to expose bluez as an interface on classic so my snap can connect to bluez as a plug11:52
zyga-suse_willdeberry: the bluez interface won't work as-is on classic11:53
zyga-suse_willdeberry: can you pastebin your diff from master please11:53
willdeberryhttps://github.com/willdeberry/snapd/commit/11001ff172e7ddc96f6109a0c8ff9b3e63595ad411:53
ogra_zyga-suse_, why wouldnt bluez work as-iis on classic ? the files, devices and services it needs to access should be the same11:55
zyga-suse_ogra_: look at the diff11:55
zyga-suse_willdeberry: the diff look ok actually11:55
zyga-suse_willdeberry: can you please test both variants in each backend, ensuring that we don't get unexpected things?11:56
zyga-suse_ogra_: the confinement is very specific11:56
ogra_zyga-suse_, yeah, i meant more the interface itself ... not the conditions that make it available11:56
zyga-suse_ogra_: down to the point where peer is confined with a specific label11:56
zyga-suse_ogra_: and on classic that would be "unconfined"11:56
ogra_ah11:56
zyga-suse_ogra_: but the patch handles that so it looks correct11:56
ogra_yeah11:57
willdeberryzyga-suse_: right now, the bluez tests pass. the only tests i haven't fixed yet are the basedefinition tests which run when building the deb11:57
zyga-suse_willdeberry: they are in ../policy11:57
zyga-suse_just run them11:57
willdeberryk11:57
zyga-suse_(and fix)11:57
ogra_zyga-suse_, btw, do we have any mechanism for interfaces to check that the backend they allow access to is actually working/existing ?11:58
zyga-suse_ogra_: yes, implicitly11:58
ogra_imho bluez should be hidden on classic server installs that dont have BT installed/enabled11:58
zyga-suse_ogra_: interface methods specific to a backend are not used if the backend is not loaded11:58
ogra_ok11:59
willdeberrybackend meaning the bluez service in regards to classic right?11:59
zyga-suse_ogra_: we don't do that distinction yet, I think it falls under hotplug a little, we should do it but it's fine to just have it, even if unsupplied11:59
zyga-suse_willdeberry: no, backend meaning one of the security backends in snapd11:59
willdeberrygotcha11:59
ogra_yeah, i meant the service backend actually11:59
willdeberrytrying to map terminiology in my brain :)11:59
* zyga-suse_ feels grumpy now12:02
=== zyga-suse_ is now known as zyga-suse
zyga-susepstolowski: review on 381012:13
pstolowskizyga-suse, thanks12:15
zyga-susepstolowski: if you want we can discuss the design here12:15
zyga-susepstolowski: (standup may be hard with my data)12:15
pstolowskizyga-suse, I'll answer to comments on the PR first12:16
zyga-susesure, thank you12:18
zyga-susejdstrand: reviewed https://github.com/snapcore/snapd/pull/380512:25
mupPR snapd#3805: interfaces/default,account-control: don't hardcode uid and gid. Use username and group instead <Created by jdstrand> <https://github.com/snapcore/snapd/pull/3805>12:25
zyga-suseer. make that 380412:26
willdeberryzyga-suse: just an fyi, this is what is currently failing and I will address before reach back out https://pastebin.com/8BdemMUD12:29
zyga-susejdstrand: reviewed both now, I think we have a major problem to solve before this can land12:29
zyga-susewilldeberry: yeah, just adjust those tests12:30
pstolowskizyga-suse, replied. take a look at one of the interfaces in #3120 to see how this is used (best to look at an interface that uses some attributes)12:31
pstolowskizyga-suse, i'll be avail in ~10 minutes if you want to discuss in HO12:32
zyga-susepstolowski: not sure if my data plan will allow it :)12:32
zyga-susepstolowski: replied12:32
zyga-suseI'll brew some fresh coffee and I'll return to my woes12:32
* genii 's ears perk up for a moment at the mention of coffee12:34
Chipacaogra_: do we support this guy? http://beagleboard.org/black-wireless12:34
zyga-susemvo: interesting https://paste.gnome.org/pu8sxtwdg12:35
mupPR snapd#3761 closed: Disable reexec on debian <Created by mwhudson> <Closed by zyga> <https://github.com/snapcore/snapd/pull/3761>12:35
ogra_Chipaca, well, the bbb image should boot and run but no idea about the changed network bits12:36
zyga-susesaw that when 2017-08-25 11:51:20 Error executing autopkgtest:ubuntu-17.10-amd64:tests/main/config-versions :  failed12:36
zyga-susemwhudson: ^ maybe interested in this12:36
zyga-suseruntime: address space conflict: map(0xc420100000) = 0x7f377c67700012:36
zyga-susefatal error: runtime: address space conflict12:36
jdstrandzyga-suse: thanks, I've commented in both 3804 and 380512:37
zyga-susejdstrand: thank you12:37
zyga-susejdstrand: offtopic, the /var/lib/snapd/apparmor/snap-confine.d is harder than anticiapted12:37
zyga-susejdstrand: is there a way for apparmor-parser to ignore missing directories somehow?12:37
jdstrandzyga-suse: not without code changes. I doubt they would be upstreamable12:38
ogra_Chipaca, i see we have /boot/uboot/linux-generic-bbb*/dtbs/am335x-boneblack-wireless.dtb  .... so might need an adjusted own gadget12:39
ogra_so yes, we can easily support it :)12:39
jdstrandzyga-suse: why is it difficult? I saw something about core. we create directories in core all the time...12:39
Chipacaogra_: maybe i should get one and use it as my laptop at the sprint12:39
zyga-susejdstrand: replied12:39
zyga-susejdstrand: apparently not all the time12:39
zyga-susejdstrand: I need to think but merely adding the directory to snapd.deb is not enough12:40
zyga-susejdstrand: (replied https://github.com/snapcore/snapd/pull/3805#discussion_r135248529 in case you want to see)12:40
mupPR snapd#3805: interfaces/default,account-control: don't hardcode uid and gid. Use username and group instead <Created by jdstrand> <https://github.com/snapcore/snapd/pull/3805>12:40
jdstrandzyga-suse: going back to 3805> in a strict or devmode snap on core, the shadow gid matches (fine). on classic, /etc is one of the directories we bind mount from the host into the mount namespace. *not* /etc from core. therefore, the /etc/shadow in the mount namespace is /etc/shadow from the host12:42
jdstrandzyga-suse: are you saying something changed in this regard?12:42
zyga-susejdstrand: ah, you are right12:42
zyga-susejdstrand: but files from core snap will be owned by 42, not by 1512:42
zyga-susejdstrand: this is a separate issue12:43
jdstrandzyga-suse: so?12:43
zyga-susejdstrand: but still true12:43
jdstrandthe snap never sees them12:43
zyga-susejdstrand: shadow is maybe not the best example, I bet there are users/groups that _can_ be seen that don't match host's /etc/12:43
jdstrandthe snap with account-control plugged will only modify the host. it can't get to the core snap12:43
jdstrandzyga-suse: no12:43
jdstrandzyga-suse: the have the same databases12:44
jdstrandthey*12:44
jdstrandbecause we intentionally bind mount so it is that way12:44
zyga-susejdstrand: my point is that the core snap has hardcoded values for each file12:44
zyga-susejdstrand: right in the squashfs12:44
jdstrandzyga-suse: again, so?12:44
jdstrandwe bind mount over them12:44
zyga-susejdstrand: and any disagreements with the host will be confusing12:44
jdstrandit doesn't matter12:44
jdstrandthis is precisely why we bind mount12:45
zyga-susejdstrand: are all non-root files in the core hidden?12:45
jdstrandso they are the same. no disagreements12:45
jdstrandzyga-suse: of course not, but that is a different issue12:45
zyga-suseok12:45
jdstrandzyga-suse: we aren't expsoing those files to the classic snap12:45
zyga-suseI agree about your reasoning on snap-seccomp now12:46
jdstrandcertainly not for chown12:46
jdstrandsince those files are readonly anyway12:46
ikeysnaps are swiftly becoming a pain in my ass, just wanted to share that.12:47
ikeyhttps://dev.solus-project.com/T439012:48
* zyga-suse looks12:48
jdstrandzyga-suse: this functionality isn't about weird esoteric permissions in the core snap vs classic. this is about 2 users and groups that are defined by the LSB to exist everywhere (root and daemon (daemon in a followup PR)) and shadow, which in practice exists everywhere (but we could adjust as needed (if NonCompliantDistro ; use othershadow)12:48
zyga-suseikey: more classic snaps :/12:48
jdstrandzyga-suse: after that it is all about snapd-managed users12:49
ikey"Yay"12:49
zyga-suseikey: we need a way to pester snap makers and improve snapcraft12:49
jdstrandI'll mention that in 380412:49
ikeyya because id been down this ABI path with Ubuntu before12:49
ikeyit's called Steam.12:49
ikeyand frankly it aint worth it12:49
jdstrandactually, there is the calling user too12:49
zyga-susejdstrand: calling user?12:50
jdstrandsudo foo, we may want to allow dropping to 100012:50
jdstrandI need to think about that one12:51
jdstrandmay not allow that. again, I'll think about it12:51
jdstrandthat might be a straight snap-confine (not snap-seccomp) change12:52
* zyga-suse really breaks now and goes to make that coffee12:52
jdstrandoh, we don't need Lookup and LookupGroup for that anyway12:52
* genii hears something about coffee again, goes and gets one12:53
ogra_ikey, "snap run --shell atom" ... then grep LD_LIBRARY_PATH from the env ... i bet it doesnt prefix it correctly with the in-snap lib paths12:56
zyga-suseogra_: that's not sufficient to test12:58
zyga-suseogra_: the wrapper may set that12:59
zyga-suseogra_: and --shell will not run it12:59
ogra_zyga-suse, and thats fine12:59
ogra_oh12:59
ogra_i didnt know that12:59
zyga-suseogra_: it's still a bug in the snap12:59
ogra_yes12:59
ogra_well12:59
ogra_https://forum.snapcraft.io/t/libraries-not-found-in-classic-snaps/103312:59
ogra_zyga-suse, i consider is a snapd/snap-confine bug TBH13:00
zyga-suseogra_: I don't13:00
zyga-suseogra_: I gave my reasoning why this cannot be set in snap-confine13:00
ogra_the env should be properly prefixed without needing a wrapper13:00
ogra_ikey, so close that bug and make them complain to the snap creator ;)13:01
ogra_flexiondotorg, /snap/atom/current/bin/electron-launch is missing paths for $SNAP/lib and $SNAP/lib/$TRIPLET ...13:07
ogra_(iirc atom was your snap)13:07
ogra_ikey, ^^^13:07
ogra_(it only defines $SNAP/usr/lib ... )13:07
jdstrandzyga-suse: fyi, https://github.com/snapcore/snapd/pull/3804#discussion_r13525423613:10
mupPR snapd#3804: cmd/snap-seccomp: support parsing 'u:' and 'g:' for username and groups <Created by jdstrand> <https://github.com/snapcore/snapd/pull/3804>13:10
Son_Gokuzyga-suse, I think at some point we need a way to deal with the lack of extrausers outside of Ubuntu13:16
zyga-suseSon_Goku: aha, is that breaking in the field somehow?13:16
zyga-suseI thought that was used in the core only13:16
jdstrandmvo: regarding 'restore' in the wayland spread test, that is what I initially tried (see the comment in the test: "# If this is in 'restore', it hangs the test")13:17
Son_Gokuwell, weird things are happening already because snaps like docker snap don't work right without bounding back and forth13:17
ogra_flexiondotorg, https://github.com/snapcrafters/atom/pull/3 for you13:17
mupPR snapcrafters/atom#3: add $SNAP/lib, $SNAP/lib/$TRIPLET to library path <Created by ogra1> <https://github.com/snapcrafters/atom/pull/3>13:17
zyga-susesnapcraft should offer easy-to-use knobs for snaps using classic confinement *and* binary packages and don't expect to run host programs13:19
zyga-suseand should set LD_LIBRARY_PATH13:19
zyga-susesergiusens: ^^13:19
jdstrandmvo: I think this has to do with backgrounding the daemon with '&'. execute would never return and the test would have to timeout before getting to restore13:19
sergiusenszyga-suse: no, the correct path is to compile13:19
zyga-suseI know13:19
zyga-susebut people shoot themselves in the foot repeatedly13:19
zyga-suseso let's add some aribags13:20
sergiusensif we set LD_LIBARARY_PATH we taint the environment by default13:20
zyga-suse*airbags13:20
zyga-suseI wouldn't set it by default13:20
zyga-susebut I would add a knob for classic confinement snaps13:20
zyga-suseand warn people if they didn't make any decision about it13:20
sergiusensif you do classic, you need to compile your runtime, period13:20
jdstrandmvo: now, I can try to use something like start-stop-daemon, but I didn't go that route because it isn't going to exist outside of Debian derived distros13:20
jdstrandmvo: granted, the test itself is only running on Ubuntu, but figured for futureproofing, did it this way13:20
zyga-susesergiusens: should store reject snaps that dont?13:21
jdstrandmvo: but, what I could do is also have it in restore13:21
sergiusenszyga-suse: we shouldn't block people that know what they are doing13:21
zyga-susesergiusens: yes but apparently people don't know13:21
jdstrandmvo: ok, let me do that13:21
jdstrandmvo: thanks for the chat :)13:21
sergiusenssetting LD_LIBRARY_PATH is valid for some snaps that would never shell out13:21
sergiusenslook at all the trouble I went through to get snapcraft to be classic. It is a lot more work, there are beneifts, but all these apps need to use a compiled electron and not the default build13:22
zyga-susejdstrand: can you please look at and +1 https://github.com/snapcore/snapd/pull/3808 (you looked at the patches already)13:25
mupPR snapd#3808: apparmor,release: add better apparmor detection/mocking code <Created by zyga> <https://github.com/snapcore/snapd/pull/3808>13:25
ogra_sergiusens, how does compiled vs non-compiled have any effect here ?13:26
ogra_sergiusens, $SNAP/lib is simply not in LD_LIBRARY_PATH so shipped libs are not found ... that has nothing to do with the electron binary at all13:26
mvojdstrand: hey, sorry, was in the standup13:27
mvojdstrand: but it seems like you solved everything :)13:27
sergiusensogra_: RPATH would be set for the binary13:28
sergiusensthe whole concept of classic confined snaps is that you fix the linker and what libraries can load (and disabling LD_LIBRARY_PATH)13:29
ogra_is it ? ... it doesnt compllain about missing $SNAP/usr/lib13:29
ogra_(about files from there)13:29
sergiusenshuh?13:29
ogra_only about the lib path that is missig from the launch wrapper13:30
* sergiusens needs more words to make sense of that sentence13:30
ogra_sergiusens, https://github.com/snapcrafters/atom/pull/313:30
mupPR snapcrafters/atom#3: add $SNAP/lib, $SNAP/lib/$TRIPLET to library path <Created by ogra1> <https://github.com/snapcrafters/atom/pull/3>13:30
sergiusenswhen we introduced classic we said it was supposed to be used by people who compile from source; we never made a promise about anything else13:31
jdstrandzyga-suse: sure13:31
ogra_sergiusens, starting the snap on a non-16.04 system simply fails because it doesnt find libs from $SNAP/lib13:31
sergiusensbecause atom (through electron) is not compiled in snapcraft13:31
ogra_sergiusens, nothing complains about any libs from $SNAP/usr/lib (which is inn LD_LIBRARY_PATH already13:31
sergiusensyes there are workarounds13:31
ogra_right13:31
sergiusenslike LD_LIBRARY_PATH13:31
ogra_so we should have a "classic-wrapper" part ...13:31
sergiusensbut, if you shell out, you get those libs picked first intead of those on the host13:31
ogra_that simply ships such a wrapper13:32
ogra_yes, thats what you want13:32
sergiusensno, no wrappers for classic was what we discussed in the forum months ago13:32
ogra_you never want to ever use any libs from the host13:32
sergiusensyes you do!13:32
sergiusense.g.; asciinema13:32
ogra_then hell breaks loose13:32
ogra_you want to be able to **fall back* to them13:32
sergiusense.g.; integrated shell in vscode (you want to use the python and pip on the system)13:33
ogra_but if you ship any lib in your snap you want that one to be used and preferred13:33
sergiusenstest runner tools13:33
sergiusensanything useful, you want from the host13:33
ogra_yes, you want to carefully pick what you ship inside13:33
sergiusensfor the application, yes, but not for what you invoke outside of the system13:33
ogra_but still13:33
sergiusensthe only sane thing here is rpath13:33
sergiusenseverything else is broken13:33
ogra_*if* you ship it it needs to be the preferred lib13:33
sergiusensand you are on your own should you choose to go down that path13:34
ogra_else classic will never function anywhere but 16.0413:34
sergiusensogra_: compile and it works13:34
* sergiusens is not discussing this anymore, seems to come back every two months13:34
ogra_well, wrap and it works too :P13:35
sergiusenshalf works13:35
sergiusensjust wrappers and it doesn't work on trusty, try the asciinema in the store there and try to run something python and subsequently upload the video ;-)13:35
* sergiusens reboots13:36
ogra_Chipaca, " ps eww of the toplevel process" ... is that Xorg ? gdm/lightdm ? systemd --user ? or bash ?13:43
* ogra_ bets on the latter13:43
* zyga-suse breaks for an hour13:55
mvodoes anyone know if there is a way to skip an entire suite with gopkg.in/check.v1 ?14:05
mvoI mean, in SetupSuite(c *C) skip everything if e.g. a required binary is not available14:05
pedronismvo: I think c.Skip in the SetUpSuite does that14:06
pedroniswe even use it already I think14:06
pedronismvo: see for example  asserts/gpgkeypairmgr_test.g14:06
pedroniso14:07
mvopedronis: aha, it will still run teardownsuite, this is what confused me14:07
mvopedronis: thanks!14:07
Chipacaogra_: not bash14:08
Chipacaogra_: ps fx, look for the topmost thing in the session (here it's upstart)14:08
Chipacaogra_: for jwm it was jwm itself14:08
jdstrandmvo: ok, comments addressed in https://github.com/snapcore/snapd/pull/375914:12
mupPR snapd#3759: add spread test for wayland <Created by jdstrand> <https://github.com/snapcore/snapd/pull/3759>14:12
Chipacaogra_: any ideas on how to find a case where it doesn't work?14:13
ogra_Chipaca, to exec ps you need a shell ... and even /dash reads profile14:13
Chipacaogra_: yes, but i'm looking at the environment of a different process14:14
ogra_Chipaca, dunno if there is some gtk test app that prints the env ... i guess that'd be a question for the desktop team ...14:18
mvojdstrand: thanks! approved, but one more nitpick/question related to the source of test-snapd-wayland14:20
ogra_flexiondotorg, seems the build badge markdown in README.md for atom points to https://build.snapcraft.io/user/snapcrafters/discord ... i guess you want to fix that to point to https://build.snapcraft.io/user/snapcrafters/atom14:27
flexiondotorgogra_ ty Mr. popey is on it.14:28
ogra_heh, ok14:28
ogra_bah, it is odd that a change to README.md causes a rebuild of the snap ...14:31
ogra_we should have something like .snapbuild-ignore so that build.snapcraft.io doesnt pick up such changes14:31
Chipacaogra_: well... if I were to build stuff using snapcraft in a repo, I'd use chunks of my README in the snap's description14:34
ogra_Chipaca, yeah, if you do that you want README changes to be used ...14:36
ogra_https://forum.snapcraft.io/t/build-snapcraft-io-should-not-always-rebuild/1850/1 ...14:36
ogra_but a .snapignore file could make that flexible14:36
Chipacaman, my xbill snap doesn't have a .desktop file14:38
ogra_heh14:38
Chipacai need to fix this post haste14:38
ogra_complain at microsoft :P14:38
Chipacaworse, i don't know where i built it14:39
ogra_ikey, if you feel like you can tell your users to "snap refresh atom --edge" to test the fix ;) (and to show off how quick snaps can be fixed ;) )14:41
* ogra_ hugs ikey ... the IRC-> bugtracker bot :)14:43
ikeylol14:43
kyrofamcphail, oh definitely, v12 is quite simply not a good experience in the snap right now, which is why we haven't updated stable. Take a look at https://github.com/nextcloud/nextcloud-snap/pull/33414:46
mupPR nextcloud/nextcloud-snap#334: nextcloud: update to v12.0.1 <Created by pachulo> <https://github.com/nextcloud/nextcloud-snap/pull/334>14:46
ogra_pfft ... icons ... who needs them anyway :P14:48
jdstrandmvo: https://github.com/snapcore/snapd/pull/3759#discussion_r13527593614:53
mupPR snapd#3759: add spread test for wayland <Created by jdstrand> <https://github.com/snapcore/snapd/pull/3759>14:53
mcphailkyrofa: thanks!14:59
mvozyga-suse: some ideas for 3808, sorry that its a bit long, happy to discuss more if you want15:01
kyrofazyga-suse, this is for you: https://bugs.launchpad.net/snapd/+bug/171293015:07
mupBug #1712930: snap-confine: mounts happen in the wrong order <snapd:New> <https://launchpad.net/bugs/1712930>15:07
ogra_just hold it upside down15:08
mvopstolowski: the error in 3642 looks real, there is a unit test failure in transaction_test.go:168 - could you please have a look?15:12
niemeyero/15:12
pstolowskimvo, sure15:13
niemeyerWill get a quick lunch and be here shortly15:13
* Pharaoh_Atem sighs15:13
Chipacaxbill-xaw now has a desktop file, and it works and is light enough i can ask people to test stuff with it :-D15:13
* Chipaca tries in his fedora with wayland, first15:14
* Chipaca braces for thrashing15:14
zyga-susere15:17
zyga-susemvo: thank you, will look at 380815:17
mvojdstrand: 3759 looks ready, I will merge when tests are green15:18
zyga-susekyrofa: looking, interesting from the title alone15:18
Chipacaogra_: hah! dang, i just wasted some time. the rest of the world is _already_ doing this stuff from profile.d15:19
=== cachio is now known as cachio_lunch
jdstrandmvo: thanks! :)15:21
zyga-susemvo: nice, I'll iterate after dinner15:22
zyga-susekyrofa: hey, have you tried 2.27?15:22
zyga-susekyrofa: I'm pretty sure we handle / which isn't MS_SHARED up front15:22
zyga-susekyrofa: reading the rest of the analysis now15:23
kyrofazyga-suse, no, just latest stable. Happy to try though-- which channel?15:23
zyga-susekyrofa: not sure if channels work now15:23
zyga-susekyrofa: our release-breaks-edge process15:23
zyga-susekyrofa: maybe proposed 2.27.4 would be nice15:24
zyga-susebut maybe not15:24
zyga-suselet me finish reading stgraber's analysis15:24
zyga-susehmm, forum logged me out15:25
zyga-suseand ff doesn't know my password, what?15:25
zyga-suseah15:25
zyga-susethis is not forum.snapcraft.io15:25
ogra_Chipaca, well, debian and ubuntu arent unless that changed recently15:28
Chipacaogra_: correct15:29
Chipacaogra_: but, it works15:29
ogra_well, good then :)15:29
* ogra_ is afk for a bit15:29
mupPR snapd#3372 opened: tests: add basic lxd test <Created by mvo5> <https://github.com/snapcore/snapd/pull/3372>15:30
* zyga-suse was a the sauna15:30
zyga-suseI should have been doing that all this week15:30
zyga-susesergiusens_: saunas are fun15:30
mvofgimenez: do you happen to know if the "nightly" suite of snapd is run currently? if so, is that part of spread-cron?15:30
fgimenezmvo: yes, according to https://travis-ci.org/snapcore/spread-cron/branches it was executed last night (the branch is called "snapd-nightly-suite")15:34
fgimenezmvo: this is the job executed currently https://github.com/snapcore/spread-cron/blob/snapd-nightly-suite/run-checks15:34
mvofgimenez: great, thank you!15:35
* Chipaca ~> walk15:35
fgimenezmvo: np :) i see that the unity test is still around, maybe better removing it completely and execute the whole nightly suite, that way it would be easier to add new tests there15:36
Chipacajdstrand: we don't yet support running ia32 binaries on amd64, right?15:36
mvopedronis: is the night extra-snap-assertions your baby? it looks its unhappy currently https://travis-ci.org/snapcore/spread-cron/builds/268207725#L71515:36
mvofgimenez: +1 for removing that15:36
pedronismvo: I don't even know what it is15:36
mvofgimenez: I'm mostly interessted in this because of lxd, docker which are a bit heavy for the normal runs15:37
mvopedronis: no worries, I have a look15:37
zyga-suseChipaca: we do15:37
Chipacaah! good :-D15:37
Chipacanow yes, /me -> walk15:37
pedronismvo: doesn't mean I wasn't involved but  I don't remember15:37
fgimenezmvo: pedronis i added it, it checks for the feature of using extra-snaps with assertions to ubuntu-image15:38
mvopedronis: your name does not come up in git blame, so I guess your memory is fine15:39
=== JoshStrobl|AFK is now known as JoshStrobl
pedronisConfigManager is weird, is not actually a StateManager :/15:42
zyga-suseI think it's only that because it handled hooks15:50
jdstrandroadmr: I noticed r922 is live. thanks!15:58
jdstrandstgraber: fyi, feel free to use reload-command15:59
roadmrjdstrand: oh yay! true, there was a deploy this morning15:59
=== cachio_lunch is now known as cachio
stgraberjdstrand: yay!16:00
sergiusenszyga-suse: they are indeed16:02
zyga-susesergiusens: your mainline kernel will soon-ish work16:03
jdstrandChipaca: ia32 on amd64, do you mean i386 on amd64/x86 on x86_64? if so, snapd supports this. snapcraft (I don't think yet) makes that easy16:14
zyga-susekyrofa: interesting, I'll need to ponder on this some more16:14
zyga-susejdstrand: interesting bug there btw,16:14
* jdstrand wanted to make sure you weren't talking about x3216:14
jdstrandI phrased the snapcraft bit weird. I don't think snapcraft helps at all there16:15
mupPR snapd#3793 closed: cmd: "make hack" now also installs snap-update-ns <Created by chipaca> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3793>16:15
jdstrandbut I did this work a long time ago (now) for popey's use case of wine on amd64 being able to run 32 bit binaries16:15
niemeyerHellos16:16
jdstrandand all that carried forward into recent snap-seccomp, etc16:16
jdstrandhey niemeyer :)16:16
niemeyerHow're things going today?  Anything urgent to look after, or can I jump into the review board and reviews?16:16
jdstrandniemeyer: I've not seen anything scary in backscroll, but I'll defer to others16:23
jdstrandChipaca: sudo snap install test-seccomp-compat --edge16:24
jdstrandChipaca: test-seccomp-compat.true3216:24
jdstrandChipaca: test-seccomp-compat.true6416:24
jdstrandChipaca: oh, I forgot, cmd/snap-confine/spread-tests/main/test-seccomp-compat/task.yaml :)16:25
zyga-suseniemeyer: hey16:25
zyga-suseniemeyer: how are you doing?16:25
zyga-suseniemeyer: I think we're good16:25
zyga-susejdstrand: nice snap!!!16:25
niemeyerjdstrand, zyga-suse: Sweet16:26
jdstrandzyga-suse: thanks :)16:26
jdstrandzyga-suse: the bug you mentioned earlier-- you meant snapcraft making it easy for compat archs?16:26
zyga-suseno, I mean the bug where snap-confine and / and MS_RSHARED and containers blow up16:26
zyga-susehttps://bugs.launchpad.net/snapd/+bug/171293016:26
mupBug #1712930: snap-confine: mounts happen in the wrong order <snapd:In Progress by zyga> <https://launchpad.net/bugs/1712930>16:27
zyga-susereal stuff is on https://discuss.linuxcontainers.org/t/snapd-cant-remove-old-revisions-when-running-inside-lxd/452/316:27
jdstrandah, I didn't see that one yet16:27
* zyga-suse loves this moment of the day16:27
zyga-susesun is visible because it is below clouds16:27
zyga-suseand shines through moving trees16:27
jdstrandzyga-suse: sounds nice :)16:28
jdstrandwe're looking forward to a bunch of thunderstorms this evening16:29
zyga-susehttps://twitter.com/zygoon/status/90111935087003648116:29
mupPR snapd#3811 opened: interfaces/i2c: adjust sysfs rule for alternate paths <Created by jdstrand> <https://github.com/snapcore/snapd/pull/3811>16:49
jdstrandzyga-suse: pretty :)16:52
mupPR snapcraft#1507 opened: many: simplify plugin loading <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1507>17:01
niemeyermvo: snapd#3260 seems good to go for next week17:05
mupPR snapd#3260: cmd/snap: implement userd command as replacement for snapd-xdg-open <Created by morphis> <https://github.com/snapcore/snapd/pull/3260>17:05
willdeberryfinally got it showing! :) `:bluez`17:12
willdeberrythanks zyga-suse17:12
Pharaoh_Atemzyga-suse: I wish it was good day here17:15
Pharaoh_AtemI'm stuck inside an office working on backporting crap to old stuff :(17:15
Pharaoh_AtemI can't even see the sun from where I am :(17:15
jdstrandwilldeberry: nice!17:37
Chipacaniemeyer: how can I edit somebody else's post in the forum?17:39
Chipacawanting to indent a chunk of stuff so it'll be formatted properly17:40
* Chipaca hugs Pharaoh_Atem 17:43
ChipacaPharaoh_Atem: not long to go now17:43
* Chipaca wonders why he's doing tech support in the forum17:45
* Chipaca realises the distance between "snapd doesn't work" and "my system doesn't work" is invisible to users \o/17:46
Pharaoh_AtemFINALLY17:50
Pharaoh_Atemwe're going to have the xdg-open replacement?17:50
willdeberryjdstrand: :) . I will be cleaning up and then sending over a PR sometime today for it17:51
willdeberryi did have to resort to build the deb and installing. doesn't seem like building snapd was enough17:51
niemeyerChipaca: Hm17:53
niemeyerChipaca: Not sure if you can while being a moderator17:53
Chipacaeh, never mind17:54
niemeyerChipaca: ?17:54
Chipacaniemeyer: if it was easy and obvious, sure, but it's not important17:54
niemeyerIf moderators can, then it's easy and obvious to give you that flag17:55
Chipacaah!17:55
Chipacaniemeyer: i misread your "not sure if you can while being a moderator" as "afaik being a moderator you can't"17:55
niemeyerAh, sorry.. it was more like "AFAIK I have no idea"17:56
niemeyerChipaca: See if you can now17:57
ChipacaI do have a pencil now17:57
niemeyerBingo17:57
niemeyerChipaca: Be careful when tagging now.. you won't be constrained to our standard targs17:57
niemeyertargs is great17:57
niemeyerWow.. it's even nicer than I expected.. "Targ or TARG may refer to: The Anti-Gravity Room"17:58
Chipacambuahaha17:59
* Chipaca tags all the things17:59
Chipacaniemeyer: if you want you can bump me down again :-)17:59
jdstrandwilldeberry: re building the deb-- that's definitely going to be more robust. glad that worked for you17:59
niemeyerChipaca: Will keep you up.. hopefully you can actually help on the moderation :)18:00
Chipacaniemeyer: with great power come great tee shirts, right?18:00
niemeyer(unlike zyga, which after a few days had half a dozen random tags, posts painted yellow, etc /o\)18:00
niemeyer:P18:00
niemeyerjdstrand: Do you have a moment to quickly cover the desktop interface PR?18:02
jdstrandniemeyer: yeah18:03
niemeyerjdstrand: Ok.. so the thing I'm trying to figure after the extensive conversation we had there is whether we're still solving a problem worth solving for the price of the added complexity18:03
niemeyerjdstrand: I miss some technical pieces, so would like to talk live about it.. can we have a quick hangout?18:04
jdstrandok18:04
jdstrandniemeyer: where?18:06
Pharaoh_Atemmvo: it feels weird to see mvo@ubuntu.com in the fedora changelogs: https://koji.fedoraproject.org/koji/buildinfo?buildID=956399 :D18:06
niemeyerjdstrand: hangouts.google.com/hangouts/_/canonical.com/desktop-interface?authuser=018:06
Pharaoh_Atemmvo: but it seems people appreciate the fuller changelogs, so we'll keep rolling with it18:07
jdstrandsigh, the camera stopped18:07
jdstrandgimma a sec18:07
Dee__Hey all! Dee here, New to using Snaps19:03
kyrofaHey Dee__, welcome19:10
niemeyerBye Dee :)19:22
geniiheh19:23
kyrofaI guess I wasn't the person to whom Dee wanted to talk19:30
mvoPharaoh_Atem: heh, I can offer more addresses if you want :)20:18
mvoniemeyer: re 3260> yeah, I'm quite happy with it now after the last refactor and added tests20:19
mvoPharaoh_Atem: I will also fix your point in 326020:20
mvoPharaoh_Atem: I reverted the snapd.spec file changes as you requested, will check tomorrow if tests are still happy. thanks for the review!20:28
willdeberryone last thing i am still dealing with is the randomness of error: `/snap/bjarkan/52/usr/bin/python3: 1: /snap/bjarkan/52/usr/bin/python3: Syntax error: word unexpected (expecting ")")`21:13
willdeberryi can rebuild on snapcraft.io and the next build will not have any issues21:13
willdeberrynot sure what is causing this21:13
willdeberrymakes me wonder if there is a bug in the python plugin for building21:14
kyrofaHey niemeyer, can we get your thoughts on this? https://bugs.launchpad.net/snapcraft/+bug/171206121:20
mupBug #1712061: snapcraft restricts version field too much <Snapcraft:New> <https://launchpad.net/bugs/1712061>21:20
mupPR snapcraft#1508 opened: schema: version should have a max length of 32 <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1508>21:31
niemeyerkyrofa: Sure, will look in a moment21:49
mupPR snapcraft#1507 closed: many: simplify plugin loading <Created by kyrofa> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1507>22:05
mupPR snapcraft#1509 opened: project_loader: process stage package grammar <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1509>23:35
niemeyerkyrofa: Done23:47
kyrofaThanks niemeyer. Why are you still here?!23:48
niemeyerkyrofa: My day started a bit late today.. kid was a bit under the weather last night and I went to sleep when I supposed to be waking up.. so it all shifted forward a bit23:49
kyrofaAh, poor kid. We just finished a round of the stomach flu, I know how you feel23:49
niemeyerYeah.. wonders of being in a school with other children23:50
niemeyerI guess that's how they improve their immune system23:50
niemeyer(and ours :P)_23:50
kyrofaNo kidding23:55

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