| *** mcatanzaro has quit IRC | 01:26 | |
| *** tristan has joined #buildstream | 07:28 | |
| *** noisecell has joined #buildstream | 08:50 | |
| *** bethw has joined #buildstream | 09:28 | |
| *** jonathanmaw has joined #buildstream | 09:40 | |
| *** ssam2 has joined #buildstream | 10:04 | |
| *** tpollard has quit IRC | 10:32 | |
| gitlab-br-bot | buildstream: merge request (modAndTest->master: Making changes to various documents:) #206 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/206 | 10:39 |
|---|---|---|
| juergbi | tristan: i'll take another look on the option aspect and respond on the MR, but just as a quick point from my notes, for flags the string approach doesn't seem ideal, overall it wouldn't be that simple either | 10:41 |
| tristan | I see, flags | 10:43 |
| tristan | so there is 2 approaches for ridding us of that baggage | 10:44 |
| tristan | the subtly-api-breaking-but-nobody-gonna-care road | 10:45 |
| tristan | or the any-list-in-option-values-gets-translated-to-colon-delimited-string road | 10:45 |
| juergbi | the transform support patch is pretty small and the API is all internal, that's why i went for that approach for now | 10:48 |
| juergbi | can reevaluate this, though | 10:48 |
| *** tristan has quit IRC | 10:48 | |
| *** tristan has joined #buildstream | 11:07 | |
| tristan | juergbi, gah - I've been struggling with this power adapter all day, fiddling with it, has to sit "just so" for power to flow through it | 11:10 |
| tristan | just went out and bought a new one | 11:10 |
| * tristan reads log | 11:11 | |
| tristan | juergbi, I'm wary of "it's all internal, lets do this for now..." approaches is the thing. Either we are constantly refactoring everything with every change and keeping things clean - or - we are constantly falling behind on our technical debt | 11:13 |
| tristan | I wouldnt mind to do the cleanup right now in advance of your branch, could probably get it done by days end... | 11:14 |
| juergbi | i generally agree, however, i wasn't convinced that the other solution would really be cleaner | 11:14 |
| juergbi | error messages might be worse, e.g., but i have to think about that again | 11:14 |
| juergbi | btw: please note that i still have to update that branch to work on top of the more recent multi-cache branch | 11:15 |
| tristan | for flags, yeah; it would be cleaner to break API and demand that user configuration specify "foo:bar:baz" as a string in the YAML (error message wise) | 11:16 |
| tristan | juergbi, I noticed yeah, getting ahead of myself because went through the other patches | 11:16 |
| tristan | there is a bunch of little cleanups sitting on that branch which are already landed | 11:16 |
| tristan | should make the whole thing less noisy | 11:16 |
| juergbi | right. i landed a few already in Dec but left less straight-forward ones in the branch | 11:18 |
| gitlab-br-bot | buildstream: merge request (sam/savefile-utility->master: utils.py: Add save_file_atomic utility) #179 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/179 | 11:54 |
| gitlab-br-bot | buildstream: merge request (sam/savefile-utility->master: utils.py: Add save_file_atomic utility) #179 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/179 | 12:35 |
| gitlab-br-bot | buildstream: merge request (modAndTest->master: Making changes to various documents:) #206 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/206 | 12:59 |
| *** xjuan has joined #buildstream | 13:18 | |
| *** bethw has left #buildstream | 13:35 | |
| gitlab-br-bot | buildstream: merge request (gating-on-docs-breakage->master: Documentation: Fix warnings, and make all warnings errors.) #207 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/207 | 14:13 |
| *** noisecell has quit IRC | 14:18 | |
| *** tristan has quit IRC | 14:20 | |
| *** tristan has joined #buildstream | 14:23 | |
| *** noisecell has joined #buildstream | 14:25 | |
| *** tiago has joined #buildstream | 14:37 | |
| gitlab-br-bot | buildstream: merge request (sam/savefile-utility->master: utils.py: Add save_file_atomic utility) #179 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/179 | 14:44 |
| gitlab-br-bot | buildstream: merge request (modAndTest->master: Making changes to various documents:) #206 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/206 | 14:46 |
| gitlab-br-bot | buildstream: merge request (gating-on-docs-breakage->master: Documentation: Fix warnings, and make all warnings errors.) #207 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/207 | 15:21 |
| *** gitlab-br-bot has quit IRC | 15:26 | |
| *** brlogger has joined #buildstream | 15:26 | |
| *** gitlab-br-bot has joined #buildstream | 15:27 | |
| gitlab-br-bot | buildstream: issue #179 ("Be smarter about querying summaries from remote caches") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/179 | 15:39 |
| gitlab-br-bot | buildstream: merge request (sam/multiple-caches->master: Multiple remote cache support) #166 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/166 | 16:02 |
| gitlab-br-bot | buildstream: merge request (modAndTest->master: Making changes to various documents:) #206 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/206 | 16:09 |
| *** noisecell has quit IRC | 16:20 | |
| *** mcatanzaro has joined #buildstream | 16:21 | |
| gitlab-br-bot | buildstream: merge request (gating-on-docs-breakage->master: Documentation: Fix warnings, and make all warnings errors.) #207 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/207 | 16:39 |
| gitlab-br-bot | buildstream: merge request (modAndTest->master: Making changes to various documents:) #206 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/206 | 16:55 |
| *** ssam2 has quit IRC | 17:34 | |
| nexus | what causes: Error: Failed to synchronize cache for repo 'updates' in the pipeline? | 18:25 |
| nexus | "Error: Failed to synchronize cache for repo 'updates'" | 18:25 |
| nexus | i made that message a bit unclear | 18:25 |
| tristan | nexus, nothing from buildstream afaict, maybe a dependency | 18:31 |
| nexus | i'm trying to fugure out why my pipeline failed | 18:32 |
| *** jonathanmaw has quit IRC | 18:39 | |
| *** tiago has quit IRC | 18:59 | |
| *** valentind has joined #buildstream | 20:26 | |
| *** xjuan has quit IRC | 20:31 | |
| *** valentind has quit IRC | 22:25 | |
| *** mcatanzaro has quit IRC | 22:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!