IRC logs for #buildstream for Tuesday, 2019-12-17

*** narispo has quit IRC05:27
*** narispo has joined #buildstream05:27
*** saline_retry has joined #buildstream06:20
*** saline_retry has quit IRC07:34
*** narispo has quit IRC07:57
*** narispo has joined #buildstream07:57
*** narispo has quit IRC08:20
*** narispo has joined #buildstream08:21
*** narispo has quit IRC08:32
*** narispo has joined #buildstream08:33
*** narispo has quit IRC08:37
*** rdale has joined #buildstream09:35
*** santi has joined #buildstream09:37
*** tme5 has joined #buildstream10:02
*** phildawson_ has joined #buildstream10:11
tme5so, i think i mentioned before, that on my to-do list is the plan to merge the git_tag source into the git source. I just wanted to ask about the original reason they were separate? I assume it no longer applies as git_tag is in plugins-experimental now10:40
*** lachlan has joined #buildstream10:41
coldtomgit_tag has always lived in bst-external/bst-plugins-experimental, the initial reason was that there was disagreement on what `bst track` was supposed to do, as git_tag changes this behaviour from other sources10:42
coldtomfor sources "blessed" by buildstream, `bst track` resolves a source to a specific ref, but with git_tag tracking becomes a method of updating elements10:46
tme5interesting10:47
tme5bst track --help says "Consults the specified tracking branches for new versions available to build and updates the project with any newly available references."10:47
tme5admittedly this help text is rather poor as "branches" means nothing in many cases10:48
*** lachlan has quit IRC10:52
*** traveltissues has joined #buildstream10:53
*** lachlan has joined #buildstream11:01
*** lachlan has quit IRC11:05
coldtomi think the problem is that the plugin now becomes opinionated on what it's tracking11:08
coldtomi've no problem with the two merging though, i'm of the opinion this should always have been in the `git` plugin11:08
*** lachlan has joined #buildstream11:12
*** lachlan has quit IRC11:24
*** lachlan has joined #buildstream11:27
gitlab-br-botBenjaminSchubert opened issue #1242 (`pip` element doesn't make it easy to add configuration) on buildstream https://gitlab.com/BuildStream/buildstream/issues/124211:53
*** lachlan has quit IRC11:53
*** narispo has joined #buildstream12:15
tme5could anyone explain in a bit more detail when i should use fail_temporarily on Plugin.call ?12:25
tme5"Whether any exceptions should be raised as temporary" - is that a python thing?12:25
juergbitme5: whether it makes sense to retry the action12:36
juergbie.g., connection failure could be a temporary network or server failure, so it should be retried12:37
juergbiiirc, SMTP has the same concept12:37
tme5ah ok, thanks12:44
*** santi has quit IRC13:38
benschubertDid people still encounter the "userconfig.yaml: Severely malformed YAML: 'NoneType' object is not callable" problem we used to get?13:53
*** lachlan has joined #buildstream13:55
*** lachlan has quit IRC14:06
*** santi has joined #buildstream14:07
*** lachlan has joined #buildstream14:09
*** lachlan has quit IRC14:14
*** lachlan has joined #buildstream14:19
*** phildawson_ has quit IRC14:36
*** tpollard has quit IRC14:40
*** tpollard has joined #buildstream14:41
*** lachlan has quit IRC14:46
*** lachlan has joined #buildstream15:12
*** lachlan has quit IRC15:19
*** lachlan has joined #buildstream15:19
*** lachlan has quit IRC15:25
gitlab-br-botjuergbi closed issue #1237 (Add CI job for buildbox-run-userchroot) on buildstream https://gitlab.com/BuildStream/buildstream/issues/123716:17
gitlab-br-botjuergbi merged MR !1751 (juerg/buildbox-run-userchroot->master: Add CI job to test buildbox-run-userchroot) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/175116:17
*** lachlan has joined #buildstream16:18
KinnisonIs that if you have an empty userconfig.yaml ?16:18
coldtombenschubert: I think i got that, but from a dodgy python3.8 venv, so take it with a pinch of salt16:19
tme5the callback SourceFetcher.fetch seems to take an optional keyword argument "alias_override" but i can't find an instance where it is call with that given16:25
benschubertKinnison: no, it's when ruamel.yaml.clib is not imported correctly16:32
tpollardI've not had it happen for a long time16:32
benschubertI found the reason why: if the clib extension fails to compile it's a silent error, and the CParser is set to None instead of a real python object16:33
benschubertI wonder whether we should gracefully fallback or not16:33
Kinnisonbenschubert: I think we should detect that during preflight of buildstream, report an error, and stop16:34
Kinnisonthe python parser is much slower16:34
benschubertthat would be even before preflight16:35
benschubertI wonder whether a warning would not be more user friendly16:35
KinnisonWarning won't help unless you enable fallback, but users ignore warnings16:36
Kinnisonand then complain things are slow16:36
benschubertfair16:36
* Kinnison <- cynical old man16:36
tlater[m]Kinnison: The syntax is `Kinnison: cynical old man`16:39
Kinnison"Kinnison is drawn from cynical old man in the IRC monad"16:41
*** santi has quit IRC16:41
*** santi has joined #buildstream16:46
*** lachlan has quit IRC16:51
WSalmonthe logs endup in the artifact right? so if i build and it just pulls the thing, is there a quick way to see the log?16:58
WSalmontpollard, ^16:58
WSalmon?16:58
benschubertbst artifcat log my_artifact16:58
tpollard^16:58
WSalmonin bst1,4?16:58
tpollardah, nope16:59
tpollardnot sure of the semantics for 1.417:00
coldtomi'm not sure if you even get the build logs with the artifact for 1.4?17:00
KinnisonI don't recall if logs are pushed as part of the artifact in 1.x17:00
WSalmonoh dear17:00
juergbiI think they are, in the logs directory of the artifact17:00
juergbinot 100% sure either, it's been a while17:01
juergbiyou might need to manually inspect CAS to see them, though17:01
tpollardyep they're cached as a subdir in 1.417:01
gitlab-br-bottraveltissues opened (was WIP) MR !1761 (traveltissues/1216->master: mtime support) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/176117:05
*** traveltissues has quit IRC17:06
*** lachlan has joined #buildstream17:07
*** lachlan has quit IRC17:10
WSalmonthanks juergbi17:13
*** lachlan has joined #buildstream17:22
*** lachlan has quit IRC17:25
*** tme5 has quit IRC17:26
gitlab-br-botjuergbi merged MR !1698 (jjardon/powerpc64le->master: buildstream/_platform/platform.py: Add powerpc64 and powerpc64le) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/169817:31
*** lachlan has joined #buildstream17:34
*** lachlan has quit IRC17:45
*** santi has quit IRC17:45
*** lachlan has joined #buildstream17:55
*** lachlan has quit IRC18:02
*** narispo has quit IRC18:45
*** narispo has joined #buildstream18:45
*** narispo has quit IRC18:46
*** narispo has joined #buildstream18:46
*** narispo has quit IRC19:38
*** narispo has joined #buildstream19:43
*** narispo has quit IRC19:49
*** narispo has joined #buildstream19:59
*** narispo has quit IRC20:09
*** narispo has joined #buildstream20:19
*** rdale has quit IRC21:29
*** narispo has quit IRC23:00
*** juergbi has quit IRC23:19
*** juergbi has joined #buildstream23:24

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