/srv/irclogs.ubuntu.com/2017/11/17/#snappy.txt

kyrofasergiusens, elopio snapcraft#1638 looks good (I reviewed it previously as well), but I notice no adt has run. It's kind of a big re-org, think it's a good idea?00:59
mupPR snapcraft#1638:  tests: reorganize unit and integration suites to make them easier to split for travis <Created by elopio> <https://github.com/snapcore/snapcraft/pull/1638>00:59
elopiokyrofa: let me update the other one so they run on the nightly01:01
kyrofaelopio, so don't land this yet?01:02
elopiokyrofa: ah, I thought it was landed already. Well, I guess we are in no hurry because it's already EOD. Let me trigger a few, and we land tomorrow morning.01:03
kyrofaYeah that seems reasonable. And gives me an opportunity to test the new subscriptions01:04
elopiowhere's the bot?01:04
elopiosnappy-m-o autopkgtest 1638 xenial:armhf xenial:amd6401:11
snappy-m-oelopio: I've just triggered your test.01:11
elopiosnappy-m-o github subscribe 163801:13
snappy-m-oelopio: I'll send you updates as tests complete in pull request snapcraft#1638 ( tests: reorganize unit and integration suites to make them easier to split for travis).01:13
mupPR snapcraft#1638:  tests: reorganize unit and integration suites to make them easier to split for travis <Created by elopio> <https://github.com/snapcore/snapcraft/pull/1638>01:13
elopioI'm now sure if the subscribe thing is working.01:13
kyrofasnappy-m-o, github subscribe 163801:14
snappy-m-okyrofa: I'll send you updates as tests complete in pull request snapcraft#1638 ( tests: reorganize unit and integration suites to make them easier to split for travis).01:14
kyrofaelopio, yeah me neither, but I wasn't before, either01:14
kyrofaNow we know that is MUST ping us either way01:14
sergiusenssnappy-m-o, github subscribe 163801:41
snappy-m-osergiusens: I'll send you updates as tests complete in pull request snapcraft#1638 ( tests: reorganize unit and integration suites to make them easier to split for travis).01:41
mupPR snapcraft#1638:  tests: reorganize unit and integration suites to make them easier to split for travis <Created by elopio> <https://github.com/snapcore/snapcraft/pull/1638>01:41
sergiusenselopio kyrofa by default I think the bot should just go ahead and ping (all of us or at least the author)01:41
kyrofasergiusens, it's not smart enough to tie github users to IRC nicks01:42
kyrofaI guess it would work for us though, just use the same01:42
kyrofasergiusens, we recently updated it to ping us on failure OR success though01:43
sergiusenskyrofa we can make a manual "team" map.01:43
kyrofaYeah that would be easy as well01:43
mborzeckigood morning06:02
mborzeckimvo: morning06:27
mvohey mborzecki06:27
mvomborzecki: how are you?06:28
mborzeckigood, thank you06:28
mborzeckisorry to bugging you early morning but would you mind taking a look at https://github.com/snapcore/snapd/pull/4235 ?06:28
mupPR #4235: cmd: pretend we're running on Ubuntu in TestExecInCoreSnapUnsetsDidRe… <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/4235>06:28
mvomborzecki: sure06:28
mborzeckithis is the last thing that's currently blocking unit tests on arch06:29
mvomborzecki: aha, ok. sure, I have a look. I'm still slightly puzzled why its needed but I will poke at it a bit (and may come back with questions)06:29
mborzeckiok06:29
mvopedronis: thanks a lot for 4222!06:30
mvomborzecki: I just checked the test, the issue is that the test is using /snap unconditionally when it needs to use dirs.SnapMountDir (silly test). so http://paste.ubuntu.com/25979289/ fixes the test for me without the need to do the mocking (its also more correct). could you please double check and if it also works for you just patch -R the current diff and force push the smaller diff?06:49
mvomborzecki: or not force push just keep it as a regular commit06:50
mvomborzecki: and when can squash-merge it06:50
mborzeckithanks, i'll check it locally and let you know06:50
mvothanks06:51
mborzeckimvo: the snippet you posted works locally ;) i'll push changes shortly07:07
mvomborzecki: yay, thank you07:07
mborzeckimvo: revert & patch are pushed now, thanks for the hint on fixing this, no need to go through the silly mock/restore trampoline :)07:15
mvomborzecki: thanks! indeed but the real silliness was in the test to hardcode the /snap prefix07:17
kalikianao/07:40
=== mardy_ is now known as mardy
pedronismvo: hi, was this fixed:  https://forum.snapcraft.io/t/spread-cron-is-not-running-snapd-vendor-sync/2739  ?08:46
mvopedronis: yes, this is working again.08:48
mvopedronis: cachio fixed it some days ago and it seems to be ok, I just checked the build history08:49
mvomborzecki: btw, do you know the status of the lxd issue? I guess zyga worked late(?)08:50
mborzeckimvo: https://forum.snapcraft.io/t/snapcraft-adt-failures-with-the-new-core-release/2850/62 we're waiting for a patch from zyga08:51
mborzeckiother than that g+s08:51
mvomborzecki: thank you08:51
mborzeckiwhat's the plan for .4 release?08:52
mvomborzecki: we do it as soon as we have a patch, ideally today08:54
pedronismvo: are you going to do a separate PR for checking pending refreshes when refreshes are managed?08:57
pedronismvo: using #4232 basically08:58
mupPR #4232: store: add support for flags in ListRefresh() <Created by mvo5> <Merged by zyga> <https://github.com/snapcore/snapd/pull/4232>08:58
mvopedronis: either way is fine with me, I can separate it out or put it into the main one, whatever is easier to review for you08:58
pedronismvo: it's going to be a separate cycle, right?  separate PR would seem better in that case08:59
mvopedronis: I hope we can get it in for 2.3009:00
pedronis?09:00
mvopedronis: but separate is fine with me, probably easier to review09:00
pedronisI don't think we can merge the rest without that, afaiu, so it needs to be in 2.3009:01
mvopedronis: yes, then we are in agreement. I misunderstood when you said separate cycle09:01
pedronisI mean separate cycle09:01
pedronisin the sense of the scheduling inside snapd09:01
mvopedronis: yes, I think so09:01
mvopedronis: that seems to make most sense09:02
pedronisok09:02
pedronisI think we are in agreement09:02
mvopedronis: cool, its my next task, I hope to get something ready by EOD09:03
mvopedronis: firefox 57 happend and all my extension (not all, but almost) are dead so I had a bit of unplanned fixtime on this this morning :(09:03
mvopedronis: anyway, I get back to schedule (and review feedback fixing and code reivews)09:03
pedronisit's ok, anyway it's still a bit until 2.30 beta? or I'm reading the roadmap wrong?09:04
pedroniss/I'm/am I/09:04
mvopedronis: we still have a bit of time yes09:04
mvopedronis: its adjusted for .4 :/09:05
pedronismvo: is just me of mborzecki  stuff seems a bit unlikey for 2.3009:05
mvopedronis: also autopkgtests on i386 are failing access the board for unknown reasons09:05
mborzeckipedronis: refresh timers?09:05
mvopedronis: its unlikely :) but reflecting what is being worked on, I think its ok if it moves to 2.3109:05
pedronistime servers09:05
pedronisservices09:05
pedronismvo: ok, was just confused09:06
mvopedronis: no worries :)09:06
mborzeckitime servers... you made my hear skip a beat :)09:06
pedronismborzecki: it's was a comment on you or anything, just a bit confused because it will take a while to code and get through review, especially given we would like to clean up the current stuff too09:06
mborzeckino worries :)09:07
pedronismvo: I want to produce a couple of small PRs about things I mentioned yesterday at standup then I will look at your main PR again09:10
mvopedronis: sounds good09:12
mupPR snapd#4239 opened: tests: more debug info for classic-ubuntu-core-transition <Created by mvo5> <https://github.com/snapcore/snapd/pull/4239>09:13
mupPR snapd#4240 opened: spread.yaml: increase workers for opensuse to 3 <Created by mvo5> <https://github.com/snapcore/snapd/pull/4240>09:51
mupPR snapcraft#1638 closed:  tests: reorganize unit and integration suites to make them easier to split for travis <Created by elopio> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1638>09:56
=== __chip__ is now known as Chipaca
zyga-ubuntuo/10:14
zyga-ubuntusorry for being so late, I need to balance my sleep/wake cycle :/10:14
mborzeckizyga-ubuntu: hey10:14
jameshzyga-ubuntu: hi.  I pushed through some changes to my user-mounts PR (rebasing on master and converting it to use snap-update-ns).  It probably still isn't quite right, but it seems to handle the simple cases I tried.10:17
mvohey zyga-ubuntu ! good morning10:17
zyga-ubuntujamesh: hey! thank you for doing that, I saw the notification and I'll have a look today10:17
zyga-ubuntujamesh: what are your next plans?10:18
zyga-ubuntumvo: hey, sorry :|10:18
zyga-ubuntumvo: but but, were on one page of PRs :)10:18
jameshzyga-ubuntu: I'd like to try and get portals working10:18
zyga-ubuntujamesh: what bits are missing?10:18
jameshso updating those branches from months back, probably folding it into the desktop interface10:19
mvozyga-ubuntu: :)10:19
mvozyga-ubuntu: we just need to merge one, e.g. the opensuse one and we are back on a single page10:20
jameshzyga-ubuntu: the user-mounts thing is the main missing infrastructure.  We might run into some issues with how we change $XDG_RUNTIME_DIR though10:20
jameshparts of document portal expect certain paths to be the same inside and outside the mount namespace.10:21
zyga-ubuntujamesh: please update the forum thread with your plans; I'm slowly getting layout code in place and I can help10:21
zyga-ubuntutoday I'm a bit dizzy for complex things but I should have fresh head on Monday10:22
zyga-ubuntumvo: not sure if you were following the LXD discussion last night, I'll do what we agreed with jdstrand; make snap-confine setgid, drop the gid part early on and only restore it for that single freeze operation10:24
zyga-ubuntumvo: I should have that ready in ~1.5 hours10:24
zyga-ubuntumvo: hopefully by EOD we can push to beta10:24
mvozyga-ubuntu: \o/10:25
mvozyga-ubuntu: I'm trying to understand a pkgtest failure on i386 in parallel, worst case is that we need to disable one test in autopkgtest (ubuntu-core-transition is failing *only* on i386)10:25
mvozyga-ubuntu: so +1 for a .410:26
zyga-ubuntuonly on i386, curious10:26
zyga-ubuntumvo: thank you, good luck10:26
pstolowskimvo, hey, what's up with https://github.com/snapcore/snapd/pull/4063 ?10:29
mupPR #4063: tests: add new kernel refresh/revert test for spread-cron <Created by mvo5> <https://github.com/snapcore/snapd/pull/4063>10:29
mborzeckizyga-ubuntu: took a deeper look into https://forum.snapcraft.io/t/installing-vscode-snap-on-arch-linux/2871/5 imo there's something wrong with what's shipped in the snap, my best guess those libs are pieces of fedora userspace10:31
zyga-ubuntumborzecki: vscode is a classic snap10:32
zyga-ubuntumborzecki: no namespace magic, problems10:32
pedroniszyga-ubuntu: hi, what's the status of #410910:32
mupPR #4109: cmd/libsnap: fix parsing of empty mountinfo fields <Created by zyga> <https://github.com/snapcore/snapd/pull/4109>10:32
zyga-ubuntupedronis: it's not critical, just something I found while hacking10:32
zyga-ubuntupedronis: there's a cleanup that I could move out of the function10:33
zyga-ubuntupedronis: and a fix itself10:33
zyga-ubuntupedronis: other than that not sure10:33
mvopstolowski: meh, no time yet for this10:33
pstolowskimvo, ah, ok, no pressure, just checking10:33
cachiozyga-ubuntu, hey11:24
cachiozyga-ubuntu, I have defined for a test this snap https://github.com/sergiocazzolato/snapd/blob/tests-interface-network-status/tests/lib/snaps/test-snapd-network-status-provider/snapcraft.yaml11:26
cachiozyga-ubuntu, but I am getting some denials when I do "sudo systemd-run --unit dbus-provider-2 test-snapd-network-status-provider.provider"11:27
=== JoshStrobl|zzz is now known as JoshStrobl
zyga-ubuntucachio: hmm11:28
zyga-ubuntucachio: what kind of denials11:28
cachiozyga-ubuntu, https://paste.ubuntu.com/25980351/11:30
zyga-ubuntuaha11:30
zyga-ubuntuis this blocking the test?11:30
zyga-ubuntuit seems that snap wants to look at the mount table11:30
zyga-ubuntuthat should be handled by mount-observe11:30
zyga-ubuntunot sure why it needs it, just saying11:30
cachiozyga-ubuntu, I already tried that11:31
zyga-ubuntuowner @{PROC}/@{pid}/mounts r,11:31
zyga-ubuntumount-observe grants this11:31
zyga-ubuntuwhat did you try?11:31
cachioyes, let me try again11:32
pstolowskiChipaca, hello! is this clear what we want wrt to environ in commands here https://forum.snapcraft.io/t/papercuts-mouth-sized-bugs/228 ?11:32
cachiozyga-ubuntu, I plug mout-observe and I see this https://paste.ubuntu.com/25980371/11:35
zyga-ubuntucachio: is this test looking at its own mount table?11:35
zyga-ubuntuthe rule says 'owner'11:36
cachiono11:36
cachioit is trying just to own the com.ubuntu.connectivity1.NetworkingStatus dbus interface11:37
zyga-ubuntucachio: I'm sorry but I don't know why it would also try to access the mount table11:37
Chipacapstolowski: what do you mean?11:44
cachiozyga-ubuntu, I have to go to the doctor11:45
zyga-ubuntucachio: ok11:45
cachiozyga-ubuntu, I'll be 5-10 minutes late for the standup11:45
Chipacapstolowski: the one about them being too restrictive?11:46
pstolowskiChipaca, no; do we want to interpolate "command: FOO=$SNAP/bar actual_command" or "command: env PATH=$SNAP/usr/local/bin:$PATH desktop-launch" ?11:47
Chipacapstolowski: me, i'd explore something like: first, support having env vars as arguments to commands (even the first argument). Second, support arbitrary arguments but with no quoting nor escaping (so just disallow '"\)11:47
Chipacapstolowski: and third, make commands be either a string, or a list; if a list, pass to exec as is11:48
Chipacathe reason for not getting into quoting and such is because it's a nightmare11:48
Chipacaand the list approach makes it not needed11:49
Chipaca(the list would take quotes and backslashes and pass them through)11:49
Chipacaum11:49
Chipacapstolowski: answering your question there, no that's not the idea at all11:49
Chipacapstolowski: commands already have an environment11:49
Chipacathat is, apps have an environment11:50
Chipacapstolowski: this is about support “command: foo $BAR $BAZ”11:50
mupPR snapcraft#1742 opened: lxd: always remove tmp_dir after execution <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1742>11:51
Chipacait's possible that work is already done though; not sure11:51
pstolowskiChipaca, ok, that should work if I read the code correctly11:51
Chipacathat post is old11:51
pstolowskiChipaca, yeah, and it originated from https://forum.snapcraft.io/t/declaratively-defining-environment-variables/175/611:51
pstolowskiChipaca, I'm trying to understand if what's done or not, there were a couple of aspects discussed there, I'm unclear what do we want at the end11:52
Chipacapstolowski: I think there is still work to be done here, but i've not looked in a while as i say11:54
* kalikiana time for a break11:55
Chipacapopey: is the mailspring snap doing something wrong, or do none of the desktop integration bits work for anybody?11:56
Chipacai mean system tray and notifications, in particular11:56
Chipaca(my indicators don't even work for regular apps for whatever reason but that's another discussion))11:57
pstolowskiChipaca, I see, thanks11:58
sergiusensChipaca indicators fail completely for me, OSD works fine for mailspring12:16
popeyChipaca: what version of Ubuntu?12:24
Chipacapopey: 16.0412:25
popeyzyga-ubuntu / mvo your favorite topic! https://forum.snapcraft.io/t/vdpau-support-in-snaps-on-nvidia/287612:25
popey:)12:25
Chipacapopey: i thin right now lxd is more favourite12:25
Chipacaalso my keyboard is dying :-(12:25
jdstrandikey|zzz: hey, regarding your forum post (https://forum.snapcraft.io/t/manual-review-of-base-snaps/2839/4), both zyga and I responded to you (so not talking into the void). we need an architect to participate in the discussion, hence the open question to niemeyer in that topic12:29
* ogra_ hands Chipaca https://www.massdrop.com/buy/xmit-hall-effect-mechanical-keyboard12:29
* popey bans ogra_ for posting massdrop links, which will cause the collapse of the economy as we all buy them12:31
ogra_hahaha12:31
Chipacaalso, that import duty12:33
Chipacaogra_: i've got a lot of nice keyboards! the one that's dying is the one for my tablet (that i'm currently on)12:33
ogra_ah, thats bad indeed12:34
ogra_well, the above is indistructable ... (at least the switches)12:34
jdstrandpedronis: hey, I think that the base declaration is correct and the spread test failure, while real, is the right behavior12:34
pedronisjdstrand: my point is mostly that people that were testing snaps doing this will need to change their tests12:35
jdstrandpedronis: "Then the plug is connected by default"12:35
jdstrandthe test should be adjusted12:35
pedronisbecause they don't get autoconnection for their --dangerous install anymore12:36
jdstrandI understand that12:36
pedronisjust making clear this will be  a "regression" for them12:36
jdstrandthat is the correct behavior. there are 5 snaps that use this12:36
pedronisI know12:37
jdstrandso I can communicate that to them and in documentation12:37
pedronisok12:37
pedronisI fixed the integration test already12:37
jdstrandbut it does mean the spread test needs to be adjusted12:37
pedronisI did that12:37
pedronisI think we are getting other spurious failures now12:37
jdstrandoh I missed that12:37
* jdstrand looks12:38
pedronisit's just taking too long12:38
pedronisthere's a PR by mvo that could help with that12:38
pedronismvo: we still got prepare failures in you add opensuse workers PR :/12:48
pedronis*your12:48
Son_Gokupedronis, have we sync'd the Fedora 27 VM base image to the current Fedora 27 GA?12:49
Son_Gokuand resync'd Rawhide to a recent snapshot of it?12:49
pedronisno clue12:50
pedronisI'm not on top of that12:50
mupPR snapd#4241 opened: store: bit less aggressive retry strategy <Created by pedronis> <https://github.com/snapcore/snapd/pull/4241>12:50
Son_Gokuwho is?12:50
pedronismvo, zyga-ubuntu, cachio12:50
pedronismaybe12:51
pedronisChipaca: could you look at #4241 (it's tiny)12:51
mupPR #4241: store: bit less aggressive retry strategy <Created by pedronis> <https://github.com/snapcore/snapd/pull/4241>12:51
zyga-ubuntuSon_Goku: cachio is working on updating the debian images, we alredy said that other images must follow12:51
Chipacasure12:51
Chipacapedronis: +112:52
pedronisthx12:52
mupPR snapd#4242 opened: panic early if snapd is pointed to staging but staging keys are not compiled-in <Created by pedronis> <https://github.com/snapcore/snapd/pull/4242>13:20
zyga-ubuntumvo: I'll push the branch in a moment, just want to see it work one more time13:26
pedronismvo: do we need to disable the opensuse tests until we understand more?13:26
zyga-ubuntupedronis: +1, also for fedora if that is broken and would impede release13:26
cachioSon_Goku, zyga-ubuntu the new fedora images will be ready early next week13:26
Son_Gokucool13:27
cachiomvo, zyga-ubuntu the debian images are not updated yet13:37
sborovkovHi13:45
sborovkovNo matching distribution found for python-distutils-extra (from snapcraft)13:45
sborovkovI am getting this when doing pip install snap craft on ubuntu13:45
sborovkovHow do I work around this?13:45
=== petevg is now known as petevg_afk
zyga-ubuntumvo: can you look at https://github.com/snapcore/snapd/pull/4230/commits/7722c0404b97fa0ac119acb495caa62c3f5ab321 please13:57
mupPR #4230: tests: add test to run snap inside lxd as a user <Created by mvo5> <https://github.com/snapcore/snapd/pull/4230>13:57
sergiusenssborovkov `pip install -r https://raw.githubusercontent.com/snapcore/snapcraft/master/requirements.txt`  python-apt and -distutils-extra are not on pypi14:01
sborovkovsergiusens, thanks14:02
mvozyga-ubuntu: sure, looking14:02
cachiojdstrand, hey14:03
mvozyga-ubuntu: looks good, thank you14:03
cachiojdstrand, I am creating s snap with this https://paste.ubuntu.com/25980439/14:05
cachiojdstrand, this is the python code: https://paste.ubuntu.com/25981110/14:05
cachiojdstrand, and when I do "sudo systemd-run --unit dbus-provider-2 test-snapd-network-status-provider.provider"14:05
cachioI see this denial14:05
cachiojdstrand, https://paste.ubuntu.com/25980371/14:05
cachiojdstrand, any idea what could be the reason?14:06
zyga-ubuntujdstrand: can you look as well please, that's the thing you suggested last night14:08
jdstrandcachio: does the snap fail to work with that? most often that is a noisy denial and not a problem. The snap could plugs the mount-observe interface to get rid of it14:08
jdstrandcachio: python is one of those environments that does that (noisy denial but works otherwise)14:09
cachiojdstrand, ok14:10
cachioI'll check that14:10
jdstrandzyga-ubuntu: sure14:11
cachiojdstrand, https://paste.ubuntu.com/25981162/14:14
cachiojdstrand, it is not from a snap14:14
mborzeckii'm calling it a day, have a nice weekend everyone14:14
mvocachio: thanks for checking 4240 - is there something we can do to help the opensuse tests to run faster again?14:38
mvocachio: i.e. caching the downloads and storing that in our test-image or something?14:39
cachiomvo, I think the best we can do is to update regularly the images14:39
cachioso all the provisionning part is reduced almost to 014:39
cachiomvo, I'll work on that next week14:40
cachioI'll need a new key for this14:40
mvocachio: ok, thanks14:42
mvocachio: should we disable opensuse until then?14:42
sergiusenselopio kyrofa updating the tests from other branches takes a bit14:43
cachiomvo, I'll take a look to opensuse14:44
mvocachio: thank you14:44
cachiomvo, not sure why it is failing noe14:44
cachionow14:44
cachioit is a timeout in the prepare trying to install dependencies14:44
cachiomvo, it shouldn't happen14:45
mupPR snapd#4243 opened: tests: disable classic-ubuntu-core-transition on i386 temporarly <Created by mvo5> <https://github.com/snapcore/snapd/pull/4243>14:52
zyga-ubuntumvo: tests/main is green, let's release this14:53
zyga-ubuntucan everyone available please review 423014:53
mvozyga-ubuntu: which one is green?14:53
zyga-ubuntumvo: (tests/main ran locally)14:53
mvozyga-ubuntu: for what PR (sorry, I lack context somehow)14:54
zyga-ubuntumvo: 423014:54
mvozyga-ubuntu: don't we need a +1 from jamie first?14:54
zyga-ubuntumvo: right, that's why I asked for reviews :)14:54
mvozyga-ubuntu: aha, ok. yeah, +1 for releasing asap14:55
elopiogood morning.14:55
mvozyga-ubuntu: but it looks like your PR will fail on opensuse :/ only 3 min left and14:55
mvo80 tests to run14:55
zyga-ubuntuoh drat14:56
zyga-ubuntumvo: well, this is master14:56
mvozyga-ubuntu: yeah14:56
zyga-ubuntumvo: shall we disable suse or shall we bump the number of workers by one?14:56
zyga-ubuntumvo: we can do a tailored PR in 2.2914:56
mvozyga-ubuntu: adding a worker was not enough, I'm in favor of disabling14:57
mvozyga-ubuntu: but cachio said he is looking into it14:57
zyga-ubuntumvo: ok14:57
cachiomvo, zyga-ubuntu I am reproducing the error here15:01
cachiowe can disable it until the problem is fixed15:01
zyga-ubuntudo we have any idea why it might be slower?15:01
cachiono yet, waiting for the timeout yet15:02
cachiozyga-ubuntu, let's disabe opensuse, then we can enable it again15:02
cachiozyga-ubuntu, mvo I tested here and it worked15:15
cachioperhaps is a temporal issue15:15
=== cachio is now known as cachio_lunch
zyga-ubuntucachio_lunch: did oyu test on linode?15:17
kyrofasergiusens, do we want to determine a landing order for these other PRs?15:25
kyrofa(so we know which ones to fix first)15:25
mvozyga-ubuntu: you have review feedback for 423015:30
zyga-ubuntulooking15:30
mvozyga-ubuntu: let me know if you want help :)15:30
jdstrandyes, was just going to ping you on that :)15:30
mvozyga-ubuntu: but looks straightforward15:30
zyga-ubuntuthank you jdstrand15:30
zyga-ubuntumvo: testing and I'll push in a sec15:45
mupPR snapcraft#1743 opened: catkin plugin: support building entire workspace <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1743>15:49
zyga-ubuntumvo: did you disable suse?15:51
zyga-ubuntumvo: I'd like to pull master and then push this15:51
zyga-ubuntumvo: you can, I think, just git push the backend to manual15:52
zyga-ubuntumvo: to save us some time15:52
zyga-ubuntumvo: I pushed, please merge/squash15:55
zyga-ubuntujdstrand: https://github.com/snapcore/snapd/pull/4230/files15:56
mupPR #4230: tests: add test to run snap inside lxd as a user <Created by mvo5> <https://github.com/snapcore/snapd/pull/4230>15:56
* zyga-ubuntu needs to play with those real/effective/saved ids thing more 15:57
=== cachio_lunch is now known as cachio
cachiozyga-ubuntu, I tested on linode15:59
cachiozyga-ubuntu, I executed again the change to add a new worker for suse15:59
cachiolet's see if it fails again16:00
cachioif it fails I'll disable opensuse16:00
jdstrandzyga-ubuntu: done. sorry I forgot something in my suggestion (teeny change)16:01
zyga-ubuntujdstrand: aha16:01
zyga-ubuntujdstrand: pushed :)16:02
* zyga-ubuntu needs to run16:03
=== zyga-ubuntu is now known as zyga-afk
cachiojdstrand, https://paste.ubuntu.com/25981162/16:03
cachio jdstrand, it is not from the tests16:03
* kalikiana dinner time16:03
zyga-afkmvo: please get this to beta tonight if we can16:04
zyga-afkmvo: I'll be back in some hours16:04
mvozyga-afk: I will push for it16:05
kyrofaelopio, one issue with the refactor now that I'm using it: it tells me tests are skipped, but doesn't say it's because they were slow16:11
kyrofaBefore the refactor it was helpful and said "skipping slow test" or something like that16:11
mvozyga-afk: once tests are in I will run the machinery16:11
mvozyga-afk: thanks a bunch of fixing this!16:11
pedronismvo: I did another pass on managed,  but didn't see your last commit, now I saw, I think the placement of that code is problematic16:15
pedronis*saw it16:16
sergiusenskyrofa oh, if that is happening, I'd agree16:16
kyrofaAlright, fixing16:16
sergiusenskyrofa also, rebase on top of new master16:17
sergiusenskyrofa I did minor piggybacking on that last merge (sorry, but sorry)16:17
kyrofacratliff, catkin-tools has finally landed!16:17
sergiusenskyrofa and take extra care on your rebases/merges as things need to manually move on your side for tests to keep on running ;-)16:18
kyrofasergiusens, yeah I'm working through the ament one now16:18
kyrofaBasically, if you added any new test files, they need to move, but if you modified ones that were already there, you're mostly there16:18
mupPR snapcraft#1593 closed: catkin tools plugin: add catkin tools support <Created by cratliff> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1593>16:19
pedronismvo: we can discuss options on Monday16:21
=== ikey|zzz is now known as ikey
pedronismvo: do you need anything more from me today?16:26
mvopedronis: thank you, I think I'm fine, I will baby-sit the build and do 2.29.416:29
elopiokyrofa: how weird. Please report a bug, and I'll look at it as soon as I'm over with the call for testing.16:31
elopioflexiondotorg: popey: can I send a coschedule post linking to the call for testing? https://forum.snapcraft.io/t/call-for-testing-snapcraft-2-35/288016:31
mupPR snapd#4244 opened: disabling opensuse until timeout issue is fixed <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/4244>16:33
cachiomvo, failed again on opensuse, this is the PR https://github.com/snapcore/snapd/pull/424416:33
mupPR #4244: disabling opensuse until timeout issue is fixed <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/4244>16:33
cachioI am still trying to reproduce it for here16:33
mvocachio: thanks16:37
mvocachio: once its green I will merge and push into the other two 2.29 prs16:37
cachiomvo, good16:37
popeyelopio: great idea!16:42
pedroniscachio: thanks16:42
popeyelopio: make sure you use the link button to give you a tagged link :)16:42
popeyget them vaulable internet points!16:42
kyrofasergiusens, alright, ament plugin is up to date16:43
popeyhttps://forum.snapcraft.io/t/call-for-testing-snapcraft-2-35/2880?u=elopio <-16:43
elopiopopey: what is a tagged link? Does my user get internet points?16:52
popeyYes!16:52
popeysee the url above, got your name in it16:53
elopiooh man, I've been stripping that u everywhere! So much karma lost :(16:53
popeysad4u16:53
cachiomvo, https://paste.ubuntu.com/25982089/16:53
cachioit took 5 minutes to install python3-docutils in opensuse16:53
elopiohttps://en.wikipedia.org/wiki/Success_Kid#/media/File:SuccessKid.jpg16:53
cachioit is in linode16:54
cachiomvo, it should take few minutes16:54
cachiomvo, few seconds16:54
cachio<30s16:54
mvocachio: woah16:55
mupPR snapd#4245 opened: interfaces/screen-inhibit-control: handle ScreenSaver/Screensaver in DBus API <Created by jdstrand> <https://github.com/snapcore/snapd/pull/4245>16:55
cachiomvo, testing in a vm on aws to compare17:00
mvocachio: thanks. 4244 has not even started yet, slightly annyoing17:03
mvojdstrand: does 4230 looks good to you now?17:05
cachiomvo, in aws works pretty fast17:06
cachiomvo, the problem is in linode17:06
mupPR snapd#4243 closed: tests: disable classic-ubuntu-core-transition on i386 temporarly <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4243>17:06
cachiomvo, any idea why I could get this error?17:11
cachiohttps://paste.ubuntu.com/25981162/17:11
cachiomvo, I have a snap running as a service, this is the snapcraft.yaml17:12
cachiomvo, also getting this denial when I start the service17:12
cachiohttps://paste.ubuntu.com/25980351/17:12
mupPR snapd#4230 closed: tests: add test to run snap inside lxd as a user <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/4230>17:15
mupPR snapd#4246 opened: snap-confine: fix snap-confine under lxd <Created by mvo5> <https://github.com/snapcore/snapd/pull/4246>17:15
mvojdstrand: -^17:15
mvocachio: new beta is still 1-2h away, tests are slow :(17:19
cachionp17:19
cachiomvo, any idea about the problem with the network-status-provider snap?17:24
kyrofasergiusens, pip extraction has also been updated17:31
magicaltrouthello folks17:35
magicaltrouthow do you install a classic snap from a local build?17:35
magicaltrout sudo snap install --classic ./my-snap-name_0.1_amd64.snap17:35
magicaltrouterror: cannot find signatures with metadata for snap "./my-snap-name_0.1_amd64.snap"17:35
naccmagicaltrout: --dangeours17:37
nacc*dangerous17:37
magicaltroutwinning17:37
naccmagicaltrout: it basically says there is no store data for it17:37
jdstrandmvo: yep, done (as per other channel)17:51
mvojdstrand: thank you17:51
jdstrandmvo: is 2.30 already branched?17:52
jdstrandI guess I can figure that out myself17:52
jdstranddoesn't look like it17:53
mvojdstrand: not branched yet, why?17:53
jdstrandmvo: just wanted to know if I needed to send up 2.30 branches too17:54
mvothanks17:55
cratliffkyrofa  That's great.  I saw the .36 milestone and thought it would be a while.  Glad to see it's in.  I should try checking out the extended workspaces sometime soon.17:58
sergiusenskyrofa I have a new branch up too which could use a look18:07
kyrofasergiusens, as do I-- the catkin-build-entire-workspace one18:07
mupPR snapcraft#1744 opened: elf: conversion from libraries <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1744>18:07
sergiusenskyrofa I saw it ;-)18:07
* kyrofa reads PEP 48418:10
mupPR snapd#4244 closed: disabling opensuse until timeout issue is fixed <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4244>18:14
kyrofaDarn, autopkgtest queue has built up a bit again18:24
kyrofaelopio, pretty sure the bot is not pinging subscribers18:33
kyrofaI don't think it's ever pinged me18:34
elopiokyrofa: yup. Pretty sure you broke it ;)18:34
kyrofaNu uh! It NEVER pinged me, I swear18:34
elopiokyrofa: so, I think next week I can add tests, and then it should be obvious what's wrong. No hurries there, right?18:34
kyrofaelopio, yeah, no rush other than curiosity, haha18:34
kyrofaelopio, kinda fun to poke at it. It'll be even better with tests18:35
kyrofaelopio, let me know? Happy to review18:35
kyrofaDoes errbot have some sort of testing lib?18:37
elopiokyrofa: yes, it has. Should be easy to fake integration tests.18:38
kyrofaSweet18:38
kyrofaman this new firefox is ugly18:45
kyrofasnappy-m-o, autopkgtest 1583 xenial:i38618:47
snappy-m-okyrofa: I've just triggered your test.18:47
kyrofasnappy-m-o, autopkgtest 1607 xenial:i38618:48
snappy-m-okyrofa: I've just triggered your test.18:48
brunosferHi19:02
pdefreitashi19:03
brunosferhi19:04
pdefreitassnapamos19:04
brunosferhi19:10
brunosferhi19:12
brunosferhi19:15
pdefreitashi19:15
brunosferI'm trying to build a snap for offline connectivity. Do you know any good tutorial on how to begin with that?19:16
naccbrunosfer: "for offline connectivity"?19:16
sergiusenskyrofa it is not pinging19:37
sergiusensand never worked in this irc form either19:37
cachiomvo, news about the beta?19:39
mupPR snapd#4247 opened: interfaces: allow /bin/chown and fchownat to root:root <Created by jdstrand> <https://github.com/snapcore/snapd/pull/4247>19:41
jamesbensonchipaca?19:42
sergiusenskyrofa is my use of mypy your only concern in the PR?20:00
mvocachio: unfortunately not, tests still not finished :(20:02
kyrofasergiusens, not done looking through it yet20:02
cachiomvo, np20:04
om26erjdstrand: thanks for the approval. One question: since I filed that Android Studio request, I made many changes to the packaging and the latest revision(15) is what we want to release. Will that need a separate approval from you ?20:11
om26erhttps://dashboard.snapcraft.io/dev/snaps/8605/rev/15/20:12
om26eroops, it seems that got approved just now.20:14
jdstrandom26er: no. you are good to go20:24
mupPR snapcraft#1745 opened: static tests: upgrade to the newest flake8 <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1745>20:41
kyrofasergiusens, variable annotations are python 3.6 only20:43
sergiusenskyrofa do they fail for you?20:50
sergiusenskyrofa snapcraft is 3.620:51
kyrofasergiusens, not in the deb :)20:51
sergiusensunless you mean 3.5, for which they would be silently ignored20:51
kyrofaI've not tried, was concerned they'd be syntax errors20:51
sergiusenskyrofa does it fail? it says 3.5 here -> https://docs.python.org/3/library/typing.html20:51
sergiusenswhere did you get 3.6?20:52
kyrofasergiusens, comment on the PR. I'm specifically talking about syntax like this: ldd_out: List[str] = []20:52
kyrofaAnnotations on variables, not functions/classes20:53
sergiusensoh, ic20:53
sergiusenskyrofa ok, enough with static checking, what about the rest?20:54
zyga-afkre20:54
=== zyga-afk is now known as zyga
zygamvo: still here?20:54
mvozyga: yes20:54
sergiusensthe comment notation is sad20:54
zygamvo: how are things? I saw the PR20:54
mvozyga: we have unhappy tests, otherwise all is well20:54
zygamvo: unhappy with LXD or random annoying failing test?20:54
mvozyga: random20:55
mvozyga: opensuse20:55
mvozyga: and also random20:55
sergiusenselopio snapcraft#1745 is for you btw ;-)20:55
mupPR snapcraft#1745: static tests: upgrade to the newest flake8 <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1745>20:55
sergiusenskyrofa after that I'll propose a general mypy sanity fix and rebase my branch on that20:55
kyrofaAlright. Noticed mypy isn't in the archives, too bad20:56
zygamvo: drat20:56
zygamvo: restart-travis-as-a-service20:56
kennylogginsdo I just apt update to get the new snapd ?20:56
zyga"one does not just apt update to get new snapd" (with the mordor thing meme)20:57
zygakennyloggins: it depends20:57
zygakennyloggins: sometimes a core-reexec issue is faster than apt update20:57
kennylogginsIDK - i just went with stable - tell me what to do ?20:58
zygakennyloggins: stable is usually fine unless a specific issue is affecting you20:58
kennylogginsIam just a user.20:58
sergiusenskyrofa why would that matter?20:59
sergiusenswe don't depend on the archives for any of the static tests20:59
zygakennyloggins: in that case I'd suggest sticking to stable20:59
mupPR snapd#4248 opened:  snap-confine: fix snap-confine under lxd <Created by mvo5> <https://github.com/snapcore/snapd/pull/4248>21:00
kyrofasergiusens, just makes things easier, that's all. Makes our dev guide harder21:00
kennylogginsokay - I shallnot do anthing then. just saw this post:21:00
kennylogginshttps://forum.snapcraft.io/t/call-for-testing-snapcraft-2-35/2880/521:00
kyrofaNot a big deal21:00
zygakennyloggins: that's for snapcraft, right?21:04
zygakennyloggins: I usually think about snapd, not snapcraft, sorry, my bias21:05
kennylogginszyga: coolbeans. No idea wat I am doing ! Weeeeeeeeee21:08
kennylogginshttps://v.gd/4vCGRy21:08
sergiusenskyrofa but it doesn't change, it still is `pip install -r requirements-devel.txt`21:35
mupPR snapd#4241 closed: store: bit less aggressive retry strategy <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/4241>21:40
mupPR snapd#4248 closed:  snap-confine: fix snap-confine under lxd (2.29) <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4248>21:54
=== JoshStrobl is now known as JoshStrobl|Away
=== pbek_ is now known as pbek
zygamvo: what happened with that merge btw? why did you have to open a new PR?22:05
mvozyga: yes, messup22:07
kyrofaHaha, sergiusens you need to start using a github mobile app or something22:07
kyrofaThe emails are never threaded22:07
mvozyga: when I merge the opensuse fix something strange happend22:08
sergiusenskyrofa oh, but why, I am using mailspring ;-)22:08
mvozyga: 2.29.4 is building in the ppa now finnaly22:08
mvofinally22:08
sergiusensthreads are a thing of the past :-P22:08
kyrofaHahaha22:08
zygamvo: thank you :-)22:08
zygamvo: so, tomorrow morning my lxd container should work ok22:09
zygamvo: (well, until I fix the other bug it will be affected by reomve)22:09
zygamvo: but that should give us a chance to releas next week22:10
kyrofaWhat the heck... sergiusens all of a sudden libpython2.7-minimal is including a sitecustomize in /etc/python2.7/22:19
sergiusenskyrofa darn, I got distracted with your comment :-P focused the window to say ... er just that22:19
kyrofaI didn't know that was a valid location!22:20
mvozyga: yeah, I hate to leave this broken over the WE, it needs to be at least in beta imo22:21
sergiusenskyrofa but, there has been no updates to python in a while on 16.04 https://launchpad.net/ubuntu/+source/python2.722:21
sergiusenskyrofa did you shuffle code around?22:22
kyrofaIt was one of the PRs I rebased22:22
kyrofaNot sure why they're clashing anyway... I suspect I broke something22:23
kyrofaOh. No, that's where we're placing ours now. What...22:24
kyrofaEh, I'll figure it out22:24
kyrofaOh it's a symlink. How interesting22:25
zygamvo: if you manage to let's also write something on the forum22:30
zygamvo: (I can do that if you'd like)22:30
gsilvaptHello all. I want to follow elopio suggestion on the forum to test the latest snapcraft version22:31
gsilvaptI can do it by cloning the repo and using the snapcraft's binaries to perform stuff, right?22:31
zygagsilvapt: did his post include instructions?22:31
zygagsilvapt: sorry for a naive question, I didn't read it22:32
gsilvapthttps://forum.snapcraft.io/t/call-for-testing-snapcraft-2-35/288022:32
gsilvaptzyga, kind of. I bet my question is fairly simple but the problem is I already have snapcraft installed22:32
gsilvaptBut I know there is a way to have 2 installations working22:32
kyrofagsilvapt, you can just test the snap if you like22:33
kyrofaBut yes, you can also run it from source22:33
zygagsilvapt: I'm sure that's true, I'm sorry I canont help you more22:33
gsilvaptkyrofa, under the snapcraft folder, right?22:34
gsilvaptI need to get used to these things if I want to contribute to snapcraft :P22:34
gsilvaptno worries, zyga :D22:34
kyrofagsilvapt, here you go: https://github.com/snapcore/snapcraft/blob/master/HACKING.md22:34
kyrofagsilvapt, I suggest doing it that way22:34
gsilvaptok, I have that installed. I need to checkout to branch 2.5 and perform the steps there, right?22:35
gsilvapts/2.5/2.3522:35
gsilvaptok, nevermind, I'm using the version I need :-=)22:36
gsilvaptok, nevermind, I'm using the version I need :-)22:36
mvojdstrand, zyga store upload of 2.29.4 core fails because of the review scripts apparently, the error is "found errors in file output: unusual mode 'rws...22:37
zygaah22:38
zygadrat22:38
zygamvo: store checks for setgid22:38
zygamvo: we need a fix from jdstrand and roadmr22:38
roadmrzyga: hi! which version of the review tools do you need for this?22:38
zygaroadmr: the one unwritten :)22:39
roadmrzyga: I have 2 in the queue right now, aiming for a Monday rollout22:39
roadmrzyga: dagnabbit :D22:39
zygaroadmr: in reality we need one off approval22:39
mvoroadmr, jdstrand, zyga here is an example https://launchpad.net/~snappy-dev/+snap/core/+build/10828122:39
zygaroadmr: we can fix this properly next week22:39
zygabut it would be good if that upload was approved22:39
gsilvaptkyrofa, are you available in 30 minutes? I would like to implement the `snapcraft version` command requested that we discussed in the other night. I haven't tried any further but I think tonight is the night :)22:39
roadmrzyga: so do you indeed want that weird mode?22:42
roadmrmvo: ^^22:42
roadmrzyga: I'm not really sure what to do :( because the status on the store is "failed review", not "awaiting manual review". The latter, I could possibly review and approve; but the former, and it being core, and it being Friday evenight, sounds like a recipe for getting yelled at :(22:43
mvoroadmr: yes, its a long story22:43
mvoroadmr: right, yes, I think your concerns are sensible22:43
mvoroadmr: the backstory is https://forum.snapcraft.io/t/snapcraft-adt-failures-with-the-new-core-release/2850/40 and we are quite eager to fix this regression but if its complicated/dangerous its not the best time on friday :(22:45
roadmrmvo: I think it is, for reasons explained above :( I might chance it if I had more experience with reviews, but in truth, jdstrand handles those hairy bits so I'd be doing it for the first time.22:45
mupPR snapd#4246 closed: snap-confine: fix snap-confine under lxd <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4246>22:46
kyrofagsilvapt, yes, I'll be here :)22:47
kyrofaHappy to help22:48
zygamvo: it's kind of depressing we dind't realize this yesteray22:48
mvozyga: yes, slightly depressing22:49
zygaroadmr: can we make the modification to the store to allow that different mode?22:51
roadmrzyga: it's click-reviewer-tools rejecting the mode22:52
roadmrit just spits out an error in the review results, and that's what causes the store to consider the upload rejected22:52
mvozyga: please send me a tg if this lands in the store, then I will promote to beta tomorrow morning, otherwise it will be monday (slightly sad)22:54
zygaok22:55
zygaroadmr: who maintains c-r-t?22:55
roadmrzyga: jdstrand does22:55
zygajdstrand: can you make that modification?22:55
zygaok, I modified c-r-t23:03
zygaroadmr: would you deploy my change or is that too outside of protocol?23:03
roadmrzyga: I can't even deploy it myself, we'd need complicity from a webop23:04
zygathe non-test diff is: http://paste.ubuntu.com/25984118/23:05
roadmrone-character diff :)23:06
zygahmm, setup.py test on clean master seems to fail23:07
zygaFAILED (failures=4)23:07
roadmr:(23:07
gsilvaptkyrofa,I'm just not getting how the method to get the version is called using --version23:08
gsilvaptIf you could help understand that bit, perhaps I could work on a solution to have it printing snapcraft's version using version and --version23:08
kyrofagsilvapt, sure, let me look, here23:09
gsilvaptWhen I look at the internals/__init__.py, I understand the method that gets the version number is the _get_version and that method can be reused. I'm just not understanding where is that method called23:10
zygaok, I give up23:10
kyrofagsilvapt, we use a library called "click" for our CLI handling23:10
zygajdstrand: if you make that modificaiton please leave me or mvo a message23:10
* zyga waves good night23:10
kyrofagsilvapt, one of its features is that it has a `version_option` decorator accepts a version number, and adds a --version option23:11
gsilvaptHum, I remember seeing some bits of code with that somewhere23:11
kyrofagsilvapt, this is done in snapcraft/cli/__init__.py23:11
kyrofagsilvapt, so I suggest you continue using the snapcraft.__version__ attribute23:13
kyrofagsilvapt, but you'll need to add a new command23:13
gsilvaptAh, I see it. I once stepped into this when I was trying to find what was going on23:13
gsilvaptHum. So there is no chance of re-using the click library? I haven't look at the code but lets see if I can figure this out with these instructions23:14
kyrofagsilvapt, I suggest doing that in a new file, snapcraft/cli/version.py23:14
kyrofaOh yes, you'll use the lib, but I had a quick look and it doesn't look like it supports magically adding a "version" command like it did for the --version option23:14
mupPR snapcraft#1745 closed: static tests: upgrade to the newest flake8 <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1745>23:14
kyrofaBut that's okay, adding a command is still pretty easy23:14
kyrofagsilvapt, I suggest you refer to snapcraft/cli/help.py23:15
kyrofaYours should look pretty similar23:15
kyrofaBut even simpler23:15
gsilvaptJust to check: This line is the one that actually adds the --version option, right? `@click.version_option(version=snapcraft.__version__)`23:16
gsilvaptkyrofa, would it make sense to have both commands in the same file?23:19
kyrofagsilvapt, indeed, that line23:20
kyrofagsilvapt, that wouldn't follow the convention established for other commands23:20
kyrofagsilvapt, sticking to established conventions is one way we're all able to share a codebase23:21
kyrofagsilvapt, so no, I suggest putting it in snapcraft/cli/version.py23:21
gsilvaptkyrofa, that's not my suggestion, I did not use the most correct words. Shouldn't --version and version commands be established in the same file? i.e, should I move the --version option to this new file so that they stay together?23:22
kyrofagsilvapt, ah, I see23:23
kyrofagsilvapt, as far as I can see, that's not actually possible, as the --version option must be specified on the root command, `snapcraft`, which is defined right there in __init__.py23:23
gsilvaptHum, ok I see23:24
gsilvaptSo, lets get some implementations done and hopefully running23:24
kyrofaJust look closely at how help is done23:25
kyrofasnappy-m-o, autopkgtest 1743 xenial:arm6423:29
snappy-m-okyrofa: I've just triggered your test.23:29
kyrofasnappy-m-o, autopkgtest 1743 artful:amd6423:30
snappy-m-okyrofa: I've just triggered your test.23:30
gsilvaptI think I may have found a working solution but I think I have a poor configuration of lxc23:30
gsilvapttried running ./runtests.sh snapcraft/tests/unit/commands and got this in the first lines: https://pastebin.com/GSvYFCt123:31
gsilvaptWell, I keep getting those after a few tests23:33
kyrofaHmm.... not seen that before23:33
gsilvaptunittest generates a full report when it's done, am I correct? It's hard to keep looking at all these tests23:34
kyrofagsilvapt, it'll die if it errors23:36
kyrofaAlthough I suspect it won't if all you did was add a new command. Did you write tests for it as well?23:36
gsilvaptyes, it said it failed anyway. And my test failed :-D23:37
gsilvaptToo bad I can't use Pycharm's test feature to just test the one I wrote23:37
gsilvaptyes, I did a similar test to the existing one using version23:37
kyrofaYou can always try `python3 -m unittest snapcraft.tests.unit.commands.test_version (or whatever you called it)23:37
gsilvapthum, right23:38
gsilvaptbut is there an actual reason why PyCharm can't compile snapcraft?23:38
gsilvaptI have I feeling I have lots of tools poorly configured :|23:38
kyrofagsilvapt, I'm afraid I don't know-- I don't use pycharm23:39
gsilvaptkyrofa, vim user?23:42
kyrofaA mixture of vim and atom23:42
kyrofaWhen atom decides not to crash X23:42
gsilvaptI used to use Vim but recently started using IntelliJ for Java and just fell in love with its features. So I decided to give a try to PyCharm. I still use Vim modal editing style. I can't go back to not use it23:44
gsilvaptBy the way, I think I have some dependencies missing, even after installing requirements and requirements-devel23:44
gsilvaptkyrofa, elopio, in case you have seen anything like this before: https://pastebin.com/BDZFzmrQ23:47
gsilvaptI'm considering removing and reinstalling lxc/lxd23:47
elopiogsilvapt: weird. I have seen weird lxd errors in the past, but now It's pretty much stable for me.23:53
elopioI'm on xenial, using the lxd snap. If the snap doesn't work for you, you can try the deb.23:53
gsilvaptSo, purge lxd/lxc, reinstall and try again23:54
gsilvaptok, glad I chose a free night for this, ehehe23:54
gsilvaptBut that basically means it is failing to start containers somehow23:54
elopiogsilvapt: if you installed with the deb, you need to purge lxd and lxd-client23:55
gsilvaptI think the system is lxd clean23:56
gsilvaptelopio, do I need a particular version of Ubuntu in the container?23:58
gsilvaptI previously had 17.1023:58
gsilvaptSince you're here and the expert using lxd, should I remove the containers after using them?23:59

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