[03:41] <mup> Bug #1812605 opened: snap find crached <Snappy:New> <https://launchpad.net/bugs/1812605>
[06:07] <mborzecki> morning
[07:13] <zyga> Hi
[07:14] <zyga> A bit in recovery mode
[07:14] <mborzecki> zyga: hey
[07:14] <zyga> Fell off the stairs last night
[07:14] <mborzecki> zyga: oi, are you ok?
[07:14] <zyga> I may join standup but don’t wait for me
[07:15] <zyga> When I can I will work on packaging
[07:15] <zyga> So so
[07:15] <mborzecki> zyga: was the trip back ok at least? :)
[07:15] <zyga> Some bruises and stuff like that. Some more back pain
[07:16] <zyga> Currently with wife for pregnancy checkup
[07:16] <zyga> Yeah, the trip was ok :-)
[07:16] <zyga> Just long and not much sleep on the way
[08:09] <mup> PR 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:12] <mborzecki> yay
[08:12] <mborzecki> mvo: hey
[08:15] <mup> PR snapd#6391 closed: tests: simplify interfaces-contacts-service test <Created by sergiocazzolato> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6391>
[08:16] <mvo> hey mborzecki - good morning!
[08:16] <mvo> mborzecki: yeah, indeed yay :)
[08:16] <mborzecki> mvo: 1.9 is the new minimum now?
[08:16] <mvo> mborzecki: correct, we should probably update our readme too
[08:17] <mborzecki> mvo: hm there's no metion of any particular go version in the readme
[08:18] <mvo> mborzecki: yeah, I think we soon add a mention
[08:18] <mvo> mborzecki: its not OMGnow important though :)
[08:18] <mvo> mborzecki: 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 unaware
[08:19] <mvo> geh, everyone is aware (double negatives are terrible!)
[08:19] <mborzecki> we can replace golang.org/x/net/context with context now
[08:19] <mborzecki> and update govendor too
[08:21] <mvo> mborzecki: 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 good
[08:22] <mborzecki> i can do a quick cleanup and open a PR if we are greenlit to move forward with this
[08:29] <mvo> mborzecki: sounds great, please do
[08:30] <mborzecki> wow, we even used ctxhttp
[08:37] <mborzecki> 35 files changed, 36 insertions(+), 66 deletions(-)
[08:37] <mborzecki> not an awful lot
[08:38] <mvo> nice
[08:40] <mborzecki> golang.org/x/net/context/ctxhttp goes away, but x/net/context is imported by tomb, so it has to stay around
[08:41] <mvo> mborzecki: ok
[08:58] <mborzecki> we could also drop the check for numerical http return codes
[08:58] <mborzecki> and use consts from net/http
[09:02] <mup> PR snapd#6402 opened: spread: increase default kill-timeout to 30min <Created by mvo5> <https://github.com/snapcore/snapd/pull/6402>
[09:09] <Chipaca> morning peeps
[09:12] <mvo> Chipaca: hey, good morning!
[09:12] <mborzecki> Chipaca: morning
[09:17] <mup> PR snapd#6403 opened: many: cleanup golang.org/x/net/context <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6403>
[09:34] <mvo> mborzecki: nice, thank you!
[09:34] <mvo> mborzecki: looks like unit tests are unhappy in this PR, seems like new stuff that the new go checks
[09:34] <mborzecki> mvo: mhm, pushing in a minute
[09:34] <mvo> mborzecki: also only in tests so shouldn't be bad :)
[09:34] <mvo> mborzecki: no worries, just wanted to let you know
[09:34]  * mvo is quite happy about this PR
[09:45] <mborzecki> mvo: and pushed
[09:49] <mvo> ta
[10:11]  * Chipaca removes 1.6, installs 1.10
[10:11]  * Chipaca feels like it's a party
[10:12]  * mborzecki can drop 1.6 gofmt wrapper now
[10:15] <Chipaca> oooh, there's a bunch of things we can drop
[10:15] <mborzecki> Chipaca: can you take a look at #6403 ?
[10:15] <mup> PR #6403: many: cleanup golang.org/x/net/context <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6403>
[10:16] <Chipaca> mborzecki: egrep -r --exclude-dir vendor '// \+build \!?go'
[10:17] <mborzecki> mhm
[10:17] <Chipaca> I need to double-check but IIRC ctrl17 was good up to 1.10 at least
[10:17] <mborzecki> Chipaca: was there some fancy json decoding too?
[10:17] <Chipaca> 1.11 might be using an even newer unicode but ctrl didn't see much change
[10:18] <Chipaca> mborzecki: ctrl is part of that, if you mean the 'clean' json
[10:18] <Chipaca> but
[10:18] <Chipaca> mborzecki: there are places where we use *json.RawMessage instead of json.RawMessage, that we could drop
[10:18] <Chipaca> and see some perf improvement
[10:18] <Chipaca> perhaps
[10:18] <mborzecki> pr by pr :P
[10:20] <Chipaca> ep
[10:30] <Chipaca> had to get https://packages.ubuntu.com/bionic/amd64/golang-golang-x-tools by hand to get godoc to play nice again
[10:30] <Chipaca> (i'm on xenial here)
[10:31]  * sparkiegeek wonders if Chipaca could have used https://launchpad.net/adapt
[10:33] <Chipaca> sparkiegeek: no idea what that is :-)
[10:34] <Chipaca> ah
[10:34] <Chipaca> sparkiegeek: no need to rebuild the package, it just works as is
[10:35] <sparkiegeek> Chipaca: 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 series
[10:35]  * Chipaca imagines using X from bionic o n xenial, and shudders
[10:36] <sparkiegeek> hah :)
[11:19] <zyga> o/
[11:19] <mvo> hey zyga
[11:19] <zyga> hey mvo :)
[11:19] <mvo> zyga: welcome back!
[11:20] <mvo> zyga: but please take it easy today
[11:20] <zyga> mvo: I'm sorry about expensify, it's super easy to click the wrong button while on the go
[11:20] <zyga> (about that "let me expense one coffee thing")
[11:20] <zyga> I'm sorting out my travel topics now
[11:20] <zyga> and recovering after the small accident yesterday :)
[11:20] <zyga> but all will be fine
[11:21] <zyga> anything urgent, any news?
[11:22] <mvo> zyga: no fires
[11:22] <mvo> zyga: your go-1.10 pr is in \o/
[11:23] <zyga> I noticed, thank you, good idea to notify other teams about that
[11:49] <mborzecki> zyga: looking into bcond_with{,out}, do we have 15.0 image for spread?
[11:51] <zyga> mborzecki: mmm, not sure
[11:51] <zyga> if you have a branch I have all the VMs ready
[11:51] <zyga> oh
[11:51] <zyga> one thing
[11:51] <zyga> so
[11:51] <zyga> tl;dr; I spent last week working on top of ubuntu in hyper-v and multipass also on top of hyperv
[11:52] <zyga> mborzecki: I think we could start standardizing on using multipass for test-builds on native kernel
[11:52] <zyga> it's very flexible and qucick
[11:52] <zyga> quick*
[11:52] <mborzecki> does it work well with non ubuntu vms?
[11:52] <zyga> yes
[11:52] <zyga> only requirement is cloud-init in the image
[11:52] <zyga> I've got a pipeline for debian builds now
[11:53] <zyga> http://download.opensuse.org/repositories/Cloud:/Images:/Leap_42.3/images/
[11:53] <zyga> I will give this a try (though not with any priority today)
[11:54] <zyga> mborzecki: it's as simple as "multipass launch $URL-to-qcow2 -c 10 -m 4G -n vm-name
[11:54] <zyga> then you have multipass commands to exec stuff inside, copy stuff around, shell interactively
[11:54] <zyga> multipass handles networking and caching
[11:54] <mborzecki> ah, haven't tried anything else than ubuntu with multipass yet
[11:55] <mborzecki> what if the image has no cloud-init?
[11:55] <zyga> nogo
[11:55] <zyga> I think it's a requirement
[11:55] <zyga> ask Saviq once he's back
[11:55] <Saviq> I'm here
[11:55] <mborzecki> Saviq: is cloud-init a hard requirement?
[11:56] <Saviq> yes, otherwise we can't access the launched instance
[11:56] <zyga> Saviq: did I say thank you for multipass, it's awesome
[11:56] <Saviq> mborzecki: otherwise you'd need to know (and pass it to us) a default username/password
[11:57] <Saviq> and we don't support that right now, nor look forward to it really
[11:57] <zyga> mborzecki: does arch have any cloud images?
[11:57] <zyga> everything else we support has them
[11:58] <Saviq> anything else, we'd need to expose a graphical and/or serial console
[11:58] <mup> PR snapd#6404 opened: snapstate: auto transition on experimental.snapd-snap=true <Created by mvo5> <https://github.com/snapcore/snapd/pull/6404>
[11:58] <Saviq> and it'd be a hands-free experience from then as far as multipass is concerned
[11:58] <mborzecki> zyga: nah :P no cloud images afaik
[11:58] <mborzecki> you can probably build one
[11:58] <mborzecki> well, we built one for spread
[11:58] <mborzecki> but it's not using cloud-init afaik
[11:59] <Saviq> probably has a well-know root password :)
[11:59] <Saviq> +n
[11:59] <mborzecki> yup, something no trivial, like 123 :P
[11:59] <Saviq> https://wiki.archlinux.org/index.php/Cloud-init
[12:00] <Saviq> https://wiki.archlinux.org/index.php/Arch_Linux_AMIs_for_Amazon_Web_Services
[12:00] <Saviq> may very well be possible to use those
[12:01] <mborzecki> mh
[12:02] <mborzecki> Saviq: while at it, is it possible to launch throw-away instances with multipass? i often work directly with cloud images and use qemu .. -snapshot
[12:04] <Saviq> mborzecki: 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 hypervisors
[12:13] <mup> PR snapd#6405 opened: run-checks: ensure we use go-1.10 if available <Simple 😃> <Created by mvo5> <https://github.com/snapcore/snapd/pull/6405>
[13:35] <zyga> mvo: mount error still happens
[13:35] <zyga> error: 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] <zyga> See "systemctl status "snap-network\\x2dconsumer-x1.mount"" and "journalctl -xe" for details.
[13:35] <zyga> )
[13:35] <zyga> this is from the PR above
[13:46] <Chipaca> zyga: is the go version relevant enough to be in 'snap version'?
[13:46] <zyga> Chipaca: dunno
[13:46] <zyga> perhaps?
[13:46] <Chipaca> zyga: or would this be a case for a --verbose flag or sth?
[13:46] <zyga> yeah
[13:46] <zyga> or even a debug
[13:46] <zyga> snap debug build-info
[13:46] <Chipaca> i mean, if it's for the tests, just add a 'go version' :-)
[13:46] <zyga> could show other helpful stuff
[13:46] <zyga> +1
[13:46] <zyga> go version can be elusive :)
[13:46] <zyga> if we have many
[13:51] <Chipaca> ooh, ooh! I remembered what I wanted to do when we moved past 1.7!
[13:52] <Chipaca> https://gist.github.com/chipaca/10ecdf364db962c9b5ee590f0f610b2a
[14:08] <zyga> sorry for cutting me video feed but I'm out of power on the phone
[14:13] <Chipaca> zyga: we lost you
[14:22] <mvo> zyga: yeah, cachio mentioned this as well, on what system did that happen for you?
[14:32] <zyga> mvo: that was in the PR you sent, I don't recall
[14:32] <zyga> mvo: I only saw it there
[14:38] <zyga> small helper I use for packaging work
[14:38] <zyga> packaigng helper https://www.irccloud.com/pastebin/oPwDE5pt/
[14:38] <zyga> mvo: ^
[14:39] <zyga> mborzecki: ^
[14:41] <mborzecki> hm we should add a multipass backend to spread :)
[14:43] <mup> PR # 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] <mup> snapd#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#6405
[14:43] <mup> PR # closed: core-build#11, core-build#22, core-build#26, core-build#37
[15:07] <cachio> mvo, should I remove the manual from debian sid after the imagge is updated?
[15:07] <mvo> cachio: yeah, lets try that
[15:07] <mvo> cachio: once its updated, lets push a PR that enables sid again
[15:07] <cachio> mvo, sure, thanks
[15:07] <mvo> cachio: and then we can check how well that goes - it will be a good prep for the packaging merge PR. thank you!
[15:54]  * cachio lunch
[16:28] <om26er> popey, ping
[16:28] <popey> om26er: hi
[16:28] <om26er> popey, could you take a look at this one https://github.com/asciinema/asciinema-snap/issues/1#issuecomment-454529232
[16:28] <popey> will do
[16:42] <mup> PR 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:43] <zyga> my debian foo has grown the SBUILD_SHELL trick
[16:53] <mvo> I uploaded new test-snapd-dbus-{consumer,producer} snaps to edge, if tests suddenly explode, this is why
[16:56] <mvo> (the PR below addresses this)
[16:56] <mup> PR snapd#6407 opened: tests: get test-snapd-dbus-{provider,consumer} from the beta channel <Created by mvo5> <https://github.com/snapcore/snapd/pull/6407>
[17:01] <mup> PR snapd#6408 opened: tests: add spread test for system dbus interface <Created by mvo5> <https://github.com/snapcore/snapd/pull/6408>
[17:41] <zyga> wooot
[17:41] <zyga> I solved the debian unit test failures :)
[18:56] <cachio> zyga, this is the error you have fixed? https://paste.ubuntu.com/p/cmQTKpDhh6/
[18:56] <zyga> no
[18:57] <zyga> this is a known issue about go fmt changing behavior across go versions
[19:00] <cachio> zyga, ahh, ok
[21:01] <mup> Bug #1812751 opened: Several remove event for unknown device <Snappy:New> <https://launchpad.net/bugs/1812751>