| *** ironfoot has quit IRC | 02:06 | |
| *** ironfoot has joined #buildstream | 02:06 | |
| *** finn has joined #buildstream | 07:50 | |
| *** finn_ has joined #buildstream | 07:52 | |
| *** finn has quit IRC | 07:54 | |
| *** bochecha_ has joined #buildstream | 08:29 | |
| *** jonathanmaw has joined #buildstream | 08:41 | |
| gitlab-br-bot | buildstream: merge request (richardmaw/cache-fail->master: WIP: Store failed builds in the cache) #475 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/475 | 08:46 |
|---|---|---|
| gitlab-br-bot | buildstream: 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/470 | 08:47 |
| *** bethw has joined #buildstream | 08:54 | |
| gitlab-br-bot | buildstream: merge request (tlater/context-trailing-slash->master: tests/context/context.py: Don't break with trailing slashes) #470 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/470 | 09:17 |
| gitlab-br-bot | buildstream: merge request (jmac/cas_virtual_directory->master: WIP: CAS-backed virtual directory implementation) #481 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/481 | 09:22 |
| gitlab-br-bot | buildstream: merge request (caching_build_trees->master: WIP: Caching buildtrees) #474 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/474 | 09:22 |
| *** dominic has joined #buildstream | 09:24 | |
| *** finn_ has quit IRC | 09:38 | |
| *** finn has joined #buildstream | 09:38 | |
| *** finn has quit IRC | 09:41 | |
| *** finn has joined #buildstream | 09:43 | |
| tlater | ta for the merge juergbi :) | 09:44 |
| tlater | Nexus: Any branch that's up-to-date with master will no longer cause those annoying context failures | 09:45 |
| tlater | So knock yourself out using docker for testing... | 09:45 |
| Nexus | woo!!! | 09:46 |
| * Nexus can stop pushing test code up to the CI | 09:46 | |
| *** finn has quit IRC | 09:50 | |
| jmac | juergbi: Do you know where I can find a list of methods which operate on `repeated` fields in protocol buffers? Google isn't helping much. | 10:45 |
| jmac | juergbi: Nevermind, think I've found it (google.protobuf.pyext._message.RepeatedCompositeContainer) | 10:49 |
| juergbi | jmac: the official documentation is here: https://developers.google.com/protocol-buffers/docs/reference/python-generated#repeated-fields | 10:57 |
| jmac | Yes, I'd found that, but it doesn't list all the methods | 10:59 |
| juergbi | yes, it's a bit lacking | 10:59 |
| *** finn has joined #buildstream | 11:02 | |
| noisecell | tristan, continuing with or discussion yesterday regarding to know the dependencies using some commands or the logs. I've run "bst show --deps run" on my element and it only shows itself, is this the expected output? | 11:06 |
| noisecell | https://gitlab.com/baserock/definitions/blob/master/elements/systems/minimal-system-image-x86_64.bst | 11:06 |
| noisecell | https://paste.baserock.org/uyexeraqax | 11:06 |
| noisecell | I've posted this 2 links for reference ^^ | 11:07 |
| noisecell | are there any command that shows the files installed from an element and which element comes from (if the element depends on others)? | 11:08 |
| *** jsgrant has quit IRC | 11:29 | |
| *** mohan43u has quit IRC | 11:41 | |
| *** mohan43u has joined #buildstream | 11:44 | |
| *** toscalix has joined #buildstream | 11:48 | |
| gitlab-br-bot | buildstream: merge request (jmac/cas_virtual_directory->master: WIP: CAS-backed virtual directory implementation) #481 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/481 | 11:52 |
| *** toscalix has quit IRC | 11:58 | |
| *** jsgrant has joined #buildstream | 12:08 | |
| gitlab-br-bot | buildstream: merge request (caching_build_trees->master: WIP: Caching buildtrees) #474 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/474 | 12:12 |
| *** bethw has quit IRC | 12:27 | |
| *** finn has quit IRC | 12:38 | |
| *** finn has joined #buildstream | 12:38 | |
| gitlab-br-bot | buildstream: merge request (jmac/cas_virtual_directory->master: WIP: CAS-backed virtual directory implementation) #481 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/481 | 13:26 |
| *** bethw has joined #buildstream | 13:28 | |
| noisecell | tristan, https://gitlab.com/baserock/definitions/blob/master/elements/systems/minimal-system-image-x86_64.bst is probably an special case. It does uses a plugin, so it seems that its dependencies are only build-dependencies. But to me it should use the "config: base: input:" information to get the runtime dependencies, IMO. | 13:45 |
| noisecell | and if we look at https://gitlab.com/baserock/definitions/blob/master/elements/systems/minimal-system.bst -- something similar happens. Its dependencies are only build dependencies but its configuration mark its output as runtime. But "bst show --deps run systems/minimal-system.bst" only shows itself as runtime dependency. | 13:59 |
| noisecell | Reading about the compose element: https://buildstream.gitlab.io/buildstream/elements/compose.html#module-elements.compose -- it says that this is a typical case. | 13:59 |
| gitlab-br-bot | buildstream: merge request (jmac/cas_virtual_directory->master: WIP: CAS-backed virtual directory implementation) #481 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/481 | 14:06 |
| gitlab-br-bot | buildstream: merge request (richardmaw/cache-fail->master: WIP: Store failed builds in the cache) #475 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/475 | 14:12 |
| *** xjuan has joined #buildstream | 14:24 | |
| *** ernestask has joined #buildstream | 14:33 | |
| gitlab-br-bot | buildstream: issue #404 ("Context directories may contain double slashes") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/404 | 15:03 |
| gitlab-br-bot | buildstream: merge request (reduce_history_in_cache->master: WIP: Reduce history in cache) #482 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/482 | 15:09 |
| gitlab-br-bot | buildstream: merge request (caching_build_trees->master: WIP: Caching buildtrees) #474 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/474 | 15:13 |
| gitlab-br-bot | buildstream: issue #418 ("Should we use a CAS-based source cache as a mirror?") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/418 | 15:14 |
| gitlab-br-bot | buildstream: merge request (reduce_history_in_cache->master: WIP: Reduce history in cache) #482 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/482 | 15:14 |
| *** Phil has joined #buildstream | 15:17 | |
| noisecell | I've updated https://gitlab.com/BuildStream/buildstream/issues/406 which is something similar that I have been looking in the previous discussion with tristan | 15:24 |
| *** xjuan has quit IRC | 15:47 | |
| *** tristan has joined #buildstream | 15:50 | |
| *** xjuan has joined #buildstream | 15:56 | |
| gitlab-br-bot | buildstream: merge request (jmac/cas_virtual_directory->master: WIP: CAS-backed virtual directory implementation) #481 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/481 | 16:01 |
| gitlab-br-bot | buildstream: merge request (jmac/cas_virtual_directory->master: WIP: CAS-backed virtual directory implementation) #481 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/481 | 16:01 |
| jennis | tristan, hi, whenever you have some time, !477 (remote expiry - using disk space available on the filesystem) is ready for maintainer review :) | 16:17 |
| gitlab-br-bot | buildstream: merge request (richardmaw/cache-fail->master: Store failed builds in the cache) #475 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/475 | 16:22 |
| gitlab-br-bot | buildstream: merge request (gokcen/sandbox_cleanup_fix->master: Fix post-bwrap cleanup behaviour) #480 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/480 | 16:23 |
| gitlab-br-bot | buildstream: merge request (gokcen/sandbox_cleanup_fix->master: Fix post-bwrap cleanup behaviour) #480 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/480 | 16:23 |
| gitlab-br-bot | buildstream: merge request (gokcen/sandbox_cleanup_fix->master: Fix post-bwrap cleanup behaviour) #480 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/480 | 16:23 |
| *** dominic has quit IRC | 16:30 | |
| *** jsgrant has quit IRC | 16:41 | |
| *** Phil has quit IRC | 16:50 | |
| jonathanmaw | valentind: if you haven't gotten round to reviewing my mirrored fetching code yet, I'm just about to push some changes that makes it a bit nicer | 16:54 |
| jonathanmaw | (when I rolled back some commits I lost the nice tidying of an algorithm that had previously been vexing me) | 16:54 |
| *** jonathanmaw has quit IRC | 17:00 | |
| gitlab-br-bot | buildstream: merge request (328-support-for-downloading-sources-from-mirrors->master: WIP: Resolve "Support for downloading sources from mirrors") #404 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/404 | 17:02 |
| gitlab-br-bot | buildstream: issue #379 ("_sandboxbwrap.py: Post-bwrap cleanup shouldn't attempt to remove preexisting folders") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/379 | 17:02 |
| gitlab-br-bot | buildstream: merge request (gokcen/sandbox_cleanup_fix->master: Fix post-bwrap cleanup behaviour) #480 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/480 | 17:02 |
| *** bethw has quit IRC | 17:06 | |
| tristan | jennis, how come we are using PushException for indicating the artifact is too big now ? | 17:09 |
| tristan | jennis, did it happen that the only error which the receiver recognizes is the artifact too big error ? | 17:10 |
| tristan | tlater, ^^^ ? | 17:12 |
| tristan | jennis, using ansii2html (from pip), I can render bst output with colors in docs :) | 17:22 |
| *** awacheux has quit IRC | 17:42 | |
| *** tristan has quit IRC | 17:52 | |
| *** ernestask has quit IRC | 17:53 | |
| *** tristan has joined #buildstream | 17:57 | |
| *** jsgrant has joined #buildstream | 17:57 | |
| gitlab-br-bot | buildstream: merge request (correcting_typo->master: Correcting a typo in safe_remove's comment) #472 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/472 | 18:00 |
| gitlab-br-bot | buildstream: merge request (correcting_typo->master: Correcting a typo in safe_remove's comment) #472 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/472 | 18:35 |
| gitlab-br-bot | buildstream: merge request (chandan/add-argument-bst-pushreceive->master: _artifactcache/pushreceive.py: Add Click type for CLI argument 'repo') #476 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/476 | 19:34 |
| tlater | tristan: Sorry, didn't keep up to date on that branch | 19:42 |
| tlater | I think I recommended that because the ArtifactTooBigError seemed a bit specific? | 19:43 |
| tristan | tlater, right, that is intentionally specific | 19:49 |
| tristan | tlater, the program was otherwise returning !0 exit status and spitting a useful stack trace in the case of errors | 19:50 |
| tlater | Ah, I didn't realize | 19:50 |
| tristan | now it assumes all errors are caused by a gracefully ignorable "artifact too big" state | 19:50 |
| tristan | s/state/condition | 19:50 |
| tristan | :) | 19:50 |
| tlater | Hm, in my mind Buildstream shouldn't crash when a network error occurs | 19:51 |
| tlater | Oh, wait, it's just the subprocess that returns !0 | 19:51 |
| tristan | you mean BuildStream :) | 19:51 |
| tlater | Darnit, my mobile phone has the wrong spelling | 19:52 |
| * tlater corrects it | 19:52 | |
| tristan | tlater, right, it's easy enough to deal with a stack trace for things that really brake, we could go one further and detail the really recognizable system errors without the stack trace | 19:52 |
| tristan | but anyway, returning 0 with a warning on every error is not bright | 19:53 |
| tlater | Yep, agreed, sorry jennis for screwing you over here | 19:53 |
| gitlab-br-bot | buildstream: issue #409 ("bst-artifact-receive: CLI should not throw stacktrackes when repo argument is invalid") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/409 | 20:07 |
| gitlab-br-bot | buildstream: merge request (chandan/add-argument-bst-pushreceive->master: _artifactcache/pushreceive.py: Add Click type for CLI argument 'repo') #476 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/476 | 20:07 |
| *** tristan has quit IRC | 20:30 | |
| *** xjuan has quit IRC | 20:48 | |
| *** bochecha_ has quit IRC | 20:53 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!