IRC logs for #buildstream for Thursday, 2019-09-05

*** narispo has quit IRC00:20
*** narispo has joined #buildstream00:20
*** narispo has quit IRC00:50
*** narispo has joined #buildstream00:50
*** narispo has quit IRC01:20
*** narispo has joined #buildstream01:21
*** narispo has quit IRC01:51
*** narispo has joined #buildstream01:51
*** narispo has quit IRC02:22
*** narispo has joined #buildstream02:22
*** narispo has quit IRC02:48
*** raoul has quit IRC02:48
*** jward has quit IRC02:48
*** pro[m] has quit IRC02:48
*** awacheux[m] has quit IRC02:48
*** connorshea[m] has quit IRC02:48
*** krichter[m] has quit IRC02:48
*** tchaik[m] has quit IRC02:48
*** segfault3[m] has quit IRC02:48
*** abderrahim[m] has quit IRC02:48
*** dbuch has quit IRC02:48
*** gimpnet-irc[m] has quit IRC02:48
*** hergertme has quit IRC02:48
*** valentind has quit IRC02:48
*** jswagner has quit IRC02:48
*** lchlan has quit IRC02:48
*** benschubert_ has quit IRC02:48
*** cs-shadow has quit IRC02:48
*** cgmcintyre[m] has quit IRC02:48
*** verdre[m] has quit IRC02:48
*** Demos[m] has quit IRC02:48
*** asingh_[m] has quit IRC02:48
*** doras has quit IRC02:48
*** reuben640[m] has quit IRC02:48
*** rafaelff[m] has quit IRC02:48
*** bethw has quit IRC02:48
*** swick has quit IRC02:48
*** jennis has quit IRC02:48
*** persia has quit IRC02:48
*** juergbi has quit IRC02:48
*** coldtom has quit IRC02:48
*** kapip has quit IRC02:48
*** ironfoot has quit IRC02:48
*** jjardon has quit IRC02:48
*** aevri has quit IRC02:48
*** phildawson has quit IRC02:48
*** tpollard has quit IRC02:48
*** slaf has quit IRC02:48
*** m_22[m] has quit IRC02:48
*** ssssam[m] has quit IRC02:48
*** nielsdg has quit IRC02:48
*** dineshdb[m] has quit IRC02:48
*** tlater[m] has quit IRC02:48
*** Trevinho[m] has quit IRC02:48
*** waltervargas[m] has quit IRC02:48
*** mattiasb has quit IRC02:48
*** mrmcq2u[m] has quit IRC02:48
*** skullone[m] has quit IRC02:48
*** theawless[m] has quit IRC02:48
*** jjardon[m] has quit IRC02:48
*** albfan[m] has quit IRC02:48
*** kailueke[m] has quit IRC02:48
*** mohan43u has quit IRC02:48
*** coldtom has joined #buildstream02:48
*** lchlan has joined #buildstream02:49
*** jjardon has joined #buildstream02:49
*** raoul_ has joined #buildstream02:49
*** phildawson has joined #buildstream02:49
*** tpollard has joined #buildstream02:49
*** slaf has joined #buildstream02:49
*** m_22[m] has joined #buildstream02:49
*** ssssam[m] has joined #buildstream02:49
*** nielsdg has joined #buildstream02:49
*** dineshdb[m] has joined #buildstream02:49
*** tlater[m] has joined #buildstream02:49
*** Trevinho[m] has joined #buildstream02:49
*** waltervargas[m] has joined #buildstream02:49
*** mattiasb has joined #buildstream02:49
*** mrmcq2u[m] has joined #buildstream02:49
*** skullone[m] has joined #buildstream02:49
*** theawless[m] has joined #buildstream02:49
*** jjardon[m] has joined #buildstream02:49
*** albfan[m] has joined #buildstream02:49
*** kailueke[m] has joined #buildstream02:49
*** irc.acc.umu.se sets mode: +o jjardon02:49
*** aevri has joined #buildstream02:49
*** ironfoot has joined #buildstream02:50
*** ChanServ sets mode: +o ironfoot02:50
*** mohan43u has joined #buildstream02:50
*** cs-shadow has joined #buildstream02:50
*** benschubert_ has joined #buildstream02:50
*** hergertme has joined #buildstream02:50
*** persia has joined #buildstream02:51
*** narispo has joined #buildstream02:52
*** jswagner has joined #buildstream02:52
*** narispo has quit IRC02:52
*** kapip has joined #buildstream02:53
*** narispo has joined #buildstream02:53
*** bethw has joined #buildstream02:53
*** swick has joined #buildstream02:53
*** juergbi has joined #buildstream02:54
*** valentind has joined #buildstream02:54
*** jennis has joined #buildstream02:55
*** jward has joined #buildstream02:57
*** narispo has quit IRC03:22
*** narispo has joined #buildstream03:53
*** narispo has quit IRC03:53
*** narispo has joined #buildstream03:54
*** narispo has quit IRC04:23
*** narispo has joined #buildstream04:24
*** narispo has quit IRC04:54
*** narispo has joined #buildstream04:55
*** narispo has quit IRC05:24
*** narispo has joined #buildstream05:25
gitlab-br-botmarge-bot123 merged MR !1564 (juerg/cache-usage->master: Display cache usage in status bar) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/156405:47
*** awacheux[m] has joined #buildstream05:52
*** narispo has quit IRC05:55
*** narispo has joined #buildstream05:56
*** narispo has quit IRC06:25
*** narispo has joined #buildstream06:26
*** narispo has quit IRC06:57
*** narispo has joined #buildstream06:57
*** narispo has quit IRC07:26
*** narispo has joined #buildstream07:27
*** narispo has quit IRC07:57
*** narispo has joined #buildstream07:58
*** rdale has joined #buildstream08:01
jjardonjuergbi: hi, would it be possible to tag the commits referenced at https://buildstream.build/buildbox_install.html#building-buildbox-components; nothing formal 0.0.1alpha or similar would be enough. It would make easier to identify the commits on the future08:15
*** narispo has quit IRC08:27
*** narispo has joined #buildstream08:28
*** connorshea[m] has joined #buildstream08:28
juergbijjardon: that could make sense. I'll check with the other BuildBox devs08:30
gitlab-br-botmarge-bot123 merged MR !1568 (jennis/update_push_pull->master: Update artifact push and pull so that they can handle artifact refs) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/156808:40
*** segfault3[m] has joined #buildstream08:48
*** narispo has quit IRC08:58
*** narispo has joined #buildstream08:59
*** rdale has quit IRC09:17
*** jonathanmaw has joined #buildstream09:17
*** rdale has joined #buildstream09:19
*** abderrahim[m] has joined #buildstream09:19
*** traveltissues has joined #buildstream09:22
*** narispo has quit IRC09:28
*** narispo has joined #buildstream09:29
*** cgmcintyre[m] has joined #buildstream09:37
*** narispo has quit IRC10:00
*** narispo has joined #buildstream10:00
*** shash has joined #buildstream10:20
shashHey Kinnison10:21
shashI'd appreciate a review on https://gitlab.com/BuildStream/buildstream/merge_requests/1458 :)10:21
KinnisonSure, I'll look in a bit10:26
*** narispo has quit IRC10:30
*** narispo has joined #buildstream10:30
*** doras has joined #buildstream10:31
jennistlater[m], i'm having a look at your MR10:31
jennisCan we have a situation where we have storage remotes *only*?10:31
tlater[m]jennis: No, not post-changes from two days ago10:31
jennishuh?10:31
tlater[m]Two days ago you could10:31
tlater[m]But I changed my mind; it doesn't make much sense10:32
jennisYeah, fair enough10:32
jennisSo if you want to make use of another CAS client for storage, you still need to run an index remote for BuildStream to "work"?10:32
jennisThat makes sense in my head, I'd imagine it'd be difficult to get things working otherwise10:32
Kinnisonshash: my 👍 remains10:33
tlater[m]jennis: Yup, that's the intention10:33
Kinnisonshash: if you can't get through to gökçen and the pipe is green, I suggest you resolve that discussion anyway and marge it10:34
jennistlater[m], cool, thanks10:34
*** narispo has quit IRC10:36
*** narispo has joined #buildstream10:37
*** rafaelff[m] has joined #buildstream10:47
*** verdre[m] has joined #buildstream10:50
*** tme5 has joined #buildstream10:50
*** narispo has quit IRC10:58
*** narispo has joined #buildstream10:58
*** reuben640[m] has joined #buildstream11:01
*** dbuch has joined #buildstream11:09
shashWill do! Thanks!11:21
gitlab-br-botmarge-bot123 merged MR !1458 (shashwatdalal/docker-source-patch->master: Add default permissions when adding to CAS and when linking files) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/145811:22
gitlab-br-bottraveltissues approved MR !1565 (jennis/update_checkout->master: Update artifact checkout to handle artifact refs) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/156512:00
*** asingh_[m] has joined #buildstream12:03
*** gimpnet-irc[m] has joined #buildstream12:05
*** tchaik[m] has joined #buildstream12:32
*** krichter[m] has joined #buildstream12:34
*** Demos[m] has joined #buildstream12:43
gitlab-br-botmarge-bot123 merged MR !1565 (jennis/update_checkout->master: Update artifact checkout to handle artifact refs) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/156513:05
*** pro[m] has joined #buildstream13:07
*** tiagogomes_ has quit IRC13:19
*** tiagogomes_ has joined #buildstream13:27
*** tme5 has quit IRC13:29
*** tiagogomes_ has quit IRC13:30
*** tme5 has joined #buildstream13:34
gitlab-br-botjennis closed issue #822 (Mark bst checkout/pull/push as obsolete) on buildstream https://gitlab.com/BuildStream/buildstream/issues/82213:51
*** tiagogomes_ has joined #buildstream14:11
gitlab-br-bottlater closed issue #1011 (Create artifact cache for buildstream tutorial builds) on buildstream https://gitlab.com/BuildStream/buildstream/issues/101114:18
gitlab-br-botjennis closed issue #1006 (New YAML world order API is not good) on buildstream https://gitlab.com/BuildStream/buildstream/issues/100614:20
gitlab-br-botjennis closed issue #988 (Standalone test failure for buildtrack) on buildstream https://gitlab.com/BuildStream/buildstream/issues/98814:27
*** jonathanmaw has quit IRC15:04
*** dtf has joined #buildstream15:04
*** jonathanmaw has joined #buildstream15:17
*** traveltissues has quit IRC15:19
*** traveltissues_ has joined #buildstream15:19
juergbijjardon: I've tagged 0.0.1 for buildbox-common and buildbox-casd15:21
jjardonjuergbi: thanks a lot!15:21
*** jonathanmaw has quit IRC15:29
*** jonathanmaw has joined #buildstream15:32
*** traveltissues_ has quit IRC15:37
*** traveltissues_ has joined #buildstream15:38
gitlab-br-bottlater opened (was WIP) MR !1540 (tlater/cache-endpoints->master: Support separate end points for artifact caches) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/154015:42
jennistlater[m] and I have noticed that when we run the test suite we see a DeprecationWarning from one of our dependencies, pluginbase: /home/jennis/Software/buildstream/.tox/py37/lib/python3.7/site-packages/pluginbase.py:439: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 i15:52
jennist will stop working15:52
jennisNow, we're using pluginbase 0.7, it looks like in the latest release (1.0.0) it's not longer a problem.15:52
jennisI'm unsure what the procedure is for this, from reading CONTRIBUTING it says we should `make -C requirements` to update deps, but is it ok to just do one huge dependency update?15:52
tlater[m]juergbi: Your conftest.py blows up my test suite15:55
tlater[m]It really doesn't like having multiple conftest.pys15:56
juergbitlater[m]: what exactly is the issue?15:56
tlater[m]Let me just get a fresh stacktrace...15:56
tlater[m]juergbi: https://hastebin.com/ujubacozes.cs15:57
tlater[m]It's not related to pycache shenanigans, I've git clean -fxded a couple of times now, and manually removed those things too15:58
juergbiyou see this also on master or only with your branch?15:58
tlater[m]Only with my branch, strangely enough15:59
tlater[m]But I never played with any conftest things15:59
tlater[m]Getting rid of that file fixes the world though :)16:00
juergbitrying it here16:00
tlater[m]Oh, nevermind, it breaks on master too16:01
* tlater[m] might have not had that file previously16:01
juergbiand it doesn't blow up here with your branch16:01
juergbiseems odd why there would be any differences between systems with tox16:01
tlater[m]Eugh16:01
juergbiCI is happy and nobody else complained, afaik16:01
tlater[m]Tox was supposed to keep us from having these issues!16:02
juergbitlater[m]: http://wisercoder.com/importmismatcherror-python-fix/16:02
tlater[m]juergbi: Could you try py37-nocover?16:02
juergbirelated?16:02
juergbisure16:02
tlater[m]No, that was my first guess too16:02
tlater[m]I've blown up any and all pyc and pycache related things16:03
tlater[m]In every way I could think of16:03
juergbinow trying with your exact command line16:04
tlater[m]Eugh, what I'd give to have your build speeds16:04
* tlater[m] is still retrying master after `git clean -fxd`16:04
juergbioh, can reproduce it with -nocover16:05
tlater[m]Ah, that's a lead16:06
* tlater[m] wonders what that change does anyway16:06
tlater[m]And indeed, it doesn't break without -nocov for me16:07
tlater[m]I don't see why these fixtures would break the test suite16:09
juergbitlater[m]: pytest usedevelop=True seems to be the option making the difference16:09
tlater[m]Ah16:10
tlater[m]Huh16:10
* tlater[m] is a little stumped16:10
juergbimaybe the issue is that conftest.py from src/* gets installed?16:10
juergbiand then some kind of conflict16:10
* juergbi only knows the bare minimum of pytest16:11
tlater[m]Hm, maybe16:11
* tlater[m] wants usedevlop on for nicer stacktraces anyway16:11
tlater[m]And faster build times16:12
juergbiis there any downside of usedevelop for nocov?16:12
tlater[m]Maybe we should just forego that !nocover there?16:12
tlater[m]Hm, I don't see much of a point, no16:12
juergbithat's what I'm thinking16:13
tlater[m]It looks like pyc and pycache things are propagated anyway16:13
tlater[m]Although I don't understand how, given that the whole codebase should be yanked into the virutalenv16:13
* tlater[m] will locally patch that out for the time being, and raise an issue about it16:14
tlater[m]Though it is slightly concerning that we can't run pytest on a proper install of our codebase16:14
juergbitlater[m]: we don't install our tests anyway, do we?16:21
juergbiexcept for source tests16:21
tlater[m]Hm, I suppose16:22
juergbitlater[m]: let me know if you need any help with the fork-allowed changes. it's a bit different but should mostly be simpler16:27
tlater[m]juergbi: I might do16:28
tlater[m]But that might have to happen tomorrow due to imminent badminton16:28
* tlater[m] has already stumbled on them - might be difficult to sync with them without accessing context16:29
tlater[m]But I guess I can just jump through basecache.16:30
juergbishould actually not be necessary with the new approach16:33
gitlab-br-bottlater opened issue #1121 (pylint fails to import `testing/_sourcetests/conftest.py` when running without usedevelop) on buildstream https://gitlab.com/BuildStream/buildstream/issues/112116:34
tlater[m]Yeah, sorry juergbi, I'll prompt you some other day I need help with that, but hopefully should not :)16:36
*** tme5 has quit IRC16:36
tpollarda powercut in our office knocked the WSL runner offline, should be available again now16:49
*** traveltissues_ has quit IRC17:24
*** traveltissues has joined #buildstream17:53
*** traveltissues has quit IRC17:57
*** rdale has quit IRC21:10
*** narispo has quit IRC21:32
*** narispo has joined #buildstream22:18
*** narispo has quit IRC22:22
*** narispo has joined #buildstream22:22
*** narispo has quit IRC23:11
*** narispo has joined #buildstream23:11

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