mupBug #1812605 opened: snap find crached <Snappy:New> <https://launchpad.net/bugs/1812605>03:41
zygaA bit in recovery mode07:14
mborzeckizyga: hey07:14
zygaFell off the stairs last night07:14
mborzeckizyga: oi, are you ok?07:14
zygaI may join standup but don’t wait for me07:14
zygaWhen I can I will work on packaging07:15
zygaSo so07:15
mborzeckizyga: was the trip back ok at least? :)07:15
zygaSome bruises and stuff like that. Some more back pain07:15
zygaCurrently with wife for pregnancy checkup07:16
zygaYeah, the trip was ok :-)07:16
zygaJust long and not much sleep on the way07:16
mupPR snapd#6294 closed: packaging/ubuntu: build with golang 1.10 <Core18> <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6294>08:09
mborzeckimvo: hey08:12
mupPR snapd#6391 closed: tests: simplify interfaces-contacts-service test <Created by sergiocazzolato> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6391>08:15
mvohey mborzecki - good morning!08:16
mvomborzecki: yeah, indeed yay :)08:16
mborzeckimvo: 1.9 is the new minimum now?08:16
mvomborzecki: correct, we should probably update our readme too08:16
mborzeckimvo: hm there's no metion of any particular go version in the readme08:17
mvomborzecki: yeah, I think we soon add a mention08:18
mvomborzecki: its not OMGnow important though :)08:18
mvomborzecki: also I just sent a mail out to foundations, security, sru etc to make sure they know we really did that, we did discuss it but I want to double check noone is unaware08:18
mvogeh, everyone is aware (double negatives are terrible!)08:19
mborzeckiwe can replace golang.org/x/net/context with context now08:19
mborzeckiand update govendor too08:19
mvomborzecki: yes, lets discuss in the standup. I would like to wait until we got feedback from foundations/security etc to make sure we don't need to revert anything. but having PRs will be good08:21
mborzeckii can do a quick cleanup and open a PR if we are greenlit to move forward with this08:22
mvomborzecki: sounds great, please do08:29
mborzeckiwow, we even used ctxhttp08:30
mborzecki35 files changed, 36 insertions(+), 66 deletions(-)08:37
mborzeckinot an awful lot08:37
mborzeckigolang.org/x/net/context/ctxhttp goes away, but x/net/context is imported by tomb, so it has to stay around08:40
mvomborzecki: ok08:41
mborzeckiwe could also drop the check for numerical http return codes08:58
mborzeckiand use consts from net/http08:58
mupPR snapd#6402 opened: spread: increase default kill-timeout to 30min <Created by mvo5> <https://github.com/snapcore/snapd/pull/6402>09:02
Chipacamorning peeps09:09
mvoChipaca: hey, good morning!09:12
mborzeckiChipaca: morning09:12
mupPR snapd#6403 opened: many: cleanup golang.org/x/net/context <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6403>09:17
mvomborzecki: nice, thank you!09:34
mvomborzecki: looks like unit tests are unhappy in this PR, seems like new stuff that the new go checks09:34
mborzeckimvo: mhm, pushing in a minute09:34
mvomborzecki: also only in tests so shouldn't be bad :)09:34
mvomborzecki: no worries, just wanted to let you know09:34
* mvo is quite happy about this PR09:34
mborzeckimvo: and pushed09:45
* Chipaca removes 1.6, installs 1.1010:11
* Chipaca feels like it's a party10:11
* mborzecki can drop 1.6 gofmt wrapper now10:12
Chipacaoooh, there's a bunch of things we can drop10:15
mborzeckiChipaca: can you take a look at #6403 ?10:15
mupPR #6403: many: cleanup golang.org/x/net/context <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6403>10:15
Chipacamborzecki: egrep -r --exclude-dir vendor '// \+build \!?go'10:16
ChipacaI need to double-check but IIRC ctrl17 was good up to 1.10 at least10:17
mborzeckiChipaca: was there some fancy json decoding too?10:17
Chipaca1.11 might be using an even newer unicode but ctrl didn't see much change10:17
Chipacamborzecki: ctrl is part of that, if you mean the 'clean' json10:18
Chipacamborzecki: there are places where we use *json.RawMessage instead of json.RawMessage, that we could drop10:18
Chipacaand see some perf improvement10:18
mborzeckipr by pr :P10:18
Chipacahad to get https://packages.ubuntu.com/bionic/amd64/golang-golang-x-tools by hand to get godoc to play nice again10:30
Chipaca(i'm on xenial here)10:30
* sparkiegeek wonders if Chipaca could have used https://launchpad.net/adapt10:31
Chipacasparkiegeek: no idea what that is :-)10:33
Chipacasparkiegeek: no need to rebuild the package, it just works as is10:34
sparkiegeekChipaca: it installs Ubuntu packages from 'other' series in a LXD and wraps them to make a cheap way of getting package X available from a given series10:35
* Chipaca imagines using X from bionic o n xenial, and shudders10:35
sparkiegeekhah :)10:36
mvohey zyga11:19
zygahey mvo :)11:19
mvozyga: welcome back!11:19
mvozyga: but please take it easy today11:20
zygamvo: I'm sorry about expensify, it's super easy to click the wrong button while on the go11:20
zyga(about that "let me expense one coffee thing")11:20
zygaI'm sorting out my travel topics now11:20
zygaand recovering after the small accident yesterday :)11:20
zygabut all will be fine11:20
zygaanything urgent, any news?11:21
mvozyga: no fires11:22
mvozyga: your go-1.10 pr is in \o/11:22
zygaI noticed, thank you, good idea to notify other teams about that11:23
mborzeckizyga: looking into bcond_with{,out}, do we have 15.0 image for spread?11:49
zygamborzecki: mmm, not sure11:51
zygaif you have a branch I have all the VMs ready11:51
zygaone thing11:51
zygatl;dr; I spent last week working on top of ubuntu in hyper-v and multipass also on top of hyperv11:51
zygamborzecki: I think we could start standardizing on using multipass for test-builds on native kernel11:52
zygait's very flexible and qucick11:52
mborzeckidoes it work well with non ubuntu vms?11:52
zygaonly requirement is cloud-init in the image11:52
zygaI've got a pipeline for debian builds now11:52
zygaI will give this a try (though not with any priority today)11:53
zygamborzecki: it's as simple as "multipass launch $URL-to-qcow2 -c 10 -m 4G -n vm-name11:54
zygathen you have multipass commands to exec stuff inside, copy stuff around, shell interactively11:54
zygamultipass handles networking and caching11:54
mborzeckiah, haven't tried anything else than ubuntu with multipass yet11:54
mborzeckiwhat if the image has no cloud-init?11:55
zygaI think it's a requirement11:55
zygaask Saviq once he's back11:55
SaviqI'm here11:55
mborzeckiSaviq: is cloud-init a hard requirement?11:55
Saviqyes, otherwise we can't access the launched instance11:56
zygaSaviq: did I say thank you for multipass, it's awesome11:56
Saviqmborzecki: otherwise you'd need to know (and pass it to us) a default username/password11:56
Saviqand we don't support that right now, nor look forward to it really11:57
zygamborzecki: does arch have any cloud images?11:57
zygaeverything else we support has them11:57
Saviqanything else, we'd need to expose a graphical and/or serial console11:58
mupPR snapd#6404 opened: snapstate: auto transition on experimental.snapd-snap=true <Created by mvo5> <https://github.com/snapcore/snapd/pull/6404>11:58
Saviqand it'd be a hands-free experience from then as far as multipass is concerned11:58
mborzeckizyga: nah :P no cloud images afaik11:58
mborzeckiyou can probably build one11:58
mborzeckiwell, we built one for spread11:58
mborzeckibut it's not using cloud-init afaik11:58
Saviqprobably has a well-know root password :)11:59
mborzeckiyup, something no trivial, like 123 :P11:59
Saviqmay very well be possible to use those12:00
mborzeckiSaviq: while at it, is it possible to launch throw-away instances with multipass? i often work directly with cloud images and use qemu .. -snapshot12:02
Saviqmborzecki: we have a handful of approaches planned for speeding up boot, but we're unlikely to add full snapshot support any time soon, don't want to become just another CLI for hypervisors12:04
mupPR snapd#6405 opened: run-checks: ensure we use go-1.10 if available <Simple 😃> <Created by mvo5> <https://github.com/snapcore/snapd/pull/6405>12:13
zygamvo: mount error still happens13:35
zygaerror: cannot perform the following tasks:13:35
zyga- Mount snap "network-consumer" (unset) ([start snap-network\x2dconsumer-x1.mount] failed with exit status 1: Job for snap-network\x2dconsumer-x1.mount failed.13:35
zygaSee "systemctl status "snap-network\\x2dconsumer-x1.mount"" and "journalctl -xe" for details.13:35
zygathis is from the PR above13:35
Chipacazyga: is the go version relevant enough to be in 'snap version'?13:46
zygaChipaca: dunno13:46
Chipacazyga: or would this be a case for a --verbose flag or sth?13:46
zygaor even a debug13:46
zygasnap debug build-info13:46
Chipacai mean, if it's for the tests, just add a 'go version' :-)13:46
zygacould show other helpful stuff13:46
zygago version can be elusive :)13:46
zygaif we have many13:46
Chipacaooh, ooh! I remembered what I wanted to do when we moved past 1.7!13:51
zygasorry for cutting me video feed but I'm out of power on the phone14:08
Chipacazyga: we lost you14:13
mvozyga: yeah, cachio mentioned this as well, on what system did that happen for you?14:22
zygamvo: that was in the PR you sent, I don't recall14:32
zygamvo: I only saw it there14:32
zygasmall helper I use for packaging work14:38
zygapackaigng helper https://www.irccloud.com/pastebin/oPwDE5pt/14:38
zygamvo: ^14:38
zygamborzecki: ^14:39
mborzeckihm we should add a multipass backend to spread :)14:41
mupPR # closed: snapd#5644, snapd#5822, snapd#5962, snapd#6016, snapd#6034, snapd#6079, snapd#6098, snapd#6106, snapd#6108, snapd#6111, snapd#6162, snapd#6177, snapd#6238, snapd#6252, snapd#6258, snapd#6270, snapd#6280, snapd#6281, snapd#6313, snapd#6320, snapd#6322, snapd#6324, snapd#6325,14:43
mupsnapd#6327, snapd#6329, snapd#6333, snapd#6341, snapd#6347, snapd#6348, snapd#6356, snapd#6360, snapd#6363, snapd#6367, snapd#6376, snapd#6380, snapd#6381, snapd#6383, snapd#6387, snapd#6389, snapd#6394, snapd#6396, snapd#6400, snapd#6401, snapd#6402, snapd#6403, snapd#6404, snapd#640514:43
mupPR # closed: core-build#11, core-build#22, core-build#26, core-build#3714:43
cachiomvo, should I remove the manual from debian sid after the imagge is updated?15:07
mvocachio: yeah, lets try that15:07
mvocachio: once its updated, lets push a PR that enables sid again15:07
cachiomvo, sure, thanks15:07
mvocachio: and then we can check how well that goes - it will be a good prep for the packaging merge PR. thank you!15:07
=== sgclark is now known as sgmoore
* cachio lunch15:54
om26erpopey, ping16:28
popeyom26er: hi16:28
om26erpopey, could you take a look at this one https://github.com/asciinema/asciinema-snap/issues/1#issuecomment-45452923216:28
popeywill do16:28
mupPR snapd#6406 opened: tests: enable debian sid as part of the main suite on travis <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6406>16:42
zygamy debian foo has grown the SBUILD_SHELL trick16:43
mvoI uploaded new test-snapd-dbus-{consumer,producer} snaps to edge, if tests suddenly explode, this is why16:53
mvo(the PR below addresses this)16:56
mupPR snapd#6407 opened: tests: get test-snapd-dbus-{provider,consumer} from the beta channel <Created by mvo5> <https://github.com/snapcore/snapd/pull/6407>16:56
mupPR snapd#6408 opened: tests: add spread test for system dbus interface <Created by mvo5> <https://github.com/snapcore/snapd/pull/6408>17:01
zygaI solved the debian unit test failures :)17:41
cachiozyga, this is the error you have fixed? https://paste.ubuntu.com/p/cmQTKpDhh6/18:56
zygathis is a known issue about go fmt changing behavior across go versions18:57
cachiozyga, ahh, ok19:00
mupBug #1812751 opened: Several remove event for unknown device <Snappy:New> <https://launchpad.net/bugs/1812751>21:01

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!