mupPR snapcraft#2838 opened: add support for system-usernames <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2838>00:11
cjp256mup: well done! you are quite responsive today! :D00:12
mupcjp256: In-com-pre-hen-si-ble-ness.00:12
mupPR snapd#7877 opened: tests: avoid mask rsyslog service in case is not enabled on the system <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/7877>01:52
sdhd-saschagood morning07:19
zygaGood morning07:33
zygaI will start late today07:33
zygaKids have no regular school and nobody woke up early07:34
mborzeckizyga: no school today? have i missed something?07:49
mupPR snapd#7821 closed: interfaces/seccomp: parallelize seccomp backend setup <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/7821>07:50
mupPR snapd#7872 closed: doc: HACKING.md change autopkgtest-trusty-amd64.img name <Created by sd-hd> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/7872>08:04
mupPR snapd#7876 closed: seed: fix seed location of local but asserted snaps <UC20> <Created by pedronis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7876>08:11
mvomborzecki: do you think you could have a look at 7870?08:21
mborzeckimvo: sure08:21
mvomborzecki: thank you!08:22
mupPR snapd#7874 closed: seed: support ModeSnaps(mode) for mode != "run" <UC20> <Created by pedronis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7874>08:22
zygamborzecki: no, theatre and medical examination for Iza and Jan respectively08:34
zygaNothing makes the morning lively as a broken jar of jam on the carpet08:34
zyga But on the upside the carpet was not this clean in weeks08:35
zygaAnd I also cleaned the dust filter in Iza’s PC08:35
zygaNow I only need breakfast and shower an I can get to work08:36
zygaeveryone's gone and I can work09:44
zygaI'm on bug triage duty today09:45
mupPR snapd#7870 closed: boot,bootloader: setup the snap recovery system bootenv <UC20> <Created by mvo5> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/7870>10:33
mupPR snapd#7817 closed: boot,bootloader: make MakeBootable() uc20 aware <UC20> <Created by mvo5> <Closed by bboozzoo> <https://github.com/snapcore/snapd/pull/7817>10:34
mupPR snapd#7878 opened: cmd/snapd-generator: fix unit name for non /snap mount locations <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7878>10:44
mborzeckizyga: mvo: ^^ trivial fix10:45
pedronispstolowski: we can try to have our chat about services after the standup today?10:56
pstolowskipedronis: great. i'll need a 10 minute break right after the standup for a very quick errand10:58
pedronispstolowski: that's fine with me10:59
pedronismvo: suggested comment for one of your follow ups as discussed:  https://paste.ubuntu.com/p/PTFrKZGgz5/11:00
Chipacawhoops, forgot to turn on irc today for some reason11:04
Chipacaonly realised because i thought to share: i need to measure, but the impression i get is that interacting with snapd perceptibly slows down as the number of changes grows11:05
Chipacanow this might just mean i get more impatient as the test i'm working on progresses :)11:05
Chipacaso i need to measure, but, that11:06
pedronisChipaca: it's not impossible we do quite a few things that in the order of #changes11:07
Chipacapedronis: yup11:07
Chipacapedronis: it shouldn't be perceptible though :)11:07
Chipacain the sense that if it is we might want to fix it11:07
Chipacaso i'll be measuring that at some point11:07
pedronisChipaca: yes, but please channels first11:07
mvopedronis: thanks, in a meeting11:07
pedronismborzecki: mvo: #7873 needs a master merge now I think11:08
mupPR #7873: image: set recovery system label when creating the image <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7873>11:08
zygamborzecki: looking11:08
Chipacapedronis: that's the test i'm working on that i saw this in11:08
mborzeckijust yesterday I read an article about O(n^2) in WMI, maybe we're hitting a similar sweet-spot with changes11:08
Chipacachannels i mean11:08
mborzeckipedronis: already updated11:08
pedronismborzecki: thx, just saw that11:08
mvopedronis: will do after my meeting, thank you!11:11
pedronismborzecki did it already11:11
pedronisI also +1ed11:11
mvopedronis: \o/11:27
mvopedronis, mborzecki thank you so much!11:27
mupPR snapd#7878 closed: cmd/snapd-generator: fix unit name for non /snap mount locations <Simple 😃> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7878>11:41
mupPR snapd#7873 closed: image: set recovery system label when creating the image <UC20> <Created by mvo5> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/7873>12:05
mupPR snapd#7879 opened: [RFC] devicestate: use httputil.ShouldRetryError() in prepareSerialRequest <Simple 😃> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7879>12:35
mupPR snapd#7880 opened: snapstate: add support for UC20 gadget in {Config,Gadget}Defaults <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7880>12:37
cachiomvo, hey12:58
cachiomborzecki, hey12:59
cachioafter we run some tests the rsyslog remains masked12:59
cachioan example is ubuntu-core-config-defaults-once12:59
cachioit happens on ubuntu core 1813:00
cachiowhere the rsyslog is not part of the system13:00
cachioseems to be a bug13:00
cachiozyga, ~13:00
cachiosomeone could confirm if it is working as design or it is something to fix?13:01
mvohey cachio ! thanks for this report13:20
cachiomvo, yaw13:26
mborzeckicachio: snap-set-core-config needs a fix, but that other test does not, my comment was about the other test13:34
cachiomborzecki, my question is about the problem seems to be in snapd itself because it leaves the rsyslog as masked in case hte service does not exist in the system13:38
cachioI mean, in case rsyslog does not exist, when we restore everyting in the test the rsyslog service should not appear any more in systemd right?13:39
zygacachio: are you saying test restore should undo the masking13:44
zygacachio: or snapd should undo the masking under some condition?13:44
cachiozyga, snapd should do it if possible13:45
zygacachio: ok, what is the condition when snapd should unmask rsyslog?13:45
cachiobut if not, I'll change the test ot do it in the test13:45
cachiozyga, in core18 rsyslog is not installed by default13:45
cachioso we mask it but it not installed13:46
zygacachio: do we ever ask snapd to unmask it?13:52
cachiozyga, but perhaps we should no mask a service which does not exist13:52
zygaso the fact it is not installed is not relevant13:53
zygaif it was installed the bug would be back13:53
zygawe either need to ask snapd to unmask it13:53
zygaor know about this in global test restore and undo it13:53
cachiozyga, I can unmask the service in the test13:53
cachioit is easy13:53
zygaconsider doing that globally as well13:54
zygasince it is part of the "state" we undo13:54
cachiozyga, ok13:54
cachioI am working in another PR to make a set of validations during hte restore13:54
cachioI'll include that on this one13:54
zygasure, as you wish13:55
=== ricab is now known as ricab|lunch
pstolowskipedronis: i'm in, can you hear me?14:47
pedronispstolowski: sorry, one sec14:48
* cachio afk 10 mins14:56
=== ricab|lunch is now known as ricab
ijohnsonhey folks, quick question how can I create cohorts for a demo of how they work? (maybe pedronis can point me in the right direction?)15:22
jdstrandpedronis: hey, can you take a look at https://forum.snapcraft.io/t/synchrorep-need-classic-confinement/13347/8 (and https://forum.snapcraft.io/t/synchrorep-need-classic-confinement/13347/9). no need to say anything unless you want to overrule me15:31
zygaijohnson: hey15:43
ijohnsonzyga hey15:43
zygaijohnson: you need to set a cohort key via snap set AFAIK15:43
zygaijohnson: any number of machines can set the same key15:43
ijohnsonbut how do I create a cohort key15:43
zygaijohnson: and then they will see the same view of revisions15:43
zygaijohnson: it's just a string15:43
zygabut I can be wrong15:43
zygaChipaca: ^ quick help please15:43
* cachio lunch15:44
pedronisijohnson: snap create-cohort snap-names15:45
zygathank you pedronis!15:45
ijohnsonperfect thanks!15:46
pedronisijohnson: you can one key per snap15:47
pedronisthat you care about15:47
jdstrandpedronis, mvo: fyi, I'm going through store reviews today and then moving to PR reviews https://github.com/snapcore/snapd/pulls/review-requested/jdstrand (jeez, a ton came in since the sprint)15:49
Chipacazyga: in a meeting sorry15:50
zygaChipaca: no worries, pedronis already gave the answer15:50
jdstrandpedronis, mvo: I am going to start with 7238, then 5822 and 7490 (the priority that pedronis gave me in Vancouver)15:51
Chipacaijohnson: snap create-cohort15:51
jdstrandpedronis, mvo: fyi, I had a huge pile of stuff that accumulated due to sprint prep, sprint immediate asks, postsprint catchup, then holidays15:51
pedronisjdstrand: yes, that's why we setup a meeting later in the week to try to prioritize those a bit, and reunderstand the very old ones15:51
Chipacaijohnson: gives you a yaml that you then ship everywhere that wants to use the cohort15:52
jdstrandpedronis, mvo: which is why I am behind. we can meet on thur as requested, but honestly, I'd prefer an email with the PRs in the priority you'd like them reviewed. then I can work that list and give updates until it is done15:52
jdstrandpedronis, mvo: ok, well, that's fine. I'm just going off what you want, but we can have the meeting15:53
jdstrandhopefully the list will be a little shorter by then15:53
Chipacapedronis: mvo: as a conclusion I don't think we need to block cutting the next release, but we do need to not actually release it without the TBD fixes15:56
Chipacapedronis: mvo: so maybe a -pre would be more honest :)15:56
sdhd-saschazyga: hope it was ok to write you per mail directly?16:07
zygasdhd-sascha: that's ok, I haven't seen your mail yet though16:07
zygaI don't really follow email that well (>>>1000 email unread)16:07
pedronismvo_: seems lot of Core 20 bits were not in 2.42 but after16:36
mvo_pedronis yeah, that sounds plausible16:44
mvo_pedronis 2.42 is pretty "old" :/16:45
mvo_pedronis why is it a concern? is anyone using stable for uc20?16:45
=== mvo_ is now known as mvo
Chipacazyga: what was the name of the forum thing where we document some missing things in snapd, like homes needing to be there all the time?16:49
Chipacaah, limitations in snapd, got it16:49
zygaChipaca: ah, let me find it16:50
zygaah you found it :D16:50
Chipacazyga: context is https://forum.snapcraft.io/t/updates-delete-user-application-data/14568 fwiw16:51
sdhd-saschazyga: just want to say that you can also give me some  tasks.16:55
zygasdhd-sascha: if you want to get into development my best advice would be follow code reviews16:57
zygawe have lots of things going on16:57
zygaand we have a specific style of interaction that may not be common in other projects16:58
zygamost churn goes into RPs at the top of the list16:58
zygaolder PRs tend to "sink" to the bottom where they see less activity16:58
zygareviewing code has two benefits: you get to familiarize yourself with what's going on and where16:58
zygaand you get to see how the review process looks like16:58
zyganote, I have not read your mail yet16:59
mvojdstrand: thanks for your feedback earlier, I will think about it17:02
mupPR snapd#7768 closed: interfaces: add raw-volume interface for access to partitions <Created by jdstrand> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/7768>17:22
pedronisjdstrand: I merged ^ , now 7875 has some conflicts17:23
jdstrandpedronis: thanks and ack (weird, I branched off my last commit to raw-volume, but I'll fix)17:24
pedronisjdstrand: it's likely because I squased 7768 because there was too much back and forth in its history17:25
mvohm, 7830 has two +1 and is green - can I merge it?17:26
pedronismvo: it would be ok for me17:27
mupPR snapd#7830 closed: interfaces: include hooks in plug/slot apparmor label <Bug> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7830>17:30
sdhd-saschajdstrand: hi. If i understand it right, then classic will be in future only supported in a few cases. For me it's ok if you write into the forum, that it want be supported and there will be soon other ways to launch other applications. E.g. "app-launch". Then i can close my issue on github and point to the forum. I personally want to concentrate on the strict wayland snap.18:17
blackboxswhi folks, are people seeing issues where snapd.seeded never completes on lxc (disco && eoan) https://bugs.launchpad.net/ubuntu/+source/snapd/+bug/1806070?20:10
mupBug #1806070: snapd.seeded.service never completes preventing full boot to default target <amd64> <apport-bug> <bionic> <snapd:Fix Released> <snapd (Ubuntu):Fix Released> <https://launchpad.net/bugs/1806070>20:10
jdstranddegville: hey, fyi, took a stab at https://forum.snapcraft.io/t/the-raw-volume-interface/1457820:26
degvillejdstrand: brilliant, thanks you!20:40
ograblackboxsw, given that bug was fixed a year ago, you should really better open a fresh one ... (your error also does look different, snapd isnt stuck in the "Doing" state for the initialization but actually errored)20:52
blackboxswogra: will do. Thanks, I'll try grabbing more details about the symptoms I'm seeing and file fresh20:52
blackboxswworth opening a bug or snapcraft forum discussion?20:53
ogra(you probably want to include the output of "snap tasks 1" as well)20:53
* blackboxsw reads : https://forum.snapcraft.io/t/when-to-use-forum-vs-irc/38 20:53
blackboxswthx will do20:53
ograi'd open a bug ... but thats really up to you, the snapd guys look at both, launchapd as well as the forum20:53
blackboxswok thanks20:53
* cachio eod21:08
sdhd-saschazyga: it's not urgent. but maybe, you have a hint for me. I want to allow access to /run/systemd/sessions/<id> . No i found that commonFilesInterface doesn't allow apparmor wildcards... Is there another where, instead of using "system-files" ?21:34
sdhd-saschazyga: sorry... "login-session-observe"21:35
sdhd-saschai found it21:35
mupPR pc-amd64-gadget#25 opened: Fix kernel path in recovery grub.cfg <Created by cmatsuoka> <https://github.com/snapcore/pc-amd64-gadget/pull/25>22:30
jdstranddegville: fyi, one more for you: https://forum.snapcraft.io/t/the-login-session-observe-interface/1458022:31
zygasdhd-sascha: I think /run is special cased to be off-limits by some of the special interfaces22:47
zygasdhd-sascha: ping me tomorrow please22:47
sdhd-saschazyga: login_session_observe from jdstrand, 18 days ago seems to work22:47
zygasdhd-sascha: that's cool, I meant that the generic "gimme a path" interfaces may not work22:48
sdhd-saschazyga: i have found a missing apparmor method in login-session-control22:48
sdhd-saschai will push it later22:48
zygaobviously a precise interface can open any location22:48
sdhd-saschabut i didn't found a interface for policykit ...22:48
sdhd-saschabut let's talk tomorrow22:48
sdhd-sascha"gimme a path" ?22:49
sdhd-saschazyga: good night22:50
mupPR pc-amd64-gadget#25 closed: Fix kernel path in recovery grub.cfg <Created by cmatsuoka> <Merged by xnox> <https://github.com/snapcore/pc-amd64-gadget/pull/25>22:56
degvillejdstrand: looks good - thanks for letting me know!23:07
mupPR snapd#7881 opened: login-session-control - added missing dbus commands <Created by sd-hd> <https://github.com/snapcore/snapd/pull/7881>23:14
jdstrandrbasak: fyi, I haven't forgotten about certbot. it is first up tomorrow23:47

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