/srv/irclogs.ubuntu.com/2019/08/29/#snappy.txt

mupPR snapcraft#2692 opened: internal: unknown apt packages aren't installed <Created by stefanor> <https://github.com/snapcore/snapcraft/pull/2692>02:17
mupPR snapd#7272 closed: interfaces/bluez: enable communication between bluetoothd and meshd via dbus <Created by jrigling> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7272>04:57
mupPR snapd#7341 closed: many: introduce package seed and seedtest <Created by pedronis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7341>05:23
mupPR snapd#7336 closed: tests: add debug section to interfaces-contacts-service <Test Robustness> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7336>05:24
mupPR snapd#7316 closed: tests: add unit tests for cmd_whoami <Created by ardaguclu> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7316>05:25
mvozyga: looks like 7370 has some real spread failurs (just fyi, not urgent :)05:44
mupPR snapd#7369 closed: overlord/snapstate: check channel names on install (2.41) <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7369>06:40
mupPR snapd#7344 closed: snapstate: use strutil.VersionCompare() in checkVersion() <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/7344>06:52
=== pstolowski|afk is now known as pstolowski
pstolowskimorning07:02
zygamvo: yeah, looking07:02
zygamvo: I tested a smaller subset last night so perhaps there's some outcome of the change07:03
zygathat is not handled by the rest of the code07:03
zygaahhh07:04
zygainteresting07:04
zygait failed on opensuse 1507:04
zygaperhaps unified cgroup does not have that feature there, let me look07:04
zygait's odd that it only _sometimes_ failed though07:04
zygathat's very weird07:04
mupPR snapd#7371 opened: snapstate: add comment to checkVersion vs strutil.VersionCompare <Created by mvo5> <https://github.com/snapcore/snapd/pull/7371>07:05
mvohey pstolowski and zyga ! good morning07:05
zygabtw07:05
mvozyga: aha, yeah, I did not look at what system failed just noticed it looked legit07:05
zygadoes anyone know what may be behind07:05
zygashell-init: error retrieving current directory: getcwd: cannot access parent directories: No such file or directory07:05
zygait spams all the logs07:05
mvozyga: I have not seen (at least consciously) this07:06
pstolowskimvo, pedronis i think we have at least one more issue wrt track/risks change, see https://forum.snapcraft.io/t/behavior-change-risk-only-channel-specifications-will-not-switch-track/11769/707:45
pstolowskimvo, pedronis related to 2nd part of my reply, one issue is the migration of old state (which i suppose Chipaca's patch will cover?), second is simply `snap install <foo>` which defaults to "stable" in the state and again will be reported as such with v2/snaps tracking-channel (and confuse GUI)07:49
mvopstolowski: indeed, that looks like a problem07:50
pedronispstolowski: I am a bit confused07:57
pedronispstolowski: didn't you discuss this with them under https://github.com/snapcore/snapd/pull/7255 ?07:59
mupPR #7255: store: use track/risk for "channel" name when parsing store details <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7255>07:59
pstolowskipedronis: yes, and the channels map is as we discussed. but there is also channel name at the top level which i haven't touched (shown in the snippet from Robert in that forum post)08:01
pedronismvo: pstolowski: I start to think that the overall change was premature08:15
mvopedronis: I was wondering the same, if we should just revert it for 2.4108:17
pedronismvo: let's talk after the town hall08:18
Chipacawhich change?08:19
pstolowskiChipaca: track/risk behavior change08:19
pedronisChipaca: https://github.com/snapcore/snapd/pull/719908:20
mupPR #7199: overlord/snapstate: keep current track if only risk is specified <Created by stolowski> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/7199>08:20
pedronisand all that descended from that08:20
pstolowskiChipaca: https://forum.snapcraft.io/t/behavior-change-risk-only-channel-specifications-will-not-switch-track/11769/708:21
pedronisI think we do want that change08:21
pedronisbut we really need to roll it out a bit more carefully08:21
pedronisand right now we are scrambling a bit08:21
Chipacagiven the amount of work we now see is missing, i agree it looks like it's shorter to roll it back08:22
pedronisChipaca: I also not sure while the change makes sense at the CLI level08:23
pedroniswhether it makes sense at the API level08:23
pedroniswhich means API tweaks08:23
pedronisdoes gnome-software do refreshes now?08:37
Chipacapedronis: i don't think so08:40
Chipacapedronis: why?08:40
pedronisChipaca: I'm confused why we did #7255 already08:58
mupPR #7255: store: use track/risk for "channel" name when parsing store details <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7255>08:58
pstolowskipedronis: this was believed to be transparent to the ubu/gnome store as discussed with Robert before, and not requiring any special-casing on their side.. anyway, i'm prepping a revert of all this09:02
pstolowskimvo: btw, the test-snapd-tools track 2.0 didn't work?09:09
mvopstolowski: not yet, sry, let me pick this up again09:09
pedronispstolowski: ok, either we should have a chat with mvo and Chipaca to see where we stand09:12
mvopstolowski: its open now09:13
pstolowskimvo: \o/ thy!09:15
Chipacapedronis: pstolowski: reverting from 2.41, right?09:18
pedronisChipaca: yes, but we have many pieces to revert, and we need to talk what next09:19
pedronisChipaca: also if you started working on a patch, we will need it at some point, so don't throw it away09:20
Chipacapedronis: was planning on stashing it for now09:21
Chipacapedronis: OTOH i might finish it first, not sure09:21
pedronisChipaca: if you are close, please finish it09:21
pstolowskiChipaca: i prepared revert for master with a plan to propose again after fixes. and yes revert for 2.41 as well09:25
pstolowskiChipaca: and i'd like to discuss that patch in the light of what was discussed this morning09:26
pedronisChipaca: pstolowski: mvo: does it work to have a HO now?09:27
pstolowskipedronis: yes, give me 3 minutes pls09:27
Chipacacan i get 10 minutes?09:27
Psil0Cybinhow the hell do  iget snapcraft working on debian based distros whjen it says / does not have /home09:27
Psil0Cybinfor the app09:27
Psil0Cybinwhat the hell09:27
pedronisChipaca: yes09:28
Chipaca👍09:28
ChipacaPsil0Cybin: could you try asking that again please?09:28
Chipacamaybe a little less sweary and with more context for us to understand it09:28
Psil0CybinChipaca, sorry on debian when /home is somewhere else snapd does not work09:30
Psil0Cybinand wont install / run an app09:30
ChipacaPsil0Cybin: snapd needs homes to be in /home, yes; taht is a limitation of snapd that is unlikely to go away in the short term09:30
ChipacaPsil0Cybin: especially because there's no good reason to not have homes in /home, either directly or via bind mounts09:31
ChipacaPsil0Cybin: also note the homes need to be always there, ie not automount, not auto-decrypt-on-login09:31
Psil0Cybinokay but for example im using kali, how can i use snapd?09:31
Psil0Cybinin that case?09:31
Chipacai thought you said you were using debian09:31
pstolowskipedronis: ok ready when you are09:32
ChipacaPsil0Cybin: kali does many things differently, and breaks/blocks the way we set up confinement09:32
ChipacaPsil0Cybin: including confinement of x11 apps for example, they won't work on kali. Also apps that use the network.09:33
Psil0CybinChipaca, i said debian because kali is debian based?09:34
Psil0Cybinokay on that note, what if an application only has insturctions for snapd installs how do i go about obtaining that software?09:34
pedronispstolowski: Chipaca: I made an actual invite09:34
Psil0Cybinanyway of getting around that, for example using this guide? (https://miloserdov.org/?p=2448&PageSpeed=noscript)09:35
Psil0Cybin?09:35
Chipacapedronis: thank you09:36
=== ricab_ is now known as ricab
Psil0CybinChipaca, or are you suggesting im out of luck from community support basically?09:40
Psil0Cybinjust would like to understand thx09:40
Psil0Cybini am assuming becasue your mia, im just out of luck?09:57
ChipacaPsil0Cybin: in a meeting, not mia10:00
Psil0CybinChipaca, apologies.10:00
Psil0Cybini will wait now, sorry.10:01
ChipacaPsil0Cybin: is this with kali from a live cd, or installed?10:21
abeatozyga, hey, I'm seeing something weird related to the content interface10:21
ChipacaPsil0Cybin: some of the issues we've seen on kali are only on the live cd10:21
zygatell me10:21
Chipacapedronis: could you update the forum to let people (including rob) know we're reverting?10:22
abeatozyga, wifi-ap snap shares a socket named $SNAP_DATA/sockets/control10:22
abeatozyga, declared in slot as    write:10:22
abeato      - $SNAP_DATA/sockets10:22
pedronisChipaca: yes10:23
abeatozyga, and used in another snap as 'target: $SNAP_COMMON/sockets'10:23
abeatozyga, what I see in the target is $SNAP_COMMON/control , should it not be inside a sockets folder?10:24
abeatozyga, all that gets written in  $SNAP_COMMON in the target is seen in $SNAP_DATA/sockets in the provider (wifi-ap)10:25
pstolowskipedronis: did you mean to revert also #7359 completely, or just https://github.com/snapcore/snapd/pull/7359/commits/4e54b6d8a2b6ffd1d539abdc9bf4af2438e691c6 ?10:25
mupPR #7359: overlord/snapstate: check channel names on install <Squash-merge> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7359>10:26
zygaabeato: I understand, give me a moment to finish something here10:27
abeatosure10:27
pedronispstolowski: yes10:28
pedronisChipaca: https://forum.snapcraft.io/t/behavior-change-risk-only-channel-specifications-will-not-switch-track/11769/8?u=pedronis10:28
pstolowskipedronis: yes - #7359 completely?10:28
mupPR #7359: overlord/snapstate: check channel names on install <Squash-merge> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7359>10:28
pedronispstolowski: yes10:29
pstolowskipedronis: ok, ty. in that case i think i'll prep 2 reverts, otherwise it'll be slightly bug10:29
pstolowski*bug10:30
pstolowski*bug10:30
pstolowski*big10:30
pstolowskigrr10:30
pedronispstolowski: thx, please mark me for reviewing those reverts10:31
mupPR snapd#7372 opened: overlord/snapstate: revert track risk behavior change for 2.41 <Created by stolowski> <https://github.com/snapcore/snapd/pull/7372>10:36
mupPR snapd#7373 opened: overlord/snapstate: revert track risk behavior change <Needs Samuele review> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7373>10:39
abeatozyga, fyi, if I change the name of the target from $SNAP_DATA/sockets to something different to the folder in the provider, like $SNAP_COMMON/sockets-wifi-ap, I actually see $SNAP_COMMON/sockets-wifi-ap/control, as expected10:45
mupPR snapd#7371 closed: snapstate: add comment to checkVersion vs strutil.VersionCompare <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7371>10:46
pstolowskipedronis: i think i'll actually revert all that in same PRs, otherwise resolveChannel change will be annoying to review10:46
pstolowskimvo: ^10:48
Chipacamvo: patch patch sent sent10:50
* mvo hugs Chipaca 10:52
mvoChipaca: thank you10:52
mvopstolowski: thank you as well!10:52
pstolowskimvo: sorry, more stuff coming to the reverts you just reviewed..10:52
mvopstolowski: no worries10:53
mvopedronis: 7345 has two +1 and is green, shall I merge?10:54
mupPR snapcraft#2693 opened: rust plugin: support for s390x <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2693>11:03
abeatozyga, this is the bug: https://bugs.launchpad.net/snapd/+bug/183853711:04
mupBug #1838537: Analyze wifi-ap's disk usage in SNAP_DATA <snapd:New> <snappy-hwe-snaps:New> <https://launchpad.net/bugs/1838537>11:04
pedronismvo: yes, was about to (will apply couple of tweaks in the later work)11:04
mvopedronis: cool, go for it!11:08
mupPR snapd#7345 closed: overlord/devicestate,seed:  small step, introduce seed.LoadAssertions and use it from firstboot <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/7345>11:09
pedronismvo: #7368 is the next in that series11:11
mupPR #7368: cmd/snap,image,seed:  move image.ValidateSeed to seed.ValidateFromYaml <Created by pedronis> <https://github.com/snapcore/snapd/pull/7368>11:11
* pstolowski lunch and a walk11:13
* Chipaca hugs pstolowski 11:13
abeatozyga, actually, hold on, I'm not 100% sure that is what is happening11:15
mupPR snapd#7364 closed: overlord/snapstate: add migration function to fix invalid channel spec <⛔ Blocked> <Created by anonymouse64> <Closed by pedronis> <https://github.com/snapcore/snapd/pull/7364>11:15
cachiomvo, hey11:23
cachiomvo, is protocol error fixed on 2.41?11:24
mvocachio: it should be, yes11:28
cachiomvo, still see this https://travis-ci.org/snapcore/snapd/jobs/578207214#L705411:28
mvocachio: thanks, let me look11:33
mvocachio: thanks, I cherry-picked one more patch from master, hopefully that fixes it. really good catch!11:35
cachiothanks!11:35
* zyga needs to take half of the day off11:36
zygaI must skip standup. Sorry11:36
zygaI will catch up in the evening11:36
pstolowskihmm google:ubuntu-core-16-64:tests/main/snap-info failure on my revert in master12:08
Chipacapstolowski: can i see?12:09
pstolowskiChipaca: https://api.travis-ci.org/v3/job/578294589/log.txt12:10
Chipacapstolowski: you should be able to run that one locally, if it helps12:10
Chipacabut12:11
Chipacapstolowski: but12:11
Chipacapstolowski: that's actually broken, now12:11
Chipacapstolowski: I blame mvo12:11
Chipacathe test needs fixing, not related to your revert12:11
pstolowskiChipaca: ok, good, it got me confused12:11
Chipaca:)12:12
Chipacapstolowski: I'll push a pr to fix it12:12
pstolowskity12:12
Chipacapstolowski: you're working on master, yes?12:12
Chipacapstolowski: 7374 has the fix12:22
mupPR snapd#7374 opened: tests/main/snap-info: update check.py for test-snapd-tools 2.0 <Created by chipaca> <https://github.com/snapcore/snapd/pull/7374>12:23
pstolowskiChipaca: aaa it's because the new track i asked mvo to create :)12:24
Chipacapstolowski: self-inflicted pain i see12:24
pstolowskiit's really not a great day today12:26
pstolowskiand super hot here12:26
pstolowski#7374 needs second review and it's required to unbreak master12:33
mupPR #7374: tests/main/snap-info: update check.py for test-snapd-tools 2.0 <Simple 😃> <Created by chipaca> <https://github.com/snapcore/snapd/pull/7374>12:33
pstolowski... and we will need to cherry pick it for 2.41 i suppose12:34
=== ricab is now known as ricab|lunch
mupPR snapd#7343 closed: tests: moving tests to nightly suite <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/7343>12:58
Psil0CybinChipaca, i have it installed13:57
Psil0Cybinon my hard drive,i am trying to see if perhaps there is a work around i uninstalled it and purged the install i am wondering if i should  try again?13:57
ChipacaPsil0Cybin: where is your home?13:58
Psil0CybinI just had so many errors, after getting snap.d to run it would not run the program i forgot the exact error message, but I am wondering if maybe you had experience before I attempted again?13:58
Psil0CybinChipaca, its /username13:59
ChipacaPsil0Cybin: why?13:59
Psil0Cybininstead of /home/username i assume13:59
Psil0Cybinno idea13:59
Chipacaquite14:00
Psil0Cybindefault installed it like so14:00
ChipacaPsil0Cybin: you can work around that in particular with a bind mount14:00
ChipacaPsil0Cybin: shouldn't be a blocker14:00
Psil0CybinOh amazing! Perhaps you have a guide i can reference too when i install it (i am working from home today -- remote)14:00
Psil0Cybinso i appreciate that!14:00
ChipacaI do not have a guide, at present14:00
=== ricab|lunch is now known as ricab
Chipacanot enough kali users to write a guide14:01
ChipacaPsil0Cybin: but you could write one14:01
Psil0Cybinyou know what I will its about time i contribute to the community, okay im going to give it an install after a webex meeting and give it a go :)14:01
Psil0CybinThanks! Chipaca you gave me hope14:01
ChipacaPsil0Cybin: i'll be here for anohter couple of hours, otherwise we can continue tomorrow14:01
Psil0Cybinthank you Chipaca you are amazing!14:03
* Chipaca writes that down14:04
mupPR snapd#7374 closed: tests/main/snap-info: update check.py for test-snapd-tools 2.0 <Simple 😃> <⚠ Critical> <Created by chipaca> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/7374>14:07
mupPR snapd#7375 opened: tests: fix snap info test <Simple 😃> <⚠ Critical> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7375>14:11
pstolowski^ cherry pick for 2.4114:11
cachiomvo, tests on spread https://travis-ci.org/snapcore/spread/builds/578357987?utm_source=github_status&utm_medium=notification14:17
cachiomvo, really good news14:17
cjp256https://www.irccloud.com/pastebin/6p15warB/ - installing snaps causes my USB devices to exhibit  some strange behavior - effectively disconnects my monitors through my USB-C dock (and maybe they come back without having to unplug). anyone have any ideas on what may be causing that?14:17
diddledancjp256: I bet that's because we do a udev rebuild14:28
diddledanI believe we trigger a hotplug or something similar to reparse everything14:28
pstolowskiyes, we trigger udevadm control --reload-rules and/or a few variants of udevadm trigger ... depending on circumstances14:34
mvocachio: nice14:35
jdstrandroadmr: hi! can you pull 20190829-1435UTC?14:36
roadmrjdstrand: sure!14:37
cachiomvo could you please take a quick look to #7375 ?14:41
mupPR #7375: tests: fix snap info test <Simple 😃> <⚠ Critical> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7375>14:42
cachiomvo, we will need it as part of 2.41 as well :)14:42
cachiopstolowski, thanks for the fix14:42
pstolowskicachio: credit goes to Chipaca14:42
mvocachio: thanks, cherry picked14:43
cjp256thanks diddledan pstolowski, that's helping me narrow it down to which udev calls are doing it :)14:45
pstolowskicjp256: this is related to the installation of specific snaps and interfaces (plugs) that they have, some interfaces declare udev rules and interact with udevadm when connected14:49
pstolowskimvo: #7375 is already a cherry-pick for 2.41 :)14:51
mupPR #7375: tests: fix snap info test <Simple 😃> <⚠ Critical> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7375>14:51
diddledanI'm eagerly awaiting 2.41 to be in the candidate or stable channels14:52
* cachio bank14:56
cjp256pstolowski: any workaround to preventing the kind of behavior i'm seeing? for me, it pretty much prohibits the use of my usb-c dock when working with snaps, particularly for displays. Probably due to X/gnome, but resorting to unplugging/plugging the displays is only about a 1 in 3 shot of recovering.14:57
popeydiddledan: what in particular?14:57
diddledanpopey: code that I added that will enable me to progress with makemkv stabilisation :-)14:57
cjp256diddledan: me too :D PROTOCOL_ERROR fix *crosses fingers*14:57
popeypstolowski: also, my audio device is on usb and it freaks out when this happens.14:57
popeydiddledan: nice!14:57
diddledanI'm really keen to get makemkv stable :-)15:00
diddledanI've had quite a few people requesting it15:00
cjp256`udevadm trigger --subsystem-nomatch=input` appears to be the source of the behavior15:02
cjp256or perhaps at least one of them15:02
pstolowskicjp256, popey : we probably need to revisit our udevadm backend and see if we can restrict that reloading more to specific classes of devices (nb, it is not related to our hotplug support, it has always been like that with our udev backend)15:05
* pstolowski quick errand, bbiab15:13
pstolowskire15:46
diddledanfwd15:47
pstolowski#7375 needs second +115:47
mupPR #7375: tests: fix snap info test <Simple 😃> <⚠ Critical> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7375>15:47
pstolowskimvo: and i've requested your re-review on 7373 since there was one more revert after you initial review15:48
mvook15:49
mvopstolowski: in a meeting right now, will look after it15:49
cjp256wrt to my display issues, I think it boils down to trigger subsystem=drm15:57
cjp256and triggering it alongside other subsystems at the same time makes breakage much more likely15:58
diddledanaww, netflix doesn't like snapped chromium :-(16:05
roadmrnetflix and chi.. er no :(16:07
diddledannot for linux users :-(16:10
diddledans/linux/snap/16:11
pstolowskicjp256: would you mind filing a bug report (https://bugs.launchpad.net/snapd) and capture all the details you found?16:11
mupPR snapd#7375 closed: tests: fix snap info test <Simple 😃> <⚠ Critical> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/7375>16:43
=== pstolowski is now known as pstolowski|afk
cjp256pstolowski: will do17:06
cjp256pstolowski: https://bugs.launchpad.net/snapd/+bug/1841971 i'll update it when I figure out anything more, thank you :)17:36
mupBug #1841971: installing and removing snaps triggers display failures <snapd:New> <https://launchpad.net/bugs/1841971>17:37
mupPR snapcraft#2693 closed: rust plugin: support for s390x <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2693>18:34
=== ricab is now known as ricab|brb
mupPR snapd#7376 opened: image,o/devicestate,seed: oops, make sure to clear seedtest helpers <Created by pedronis> <https://github.com/snapcore/snapd/pull/7376>19:30
mupPR snapcraft#2692 closed: internal: unknown apt packages aren't installed <Created by stefanor> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2692>22:59

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