/srv/irclogs.ubuntu.com/2018/10/31/#snappy.txt

fayg0hey anyone here have any luck with rocket chat irc federation ?03:05
mborzeckimorning06:21
zygao/06:23
zygahey mborzecki06:23
pieqalready up zyga ?! :)06:35
zygayes06:35
zygahow are you Pierre? :)06:36
pieqzyga, good good! And you?07:11
pieqit's finally getting cold here in Taipei :)07:11
zygareally?07:20
zygait was 22C yesterday07:20
zygaweather anomaly like nothing before07:21
zygatoday it is colder but not anywhere near being really cold yet07:21
pieqwell... it's 19 °C07:27
pieqI have to put something on top of my t-shirt, that's my definition of "cold" :)07:27
zygahaha, yes, that's Spain-level cold :)07:29
mborzeckihmm would like to get https://github.com/snapcore/snapd/pull/5696 to a state where it can be landed07:47
mupPR #5696: interfaces/opengl: add additional accesses for cuda <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/5696>07:47
mborzeckidamn, cuda-samples from ijohnson is huuge07:50
mborzeckiand fedora 29 is out, the mirrors are down :/07:55
zygahmmm07:57
zygamborzecki: heh, yes07:57
zygaI'm pulling 29 to try07:57
zygamborzecki: did you see my suggestion on 569607:57
zygamaybe we should just drop that part07:57
zygaand even hardcode the mknod in snap-confine?07:58
zygagolang question:07:58
zygaI have a struct with another struct inside07:58
zygathe inner struct is defined inline, that is, it has no type name07:58
zygahow do I make literals of that type?07:58
mborzeckizyga: yeah, i want to check if you can precreate the node and if stuff continues to work then we should be fine07:59
mborzeckizyga: iirc it used to work like this in early cuda versions, you had to create the node manually and so on07:59
zygaI think it must work08:00
zygaafter all, cuda creates the nodea08:00
zygaand then it is there08:00
zygaso ...08:00
zygaif it is a fixed major/minor08:00
zygalet's just do it08:01
mborzeckizyga: i'm installing a snap from ian, if it works, then i'll update the PR and we can land it08:02
zygaok08:02
zygacool :)08:02
pstolowskiheyas08:03
mborzeckizyga: there was a topic about cuda and ubuntu core, left a note there, https://forum.snapcraft.io/t/nvidia-cuda-on-ubuntu-core/292/45 maybe you have some ideas08:03
mborzeckipstolowski: hey08:03
zygahey pawel08:04
mupPR snapd#6068 closed: ifacestate/helpers: findConnsForHotplugKey helper <Hotplug 🔌> <Simple 😃> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6068>08:05
zygamborzecki: interesting08:05
zygamborzecki: based on what you said I'd say the most stable, long term solution for loading firmware would be where snapd would run an internal helper in the main mount namespace08:06
zygaand that helper would arrange for the firmware to be loaded08:06
zygato both detach from the need for granting apps the permission08:06
zygaand to be in control of the mount namespace08:06
mborzeckizyga: mhm, something like that08:07
mborzeckizyga: snapctl request-firmware :)08:07
mborzeckiand dd that to sys node08:07
zygacould be just snapd doing it itself08:07
zygawe may not really need a new command08:08
zygaif it is really an ioctl and dd08:08
* zyga installs F29 now08:13
mborzeckizyga: yup, seems to work with manually created node08:19
zygaanother hack in Nvidia land08:19
zyga but ... whatever :)08:19
zygaperhaps it should be a part of the mount profile?08:19
zygawe can create symlinks08:20
zygawe could create device nodes08:20
mupPR snapd#6077 opened: overlord/ifacestate: use map[string]*connState when passing conns around <Created by stolowski> <https://github.com/snapcore/snapd/pull/6077>08:22
mborzeckizyga: we'd need to teach snap-update-ns to create nodes08:23
zygayeah but that's not difficult in the current architecture08:23
mborzeckimhm08:23
zygait's really a minor bump over symlinks08:23
mborzeckiwow, the tests are really slow on my gt103008:24
Chipacao/08:31
zygahey Chipaca08:31
zygamborzecki: what does the test entail?08:31
pstolowskimorning Chipaca!08:31
mborzeckizyga: heh, i think it's stuck08:32
ChipacaI'm seeing red today08:33
Chipacaand not in a berserker kind of way08:33
mborzeckiChipaca: fedora got a new release :)08:38
mborzeckiheh maybe we should start putting fedora on manual ahead of the release08:39
Chipacamborzecki: I'm seeing failures in ubuntu things that shouldn't be failing08:40
Chipacabah08:40
Chipacayesterday the new core release meant the store was unusable from tests for a few hours08:41
Chipacathen the travis osx image stopped shipping ruby -> no homebrew -> red osx test08:41
Chipacathe ubuntu thing hadn't properly resolved before I had to sleep, at ~1am08:41
Chipacaand now travis is fixed, and the store seems to be fine, but I'm still seeing some failures that make no sense unless the store is not fine08:42
Chipacatests/regression/lp-1693042 in particular08:42
zygaregression tests are useful08:44
zygawell08:44
zygainteresting day08:44
* zyga needs coffee08:45
zygare09:03
* zyga got a dentist appointment at 14:30 09:10
zygabut I can do the standup on the go09:10
chestyI finally got x2goclient snap working, thanks for the help. After I worked around the bug in libssh x2go tried to open /tmp/.X11-unix/X0 and failed, I didn't spend anytime on looking into it as I figured there's no way around it other than classic mode, and it worked in classic mode so I'm happy. cheers09:25
pstolowski:304309:36
pstolowskiups, wrong window09:36
Chipacachesty: do you have the x11 interface?09:37
chestyi do, yes09:39
chestyi don't have desktop or desktop-legacy09:40
Chipacachesty: the x11 interface allows access to /tmp/.X11/X009:40
Chipacaalso /tmp/.X11-unix/X009:41
Chipaca(first one was a typo)09:41
chestyweird, I just double checked, I definitely had and still have the x11 plug09:41
Chipaca*also* /tmp/.X11-unix/X1234567654323456765434567654345676543234567809:41
Chipaca:-)09:41
Chipacazyga: mborzecki: who's best for looking into ^ x11 socket denials?09:42
chestyi only have 12345676543234567654345676543456765432345677 xservers running, so I don't require 1234567654323456765434567654345676543234567809:42
Chipacachesty: ah, that's your issue, 12345676543234567654345676543456765432345677 is prime09:43
* Chipaca hasn't really checked09:43
chestyhehe, don't worry, I'm on the internet so I believe everything I read09:43
chestyI've now forwarded to all my facebook friends and they believe it too09:44
* Chipaca becomes president of the american mathematical society09:44
mborzeckiChipaca: /tmp/.X11-unix/X12345676543234567654345676543456765432345678 as in nvidia prime?09:44
chestyand further more I didn't see any apparmor denies09:46
chestyor warnings, it was in devmode09:46
Chipacaalas, 12345676543234567654345676543456765432345677 = 7×13×1305511×602162345977×172575591847420434324601 (5 distinct prime factors)09:46
Chipacachesty: wait09:46
chestyFAKE NEWS09:46
Chipacachesty: if it's in devmode, nothing is blocking your x1109:47
mborzeckiChipaca: unless it's x itself09:47
Chipacachesty: and obviously x11 apps work snapped, you don't need classic for that09:47
Chipacaexactly09:47
mborzeckixhost + ?09:47
Chipacano :-(09:47
Chipacano xhost +09:47
Chipacabad mborzecki09:47
mborzeckihehe :)09:48
Chipacabad!09:48
Chipacachesty: but, look at things that are not (strictly) snappy, for that denial09:48
chestyx2goclient worked, I could see its GUI, but I think it spawns nxproxy which couldn't open /tmp/.X11-unix09:48
Chipacachesty: what happens without devmode?09:49
chestyI don't understand, I might have my terms mixed up, is devmode a confinement?09:50
Chipacachesty: things are set up as in strict, but apparmor and seccomp use the unconfined profiles09:51
* Chipaca thinks he got the terms right there09:51
chestyso when you say without devmode, I have to replace it with either strict or classic?09:52
Chipacachesty: strict, is what i meant09:53
Chipacaclassic is a different kettle of fish09:53
chestyok, I'm rebuilding with strict. it's take a few minutes so I'll ping you when it's done09:54
Chipacaok09:55
mborzeckipstolowski: conflicts in https://github.com/snapcore/snapd/pull/607109:56
mupPR #6071: ifacestate/hotplug: updateDevice helper <Hotplug 🔌> <Simple 😃> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6071>09:56
popeychesty: also, you can snap install snappy-debug, then run "snappy-debug.security scanlog" in a terminal when you run your snap to see what it thinks is missing in your interfaces09:56
pstolowskimborzecki: thanks! will fix in a moment09:56
popeychesty: https://docs.snapcraft.io/debugging-building-snaps/627409:56
chestycheers popey, I actually stumbled on that last night09:56
chestypreviously I was looking at dmesg09:56
chestyhas anyone written a book yet on snapcraft? possibly hard to do atm being newish with lots of exciting changes, but I think I learn better from reading a book from page 1 to page 300. then I learn all the things I didn't know I needed to learn09:58
Chipacachesty: ssh, don't say that too loud or degville might hear you and get ideas10:00
ograbeing newish totally helps with that ... take notes on the side and once you are not "newish" anymore, write  the book from the notes ;)10:00
Chipacaor what ogra said!10:00
chestyah, son of a gun, I need to rebuild it with desktop-launch, I have no fonts atm. I'll pastebin my yaml while I'm at it10:01
chestyhttps://pastebin.com/Zc9zPw6M10:03
popeyI would make that command: desktop-launch $SNAP/bin/x2goclient (or whatever the path is to the executable)10:03
Chipacachesty: you don't need an empty . in the description, in yaml10:04
mupPR snapd#6078 opened: ifacestate/ifacemgr: don't reload hotplug-gone connections on startup <Hotplug 🔌> <Simple 😃> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6078>10:05
Chipacachesty: also your last two lines of the description are indented more than the first line, which might look odd10:05
chestyanother weird thing I found after working around libssh was I was getting a message about libXcomp.so.5: file not found the file was there so I figured it was a shared library that was missing, I ran ldd on libXcomp and I added a bunch of debs explicitly in package-stage that made sure all the libraries were present that were listed in ldd and it f10:06
chestyixed that problem10:06
popeyi wrote a simple script which turns the output of ldd into a list of debs if that's any use?10:07
popeyhttps://gist.github.com/85644e69b4e55eed782908a79d9ed208#file-lddtostage10:08
popeyjust point it at a binary and it will spit out a bunch of deb names10:08
chestyI just had a black out. I'm back. thanks for the yaml tip Chipaca, the only reason I'm doing this is because x2goclient in 18.04 segfaults, and no one has responded to my bug report. I hadn't intended to upload it to the snap store, I'm just going to use it at work10:17
popeychesty: got a link to the bug?10:18
chestyhere's the one against the package in ubuntu I filed. https://bugs.launchpad.net/ubuntu/+source/x2goclient/+bug/179984910:20
mupBug #1799849: segfault when starting new session <x2goclient (Ubuntu):New> <https://launchpad.net/bugs/1799849>10:20
chestyI also send a message, not a bug report, to x2go people, last I check I hadn't got a reply10:20
chestyI think something changed in libssh threading, not sure though10:22
chestyI got a DENIED on /home/michaelc/.ssh/known_hosts but I didn't have the ssh plug listed, rebuilding10:30
ChipacaI love the "snap isn't working form me on WSL, what am I doing wrong?" bug report10:45
Chipacaguess what we'll need to do someday =)10:45
sparkiegeekChipaca: +110:45
Chipacathe answer is "buy microsoft", if you were wondering10:45
chestylots of DENIED messages https://gist.github.com/chesty/428ed208c467f7f4e20c1244cfc3210a there were more than these, this is just the last. the gui still loads with these messages10:46
Chipacahuh, i wonder why libuuid is blocked10:48
mborzeckidoesn't make sense for those libs to be blocked10:48
chestyI added 2 entries regarding writing to known_hosts at the top of the file10:50
chestyI'll gist my current yaml, I'm doing something wrong with the plugs, they are being ignored https://gist.github.com/chesty/903cc48cca629ccab4d5447bae4091d410:52
popeyyou are building this on 16.04, right chesty ?11:03
chestyin the docker image snapcore/snapcraft11:04
chestyDISTRIB_RELEASE=16.0411:05
popeyah okay11:05
popeygroovy11:06
chestyand I'm running it on DISTRIB_RELEASE=18.04 snap    2.36+git987.40fc2bf~ubuntu16.04.111:15
chestyI just noticed snap    2.36+git987.40fc2bf~ubuntu16.04.1 says ubuntu-16.04.1 but maybe that's right11:15
* pstolowski lunch11:20
Chipacaoh traivs, y u so travis11:25
popeyHey! I won't hear a bad word said about my mate travis!11:25
* ogra senses a custard-pie battle dawning11:26
popeyChipaca: is there any particular reason there is no --quiet option for snap install?11:38
mupPR snapd#6079 opened: [RFC] `snap connections` command <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6079>11:51
mborzeckilow tech `snap connections`, RFC before I spend more time on it ^^11:51
zygamborzecki: nice!11:55
mborzeckii'll start looking into cross snap service ordering if nobody minds?11:56
mborzeckiChipaca: ^^ or is it something you wanted to look at?11:57
zygamborzecki: it looks very promising11:58
zygamborzecki: some ideas to think about:11:59
zygamborzecki: add --system to show system connections, hide those by default11:59
zygamborzecki: add "manual" note for non-auto connected stuff12:00
zygamborzecki: add "never" note for manually disconnected stuff12:00
zygamborzecki: add "content(foo)" note for content connections12:01
zygamborzecki: add interface(foo) if plug or slot name doesn't match interface name, remove the interface column12:01
mborzeckizyga: manual is wip, some changes in the backend needed, i like the idea of adding 'never' for autoconnected but disconnected manually12:01
zygamborzecki: that's all for quick ideas12:01
zygamborzecki: some color coding or ordering could be tweaked for usability once we have a better feeling12:02
zygabut I think this is super nice12:02
mborzeckii'm sure Chipaca will have ideas about that :)12:02
zygamborzecki: perhaps also add --gadget or similar for IOT devices12:02
zyga(like --system)12:02
zygaor a note about the fact that it was made by the gadget12:02
mborzeckizyga: right, connection made gadget is something we have in the backend too12:03
zygamborzecki: some more ideas12:05
zygaallow showing attributes12:05
zygaincluding labels12:06
zygaand static / dynamic attributes12:06
Saviqniemeyer: hey, did you ever get around to trying a fresh login to google for spread?12:08
kjackalHi snappy people, something is not right with snapd on debian 9. Just oppened forum topic https://forum.snapcraft.io/t/snapd-not-apt-installing-properly-on-debian-9/824012:09
zygakjackal: thank you, we will check it out12:09
kjackalzyga: Do we have an eta on the release of core?12:09
zygakjackal: core snap?12:09
kjackalyes12:09
kjackalThere is a fix on apparmor profiles that allows the deployment of kubeflow on microk8s12:10
zygakjackal: is that 2.36? I think this is coming in ~2 weeks12:11
kjackalIn about two weeks, thank you zyga12:12
zygakjackal: the problem you are experiencing is as follows12:12
zygasnapd in debian is very old but supports re-executing12:12
zygaif you install core it will automatically allow snapd to run a newer version of itself12:12
zygaif you install a snap by itself it will not work correctly because old snapd schedules the installation12:13
ograwell, if core was never installed before shouldnt the first snap trigger the core install first ?12:13
zygaI don't think we can fix this without updating snapd in Debian 912:13
ograthat doesnt seem to happen here12:13
zygaogra: it does12:13
zygaogra: but it doesn't mean new core will finish the installation12:13
zygaand this is what happens here12:13
zygaold core runs hooks12:13
zygaand things mishbehave12:13
ograah12:14
kjackalzyga: Ok, so there are no plans in fixing this anytime soon?12:14
ograwell, i dont see anything mentioning the installation of core after the "sudo snap install microk8s --classic" ... usually it prints something about core when it pulls that in12:14
zygakjackal: not that I know of12:15
zygakjackal: you must first install core12:15
zygakjackal: can you post "snap changes"12:15
zygain that thread please12:15
kjackalDone, on the thread and here: https://pastebin.ubuntu.com/p/HCzBFppdBs/12:17
kjackalzyga: should I try to address this on the snap side? Will this problem be present in other outdated distros?12:18
zygakjackal: I don't believe you can do anything about that yourself from within a snap12:18
zygaogra: ^ core is installed12:19
ograzyga, well, in change 5 ... after a manual "snap install core" it seems12:19
ograbut not triggered by the installation of the microk8s snap12:20
zygathat's weird,12:21
zygawell, still12:21
ograchange #1 should normally trigger the installation of core ... at least it does on my systems when i set up something that never had snapd12:21
zygawe cannot do anything about old snapd12:21
ograi bet it is the --classic switch that makes it not do that (but only guessing)12:21
zygapossibly12:22
* zyga quick lunch ahead of dentist12:22
kjackalDo we have other outdated distros?12:31
Chipacamborzecki: hadn't we fixed the '(ssssa{ss})', does not match expected type '(sssa{sv}a{ss})' thing?12:42
* Chipaca realises the time and rushes to make lunch12:46
zyganope12:46
zygakjackal: I think Debian 9 is special in this regard12:46
kjackalcool, good to know12:47
ChipacaI wonder what /proc/version is on WSL12:54
mborzeckiChipaca: no, we switched to using busctl to get saner error messages12:54
mborzeckiChipaca: is it happening on 2.36 branch?12:54
Chipacayes12:56
mborzeckii'll prep a pr with cherry pick of that change12:57
mborzecki(as it went away magically too)12:58
pstolowskimborzecki: standup?13:00
Chipacaand my speedup version thing broke the build somehow. possible the benchmark thing13:01
Chipacacachio: ?13:01
mupPR snapd#6080 opened: tests/main/interfces-accounts-service: switch to busctl, more debugging (2.36) <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6080>13:05
mborzeckiChipaca: ^^13:05
Chipacamborzecki: kthx13:16
ijohnsonmborzecki: I ticked the "allow edits from maintainers" box on my CUDA PR so you should be able to push to it. If you have time to take over that CUDA PR, go for it, I unfortunately haven't had time past few weeks. But I'm happy to run any tests in the background if you like, I have a few different NVIDIA cards I can test with13:20
mborzeckiijohnson: thanks, let me try pushing the branch now13:20
mborzeckiijohnson: aand pushed, yay!13:21
mborzeckiijohnson: was cuda test suite super slow for you when you ran it?13:22
ijohnsonmborzecki: eh it takes maybe 10 minutes on my GTX 1050 but only took like 3 minutes on my titan X :P13:23
ijohnsonbut it shouldn't hang anywhere, do you know which test it was hanging on for you?13:23
mborzeckiijohnson: well, it appears stuck on my gt1030 :)13:24
mborzeckiijohnson: it runs a couple of tests and appears stuck, maybe it's still crunching some numbers though it must be doing that really slow13:25
ijohnsonhmm that's unfortunate, but yes some of the tests are fairly intensive, but a 1030 should still be able to finish in a reasonablish amount of time. let me look at my test script and see how to make it more verbose on which test it's running so you can see which test it gets stuck13:26
ijohnsonoh actually it should say which sample it's running by default: https://github.com/anonymouse64/cuda-samples-snap/blob/master/bin/test-cuda.sh#L11013:27
mborzeckiijohnson: it does, and i'm apparently not smart enough to figure out that you should close the window that popped up :/13:29
ijohnsonah yeah that's a thing you do have to close the windows it opens, I wasn't able to figure out how to automatically close the windows13:30
mborzeckiijohnson: anyways, loading nvidia-uvm and creating /dev/nvidia-uvm before i ran the samples did the trick13:31
ijohnsonmborzecki: good to know, thanks for testing13:32
mborzeckiChipaca: so debian build also calls go generate then?13:36
Chipacamborzecki: yes13:36
Chipacamborzecki: export DH_GOLANG_GO_GENERATE=113:37
Chipacamborzecki: in debian/rules13:37
mborzeckiChipaca: interesting13:37
mupPR snapd#6081 opened: overlord/ifacestate: mark connections disconnected by hotplug with hotplug-gone <Hotplug 🔌> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6081>13:37
mborzeckiheh, go geneate ./... and i learned about snap blame13:38
Chipacaso it then runs this monster of a command13:38
Chipacamborzecki: look for 'go generate' in https://api.travis-ci.org/v3/job/448506794/log.txt13:38
mborzeckiChipaca: wow, looks like `go generate $(go list ./...)`13:39
Chipacamborzecki: you didn't know about snap blame? :)13:39
Chipacamborzecki: with extra pie13:39
mborzeckiChipaca: no i didn't, but now snap blame told me it's my fault :)13:40
Chipacamborzecki: it's obviously working correctly13:40
mborzeckiChipaca: i know13:40
Chipacagrah, another 2.36 failure: https://api.travis-ci.org/v3/job/448204858/log.txt13:40
mborzeckiChipaca: hm some people appear to me more at fault than others, though my N=20something13:41
Chipacamborzecki: shocking13:41
pstolowskizyga: another report re https://forum.snapcraft.io/t/cant-connect-interfaces-so-cant-run-snaps/8123/15 , can you comment on the fix?13:41
Chipacamborzecki: more pain from the generator :-(13:56
mborzeckiChipaca: uhh, logs?13:56
Chipacadh_install: usr/bin/chrorder exists in debian/tmp but is not installed to anywhere13:56
mborzeckioh, wow so it also does go install ./... ?13:56
Chipacayar13:56
Chipacamborzecki: do any of the other distros?13:57
mborzeckinot that i've seen13:57
mborzeckiChipaca: fedora, arch and suse call go build -o ...13:57
mborzeckiChipaca: it'd be fun to package a project with multiple small development tools13:59
ChipacaI don't know how to tell dh_install to ignore this file14:00
Chipacahmm14:00
mborzeckican yuu remove it afterwards?14:00
Chipacaah got it14:00
Chipacawe're already removing several14:00
Chipacamborzecki: look at override_dh_install in debian/rules fwiw14:00
mborzeckiChipaca: mhm, we do some cleanup there already14:02
zygaRe14:03
zygapstolowski: looking14:04
zygaDone14:06
pstolowski thanks14:08
mborzeckioff to pick up the kids14:08
* cachio afk14:27
* Chipaca afk14:31
Saviqcachio: FYI, Fedora 29 is out14:32
mupPR snapd#6082 opened: overlord/ifacestate: set hotplug-key of the connection when connecting hotplug slots <Hotplug 🔌> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6082>15:49
mupPR snapd#6080 closed: tests/main/interfces-accounts-service: switch to busctl, more debugging (2.36) <Created by bboozzoo> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/6080>15:56
mupPR snapd#6073 closed: strutil: let MatchCounter work with a nil regexp <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/6073>16:02
* Chipaca <- dumb16:03
Chipacaforgot to apply the delta to 1404 rules16:03
Chipacazyga: I changed #6060 to use the new nil regexp support in MatchCounter, if you could give it a second look sometime16:15
mupPR #6060: overlord/snapshotstate/backend: be more verbose when SNAPPY_TESTING=1 <Created by chipaca> <https://github.com/snapcore/snapd/pull/6060>16:15
sil2100rbasak: hey! You done with your SRU work for today?16:18
sil2100Eeek, wrong channel16:18
sil2100(please ignore)16:18
cachioSaviq, hi, I alerady created the fedora 29 image16:22
cachioI am fixing some issues araoung this now16:22
cachiobut you can try it16:22
zyga Chipaca +116:30
zygaapproved :)16:30
Saviqcachio: thanks, I'll try and add it to our CI - still can't try locally, couldn't get the right login credentials (or I've not enough privileges - Gustavo was looking into that)16:37
zygahttps://github.com/snapcore/snapd/pull/6074/files#r229779158 ?16:50
zygaChipaca: ^16:50
mupPR #6074: strutil: make VersionCompare faster <Created by chipaca> <https://github.com/snapcore/snapd/pull/6074>16:50
Chipacazyga: we usually do, yes16:50
zygausually?16:50
zygado we have any other files that we commit and generate?16:50
Chipacazyga: in all other cases where the file is generated, it's committed, afaik16:51
zygahmmm16:51
zygaI don't suppose this file will change16:51
zygabut I find it odd16:51
zygait's generated for a reason, why keep it in VCS?16:51
Chipacaperhaps excepting cmd/version_generated.go16:51
zygadrat, I have to go now16:52
zygasorry, I'll pick up reviews later16:52
pstolowskizyga: what do you think about #6077 ?16:52
mupPR #6077: overlord/ifacestate: use map[string]*connState when passing conns around <Created by stolowski> <https://github.com/snapcore/snapd/pull/6077>16:52
Chipacazyga: one reason for committing generated files is that without it you can't 'go get' the repo16:54
Chipacaunless the code has been prepared for it (in this case it hasn't)16:55
Chipacazyga: what cmd/version_generated.go is populate a variable that's set elsewhere, so that one's ok16:55
Chipacazyga: if you don't have the generated file, you get version "unknown"16:55
zygapstolowski: approved, thank you16:57
pstolowskithanks!16:57
zygaInteresting Chipaca, I didn’t know that16:57
zygaIt makes sense to commit based on this information16:57
Chipacagrah16:59
Chipacadebian now fails on 'systemctl restart snapd.socket' for no reason16:59
Saviqcachio: hmm that's not something I've seen before: https://travis-ci.org/MirServer/mir/jobs/44894806817:01
=== phoenix_firebrd is now known as murthy
cachioSaviq, we are not testing on fedora-rawhide-6417:07
Saviqcachio: we either, is there an image for rawhide now?17:07
cachiojust fedora 28 and in the future 2917:07
cachioI have to create a new one17:07
Saviqwe've been upgrading from latest to rawhide so far17:07
Saviqand that upgrade - 29 to rawhide - is what failed there17:08
cachioSaviq, ahh17:08
cachiook, I'll need to do the same to create the rawhide image17:08
cachiolet me finish the changes to run snapd on f2917:08
cachioand then I'll update fedora warhide17:09
cachiojust I need to fix one error on f2917:09
Saviqack, tx17:09
Saviqalan_g: FYI ↑17:10
alan_gNice.17:11
zygacachio: hey, do you know about mkosi?17:15
zygaIt is pretty cool for making OS images17:16
zygaAnd I know the upstream :-)17:16
zygaIt is a pet of systemd17:16
cachiozyga, no, but sounds nice17:16
cachioI'll take a look17:16
zygaIt is a single script17:17
zygaLook at github/systemd/mkosi17:17
=== murthy is now known as phoenix_firebrd_
Chipacanice change of behaviour in systemd in debian-917:18
Chipacahttps://github.com/snapcore/snapd/pull/608317:19
mupPR #6083: tests/lib: adjust to changed systemctl behaviour on debian-9 <Created by chipaca> <https://github.com/snapcore/snapd/pull/6083>17:19
Chipacathat'll need backporting to 2.36 as well as it's currently hurting17:19
mupPR snapd#6083 opened: tests/lib: adjust to changed systemctl behaviour on debian-9 <Simple 😃> <Squash-merge> <Created by chipaca> <https://github.com/snapcore/snapd/pull/6083>17:19
Chipacaand now i must go17:19
* Chipaca afk for a good while now17:20
Chipacatelegram works17:20
cachiozyga, some tests failing on fedora 29 with the same error https://paste.ubuntu.com/p/W9fyHPZwDc/17:21
cachiozyga, any idea whay it could be happening?17:21
=== phoenix_firebrd_ is now known as murthy
zygaI’m afk now17:38
zygaI’ll check back home17:38
cachiozyga, sure, tx17:40
cachiono errors on dmesg17:40
cachiono errors on journalctl17:40
kyrofaHey niemeyer, are you around?18:39
niemeyerkyrofa: Sort of.. I'm on security border line right now19:01
niemeyerkyrofa: I've seen your mail about the arm server.  Sorry I didn't manage to look into it yet19:02
kyrofaniemeyer, ah, nevermind then. Hoping to chat about extensions after the sprint, but it can wait until you're actually working!19:03
kyrofaNo worries on the server, when you have time19:04
niemeyerkyrofa: I should be in the hotel in an hour or two.. happy to talk19:04
kyrofaniemeyer, ping me if you're up for it, but no pressure19:05
niemeyerThanks, talk soon19:06
cachioPharaoh_Atem, hey, I am getting this error -> Error: Failed to synchronize cache for repo 'fedora-modular'19:59
cachioI am running fedora-upgrade from 29 to rawhide19:59
cachiodid you know about that?19:59
cachioSon_Goku, hey20:05
cachio hey, I am getting this error -> Error: Failed to synchronize cache for repo 'fedora-modular'20:05
cachio I am running fedora-upgrade from 29 to rawhide20:05
cachiodid you know about that?20:05
mupPR snapd#6066 closed: tests: fixes and new backend for tests on nested suite <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6066>20:27
zygacachio: no idea, perhaps strace it to see what is being executed21:02
cachiozyga, yes, I'll try that21:04
Pharaoh_Atemcachio: yeah21:15
Pharaoh_AtemI think the modular repo may not exist yet for rawhide21:15
cachioPharaoh_Atem, ahh, ok, I'll fix it, thanks21:31
cachioPharaoh_Atem, do you now when is it gonna be created?21:32
Pharaoh_Atemcachio: it should technically exist now, at least it exists on my rawhide mirror (I just checked)21:36
Pharaoh_Atemso I dunno21:36
cachioPharaoh_Atem, ok, thanks21:40
cachioI'll try tomorrow again21:40
popeyogra: i have a Wyse 120 physical terminal connected via USB serial to my Ubuntu Core laptop (Don't ask)...22:10
popeyhttp://0pointer.de/blog/projects/serial-console.html - any reason why that wouldn't work?22:10
popeyI have agetty running against ttyUSB3, but wondered if there was any module or something that would prevent this working22:10
popeyI see the module loaded in dmesg, so the serial usb device "works".22:11
mupPR snapd#6083 closed: tests/lib: adjust to changed systemctl behaviour on debian-9 <Simple 😃> <Squash-merge> <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/6083>22:30
mupPR snapd#6084 opened: tests/lib: adjust to changed systemctl behaviour on debian-9 <Created by chipaca> <https://github.com/snapcore/snapd/pull/6084>23:26
mupPR snapd#6076 closed: tests: linode execution is not needed anymore <Simple 😃> <Created by sergiocazzolato> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/6076>23:33
mupPR snapd#6075 closed: tests: removing fedora 26 system from spread.yaml <Simple 😃> <Created by sergiocazzolato> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/6075>23:51

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