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

mborzeckimorning04:56
newbeeAny one tell us, why the "snapcraft pull" command always create "default-jdk and java-open-Jdk" in the "parts/local/install" directory, and also guide us to use oracle-jdk.05:06
newbeeI have a maven project which has source and target versions as JDK 1.8. And I wanted to build a snap app for the same project. As per the doc it looks like snapcraft "will pull a relocatable OpenJDK via the default-jdk Ubuntu package" and since, my project relies on 1.8 building the app fails. Is there a provision through which I can build the project?05:56
mborzeckimvo: morning06:26
mborzeckimvo: i was experimenting with udev a bit yesterday, just to simulate us dropping new rules file to /etc/udev/rules.d06:27
mborzeckiwhen udev reloads the rules and tirggers events, there's a chance bad things will happen06:27
zygaGood morning06:28
zygamborzecki: chance as in randomly?06:28
mborzeckizyga: yeah, looks like so06:29
zygaI wonder if we could run just a fragment of udev rules somehow06:29
mborzeckiloosing sound, pulseuadio killed, libinput going crazy, disks spinning up06:29
mborzeckilibinput thing is most annyoing, all the key repeat settings are gone now and cannot restore them, would probably have to restart the session06:30
mborzecki(xorg shutting down is probably more annyoing, but so far it didn't happen again)06:30
mvohey mborzecki! good morning06:40
mvomborzecki: uh, that sounds nasty06:40
mvomborzecki: any idea what we can do about it?06:46
mborzeckinot yet, need to do some digging still06:46
mvota06:47
=== pstolowski|afk is now known as pstolowski
pstolowskiheyas!07:05
mvohey pstolowski07:06
=== chihchun_afk is now known as chihchun
gohanI have a maven project which has source and target versions as JDK 1.8. And I wanted to build a snap app for the same project. As per the doc it looks like snapcraft "will pull a relocatable OpenJDK via the default-jdk Ubuntu package" and since, my project relies on 1.8 building the app fails. Is there a provision through which I can build the project?07:20
mupPR snapd#5172 opened: cmd/snap-confine, interfaces/opengl: allow access to glvnd EGL vendor files <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5172>07:34
kjackaljdstrand: thank you for th update. Yes lets have a quick chat about this. I also think we will have to ship our own docker build (as we will do for etcd if we want to support other architectures).07:36
b_bhi07:42
pstolowskimvo commnted on your json.Number PR07:43
mvopstolowski: thank you!07:51
mvopstolowski: yeah, this is what I remembered as well. probably a good idea to talk during the standup about the concerns from samuele (aiui we don't use it consistently enough?)07:53
pstolowskimvo: it's possible i missed something as your PR shows...07:54
pstolowskimvo: just mentioning all this in case we want to re-consider the approach; we need to remember of the original bug that triggered the introducion of json.Number, although hopefully the tests should be good enough to prevent a regression07:54
mvopstolowski: yeah, I don't want to go back, just wanted to explore options07:55
mvopstolowski: i.e. if there is a strong opinion that we should do something else my PR would be a bit pointless :)07:55
mvo(but I think there is no such consensus yet)07:56
pedronismvo: pstolowski: as I said we don't have a strong case to do something else atm, otoh I think we discovered there are some people using client as a library, they might be surprised by Number but who knows,  ... it seems something to reconsider if somebody files a bug or when we introduce config schemas07:58
pedronismvo: btw, I will not be at standup today,  I have another meeting  (not a recurring one though)07:59
mvopedronis: +1 - its a good point that it will be surprising for some people07:59
mvopedronis: meeting> thanks, no worries07:59
pstolowskii see08:00
mvoniemeyer: could you please add sil2100 to the standup for the time being? not sure if he should attend every day but he will work with us on core1808:00
pedronismvo: pstolowski: at least in mvo's PR  the use is documented for Conf08:01
pedronisnow08:01
pedronisthe only other issue is Attrs in interfaces but not sure is super used08:02
pedronismvo: I marked #4497 as something you should look at,  it also need a 2nd Gustavo's review08:03
mupPR #4497: many: make rebooting of core on refresh immediate, refactor logic around it <Created by pedronis> <https://github.com/snapcore/snapd/pull/4497>08:03
pedronispstolowski: btw if you did a review of #5169 we could land it08:04
mupPR #5169: snap: fix `snap interface --attrs` output when numbers are used <Created by mvo5> <https://github.com/snapcore/snapd/pull/5169>08:04
pedronisour little hill of PRs seems hard to flatten08:05
pstolowskiyes, merged08:06
mupPR snapd#5169 closed: snap: fix `snap interface --attrs` output when numbers are used <Created by mvo5> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/5169>08:06
Chipacao/08:07
pedronispstolowski: thx08:08
pedronisChipaca: hi08:08
Chipacapedronis: how's things?08:09
pedronisgood-ish08:10
mborzeckineed to go out for a while, bb in 1-2h08:18
Chipacapedronis: am I right in reading that we drop changes after just 1 day?08:33
Chipacai thought it was longer than that08:33
pedronisChipaca: ready changes, yes08:34
Chipacapedronis: context of my question was https://askubuntu.com/a/1037304/71108:36
pedronismvo: I think 5157 and 5158 just needs to get green?   there's a couple of PRs marked for you to review08:57
pedroniszyga: are you working on  #5081 ?08:59
mupPR #5081: interfaces/apparmor: add chopTree <Created by zyga> <https://github.com/snapcore/snapd/pull/5081>08:59
pedronisis there somebody with a PR that needs a review from me? most seems to need Gustavo's input or j-dstrand or rework or are close to merging09:04
zygapedronis: Yes though a bit on the back of the queue09:05
zygaAnd on holidays today09:05
pedronisah09:05
zygaI will apply feedback in the evening09:05
pedronissorry09:05
zygaNo worries :-)09:07
kjackalhi all, when building with snapcraft cleanbuild the lxc ubuntu image is a xenial one? or is it whatever the host system is?09:19
popeyxenial09:19
kjackalthanks popey09:20
popeynp09:20
mborzeckimvo: shall we land #5158?09:22
mupPR #5158: boot,partition: improve tests/docs around SetNextBoot() <Simple> <Created by mvo5> <https://github.com/snapcore/snapd/pull/5158>09:22
pedronismborzecki: it looked ok tome09:26
pedronis*to me09:26
mupPR snapd#5157 closed: many: improve `snap wait` command <Created by mvo5> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/5157>09:26
mupPR snapd#5158 closed: boot,partition: improve tests/docs around SetNextBoot() <Simple> <Created by mvo5> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/5158>09:29
mvopedronis: thanks, I get to the reviews now09:44
mupPR snapd#5077 closed: overlord/snapstate,overlord/auth,store: coalesce no auth user refresh requests <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/5077>09:45
b_b'k finally got a working snap of timeline09:55
b_bhttps://gist.github.com/brunob/1155589f66171992c8661dd8a51d855509:55
b_bhave to use this trick https://forum.snapcraft.io/t/add-external-resources-into-final-snap/990/209:55
b_bsince the zip of the app doesn't provide a setup.py file09:55
b_bi was hoping to get it working with a dump of the zip downloaded, but no luck09:56
b_bcf https://gist.github.com/brunob/1155589f66171992c8661dd8a51d8555/ad3e38dd591969211daf59404c6b3e60495d069e09:57
b_bif anybody have advices or recommandations on this i'll be glad09:57
Chipacab_b: my advice would be to ask in forum.snapcraft.io10:01
b_b'k, thx Chipaca10:02
mupPR snapd#5165 closed: tests: fix user mounts test for external systems <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5165>10:11
mvomwhudson: hey, do you build the go snap for ppc64el ? and if not, would it be possible to enable it there? we use it now as part of our autopkgtest suite10:13
mwhudsonmvo: i do10:13
mvomwhudson: hm, I get a "funny" error there https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-cosmic/cosmic/ppc64el/s/snapd/20180516_134211_9b965@/log.gz10:14
mvomwhudson: "/snap/go/1877/gowrapper: line 3: /snap/go/1877/bin/go: No such file or directory"10:14
popeydiddledan: have you seen "IsADirectoryError" on corebird before?10:18
popeydiddledan: https://launchpadlibrarian.net/370632559/buildlog_snap_ubuntu_xenial_amd64_c0aad5cf3f1a1163b9cfe834a79080dc-xenial_BUILDING.txt.gz10:18
popey"IsADirectoryError: [Errno 21] Is a directory: '/build/corebird/stage/usr/share/locale'"10:18
b_b"sorry, new users can posts 2 links only"10:20
b_bnice...10:20
threshyeah the forum is a weird thing10:20
popeyit's anti-spam protection10:21
b_bhttps://forum.snapcraft.io/t/snapcraft-a-prebuilt-python-app-no-setup-py-file/548210:21
threshis it needed if all the links are nofollow? (are they?)10:21
popeyjust break the url, and post to the forum, give me the link and I'll fix it10:21
b_bdone10:21
popeyhah, it thinks setup.py is a url :)10:21
mvomwhudson: I think I know what is going on /lib64/ldso is not a relative link10:24
mvomwhudson: I can fix that in core10:24
mwhudsonmvo: uhh there were snapcraft bugs in this area but they got fixed i thought...10:24
* mwhudson is not really here btw10:24
mvomwhudson: thank you and sorry for the noise, I think I know what to do do fix this10:25
mwhudsonmvo: ok :)10:27
mupBug #1771793 opened: Interface for hardware temperature monitoring <Snappy:New> <https://launchpad.net/bugs/1771793>10:33
mborzeckire10:37
mupPR core#88 opened: hooks: make /lib64/ld64.so.2 relative to unbreak classic snaps on ppc64el <Created by mvo5> <https://github.com/snapcore/core/pull/88>10:37
b_bthe problem was with the 3rd url popey : https://forum.snapcraft.io/t/add-external-resources-into-final-snap/99010:48
b_bso i broke it10:48
Chipacapopey: b_b: backtick-quote things that it thinks are urls, fwiw10:52
sitterthere is something seriously wrong with Qt 5.10 compatibility due to the newer/different syscalls it uses compared to earlier versions. e.g. `snap install --candidate kde-frameworks-5 && snap install okular && snap run okular` lots of write errors by snapd apparently denying linkat syscalls which are used inside qtemporaryfile which in turn is used for just about config file writing business (e.g. https://github.com/qt/qtbase/blob/10:57
sitterfe5edcee602f0ab2912bbdd1a21f4309ed7dbfd6/src/corelib/io/qtemporaryfile.cpp#L466 ) I think this ultimately is also the problem here https://forum.snapcraft.io/t/qt5-kde-integration-in-vlc-snap-is-broken/546510:57
sitterseems fairly major, but I really don't know what to do with this :S10:57
sitterwell, when I say fairly I actually mean: this is preventing any and all KDE snap builds moving forward10:58
=== chihchun is now known as chihchun_afk
Chipacasitter: raising the issue in a forum topic, with a sampling of the denies, would be a good step towards fixing it11:01
Chipacasitter: we've historically been rather good at fixing these11:02
eraserpencilis it possible to have run snapcraft across several cores?11:02
ChipacaI'll tag jdstrand on that vlc+qt5 one11:03
Chipacaeraserpencil: by cores, you mean cpus?11:05
Chipacaeraserpencil: (cores is a rather loaded term...)11:05
mborzeckisame as 'classic' for that matter11:06
eraserpencilChipaca: I guess cores, or threads. Not sure of the accurate term.11:06
mborzeckieraserpencil: are you doing container builds?11:06
eraserpencilI thought CPU was the actual physical processor11:07
eraserpencilnope11:07
Chipacaeraserpencil: snapcraft itself shouldn't be cpu-bound, and you should be able to pass any arguments the things inside it need to run in parallel11:07
mborzeckieraserpencil: make -j and so on11:08
Chipacayeah11:08
eraserpencilahh kk11:13
eraserpencili'll give it a try11:13
eraserpencilsorry, I might need a more detailed hot-to11:17
eraserpencilhow-to*11:17
sitterhttps://forum.snapcraft.io/t/qt-5-10-linkat-denials-broken-kde-snaps/5484 about the qt 5.10 issue11:19
Chipacasitter: thanks11:26
b_btrying to get a minimal gtk theme for my app now11:28
b_bshould i use https://github.com/snapcrafters/gtk-common-themes ?11:29
b_bor use stage-packages like here https://gitlab.gnome.org/Community/Ubuntu/gnome-software/blob/35d29d0b6188c6b1d60644af30d18f91e773eda2/snap/snapcraft.yaml#L16311:29
jdstrandChipaca: I was already planning on looking at that11:31
=== pstolowski is now known as pstolowski|lunch
pedronisjdstrand: hi11:33
mvoa review for https://github.com/snapcore/core/pull/88 (trivial) would be great11:34
mupPR core#88: hooks: make /lib64/ld64.so.2 relative to unbreak classic snaps on ppc64el <Created by mvo5> <https://github.com/snapcore/core/pull/88>11:34
mborzeckimvo: why does it break with an aboslute path in the symlink?11:39
mvomborzecki: the go snap sets the elf interpreter to /snap/core/current/lib/ld64.so.211:39
mvomborzecki: but that links to a libc6 that is not necessarily available on the system11:40
mvomborzecki: eh, to an ld.so11:40
ogra_jdstrand, geez ... stop fixin security holes ... i only updated my packageproxy snap on the weekend for new curl :P11:40
mupPR core#88 closed: hooks: make /lib64/ld64.so.2 relative to unbreak classic snaps on ppc64el <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core/pull/88>11:41
mupPR snapd#5171 closed: ifacestate: unify reconnect and autoconnect methods <Simple> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5171>11:43
jdstrandogra_: it isn't me, it our awesome security team :)11:52
jdstrandcc ratliff ^ :)11:52
ogra_:D11:52
jdstrandit is pretty cool seeing those go out :)11:52
jdstrandpedronis: hey :)11:52
ogra_yeah, totally !11:52
jdstrandI set up all my little snaps with LP builds so I could get those :)11:53
mborzeckiheh udev reload && trigger makes x session cry12:17
ogra_mborzecki, yeah, dont do that ... you can limit trigger to certain subsystems12:42
ogra_(forgot what the option ws called ...)12:42
mborzeckiogra_: snapd does it currently when a snap is installed and it comes with some udev rules, it's control -R && trigger, i suspect it's when the x session may occasionally die as it happened on me a coulpe of times already12:44
pedronisthere's a zoo of filtering options to trigger but we aren't using them12:44
mborzeckihm maybe interface should specify a subsystem at least12:45
mvoChipaca: do you have an opinion on #5134?12:53
mupPR #5134: Shrink image generated with snap prepare <Created by kubiko> <https://github.com/snapcore/snapd/pull/5134>12:53
Chipacamvo: let me look12:53
Chipacaah. it's green now? i'll look properly then :-)12:54
ogra_mborzecki, well, snapd should inspect what subsystem the rules are for and only re-trigger that one ... blindly re-running all rules and uevents (which is what trigger does) can dremove and re-add devices ... xorg might simply not expect that12:55
ogra_-d12:55
ogra_mborzecki, it can get really nasty if udev decides to re-init your HDDs12:56
ogra_(because you didnt exclude the block devices)12:56
=== pstolowski|lunch is now known as pstolowski
mvoChipaca: not sure if it is, I merged master and addressed the review from zyga12:57
mvoChipaca: but it does lok interessting12:57
mborzeckiogra_: hehe, it does trigger the disks to spin up, also yday my sound card went away, pulseaudio died and pcspkr started working instead12:57
ogra_yeah12:57
mborzeckiwell, at least there's something to investigate now12:58
zygaIMO the question about udev is that it fails -sometimes-13:02
zygaWhy is that?13:02
niemeyerzyga: Are you holiday? :)13:03
niemeyeron13:03
zygaYes13:03
niemeyerzyga: Doesn't look like it!13:03
niemeyer:P13:03
zygaHeheh13:03
zygaTaking a break13:03
zygaWe arrived at 2am13:03
zygaAnd woke up early13:03
mupPR snapcraft#2140 opened: lp 1768233 workaround for plainbox_provider plugin <Created by jocave> <https://github.com/snapcore/snapcraft/pull/2140>13:06
skomorokhIs a "classic" confinement snap just classic in the sense that it runs unconfined as the user when they load the app or does it also let the package maintainer run scripts as root like a .deb or similar?13:57
b_bit seems override-build is what i'm looking for14:09
b_bmaybe i should put after: [desktop-gtk2] in a specific part14:10
b_bcause everytime i clean my snap the build of gtk take so long14:11
mborzeckimy snap graphics-debug-tools-bboozzoo has been flagged for manual review because i'm not shipping *.desktop files, can someone approve it?14:33
mvomborzecki: sure14:34
mborzeckiheh installing & removing this snap, pcspkr is back :)14:34
mborzeckimvo: thanks!14:34
mvomborzecki: there you go14:35
mborzeckimvo: ta14:35
mborzeckiChipaca: do you still have that nvidia laptop?14:38
mupPR snapd#5172 closed: cmd/snap-confine, interfaces/opengl: allow access to glvnd EGL vendor files <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5172>14:40
mupPR snapd#5173 opened: snap: some doc comments fixes and additions <Created by pedronis> <https://github.com/snapcore/snapd/pull/5173>14:47
Chipacamborzecki: i do15:11
mborzeckiChipaca: is it on 18.04?15:11
pedronisChipaca: your input on #5173 would be appreciated  (it's small but interesting, apparently)15:15
b_b++15:16
mupPR #5173: snap: some doc comments fixes and additions <Simple> <Created by pedronis> <https://github.com/snapcore/snapd/pull/5173>15:16
diddledanniemeyer: and jdstrand: I've updated the optical-drive write support PR I filed - take a look when you get a break from other things: https://github.com/snapcore/snapd/pull/436915:17
mupPR #4369: add write permission to optical-drive interface <Decaying> <Created by diddledan> <https://github.com/snapcore/snapd/pull/4369>15:17
jdstrandyes, I'm looking at it now15:17
diddledan\o/15:17
diddledanthe tests pass locally (unrelated tests elsewhere are broken but the tests on the optical-drive interface pass)15:18
diddledanI can't decide whether I really love golang or whether it is too different for me to understand properly :-p15:22
roadmrjdstrand: heads up! I added a task for click-reviewers-tools for bug 177184015:23
mupBug #1771840: snap upload rejected with "refresh-timer" interface attribute <Canonical Click Reviewers tools:New> <Snap Store:New> <https://launchpad.net/bugs/1771840>15:23
roadmrkyleN: ^^ relevant to you as well15:24
kyleNack15:24
jdstrandroadmr: I'll have something for you a bit later15:29
roadmryay jdstrand \o/ thanks15:29
* cachio_ lunch15:39
Chipacapedronis: all the +1's15:41
Chipacapedronis: if you see a message about a '.' at the end, i deleted it; i can't read diff :-)15:41
diddledanjdstrand: looks like I might be dereferencing a pointer somehow - my golang probably sucks :-p `if err := plug.Attr("write", &write); err == nil && write == true {` <-- I think `write` is a nil pointer here16:03
diddledanI can change the `write == true` test to put it inside the block as a nested `if`. That should fix it?16:04
mupPR core#38 closed: Add another pi-config option <Created by sergey-borovkov> <https://github.com/snapcore/core/pull/38>16:05
mupPR core#83 closed: move most of the ubuntu-core config deb into the snap snap build <Created by mvo5> <https://github.com/snapcore/core/pull/83>16:05
mupPR core#38 opened: Add another pi-config option <Created by sergey-borovkov> <https://github.com/snapcore/core/pull/38>16:06
mupPR core#83 opened: move most of the ubuntu-core config deb into the snap snap build <Created by mvo5> <https://github.com/snapcore/core/pull/83>16:06
=== pstolowski is now known as pstolowski|afk
Chipacadiddledan: if write were a pointer, it wouldn't compile16:44
diddledanit's ok, jdstrand pointed out a way to eliminate the whole block in the github comments :-)16:44
Chipacaheh16:44
mupPR snapcraft#2141 opened: jhbuild plugin: allow running as 'root' <Created by diddledan> <https://github.com/snapcore/snapcraft/pull/2141>17:19
diddledandamn that guy, diddledan, he's a nutter17:19
diddledantoo many commas17:20
* diddledan tries again17:20
diddledandamn that guy diddledan, he's a nutter!17:20
sergiusensdiddledan: thanks for that jhbuild fix!20:55
mupPR snapd#5174 opened: interfaces/default,process-control: miscellaneous signal policy fixes <Created by jdstrand> <https://github.com/snapcore/snapd/pull/5174>21:13
=== matteo| is now known as matteo
mupPR snapd#5175 opened: systemd: mock useFuse() so testsuite passes in container via lxd snap <Created by jdstrand> <https://github.com/snapcore/snapd/pull/5175>22:54

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