IRC logs for #buildstream for Friday, 2018-11-30

*** abderrah1 has quit IRC00:48
*** abderrah1 has joined #buildstream00:48
*** cs-shadow has quit IRC00:57
*** reaper has joined #buildstream01:56
*** mrjester has joined #buildstream02:04
*** kapil___ has quit IRC02:48
*** kapil___ has joined #buildstream03:14
*** abderrahim has joined #buildstream03:17
*** abderrah1 has quit IRC03:18
*** abderrahim has quit IRC04:09
*** abderrahim has joined #buildstream04:10
*** mohan43u has quit IRC04:25
*** mohan43u has joined #buildstream05:11
*** kapil___ has quit IRC05:53
*** tristan has joined #buildstream05:58
*** tristan has quit IRC06:04
*** tristan has joined #buildstream06:04
*** nimish has joined #buildstream06:34
*** abderrah1 has joined #buildstream07:09
*** abderrahim has quit IRC07:11
*** tristan has quit IRC07:34
*** tristan has joined #buildstream08:45
Kinnisonjuergbi: Good morning.  !967 still looks mergeable if you have a chance.  I'm amazed something else wasn't merged overnight08:49
*** toscalix has joined #buildstream09:03
*** finn has joined #buildstream09:09
juergbiKinnison: I'm on it09:25
KinnisonCool, thanks for looking09:27
juergbilooks mostly fine but there are a few details that need to be fixed up09:31
KinnisonHopefully Nexus can get to those pretty early today09:31
* Kinnison feels somewhat responsible for that MR because he missed some important bits early on :(09:32
*** WSalmon_ has joined #buildstream09:37
Kinnisonjuergbi: also, nice catches.  I am clearly change-blind at this point on that MR :(09:38
*** alatiera has joined #buildstream10:04
gitlab-br-botjuergbi merged MR !981 (aevri/update_man->master: man/: update with changes since Apr 2018) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/98110:09
*** tpollard has joined #buildstream10:09
* Nexus has a look10:10
*** rdale has joined #buildstream10:11
*** raoul has joined #buildstream10:11
Nexusjuergbi, Kinnison all done10:18
* Kinnison reads "The yaml is mangled as part of loading" (part of the description of !980) and is super-sad :/10:19
benschuberttristan: Hey! Would you have time to have a look at !819? I'd like to be able to push it through since a few people are waiting on it :)10:24
gitlab-br-botMR !819: Add `--deps build` option to `bst checkout` https://gitlab.com/BuildStream/buildstream/merge_requests/81910:24
*** lachlan has joined #buildstream10:30
*** AmauryPi has joined #buildstream10:39
*** nimish has quit IRC10:46
juergbiNexus: CI failed10:53
*** Davnit__ has joined #buildstream10:54
*** finn_ has joined #buildstream10:59
*** finn has quit IRC11:00
*** andy has joined #buildstream11:20
*** andy has quit IRC11:29
*** rdale has quit IRC11:44
*** rdale has joined #buildstream11:50
*** finn_ has quit IRC11:54
*** finn_ has joined #buildstream11:54
gitlab-br-botgokcennurlu opened issue #798 ('bst pull' with --remote override fails to initialize remote because 'push not allowed') on buildstream https://gitlab.com/BuildStream/buildstream/issues/79811:55
*** nimish has joined #buildstream12:02
*** QRealm has joined #buildstream12:37
*** lachlan has quit IRC12:41
*** finn_ has quit IRC12:45
*** lachlan has joined #buildstream12:45
*** kyle__ has joined #buildstream12:47
*** nimish has quit IRC12:47
*** nimish has joined #buildstream12:47
*** finn has joined #buildstream12:56
*** finn_ has joined #buildstream12:57
*** finn has quit IRC12:57
*** nimish has quit IRC13:02
*** nimish has joined #buildstream13:03
tpollardjuergbi: did the slight tweak you asked for on !97813:05
gitlab-br-botMR !978: _stream.py: Ability to pull missing buildtrees outside of pull/build https://gitlab.com/BuildStream/buildstream/merge_requests/97813:05
jmacCan I set the cache quota in project.conf?13:43
tpollardI don't believe so13:45
tpollardfor the local cache13:45
juergbino, cache / cache expiry is global, and it's configured in global user config13:46
jmacOh well13:47
jmacI wonder how the quota behaves if I have two instances of BST running with different XDG_CACHE_HOME values13:48
juergbijmac: I'd recommend to also set XDG_CONFIG_HOME in that case13:50
jmacAha, that might be what I'm after13:50
gitlab-br-botjuergbi merged MR !978 (tpollard/774->master: _stream.py: Ability to pull missing buildtrees outside of pull/build) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/97814:10
gitlab-br-botjuergbi closed issue #774 (Provide the ability to pull missing buildtrees via the scheduler outside of user invoked bst pull & build) on buildstream https://gitlab.com/BuildStream/buildstream/issues/77414:10
*** nimish has quit IRC14:12
*** nimish has joined #buildstream14:13
tpollardcheers juergbi14:14
NexusSo, the topic of "What do we call the flag for a 'bst build world'" returns. The current thoughts seem to be either `--world` or having it default to build everything if no element is named e.g. `bst build` would build the entire project. This may conflict with building from the workspace directory however, as i believe this wants to use the default to build the element which the workspace came from.14:16
NexusAnd i believe it's important that we remain consistant, so to me, i think the `default` option isn't viable. But i am open to suggestions and ideas!14:16
juergbiI don't think it would be that surprising to the user if the default target in a workspace directory builds only the workspace and the default target in the project directory builds the whole project (potentially configurable in project.conf in a future version)14:19
*** nimish has quit IRC14:21
tpollardboth have merits, I'm not sure I have a strong enough opinion on either though14:25
*** nimish has joined #buildstream14:28
tpollardis anyone getting ''pyopenssl' third-party module is not installed. This meansFTPS support will be disabled. You can install it with:'pip install pyopenssl'' when running the tests recently?14:52
juergbitpollard: yes, maybe it should be added to dev-requirements.txt if apparently it's not implicitly pulled in14:55
juergbivalentind: ^^?14:55
juergbior it might not be needed at all, and is just a confusing warning14:55
tpollardall tests still pass for me, but it's definitely only appeared in the recent days14:56
*** cs-shadow has joined #buildstream14:56
juergbiyes, it's since the netrc support14:56
*** nimish has quit IRC15:06
*** nimish has joined #buildstream15:06
*** nimish has quit IRC15:36
*** nimish has joined #buildstream15:37
gitlab-br-botjuergbi closed issue #111 (Unix platform cannot interact with artifact shares) on buildstream https://gitlab.com/BuildStream/buildstream/issues/11115:46
gitlab-br-botjuergbi closed issue #335 (Loader tests are broken) on buildstream https://gitlab.com/BuildStream/buildstream/issues/33516:06
NexusWhen are we going to have parallel tests? :'(16:07
KinnisonYou can turn them on16:07
Kinnisonjuergbi: and I did some experimentation a few months back though16:07
Kinnisonand it turns out that pytest's overhead basically meant that if you gave it 2 runners it was helpful, but 4 was harmful16:07
KinnisonIIRC16:08
gitlab-br-botjuergbi closed issue #414 (Project doesn't build when using bst 1.1.3 (it builds fine with 1.1.2)) on buildstream https://gitlab.com/BuildStream/buildstream/issues/41416:10
*** toscalix has quit IRC16:16
gitlab-br-botjuergbi closed issue #598 (Fatal warnings prints None when detail not provided) on buildstream https://gitlab.com/BuildStream/buildstream/issues/59816:16
juergbion my desktop I got the best time with using roughly a third of the hardware threads16:20
Kinnisonthat'd match my expectation (8 thread system, 2 good, 4 bad)16:27
*** lachlan has quit IRC16:29
*** lachlan has joined #buildstream16:32
*** lachlan has quit IRC16:43
*** lachlan has joined #buildstream16:54
*** tpollard has quit IRC16:58
*** nimish has quit IRC17:07
*** nimish has joined #buildstream17:07
*** nimish has joined #buildstream17:07
*** phildawson_ has quit IRC17:11
*** WSalmon_ has quit IRC17:13
*** alatiera has quit IRC17:17
*** alatiera has joined #buildstream17:19
gitlab-br-botjuergbi merged MR !967 (mandatory_suffix->master: Mandatory .bst suffix) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/96717:21
gitlab-br-botraoul.hidalgocharman opened MR !982 (raoul/628-RE-flow-optimisation->master: Raoul/628 re flow optimisation) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/98217:29
*** finn_ has quit IRC17:35
*** raoul has quit IRC17:39
*** xjuan has joined #buildstream18:07
*** lachlan has quit IRC18:49
*** xjuan has quit IRC18:51
*** xjuan has joined #buildstream18:57
*** nimish has quit IRC19:32
*** nimish has joined #buildstream19:32
*** xjuan has quit IRC20:18
*** tristan has quit IRC20:35
*** nimish has quit IRC21:11
*** nimish has joined #buildstream21:11
*** xjuan has joined #buildstream21:28
*** xjuan has quit IRC21:51
*** xjuan has joined #buildstream22:08
*** mohan43u has quit IRC22:12
*** mohan43u has joined #buildstream22:16
*** alatiera has quit IRC22:18
*** kapil___ has joined #buildstream23:35
*** cs-shadow has quit IRC23:36

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