IRC logs for #buildstream for Wednesday, 2018-05-23

gitlab-br-botbuildstream: merge request (juerg/googlecas->master: WIP: Remote Execution CAS-based artifact cache) #337 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/33705:51
*** ernestask has joined #buildstream06:06
gitlab-br-botbuildstream: merge request (juerg/googlecas->master: WIP: Remote Execution CAS-based artifact cache) #337 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/33707:17
*** toscalix has joined #buildstream07:39
*** bochecha_ has joined #buildstream08:49
*** jonathanmaw has joined #buildstream08:50
*** bethw has joined #buildstream09:00
*** tiagogomes has joined #buildstream09:08
*** slaf_ has joined #buildstream09:13
*** slaf_ has joined #buildstream09:13
gitlab-br-botbuildstream: merge request (awacheux/artifact-type-error->master: Fix artifact initialization result tuple) #469 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/46909:13
*** slaf_ has joined #buildstream09:14
*** slaf_ has joined #buildstream09:14
*** slaf_ has joined #buildstream09:14
*** slaf_ has joined #buildstream09:14
*** slaf_ has joined #buildstream09:15
*** slaf_ has joined #buildstream09:15
*** slaf has quit IRC09:15
*** slaf_ is now known as slaf09:15
*** dominic has joined #buildstream09:19
Nexushow did 2 commits called "correction :" make it past review?09:21
jmacAt a guess, they didn't09:23
gitlab-br-botbuildstream: merge request (juerg/googlecas->master: WIP: Remote Execution CAS-based artifact cache) #337 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/33709:31
juergbiNexus: yes, toscalix pushed those directly (gitlab templates, no code). I already reminded him of our commit message conventions09:32
juergbijjardon: CAS MR !337 now includes initial documentation for CAS artifact server setup. If you could give this a try for freedesktop-sdk, that would be great09:34
toscalixNexus: my bad09:38
toscalixapologies09:38
jjardonjuergbi: cool, thanks a lot09:51
*** aday has joined #buildstream09:51
gitlab-br-botbuildstream: merge request (jennis/136-clean-remote-cache->master: WIP: jennis/136 clean remote cache) #421 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/42110:21
tlatertoscalix: Should the bug template perhaps be the default? I imagine most drive-by contributors (and in fact normal contributors) will be submitting bugs rather than tasks.11:17
* tlater certainly got a bit confused trying to submit a bug this morning11:17
tlatertoscalix: I also find it a bit confusing what milestone my bug is for - unless I'm time traveling, bugs should usually be in the current milestone, no?11:26
gitlab-br-botbuildstream: issue #404 ("Context directories may contain double slashes") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/40411:28
tlater^ Writing that bug somehow didn't automatically apply the bug label either11:29
tlaterCould I also perhaps suggest changing the "review" label to "waiting for review"? I feel like that would be more useful, since that way I could request review on minor MRs without making it seem urgent :)11:33
gitlab-br-botbuildstream: merge request (tlater/context-trailing-slash->master: tests/context/context.py: Don't break with trailing slashes) #470 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/47011:34
gitlab-br-botbuildstream: merge request (tlater/context-trailing-slash->master: tests/context/context.py: Don't break with trailing slashes) #470 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/47011:36
tlaterOh, yeah, Nexus - if you run into that issue again, issue #404 tracks it. It affects you because you're using my docker container script, which sets the cache directory. Buildstream tests don't normally check setting the environment variable that governs that, and therefore fail when you use that script.11:42
tlaterIf it annoys you, just apply the patch in !470 and remove it before you create MRs - that's how I've been working around it, couldn't be bothered to write a patch...11:43
gitlab-br-botbuildstream: merge request (jennis/note_explicit_tracking_of_junctions->master: Add a note explaining that junction elements must be explictly tracked) #468 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/46811:50
jennisjuergbi, if you have a minute, I've changed the note on explicit tracking of junctions, would you mind reviewing it?11:51
jennisIf anyone else is interested: https://paste.baserock.org/abigugaxel it'd be nice if you're able to approve / suggest improvements of the note11:52
Nexustlater: ah12:05
*** mohan43u has quit IRC12:39
*** mohan43u has joined #buildstream12:42
gitlab-br-botbuildstream: issue #278 ("Filter documentation could be better") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/27812:55
gitlab-br-botbuildstream: merge request (valentindavid/331_include->master: WIP: Add support for include in project.conf) #471 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/47112:55
toscalixtlater: maybe instead of talking about milestone we can talk about version12:56
toscalixin theory a bug can affect more than one milestone12:56
tlaterI think calling it "version" in this context is less confusing12:56
toscalixthis differentiation will be interesting when we have in parallel a stable and a development version. Current bugs might or might nmot affect a past stable version12:56
tlaterI personally think about milestones as future versions12:57
toscalixwould you be so kind to create a MR?12:57
tlaterSure, I'll be in a meeting for a bit, but I'll do so after :)12:58
toscalixthanks12:58
gitlab-br-botbuildstream: merge request (tlater/context-trailing-slash->master: tests/context/context.py: Don't break with trailing slashes) #470 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/47012:59
*** jonathanmaw has quit IRC13:13
gitlab-br-botbuildstream: merge request (jennis/note_explicit_tracking_of_junctions->master: Add a note explaining that junction elements must be explictly tracked) #468 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/46813:42
tlatertoscalix: Looks like I just misread the template, perhaps because the gitlab command is /milestone. Annoying, but we can't actually change that :|13:45
gitlab-br-botbuildstream: merge request (jennis/note_explicit_tracking_of_junctions->master: Add a note explaining that junction elements must be explictly tracked) #468 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/46814:00
gitlab-br-botbuildstream: merge request (jennis/note_explicit_tracking_of_junctions->master: Add a note explaining that junction elements must be explictly tracked) #468 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/46814:33
*** xjuan has joined #buildstream14:41
gitlab-br-botbuildstream: merge request (jennis/note_explicit_tracking_of_junctions->master: Add a note explaining that junction elements must be explictly tracked) #468 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/46814:45
gitlab-br-botbuildstream: merge request (juerg/googlecas->master: WIP: Remote Execution CAS-based artifact cache) #337 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/33714:57
gitlab-br-botbuildstream: merge request (juerg/googlecas->master: WIP: Remote Execution CAS-based artifact cache) #337 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/33715:14
*** jonathanmaw has joined #buildstream15:20
gitlab-br-botbuildstream: merge request (juerg/googlecas->master: WIP: Remote Execution CAS-based artifact cache) #337 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/33715:23
gitlab-br-botbuildstream: issue #402 ("Save only the most recent build cache for each element") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/40215:27
*** dominic has quit IRC15:30
gitlab-br-botbuildstream: merge request (jennis/note_explicit_tracking_of_junctions->master: Add a note explaining that junction elements must be explictly tracked) #468 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/46815:30
*** tiagogomes has quit IRC15:56
gitlab-br-botbuildstream: issue #405 ("bst is still pushing artifacts it just have pulled") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/40516:24
finnAnyone know who "George from Uber" is in regards to BuildFarm?16:28
finnI'm fairly sure his nick is ggensure on the google-groups I'm looking at16:29
finnAh nvm, I found his email by grepping a git repo :)16:31
*** noisecell has quit IRC16:41
*** bethw has quit IRC17:03
*** toscalix has quit IRC17:05
*** jonathanmaw has quit IRC17:44
*** aday has quit IRC18:25
*** ernestask has quit IRC19:44
*** xjuan has quit IRC19:56
*** bochecha_ has quit IRC22:32
*** bochecha_ has joined #buildstream22:36
*** bochecha_ has quit IRC22:38
*** bochecha_ has joined #buildstream22:39
*** bochecha_ has quit IRC22:47
*** bochecha_ has joined #buildstream22:47
*** bochecha_ has quit IRC22:59
*** bochecha_ has joined #buildstream23:12
*** bochecha_ has quit IRC23:44
*** bochecha_ has joined #buildstream23:44
*** bochecha_ has quit IRC23:56

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