[05:29] <mborzecki> morning
[07:13] <pstolowski> morning
[07:19] <mborzecki> pstolowski: heya
[07:20] <mborzecki> pstolowski: do you need any reviews?
[07:21] <pstolowski> mborzecki: atm no, thanks
[08:25] <mup> PR snapd#10241 opened: tests: set memory limit for snapd <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10241>
[09:05] <mup> PR snapd#10242 opened: Linter overlord <Created by mardy> <https://github.com/snapcore/snapd/pull/10242>
[09:28] <mborzecki> heh, so somehow socket.gethostbyname('localhost') fails on opensuse TW with `socket.gaierror: [Errno -2] Name or service not known`
[09:33] <pedronis> mmh
[09:33] <pedronis> that is a bit weird, no?
[09:33] <pedronis> mborzecki: is this locally? or in the testes? are we not running some resolving service?
[09:34] <pedronis> *tests
[09:35] <mborzecki> pedronis: it's the interfaces-network-bind test, the network-bind-consumer snap fails, when resolving 'localhost', tbh even `import socket; socket.gethostbyname('localhost')` fails for me
[09:37] <mborzecki> hmm the same snap does work though on my laptop running tw
[09:38] <pstolowski> pedronis: hi, i think he have a bad bug with prereqs,  https://github.com/snapcore/snapd/pull/10243
[09:38] <mup> PR #10243: o/snapstate: failure of prereqs on content interface doesn't prevent install <Bug> <Skip spread> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10243>
[09:40] <mup> PR snapd#10243 opened: o/snapstate: failure of prereqs on content interface doesn't prevent install <Bug> <Skip spread> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10243>
[09:43] <pedronis> pstolowski: that's a complicated thing
[09:45] <pedronis> pstolowski: we dont have a concept of required interfaces, though many in fact sort of are
[09:51] <pstolowski> right, i see
[09:52] <mborzecki> hmm funny installing nss-mdns fixes it
[09:55] <pedronis> pstolowski: basically there might no fix that doesn't involve clarifying that somehow
[09:55] <pedronis> *be no fix
[10:15] <mup> PR snapd#10244 opened: tests: more fixes for spread suite on openSUSE <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10244>
[11:15] <pstolowski> mardy: i've reviewed your linter PR with 2 suggestions
[11:32] <mardy> pstolowski: thanks, will look at it in a short while
[11:55] <mup> PR snapd#10245 opened: cmd/snap: exit normally if "snap changes" has no changes <Created by mardy> <https://github.com/snapcore/snapd/pull/10245>
[12:18] <pstolowski> mardy: ^ nice!
[12:19] <mvo> mardy: nice!
[12:20] <mup> PR snapd#10246 opened: o/patch: check if we have snapd snap with correct snap type already in snapstate <Created by stolowski> <https://github.com/snapcore/snapd/pull/10246>
[12:28] <mardy> pstolowski, mvo: thanks, but the tests were wrong. Now it should be ok
[12:31] <pstolowski> mardy: yeah, i wonder if TestTasksLastQuestionmark doesn't need updating too?
[12:40] <mardy> pstolowski: no, it doesn't; I actually updated to initially, and that caused the tests to fail. The reason is that that test is actually testing last.go
[12:40] <pstolowski> mardy: aah ok
[12:40] <mardy> unless, of course, I should change also last.go
[12:46] <pstolowski> mardy: i was wondering about that, not sure. could be a separate PR anyway
[13:01] <mup> PR snapd#10244 closed: tests: more fixes for spread suite on openSUSE <Simple 😃> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/10244>
=== alan_g_ is now known as alan_g
[14:01] <mup> PR snapd#10247 opened: tests: update spread url <Run nested> <Simple 😃> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10247>
[14:55]  * cachio lunch
[22:37] <mup> PR snapd#10249 opened: tests: adding support for fedora-34 <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10249>
=== tomreyn_ is now known as tomreyn