| *** tristan has joined #buildstream | 05:10 | |
| *** ChanServ sets mode: +o tristan | 05:10 | |
| tristan | jjardon, nanonyme... read-only artifact caches are intended to be supported in project.conf as recommended defaults (can be overridden by buildstream.conf), this afaict doesn't change | 05:12 |
|---|---|---|
| tristan | also, as far as I understand, the "storage" service in the "remote-execution" config block, is pretty much the same as an artifact cache, but is not guaranteed to support the indexing of artifacts by artifact name (which is an additional requirement for an artifact cache) | 05:13 |
| nanonyme | tristan, so would that storage service in remote execution config block still work in project settings after these changes? | 07:17 |
| juergbi | tristan: my 'remote-cache' branch drops the `pull-artifact-files` option | 07:22 |
| juergbi | https://github.com/apache/buildstream/pull/1430/commits/613ae896168f473ab7c381cfaf77532c26d72ea5 | 07:23 |
| tristan | juergbi, I see, do you expect that to drop soon ? | 08:33 |
| tristan | nanonyme, no, the whole "remote-execution" block would be removed from project settings | 08:34 |
| tristan | nanonyme, when using remote execution, you basically always need write access to the storage | 08:35 |
| juergbi | tristan: need to get #1429 in as prerequisite. I should have an update on that today. however, I think we could pull forward the drop of that option. as per commit message, status quo is problematic, so I consider it acceptable to drop it now even if the replacement takes a bit longer to be merged | 08:37 |
| tristan | juergbi, in any case, it seems to not cause much of a fuss in my branch, I was more curious as to why it was there and if it should change - I'll just leave it as such, finish up the remaining proposal changes and we can see what to land first | 08:37 |
| juergbi | ok. I hope I can also look at all the pending things today but first need to update #1429 on top of benschubert's branch | 08:38 |
| tristan | sure, I've got three, some of it brewing for some time | 08:38 |
| tristan | it would be good to land https://github.com/apache/buildstream/pull/1433 first and then look at https://github.com/apache/buildstream/pull/1442 | 08:39 |
| *** santi has joined #buildstream | 09:33 | |
| *** tristan has quit IRC | 11:49 | |
| *** tristan has joined #buildstream | 12:04 | |
| *** ChanServ sets mode: +o tristan | 12:04 | |
| *** tristan has quit IRC | 12:31 | |
| *** tristan has joined #buildstream | 12:45 | |
| *** ChanServ sets mode: +o tristan | 12:45 | |
| *** santi has quit IRC | 18:50 | |
| *** toscalix has joined #buildstream | 20:22 | |
| *** toscalix has quit IRC | 20:42 | |
| *** toscalix has joined #buildstream | 20:44 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!