mborzecki | morning | 05:29 |
---|---|---|
pstolowski | morning | 07:13 |
mborzecki | pstolowski: heya | 07:19 |
mborzecki | pstolowski: do you need any reviews? | 07:20 |
pstolowski | mborzecki: atm no, thanks | 07:21 |
mup | PR snapd#10241 opened: tests: set memory limit for snapd <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10241> | 08:25 |
mup | PR snapd#10242 opened: Linter overlord <Created by mardy> <https://github.com/snapcore/snapd/pull/10242> | 09:05 |
mborzecki | heh, so somehow socket.gethostbyname('localhost') fails on opensuse TW with `socket.gaierror: [Errno -2] Name or service not known` | 09:28 |
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:33 |
pedronis | *tests | 09:34 |
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:35 |
mborzecki | hmm the same snap does work though on my laptop running tw | 09:37 |
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:38 |
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:40 |
pedronis | pstolowski: that's a complicated thing | 09:43 |
pedronis | pstolowski: we dont have a concept of required interfaces, though many in fact sort of are | 09:45 |
pstolowski | right, i see | 09:51 |
mborzecki | hmm funny installing nss-mdns fixes it | 09:52 |
pedronis | pstolowski: basically there might no fix that doesn't involve clarifying that somehow | 09:55 |
pedronis | *be no fix | 09:55 |
mup | PR snapd#10244 opened: tests: more fixes for spread suite on openSUSE <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10244> | 10:15 |
pstolowski | mardy: i've reviewed your linter PR with 2 suggestions | 11:15 |
mardy | pstolowski: thanks, will look at it in a short while | 11:32 |
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> | 11:55 |
pstolowski | mardy: ^ nice! | 12:18 |
mvo | mardy: nice! | 12:19 |
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:20 |
mardy | pstolowski, mvo: thanks, but the tests were wrong. Now it should be ok | 12:28 |
pstolowski | mardy: yeah, i wonder if TestTasksLastQuestionmark doesn't need updating too? | 12:31 |
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:40 |
pstolowski | mardy: i was wondering about that, not sure. could be a separate PR anyway | 12:46 |
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> | 13:01 |
=== alan_g_ is now known as alan_g | ||
mup | PR snapd#10247 opened: tests: update spread url <Run nested> <Simple 😃> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10247> | 14:01 |
* cachio lunch | 14:55 | |
mup | PR snapd#10249 opened: tests: adding support for fedora-34 <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10249> | 22:37 |
=== tomreyn_ is now known as tomreyn |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!