/srv/irclogs.ubuntu.com/2018/05/10/#snappy.txt

mborzeckimorning05:05
mupPR snapd#5116 closed: interfaces: move host font update-ns AppArmor rules to desktop interface <Created by jhenstridge> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/5116>06:04
zygaThanks!06:19
mborzeckizyga: hey, pushed more test cases  to 4504 https://github.com/snapcore/snapd/pull/4504#discussion_r18715268306:24
mupPR #4504: snap, wrappers: systemd WatchdogSec support <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/4504>06:24
mborzeckizyga: i think you missed the parenthesis in the if condition06:25
zygaAck06:37
zygaI will check shortly06:37
mborzeckistore hiccups continue today?06:43
zygaplease tell me that isn't so :)06:53
=== pstolowski|afk is now known as pstolowski
pstolowskimorning06:54
zygacześć06:56
zygajak tam? :)06:56
zygaI need to plan an upgrade from artful06:56
zygaI should probably do a clean bionic install from the final iso to see how that is like06:56
zygaand then move my data over06:57
zygajamesh: hey, I merged master into #511506:58
mupPR #5115: interfaces: add xdg-document-portal support to desktop interface <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/5115>06:58
zygais it the final state of that PR, there are some pending comments06:58
zygaalso I recall reading your report on ubuntu-devel, what's the state of the portal work, is it the last branch you expect in this area for now, is there anything buggy or missing from your pop?06:59
zygapov06:59
jameshzyga: I was just updating it now, and running some of the spread tests locally (which failed due to a 503 error from the store)07:01
zygaack, feel free to force push if it makes your life easier, just merge master to get the diff updated please :)07:01
jameshhmm.  there's a new "checks" tab on pull requests07:03
jameshzyga: fwiw, I've been able to run the full file open/file save workflows using these snapd changes07:06
mborzeckiand it's somewhat confusing, there's checks that are run on each PR listed just below the review statuses and a separate checks tab (github apps marketplace?)07:06
zygajamesh: sweet! lots of lots of people will love that07:06
zygajamesh: yes, it's a new GitHub feature apparently07:07
jameshchecks sound like it could be useful for automated syntax checking and the like07:08
jameshor a compile test.07:09
mborzeckiafk for a bit07:09
jameshThe "status" feature is probably still what you want for test suites though07:09
zygayes, but we may opt into the new thing07:10
zygait seems travis now has a unified model where we could pay to get more test slots for FOSS projectsa07:10
mborzeckire07:39
mborzeckiheh error: received an unexpected http response code (503) when trying to download https://api.snapcraft.io/api/v1/snaps/download/b8X2psL1ryVrPt5WEmpYiqfr5emixTd7_1797.snap07:40
mborzeckizyga: don't want to upset you.. but ^^07:40
zygasigh :)07:40
zygawell07:40
zygaweather is weather07:40
zygaand the store is the store07:40
zygamborzecki: can you look at https://github.com/snapcore/snapd/pull/5142 again please07:42
mupPR #5142: many: add "snap debug sandbox-features" and needed bits <Squash-merge> <Created by zyga> <https://github.com/snapcore/snapd/pull/5142>07:42
zygaI updated it and documented the behaviour on the forum07:42
mborzeckizyga: already reviewed it in the morning https://github.com/snapcore/snapd/pull/5142#pullrequestreview-11898555407:44
mupPR #5142: many: add "snap debug sandbox-features" and needed bits <Squash-merge> <Created by zyga> <https://github.com/snapcore/snapd/pull/5142>07:44
zygaoh, indeed, my tab was out of date07:44
zygareplied :)07:46
kjackalhello07:49
kjackalis there a way to fill the snap version at build time?07:49
zygayes07:49
zygaversion-script07:49
zygaAFAIR, google that there are examples around07:50
kjackalversion-script the keyword I was looking for, thanks zyga07:50
zygaogra_: https://github.com/snapcore/pi3-gadget/pull/14/files07:50
mupPR pi3-gadget#14: Update SPI slot definition <Created by tokurz> <https://github.com/snapcore/pi3-gadget/pull/14>07:50
zygaogra_: can you please have a look at this07:50
Chipacamoin moin07:55
zygahey hey07:57
mcphailkjackal: if you're using git you can also pass 'git' as the version: field but it is slightly broken07:57
kjackalthanks for the suggestion mcphail, I am packaging kubernetes so I would like the version in the snap to reflect the version of kubernetes binaries. I will take a look at git07:59
mcphailkjackal: i hit this issue with underscores in the git tags, though: https://bugs.launchpad.net/snapcraft/+bug/176951908:01
mupBug #1769519: Setting the snapcraft.yaml version field to 'git' produces undocumented and broken magic <Snapcraft:New> <https://launchpad.net/bugs/1769519>08:01
mborzeckizyga: mvo is off today?08:02
zygaI think so08:02
* zyga hears a lot of fire sirens08:03
zygahmm08:03
zygaa *lot*08:03
mborzeckizyga: too early for warsaw uprising anniversary08:03
zygafeels like something major nearby08:03
Chipacapedronis: zyga: is there anything useful we can do to not use  a lot of cpu while not making progress when there's something wrong with the seed?08:04
zygacan you give me an example08:04
Chipacamaybe #176789608:04
mupBug #1767896: Live 18.04 with persistence snapd high CPU usage <OEM Priority Project:Confirmed for swem> <snapd (Ubuntu):Confirmed> <https://launchpad.net/bugs/1767896>08:04
Chipacagrah! why does 18.04 not have keyboard shortcuts for menu items >:-(08:05
* Chipaca is glad he hasn't upgraded every time he runs it in a vm08:05
zygaChipaca: because gnome decided they are not desired or something?08:06
Chipacaogra_: do you know how the live cd dance happens? asking because i want to tweak /etc/environment before snapd gets a chance to run08:08
zygahmmmm08:10
zygaChipaca: should we refresh/seed on live CD?08:10
Chipacazyga: yep08:10
Chipacazyga: some work in to making it work, evne08:10
Chipacaeven08:10
zygapedronis said that was not expected to work08:11
Chipacabut the daily livecd broke08:11
* zyga is getting confused08:11
zygastill08:11
Chipaca?08:11
zyga2018-05-09T06:45:35-04:00 ERROR rename /var/lib/snapd/snaps/gtk-common-themes_3.snap.partial /var/lib/snapd/snaps/gtk-common-themes_3.snap: no such file or directory08:11
zygathis is the thing to debug08:11
Chipacait totally is expected to work08:11
Chipacazyga: no08:11
zygaI asked pedronis yesterday and he said no08:11
Chipacai mean08:11
Chipacazyga: the snap not being there is the cause of this bug, yes08:11
Chipacazyga: snapd trying again and again and again is less than smart08:12
Chipacaand that's what burns up cpu08:12
zygayeah, no disagreement there08:12
zygait should slow down exponentially08:12
ChipacaOTOH, if you boot the live cd _with network_, by the ~2nd try it just works08:12
Chipacabecause the download succeeds08:12
Chipacasomehow? i think08:12
zygaoh I saw that08:12
zygaI have a live USB (not persistent) of bionic08:12
zygaand I installed a snap08:12
zygaand it failed08:12
zygathen again08:12
zygaand it worked08:12
zygano idea08:12
pedronisChipaca: zyga: I'm not here, but in budapest live cd people said they had space problems with refreshes, asked how to turn them off, explained about hold, if they didn't do it, not sure whay08:16
zygaah, I see08:16
zygathanks for the clarification08:16
Chipacapedronis: I think they have done it, at least i understood they have and i see something like it in this daily08:17
Chipacapedronis: now go away08:18
mborzeckiChipaca: https://github.com/snapcore/snapd/pull/5091#issuecomment-385631992 you mean to show the log only then, i.e. one week in since the last refresh? right now there's a message logged each time refresh is held by being on a metered connection08:19
mupPR #5091: many: hold refresh when on metered connections <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5091>08:19
Chipacamborzecki: I meant “in a week's time we're going to update *no matter what* so get to some cafe or something you nitwit”08:33
mborzeckiChipaca: ah ok08:34
Chipacamborzecki: also the // TODO use warnings for this08:52
mborzeckiChipaca: pushed some updates to 5091, please rereview08:52
Chipacamborzecki: will make it easier to remember / find them :-)08:52
Chipacamborzecki: looking08:52
mborzeckiChipaca: hah totally forgot about timeutil.Human ;)08:54
Chipacamborzecki: :=D08:54
Chipacauh, that was meant to be a smile08:54
Chipacanot … anything that isn't a smile08:54
mborzeckihm one thing though, i can't drop the time in timeutil.Human right?08:55
Chipacanot with that attitude08:56
pstolowskifor that you need timeutil.Puritan ;)08:56
Chipacamborzecki: you're right08:56
Chipacamborzecki: maybe leave it for now and then add a flag to Human (or find a name for the new thing)08:56
mborzeckiHumanDays08:57
zygawe need Klingon package for i18n08:58
zyga;-)08:59
* Chipaca half-smiled half-groaned09:00
* Chipaca ~> moar coffee09:00
* Chipaca ~> really moar coffee09:02
mborzeckithe review board does not auto update?09:04
zygamborzecki: no, or if so it does so very infrequently09:05
Chipacamborzecki: gustavo mentioned it was stuck09:09
* Chipaca might have his snark levels dialed up too much today09:11
Chipacaa user said something about a bug being a issue *we're* having, and I almost replied "oh, we aren't having it; you are"09:11
zygaheh, someone is in a mood today :)09:12
zygabut that could be me, I have a food poisoning since last night09:12
zygasome cream was added to a sauce I didn't know about09:12
Chipacazyga: have you upset your wfie again09:12
zygasucks to be me today09:12
Chipaca:-)09:12
zygaChipaca: my mother in law09:12
Chipacazyga: I flagged you about something in the forum btw, if you want to pick that up09:14
=== chihchun_afk is now known as chihchun
mborzeckithe weather outside is so nice today09:29
threshmeh, no fonts in Qt apps again09:31
threshzyga, you've asked fro SNAP_CONFINE_DEBUG=1 snap run ... when this happens, so here goes: https://gist.github.com/thresheek/05e0343b19cb1e16b745ebf6162d659909:33
threshalso, good morning09:33
=== chihchun is now known as chihchun_afk
threshand again, it's fine after a reboot09:59
mborzeckithresh:  `/usr/lib/snapd/snap-discard-ns <snap>` should be enough10:03
threshmborzecki, ok, I'll try next time this happens.  What could be the cause?10:04
mborzeckithresh: maybe it's something with how the fonts are made available to snaps10:07
Chipacathresh: do you have an nvidia prime thing?10:11
Chipaca(no, it's not a subscription service from nvidia to get free shipping)10:12
Chipaca(and no it's not condoms either)10:12
threshChipaca, yes I do10:13
threshbumblbee or whatever10:13
Chipacathresh: bumblebee was the old thing that supported per-window, right?10:13
Chipacaanyway, that confuses snapd10:13
threshpossibly10:13
Chipacaevery time i switch from one to the other i need to snap-discard-ns10:14
Chipacanot sure if the fix is to tell snapd about this, or to tell whatever it is that does the switching about snapd :-)10:15
niemeyerMoin10:20
Chipacawho's the publisher of the gtk-common-themes snap?10:21
Chipacai mean, who are the devs10:21
pstolowskizyga: is there any concern still about #5075 ?10:22
mupPR #5075: snap/env: fix env duplication logic <Simple> <Created by didrocks> <https://github.com/snapcore/snapd/pull/5075>10:22
pstolowskihello niemeyer!10:22
Chipacapopey: do you know who pushes gtk-common-themes?10:22
niemeyerpstolowski: Yo o/10:23
popeyChipaca: desktop team own that. kenvandine probably10:23
Chipacakenvandine: O HI10:23
Chipaca6am for him, probably too early10:23
Chipacapopey: thanks10:24
greybacksergiusens: https://bugs.launchpad.net/snapcraft/+bug/1770364 is the issue I was struggling with yesterday10:42
mupBug #1770364: Using 2 remote parts, file not being copied from stage to prime <Snapcraft:New> <https://launchpad.net/bugs/1770364>10:42
mupPR snapd#5124 closed: many: add wait command and seeded target <Critical> <Created by mvo5> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/5124>11:09
niemeyercprov: ping for when you're around11:18
cprovniemeyer: I am in a dentist appt. and will be back in 30m. Urgent ?11:22
ogra_Chipaca, to change livecd defaults you wantz ro change casper and re-build the image11:24
ogra_*want to11:25
Chipacaogra_: I want many things, but that is not one of them11:27
Chipacaogra_: :-)11:27
ogra_Chipaca, well, thats the only sane way to change /etc/environment that i know11:28
Chipacaogra_: then i guess i won't11:28
ogra_casper creates all defaults in the livefs overlay11:28
Chipacaogra_: it was to debug a bug that i've now marked as won't fix, so ¯\_(ツ)_/¯11:30
ogra_(alternative you could hack it in on the fly ... by adding "break=bottom" to the kernel cmdline, injecting it into the "nearly booted" system from the initrd shell and then move on with the boot by exiting the shell11:30
ogra_)11:30
ogra_ah, k11:30
* zyga is unsuited for work today :/11:37
om26eris there a way to set environment variable for an installed snap, maybe using snapctl ?11:37
* zyga files a sick day, sorry folks, I cannot help it today11:48
Chipacaom26er: no11:49
ogra_for testing you can copy the wrapper script to ~/ ... edit it and bind-mount it over the one in the snap ...11:58
ogra_but thats indeed no permanent solution or in any way sane :)11:58
cprovniemeyer: ping, I am back home12:02
Chipacaom26er: what are you needing to do?12:12
om26erChipaca: my snap requires an environment variable exported for a specific feature to work, though I opted for a system-wide export and that did the trick.12:13
Chipacaom26er: it sounds like your snap should be using a config hook :-)12:13
om26erSo not blocked on anything, just wanted to know if that was possible12:13
om26erI'll look into that12:13
zygaom26er: is is a fixed flag?12:13
zygaogra_: like FOO_ENABLE=112:14
kenvandineChipaca, that's my team, jamesh most specifically12:14
om26erzyga: yeah12:14
zygaer om26er ^12:14
Chipacakenvandine: O HI12:14
Chipacakenvandine: :-)12:14
zygaom26er: can you juse add that to environment: in snapcraft.yaml?12:14
kenvandineChipaca, but didrocks does the regular pushes12:14
Chipacakenvandine: #1767896 is why i was wondering12:14
mupBug #1767896: Live 18.04 with broken seed causes snapd high CPU usage <OEM Priority Project:Confirmed for swem> <snapd (Ubuntu):Won't Fix> <https://launchpad.net/bugs/1767896>12:14
om26erzyga: don't want to make it permanent. On some systems exporting that variable is not needed, on other it is.12:14
ogra_zyga, or CRAZY_ENABLE=1, yeah :)12:15
zygais it harmful on the first group?12:15
Chipacakenvandine: daily live cd broken because one of the snaps now depends on something that isn't seeded, and can't be seeded because it's not in the channel12:15
zygaif it is a configuration then yes, use configure hook12:15
kenvandineChipaca, ugh12:15
om26erzyga: its untested, so can't really say. Would probably need a wide testing12:15
zygaif it can be globally enabled then just do so12:15
kenvandineChipaca, i'll open and close the branch12:15
om26erSo it basically tells Android Studio to use system-wide libstd instead of the one that's shipped with its source.12:16
Chipacaom26er: you could always have a wrapper check for e.g. [ -e $SNAP_USER_DATA/.do_the_crazy_thing ]12:19
Chipacakenvandine: will you update the bug when you do?12:20
kenvandineChipaca, done and done12:22
Chipacakenvandine: awe, and some12:25
kenvandine:)12:26
mupPR snapd#5117 closed: interfaces/apparmor: enable apparmor, even if partial <Blocked> <Created by zyga> <Closed by zyga> <https://github.com/snapcore/snapd/pull/5117>12:35
mborzeckizyga: there are conflicts on #509012:37
mupPR #5090: cmd/snap-update-ns: poke holes when creating source paths for layouts <Created by zyga> <https://github.com/snapcore/snapd/pull/5090>12:37
zygaoh12:37
zygathanks12:38
=== devil is now known as Guest68883
mupPR snapd#5145 closed: boot: clear "snap_mode" when needed <Critical> <Created by mvo5> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/5145>13:25
greybackkenvandine: hey, got a sec to look at https://github.com/ubuntu/snapcraft-desktop-helpers/pull/115 ?14:12
mupPR ubuntu/snapcraft-desktop-helpers#115: Fix FONTCONFIG_PATH to point to where we generate fonts.conf <Created by gerboland> <https://github.com/ubuntu/snapcraft-desktop-helpers/pull/115>14:12
Son_Gokuniemeyer: https://lists.fedoraproject.org/archives/list/server@lists.fedoraproject.org/message/XWJPNR2BIT6ZTA5M7OKSW7U6GY5EKQWH/14:12
kenvandinegreyback, saw that, it's in my queue to look at today14:14
greybackkenvandine: ack, thanks!14:14
smosero/14:28
om26erWimpress: Hey! Who owns the name 'axel' on the snap store ? Can we reclaim that ?14:34
* Wimpress goes looking...14:34
om26erI have proposed packaging upstream but they seem to be on a long vacation or something ;)14:34
om26erhttps://github.com/axel-download-accelerator/axel/pull/15614:35
mupPR axel-download-accelerator/axel#156: Add snap packaging <Created by om26er> <https://github.com/axel-download-accelerator/axel/pull/156>14:35
om26erelopio: Hey!14:37
om26erelopio: do you know what will it take to get the Keybase Desktop app as snap ? The packaging you proposed upstream looks to be the CLI client.14:38
Wimpressom26er: That name is not taken as far as I can tell14:38
om26erWimpress: hmm, says "The name 'axel' is already taken."14:39
popeysparkiegeek: ^ help. why does that snap not show in search but snapcraft thinks it's registered?14:40
niemeyerzyga: ping14:52
zygaYes?14:54
zyganiemeyer: pong14:55
niemeyerzyga: Hey14:55
om26erWimpress: is it fine to create a new repository under snapcrafters ?14:59
om26erapparently I can click the 'New' button14:59
Wimpressom26er: Do you want to repo to be called `axel`?15:17
om26erWimpress: yes.15:17
WimpressI can create it and you can submit a PR :-)15:18
om26ersure15:18
Wimpressom26er: https://github.com/snapcrafters/axel15:26
zygaPharaoh_Atem: matthew miller supports fedora base snap :)15:30
popeyom26er: https://news.ycombinator.com/item?id=17038734 - thank you!15:55
om26er\o/15:56
popey:)15:56
popeyThought you might like that :)15:56
om26erpopey: Definitely great to know and fyi android studio neared 10k installs :)15:57
popeyNice!15:59
popeyOne of them is me :)15:59
om26erIts exciting how snaps are fixing real issues, making my life a breeze every day.16:00
popeyThat's great to hear. Looking forward to the next developer tool you need to snap ;)16:00
om26erI have my eyes on https://hub.github.com/16:02
=== pstolowski is now known as pstolowski|afk
kyrofaHey niemeyer are you planning on updating blender to 2.79b?16:24
niemeyerkyrofa: Yeah, definitely, and also need to update 2.816:27
=== blackboxsw is now known as blackboxs
=== blackboxs is now known as blackboxsw
=== prime is now known as Guest32215
mupPR snapd#5148 opened: boot: clear "snap_mode" when needed (2.32) <Created by mvo5> <https://github.com/snapcore/snapd/pull/5148>17:17
mupPR snapd#5149 opened:  many: add wait command and seeded target <Created by mvo5> <https://github.com/snapcore/snapd/pull/5149>17:21
mupPR snapd#5150 opened:  snapd.core-fixup.sh: add workaround for corrupted uboot.env <Created by mvo5> <https://github.com/snapcore/snapd/pull/5150>17:25
om26erWimpress: https://github.com/snapcrafters/axel/pull/119:10
mupPR snapcrafters/axel#1: Add initial packaging <Created by om26er> <https://github.com/snapcrafters/axel/pull/1>19:10
om26erSo I think you will need to register/own that name first to be actually able to publish that to the store.19:11
popeyom26er: i am trying, getting a store problem.19:38
popeyom26er: ok, axel is now registered to snapcrafters20:05
mupPR snapd#5151 opened: tests: use journalctl cursors instead rotating logs <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/5151>20:41
om26erpopey: lets merge that PR and enable builds for it :)20:54
popeyI can't enable builds yet. https://github.com/canonical-websites/build.snapcraft.io/issues/109121:02
noise][FYI https://forum.snapcraft.io/t/new-cdn-for-snap-downloads/5363 - new CDN online, 50% mix at the moment, please post on that forum thread is you see any download issues21:09
mupPR snapcraft#2129 opened: Swap FROM in Dockerfiles from 'xenial' to 'bionic' <Created by felicianotech> <https://github.com/snapcore/snapcraft/pull/2129>22:32

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