/srv/irclogs.ubuntu.com/2019/06/06/#snappy.txt

mborzeckimorning04:56
mborzeckibrb, new kernel05:00
mborzeckiarch packages updated05:46
zygaHi05:56
zygaNice :-)05:56
zygaBiking home, 15 min05:57
mborzeckizyga: hey06:10
jameshzyga: thanks for the review comments on the user session agent branch yesterday.  It looks like this will need to be a separate process to userd though :-(06:16
jameshnot a huge deal, but means we don't have everything sitting under one roof.06:17
zygare06:38
zygajamesh: hey, because of the second socket?06:38
zygajamesh: I thought that system can model that06:38
zygajamesh: I'm catching up with the comments on the PR06:41
zygajamesh: ah, is it because of xenial's systemd user oddity?06:43
zygajamesh: hmm, that's unfortunate06:43
zygahmm hmm hmm06:44
zygajamesh: thank you for iterating on the branch06:44
zygajamesh: it would be good to try to dig why systemd behaves this way on 16.04 and if that is something we can remedy, either on snapd side or by changing the distribution package a little06:44
zygamborzecki: yesterday I implemented prediction of event propagation for snap-update-ns06:44
zygamborzecki: I need to tweak it some more and add more tests but it is promising06:44
mborzeckizyga: nice, let me know when you ahve something to review06:45
zygamborzecki: thank you, it doesn't pass all tests yet so I need to address that first06:46
mborzeckimhm06:46
zygamborzecki: maybe it is a dead end but it is seductive that it doesn't require major changes in s-u-n06:46
mborzeckizyga: see, at least you did something useful, i instead was at a meetup about migration from c++11 to c++14, not sure if that was the presenter's goal, but i left with the conclusion that you should not let clerks and lawyers design a language06:50
zygahahaha06:50
zygawhat's the issue with C++ 11 -> C++1406:50
zygaI recently used a C11 feature and I loved it (not C++)06:50
zygastandard atomics06:50
mborzeckizyga: s/is/are/06:51
mborzeckizyga: i'm still waiting for a meetup about some large codebase introducing c++17 that should be even more questionable fun06:52
jameshzyga: it would probably be possible to update the dbus upstart job to do a "systemctl --user set-environment" call, but I don't know how likely it would be for that to be accepted07:01
=== pstolowski|afk is now known as pstolowski
pstolowskimorning07:01
jameshzyga: also, I suspect we'd have problems due to two ways for "snap userd" to be started: directly by dbus-daemon if a D-Bus call comes in, or managed by systemd for socket activation07:02
zygajamesh: I think it's worth checking to ensue consistency of the snap platform07:02
zygammm07:02
zygajamesh: tricky business, thank you for staying on top of that07:02
zygagood morning pstolowski07:02
mborzeckipstolowski: heyah07:02
jameshzyga: and I suspect the the dbus-daemon invoked userd won't be able to take over the socket systemd created07:02
jameshzyga: in contrast, having two processes should work the same everywhere.07:03
zygajamesh: true though I wonder if we should design this so that a OS version on the way out dictates how it behaves everywhere in the future07:03
jameshzyga: is there much benefit to multiplexing it all into a single process though?07:05
zygajamesh: that I don't know, my whole point is we should be thoughtful about the design07:05
zygathere may be good reasons to do either approaches07:06
jameshzyga: the systemd env on xenial is pretty bare.  So in addition to not seeing the session bus, user units won't see the display07:11
zygajamesh: please capture that in the thread, I think it is relevant for reviewers07:11
zygabrb07:59
zygare08:10
pedronismborzecki: hi, I looked again at #668008:34
mborzeckipedronis: hi, thanks for the the comments08:35
Chipacamup_: you ok there?08:44
Chipacaniemeyer: can you tickle mup? it's feeling down08:44
Paddy_NIIf I wanted others to test an app that I have snapped what would be the best practise for sharing the "snap" package?09:53
jameshPaddy_NI: do you care who in particular tests it?09:55
Paddy_NINot at all09:55
Paddy_NI:-)09:55
jameshif you plan to eventually release it on the store, just register its name and publish the snap to the beta channel09:55
jameshit won't show up in search results, and you can just tell people to run "snap install --beta myapp"09:55
jameshwhen it's ready for general use, publish to stable09:56
Paddy_NIOkay, well the app is under MIT license and it's not mine.  I have reached out to the developer via email and he has not responded.  I just feel uneasy about publishing someone else's app.09:56
Paddy_NII am also not sure just how "secure" or "safe" this app is, I have been using it for a very long time and think it's great but my own personal experience is not of much value when it comes to the safety of others09:58
jameshyou've to two options then: (1) publish it with a different name (e.g. suffix it with your own name), or (2) register it with the real name and get the name transferred later if the upstream wants to take over.09:58
Paddy_NIOkay, I might just add my name that seems like less hassle :-)09:58
Paddy_NIThank you jamesh09:58
Paddy_NIWould I add my name as a suffix to the snapcraft.yaml too?09:59
jameshyes.  The "name" field in the snapcraft.yaml should match what you're publishing it as.10:00
Chipacajamesh: beta does appear in search results10:00
Paddy_NIOkay good to know :-)10:00
Chipacajamesh: from 'snap find' that is10:00
jameshChipaca: oh?  I thought it was only stable10:00
Chipacawon't appear in gnome software10:00
Chipacabut will appear on the terminal10:00
Chipacajamesh: search is now broad by default10:00
Chipacajamesh: (old behaviour is --narrow)10:00
ChipacaPaddy_NI: it's probably best to register the final name10:01
ChipacaPaddy_NI: otherwise, people will get stranded10:01
jameshChipaca: thank you for the review comments yesterday.  Converting to the new http.Server Shutdown() method turned out to be pretty easy10:01
ChipacaPaddy_NI: that is, say you register fooapp-paddy10:01
ChipacaPaddy_NI: people install it and love it10:02
ChipacaPaddy_NI: fooapp notices and starts doing their own, official, snap10:02
ChipacaPaddy_NI: so you stop maintaining fooapp-paddy10:02
ChipacaPaddy_NI: the users of which are now abandoned10:02
Paddy_NIThat does not sound ideal10:02
ChipacaPaddy_NI: instead, if you grab fooapp, when they notice and want to pick it up it's a forum post and a couple of emails to get it transferred10:02
Paddy_NIChipaca, Okay I'll do that instead :-)10:03
ChipacaPaddy_NI: it is a little bit more work for the handoff, but it doesn't suck for users as much when things go well :)10:03
Chipacago well == upstream picks it up10:03
Paddy_NIThat makes lots of sense10:04
Chipacabut we've tried to make it easy (as popey and/or Wimpress can attest)10:04
Chipacajamesh: i'm glad that refactor is easy as i've been dreading it for daemon itself :010:04
Chipacameant to write :) but :0 is also appropriate10:04
Paddy_NIThis app can be used in conjunction with other applications using "--vlc" or "--mpv" or whatever really, is there a "plug" for this functionality as I am pretty sure it does not work at the moment in my snap?10:06
jameshChipaca: https://github.com/jhenstridge/snapd/blob/user-session-agent/userd/session_agent.go#L121 <- it's just a few lines longer, plus ignoring http.ErrServerClosed in the Serve goroutine10:06
pedronisChipaca: there should be an explicit way to ask not to appear in searches though10:07
Chipacapedronis: yes but AFAIK that's not implemented yet?10:07
Chipacapedronis: at some point there was talk of not showing things that were only in edge, but I don't think that got done either10:08
Chipacae.g. 'snap find wine' finds 'paladins' which is edge-only10:08
pedronisno now find gets everything10:09
pedronisbut I thought we added a flag to hide from search10:09
Paddy_NII wonder if all those plugs are necessary or if there is some overlapping, here is the contents of my yaml https://paste.ubuntu.com/p/8ngT32WpFQ/10:10
ChipacaPaddy_NI: no plugs for vlc nor mpv, and typically those things would get bundled in a snap (unless it's a classic-mode snap which has its own drawbacks)10:12
zygaafk for a moment10:13
Paddy_NIAh I see, so I would need to choose for the user.  It would be a bit mad to bundle in VLC, MPV etc10:13
pedronisChipaca: it's there, got to https://snapcraft.io/YOUR-SNAP/settings10:13
Chipacapedronis: ooh nice10:13
pedronisremember that nowadays most publisher UI is in the store10:13
Paddy_NIPerhaps then the snap would have the bundled player suffixed to the name10:13
Paddy_NI"peerflix-mpv"10:14
ChipacaPaddy_NI: there's an "unlisted" option in the 'settings' tab of your snap on the web10:14
ChipacaPaddy_NI: as pedronis pointed out10:14
Chipacapedronis: I didn't know that was implemented, that's neat10:15
Paddy_NIChipaca, Alan Pope mentioned that in one of the snapcraft.io videos on youtube10:15
Chipacapsh, what does he know10:15
Chipaca:-þ10:15
Paddy_NII just wasn't sure about etiquette10:15
Paddy_NI:-)10:16
Chipacapedronis: we should make all the test-snapd-* snaps unlisted (unless we use them for 'snap find' integration tests)10:16
ChipacaPaddy_NI: wrt plugs etc that sounds more like a forum topic so you can get the right people's time to look10:17
pedronisChipaca: maybe10:17
ChipacaPaddy_NI: some of those are gnarly interfaces10:17
Paddy_NIChipaca, I was thinking that, I am very hesitant about sharing this with people10:18
* Chipaca is people too!10:18
Paddy_NII also noticed an "issue" on the github page which has went unaddressed10:19
Paddy_NII'll stick it on the snap store unlisted and see who wants to try it10:19
* popey_ looks at Chipaca -_-10:20
jameshPaddy_NI: I wouldn't expect a bittorrent client to need firewall-control, network-control, or fwupd10:20
jameshat a minimum10:20
jameshtry removing them and see if your snap still functions10:21
Chipacajamesh: that's probably for poking holes to get incoming connections10:21
Paddy_NIjamesh, It streams the video over your lan you see10:21
Chipacabut i dunno10:21
Paddy_NIUnless that still does not matter10:21
* Chipaca ignores popey_ and hopes for the best10:21
Paddy_NISo for instance I would do 'peerflix "magnet_link_here"' and I can stream the video or whatever using the ip address of the computer being used on port 888810:22
Paddy_NIVery handy10:22
jameshPaddy_NI: fwupd has nothing to do with firewalling10:22
Paddy_NIOkay I'll remove it and continue testing10:23
jameshit is a tool for updating firmware10:23
jameshin general you should ask for the minimal set of permissions the app needs, rather than just adding them just in case10:24
Paddy_NIIf I don't use the "home" plug then I cannot do 'peerflix path/to/torrent.torrent'10:24
jameshthat one is fine10:24
Paddy_NIjamesh, I started with none10:24
Paddy_NIThe output from snapcraft security log told me those10:24
jameshPaddy_NI: if the app runs unconfined as a regular user, it probably doesn't need the *-control plugs10:25
Paddy_NIOr rather those snappy-debug.security scanlog10:25
Paddy_NIcool10:25
Paddy_NIOkay I have lost count now of how many times I have watched the opening sequence to "Big Buck Bunny"10:30
Paddy_NIThe app works minus the aforementioned plugs. Thanks guys :-)10:31
Paddy_NIBundling in mpv would probably be wise10:31
Paddy_NIIt's not how I would use it but that's irrelevant I suppose.10:32
ChipacaPaddy_NI: there was somebody else looking into bundling mpv just yesterday10:32
ChipacaPaddy_NI: check the channel logs :)10:33
Paddy_NIOh excellent :-)10:33
Paddy_NIChipaca, I've just checked the logs from 4th, 5th and 6th of June. No mention of mpv.10:39
Paddy_NIMaybe I'm just stupid10:39
ChipacaPaddy_NI: i started searching myself as well and can't find it either10:39
Chipacamaybe i imagined it ¯\_(ツ)_/¯10:39
Paddy_NIChipaca, lol10:40
Chipacawonder how10:40
Paddy_NII'm going to google a bit10:40
ChipacaPaddy_NI: https://forum.snapcraft.io/t/classic-confinement-request-syncplay/1118910:41
ChipacaPaddy_NI: not on irc :)10:41
Paddy_NIAh great, cheers Chipaca10:41
Paddy_NII suppose there is tons of work going on with regard interfaces "plugs"?  It would be nice if there was a generic media player "plug" although that's most likely easier said than done.10:48
Paddy_NIIf someone snaps a web browser and the user installs and uses this browser then clicks on say a "mailto" link is that able to summon the external mail client?10:50
ChipacaPaddy_NI: yes10:58
ChipacaPaddy_NI: through a magic proxying of xdg-open10:58
* Chipaca ⇝ runch10:58
Paddy_NIChipaca, I suppose this works the same for magnet links etc?10:58
ChipacaPaddy_NI: i honestly don't know10:58
Chipacawhen we wrote the first iteration, no10:59
Chipacabut it's seen work since and i lost track of it10:59
Paddy_NIChipaca, That's no worries, I wonder if the same process is all that would be needed for peerflix10:59
Chipacasounds reasonable to me10:59
Paddy_NIultimately it's just telling the media player to open "http://localhost:8888"10:59
Chipacajamesh: is that something you worked on? ^10:59
jameshChipaca: I don't think the existing portals APIs will help much here.  It'll probably just open the URL in a web browser too11:01
Paddy_NIAh I see11:01
ograPaddy_NI, if you cant see big buck bunny anymore ... https://durian.blender.org/download/ is also fine for testing11:04
Paddy_NIlol11:04
Paddy_NIThanks for that :-)11:04
ogra:)11:04
Paddy_NI@ogra, I am using this list https://webtorrent.io/free-torrents11:05
Paddy_NIAmazing level of work has went into those videos, blows my mind.11:05
ograah, there is sintel on the list nevermind then11:06
Paddy_NI@ogra, I specifically need magnet links and torrent files to test this any how :-)11:07
ograah11:07
Paddy_NIIt would be shoddy of me to stick peerflix on the store in it's current form as it is technically broken.11:08
Paddy_NII could try moving on to "mps-youtube" however I believe I shall be facing the exact same issue with regards calling external media players11:09
Paddy_NIWhich then brings me back to electing one that I bundle in11:09
Paddy_NIMPV of course would be my desired player but this might not be true of others.  Also "mps-youtube" typically only works if both it and "youtube-dl" have been installed via "pip3"11:10
Paddy_NIInstalling youtube-dl via apt creates problems with paths, mps-youtube expects it to be somewhere else.11:11
ograwell, you can trivially add mplayer to your stage-packages and use that ...11:12
Paddy_NI@ogra, oh I like that idea11:12
ogralike i do here in my wasp player https://github.com/ogra1/video-kiosk-snap/blob/master/snap/snapcraft.yaml ... https://github.com/ogra1/video-kiosk-snap/blob/master/launcher11:12
Paddy_NII might actually try that now with peerflix actually11:12
Paddy_NI@ogra, Thank you :-)11:13
Paddy_NIWe are getting crazy amounts of rain here, I find it quite soothing.11:15
pedronisChipaca: mborzecki: as I said I don't think checking for nil in constructors is that typical11:23
mborzeckioff to pick up the kids12:04
zygaI have a chaotic day, feel like is should take the day off to be fair12:33
zygaSorry. I’m logged around to help with paperwork I must do in person12:33
=== ricab is now known as ricab|lunch
zygamborzecki: I’ll skip standup, in traffic now12:52
pedronisChipaca: standup?13:00
zyga(Not attending now, crying baby, in traffic)13:01
zygaback in the office13:13
zygathey are shooting a film outside again13:13
pstolowskizyga: too bad it's not GoT ;)13:16
zygapstolowski: my character would die in episode two13:17
zyga;)13:17
Chipacapedronis: dunno if you saw that I addressed your feedback from #695113:38
pedronisChipaca: yes, but no, thank you, I put it back into my queue13:40
ograChipaca, with snap interfaces being deprecated, whats the replacement that will show me all available interfaces on a UC install (for GPIO or I2C i need to find the exact name, snap interfaces properly lists that)13:43
Chipacawat13:43
Chipacaoh you mean 'snap interfaces'13:43
Chipacaphew13:43
ograeh, yes, thats why i wrote snap interfaces :)13:44
Chipacaogra: snap interfaces aren't deprecated13:44
Chipacaogra: snap interfaces is deprecated13:44
Chipacathese two sentences are both true13:45
ogralol13:45
Chipacathe plural is important13:45
ograok13:45
Chipaca:)13:45
Chipaca'sanp interfaces', the command, is deprecated13:45
ograwell, i obviously mean the deprecated command13:45
Chipacai read your original question as that plugs/slots ("interfaces") were deprecated13:45
Chipacaobviously13:45
Chipacain retrospect13:45
Chipacaanyway13:45
Chipacazyga: ^^^13:45
zygammm?13:46
zygahaha13:46
ograsomething like "snap connections --available-plugs" or so would be nice13:46
Chipacazyga: snap interfaces for properties13:46
=== ricab|lunch is now known as ricab
Chipacaogra: this is specifically about the properties of the interfaces, right?13:46
zygasnap interfaces for which properties?13:46
ograerr ... --available-slots indeed, sorry13:46
Chipacaogra: maybe 'snap interface --attrs'?13:46
zygasnap interface --attrs shows stuff13:47
ograChipaca, this is simply to find all offered slots on a UC device13:47
zygabut I really think we want snap <plug,slot> instead13:47
ograwell, i dont know the slot name and i dont know what slots are available13:47
ograso i cant really hand the command a slot name13:48
zygaogra: so you want smart tab completion that shows you what is available, in a way?13:48
ograno13:48
zygaI agree it'd be nice to say snap what-can-I-connect-to <plug-or-slot>13:49
ograi just want *some way* to list the available slots on a device13:49
zygaogra: snap slots is that13:49
ograno need for fancy tab completion13:49
zygait's just not implemented13:49
zygabut it's agreed upon13:49
zygamaybe we can do a pass for .4113:49
ograogra@stream:~$ snap interfaces|grep i2c13:49
ograpi3:i2c-0                 -13:49
ograpi3:i2c-1                 -13:49
ograpi3:i2c-2                 -13:49
Chipacasnap install zyga13:49
Chipacazyga.slots13:49
zyga:D13:49
ograsomething that gives me that output13:49
zygaChipaca: I have one slot but it says "insert A/C"13:49
ogra(even unfiltered)13:49
Chipacaogra: what's 'snap connections system'?13:50
Chipacaogra: (at the same time maybe it's premature; interfaces isn't going away yet)13:50
ograthat only seems to list a subset13:50
ogranone of the gadget ones13:51
zygaogra: it shows implicit slots13:51
ograyes13:51
zygaogra: it shows slots that are not associated with any installed snap13:51
ograsnap connections pi3 ... that works13:51
ograan internal alias would be cool here "snap connections gadget"13:52
Chipacaogra: also note --all (dunno if it helps, don't have fancy hardware here)13:52
* Chipaca hugs his xeon before it gets offended13:52
ogra--all also shows a subset but at least includes the gadget ones13:53
ograthanks, i think i can go with that ... perhaps the "snap interfaces" error should point to some possible alternatives to get the same info13:54
zygaogra: --all shows disconnected things AFAIK13:56
ograseems to show connected and disconnected stuff (i cant see a pattern here what it shows/doesnt show though)13:57
ograhttps://paste.ubuntu.com/p/GSRTXxvJ3V/13:57
ograas i said i can go with "snap connections pi3" for the particular use-case where i need to find out the interface name from a gadget ad-hoc13:58
Chipacamborzecki worked on the ux of connections13:58
Chipacai remember it being tricky to get right13:58
Chipaca(and maybe it still needs work in some cases?)13:58
ograbut it also means i need to know what the gadget name is ... thats why an alias might be cool here13:59
mborzeckiiirc we discused snap slots or similar13:59
zygabrb14:00
ackkjdstrand, hi I'm using the patched snapd with support for system users you built, I have supervisord configured to run a process as daemon:daemon (which is configured in the snap as system user), but I see messages like this in log: supervisor: couldn't setuid to 1: Could not set groups of effective user14:00
zygaExhausted by heat.14:03
jdstrandackk: I suspect it is calling setgroups(). see privmsg14:03
zygaMaybe I should turn nocturnal14:03
cachiozyga, hey14:07
cachiofound a problem running beta validation for amg64 http://paste.ubuntu.com/p/pzHMzjRZPG/14:07
cachioruntime/cgo: pthread_create failed: Resource temporarily unavailable14:07
ackkjdstrand, ah, we do use a preload which overrides setgroups/initgroups, for nginx at the moment. I guess I need to do the same for squid then14:07
mborzeckicachio: Jun 05 21:52:38 localhost.localdomain snapd[20219]: runtime/cgo: pthread_create failed: Resource temporarily unavailable14:08
mborzeckiJun 05 21:52:48 localhost.localdomain snapd[20219]: SIGABRT: abort14:08
cachiomborzecki, yes14:09
mborzeckiahh that's the policy-app-consumer, the one with bandwagon of apps14:09
cachiomborzecki, any idea why it could be happening?14:11
cachiomborzecki, I have the machine here14:11
cachiowith a debug session14:11
ackkjdstrand, but, this won't help with apps like supervisors trying to run other commands with a specifi user, will it?14:12
ackkjdstrand, I added the preload to squid but I see the same error14:12
mborzeckicachio: this one is interesting too: snap-repair[4812]: runtime/cgo: pthread_create failed: Resource temporarily unavailable14:13
mborzeckicachio: what does ps -efT | wc -l show?14:14
cachio11214:14
cachiomborzecki,14:14
jdstrandackk: there should be a policy violation in the journal, what is it?14:15
ackkjdstrand, dmesg or journalctl?14:20
ackkjdstrand, I don't see any14:21
jdstrandackk: journalctl14:21
jdstrandwell, really either14:21
ackkjdstrand, no denial14:21
jdstrandackk: are you using snappy-debug?14:22
ackkno14:22
ackkjdstrand, how does that work?14:22
jdstrandackk: sudo snap install snappy-debug ; sudo journalctl --output=short --follow --all | sudo snappy-debug14:23
jdstrandackk: that will tail the journal. then you exercise your application14:23
jdstrandackk: and it will pop out denials with suggestions if there are logged policy violations14:23
jdstrandackk: one thing it does is turn off kernel rate limiting14:24
jdstrandwhich might be why you aren't seeing a denial14:24
ackkah, I see it now14:24
jdstrandwhat is the denial?14:24
ackkjdstrand, https://paste.ubuntu.com/p/xbbC2pJDbF/14:25
jdstrandackk: what is the name of the snap command invocation? maas.what?14:26
ackkjdstrand, it's a daemon maas.supervisord14:26
* jdstrand notes he has a meeting in 4 minutes (coincidentally on this PR)14:26
ackksorry, maas.supervisor14:26
jdstrandackk: what is the output of grep setgroups /var/lib/snapd/seccomp/bpf/snap.maas.supervisor ?14:27
ackk$ grep setgroups /var/lib/snapd/seccomp/bpf/snap.maas.supervisor.src14:27
ackk# allow use of setgroups(0, NULL)14:27
ackksetgroups 0 014:27
ackksetgroups32 0 014:27
ackkjdstrand, ^14:27
jdstrandackk: ok, so something with your LD_PRELOAD isn't working14:28
ackkjdstrand, well supervisor doesn't have LD_PRELOAD14:28
jdstrandackk: setgroups needs to be called with setgroups(0, NULL)14:28
ackkjdstrand, does it also need it?14:28
ackkoh ok14:28
ackkthen I'll add it there too14:29
jdstrandackk: *any* setgrroups denial needs the preload14:29
* jdstrand -> meeting14:29
ackkjdstrand, cool, thanks14:30
ackkah, and now squids actually fails to create the shm files14:32
ackkI guess I need snapcraft-preload after all14:32
ograhmm, is there something with the store today ... i see a lot 503 errors why installing snaps (they dont show up on the second install attempt)15:05
ogras/why/while/15:05
Chipacawoo, woo15:06
Chipacacohort pr #N is in, #(N+1) is up for review15:06
Chipacawoo15:06
Chipacanow running a little late for my train but, woo :)15:06
roadmr🚂15:06
roadmrogra: https://status.snapcraft.io/ 💚15:07
Chipacapedronis: thank you for the review :)15:07
ograroadmr, well ... Download snap "chromium-mir-kiosk" (32) from channel "edge" (received an unexpected http response code (503) when trying to download https://fastly.cdn.snapcraft.io/download-origin/fastly/pKicqh8BMdA6Y6jbtWYBdI2QwhRH4v6T_32.snap?token=1559847600_743804ce162b2329f8ec18078bdcdd62d67273c6)15:08
roadmrogra: if you're consistently seeing 503s and you could turn on snapd debugging and shoot me some logs of what the store is saying, I'd be grateful - can look deeper if I have that info15:08
roadmrogra: oh! perfect! let's see15:08
ograthis took three attempts ... third one was the charm and it installs now15:08
roadmrogra: oh that's the problem - you need juju if you want to install charms 👅15:08
ogralol15:08
Chipacapedronis: dunno if you want to add this to your queue or not (it's low impact tbh): "overlord/snapstate, & fallout: give Install a *RevisionOptions" #6961 https://github.com/snapcore/snapd/pull/696115:09
* Chipaca goes to catch that train15:09
ahasenackogra: I also got a couple of 503s today15:24
ahasenackand also worked on the third attempt only15:24
roadmrahasenac, ogra: hey sooo.. yes, looks like our cdn provider is having some trouble, that explains your failure. We'll post an update in the forum/status page shortly15:24
ograwell, it still works ... just needs some run-on :)15:25
roadmrogra: we've kicked the offending server, things should be copacetic now. Thanks for the report!15:42
ograthanks for fixing !15:44
=== pstolowski is now known as pstolowski|afk
jdstrandpedronis: oh, I said next week I would work on the snap_daemon PR, but I forgot I will be at the snapcraft summit17:41
zygaa newline in the wrong spot of debian/rules makes one funny debugging session as to WTF18:04
zygaspecifically the *first* line when /bin/sh executes a makefile18:04
albertosottilejdstrand: Hi, we read your last comment about the Syncplay classic confinement request... we honestly do not believe that we will add <<more functionality in the realm of taking “input from over the internet to run arbitrary commands”>> in the foreseeable future. So, what should we do now?18:30
stgraberjdstrand: hey there, just moved lxd-demo-server from being an old core16 snap to being a snap using the core18 base and looks like this somehow triggered a manual review?18:39
stgraberjdstrand: human review required due to 'allow-installation' constraint (bool) declaration-snap-v2_plugs_installation (daemon, lxd)18:39
stgraberjdstrand: so looks like it's because it's using the lxd interface (normal as it drives LXD) but that's not a change from any of the previous revisions, so a bit confused as to why this is getting flagged now18:40
jdstrandalbertosottile: comment to that effect in the topic, then pedronis will make the final call18:40
jdstrandalbertosottile: and hello to you too :)18:41
albertosottilejdstrand: We will, thanks :)18:41
jdstrandstgraber: this doesn't have to do with the move to core18 but new tests in the review-tools. I'll grant the snap decl for you18:42
stgraberjdstrand: thanks, so I guess it's just because I didn't upload that one in a while then :)18:42
jdstrandstgraber: yeah, exactly18:43
jdstrandstgraber: ok done. both now pass automated review. you'll need to release them to a channel18:46
stgraberjdstrand: done, thanks18:47
jdstrandnp18:50
zygajdstrand: I'm sorry about missing the call today, I was tired and frustrated by chaotic home schedule and the heat19:05
jdstrandzyga: no worries. I marked you optional and figured you simply opted out :)19:18
jdstrandzyga: hope things are getting better19:18
zygajdstrand: I'm looking forward to coldness of canada19:18
jdstrandzyga: hehe :)19:21
roadmrzyga: the weather is amazing here, temps should be well above 20C next week, check the 7-day forecast: https://weather.gc.ca/city/pages/qc-147_metric_e.html (oh well, rainy tuesday)19:21
jdstrandzyga: it's been pretty warm and humid here too19:22
zygaroadmr: thank you!, looking19:22
zygatemps here are looking at close to 30 all the time19:23
zygaand my office behaves like a small oven with me inside19:23
roadmrzyga: ouch. We'll be there in about a month; July/August can be uncomfortably warm/humid19:24
roadmrsummer is a crazy thing :)19:24
jdstrandroadmr: hmm, it that my recent review-tools builds are stuck in 'Uploading build': https://code.launchpad.net/~myapps-reviewers/+snap/review-tools19:59
roadmrjdstrand: maybe due to this morning's store trouble?19:59
roadmroh this is happening right now - that's odd20:00
jdstrandI'm not aware of that trouble... if it's just catching up, that's fine. fyi since it is unusual20:00
roadmrjdstrand: right, there was some network saturation, let's give them a few and see if they catch up20:01
roadmrjdstrand: I see some of the stuck uploads finally uploaded - things may just be sluggish :/20:08
ijohnsonroadmr: fyi I have had snaps stuck in ci jobs for the past 2.5 hours trying to upload, so whatever the issue was this morning it seems to still be happening20:09
roadmrijohnson: oh thanks.... where is this ci you speak of? 🤔20:10
ijohnsonuhh you mean physically? I think actually it's somewhere near you in Canada, it's a jenkins server the Linux Foundation uses20:10
roadmrijohnson: well I meant whether it was e.g. travis20:11
ijohnsonoh haha well no it's a custom jenkins server20:11
ijohnsonroadmr: see for example https://jenkins.edgexfoundry.org/view/Snap/job/edgex-go-snap-edinburgh-stage-snap/10/console20:11
roadmrijohnson: ah :)20:12
roadmrijohnson: oh I see what happened - looks like revision 935 of that snap failed to complete auto-review, so that upload held the queue, subsequent uploads are awaiting review as well20:14
roadmrijohnson: I've retriggered 935; btw the timing on 935 matches when we were having bandwidth problems20:14
ijohnsonroadmr: hmm, but previously even when a previous snap was stuck in review snapcraft would at least stop and explain that it failed due to the queue20:15
ijohnsonroadmr: are you saying that because the revision failed with auto-review during the bandwidth problem time period this new upload got stuck?20:16
roadmrijohnson: yep, that could be it. btw 935 just passed, so the rest are being processed now20:16
ijohnsonroadmr: ack thanks, btw any idea why they failed auto-review?20:17
roadmrijohnson: probably network-related; e.g. the worker unit that runs the autoreview tries to download the snap, but it was slow/faily at that time, so it barfed, then retried... etc. They do a limited number of retries before giving up20:18
ijohnsonok that makes sense, because I don't think we added anything new to that revision that would have failed review20:19
ijohnsonthanks for explaining20:19
roadmryep, this was "task failed to complete because I couldn't download the snap", compare to "task succeeded but obtained a failed review from the review tools"20:19
ijohnson👍20:21
=== Greyztar- is now known as Greyztar
=== ackk_ is now known as ackk
=== ackk is now known as ack
=== ack is now known as Guest15910
=== ackk is now known as ack`
=== ack` is now known as ackk
=== ackk is now known as ack`
=== ack` is now known as ackk
=== ackk is now known as ack`
=== ack` is now known as ackk

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