IRC logs for #buildstream for Tuesday, 2020-03-24

*** jward has quit IRC01:02
*** ikerperez has quit IRC01:02
*** ikerperez has joined #buildstream01:03
*** robjh has quit IRC01:03
*** WSalmon has quit IRC01:03
*** paulsherwood has quit IRC01:04
*** valentind has quit IRC01:04
*** douglaswinship has quit IRC01:05
*** douglaswinship has joined #buildstream01:05
*** robjh has joined #buildstream01:05
*** jward has joined #buildstream01:05
*** valentind has joined #buildstream01:06
*** traveltissues has joined #buildstream01:07
*** WSalmon has joined #buildstream01:07
*** traveltissues has quit IRC01:07
*** paulsherwood has joined #buildstream01:09
*** traveltissues has joined #buildstream01:22
*** traveltissues has quit IRC01:25
*** hasebastian has joined #buildstream03:42
*** narispo has quit IRC03:54
*** narispo has joined #buildstream03:54
*** hasebastian has quit IRC06:40
*** traveltissues has joined #buildstream08:27
*** mohan43u has quit IRC08:36
*** benschubert has joined #buildstream08:47
*** mohan43u has joined #buildstream08:50
gitlab-br-botBenjaminSchubert approved MR !1833 (abderrahim/etag->master: _downloadablefilesource.py: don't download the file if etag matches) on buildstream https://gitlab.com/BuildStream/buildstream/-/merge_requests/183309:08
benschubertabderrahim[m]: do you have merge permissions on the project? If not, can I merge ^ ?09:08
*** tpollard has joined #buildstream09:08
*** rdale has joined #buildstream09:26
*** santi has joined #buildstream09:35
*** phildawson has joined #buildstream09:38
abderrahim[m]benschubert: Thanks. I do09:59
gitlab-br-botabderrahimk merged MR !1833 (abderrahim/etag->master: _downloadablefilesource.py: don't download the file if etag matches) on buildstream https://gitlab.com/BuildStream/buildstream/-/merge_requests/183309:59
benschubertawesome09:59
benschubertAm I the only one for which the artifactcache tests take ages? It roughly 1 minute per test...10:01
juergbiI'm running tests in parallel10:05
juergbisome buildtree tests are the slowest for me10:05
benschubertinteresting, I'll dig a bit deeper :)10:06
gitlab-br-botabderrahimk opened MR !1838 (abderrahim/etag-1->bst-1: _downloadablefilesource.py: don't download the file if etag matches) on buildstream https://gitlab.com/BuildStream/buildstream/-/merge_requests/183810:08
*** lachlan has joined #buildstream10:15
*** lachlan has quit IRC10:40
*** lachlan has joined #buildstream10:45
*** lachlan has quit IRC11:00
*** lachlan has joined #buildstream11:11
*** lachlan has quit IRC11:16
*** jib has joined #buildstream11:22
*** jib has left #buildstream11:24
*** lachlan has joined #buildstream11:25
*** narispo has quit IRC11:56
*** narispo has joined #buildstream11:56
*** lachlan has quit IRC12:43
gitlab-br-botBenjaminSchubert opened (was WIP) MR !1832 (bschubert/stricter-cas-soc->master: Don't create Buildstream-related directories in the cas - owned directory) on buildstream https://gitlab.com/BuildStream/buildstream/-/merge_requests/183212:44
benschubertjuergbi: ^ finally fixed all the tests. How does that look?12:44
*** cphang has joined #buildstream12:50
*** traveltissues has quit IRC12:52
*** traveltissues_ has joined #buildstream12:52
*** cphang has left #buildstream12:54
*** cphang has joined #buildstream12:54
*** cphang has left #buildstream12:54
*** cphang has joined #buildstream12:54
*** lachlan has joined #buildstream12:58
*** lachlan has quit IRC13:11
*** traveltissues_ has quit IRC14:06
gitlab-br-botjuergbi approved MR !1832 (bschubert/stricter-cas-soc->master: Don't create Buildstream-related directories in the cas - owned directory) on buildstream https://gitlab.com/BuildStream/buildstream/-/merge_requests/183214:25
benschubertThanks a lot :)14:26
*** lachlan has joined #buildstream14:30
*** traveltissues has joined #buildstream15:01
*** lachlan has quit IRC15:19
*** lachlan has joined #buildstream15:34
gitlab-br-botBenjaminSchubert merged MR !1832 (bschubert/stricter-cas-soc->master: Don't create Buildstream-related directories in the cas - owned directory) on buildstream https://gitlab.com/BuildStream/buildstream/-/merge_requests/183215:36
*** lachlan has quit IRC16:16
cphangHi! Many apologies if this has been answered already. For getting towards a bst 2.0 release, plugin API cleanup was one of the items as a todo. How close are we to having a stable plugin API in master? Are there any outstanding issues I could have a look at that are towards this cleanup effort? :)16:38
tpollardI'm not aware of a catchall issue16:44
coldtomaiui adopting the Remote Asset API will require plugin API changes, at least for the features proposed on the mailing list16:45
coldtombut that's the only piece of work on my radar which requires plugin API changes16:45
juergbicphang: one related WIP item is to drop support for element plugins that don't use the virtual directory API (Directory class)16:48
juergbias they are incompatible with remote execution as well as buildbox-run sandboxing16:48
juergbii.e., update the remaining plugins, and supplement the virtual directory API as necessary. I've already started with this16:49
*** lachlan has joined #buildstream16:49
juergbirelated to that, review the whole Directory API whether it's good enough as stable API for 2.016:51
juergbiI've also been thinking about whether it could make sense to drop the separation between prepare() and assemble() in the core API. it might simplify a few things in light of the need for batch operations with remote execution / buildbox-run16:52
juergbiand some general review of the batch-based sandboxing API in light of the move to buildbox-run + remote-execution16:53
cphangAha tvm juergbi, the WIP item is https://gitlab.com/BuildStream/buildstream/-/issues/1262 ?16:58
juergbiyes16:58
cphangCool ta, looks like there are 7 plugins at latest count that don't use the virtual directory API https://paste.gnome.org/p0ewexfqg17:02
cphangjuergbi would you appreciate any MRs that helped with that port from bst-plugins-experimental? Or is it something you've got already in hand? :)17:03
juergbicphang: I'd appreciate help there. just need to make sure we don't do duplicate work. my messy WIP branch is https://gitlab.com/BuildStream/bst-plugins-experimental/-/commits/juerg/vdirectory17:10
juergbiI'd definitely be happy anyone working on plugins I haven't touched yet at all17:11
juergbior when there are none left, possibly taking over WIP ones17:11
juergbiI'm currently finishing up another task related to buildbox sandboxing before coming back to this17:12
juergbithere are also a few plugins in the freedesktop-sdk repo that require work17:12
*** lachlan has quit IRC17:13
juergbifor some plugins there might not be sufficient test coverage17:13
cphangack makes sense. How about if I open up an issue in bst-plugins-experimental to track this. I can have a crack at the oci and dpkg_deploy plugins. Don't think from that branch you've started work on that?17:15
juergbisounds good, thanks17:15
juergbialso have to tag a new buildstream snapshot such that bst-plugins-experimental master can use the new directory API17:16
juergbiwanted to wait until I'm relatively certain that all the needs are covered17:17
cphangcool17:17
*** lachlan has joined #buildstream17:17
*** lachlan has quit IRC17:24
*** lachlan has joined #buildstream17:29
jjardonjuergbi: to make easier to track them; would it be ok to use a milestone or a label to mark issues we know are bst2 blockers?17:51
abderrahim[m]cphang: there is also this issue I stumbled upon (filed as #1269). The fixme here needs to be addressed https://gitlab.com/BuildStream/buildstream/-/blob/master/src/buildstream/source.py#L100917:52
gitlab-br-botIssue #1269: documentation for Source.set_ref is incomplete and incorrect https://gitlab.com/BuildStream/buildstream/-/issues/126917:52
abderrahim[m](and generally documentation needs to be updated)17:52
abderrahim[m]It's currently written for buildstream developers, a 3rd party developer can't figure out how to write a plugin from reading the documentation alone17:53
*** chipb has joined #buildstream17:54
benschubertabderrahim[m]: oh that's me who put that on. I should try to get that done :(17:56
benschubert(the TODO I mean)17:56
benschubertjjardon: I believe the milestone/tag is a great idea yep!17:56
juergbiagreed, we should use a milestone17:57
jjardonok! https://gitlab.com/BuildStream/buildstream/-/milestones/3 created17:58
jjardonexciting times17:58
benschubertawesome thanks :)17:58
cphangooh nice jjardon :D18:01
*** tpollard has quit IRC18:06
*** lachlan has quit IRC18:20
*** traveltissues has quit IRC18:23
*** lachlan has joined #buildstream18:33
*** santi has quit IRC18:39
cphangthanks abderrahim[m]  Would it be possible to add https://gitlab.com/BuildStream/buildstream/-/issues/1269 to the milestone for 2.0? I would add but don't have the necessary permissions18:43
abderrahim[m]done18:44
cphangthanks :)18:45
*** lachlan has quit IRC19:09
*** benschubert has quit IRC20:22

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