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

*** cs-shadow has quit IRC02:06
*** tristan has quit IRC05:57
*** tristan has joined #buildstream06:27
*** tristan has quit IRC07:12
*** tristan has joined #buildstream07:17
*** bochecha has joined #buildstream07:48
juergbibenschubert: I have buildbox MRs to fix the excessive memory usage on upload. Small changes, so hopefully I can get those in today and then tag/update buildbox for BuildStream08:04
juergbior at least one instance of excessive memory usage. there might be similar issues in other parts08:05
benschubertjuergbi: I'm happy to test ;)08:18
benschubertturns out building the whole llvm  suite with -flto=full is a good way of stress testing memory usage on that ;)08:18
Kinnisonbenschubert: !1605 could do with some attention from you if you have time -- I think, reading the commits, the requisite stuff is merged into casd08:29
gitlab-br-botMR !1605: WIP: Report when Buildbox-casd is not alive at the end of a run https://gitlab.com/BuildStream/buildstream/merge_requests/160508:29
benschubertKinnison: we are still waiting on a last MR from juerb before updating all our testing infra and the docs08:29
benschubert(cf just before :) )08:30
Kinnisonaah right08:30
Kinnisonok08:30
benschubertbut it's still in my head :) not forgotten08:30
KinnisonI should get tea, it's 2+ hours into the working day and I've still not had any08:30
*** tiagogomes has joined #buildstream08:37
*** tristan has quit IRC09:01
*** tpollard has joined #buildstream09:02
*** jonathanmaw has joined #buildstream09:07
*** tristan has joined #buildstream09:12
*** traveltissues has joined #buildstream09:14
Kinnisontraveltissues: The performance change noted in the benchmarking results on !1563 is very confusing to me.  I don't see how your changes could have negatively affected performance on a benchmark which doesn't use workspaces09:41
gitlab-br-botMR !1563: workspaces via sourcecache 1 https://gitlab.com/BuildStream/buildstream/merge_requests/156309:41
*** phil has joined #buildstream09:46
*** phildawson_ has quit IRC09:47
coldtombenschubert: if you have time, could you take a look at !1607 please?09:50
gitlab-br-botMR !1607: WIP: Run standardised source tests from external plugins https://gitlab.com/BuildStream/buildstream/merge_requests/160709:50
*** phil has quit IRC09:59
*** phil has joined #buildstream09:59
benschubertcoldtom: sure I'll look start of this afternoon. Is this based on my ML thread?10:06
coldtombenschubert: thanks, it is indeed based on your ML thread10:06
benschubertthanks a lot for this work :)10:07
gitlab-br-bottraveltissues opened issue #1140 (Follow-up from "workspaces via sourcecache 1" - workspace.init_workspace should raise a SourceError) on buildstream https://gitlab.com/BuildStream/buildstream/issues/114010:40
*** phoenix has joined #buildstream10:41
traveltissuesKinnison, i have some cleanup to do and i'll rebenchmark, i had better results on py3610:58
traveltissuesbut i'll be looking into that10:58
coldtomjust wanted to say, `bst artifact log` is a wonderful addition and thank you for adding it11:04
tpollardit is good!11:10
benschubert^^ completely agree :D11:26
Kinnisontraveltissues: Mmm, I just find it odd that it's not almost exactly the same11:39
traveltissuesi have a theory Kinnison11:40
*** cs-shadow has joined #buildstream13:07
*** tristan has quit IRC13:10
coldtomwould it be possible to get review on the several plugin tests I've added to bst-plugins-experimental please?13:30
coldtomon a related note, I think we need to rethink the base of the test project - the alpine image keeps needing additional dependencies adding, and for the qmake plugin I think I'd need to add gcc + glibc13:32
coldtomwould there be much opposition to maintaining a bst project to act as the base image for the plugin tests?13:32
tpollardWe've gone down that road once13:36
tpollardhttps://gitlab.com/BuildStream/buildstream/merge_requests/130013:38
benschubertmeh, freedesktop and a specialized docker image is quite different. One would be lightweight and easy to update, the other one would require caching/rebuilding a good part of a stack we don't really care about. I prefer the path of having a tailored image as a base13:47
coldtoma tailored image would *probably* be necessary anyway13:50
coldtomi was thinking of using the alpine image as a base and chopping and changing things13:50
coldtombut we'd still need to build more stuff on top of freedesktop-sdk anyway, especially if more weird and wonderful plugins appeared13:50
coldtomalso we don't *want* frequent updates in a test image13:53
benschubertcoldtom: about your prs on bst-plugins-experimental any reason they are not working with buildbox?14:51
coldtomah oops, i just copied the cmake ones and missed that mark, I'm not sure if they work with buildbox or not14:51
benschubertwould you mind checking? :)14:52
coldtomi'll check whether the cmake one is still broken while i'm at it :)14:53
benschubertawesome15:02
coldtomi think this was only a problem in buildstream CI where the plugins were tested in the `test-buildbox` tests, judging by the comment remarking things only failed in CI15:05
*** phil has quit IRC15:09
*** phildawson has joined #buildstream15:09
*** phoenix has quit IRC15:44
gitlab-br-botjuergbi opened (was WIP) MR !1601 (juerg/casd->master: Do not directly communicate with CAS server) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/160115:49
juergbican I please get an approval for https://gitlab.com/BuildStream/website/merge_requests/127 ?16:14
juergbialso, will marge !1601 soon unless there are any last minute objections16:15
gitlab-br-botBenjaminSchubert approved MR !1601 (juerg/casd->master: Do not directly communicate with CAS server) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/160116:33
gitlab-br-botBenjaminSchubert opened (was WIP) MR !1605 (bschubert/ensure-buildbox-alive->master: Report when Buildbox-casd is not alive at the end of a run) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/160516:33
benschubertcoldtom: want me to merge and rebase all your PRs to bst-plugins-experimental?16:40
gitlab-br-botmarge-bot123 merged MR !1601 (juerg/casd->master: Do not directly communicate with CAS server) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/160116:44
*** lachlan has joined #buildstream16:53
*** lachlan has quit IRC17:04
*** traveltissues has quit IRC17:27
*** phoenix has joined #buildstream18:27
*** hergertme has quit IRC19:07
*** hergertme has joined #buildstream19:11
*** hergertme has joined #buildstream19:12
*** bochecha has quit IRC22:41
*** phoenix has quit IRC23:22
*** cs-shadow has quit IRC23:45

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