*** cphang has quit IRC | 04:16 | |
*** Frazer has quit IRC | 04:17 | |
*** benschubert has joined #buildstream | 07:21 | |
*** tristan has quit IRC | 07:48 | |
*** jonathanmaw has joined #buildstream | 07:57 | |
*** santi has joined #buildstream | 08:45 | |
*** tristan has joined #buildstream | 08:53 | |
*** ChanServ sets mode: +o tristan | 08:53 | |
*** tristan has quit IRC | 10:03 | |
juergbi | WSalmon: have you read my suggested plan regarding ReferenceStorage? I'm a bit confused by your comment on !1978 | 10:04 |
---|---|---|
WSalmon | confused? | 10:05 |
WSalmon | i have skim read it, i need to go back and have another read at some point | 10:06 |
juergbi | WSalmon: you wrote "It seems the consensus on the ML is to remove this code.", however, my latest suggestion is to keep it in master for now | 10:06 |
juergbi | in today's mail | 10:06 |
WSalmon | oh, i have not noticed todays email | 10:06 |
WSalmon | ok i have removed my comment, interested to see what sander says to the ML | 10:08 |
WSalmon | i know deletion etc was only aviable as a workaround but it is a really important one, this MR will make a proper "fix" even more pressing https://gitlab.com/BuildStream/buildstream/-/merge_requests/1978#note_383636760 | 10:13 |
juergbi | true | 10:15 |
juergbi | it should still be possible to work around it from the client side, though. local bst artifact delete, then bst build without pulling and then push that | 10:17 |
WSalmon | i presume the server its self dose not know which bits of its cas are artifacts definitions and which are "just" objects so even adding a "ls" grpc call would result in a costly operation? | 10:17 |
juergbi | listing pushed URIs would theoretically be possible but it's not in any protocol | 10:18 |
juergbi | and if a server is used by many projects for a while, there might be a huge number of URIs | 10:18 |
juergbi | I'd rather prioritize support for remote delete and bst artifact show improvements as needed | 10:18 |
juergbi | remote delete is conceptually fairly easy, I think it's mainly a CLI syntax question | 10:19 |
WSalmon | oh yes thats were we should put our efforts, i was just curious about the back end | 10:20 |
*** santi has quit IRC | 15:51 | |
*** santi has joined #buildstream | 15:52 | |
*** jonathanmaw has quit IRC | 16:14 | |
*** traveltissues has quit IRC | 18:09 | |
*** WSalmon has quit IRC | 18:09 | |
*** valentind has quit IRC | 18:10 | |
*** bethw has quit IRC | 18:10 | |
*** robjh has quit IRC | 18:10 | |
*** bethw has joined #buildstream | 18:11 | |
*** robjh has joined #buildstream | 18:12 | |
*** valentind has joined #buildstream | 18:14 | |
*** mohan43u has joined #buildstream | 18:18 | |
*** bethw has quit IRC | 18:19 | |
*** bethw has joined #buildstream | 18:19 | |
*** bwh has quit IRC | 18:48 | |
*** douglaswinship has quit IRC | 18:48 | |
*** santi has quit IRC | 18:52 | |
*** santi has joined #buildstream | 18:54 | |
*** santi has quit IRC | 18:54 | |
*** bwh has joined #buildstream | 19:00 | |
bwh | I came up with a reproducer for my git source issue, and oepened: https://gitlab.com/BuildStream/buildstream/-/issues/1372 | 19:04 |
*** benschubert has quit IRC | 19:17 | |
juergbi | bwh: thanks for the reproducer. will give it a try on master | 20:09 |
*** mohan43u has quit IRC | 20:13 | |
*** mohan43u has joined #buildstream | 20:17 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!