IRC logs for #buildstream for Wednesday, 2020-04-22

*** hasebastian has joined #buildstream03:40
*** narispo has quit IRC05:21
*** narispo has joined #buildstream05:21
*** narispo has quit IRC05:23
*** narispo has joined #buildstream05:24
*** hasebastian has quit IRC06:08
gitlab-br-botjuergbi opened MR !1876 (juerg/centos-7.7->master: .gitlab-ci.yml: Update to CentOS 7.7) on buildstream https://gitlab.com/BuildStream/buildstream/-/merge_requests/187606:11
*** tristan has quit IRC07:19
*** benschubert has joined #buildstream07:21
*** ikerperez has quit IRC07:25
*** cphang has quit IRC07:25
*** tristan has joined #buildstream07:32
*** ChanServ sets mode: +o tristan07:34
*** ikerperez has joined #buildstream07:38
*** cphang has joined #buildstream07:51
*** phildawson-ct has joined #buildstream08:06
tristanjuergbi, got a moment ? I'm trying to respond to your email about the junction jungle08:07
juergbisure08:07
tristanjuergbi, specifically I'm having a hard time parsing the part where you would reverse it... in my example I had foo's declaration of baz.bst junction explicitly overriding the bar project's junction baz.bst08:09
juergbiyes, so I'd rather specify in foo's bar.bst that its baz.bst should be overridden by foo's baz.bst08:10
tristanare you saying that you would prefer to have foo's bar.bst say "override this project's junction baz.bst... with this project local declaration: baz.bst" ?08:10
juergbicorrect08:10
tristanOk08:10
tristan:)08:10
juergbias we're effectively modifying the bar subproject08:11
*** santi has joined #buildstream08:17
*** tpollard has joined #buildstream08:17
*** lachlan has joined #buildstream08:19
*** rdale has joined #buildstream08:33
*** phoenix has joined #buildstream08:33
tristanjuergbi, got it out... maybe if we can polish this off I can work on it next after these versioning things08:47
*** hasebastian has joined #buildstream08:54
*** phoenix has quit IRC08:59
*** phoenix has joined #buildstream09:05
*** hasebastian has quit IRC09:12
*** phoenix has quit IRC09:20
juergbitristan: replied. that would be great09:36
*** cs-shadow has joined #buildstream09:45
*** tristan has quit IRC10:17
gitlab-br-botjuergbi merged MR !1876 (juerg/centos-7.7->master: .gitlab-ci.yml: Update to CentOS 7.7) on buildstream https://gitlab.com/BuildStream/buildstream/-/merge_requests/187610:30
*** tristan has joined #buildstream10:52
*** tristan has quit IRC11:03
*** phoenix has joined #buildstream11:05
*** hasebastian has joined #buildstream11:14
gitlab-br-bottraveltissues opened issue #1289 (automatic cachekey updating for tests is broken) on buildstream https://gitlab.com/BuildStream/buildstream/-/issues/128912:39
WSalmonjuergbi, are you able to run tox locally for bst-plugins-experimental? im trying to tidy up git_tag and i still cant figure out whats up14:42
gitlab-br-botjuergbi closed issue #1289 (automatic cachekey updating for tests is broken) on buildstream https://gitlab.com/BuildStream/buildstream/-/issues/128914:46
*** lachlan has quit IRC14:52
*** lachlan has joined #buildstream14:52
*** phoenix has quit IRC14:53
juergbiWSalmon: no, not working properly here for some reason, haven't investigated yet14:53
WSalmonok, i have just got tpollard to run them and he has no issue, im really confused, i have tried to run them loads of diffrent versiond of python and tox and i just keep drawing a blank14:54
juergbimaybe there is an issue if bst-plugins-experimental is installed in the current venv14:55
tpollardif you want anything else just ask14:55
WSalmonthanks tpollard14:57
WSalmoni made a new venv and only installed tox14:57
juergbiWSalmon: and that works?14:58
WSalmonnope14:58
WSalmoneverything i have tried ends up with bst-plugins-experimental==0.12.0,BuildStream==1.93.114:59
juergbisame here14:59
WSalmoni even tried removing .tox14:59
WSalmonim on fedora-31 is tpolard on ubuntu?14:59
tpollard19.10 yeh15:00
WSalmonooooooooooooo15:02
WSalmoni did a fresh clone and it works15:02
WSalmonjuergbi, maybe a git clean with the right letters is needed..15:03
*** lachlan has quit IRC15:05
WSalmon`git clean -fX` dosent seem to be enough tho15:06
juergbiWSalmon: I had a very old bst_plugins_experimental.egg-info/15:06
juergbigit clean -dfX should suffice15:06
WSalmonim confused how that happened tho15:08
WSalmonyep `git clean -dfX` did it15:09
WSalmonbenschubert, your our expert at tox do you have any idea how this could have happened? i dont mind adding a patch to try and stop it but i dont know how to start15:10
jjardonhi, I need to do a full rebuild of a bst project: what would be the best way? ie, best/cleanest way to force a change in the cache key15:33
coldtomjjardon open a workspace on the bottom element would be a way i guess? or configure buildstream  to not look in your cache15:35
coldtom(that is, change the location of your cache to somewhere else15:36
jjardonah, but I want the cache to be confgured and used, but not to pull basically15:36
coldtomah, so you want to force a full build to be pushed to a remote cache?15:37
cphangcan an environment variable be injected via project.conf?15:39
coldtom^that should change cache keys aiui15:40
juergbijjardon: is this for test/validation, or what's the reason for this?15:42
tpollardif you don't want to allow pull, set the fetchers to zero or --remote to something dummy depending on your bst version15:42
jjardonjuergbi: test15:42
jjardoncoldtom: exactly that :)15:43
jjardontpollard: does that work with bst 1.4.2 ?15:43
coldtomtpollard, fetchers are also used for `fetch` aren't they?15:43
coldtomi guess you could do it in multiple steps15:44
tpollardcoldtom: maybe, not 100% sure15:44
tpollardI would assume yes, that it's source fetchers too15:46
coldtomif it's possible to do the build first, then configure the cache, then push the artifacts i think that'd be the way to go, but then you will only push (--deps all) on successful builds (i guess you could ignore build failures and wait for the push failure when it tries to push non-existent artifacts)15:46
*** hasebastian has quit IRC15:52
*** lachlan has joined #buildstream15:59
WSalmonhttps://gitlab.com/BuildStream/bst-external/-/blob/master/bst_external/sources/git_tag.py#L564 this allows my plugin to know if the user selected false or didnt spesifiy false, if the user didnt specify false/true and the repo has gitlfs then i can raise a configureable warning. how do i do this with master, what do i call on node? i dont seem to be able to have a boolin default to None16:03
*** lachlan has quit IRC16:05
WSalmonits wiered having a boolin default to None but it works so well as i can still just do `if var:` and var can be None or false and it still behaves right but if i need to check if it has been spesified then i can expelicity check for None16:05
juergbivalentind: I can now build all fd-sdk oci images with the juerg/vdirectory branches of buildstream and bst-plugins-experimental16:10
tpollardWSalmon: maybe look at the ref not in track configurable warning?16:10
juergbivalentind: will you build them yourself for testing or shall I send you the artifacts/tarballs somehow?16:10
juergbiideally, we would be testing them built with the default bwrap sandbox as well as BST_FORCE_SANDBOX=buildbox-run16:11
juergbithey shouldn't differ but exercise different code paths in buildstream core16:11
tpollardWSalmon: or the configurable submodule warnings16:11
juergbiWSalmon: I've recently added support for `None` as default value for ints16:18
juergbiwe could likely do the same for bool16:18
juergbialternatively, I think you could check something like: "use-lfs" in node16:19
juergbiand then make the get_bool call conditional16:19
*** lachlan has joined #buildstream16:20
*** tristan has joined #buildstream16:27
*** douglaswinship has quit IRC16:27
benschubertyou also have `node.is_none()` which would work16:27
benschubertWSalmon: for the tox question, not sure what there is to fix? I don't know how this old bst-plugins-experimental version got installed there16:28
gitlab-br-botjuergbi closed MR !1874 (juerg/validate-path->master: _casbaseddirectory.py: Validate path components) on buildstream https://gitlab.com/BuildStream/buildstream/-/merge_requests/187416:30
*** lachlan has quit IRC16:34
valentindjuergbi, I will test tomorrow. Thank you.16:47
juergbita16:47
*** lachlan has joined #buildstream16:53
*** lachlan has quit IRC16:56
*** lachlan has joined #buildstream16:56
*** tpollard has quit IRC17:01
*** lachlan has quit IRC17:02
*** santi has quit IRC17:15
*** narispo has quit IRC17:35
*** phoenix has joined #buildstream17:51
*** cphang has quit IRC17:52
*** ikerperez has quit IRC17:52
*** bethw has quit IRC17:54
*** jward has quit IRC17:55
*** WSalmon has quit IRC17:55
*** valentind has quit IRC17:55
*** robjh has quit IRC17:55
*** jward has joined #buildstream17:56
*** robjh has joined #buildstream17:56
*** WSalmon has joined #buildstream17:56
*** valentind has joined #buildstream17:56
*** cphang has joined #buildstream17:59
*** bethw has joined #buildstream18:00
*** rdale has quit IRC18:09
*** ikerperez has joined #buildstream18:15
*** ceibal has joined #buildstream18:16
*** ceibal has quit IRC18:45
*** benschubert has quit IRC19:19
*** rdale has joined #buildstream19:25
*** phildawson-ct has quit IRC20:09
gitlab-br-botcs-shadow closed issue #1278 (Remote execution test is failing) on buildstream https://gitlab.com/BuildStream/buildstream/-/issues/127820:10
gitlab-br-botcs-shadow opened issue #1290 (Some tests break with pytest 5.4.1) on buildstream https://gitlab.com/BuildStream/buildstream/-/issues/129020:26
gitlab-br-botcs-shadow closed issue #1248 (Some public API functions can raise private LoadError exception) on buildstream https://gitlab.com/BuildStream/buildstream/-/issues/124820:29
*** cs-shadow has quit IRC22:41

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