IRC logs for #buildstream for Monday, 2020-01-13

*** delli3 has joined #buildstream04:40
*** delli3 has joined #buildstream04:45
*** delli3 has quit IRC04:58
*** tristan has joined #buildstream05:09
*** delli3 has joined #buildstream05:09
*** traveltissues has joined #buildstream07:43
*** traveltissues has quit IRC07:46
*** traveltissues has joined #buildstream07:46
*** traveltissues has quit IRC08:03
*** traveltissues has joined #buildstream08:03
*** traveltissues has quit IRC08:06
*** traveltissues has joined #buildstream08:07
*** traveltissues has quit IRC08:22
*** traveltissues has joined #buildstream08:22
*** bochecha has joined #buildstream09:21
*** tpollard has joined #buildstream09:25
*** rdale has joined #buildstream09:33
*** tme5 has joined #buildstream09:39
*** tiagogomes has joined #buildstream09:41
coldtomanyone got time to give https://gitlab.com/BuildStream/bst-plugins-experimental/merge_requests/61 a quick review?09:52
benschubertcoldtom: done :)09:55
benschubertout of interest: did you manage to build Bazel with Bazel? :D09:56
benschubertin buildstream09:56
coldtomtyvm benschubert, and yup, here's an element to do it: https://gitlab.com/celduin/burn-sdk/blob/master/elements/bazel.bst09:57
benschuberthow did you get around Bazel trying to download a Java SDK ?09:58
coldtomthat's what the bazel_source element does, using --experimental_repository_resolved_file and `bazel cquery` you can just about manage to get bazel to give you a list of the sources it's going to download10:00
benschubertah nice :)10:00
coldtoms/element/plugin/10:00
benschubertI'll need to take a deeper look into it10:01
benschubertcoldtom: also, I'm planning in publishing a new version of bst-plugins-experimental now, is that fine? That will allow the tests on buildstream around sources to be finally merged10:01
coldtomunfortunately due to the extensibility of bazel, avoiding bazel downloading things is hard10:02
coldtomfine with me, finally there'll be a tag compatible with master \o/10:02
juergbiit will be incompatible again as soon as !1739 is merged, though, afaict10:03
gitlab-br-botMR !1739: WIP: Optimize consistency and state handling https://gitlab.com/BuildStream/buildstream/merge_requests/173910:03
coldtomheh, compatible with the latest dev snapshot then :P10:04
benschubertcorrect10:04
benschubertI want to tests first hand the process of updating with the two repos having breaking changes :D10:05
benschubertjuergbi: btw thanks for the thorough review, I'll go through it10:05
juergbiyw10:05
*** jonathanmaw has joined #buildstream10:12
benschubertAlso added the 'real world' benchmark on !1739: https://gitlab.com/BuildStream/buildstream/merge_requests/1739#using-a-real-internal-project :D10:13
juergbinice speedup :)10:16
tpollardnice10:16
benschubertwent from driving me crazy to acceptable :)10:17
benschubertbut gosh how tangled was all of this10:17
coldtomwow! that's an amazing speedup!10:18
benschubertbst-plugins-experimental 0.13.0 is out! https://gitlab.com/BuildStream/bst-plugins-experimental/-/tags/0.13.010:22
tme5impressive optimisation10:22
*** lachlan has joined #buildstream10:34
*** lachlan has quit IRC10:37
*** lachlan has joined #buildstream10:38
*** lachlan has quit IRC10:41
gitlab-br-botjuergbi opened MR !1793 (juerg/batch-command-logging->master: _sandboxreapi.py: Improve batch command logging) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/179310:44
gitlab-br-botBenjaminSchubert approved MR !1793 (juerg/batch-command-logging->master: _sandboxreapi.py: Improve batch command logging) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/179310:46
juergbita10:48
*** lachlan has joined #buildstream11:02
gitlab-br-botBenjaminSchubert opened (was WIP) MR !1757 (bschubert/standardized-tests->master: introduce cross-repo standardized source tests) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/175711:05
benschubertcoldtom: ^ if you want to give a last review on this. After multiple months we're finally here :)11:06
*** lachlan has quit IRC11:07
coldtombenschubert: sure, I'll take a look11:07
*** lachlan has joined #buildstream11:13
*** santi has joined #buildstream11:16
*** santi has quit IRC11:17
gitlab-br-botcoldtom approved MR !1757 (bschubert/standardized-tests->master: introduce cross-repo standardized source tests) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/175711:18
*** lachlan has quit IRC11:19
*** santi has joined #buildstream11:19
*** lachlan has joined #buildstream11:22
*** lachlan has quit IRC11:25
*** lachlan has joined #buildstream11:27
*** lachlan has quit IRC11:31
*** santi has quit IRC11:40
*** santi has joined #buildstream11:43
*** lachlan has joined #buildstream11:51
*** traveltissues has quit IRC11:57
gitlab-br-botjuergbi merged MR !1793 (juerg/batch-command-logging->master: _sandboxreapi.py: Improve batch command logging) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/179312:00
*** traveltissues has joined #buildstream12:09
*** traveltissues has quit IRC12:16
juergbitlater[m]: thanks for the update of !1769. I think this has fixed the remaining edge cases12:16
gitlab-br-botMR !1769: WIP: storage/_casbaseddirectory.py: Add `diff` methods https://gitlab.com/BuildStream/buildstream/merge_requests/176912:16
tlater[m]Awesome, thanks juergbi :)12:16
*** traveltissues has joined #buildstream12:17
tlater[m]I've also changed the target as traveltissues requested12:17
juergbiok12:18
*** lachlan has quit IRC12:20
*** lachlan has joined #buildstream12:30
*** lachlan has quit IRC12:33
*** santi has quit IRC12:40
*** santi has joined #buildstream12:40
*** santi has quit IRC12:42
*** santi has joined #buildstream12:42
*** santi has quit IRC12:44
*** santi has joined #buildstream12:45
tme5if you specify to a git source that you want to track the latest tag of the branches, what do you think the behaviour should be if a branch doesn't have a tag?12:48
tpollardconfigurable warning, imo12:49
tme5currently git_tag will choose the latest commit instead12:49
tpollardlike the commit not existing in a specific branch12:49
tme5yeah, i'm thinking issue a warning12:49
*** lachlan has joined #buildstream12:53
*** lachlan has quit IRC12:59
*** rdale has quit IRC13:01
*** xjuan has joined #buildstream14:09
*** lachlan has joined #buildstream14:16
*** lachlan has quit IRC14:19
*** traveltissues has quit IRC14:45
*** traveltissues has joined #buildstream14:46
*** traveltissues has quit IRC14:50
tpollardhmm, might be issues with our digital ocean runners15:47
tpollarddashboard is only claiming 1 runner active15:48
tpollardyep it's journactl is full of warnings15:49
tpollardI'll restart the bastion servive15:49
tpollard*service15:49
tpollardshould be back now15:52
*** xjuan has quit IRC15:56
*** lachlan has joined #buildstream16:22
*** lachlan has quit IRC16:42
*** bochecha has quit IRC16:47
*** phildawson has joined #buildstream16:49
*** lachlan has joined #buildstream16:53
*** santi has quit IRC16:54
*** lachlan has quit IRC17:00
*** phildawson-ct has joined #buildstream17:00
*** phildawson has quit IRC17:01
juergbitpollard: as I understand it the remaining blocker of !1745 is a final decision on whether it should be enabled by default? and possibly benchmark on more systems (maybe benschubert)?17:01
gitlab-br-botMR !1745: Make message & status rendering be tick driven by default https://gitlab.com/BuildStream/buildstream/merge_requests/174517:01
benschubertI have a good usecase for this. :D will take me a bit of time though17:03
benschubertwill try to do that tomorrow17:03
juergbigreat17:04
benschubertjuergbi: I believe I tackled everything you mentionned in !173917:06
gitlab-br-botMR !1739: WIP: Optimize consistency and state handling https://gitlab.com/BuildStream/buildstream/merge_requests/173917:06
*** santi has joined #buildstream17:06
benschubertI'll look at the plugins as soon as my MR around testing them is in :)17:06
*** phildawson has joined #buildstream17:06
tpollardjuergbi: I think just a final decision if there's no comments on the actual code17:06
*** phildawson-ct has quit IRC17:07
*** phildawson-ct has joined #buildstream17:08
*** phildawson has quit IRC17:09
*** phildawson has joined #buildstream17:20
*** phildawson-ct has quit IRC17:20
*** lachlan has joined #buildstream17:29
*** xjuan has joined #buildstream17:31
*** xjuan has quit IRC17:34
*** lachlan has quit IRC17:34
*** tme5 has quit IRC17:36
juergbibenschubert: we can always make exceptions for the odd test case to avoid spending too much effort on it. however, would certainly be good to at least be close, so thanks17:37
benschubertjuergbi: I might have missed it, but I thought I didn't have any odd tests anymore? did I completely miss one part?17:37
juergbibenschubert: no, I just meant that if we can't get each commit in the branch passing the whole test suite because of one or a few test cases, we can consider an exception17:38
*** xjuan has joined #buildstream17:39
benschubertoh fair enough, I'll give it a bit of time and if I can't I'll give up :)17:39
benschubertbut should be fine17:39
juergbita17:39
benschubertI also still need to send it through the ML since it's a breaking change17:39
gitlab-br-botBenjaminSchubert opened issue #1249 (On build error, can't enter twice in the shell) on buildstream https://gitlab.com/BuildStream/buildstream/issues/124917:42
*** phildawson has quit IRC17:44
*** adds68 has quit IRC17:46
*** lachlan has joined #buildstream17:47
*** lachlan has quit IRC18:03
*** lachlan has joined #buildstream18:06
*** lachlan has quit IRC18:09
*** jonathanmaw has quit IRC18:10
*** lachlan has joined #buildstream18:17
*** tiagogomes has quit IRC18:23
*** phildawson has joined #buildstream18:23
gitlab-br-botBenjaminSchubert merged MR !1757 (bschubert/standardized-tests->master: introduce cross-repo standardized source tests) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/175718:25
*** phildawson has quit IRC18:26
*** lachlan has quit IRC18:27
douglaswinshipIf i'm writing a plugin, is there any way I can get it to access a particular file, in a particular location in the project repository?18:28
douglaswinshipI'm basically looking for a variable that resolves to the top directory of the project.18:29
douglaswinshipOr failing that, to the path of the element.18:29
douglaswinshipI can't use an absolute path, because the repo will be copied to different places by different users. But i don't want to use a path relative to the current folder, because users might use the "bst build" command from different starting folders18:31
*** phildawson has joined #buildstream18:34
juergbidouglaswinship: you can do this from source plugins with self.get_project_directory()18:36
juergbiI don't think public API allows this in element plugins18:36
*** lachlan has joined #buildstream18:51
*** lachlan has quit IRC18:55
*** lachlan has joined #buildstream18:58
cs-shadowdouglaswinship: may i ask the use case for this? (perhaps there's another of achieving that you are trying to do without requiring said access)19:03
*** phildawson has quit IRC19:07
*** phildawson has joined #buildstream19:07
*** lachlan has quit IRC19:32
*** lachlan has joined #buildstream20:09
*** lachlan has quit IRC20:12
*** phildawson has quit IRC20:13
*** lachlan has joined #buildstream20:17
*** lachlan has quit IRC20:20
*** lachlan has joined #buildstream20:52
*** phildawson has joined #buildstream20:52
*** lachlan has quit IRC21:01
*** lachlan has joined #buildstream21:01
*** santi has quit IRC21:17
*** lachlan has quit IRC21:19
*** phildawson has quit IRC23:03

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