[02:32] <mup> PR snapcraft#3263 closed: colcon v2 plugin: honour http(s) proxy for all build commands <bug> <Created by cjp256> <Closed by cjp256> <https://github.com/snapcore/snapcraft/pull/3263>
[06:23] <zyga-x240> o/
[06:23] <zyga-x240> mvo: I saw your message yesterday
[06:23] <zyga-x240> oh boy I open my laptop and Lucy wakes up
[06:24] <mvo> zyga-x240: good morning
[06:24] <zyga-x240> :)
[06:24] <mvo> zyga-x240: no worries, either way is fine
[06:25] <zyga-x240> until she's fully awake, changed and fed I will not be fully available
[06:25] <zyga-x240> just a few more days till next week
[06:26] <mvo> zyga-x240: no worries
[06:32] <mvo> can someone please approve 9214? it's just changelog updaes
[06:40] <mup> PR snapd#9215 closed: mkversion.sh: do not use git describe --dirty <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/9215>
[06:45] <mup> PR snapd#9215 opened: mkversion.sh: do not use git describe --dirty <Created by mvo5> <https://github.com/snapcore/snapd/pull/9215>
[07:04] <pstolowski> morniing
[07:05] <mvo> good morning pstolowski
[07:18] <zyga> re
[07:18] <zyga> mvo looking
[07:19] <zyga> mvo merged
[07:20] <mup> PR snapd#9214 closed: release: 2.46 <Skip spread> <Created by mvo5> <Merged by zyga> <https://github.com/snapcore/snapd/pull/9214>
[07:23] <mvo> zyga: \o/
[07:24] <zyga> mvo in a way it's good we'll need 2.47
[07:24] <zyga> we should release it in a few weeks at most
[07:24] <zyga> and get back to small releases
[07:24] <zyga> that are fast
[07:25] <mup> PR snapd#9219 opened: run-checks: only check files in git for misspelling <Created by mvo5> <https://github.com/snapcore/snapd/pull/9219>
[07:27] <zyga> mvo reviewed with a question
[07:28] <mvo> zyga: \o/
[07:28] <zyga> mvo I'm available now, shall I pick up snapshot export PR?
[07:28] <mvo> zyga: if you want, I did not manage to and have meetings soon :/
[07:28] <zyga> ok
[07:28]  * mvo hugs zyga
[07:28] <zyga> gladly
[07:28] <zyga> on it
[07:28] <zyga> it's a new topic
[07:28] <zyga> that feels good sometimes
[07:28] <zyga> btw, offtopic, my lxd install stopped working
[07:28] <zyga> it says it needs manual network configuration
[07:29] <zyga> as it cannot find a free subnet or something
[07:29] <zyga> I have a 10.x.y.z/8 netmask
[07:29] <zyga> so I'm quite surprised
[07:31] <zyga> mvo shellcheck complained about the spell changing
[07:34] <mvo> zyga: yeah, I am a muppet
[07:34] <mvo> zyga: I force pushed a fix, I don't have $GOPATH/bin in my $PATH
[07:34] <zyga> k
[07:34] <mvo> zyga: so for local testing I did that (I don't like the idea of something like $GOPATH/bin in my PATH, feels too dangerous)
[07:34] <mvo> anyway
[07:34] <mvo> silly me
[07:34] <zyga> it's okay
[07:35] <mvo> zyga: I replied, I can add comments about where the ignore comes from, maybe a good idea
[07:36] <zyga> mvo replied too
[07:50] <mup> PR snapd#9205 closed: boot/initramfs_test.go: reset boot vars on the bootloader for each iteration <Simple 😃> <Test Robustness> <UC20> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9205>
[09:08] <zyga> mvo I've updated https://github.com/snapcore/snapd/pull/8982 and left two unresolved comments:
[09:08] <mup> PR #8982: snapshots: export of snapshots <Created by slimjim777> <https://github.com/snapcore/snapd/pull/8982>
[09:08] <zyga> mvo one is about the help message of export-snapshot, I need to ask Graham for help here
[09:08] <zyga> it can also happen in a follow up I guess but it might be easy enough to do todat
[09:08] <zyga> *today
[09:09] <zyga> the second one is about error handling in https://github.com/snapcore/snapd/pull/8982#discussion_r476372693
[09:09] <zyga> and I think I need pstolowski help on that
[09:09] <zyga> can you look and express your opinion please pawel?
[09:09] <zyga> other code seems to handle errors from requests in a different way
[09:09] <zyga> and can use the body to unpack error details
[09:09] <zyga> we should either be consistent about it or leave a comment why we are not doing this
[09:10] <zyga> mvo I left the tar improvements out for a new PR
[09:10] <zyga> and adjusted everything else
[09:10] <zyga> so it means someone other than me should do a review
[09:10] <mvo> zyga: in a meeting, but *thanks*
[09:11] <pstolowski> zyga: sure
[09:15] <pedronis> I can look about the error handling as well but in a bit, I suspect the difference might relate that the happy response is not json here
[09:15] <pedronis> but I suspect the error ones should be
[09:19] <zyga> yeah, I think so too
[09:19] <zyga> thanks!
[09:20] <zyga> I'll return to my other tasks and try to finish the tar optimize after PT in the evening
[09:26] <pstolowski> zyga: it looks like e.g. maintenance status on error may be interesting to have
[10:03] <mvo> zyga: just looked over your changes, looks fine
[10:03] <mvo> zyga: thank you!
[10:05] <zyga> mvo cheers :)
[10:05] <mvo> pstolowski: I wonder if the client error handling should be done in a followup, I'm not sure we need more here tbh but even if it seems the exiting code is not wrong, just that it could be improved(?)
[10:05] <mvo> pstolowski: I'm in favor of merging (if green :)
[10:05] <pstolowski> mvo: yes, absolutely, a followup is gine
[10:05] <pstolowski> *fine
[10:05] <zyga> mvo I'm okay with that as well
[10:26] <pstolowski> hmm services PR failed and the failure looks real on 14.04
[10:41] <mup> PR snapd#9220 opened: o/snapstate: disk space check with single snap install <Disk space awareness> <⛔ Blocked> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9220>
[10:55] <zyga> pstolowski how did it fail?
[10:56] <pstolowski> zyga: state of services not as expected
[10:56] <zyga> mmm
[10:56] <pstolowski> oddly only on 14.04. i'm going to investigate in a bit
[10:57] <zyga> what was the unexpected status?
[11:06] <pstolowski> zyga: enabled/inactive instead of disabled/inactive
[11:06] <mup> PR snapd#9221 opened: tests: disk space awareness spread test <Disk space awareness> <⛔ Blocked> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9221>
[11:16] <mup> PR snapd#8982 closed: snapshots: export of snapshots <Created by slimjim777> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8982>
[11:21] <mup> PR snapd#9199 closed: snapstate: installSize helper that calculates total size of snaps and their prerequisites <Disk space awareness> <Needs Samuele review> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9199>
[11:31] <pedronis> pstolowski: should I re-review the service PR or wait?
[11:51] <mup> PR snapd#9222 opened: osutil: add a package doc comment (via doc.go) <Simple 😃> <Created by pedronis> <https://github.com/snapcore/snapd/pull/9222>
[12:04] <pstolowski> pedronis: please wait, i'll let you know
[12:04] <pedronis> pstolowski: ok, thx
[12:05] <pedronis> pstolowski: when you have a moment, can you merge master into #9211 ?
[12:05] <mup> PR #9211: o/snapstate: disk space check with InstallMany <Disk space awareness> <⛔ Blocked> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9211>
[12:07] <pstolowski> yes
[12:11] <pedronis> mvo: #9222 is small/simple and touches something you did
[12:11] <mup> PR #9222: osutil: add a package doc comment (via doc.go) <Simple 😃> <Created by pedronis> <https://github.com/snapcore/snapd/pull/9222>
[12:58] <mvo> pedronis: thank you
[13:01] <mup> PR snapd#9206 closed: boot: complain about reused asset name during initial install <UC20> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9206>
[13:01] <mup> PR snapd#9222 closed: osutil: add a package doc comment (via doc.go) <Simple 😃> <Created by pedronis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9222>
[13:09] <Chipaca> hey-o
[13:09] <Chipaca> i've recently found that i need to restart snapd.socket manually every so often
[14:06] <mvo> pstolowski: 9211 has some unhappy unit test but looks great otherwise
[14:08] <pstolowski> mvo: ah, i forgot to rename after mergin master, thanks
[14:08] <cachio> zyga, hey
[14:08] <cachio> about the error related to the failober tests
[14:09] <zyga> re
[14:09] <cachio> of rexternal backend
[14:09] <zyga> Chipaca, oh what's the state of the socket when you do?
[14:09] <zyga> cachio, yeah?
[14:09] <cachio> do you have in mind anython to fix that?
[14:09] <Chipaca> zyga: a fair question. i'll look next time it happens.
[14:09] <zyga> cachio sorry, the failure we talked about before or some new one?
[14:09] <zyga> Chipaca I did observe something weird related to pulse socket like that
[14:09] <zyga> oh, sorry
[14:09] <zyga> not to the pulse socket
[14:09] <cachio> zyga, the one we talked before
[14:09] <zyga> to the snapd socket in fact
[14:10] <zyga> it was related to snapd-userd socket
[14:10] <zyga> when we changed the socket definition
[14:10] <zyga> (just rewrite it)
[14:10] <zyga> and not reload systemd-userd
[14:10] <cachio> zyga, which failed because external backend does "sudo su to run the spread script"
[14:10] <zyga> cachio right
[14:10] <zyga> cachio that's fundamentally broken sadly :/
[14:11] <zyga> cachio we have to change that
[14:11] <zyga> it should, at minimum, do sudo -l
[14:11] <cachio> zyga, so change spread
[14:11] <zyga> er, wait
[14:11] <zyga> not -l
[14:11] <zyga> su -l, for sudo that's ....
[14:12] <mup> PR snapd#9217 closed: run-checks: add typos from auto-tools when using `make hack` <Simple 😃> <Test Robustness> <Created by anonymouse64> <Closed by anonymouse64> <https://github.com/snapcore/snapd/pull/9217>
[14:12] <zyga> cachio sudo -i (or sudo --login for clarity)
[14:12] <zyga> not tested
[14:12] <zyga> but that should be much better if it works as I expect
[14:12] <zyga> if that's convenient, change spread and see if this fixes it
[14:12] <cachio> zyga, ok, I'll make the change
[14:12] <zyga> ok
[14:12] <cachio> zyga, thanks for the suggestion
[14:13] <zyga> Chipaca in any case, thanks for the heads up
[14:13] <zyga> I'm running snapd without rebooting on metal to observe weird things as well but I have been unable to spend the time to dig into the things I saw
[14:13] <zyga> (insert commas for best sense)
[14:16] <ijohnson> zyga: mvo: so for #9216, what should I do? the changelog typo fix is not on release/2.46 so I'm not sure what to force push ?
[14:16] <mup> PR #9216: vendor.json: update mysterious secboot SHA again <⚠ Critical> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9216>
[14:17] <mup> PR snapd#9215 closed: mkversion.sh: do not use git describe --dirty <⚠ Critical> <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/9215>
[14:17] <zyga> ijohnson I was unclear, sorry, I expected to find only the hash change, not the typo fixes
[14:17] <ijohnson> zyga: sure but then no tests run against the PR if I don't fix the typos here
[14:17] <zyga> are they necessary in this branch?
[14:17] <zyga> oh
[14:17] <zyga> meh
[14:17] <zyga> that sucks, can we just merge the fixes separately?
[14:17] <ijohnson> i.e. open any PR against release/2.46 and tests don't run
[14:17] <zyga> it's not a biggie
[14:18] <ijohnson> I'll leave to mvo to decide what to do, I'll leave the PR as-is for now
[14:18] <zyga> ijohnson due to the typos?
[14:18] <zyga> ok
[14:18] <ijohnson> zyga: yes
[14:18] <zyga> I trust mvo's merge button
[14:18] <ijohnson> the unit / static tests fail on the typos and don't do spread runs
[14:18] <mup> PR snapcraft#3264 closed: colcon v2 plugin: don't strip environment for stage-runtime-dependencies <bug> <Created by cjp256> <Closed by cjp256> <https://github.com/snapcore/snapcraft/pull/3264>
[14:18] <ijohnson> normally I'd say it's fine but this sha thing is mysterious enough to make me want to see spread results of building with this vendor.json change
[14:20] <zyga> ijohnson I would just merge the typos separately
[14:20] <zyga> and then rebase this branch
[14:20] <zyga> so we can see a single thing to track, not that with other bits
[14:22] <ijohnson> zyga: sure but I would rather wait til mvo says what he prefers
[14:22] <zyga> ok
[14:23] <mup> PR snapcraft#3265 opened: colcon v2 plugin: honour http(s) proxy for stage-runtime-dependencies <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3265>
[14:29] <pedronis> mvo: pstolowski: there is no daemon support yet for ErrInsuffiecientSpace ? also that name is not following style conventions
[14:30] <pstolowski> pedronis: correct, no daemon yet
[14:35] <cachio> zyga, so currently it is doing "sudo -i /bin/bash /tmp/tmp.zwRt33mi04"
[14:35] <cachio> to execute the script
[14:36] <pedronis> pstolowski: I left some comments
[14:36] <zyga> cachio currently as in with some changes or vanilla?
[14:37] <mvo> ijohnson, zyga sorry, meetings will look at the backlog
[14:37] <cachio> zyga, vanilla
[14:37] <zyga> cachio ok, perhaps it needs some more oomph
[14:37] <pstolowski> pedronis: ty
[14:38] <zyga> can you try changing that to su -l
[14:38] <cachio> zyga, sure
[14:40] <ijohnson> mvo: I opened https://github.com/snapcore/snapd/pull/9223 to fix the --dirty handling in mkversion.sh
[14:40] <mup> PR #9223: mkversion.sh: simple hack to include dirty in version if the tree is dirty <Bug> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9223>
[14:41] <ijohnson> hopefully that is the best of all possible worlds (but all possible worlds are still terrible)
[14:41] <ijohnson> mvo: also please advice on next steps for #9216
[14:41] <mup> PR #9216: vendor.json: update mysterious secboot SHA again <⚠ Critical> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9216>
[14:42] <mup> PR snapd#9223 opened: mkversion.sh: simple hack to include dirty in version if the tree is dirty <Bug> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9223>
[15:07] <mup> PR snapd#9216 closed: vendor.json: update mysterious secboot SHA again <⚠ Critical> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9216>
[15:07] <pedronis> pstolowski: mvo: do we plan to have checks also for refresh ?
[15:08] <mup> PR snapcraft#3266 opened: cli: add --enable-experimental-extensions option for expand-extensions <enhancement> <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3266>
[15:09] <mvo> pstolowski, pedronis: I think that would be good
[15:09] <pstolowski> pedronis: yes i think so
[15:15] <pedronis> pstolowski: mvo: given that we remove old revisions only after installing the new ones, it's probably not too different form install
[15:15] <pedronis> *from
[15:15] <mvo> pedronis: yeah
[15:16] <mvo> pedronis: one interessting point for really low disk space system is that we may need to provide a mode in the future to change this behavior to "remove-before-install"
[15:19] <pedronis> ok, sounds something to think about when we get there
[15:21] <pedronis> it means no rollback unless you keep two old revisions
[15:22] <pedronis> which would take more space
[15:23] <pedronis> mvo: if you really want just one revision there's no undo
[15:28]  * cachio lunch
[15:29]  * mvo nods
[15:42] <mup> PR snapd#9224 opened: Added MIRKey to U2F devices list in interfaces/builtin <Created by kobusgrobler> <https://github.com/snapcore/snapd/pull/9224>
[15:47] <pstolowski> hmm there is something wonky with 'systemctl enable --root=/ on 14.04 ...'
[15:48] <pstolowski> mvo: interesting idea
[15:49] <mvo> pstolowski: all later
[15:49] <pstolowski> sure, ack
[15:49] <mvo> pstolowski: just an idea because we were talking about really low disspace systems
[15:51] <pstolowski> systemctl enable --root=/ on 14.04 creates symlinks sunch as "snap.disabled-svcs-kept.svc.service -> ///etc/systemd/system/snap.disabled-svcs-kept.svc.service" and things seem to go wrong from there
[15:52] <pedronis> pstolowski: what is using enable --root ? a test?
[15:53] <pstolowski> pedronis: Enable(..) in our systemd package, and it's now used in my services PR. no problems except for 14.04
[15:54] <pstolowski> so it's used for real, not in test
[15:55] <pedronis> pstolowski: is that related to preseeding ?
[15:55] <pstolowski> pedronis: no
[15:55] <pedronis> or it was always like that?
[15:56] <pstolowski> pedronis: as far as systemd package is concerned , --root is 1 year old
[15:56] <pedronis> it's related to user services it seems
[15:56] <pstolowski> pedronis: but it's now used for snap start.. in my PR. i need to compare with master
[15:57] <pstolowski> pedronis: likely, the change was from James
[15:57] <pedronis> not sure
[15:57] <pedronis> James didn't change that
[15:58] <pedronis> at least not in the obvious PR
[15:59]  * mvo hockey, available via tg for ugent cases
[16:01] <pstolowski> pedronis: right, he just touched the line but didn't change semantics. anyway, something in my PR to fix, i need to check what did we do in master, most likely --root wasn't and shouldn't be passed in this case
[16:02] <pedronis> afaict even on master we pass it everywhere
[16:02] <pedronis> ah, no
[16:02] <pedronis> sorry, I'm confused is the problem enable or start?
[16:04] <pstolowski> pedronis: start --enable our our side, which translates to separate enable followed by start
[16:06] <pedronis> pstolowski: I see, anyway --root was there since that code started using systemctl in 2016
[16:07] <pedronis> pstolowski: 5106ed3bcb193bf
[16:07] <mup> PR snapd#9225 opened: many: cloud-init cleanups from previous PR's <Cleanup :broom:> <Simple 😃> <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9225>
[16:11] <pstolowski> pedronis: yes, but afaict it wasn't used with this exec-command tasks where systemctl was called directly, that's the thing
[16:11] <pstolowski> anyway i need to eod, appointment
[16:12] <mup> PR snapd#9226 opened: cmd/snap-bootstrap/initramfs-mounts: compute string outside of loop <Cleanup :broom:> <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9226>
[16:17] <pedronis> ijohnson: does #9225 need to run with nested tests enabled?
[16:17] <mup> PR #9225: many: cloud-init cleanups from previous PR's <Cleanup :broom:> <Simple 😃> <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9225>
[16:18] <cachio> zyga, no luck with sudo -l
[16:18] <ijohnson> pedronis: ah yes good point, let me re-open with that label
[16:21] <pedronis> ijohnson: I reviewed the cleanup PRs
[16:21] <ijohnson> thanks
[16:27] <mup> PR snapd#9219 closed: run-checks: only check files in git for misspelling <Skip spread> <Created by mvo5> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/9219>
[16:29] <ijohnson> cachio: can you add to your nested test TODO to fix the regexp here for the ubuntu-16.04-64:tests/nested/core/extra-snaps-assertions test? https://pastebin.ubuntu.com/p/8y5pYY5PQw/
[16:30] <ijohnson> pedronis: the last change you requested re cloud-init was in nested.sh from the private reviews, but I pushed it to https://github.com/snapcore/snapd/pull/9208/commits/9d1a8d64f0eb1242ce912875fc9e9f977fe72fe3 since that branch needed a restart anyways and is already nested.sh related
[16:30] <mup> PR #9208: tests/nested/core20/kernel-failover: add test for failed refresh of uc20 kernel <Run nested> <Test Robustness> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9208>
[16:30] <ijohnson> pedronis: you originally requested it here: https://github.com/anonymouse64/snapd-private/pull/8/files/bbfe9f9fc266d9cc43fe2c45c44620f4ad6ff2d0#r451357700
[16:30] <cachio> ijohnson, to fix?
[16:30] <cachio> let me check
[16:31] <ijohnson> cachio: see the pastebin, the regexp for the core snap is wrong, it doesn't expect to see a pre version there
[16:32] <mup> PR snapd#9227 opened: snap: add stat to the random access file return interface <Simple 😃> <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/9227>
[16:33] <cachio> ijohnson, ahhh, the pre + git
[16:33] <ijohnson> yes
[16:33] <cachio> in my pr
[16:33] <cachio> I am not checkin hte version when the rev ix x1
[16:34] <cachio> because we test that as part of the listing test in main suite
[16:34] <cachio> so what I did is to simplify what we test in case we build core
[16:34] <cachio> ijohnson, does it make sense for you?
[16:35] <ijohnson> cachio: sure if it doesn't make sense to check that you can just remove that check that's fine
[16:35] <ijohnson> just wanted to make sure you knew about the failure
[16:38] <cachio> ijohnson, nice, so that problem is covered in #9098
[16:38] <mup> PR #9098: tests: new organization for nested tests <Run nested> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/9098>
[16:39] <ijohnson> great
[16:51] <zyga> cachio, can you check if if su -l creates a logind session?
[16:51] <cachio> zyga, sure
[16:51]  * zyga feels so so
[16:52] <zyga> I think I may be getting the same thing my wife has
[16:52] <cachio> zyga, go to rest
[16:52] <ijohnson> zyga: oh no
[16:53] <zyga> cachio alternatively try this
[16:53] <zyga> cachio run without sudo
[16:53] <zyga> cachio use sudo to create a systemd unit, maybe via systemd-run or manually
[16:53] <zyga> have the unit run the shell script
[16:54] <cachio> zyga, run the script without sudo?
[16:54] <zyga> and start the unit similarly to how session-tool does it, with runuser -l
[16:55] <zyga> cachio start the script with runuser -l, as the desired user (root) but inside a systemd system unit
[16:55] <zyga> cachio it's all a bit convoluted but that's one scenario that I know should behave correctly
[16:55] <zyga> cachio you can then try to reduce this
[16:56] <cachio> zyga, ok, I'll try that
[17:06] <ijohnson> cmatsuoka: what's the keyboard shortcut to open a debug shell for a uc20 vm again? I seem to remember it was ctrl+f8 or something but that doesn't do anything :-/
[18:04] <zyga> ijohnson, cmatsuoka: please review simple bug fix https://github.com/snapcore/snapd/pull/9228
[18:04] <mup> PR #9228: interfaces/systemd: compare dereferenced Service <Bug> <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/9228>
[18:04] <zyga> customer bug
[18:04] <zyga> and my shame
[18:04] <zyga> as I wrote this
[18:04] <ijohnson> cmatsuoka: hey on your spread runner do you have python installed?
[18:04] <ijohnson> cmatsuoka: see https://github.com/snapcore/snapd/runs/1032308198?check_suite_focus=true
[18:04] <zyga> ijohnson: python-launchpadlib probably
[18:05] <zyga> or python3-launchpadlib
[18:05] <ijohnson> the CLA checker failed because it couldn't find python
[18:05] <zyga> anyway
[18:05] <ijohnson> zyga: sure I'll have a look
[18:05] <cachio> zyga, back to this
[18:05] <cachio> zyga, you said start the script with runuser -l
[18:05] <cachio> which script are you talking about?
[18:05] <zyga> cachio runuser -l is like sudo
[18:05] <cachio> zyga, yes
[18:06] <zyga> but it has the right pam config so it gives you a logind session
[18:06] <zyga> the logind session is only created if the invoking process does not belong to a cgroup which is considered part of a session
[18:06] <cachio> you mean spread has do to that?
[18:06] <cachio> instead of doing sudo -i
[18:06] <zyga> hence the workaround with running a systemd unit to run runuser -l
[18:06] <zyga> yes
[18:06] <cachio> zyga, ah, ok
[18:06] <zyga> or offer a way for us to do this via spread
[18:07] <cmatsuoka> ijohnson: installing python-is-python3
[18:07] <zyga> otherwise, the external user has a session
[18:07] <zyga> and root does not
[18:07] <ijohnson> cmatsuoka: thanks
[18:07] <zyga> and our tests are not compatible wiith  that
[18:07] <cmatsuoka> zyga: will check asap
[18:07] <ijohnson> cmatsuoka: also did you see my ping before you logged off about the debug shell on a VM?
[18:07] <mup> PR snapd#9228 opened: interfaces/systemd: compare dereferenced Service <Bug> <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/9228>
[18:08] <cmatsuoka> ijohnson: yes, I answered it (alt+f9) but maybe it was lost in a reconnection?
[18:08] <ijohnson> cmatsuoka: yeah I didn't see the response
[18:08] <zyga> Thank you for the quick review Ian
[18:08] <zyga> btw, if my wife is okay and she goes back to work next week
[18:08] <zyga> I will shift my hours to start at 11 AM or perhaps even at noon
[18:08] <ijohnson> cmatsuoka: hmm so I added rd.systemd.debug-shell=1 to the kernel command line via grub.cfg but alt-f9 still doesn't work
[18:08] <zyga> to be with lucy in the morning
[18:08] <ijohnson> zyga: that's great that your wife is okay
[18:08] <ijohnson> happy to hear that
[18:08] <zyga> and then hand off to her (my wife will work part time)
[18:09] <zyga> ijohnson: she's not yet, I meant "iff she's okay"
[18:09] <ijohnson> zyga: ah I missed the most critical word there
[18:09] <zyga> ijohnson: so if that all works out I will be working more with you guys :)
[18:09] <ijohnson> :-)
[18:09] <zyga> more overlap during the day
[18:09] <cmatsuoka> zyga: reviewed
[18:10] <zyga> thank you guys!
[18:10] <zyga> complex day
[18:10] <zyga> older kids fighting
[18:11] <zyga> lucy not seeing mom all day, being very grumpy about that
[18:11] <zyga> wife coming home sick
[18:11] <zyga> eh
[18:11] <zyga> eh
[18:11] <zyga> and now bugs
[18:34] <mup> PR snapcraft#3267 opened: ci: re-enable appveyor artifact collection <tooling> <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3267>
[18:43] <cachio> zyga, tried that but the session was not created
[18:44] <cachio> allways have the external and test sessions but can make it for the root session
[18:44] <cachio> I need to go to the kinesiologist now
[18:44] <cachio> I tried manuall
[18:44] <cachio> y
[18:45] <cachio> once I am back I'll continue reading documentation
[18:45] <cachio> zyga, if you have any idea please leave a note
[18:45] <cachio> zyga, thanks
[18:45]  * cachio -> kinesiologist
[19:07] <cmatsuoka> ijohnson: did you add dangerous too?
[19:07] <ijohnson> cmatsuoka: yes I did still didn't work, but I was able to work around it anyways
[19:09] <cmatsuoka> ijohnson: it works for me with regular systemd.debug-shell, it's been a long time since I used rd.systemd but it should work using the same mechanism
[19:11] <cmatsuoka> afk, /me forgot to pay some bills
[21:22] <ijohnson> cachio: I reviewed #9098, I think it is getting close just a few more comments
[21:22] <mup> PR #9098: tests: new organization for nested tests <Run nested> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/9098>
[21:22] <cachio> ijohnson, thanks
[21:23] <ijohnson> np
[21:23] <cachio> ijohnson, any idea how to create a user session for root when I an using external user?
[21:23] <cachio> I tried what zyga suggested
[21:23] <cachio> but no luck
[21:24] <ijohnson> ahh no unfortunately I don't I would need to spend a bit of time looking at the problem first, I haven't looked at the user session stuff in a while
[21:24] <ijohnson> if you still can't get it worked out tomorrow maybe I can have a look tomorrow afternoon
[21:24] <cachio> ijohnson, np, tomorrow I'll continue with zyga
[21:25] <ijohnson> sounds good
[21:25] <cachio> not it is too late for him