/srv/irclogs.ubuntu.com/2021/05/06/#snappy.txt

mborzeckimorning05:29
pstolowskimorning07:13
mborzeckipstolowski: heya07:19
mborzeckipstolowski: do you need any reviews?07:20
pstolowskimborzecki: atm no, thanks07:21
mupPR snapd#10241 opened: tests: set memory limit for snapd <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10241>08:25
mupPR snapd#10242 opened: Linter overlord <Created by mardy> <https://github.com/snapcore/snapd/pull/10242>09:05
mborzeckiheh, so somehow socket.gethostbyname('localhost') fails on opensuse TW with `socket.gaierror: [Errno -2] Name or service not known`09:28
pedronismmh09:33
pedronisthat is a bit weird, no?09:33
pedronismborzecki: is this locally? or in the testes? are we not running some resolving service?09:33
pedronis*tests09:34
mborzeckipedronis: 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 me09:35
mborzeckihmm the same snap does work though on my laptop running tw09:37
pstolowskipedronis: hi, i think he have a bad bug with prereqs,  https://github.com/snapcore/snapd/pull/1024309:38
mupPR #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
mupPR 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
pedronispstolowski: that's a complicated thing09:43
pedronispstolowski: we dont have a concept of required interfaces, though many in fact sort of are09:45
pstolowskiright, i see09:51
mborzeckihmm funny installing nss-mdns fixes it09:52
pedronispstolowski: basically there might no fix that doesn't involve clarifying that somehow09:55
pedronis*be no fix09:55
mupPR snapd#10244 opened: tests: more fixes for spread suite on openSUSE <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10244>10:15
pstolowskimardy: i've reviewed your linter PR with 2 suggestions11:15
mardypstolowski: thanks, will look at it in a short while11:32
mupPR 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
pstolowskimardy: ^ nice!12:18
mvomardy: nice!12:19
mupPR 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
mardypstolowski, mvo: thanks, but the tests were wrong. Now it should be ok12:28
pstolowskimardy: yeah, i wonder if TestTasksLastQuestionmark doesn't need updating too?12:31
mardypstolowski: 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.go12:40
pstolowskimardy: aah ok12:40
mardyunless, of course, I should change also last.go12:40
pstolowskimardy: i was wondering about that, not sure. could be a separate PR anyway12:46
mupPR 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
mupPR snapd#10247 opened: tests: update spread url <Run nested> <Simple 😃> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10247>14:01
* cachio lunch14:55
mupPR 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!