/srv/irclogs.ubuntu.com/2017/09/18/#snappy.txt

=== ikey|work is now known as ikey|zzz
=== wgrant__ is now known as wgrant
=== JoshStrobl is now known as JoshStrobl|zzz
zyga-ubuntumvo: hello07:01
zyga-ubuntumvo: how are you doing?07:01
zyga-ubuntumvo: I updated the mini-lab, the dragon board now has a HDD and swap so should be suitable for more tasks07:02
mvohey zyga-ubuntu - good monring. I'm doing well, how are you?07:02
zyga-ubuntumvo: I also made ssh keys sync hourly, though not to each device yet (just to the ssh hop)07:02
zyga-ubuntumvo: I'm iterating on layouts and feeling good :-)07:02
zyga-ubuntumvo: the damp weather is a good motivator to keep warm and code :)07:02
zyga-ubuntumvo: I will also add power control to dragon-1, it's already wired but not scripted yet07:03
zyga-ubuntumvo: how are your purchases going?07:03
zyga-ubuntumvo: while not perfect I found a working way to have home on an external HDD on ubuntu core07:05
mvozyga-ubuntu: nice progress! no news on purchases, still pondering07:05
zyga-ubuntumvo: same with swap support07:05
mvozyga-ubuntu: oh, nice, how do you do it?07:05
zyga-ubuntumvo: I created two systemd mount units07:05
zyga-ubuntumvo: ssh and have a look, it's remarkably working fine :)07:06
zyga-ubuntumvo: I ran some tests on it overnight and I have an idea07:06
zyga-ubuntumvo: when idle it could just pull master and run tests in a loop07:06
zyga-ubuntumvo: and report failures07:06
zyga-ubuntumvo: I saw a few things like "settle" something fail because of timing07:06
zyga-ubuntumvo: also noticed that the board only reports 0.7GB RAM, I suspect the rest is used up by GPU, modems and what not07:07
zyga-ubuntumvo: and I wanted to ask ogra if that could be tweaked for a headless builder07:07
mvozyga-ubuntu: interessting. I have a look in a bit once I finished look at PRs, mail etc :)07:08
zyga-ubuntumvo: sure :)07:08
* zyga-ubuntu -> breakfast07:21
sborovkovogra_: btw. commenting out that stuff in the psplash.c. It actually does not disable psplash-write. So may be you should disable it as well. When psplash-write is called text is drawn just as expected.07:22
zyga-ubunture07:32
zyga-ubuntupstolowski: hello07:32
zyga-ubuntupstolowski: question, there's a debian bug report about missing manual page for snapctl07:32
pstolowskizyga-ubuntu, good morning!07:32
zyga-ubuntupstolowski: would you like to write one in restructured text like some of our other manual pages?07:33
pstolowskizyga-ubuntu, sure. do we generate man pages from built-in help?07:36
zyga-ubuntupstolowski: I think that's largely insufficient, look at the manual page for snap-confine for instance07:41
zyga-ubuntupstolowski: the help part is really one element07:41
pstolowskizyga-ubuntu, ok, will do, just looked at .spec for fedora and saw man page generated from snap --man tere07:41
pstolowski*there07:41
zyga-ubuntupstolowski: that's not the same, again07:42
zyga-ubuntupstolowski: unless you have 10s of options I really strongly encourage you to write it manually07:42
pstolowskizyga-ubuntu, okay. snap-confine.rst it is07:44
zyga-ubuntupstolowski: thanks, I'll gladly help if you need anything07:45
pstolowskithanks zyga-ubuntu07:46
* zyga-ubuntu fights the 14.04 battle08:20
zyga-ubuntumvo: would it be possible to build snapd with golang 1.6 like on 16.04?08:21
zyga-ubuntumvo: I just got a failure about C99 constructs that are probably because of older golang defauls08:21
mvozyga-ubuntu: on 14.04? we use a backported go-1.608:21
mvozyga-ubuntu: bug of course there is an older gcc there08:21
zyga-ubuntumvo: aah08:21
zyga-ubuntumvo: that makes sense!08:22
zyga-ubuntuthank you08:22
zyga-ubuntuwell, one more rn08:23
zyga-ubuntu*run08:23
ogra_zyga-ubuntu, you can turn off the GPU ram but wont be able to run accellerated graphics stuff... just drop the last line in /booot/uboot/config.txt08:27
zyga-ubuntuogra_: oh, thank you08:27
zyga-ubuntuogra_: do you think that would make sense as a core config option?08:27
ogra_sborovkov, perfect, yeah, i''ll do that08:27
ogra_zyga-ubuntu, probably ... we need to have it on by defautl so trhat mir works thugh08:28
zyga-ubuntuogra_: yes, certainly08:30
zyga-ubuntuogra_: I don't have /boot/uboot/config.txt - just uboot.env which is binary08:31
ogra_zyga-ubuntu, bah, sory, i mis-read08:32
ogra_i thougth you were n a Pi08:32
zyga-ubuntuaha, right08:32
ogra_i dont think there is a way to tune that on a dragonboard08:32
zyga-ubuntuogra_: ok, thanks08:32
ogra_despite using your own patched kernel or so08:32
zyga-ubuntuone of those days we will get sodimms on the devkits :)08:33
ogra_zyga-ubuntu, also ... the dragonboard is am64 ... the ram is not really eaten by GPU but by the fact that your binaries are about 1/3 bigge in ram08:33
ogra_arm64 on small boards is a massive insanity08:33
zyga-ubuntuogra_: oh? why does linux report 0.7GB memory total then?08:34
ogra_(especially if the same board can run 32bit and eat half the ram in the end)08:34
ogra_well, part of it is surely GPU ... but you will also use massively more ram for the binaries08:35
zyga-ubuntusure but it looks like 0.3 is just reserved up front08:36
ogra_yeah08:37
ogra_as i said, i dont think thats tunable08:37
ogra_despite recompilling parts08:37
mupIssue snapcraft#1543 closed: Unable to specify make arguments <Created by akashrajkn> <Closed by akashrajkn> <https://github.com/snapcore/snapcraft/issue/1543>08:39
ogra_just a FYI ... htop in a beaglebone repts 48MB in use on a idle system ... with the same set of snaps installed, htop on rteh dragonboard reports 122MB08:39
ogra_(there is some minimal overhead because the dragonboard uns wpa_supplicant and stuff, but you get the difference between arm64 and armhf)08:40
sborovkovogra_: hi :) so from that script you provided me for using on the first boot. File system is always available? I am a bit confused about what's going on there.. I tried this if [ ! -d "/root/system-data/var/snap/screenly-client/" ] ; then psplash-write.... And checking for snap.json file. Somehow it executes that line on every boot :-(08:48
ogra_sborovkov, oh, sorry, i improved that a lot already ... one sec. let me dicg it up08:48
ogra_sborovkov, checking for state.json is the best thing to do here http://paste.ubuntu.com/25563809/08:50
ogra_sborovkov, it works fine here08:50
ogra_(only shows on first boot)08:51
sborovkovgreat.08:51
sborovkovthanks08:51
ogra_:)08:51
sborovkovI was checking it with -f08:51
sborovkovnot sure why it shows up every time08:51
sborovkovah08:53
sborovkovcause I did not check for /root/writable08:53
sborovkovbut /root/system-data instead08:53
ogra_ah, yeah08:53
Chipacain the forum, when something is fix-committed, do i do anything or do i leave it on upcoming?08:54
Chipacado i leave myself tagged in it?08:54
ogra_isnt there ar "solution" tag that seta a gee checkmark and such ?08:56
ogra_*sets08:56
ogra_*green08:56
pedronisChipaca: I think we remove upcoming and name only when is fix-release08:59
pedronisd09:00
Chipacak09:00
pedronisChipaca: unless is some kind of code reorg/internal feature, then probably landing to master is enough09:00
mwhudsonzyga-ubuntu: did you see that maciattobin board thing?09:08
mwhudsonzyga-ubuntu: that looks like an arm64 device you could use without endless swearing09:08
zyga-ubuntumwhudson: yeah but it's not a ref device, I would probably consider getting one of the server boards if that was one09:12
mwhudsontrue09:12
zyga-ubuntumwhudson: (one of the older armv8 server boards with DDR slots and sata and everything)09:13
mwhudsonwhich of those?09:13
zyga-ubuntumwhudson: any that's available :)09:13
mwhudsonah heh09:13
zyga-ubuntumwhudson: it would be interesting to have an UEFI device09:13
Chipacazyga-ubuntu: mwhudson: and the OpenQ 820?09:13
* Chipaca doesn't know what makes something a 'reference device'09:13
zyga-ubuntuChipaca: our decision09:14
Chipacaah :-) ok09:14
zyga-ubuntuChipaca: btw, can you perhaps ssh into office.zygoon.pl?09:14
ChipacaI dunno09:14
Chipacaproperly get cooties09:14
zyga-ubuntuI wanted to check if this works for you09:14
mwhudsonzyga-ubuntu: that's still a mobile phone cpu isn't it?09:14
zyga-ubuntumwhudson: ye09:14
Chipacazyga-ubuntu: what's your fingerprint?09:14
mwhudsonzyga-ubuntu: at least it has usb 3 and pcie so the fastest io probably isn't the wifi :)09:15
Chipaca(i read about the macchiatobin and the 820 reading about arm64 laptopts)09:16
zyga-ubuntu256 SHA256:C3V0UBPGQHPToLeX/qcEAAGX02XbbwHDx0zQkVQtM74 mini (ECDSA)09:16
zyga-ubuntu2048 SHA256:x4RFVP/TtE/nLtJ2WUO3dlLsavKWny7qeXNR1gHL3HA mini (RSA)09:16
zyga-ubuntu256 SHA256:adjGbm4pxRSXYErI0y/bnQeaeJqCDfJKYPHPDLQxKoM mini (ED25519)09:16
Chipacahttps://lwn.net/Articles/733837/09:16
mwhudsonoh wait, last two comments should have been at Chipaca :)09:17
Chipacamwhudson: :-)09:17
Chipacazyga-ubuntu: i'm in09:18
Chipacazyga-ubuntu: minicom says i can't use the serial port09:18
zyga-ubuntuChipaca: can you ssh into dragon-1 now09:18
zyga-ubuntuChipaca: oh, I'll adjust that09:19
zyga-ubuntuChipaca: you should be good now, also you should be able to hop onto dragon-109:19
Chipacazyga-ubuntu: the pi3 asks me for my password09:24
Chipacazyga-ubuntu:  the dragon works09:24
zyga-ubuntuChipaca: yes, pi is not configured yet, when I have more time I'll improve the script that sets this up to push accounts there as well09:25
zyga-ubuntuChipaca: I mainly wanted to make the dragon available as most people have a working PI already09:25
Chipacaah ok09:25
zyga-ubuntuChipaca: and the dragon is a hit and miss it seems09:26
zyga-ubuntuChipaca: feel free to use that any time09:26
Chipacazyga-ubuntu: i can confirm minicom now lets me talk to the dragon09:26
zyga-ubuntuChipaca: nice09:26
Chipacazyga-ubuntu: why minicom and not screen, btw?09:26
zyga-ubuntuChipaca: I will add a way to power the device on and off from mini09:26
zyga-ubuntuChipaca: screen was corrupting input heavily09:26
zyga-ubuntuChipaca: and minicom was solid09:26
zyga-ubuntuChipaca: no idea, try screen09:26
zyga-ubuntuChipaca: I also like minicom :)09:27
Chipaca:-)09:27
mupPR snapd#3930 opened: cmd/snap-repair: integrate root public keys for repairs <Created by pedronis> <https://github.com/snapcore/snapd/pull/3930>09:36
mupBug #1717857 opened: UDisks2 interface restricts sending DBus.Properties.Get message from the client to udisksd service <Snappy:Confirmed> <snapd (Ubuntu):Confirmed> <https://launchpad.net/bugs/1717857>09:43
mupPR snapd#3931 opened: interfaces/builtin: allow receiving dbus messages <Created by zyga> <https://github.com/snapcore/snapd/pull/3931>09:44
mvozyga-ubuntu: hm, is 3931 a regression?09:45
zyga-ubuntumvo: no, it's an old bug09:46
mvozyga-ubuntu: interessting that we have not gotten bugreports earlier09:48
zyga-ubuntumvo: yep, looks like nobody really tried tihs09:48
ogra_ppisati, i have been getting reports about SD corruption lately and i think we are hitting https://github.com/raspberrypi/firmware/issues/397#issuecomment-219287224 with some newer SD cards ... i suspect we'll need to collect some data and will need to ship some quirks like https://github.com/respeaker/openwrt/blob/master/target/linux/brcm2708/patches-4.4/0354-mmc-Apply-QUIRK_BROKEN_ERASE-to-other-capacities.patch09:48
ogra_ppisati, that is raspi2 only though09:49
ogra_(i'm still collecting info, just a FYI)09:51
ppisatiogra_: kernel version?10:03
ppisatiogra_: and we already have those quirks applied10:07
ppisatiogra_: i'm interested to know, if they experienced this on a 4.4.0-1071.79+ version10:08
pedronismvo: I just noticed reading the topic that the plan was that the script would just call 'repair' (not 'snap-repair') using some kind of symlink, it's a bit annoying but makes sense10:10
mvopedronis: I was looking at the summary in the snap-repair show and was wondering if it makes sense to include the summary into the log output, it would make it easier for humans to inject the snap-repair directories using find/less/grep10:11
mvopedronis: re 'repair'> sure, I can create a symlink10:11
pedronismvo:   we could, a bit of a question of how to format it there10:12
pedronisand how to present this in show itself10:13
mvopedronis: yeah, I was thinking a tiny header like: "repair: canonical-1\nsummary: repair one\noutput:..." and we can/could either show in snap-repair show (even though slightly redundant) or just filter out in the command10:14
mvopedronis: feels nice to have the relevant info in the file itself, was thinking e.g. filesystem corruption where fsck might not reconstruct the filename but the content. anyway, maybe overthinking this10:15
zyga-ubuntuhmm10:16
zyga-ubuntuUnpacking manpages-dev (4.13-2) ...10:16
zyga-ubuntudpkg: error processing archive /tmp/apt-dpkg-install-FOEqdJ/39-manpages-dev_4.13-2_all.deb (--unpack):10:16
zyga-ubuntu trying to overwrite '/usr/share/man/man4/console_ioctl.4.gz', which is also in package manpages 4.10-210:16
zyga-ubuntumust be my lucky day10:16
zyga-ubuntuthe systemd-virt-detect bug fixed itself10:16
zyga-ubuntuand now this strikes back from hell10:17
mvopedronis: I will modify 3925 to create a dir with "repair" that symlinks to snap-repair and have that in PATH. we only need one,right? i.e. not snap-repair and repair (both names) for the repair script?10:17
pedronismvo: yes, sounds rights10:18
pedronis*right10:18
mvota10:18
ogra_ppisati, it is the curent pi2-kenel snap, namely AlbertA and bschefer are using the same SD type and brand and both have a corrupted booot pattiion after every core upgrade (which means daily if you use the edge channel... )10:18
ogra_ppisati, some 32GB sandisk ... i'm waiting for them to get online to get the manfid and stuff10:19
ppisatiogra_: 4.4.0.1071.71 - edge10:20
ogra_(sadly i shuffled the card into another board and cant currently use it in the pi)10:20
ppisatiogra_: then no, it is not what i think10:20
ogra_it seems to be oonly a small set of SDs but i hear about it regulary from users10:20
pedronismvo: about summary in output, I think it makes sense,  just a bit wondering that it would be best to strip it out in show and have it separate, but we need to think what to do about newlines10:21
ogra_ppisati, and i know we already have a quirks patch ... i just think it needs to cover some more cards10:21
ppisatiogra_: ack, is there a way to reproduce it?10:22
ogra_ppisati, for bschefer and AlbertA the overlays subdir on the boot partition gets messed up so the kms dtb doesnt get loaded and they end up without /dev/dri ... for them this is reproducable with every update10:23
ogra_foo me on my card it used to be uboot.env that got corrupt, also reliably with every update10:24
mvopedronis: yeah, I will ponder a bit10:25
ogra_ppisati, my idea was to cllect their SD data, and let them try a patched kenel t see if it help ... but the behaviour sounds suspiciously like the erase issue from above10:25
ogra_(specifically because they both have the same card exposing the same issue 100% of the time)10:26
mupPR #100: Ongoing work on the capability APIs <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/100>10:26
ogra_*** baa-dum tish ***10:27
ogra_zyga-ubuntu wins the price fr the 100st PR !!!10:27
* ogra_ throws confetti10:27
ppisatiogra_: makes sense - do you have an old edge image around? so i can try with some sd cards here, and to some update cycles10:27
ogra_ppisati, http://people.canonical.com/~ogra/snappy/all-snaps/daily/ just pick an old one there10:28
zyga-ubuntuheh :-)10:28
ppisatiogra_: cool, will do - pi2 or pi3?10:28
ogra_ppisati, pi3 here but i bet it doesnt matter10:29
* zyga-ubuntu tries a fix for the debian bug and goes to make more tea10:29
* zyga-ubuntu itereates on manpages bug10:40
zyga-ubuntuand while spread runs let's test Fedora 25 update10:41
zyga-ubuntumvo: that's something for you10:45
zyga-ubuntuhttps://github.com/snapcore/snapd/pull/393210:45
mupPR #3932: spread: work around temporary packaging issue in debian sid <Created by zyga> <https://github.com/snapcore/snapd/pull/3932>10:45
zyga-ubuntumvo: locally still going10:45
mupPR snapd#3932 opened: spread: work around temporary packaging issue in debian sid <Created by zyga> <https://github.com/snapcore/snapd/pull/3932>10:45
mupPR snapd#3933 opened: snap-repair: make `repair` binary available for repair scripts <Created by mvo5> <https://github.com/snapcore/snapd/pull/3933>10:46
mvozyga-ubuntu: ta10:48
zyga-ubuntumvo: tests are rolling so I think this is a good fix10:52
ppisati$ snap list10:58
ppisatiNo snaps are installed yet. Try "snap install hello-world".10:58
ogra_ppisati, hrm10:58
ppisatiogra_: pi3 image of 3 days ago, i feel confused10:58
ogra_ppisati, sounds like fixrct didnt run10:58
ppisati14 of sept image10:58
zyga-ubuntudarn, I ran out of disk space10:59
ogra_ppisati, very weird, i tested them after fixrtc got fixed again11:00
ogra_oh ... one sec11:00
ppisatiogra_: let me try with the 15th and 16th of sept image11:00
* ogra_ checks if edge actually has the latest kernel 11:00
ogra_hmm, no, latest kernel there11:02
ogra_i thought perhaps edge would be behind11:02
mvopedronis: do we allow newlines in the summary currently?11:03
zyga-ubuntuSon_Goku: thank you for pushing the 2.27.6 release out11:15
Son_Gokumy 2.27.6 is special, though :)11:15
zyga-ubuntuSon_Goku: yes, I know :)11:15
Son_Gokuit's better than all the other snapd 2.27.6 :)11:15
zyga-ubuntuSon_Goku: I'm testing it on F25 now11:15
zyga-ubuntuSon_Goku: though it's out already I wanted to finish it11:15
zyga-ubuntuSon_Goku: sorry for being slow, the weather is getting the better of me :/11:15
Son_Gokuit's fine11:16
zyga-ubuntuhow are you doing?11:17
pedronismvo: in principle yes, we don't check for them not to be there? we could prohibit them in asserts/repair.go if we want11:19
Son_Gokuzyga-ubuntu: alright I suppose11:25
ogra_zyga-ubuntu, you should use stgraber's in-browser terminal that he uses for https://linuxcontainers.org/lxd/try-it/ and hook that up to native containers on the pi and dragonboard ;)11:37
ogra_(for your device lab that is)11:37
zyga-ubuntuogra_: is there a snap for the client part?11:37
ogra_no idea :)11:38
zyga-ubuntuogra_: I've redirected ports for mosh but I don't think we have a working mosh snap11:38
ogra_but that sounds like an awesome spare-time project :)11:38
zyga-ubuntuogra_: but you can mosh into the mac mini allright11:38
ogra_having a central website to try ubuntu core in a browser on specific supported boards11:38
zyga-ubuntuogra_: I need to get a 2nd hand desk rack11:39
zyga-ubuntuogra_: I will add my other boards there but I have no space to make this reliable now11:39
zyga-ubuntuogra_: btw, how do you power the dragon board?11:42
zyga-ubuntuogra_: I've set 12V and 2A11:42
ogra_yeah, that should be fine11:42
zyga-ubuntuogra_: I wonder what the on-board regulators do11:42
ogra_the 96boards boards all support 12-19V iirc11:42
zyga-ubuntuogra_: is thre an optimal value that produces least waste heat11:43
ogra_i think 12V/2A is fine11:43
ogra_not sure how much lower you can go, there is a lot HW on the dragonboard11:43
zyga-ubuntuogra_: fun fact11:44
zyga-ubuntuogra_: I powered it last week using the PSP power adapter11:44
zyga-ubuntuogra_: that's 5V11:44
ogra_https://www.96boards.org/product/power/11:45
ogra_"The 96Boards CE boards require an 8-18V 2A power supply."11:45
ogra_so you were lucky i guess :)11:45
zyga-ubuntuwell now it should be solid :)11:45
zyga-ubuntuI was thinking about setting the limit to 3A11:45
zyga-ubuntubut even under load I see at most 0.5A11:46
tbrfun fact: the spec was *specifically* written to exclude powering it from USB/5V "because that's just too weak"11:46
ogra_because you are not using all HW ... iirc there is GPS and such11:46
tbralso hook up some USB devices that go to full spec load of 0.5A11:47
ogra_yep11:47
zyga-ubuntutbr: it powers an external HDD using two USB ports11:47
zyga-ubuntutbr: but now it's on 12V/2A supply11:48
ogra_the 12/2 thing will guarantee that it stays fully functional if all onboard HW is powered and used11:48
tbrand all USB ports used11:49
tbrpower budget is always spec'd to full load *everything* and a bit of safety margin (at least one would hope)11:49
ogra_right11:49
* Chipaca grabs the painkillers and heads back to bed11:50
* zyga-ubuntu hugs Chipaca 11:50
zyga-ubuntuChipaca: stay strong, it's not even winter yet11:50
Chipacaheh11:50
Chipacazyga-ubuntu: it's my back having a bad day, is all11:50
* zyga-ubuntu reads https://www.securecoding.cert.org/confluence/display/c/MSC06-C.+Beware+of+compiler+optimizations11:50
Chipacai must've done something unexpected over the weekend11:50
zyga-ubuntuChipaca: ouch, even more sympathy then :/11:50
Chipacalike, the dishes /o\11:51
zyga-ubuntuI know how bad that is11:51
mupBug #1710637 opened: Input falls through to gdm3 and terminates the session on Ctrl+C <amd64> <apport-bug> <artful> <gnome-17.10> <rls-aa-incoming> <wayland-session> <Snappy:New> <console-setup (Ubuntu):Fix Released> <gdm3 (Ubuntu):Incomplete> <gnome-shell (Ubuntu):Incomplete> <mutter12:05
mup(Ubuntu):Incomplete> <https://launchpad.net/bugs/1710637>12:05
zyga-ubuntuwhaat?12:07
ogra_check the desktop channel :)12:07
zyga-ubuntumvo: hey, just FYI in case it blows up on us: https://forum.snapcraft.io/t/snap-remove-somesnap-triggers-keyboard-input-fallthrough-to-vt-ctrl-c-in-terminal-logs-out-of-current-gdm-session/2162/6 and https://bugs.launchpad.net/ubuntu/+source/gdm3/+bug/1710637 -- we may need another release for that12:27
mupBug #1710637: Input falls through to gdm3 and terminates the session on Ctrl+C <amd64> <apport-bug> <artful> <gnome-17.10> <rls-aa-incoming> <wayland-session> <Snappy:New>12:27
mup<console-setup (Ubuntu):Fix Released> <gdm3 (Ubuntu):Incomplete> <gnome-shell (Ubuntu):Incomplete> <mutter (Ubuntu):Incomplete> <https://launchpad.net/bugs/1710637>12:27
* zyga-ubuntu breaks for lunch before standup12:28
=== ShalokShalom_ is now known as ShalokShalom
ackkniemeyer, when you have time, could you have a look at https://github.com/snapcore/snapd/pull/3916 for the addition to the snap.yaml content for apps?13:11
mupPR #3916: add support for socket activation in apps <Created by albertodonato> <https://github.com/snapcore/snapd/pull/3916>13:11
zyga-ubuntujdstrand: hello13:17
zyga-ubuntujdstrand: thank you for the review, I pushed more things and I think it's ready for re-review13:17
zyga-ubuntujdstrand: I'd like to start proposing new PRs instead of iterating on this one as it's veryt hard to keep track of everything13:18
jdstrandzyga-ubuntu: I'm done with my big reviews, so anything extra is just making sure what I commented on is done13:19
jdstrandzyga-ubuntu: I assume you are talking about PR 3621?13:20
mupPR #3621: cmd/snap-{confine,update-ns}: apply mount profiles using snap-update-ns <Created by zyga> <https://github.com/snapcore/snapd/pull/3621>13:20
jdstrandzyga-ubuntu: that PR is tricky because we are doing retroactive reviews. ie, it is arguably fine when not called from snap-confine but when called from snap-confine, needs changes13:22
zyga-ubuntujdstrand: yes13:22
zyga-ubuntujdstrand: let's mark it as good then13:22
zyga-ubuntujdstrand: and land it :)13:22
zyga-ubuntujdstrand: I just cannot land it before you agree13:22
jdstrandbut anyway, like I said, I finished the big reviews, all that is left is making sure that comments are addressed, so I think it will be mergeable soon13:22
zyga-ubuntuthank you13:22
* jdstrand nods13:22
ogra_ppisati, did you get anywhere with the newer images ?13:37
ogra_koza, hmm, https://docs.ubuntu.com/core/en/stacks/bluetooth/bluez/docs/reference/sending-files#receiving-files ... it doesnt seem like the mentioned "bluez-test" snap from there is in the store13:39
ogra_*bluez-tests13:39
ppisatiogra_: yes, so all the available daily (14/15/16 of sept) behave the same - snap list is empty13:40
ppisatiogra_: but when installing the hello world snap, tra triggered the update of core13:40
ppisatiogra_: so it did the update and rebooted fine13:40
ogra_ppisati, seems the builder was stuck, there are new images nnow13:40
ppisati$ snap list13:40
ppisatiName         Version    Rev   Developer  Notes13:40
ppisaticore         16-2.27.6  2849  canonical  core13:40
ppisatihello-world  6.3        27    canonical  -13:40
ogra_ppisati, no, thats completely broken13:41
ppisatiogra_: yeah, it should show more snaps, but i was looking after the corruption13:41
ogra_ppisati, the device didnt get initialized ... else you would see the gadget and kernel snap in list ... what you have there is simply snapd that pulled in *another* core to be able to run hello-world13:41
ogra_ppisati, well, lets wait for bschaefer or AlbertA ... they can definitely reproduce 100%13:42
* ppisati tries the new daily13:43
ogra_ppisati, in the case where you have an uninitialized image an upgrade wont actually upgrade the bootloader bits (because the image simply doesnt know about the bootloader) ... so thats pretty different13:43
ogra_on proper images a core update re-writes the bootloader config13:44
ogra_and thats where the corruption usually happens ... the vfat seems to be more prone to corruption than the ext4 partition in that case13:44
=== ikey|zzz is now known as ikey
=== chihchun_afk is now known as chihchun
ppisatiogra_: the latest daily (18 of sept) is good, snap list report a list of snaps14:02
ogra_phew14:03
ogra_so the stale builds used soome old kernel before the no-change rebuild14:03
ppisati$ snap refresh core14:04
ppisatisnap "core" has no updates available14:04
ogra_yeah ...14:04
ogra_do "snap refresh core --stable"14:04
ogra_that force-switches the channel so it will definitely up/downgrade14:05
=== JoshStrobl|zzz is now known as JoshStrobl
ppisatiogra_: did it, it rebooted fine and i'm back in - mmcblk0p1 is mounted and i can see its content14:18
ogra_ppisati, great14:18
ogra_ppisati, so your SD isnt affected14:18
* ppisati tries with another sd card, just in case...14:21
* zyga-ubuntu forgot about his daughter because of the 2nd call14:25
zyga-ubuntupstolowski: hey14:35
zyga-ubuntupstolowski: I'll gladly help you on that effort14:35
zyga-ubuntumvo: can we land https://github.com/snapcore/snapd/pull/3932?14:36
mupPR #3932: spread: work around temporary packaging issue in debian sid <Created by zyga> <https://github.com/snapcore/snapd/pull/3932>14:36
zyga-ubuntujdstrand: thank you for a re-review14:39
pedroniszyga-ubuntu: pstolowski: a bit worried that this will mean even more Plug/SlotInfo around and that will be confusing14:40
=== JanC_ is now known as JanC
zyga-ubuntupedronis: if we add a interfaces.Connection type that stores PlugInfo, SlotInfo pointers nothing will (yet) change14:44
zyga-ubuntupedronis: then some reshuffling to represent things this way14:44
zyga-ubuntupedronis: as long as we don't land the plugdata branches it should not get more confusing than it is now14:45
zyga-ubuntujdstrand: I didn't miss those btw, I was still under the impression that those can be merged separately14:46
zyga-ubuntujdstrand: (so that this PR can land)14:46
zyga-ubuntujdstrand: but I can make them here as well if you think this is necessary14:46
mvozyga-ubuntu: is there a bts bug already about this issue in debian? if so, I think we should include a link to that so that we can remove this workaround once its fixed14:46
pedroniszyga-ubuntu: well we have things like Plugs()  []*Plug,  Plug have identity sort of, PlugInfo don't14:47
zyga-ubuntumvo: I didn't look14:48
zyga-ubuntupedronis: yes but that won't yet change, I think14:48
zyga-ubuntupedronis: unless I'm mistaken and we need to cross that bridge soon14:48
jdstrandzyga-ubuntu: there so small I think here is good. if we land them and something happens and the next one gets released, that isn't good. I could try to separate out the must haves and the should haves, but then it just makes the other PR review difficult cause we need to reference back to this one14:49
jdstrandthy're*14:49
jdstrandmeh14:49
jdstrandthey're*14:49
zyga-ubuntujdstrand: yes, that's a good point (about the release)14:49
jdstrandzyga-ubuntu: I mean, I guess technically you create a PR for just the security audit changes, land that before 3621, then merge that into 3621, but I don't see any gain in that14:52
* jdstrand is trying to get this to land fast too :)14:52
zyga-ubuntujdstrand: I didn't want to do the simplification changes as I think they are not critical and don't have a security impact14:52
jdstrandzyga-ubuntu: those are the should have column14:53
jdstrandlike I said, we can split it up, but I don't see any reason14:53
zyga-ubuntujdstrand: ok14:53
jdstrandI mean, we can make things harder on ourselves if we want :P14:54
jdstrandI'll leave the to split vs not up to you, but will say if split, please get the audit feedback in before 3621, so it is in place whenever 3621 is14:55
zyga-ubuntujdstrand: I'll do them in the PR14:55
jdstrandok cool. it is really close to landing14:55
zyga-ubuntujdstrand: can you please edit https://github.com/snapcore/snapd/pull/3621#pullrequestreview-63369978 to indicate if something is optional14:55
mupPR #3621: cmd/snap-{confine,update-ns}: apply mount profiles using snap-update-ns <Created by zyga> <https://github.com/snapcore/snapd/pull/3621>14:55
zyga-ubuntujdstrand: I'm not sure if all of that is mandatory or not14:55
zyga-ubuntusorry for being confused, I just dropped from a call and I'm swapping context14:56
jdstrandzyga-ubuntu: I did. I consider it all mandatory except the one that says (not a blocker)14:56
zyga-ubuntujdstrand: why are the simplifications mandatory?14:56
jdstrandzyga-ubuntu: I gave my feedback on why the simplification is important last week in response to your question14:56
zyga-ubuntuah14:56
zyga-ubuntulet me recheck14:56
jdstrandzyga-ubuntu: to be clear, there aren't security bugs there, but see my comment14:57
zyga-ubuntuah, I see that14:58
zyga-ubuntuI must have missed that, sorry, the thread is so long now14:58
jdstrandindeed14:58
jdstrandzyga-ubuntu: the basic idea is since it is security sensitive, make it easy to verify14:59
pstolowskizyga-ubuntu, pedronis hey, sorry, was distracted and missed your earlier messages15:11
=== cachio is now known as cachio_lunch
mupPR snapd#3810 closed: interfaces/hooks: PlugData and SlotData wrappers <Created by stolowski> <Closed by stolowski> <https://github.com/snapcore/snapd/pull/3810>15:13
pstolowskipedronis, what was your worry? about having PlugInfo & SlotInfo in the "Connection" struct?15:32
pedronispstolowski: not that, but when we remove Plug and Slot15:32
ogra_AlbertA, bschaefer so my pi did behave just fine over the weekend, no corruption ... but i found https://github.com/raspberrypi/firmware/issues/397#issuecomment-219287224 ... we might need something similar to https://github.com/respeaker/openwrt/blob/master/target/linux/brcm2708/patches-4.4/0354-mmc-Apply-QUIRK_BROKEN_ERASE-to-other-capacities.patch for your specific SD card15:33
bschaeferogra_, o interesting15:34
ogra_AlbertA, bschaefer, the output of "grep . /sys/class/mmc_host/mmc0/mmc0:*/* 2>/dev/null" (as mentioned in the issue above) might be interesting ... ppisati was also looking into it but has no SD he can reproduce it with15:35
bschaeferogra_, i bought two of the SD cards15:35
ogra_i assume both of yur boards ended up with a corrupt overlays dir again ?15:35
bschaeferi can give you or him one in new york (the one that was acting an issue)15:35
bschaeferor test some things out15:35
bschaeferogra_, monday for me today + i had that hack of manually untaring15:35
bschaeferlet me test agian15:36
bschaefer(monday 8:30am :)15:36
ogra_yeah, the patch is trivial, once we have the data from the grep above we ca surely cook something up15:36
ogra_*can15:36
mupPR snapd#3934 opened: snap-repair: implement `snap-reapir {listt,show}` <Created by mvo5> <https://github.com/snapcore/snapd/pull/3934>15:36
zyga-ubuntumvo: llist15:37
zyga-ubuntuer, sorry, I meant listt15:37
bschaeferogra_, cool let me re-flash and boot up15:37
mvopedronis: I reworked 3777 quite a bit (I think its nicer now) in 3934 - I can either push this over 3777 or close 3777 either way is fine with me15:38
mvozyga-ubuntu: thanks, fixed15:38
Son_Gokuzyga-ubuntu: are you going to be at the sprint/rally/thing next week?15:39
Son_Gokumvo: ^?15:39
ogra_mvo, is reapir an aggressive brother of tapir ?15:39
ogra_:P15:39
zyga-ubuntuSon_Goku: no, I don't think I will15:39
pedronismvo: I'll look in a bit15:39
mvopedronis: no rush, I need to leave soonish anyway to play hockey15:40
mvoSon_Goku: I will be there15:40
Son_Goku\o/15:40
mvoogra_: its what I turn into when I did not have enough tea sadly15:40
ogra_heh15:40
mvoSon_Goku: and more of the folks you know, we will have a blast!15:41
* ogra_ gives mvo more tea15:41
AlbertAogra_: ok I'll try to get those logs a bit later15:43
bschaeferogra_, http://paste.ubuntu.com/25566047/15:44
ogra_thx15:44
bschaefernp! Let me know if theres any fixes you want me to test out as well15:45
ogra_will do15:45
=== cachio_lunch is now known as cachio
AlbertAogra_: http://pastebin.ubuntu.com/25566134/15:58
* Chipaca ~> lunch15:58
ogra_AlbertA, bschaefer oooh, they differ !15:58
ogra_(different oemid)15:59
mupPR snapd#3935 opened: cmd/snap-repair: implement the repair run loop <Created by pedronis> <https://github.com/snapcore/snapd/pull/3935>16:01
bschaeferogra_, though i think we both had corrupting issues :)16:01
ogra_yes, and obviously the same16:02
bschaeferAlbertA, your SD is old!16:02
bschaefer201516:02
AlbertA:)16:02
AlbertAogra_: bschaefer: interesting, another core upgrade but this time after a second reboot, system won't boot at all (stuck at the "Starting Kernel..." screen)16:11
bschaeferAlbertA, that use to happen to me16:11
bschaeferbut i blamed by 8GB SD card16:11
AlbertAogra_: which SD cards do you use?16:11
bschaeferand got these 32GB ones and havent had an issue with that since16:11
ogra_AlbertA, totally random ones i buy at the electronics discounter next door16:12
* bschaefer never knew the intricacies of SD cards16:12
ogra_i have some toshibas, sandisk, terratec ... apacer16:12
ogra_whatever they have in the offers when i need new ones, i explicitly dont pick a particular one (to be able to run into issues by my weird choice ... obviously that never works :P )16:14
ogra_AlbertA, well, you can check the card in your PC reader if it is really corrupt16:14
ogra_ppisati, see above ...16:14
ogra_ppisati, i'll cook up a patch that supppresses erase on both cards tomorrow16:15
bschaeferogra_, ill be sure to pack the current one im using plus the other new one i have unopened16:16
bschaeferif you need more testing on those16:16
bschaeferthough hopefully this is enough :)16:16
ogra_well, i should have a kernel snap or complete image for you guys to test tomorrow ... we'll then see if that helps16:16
ogra_up to now it is just a theory anyway16:16
AlbertAogra_: yeah16:17
AlbertAthxs!16:17
ogra_:)16:17
bschaeferogra_, awesome, and thanks!16:17
ppisatiogra_: ack, FWIW i tested on two more sd cards, everything was fine16:28
ogra_ppisati, yeah, i have like 30 cards here and only one that ever has shown such an issue16:29
mupIssue snapcraft#1489 closed: Custom path of desktop file - should not be expanded in prime directory <Created by koppor> <Closed by kyrofa> <https://github.com/snapcore/snapcraft/issue/1489>16:35
mupIssue snapcraft#1506 closed: Inconsistent formatting when opening vs. closing channels <Created by Roadmaster> <Closed by kyrofa> <https://github.com/snapcore/snapcraft/issue/1506>16:35
zyga-ubuntuChipaca: how are you feeling?16:42
Chipacazyga-ubuntu: better, thank you16:42
Chipacahad lunch even16:42
Chipacadid i miss much?16:42
bschaeferhmm is there a bug for this? "snap install <snap_name_not_complete><hit_tab_to_autocomplete> --devmode" turns into "snap install --devmode --devmode"16:45
zyga-ubuntuChipaca: no, I think all is good16:46
zyga-ubuntuChipaca: some discussion around iface repository and connections16:46
ogra_bschaefer, tab completion is Chipaca's baby16:47
Chipacabschaefer: hm, that's a new one16:48
bschaeferChipaca, i mainly hit it when i have mir-kiosk_*snap and mir-kiosk-apps*snap16:48
bschaeferand im install both with --devmode16:48
bschaeferso i just try to auto tab from mir-kisok-<tab> but since --devmode is there seems... to want to reaplce the string with that :)16:49
Chipacabschaefer: I'll take a look, but just for the record a lot of things don't auto-complete properly from the middle of the command16:49
niemeyerackk: Will check it out.. sorry for not replying earlier.. we were in the standup and then I forgot to reply16:49
bschaeferChipaca, yeah figured it was an strange edge case :)16:49
bschaefernot really a huge issue16:49
Chipacabschaefer: it _should_ work, though, and i know to fix it if it doesn't16:49
Chipacaso, i'll take a look16:49
bschaeferChipaca, cool :), tahnks!16:50
mupBug #1666386 changed: Snap apps do not work on Lubtuntu <Snappy:Invalid by zyga> <lubuntu-meta (Ubuntu):New> <https://launchpad.net/bugs/1666386>17:08
naccfor those using LP to build their snap, how are people organizing their repository to deal with stable/edge branches, etc. It seems I can only have one repository <-> package connection, which means I have to publish to edge, and manually promote to stable at the appropriate time? Can I have a stable branch pushing to stable and an edge branch pushing to edge?17:20
=== cachio is now known as cachio_afk
kyrofanacc, to be clear, there are multiple ways to use LP to build snaps. You cannot achieve what you're asking with build.snapcraft.io, but you can if you simply use LP differently17:46
pedronismvo: I think it's ok to close the previous one17:46
kyrofanacc, so from the limitations you've mentioned, I assume you're using build.snapcraft.io, correct?17:47
nacckyrofa: i'm using LP directly17:48
nacc(no b.s.io)17:48
kyrofaOh! Well then yes, you can definitely do as you're asking17:48
nacckyrofa: ok! link to docs?17:48
kyrofanacc, don't make me laugh17:48
nacckyrofa: if i click on the second branch and try to associate it to the same snap, LP says no17:49
kyrofanacc, "snaps" in LP are really snap _recipes_17:49
kyrofanacc, you can only create one per branch, and they must be named differently, but when they're uploaded to the store they're uploaded using the name in the YAML17:49
kyrofanacc, which means they're the same snap17:49
nacckyrofa: yep, that makes sense17:50
kyrofanacc, so you're on the right path17:50
kyrofanacc, create a new snap, called <mysnap>-stable17:50
mupPR snapd#3936 opened: data/completion: small tweak to snap completion snippet <Created by chipaca> <https://github.com/snapcore/snapd/pull/3936>17:50
kyrofaAssociate it with the stable branch, auto-upload to stable, and voila17:50
Chipacabschaefer: ^ :-)17:50
nacckyrofa: oh! so basically lie to LP's recipe creator? :)17:50
nacckyrofa: fun :)17:50
bschaeferChipaca, awesome thanks!17:51
kyrofanacc, whatever it takes :P17:51
nacckyrofa: yeah :)17:51
Chipacabschaefer: if you drop data/completion/snap from that pr into your /usr/share/bash-completion/completions, your issue should go away (and nothing bad could possibly happen™)17:51
nacckyrofa: thannks! i'll try it now17:51
kyrofanacc, you should see the gymnastics we do in the nextcloud snap17:51
bschaeferChipaca, o sweet yeah ill give that a try now17:52
nacckyrofa: are there bugs open for this to be easier?17:52
nacckyrofa: as it seems like a pretty obvious workflow to me17:52
nacckyrofa: do you make -stable -edge or whatever other snaps you create private so they don't show up int he store?17:52
nacckyrofa: they are just placeholders, it feels like17:52
kyrofanacc, I'm afraid I don't know about bugs17:52
kyrofanacc, the snap recipes you create in LP do not show up in the store17:53
kyrofanacc, the names are specific to LP17:53
nacckyrofa: oh right and since there will be no uploads to git-ubuntu-stable, e.g., it will never show up17:53
nacckyrofa: funny17:53
kyrofanacc, there are fields on the recipe that cover "what is this named in the store"17:53
bschaeferChipaca, sweet, confirmed working /me comment17:59
Chipacabschaefer: ta17:59
mupPR snapd#3937 opened: interfaces/udev: only 'trigger --action=change', not 'control --reload-rules' <Created by jdstrand> <https://github.com/snapcore/snapd/pull/3937>18:17
=== cachio_afk is now known as cachio
mupPR snapcraft#1551 closed: dirs: set plugin, schema, and library dir for snap <bug> <Created by kyrofa> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1551>19:35
cachioniemeyer, I am looking at the test ubuntu-core-services which check the status for the different snapd services21:00
cachioi am looking the results on dragonboard and it is failing21:00
cachiobasically snapd.autoimport.service and snapd.snapd.refresh.timer are not active on dragonboard21:00
cachioi am not sure if it is expected or it is an issue21:01
cachioniemeyer, any idea?21:01
niemeyerHmm21:11
niemeyercachio: Refresh timer is not supposed to be active anymore I believe. This is now internal.21:11
niemeyercachio: As for autoimport, I *think* this is actually a once only thing on startup.. it's not supposed to be active the whole time.. but that's been a while ago and my memory fails21:12
cachioniemeyer, ok, i'll ask tomorrow to mvo because the test he just added is failing on ubuntu-core21:13
pedronisalso test themselves manipulate those (maybe in different ways between core and classic)21:14
cachiopedronis, yes, that could be a reason, because i am running with external backend21:15
pedronisanyway mvo is indeed the best person to talk about this21:17
cachioniemeyer, the test-snapd-upower-observe-consumer snap is not available for arm-64 and a test is failing because of that, I don't have permissions to upload that to prod, do you?21:33
niemeyercachio: I don't as well21:51
mupPR snapd#3938 opened: interfaces/opengl: don't udev tag nvidia devices and use snap-confine instead <Created by jdstrand> <https://github.com/snapcore/snapd/pull/3938>21:52

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