| mup | PR snapcraft#2952 opened: spread: introduce appstream parse-info test <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2952> | 03:25 |
|---|---|---|
| === mcphail3 is now known as mcphail | ||
| === popey8 is now known as popey | ||
| === Eleventh_Doctor is now known as Pharaoh_Atem | ||
| mborzecki | morning | 06:11 |
| mborzecki | school run | 06:40 |
| mborzecki | re | 07:08 |
| mup | PR snapd#8184 closed: cmd/libsnap, tests: fix C unit tests failing as non-root <Test Robustness> <⚠ Critical> <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8184> | 07:40 |
| mvo | ijohnson: meh, it looks like 2020-02-24 22:05:33 Error executing google:ubuntu-core-18-64:tests/core/snapd-failover : is still failing on master even after merging your latest fix for it :( | 07:41 |
| mvo | ijohnson: https://api.travis-ci.org/v3/job/654625002/log.txt which is a run after 8171 got merged | 07:42 |
| mborzecki | mvo: hey | 07:42 |
| mvo | hey mborzecki | 07:42 |
| mborzecki | ouch, failover still failing? :/ | 07:43 |
| mvo | mborzecki: at least one log still has it | 07:43 |
| mborzecki | mvo: there's one oustanding PR with fixes iirc | 07:43 |
| mvo | mborzecki: aha, there is? | 07:43 |
| mvo | mborzecki: 8169 ? | 07:44 |
| mborzecki | hm i think it was #8140 | 07:44 |
| mup | PR #8140: tests: enable more tests for UC20/UC18 <Test Robustness> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8140> | 07:44 |
| mborzecki | let me check, it was referenced in one that got merged | 07:44 |
| mborzecki | mvo: right 8169 | 07:45 |
| zyga | good morning | 08:03 |
| * zyga is sleepy today, sorry for starting late | 08:04 | |
| mvo | zyga: hey! | 08:04 |
| zyga | mvo: hey, I fixed the error from last evening | 08:04 |
| mvo | zyga: I saw, thanks for this! merged already | 08:04 |
| zyga | ah, good | 08:04 |
| zyga | mvo: I wanted to raise https://github.com/snapcore/snapd/pull/8123 | 08:04 |
| mup | PR #8123: interfaces/network-control: bring /var/lib/dhcp from host (approach b) <Bug> <Created by zyga> <https://github.com/snapcore/snapd/pull/8123> | 08:04 |
| zyga | mvo: jamie gave it a +1 and expressed ok to put this in 2.44 | 08:05 |
| zyga | my reasoning for raising it is that I prefer one variant rather than two | 08:05 |
| pstolowski | morning | 08:05 |
| zyga | mvo: if 2.44 ships (a) (merged) and then we switch to (b) in 2.45 that's more churn than just going with (b) | 08:05 |
| zyga | mvo: at the same time, it's not a priority fix, just for your consideration | 08:05 |
| mborzecki | zyga: pstolowski: hey guys | 08:05 |
| zyga | hey maciek! | 08:05 |
| * zyga is sleepy, couldn't sleep last night | 08:06 | |
| zyga | I'll make a 2nd coffee and be back soon | 08:07 |
| mborzecki | pstolowski: can you take a look at https://github.com/snapcore/snapd/pull/8081 ? | 08:10 |
| mup | PR #8081: tests/main/user-session-env: add test verifying environment variables inside the user session <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8081> | 08:10 |
| pstolowski | sure | 08:10 |
| mup | PR core20#23 closed: hooks: ensure console-conf@ is started after snapd.recovery-chooser-trigger.service <Created by bboozzoo> <Closed by bboozzoo> <https://github.com/snapcore/core20/pull/23> | 08:13 |
| mborzecki | pstolowski: thanks! | 08:13 |
| mvo | zyga: thanks, it looks like samuele wants to have another look, I'm not against this (at all) | 08:14 |
| zyga | yep, I saw, | 08:14 |
| mup | PR snapd#8188 opened: spread.yaml: make qemu ubuntu-core-20-64 use ubuntu-20.04-64 <Simple 😃> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8188> | 08:14 |
| zyga | what's the timeline for 2.44? | 08:14 |
| zyga | PPA is busted? | 08:16 |
| zyga | https://www.irccloud.com/pastebin/BCFak9n5/ | 08:17 |
| * zyga wonders if we could just get on with slack | 08:17 | |
| * mborzecki prepares to buy another 16GB of ram | 08:21 | |
| zyga | mborzecki: that's easy though :) | 08:22 |
| zyga | mborzecki: remember when vista was around | 08:22 |
| zyga | mborzecki: all that free memory | 08:22 |
| mborzecki | i think i wouldn't mind matrix really | 08:30 |
| pstolowski | mborzecki: +1 with small remark | 08:30 |
| mborzecki | pstolowski: thanks, pushing the update now | 08:30 |
| mborzecki | seems like there's a bunch of mobile apps, probably none as well supported as slack app, but still looks better than mattermost | 08:33 |
| mborzecki | mvo: so, about the chooser ui, i think we should have some intiial prompt rather than jumping straight away to options that do something | 08:34 |
| mborzecki | mvo: for instnace, on serial you may have trouble with baud rate setting, or the output may not apepar right away, if the user presses enter and that activates some 'funky' option there may be trouble ahead | 08:35 |
| zyga | mborzecki: does matrix have a good phone app? | 08:35 |
| mvo | mborzecki: good point | 08:35 |
| mborzecki | zyga: hmm https://play.google.com/store/apps/details?id=im.vector.app&hl=en_US | 08:36 |
| mborzecki | 677 reviews, heh | 08:36 |
| zyga | seems poor | 08:36 |
| mborzecki | there's this one too https://play.google.com/store/apps/details?id=im.vector.riotx&hl=en_US | 08:36 |
| zyga | 5k installs | 08:37 |
| mborzecki | yeah | 08:37 |
| degville | zyga / mborzecki: I've used Riot.im for some time, and it has really improved over the last 12 months. I use it mostly as an IRC bridge though. RiotX looks good - it would be great to see how Mozilla gets on with Matrix, because it's scale that worries me and can't test. | 09:19 |
| zyga | degville: I wonder if matrix can be summarized as "good desktop experience, irrelevant mobile experience" | 09:24 |
| zyga | degville: which might explain why mozilla is OK with it | 09:24 |
| zyga | degville: I would probably not use slack on my phone outside of conferences and occasional "let's check that one thing from bed" moments | 09:25 |
| degville | zyga: me neither. my phone is too old and it will probably reduce battery to 30 mins. I used the web client when I needed to use slack. It does seem a little 'over engineered'. It would be great to get behind an open replacement for IRC imo, because it's not just us who could benefit. | 09:26 |
| degville | though I did get the weechat (IRC client) to slack plugin working, which is a nice compromise. | 09:27 |
| zyga | degville: I think one thing has changed though, the acceptance level has shifted up | 09:37 |
| zyga | degville: people expect a more polished experience | 09:37 |
| zyga | degville: and in the endless quarrels of FOSS people we have not managed to raise above that to produce an excellent opinionated product that is also free | 09:37 |
| zyga | degville: I fear we'll go to a non-free product just because it is actually working well | 09:38 |
| zyga | degville: a bit like going with google for mai | 09:38 |
| zyga | 09:38 | |
| zyga | can we run our own mail server, sure | 09:38 |
| zyga | is it nice to most people, not as much | 09:38 |
| zyga | I think IRC will be exactly the same thing | 09:38 |
| zyga | (and email has much nicer desktop apps than IRC ever did) | 09:39 |
| degville | yeah, you're right. I really don't h | 09:39 |
| degville | have a problem with non-free at all. It is about the best tool. | 09:39 |
| degville | but... it's also a change to differentiate, and also flex some open source credentials when other people may be going the other way. | 09:40 |
| zyga | I wish there was a monetarily supported free product that's really nice and offers good experience | 09:40 |
| zyga | I wonder what suse and rh are doing in this regard | 09:41 |
| degville | yep. I was seriously considering getting in touch with a couple of friends to see if they wanted to create a startup :) | 09:41 |
| zyga | I think being able to host your own FOSS server would be 90% of what we want | 09:42 |
| degville | well, RH has just gone with Slack. | 09:42 |
| degville | over MS teams (https://www.theverge.com/2020/2/10/21132060/ibm-slack-chat-employee-rollout-microsoft-teams-competition) | 09:42 |
| zyga | the apps could just be closed source for all I care (mobile) and web apps running from the same server | 09:42 |
| zyga | oooooh | 09:42 |
| zyga | RH slack? | 09:42 |
| mborzecki | degville: as in 'just now' or just (as in no other options considered) ? | 09:42 |
| zyga | are they using the snap? ;) | 09:43 |
| mborzecki | ah ok, IMB | 09:43 |
| degville | ahahah... | 09:43 |
| mborzecki | IBM | 09:43 |
| degville | yeah, true. I didn't realise it was only 350k employees either, which is a drop in the IBM ocean. | 09:44 |
| zyga | wait, what's 350k employees? | 09:44 |
| degville | IBM's employees. | 09:44 |
| zyga | ouch | 09:45 |
| zyga | that must cost some pretty penny | 09:45 |
| degville | ...actually, all of it's employees. | 09:45 |
| degville | its | 09:45 |
| * mborzecki wonders about uinput for snap-boostrap recovery-chooser-trigger spread tests | 09:45 | |
| zyga | uinput? | 09:45 |
| zyga | what's that? | 09:46 |
| mborzecki | zyga: userspace driven input device, basically create /dev/input/<event*> from userspace and inject input events | 09:46 |
| zyga | aha | 09:46 |
| zyga | all this input devices are now yours | 09:46 |
| zyga | except evdev | 09:46 |
| zyga | attempt no open there | 09:46 |
| zyga | ;) | 09:46 |
| * zyga goes back to hitting his head on sessions | 09:47 | |
| MattJ | zyga, degville: there are multiple options for self-hosted FOSS team chat... RocketChat, Mattermost, etc. - curious why you're dismissing them (iirc this community used to be on RocketChat, what happened?) | 09:47 |
| MattJ | I've been working on FOSS chat stuff for ~15 years, so I'm 100% curious to hear from folk (and happy to chat in private if it's too off-topic here) | 09:48 |
| zyga | MattJ: I don't know - do any of those offer identity management and mobile apps? | 09:48 |
| * zyga wonders what the requirements are | 09:48 | |
| MattJ | Yes | 09:48 |
| mborzecki | hm mattermost mobile app was pretty crappy tbh | 09:50 |
| MattJ | I've never used their mobile app myself, but the rest of Mattermost is generally fairly polished (it's essentially a Slack clone) | 09:51 |
| MattJ | Main issue with Mattermost is that they do the do the "enterprise edition" thing, and refuse to add (or accept contributions for) certain features in the community edition | 09:51 |
| zyga | typical open core | 09:52 |
| zyga | is that bad tough | 09:52 |
| zyga | otherwise the open part would not be there either | 09:52 |
| MattJ | *shrug* - that's an entire debate in itself :) | 09:52 |
| MattJ | I'm personally not a fan of it, but I do also find it weird how many stories I've heard of companies running the open-source version with bunches of patches applied because they don't want to pay for the commercial version | 09:53 |
| MattJ | but then they basically have to maintain their own fork | 09:53 |
| MattJ | but I guess dev/ops time spent on that doesn't appear explicitly on balance sheets, so it's ok | 09:54 |
| zyga | combinations of imperfect realities | 09:54 |
| zyga | we should all use git irc | 09:55 |
| zyga | like irc | 09:55 |
| zyga | just with the usability of git ;) | 09:55 |
| zyga | commit your messages | 09:55 |
| zyga | don't rebase public channels | 09:55 |
| zyga | that kind of stuff | 09:55 |
| zyga | for usability it comes with a man page generator | 09:55 |
| MattJ | There are plenty of blockchain chat solutions if that really tickles your fancy :D | 09:55 |
| mborzecki | ICO included ;) | 09:56 |
| mborzecki | looks like it's a choice between lest crappy solution | 09:57 |
| MattJ | Well there are plenty of annoyances with Slack and other proprietary solutions too, so... welcome to software :) | 09:58 |
| MattJ | I'm looking to possibly release something in this space later in the year, so I've been doing a fair amount of research | 09:58 |
| * zyga is feeling so so today | 10:05 | |
| mup | PR snapd#8177 closed: cmd/snap-bootstrap/initramfs-mounts: mount the snapd snap in run-mode too <UC20> <Created by anonymouse64> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/8177> | 10:06 |
| zyga | [Tue Feb 25 08:37:42 2020] audit: type=1400 audit(1582619862.212:29075): apparmor="DENIED" operation="open" profile="snap.test-snapd-audio-record.play" name="/etc/pulse/client.conf" pid=11535 comm="paplay" requested_mask="r" denied_mask="r" fsuid=12345 ouid=0 | 10:06 |
| zyga | https://www.irccloud.com/pastebin/06KY87al/ | 10:07 |
| ackk | hi, is there a way to programmatically (from a script) check for snapd support for default tracks? | 10:30 |
| zyga | ackk: apart from version query, probably not | 10:32 |
| ackk | zyga, ok. so >=2.44 ? | 10:32 |
| zyga | ackk: I don't know, please ask mvo about the details | 10:32 |
| * zyga is deep in systemd-pam | 10:32 | |
| * ackk looks around for mvo :) | 10:33 | |
| mvo | ackk: yeah, 2.44 is the best bet right now | 10:36 |
| ackk | mvo, ok, thanks | 10:36 |
| mvo | ackk: we could build something programmatic if needed | 10:37 |
| ackk | mvo, are versions guaranteed to be x.y (just two numbers)? | 10:37 |
| mup | PR snapd#8189 opened: seed,cmd/snap-boostrap: introude seed.Snap.EssentialType, simplify bootstrap code <Created by pedronis> <https://github.com/snapcore/snapd/pull/8189> | 10:38 |
| pedronis | mvo: mborzecki: ^ this is relatively simple, the size is mostly from test updates that change indent | 10:51 |
| mvo | pedronis: thanks | 10:51 |
| mup | PR snapd#8156 closed: cmd/snap-bootstrap: subcommand to detect UC chooser trigger <UC20> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/8156> | 10:51 |
| mvo | ackk: following the ubuntu versioning schema, so 2.44 or 2.44.1 | 10:51 |
| pedronis | is not the step that reduces validations but is on the way there | 10:52 |
| ackk | mvo, thanks | 10:57 |
| pedronis | mborzecki: thanks for review and for spotting the image_test.go issue | 11:14 |
| pedronis | image_test.go fixed | 11:27 |
| mup | PR snapd#8190 opened: overlord, taskrunner: exit on task/ensure error when preseeding <Preseeding 🍞> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8190> | 11:43 |
| jdstrand | pedronis: hi! curious, is there a store api to determine the highest snap-declaration format the store supports? | 12:00 |
| pedronis | jdstrand: not at the moment | 12:08 |
| jdstrand | ok, thanks | 12:08 |
| pedronis | it could be added if needed | 12:08 |
| pedronis | but needs to involve the store (obviously) | 12:09 |
| jdstrand | pedronis: sure, it isn't important | 12:11 |
| jdstrand | pedronis: I'm just writing a small tool to help reviewers working with store apis. I can hardcode it with a pointer to maxSupportedFormat in asserts/asserts.go | 12:13 |
| pedronis | jdstrand: fwiw, it doesn't support 4 yet | 12:13 |
| jdstrand | I know | 12:13 |
| pedronis | I'll ask for a bump during 2.44 release cycle | 12:13 |
| * jdstrand nods | 12:14 | |
| mup | PR snapd#8191 opened: [RFC] cmd/snap-recovery-chooser: add a recovery chooser <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8191> | 12:26 |
| === Eighth_Doctor is now known as Conan_Kudo | ||
| === Conan_Kudo is now known as Eighth_Doctor | ||
| * zyga is deep into PAM but feels like it is actually working now | 12:35 | |
| zyga | mborzecki: fun stuff | 12:36 |
| mborzecki | zyga: pam? | 12:36 |
| mborzecki | zyga: the session/scope thing? | 12:36 |
| mup | PR core20#25 opened: hooks/200-console-conf-after.chroot: perform console-conf ordering checks <Created by bboozzoo> <https://github.com/snapcore/core20/pull/25> | 12:37 |
| mborzecki | hmm signal? | 12:40 |
| zyga | mborzecki: yeah, I feel I'm starting to understand WTF when you log in | 12:41 |
| zyga | I need to take the dog out | 12:41 |
| zyga | ttyl | 12:41 |
| zyga | I'm taking off | 12:53 |
| zyga | cannot stand smell of food and cooking at home | 12:53 |
| zyga | $motherinlaw is on a cooking spree | 12:54 |
| mborzecki | hmm looks like all spread jobs are failing? | 13:08 |
| cmatsuoka | mborzecki: the error message also looks wrong | 13:09 |
| cmatsuoka | 'If that is intentional, please update the package list in the' | 13:10 |
| mborzecki | cmatsuoka: the error is way up in the log | 13:10 |
| cmatsuoka | ah ok, it continues after in a new echo | 13:10 |
| cmatsuoka | never mind | 13:10 |
| mborzecki | but i meant that snapd spread jobs are failing due to snapd-failover | 13:11 |
| cmatsuoka | mborzecki: make install error, that's strange | 13:12 |
| cmatsuoka | did we land ian's snapcraft.yaml fix? | 13:13 |
| cmatsuoka | anyway, I'm not familiar with this usr/share/snappy/dpkg.list | 13:16 |
| cmatsuoka | mborzecki: ah, failover again? I thought mvo fixed that a few days ago | 13:18 |
| mvo | cmatsuoka: I thought so too :( | 13:23 |
| mvo | cmatsuoka: investigating this again, it's a PITA | 13:23 |
| cmatsuoka | mvo: btw dimitri's fix worked and now we have run mode working, with and without tpm | 13:25 |
| * zyga is at a cafeteria | 13:26 | |
| zyga | finally no food smell | 13:26 |
| roadmr | what :) | 13:28 |
| mvo | cmatsuoka: \o/ | 13:30 |
| mborzecki | zyga: must be a bad cafeteria if you're no smelling any food | 13:35 |
| zyga | mborzecki: are you kidding me? | 13:36 |
| zyga | mborzecki: there's coffee and food but you don't smell any of it | 13:36 |
| zyga | mborzecki: it's not a kitchen | 13:36 |
| mborzecki | zyga: sure, it doesn't smell like beef steaks, but these places have a very distinct smell too (one i actually enjoy) | 13:41 |
| zyga | mborzecki: I think I'm just overly happy that it doesn't smell like the back of a kitchen anymore :) | 13:46 |
| mborzecki | hahaha ;) fair enough | 13:46 |
| zyga | mborzecki: but on topic, pam is really key to my understanding of the whole problem | 13:47 |
| zyga | mborzecki: building pam with debugging and setting up the debug log | 13:47 |
| zyga | mborzecki: also forkstat | 13:47 |
| zyga | that opened my eyes as to what's going on on login | 13:47 |
| zyga | in various contexts | 13:48 |
| zyga | I'm going through the config and the pam modules that interact with default sessions you may open, piecing together the picture of how it works | 13:48 |
| mborzecki | zyga: you'll have to do a write up for everyone | 13:48 |
| zyga | mborzecki: yep, including instructions on how to get stuff into debuggable state :) | 13:48 |
| mborzecki | and not break your system in the process | 13:49 |
| zyga | mborzecki: pam_systemd.c (in systemd's tree) is really 99% of what I care about | 13:50 |
| zyga | although some other bits are also relevant in other modules | 13:50 |
| zyga | heh | 13:51 |
| zyga | this is a fun comment | 13:51 |
| zyga | https://www.irccloud.com/pastebin/zk6NdIwe/ | 13:51 |
| mup | PR snapd#8192 opened: tests: add more debug output to the snapd-failure handling <Created by mvo5> <https://github.com/snapcore/snapd/pull/8192> | 13:52 |
| zyga | hmm, how to set syslog priority level | 13:55 |
| zyga | mvo: I'll skip standup; my update is a deep-dive into logind, pam, ssh, getty and systemd in order to write proper tests for the issue discovered by jamie | 13:58 |
| ijohnson | hey folks | 13:59 |
| zyga | mvo: I'll update the standup notes later today, as I've been taking notes on with pen&paper | 13:59 |
| ijohnson | zyga: sounds like fun :-) | 13:59 |
| zyga | ijohnson: yeah, and useful | 13:59 |
| zyga | I didn't know this at all | 13:59 |
| zyga | but I'm slowly building towards a tool that will let us run stuff as a user in a session of given properties | 14:00 |
| mvo | zyga: ok | 14:07 |
| mborzecki | ijohnson: looks like we need to sync about core20/subiquity PRs | 14:10 |
| mborzecki | ijohnson: opened this one https://github.com/CanonicalLtd/subiquity/pull/638 to master | 14:11 |
| mup | PR CanonicalLtd/subiquity#638: debian/console-conf.service: ensure correct order with respect to snapd services <Created by bboozzoo> <https://github.com/CanonicalLtd/subiquity/pull/638> | 14:11 |
| ijohnson | mborzecki: hmm talk more after we talk after SU about the other thing? | 14:11 |
| mborzecki | ijohnson: and this one to core20 https://github.com/snapcore/core20/pull/25 | 14:11 |
| mup | PR core20#25: hooks/200-console-conf-after.chroot: perform console-conf ordering checks <Created by bboozzoo> <https://github.com/snapcore/core20/pull/25> | 14:11 |
| mborzecki | ijohnson: yeah, let's stay after standup | 14:12 |
| zyga | mborzecki: this is cool | 14:18 |
| zyga | https://www.irccloud.com/pastebin/XACC4Nqv/ | 14:18 |
| zyga | for example, this is running "true" via ssh | 14:20 |
| zyga | https://www.irccloud.com/pastebin/9Bmj2q1M/ | 14:20 |
| zyga | (so much noise and garbage) | 14:21 |
| zyga | it's also interesting as to what PATH is set to specific components, e.g. update-motd | 14:21 |
| zyga | or the number of times python has to run before you get the shell prompt | 14:22 |
| zyga | each time to run lsb_release | 14:22 |
| zyga | mborzecki: comparison of /etc/pam.d/{su,sudo} is interesting | 14:33 |
| zyga | one involves common-session and the other common-session-noninteractive | 14:34 |
| zyga | common-session invokes pam_systemd.so while common-session-noninteractive does not | 14:34 |
| pstolowski | cmatsuoka: hey, you're probably aware of https://bugs.launchpad.net/snapd/+bug/1863886 ? | 14:45 |
| mup | Bug #1863886: snap-bootstrap should validate which ubuntu-data is mounted <core20> <snapd:New> <https://launchpad.net/bugs/1863886> | 14:45 |
| cmatsuoka | pstolowski: ah yes | 14:46 |
| cmatsuoka | pstolowski: we still must check the best way to do that | 14:46 |
| pstolowski | sure | 14:48 |
| cmatsuoka | but yes, it's in my todo list | 14:49 |
| pstolowski | cmatsuoka: ok, i'll assign it to you then | 14:49 |
| cmatsuoka | pstolowski: ok, thanks! | 14:49 |
| mborzecki | ijohnson: added your patch to https://github.com/CanonicalLtd/subiquity/pull/638 | 14:56 |
| mup | PR CanonicalLtd/subiquity#638: debian/console-conf.service: ensure correct order with respect to snapd services <Created by bboozzoo> <https://github.com/CanonicalLtd/subiquity/pull/638> | 14:56 |
| ijohnson | mborzecki: cool sounds good, I hadn't noticed your branch didn't have the other old uc18 specific service | 15:01 |
| mborzecki | ijohnson: can you do a quick review of https://github.com/snapcore/core20/pull/25 maybe? | 15:02 |
| mup | PR core20#25: hooks/200-console-conf-after.chroot: perform console-conf ordering checks <Created by bboozzoo> <https://github.com/snapcore/core20/pull/25> | 15:02 |
| ijohnson | mborzecki: sure looking now | 15:02 |
| mborzecki | ijohnson: thanks! | 15:03 |
| mup | PR core20#24 closed: hooks: delete console-conf hack <Created by anonymouse64> <Closed by anonymouse64> <https://github.com/snapcore/core20/pull/24> | 15:05 |
| * zyga has a small eureka moment!!! | 15:32 | |
| zyga | in case I get hit by a bus on my way home | 15:48 |
| zyga | the magic to run a command in a session | 15:48 |
| zyga | is | 15:48 |
| zyga | systemd-run /sbin/runuser -l USER -c COMMAND | 15:49 |
| zyga | one can also pass --send-sighup and --tty to systemd-run, for instant gratification | 15:49 |
| zyga | the rationale is as follows: | 15:49 |
| zyga | systemd-run runs the rests as a servicem outside of our existing logind session | 15:50 |
| zyga | runuser -l invokes pam_systemd which asks logind for a new session - this normally fails if we already have one (hence systemd-run) | 15:50 |
| zyga | the session lives for the duration of the executed command | 15:50 |
| zyga | I'll work on making this nice and useful for writing tests | 15:50 |
| zyga | CC ^ mborzecki | 15:51 |
| ijohnson | zyga: nice work that's pretty great to know | 15:51 |
| mborzecki | *magic* | 15:52 |
| zyga | I'll make some test primitives | 15:52 |
| zyga | and adjust our test code where it matters | 15:52 |
| zyga | I have extra notes on how to get debug information | 15:53 |
| zyga | the critical bit was tracing logind code paths to understand that it fails to create a session if the requesting PID already inhabits one | 15:53 |
| zyga | I'm happy about this day now :) | 15:53 |
| zyga | I think I deserve lunch now | 15:54 |
| zyga | I'll head home | 15:54 |
| zyga | and sit some more to write down the code in the evening | 15:54 |
| zyga | CC jdstrand ^ (how to test in a user session above) | 15:54 |
| zyga | I need to check this on more distributions as PAM config matters | 15:54 |
| zyga | mainly to check if there's any important skew from one distro to another | 15:54 |
| jdstrand | zyga: nice! :) | 16:01 |
| zyga | jdstrand: the journey through PAM was interesting | 16:01 |
| zyga | jdstrand: one-more-way-to-setenv | 16:01 |
| zyga | perhaps a way out of some of our environment problems | 16:01 |
| * jdstrand nods | 16:07 | |
| zyga | ogra: is this reliable? https://github.com/ogra1/snapd-docker | 16:18 |
| === alan_g_ is now known as alan_g | ||
| mvo | hm, I get "remote failed to report status" when I try to push to github | 16:38 |
| mvo | anyone else seeing this? pushing a new branch | 16:38 |
| zyga | systemd-run --pipe --quiet --wait /sbin/runuser -l test -c "COMMAND" | 16:39 |
| zyga | wooot | 16:39 |
| zyga | that's what we need :) | 16:39 |
| zyga | https://status.github.com/ | 16:39 |
| zyga | increased error rate | 16:40 |
| zyga | so probably broken | 16:40 |
| zyga | partial outage | 16:40 |
| mvo | aha | 16:40 |
| mvo | it's very annoying, it *may* be the fix for the failover | 16:40 |
| zyga | hahaha | 16:42 |
| zyga | that's indeed a bit annoying | 16:42 |
| zyga | ok, I need to stop coding and wrap up | 16:43 |
| * zyga waits for spread to finish | 16:43 | |
| mup | PR snapd#8193 opened: snapstate: do not restart in undoLinkSnap unless on first install <Created by mvo5> <https://github.com/snapcore/snapd/pull/8193> | 16:43 |
| mvo | ijohnson: I pushed 8193 with something that worked for me at least once, let's hope it proves to be ok | 16:44 |
| ijohnson | mvo: nice I'll try that in combination with 8169 | 16:44 |
| zyga | oooh | 16:44 |
| zyga | wow | 16:44 |
| zyga | mvo: is this intentional? https://github.com/snapcore/snapd/pull/8193/files#diff-5a87044d1fc20ea2664be6c683755bffR129 | 16:45 |
| mup | PR #8193: snapstate: do not restart in undoLinkSnap unless on first install <Created by mvo5> <https://github.com/snapcore/snapd/pull/8193> | 16:45 |
| ijohnson | zyga: I have that in my branch too | 16:45 |
| ijohnson | zyga: it's quite useful I think because if we are at the point where snap-failure is running, something has went wrong so having debug info from snapd when it is invoked from snap-failure is very useful | 16:46 |
| zyga | I don't doubt usefulness | 16:46 |
| zyga | I wonder what the consequences of running with debug are | 16:46 |
| ijohnson | it's rather quiet because in this case snapd should only revert the snapd snap and exit | 16:46 |
| zyga | I'll review stuff with clear head later | 16:48 |
| zyga | I think working outside is a bit more tiresome than indoors | 16:48 |
| zyga | but this was a good day | 16:48 |
| zyga | (at least for me) | 16:48 |
| * zyga EODs | 16:50 | |
| mvo | zyga: yeah, it's on prupose, like ijohnson says, the output is actually quite short but very useful | 16:51 |
| cjwatson | zyga: FYI I released man-db 2.9.1 with the /snap/man change | 17:23 |
| cjwatson | as discussed | 17:23 |
| ijohnson | mvo: I've done 3-4 spread runs of your branch with mine on top on core18 qemu and haven't run into the race yet | 17:28 |
| ijohnson | but to be fair I was only seeing it once every 30 or so runs, so not sure how excited to get about that data point | 17:29 |
| zyga | mvo: ack, I just wanted to check | 17:33 |
| * zyga is back with fixed bikes | 17:33 | |
| zyga | cjwatson: ack, thank you | 17:33 |
| zyga | cjwatson: I'll look at the state on our side soon | 17:33 |
| * zyga spawns some more tests and goes for dinner | 17:34 | |
| * zyga is starving | 17:34 | |
| mvo | ijohnson: ohhhhhh | 17:38 |
| mvo | ijohnson: fingers crossed - I will stop irc now and pretend the bug is fixed :) | 17:38 |
| ijohnson | haha | 17:38 |
| * mvo hugs ijohnson | 17:41 | |
| zyga | drat | 18:28 |
| zyga | my laptop suspended while running tests | 18:28 |
| zyga | oh well | 18:28 |
| kyrofa | Hey zyga, when I run `snap login`, does that macaroon ONLY have the ability to install snaps? Nothing else? | 18:49 |
| zyga | kyrofa: mmmm, no | 18:49 |
| zyga | kyrofa: I think that is equivalent to root | 18:49 |
| kyrofa | What other possibilities are there? | 18:49 |
| zyga | kyrofa: the details are in api.go | 18:49 |
| zyga | kyrofa: you can remove snaps, configure them, etc | 18:49 |
| zyga | kyrofa: let me check | 18:50 |
| kyrofa | zyga, right, but none of those things relate to the macaroon | 18:50 |
| kyrofa | For example, a macaroon can have the ability to upload snaps as well, which we use in snapcraft | 18:50 |
| zyga | I think it is subtle | 18:50 |
| zyga | one moment | 18:50 |
| kyrofa | Alright, thanks :) | 18:50 |
| zyga | https://github.com/snapcore/snapd/blob/master/daemon/api.go#L192 | 18:51 |
| zyga | look at all the UserOK: true things | 18:51 |
| zyga | those can be done as an authenticated user | 18:51 |
| zyga | you can get app icon, search the store, manage snaps, interfaces, changes, and more | 18:52 |
| kyrofa | zyga, does the polkit integration end up with the same permissions? | 18:52 |
| zyga | not quite | 18:52 |
| zyga | for example an authenticated user can search | 18:52 |
| zyga | and that doesn't require any polkit permissions | 18:52 |
| zyga | UserOK means a non-root can GET the endpoint | 18:53 |
| kyrofa | zyga, to clarify: if app A uses polkit and app B runs `snap login`, do they end up being able to do the same things, or does one have more ability than the other? | 18:53 |
| zyga | ah wait | 18:53 |
| zyga | wait | 18:53 |
| * zyga is misreading stuff | 18:53 | |
| zyga | kyrofa: I think they are not identical and thus could be different | 18:54 |
| zyga | for example, /v2/logs has PolkitOK | 18:54 |
| zyga | so it requires a polkit prompt to see | 18:55 |
| kyrofa | Ah ha | 18:55 |
| zyga | https://github.com/snapcore/snapd/blob/master/daemon/daemon.go#L132 is relevant | 18:55 |
| kyrofa | zyga, exactly what I needed | 18:55 |
| zyga | https://github.com/snapcore/snapd/blob/master/daemon/daemon.go#L139 | 18:56 |
| zyga | this seems to suggest that if you are logged in | 18:56 |
| zyga | then you can access anything that's not specifically walled off | 18:56 |
| zyga | so if you log in you get more power | 18:56 |
| zyga | which is weird, nothing sets RootOnly? | 18:57 |
| zyga | aha, I see | 18:57 |
| zyga | those are now split up to separate files | 18:57 |
| zyga | kyrofa: creating users require to be root | 18:57 |
| zyga | kyrofa: also profiling snapd | 18:58 |
| kyrofa | Thanks zyga | 19:04 |
| zyga | kyrofa: pleasure :) | 19:12 |
| mup | PR snapd#8194 opened: o/devicestate: unset recovery_system when done seeding <Simple 😃> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8194> | 19:58 |
| mup | PR snapd#8193 closed: snapstate: do not restart in undoLinkSnap unless on first install <⚠ Critical> <Created by mvo5> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/8193> | 20:27 |
| === gremmlyn is now known as gremmly | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!