*** Prince781 has quit IRC | 00:04 | |
*** Prince781_ has joined #buildstream | 00:04 | |
*** Prince781_ is now known as Prince781 | 00:05 | |
*** janeric has joined #buildstream | 00:58 | |
*** bochecha has quit IRC | 01:27 | |
*** dineshdb[m] has quit IRC | 02:14 | |
*** inigomartinez has quit IRC | 02:14 | |
*** waltervargas[m] has quit IRC | 02:15 | |
*** segfault3[m] has quit IRC | 02:15 | |
*** ssssam[m] has quit IRC | 02:15 | |
*** rafaelff[m] has quit IRC | 02:15 | |
*** mattiasb has quit IRC | 02:15 | |
*** albfan[m] has quit IRC | 02:15 | |
*** alatiera has quit IRC | 02:15 | |
*** cgmcintyre[m] has quit IRC | 02:15 | |
*** krichter[m] has quit IRC | 02:15 | |
*** kailueke[m] has quit IRC | 02:15 | |
*** tlater[m] has quit IRC | 02:15 | |
*** awacheux[m] has quit IRC | 02:15 | |
*** jjardon[m] has quit IRC | 02:15 | |
*** doras[m] has quit IRC | 02:15 | |
*** asingh_[m] has quit IRC | 02:15 | |
*** oknf[m] has quit IRC | 02:15 | |
*** Demos[m] has quit IRC | 02:15 | |
*** connorshea[m] has quit IRC | 02:15 | |
*** pro[m] has quit IRC | 02:15 | |
*** theawless[m] has quit IRC | 02:15 | |
*** m_22[m] has quit IRC | 02:16 | |
*** abderrahim[m] has quit IRC | 02:16 | |
*** xjuan has quit IRC | 02:22 | |
*** Prince781 has quit IRC | 02:26 | |
*** dineshdb[m] has joined #buildstream | 02:38 | |
*** waltervargas[m] has joined #buildstream | 02:41 | |
*** inigomartinez has joined #buildstream | 02:52 | |
*** segfault3[m] has joined #buildstream | 02:52 | |
*** Prince781 has joined #buildstream | 02:57 | |
*** jjardon[m] has joined #buildstream | 03:02 | |
*** asingh_[m] has joined #buildstream | 03:07 | |
*** oknf[m] has joined #buildstream | 03:07 | |
*** doras[m] has joined #buildstream | 03:08 | |
*** Prince781 has quit IRC | 03:09 | |
*** connorshea[m] has joined #buildstream | 03:13 | |
*** Demos[m] has joined #buildstream | 03:18 | |
*** eldritch has joined #buildstream | 03:23 | |
*** inigomartinez has quit IRC | 03:37 | |
*** jjardon[m] has quit IRC | 03:37 | |
*** connorshea[m] has quit IRC | 03:37 | |
*** doras[m] has quit IRC | 03:37 | |
*** waltervargas[m] has quit IRC | 03:37 | |
*** Demos[m] has quit IRC | 03:37 | |
*** dineshdb[m] has quit IRC | 03:37 | |
*** oknf[m] has quit IRC | 03:37 | |
*** segfault3[m] has quit IRC | 03:37 | |
*** asingh_[m] has quit IRC | 03:38 | |
*** theawless[m] has joined #buildstream | 04:29 | |
*** pro[m] has joined #buildstream | 04:31 | |
*** abderrahim[m] has joined #buildstream | 04:46 | |
*** m_22[m] has joined #buildstream | 04:52 | |
*** albfan[m] has joined #buildstream | 05:18 | |
*** ssssam[m] has joined #buildstream | 05:32 | |
*** mattiasb has joined #buildstream | 05:55 | |
*** rafaelff[m] has joined #buildstream | 05:56 | |
*** cgmcintyre[m] has joined #buildstream | 06:15 | |
*** alatiera has joined #buildstream | 06:38 | |
*** krichter[m] has joined #buildstream | 06:48 | |
*** kailueke[m] has joined #buildstream | 06:51 | |
*** tlater[m] has joined #buildstream | 07:01 | |
*** awacheux[m] has joined #buildstream | 07:04 | |
*** asingh_[m] has joined #buildstream | 07:14 | |
*** jjardon[m] has joined #buildstream | 07:20 | |
*** segfault3[m] has joined #buildstream | 07:23 | |
*** oknf[m] has joined #buildstream | 07:24 | |
*** connorshea[m] has joined #buildstream | 07:25 | |
*** inigomartinez has joined #buildstream | 07:35 | |
*** doras[m] has joined #buildstream | 07:57 | |
*** waltervargas[m] has joined #buildstream | 08:05 | |
*** Demos[m] has joined #buildstream | 08:05 | |
*** dineshdb[m] has joined #buildstream | 08:08 | |
gitlab-br-bot | jonathanmaw approved MR !931 (jmac/remote-sandbox-errors->master: _sandboxremote.py: Use the standard SandboxError) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/931 | 08:40 |
---|---|---|
*** toscalix has joined #buildstream | 09:02 | |
valentind | Just to be sure. If I need some extra python dependencies in a branch where the tests depend on that dependency. I need first to make a merge with just the dependency added so that the when it is merged, the docker images are generated with the dependency. And then I can merge the rest. | 09:03 |
valentind | Am I right? | 09:03 |
valentind | Ah, no there is a dev-requirements.txt in the buildstream-docker-images itself. | 09:05 |
jmac | I've finished the performance test on !911 and it looks roughly equivalent. If anyone's got time to review it, that would be welcome. | 09:40 |
gitlab-br-bot | MR !911: Direct CAS-to-CAS import https://gitlab.com/BuildStream/buildstream/merge_requests/911 | 09:40 |
juergbi | jmac: I will take a look | 09:41 |
*** benschubert has joined #buildstream | 09:48 | |
*** tiagogomes has joined #buildstream | 09:50 | |
*** raoul has joined #buildstream | 10:03 | |
WSalmon | I found that if i import stream in to the frontend/cli.py then the auto completion brakes but the tests do not capture this, is this know acceptable behavour or should we have a issue to catch this sort of error or have we tried and found it too hard? the issue seems to be a import of a import of stream doing 'from ... import foo' and python not getting the loaction right when run to give auto complete | 10:14 |
WSalmon | the test would be to check for this failure mode of auto completion rather than to allow stream to be imported | 10:15 |
*** bochecha has joined #buildstream | 10:16 | |
juergbi | WSalmon: if you can think of a simple way to improve/extend the completion tests to cover this, that would certainly be great | 10:17 |
juergbi | however, if it requires a large effort, there are probably more important tasks to tackle first | 10:18 |
WSalmon | i havent really thought about testing, i just had a panic that one of my MR that was close to landing was going to brake auto complete and thought this might help some one else not have my same panic, (i have fixed my MR now) | 10:19 |
WSalmon | i will have a think about testing | 10:19 |
juergbi | yes, it would definitely be good to catch that. one option might also be to change the way imports are handled for completion to actually allow this (just a random thought, don't know whether this makes sense) | 10:20 |
*** alatiera_ has joined #buildstream | 10:33 | |
*** lachlan has quit IRC | 10:46 | |
*** lachlan has joined #buildstream | 11:15 | |
*** lachlan has quit IRC | 11:23 | |
*** lachlan has joined #buildstream | 11:36 | |
Kinnison | Are Provenance objects ever altered? | 11:36 |
* Kinnison is trying to understand why they have a clone() method | 11:37 | |
*** lachlan has quit IRC | 11:39 | |
*** TacoThief has joined #buildstream | 11:45 | |
gitlab-br-bot | willsalmon opened issue #751 (test for import failures for auto completes) on buildstream https://gitlab.com/BuildStream/buildstream/issues/751 | 11:48 |
jennis | Does anyone have the link for our IRC logs? :) | 11:50 |
jmac | The one in the topic? Why the smiley? | 11:50 |
jennis | Just being friendly | 11:51 |
jennis | and thanks | 11:51 |
* jennis did not notice that before | 11:51 | |
jmac | Ah, OK, I wondered if I was missing a joke | 11:51 |
*** lachlan has joined #buildstream | 11:54 | |
*** jonathanmaw has joined #buildstream | 12:15 | |
*** lachlan has quit IRC | 12:19 | |
*** lachlan has joined #buildstream | 13:00 | |
*** lachlan has quit IRC | 13:10 | |
*** lachlan has joined #buildstream | 13:19 | |
*** lachlan has quit IRC | 13:24 | |
tpollard | juergbi: I've rebased !768 on top of !922 that landed in master, hopefully it's acceptable | 13:33 |
gitlab-br-bot | MR !768: Retries should fail Backport 1.2 https://gitlab.com/BuildStream/buildstream/merge_requests/768 | 13:33 |
gitlab-br-bot | MR !922: Split up artifact cache and CAS cache https://gitlab.com/BuildStream/buildstream/merge_requests/922 | 13:33 |
tpollard | erm, that should be !786 | 13:33 |
gitlab-br-bot | MR !786: Don't pull artifact buildtrees by default https://gitlab.com/BuildStream/buildstream/merge_requests/786 | 13:33 |
juergbi | tpollard: will take a quick look at the cas/artifactcache part | 13:34 |
tpollard | cheers :) | 13:35 |
juergbi | tpollard: looks fine at quick glance. just two tiny nitpicks | 13:43 |
tpollard | juergbi: cool | 13:45 |
tpollard | juergbi: do you think initial or original could be used in place of element in that variable name? | 13:47 |
juergbi | original sounds fine to me | 13:48 |
tpollard | ty! | 13:50 |
alatiera | hello, I recall a while ago I was talking with someone about a plugin that generates oci/docker images | 13:53 |
alatiera | does this exist, and is it functional? | 13:53 |
alatiera | or was it just a concept/future problem, I have a hard time recalling | 13:53 |
WSalmon | ^cs-shadow finn tlater[m] | 13:55 |
finn | docker image plugin | 13:58 |
finn | I don't think I ever used it to generate docker images | 13:58 |
finn | I've used the docker source plugin | 13:58 |
WSalmon | dose it live in bst-external? | 13:59 |
Kinnison | alatiera: It was mostly conceptual though there is a prototype which uses bst checkout and docker's tooling | 13:59 |
Kinnison | alatiera: https://gitlab.com/BuildStream/buildstream/blob/master/contrib/bst-docker-import | 13:59 |
alatiera | Kinnison, Nice seems straight forward overall | 14:01 |
alatiera | Kinnison: so it can be done with just making a tarball of the bst checkout? Pretty cooL! | 14:02 |
Kinnison | alatiera: I'll admit I'm not a docker/oci expert, I just remember Chandan merging that | 14:03 |
alatiera | I am intrested in using Buildstream to generate images for the gitlab CI in gnome mainly, docker is a pain to deal with | 14:06 |
alatiera | looks doable, thanks a lot for the helP! | 14:06 |
gitlab-br-bot | richardmaw-codethink approved MR !799 (Qinusty/unit-test-utils->master: Fix issue with _pretty_size with large numbers of bytes) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/799 | 14:12 |
gitlab-br-bot | richardmaw-codethink merged MR !799 (Qinusty/unit-test-utils->master: Fix issue with _pretty_size with large numbers of bytes) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/799 | 14:12 |
*** lachlan has joined #buildstream | 14:22 | |
*** jonathanmaw_ has joined #buildstream | 14:26 | |
*** jonathanmaw has quit IRC | 14:26 | |
*** lachlan has quit IRC | 14:37 | |
*** lachlan has joined #buildstream | 14:39 | |
*** Prince781 has joined #buildstream | 14:52 | |
*** Prince781 has quit IRC | 14:54 | |
*** lachlan has quit IRC | 14:57 | |
*** lachlan has joined #buildstream | 15:00 | |
gitlab-br-bot | tpollard approved MR !931 (jmac/remote-sandbox-errors->master: _sandboxremote.py: Use the standard SandboxError) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/931 | 15:07 |
*** lachlan has quit IRC | 15:12 | |
*** jonathanmaw_ has quit IRC | 15:16 | |
*** jonathanmaw has joined #buildstream | 15:16 | |
*** lachlan has joined #buildstream | 15:20 | |
*** lachlan has quit IRC | 15:28 | |
*** lachlan has joined #buildstream | 15:32 | |
*** lachlan has quit IRC | 15:36 | |
*** lachlan has joined #buildstream | 15:45 | |
*** lachlan has quit IRC | 15:48 | |
*** lachlan has joined #buildstream | 16:16 | |
*** lachlan has quit IRC | 16:24 | |
*** ctolentino has quit IRC | 16:26 | |
*** lachlan has joined #buildstream | 16:46 | |
*** toscalix has quit IRC | 16:46 | |
gitlab-br-bot | jennis opened MR !932 (jennis/docs_patch->master: Update our instructions/examples regarding how we declare remote caches) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/932 | 16:48 |
jennis | A somewhat lengthy docs MR ^ I would appreciate if anyone could spare the time to review | 16:49 |
jennis | Please try and understand the MR description before reviewing | 16:49 |
adds68 | skullman, hey what is the method for printing to stdout during a build again? | 16:56 |
skullman | if it's in a plugin it's probably self.status | 16:56 |
skullman | or self.debug | 16:56 |
adds68 | skullman, that's it! Thanks | 16:57 |
*** lachlan has quit IRC | 17:14 | |
*** Prince781 has joined #buildstream | 17:15 | |
gitlab-br-bot | jmacarthur closed issue #746 (SandboxRemote causes bugs instead of errors) on buildstream https://gitlab.com/BuildStream/buildstream/issues/746 | 17:16 |
gitlab-br-bot | jmacarthur merged MR !931 (jmac/remote-sandbox-errors->master: _sandboxremote.py: Use the standard SandboxError) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/931 | 17:16 |
gitlab-br-bot | danielsilverstone-ct opened MR !933 (danielsilverstone-ct/further-optimisations->master: WIP: Some more microoptimisations) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/933 | 17:20 |
Kinnison | For those who might care, https://gitlab.com/BuildStream/buildstream/merge_requests/933 carries two more possible optimisations for pre-scheduler time in `bst build` totalling around 28s improvement in my test case | 17:21 |
Kinnison | it's WIP because I'm trying to find further improvements | 17:21 |
*** lachlan has joined #buildstream | 17:23 | |
adds68 | When using a mirror for a bst source, where would i find it once it has been resolved? | 17:23 |
adds68 | I have a list of sources for a given element, calling source.mirror.url still returns the mirror name, rather than the resolved url | 17:24 |
*** abderrahim2 has joined #buildstream | 17:26 | |
*** abderrahim1 has quit IRC | 17:27 | |
ikerperez | How long will last "bst source-bundle" on Buildstream? is it worth to start working with it or should I wait for this request be approved https://gitlab.com/BuildStream/buildstream/merge_requests/820 | 17:31 |
*** lachlan has quit IRC | 17:41 | |
persia | ikerperez: Probably not worth working with it. | 17:50 |
persia | That said, if you have a use case that it uniquely solves, that is worth discussion. Maybe the use case can be addressed another way, or maybe there are reasons not to deprecate it. | 17:50 |
persia | In the absence of argument, I'm expecting it to be announced as deprecated in the next release. | 17:51 |
ikerperez | ok, thanks persia | 17:57 |
gitlab-br-bot | valentindavid opened MR !934 (valentindavid/script-artifact-corruption->master: Fix cache corruption by scripts when layout and integration commands are used) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/934 | 17:58 |
*** finn has quit IRC | 18:03 | |
*** tpollard has quit IRC | 18:03 | |
*** tiagogomes has quit IRC | 18:03 | |
*** jonathanmaw has quit IRC | 18:03 | |
*** phildawson_ has quit IRC | 18:03 | |
*** WSalmon has quit IRC | 18:03 | |
*** raoul has quit IRC | 18:03 | |
*** rdale has quit IRC | 18:04 | |
*** toscalix has joined #buildstream | 18:04 | |
*** toscalix has quit IRC | 18:05 | |
*** toscalix has joined #buildstream | 18:06 | |
*** toscalix has quit IRC | 18:07 | |
*** tiagogomes has joined #buildstream | 18:22 | |
*** tpollard has joined #buildstream | 18:22 | |
*** WSalmon has joined #buildstream | 18:23 | |
*** Prince781 has quit IRC | 18:29 | |
*** lachlan has joined #buildstream | 18:39 | |
*** finn has joined #buildstream | 18:40 | |
*** bethw has quit IRC | 18:40 | |
*** lachlan has quit IRC | 18:40 | |
*** lachlan has joined #buildstream | 18:40 | |
*** finn has quit IRC | 18:40 | |
*** benbrown has quit IRC | 18:40 | |
*** benbrown has joined #buildstream | 18:40 | |
*** aiden has quit IRC | 18:40 | |
*** lachlan has quit IRC | 18:40 | |
*** aiden has joined #buildstream | 18:40 | |
*** adds68 has quit IRC | 18:40 | |
*** coldtom has quit IRC | 18:40 | |
*** coldtom has joined #buildstream | 18:41 | |
*** adds68 has joined #buildstream | 18:41 | |
*** bethw has joined #buildstream | 18:41 | |
*** Prince781 has joined #buildstream | 19:24 | |
*** xjuan has joined #buildstream | 19:38 | |
*** greppy has joined #buildstream | 20:14 | |
*** Prince781 has quit IRC | 20:42 | |
*** benschubert has quit IRC | 20:53 | |
*** bjurr has joined #buildstream | 21:36 | |
*** alatiera_ has quit IRC | 23:50 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!