IRC logs for #buildstream for Wednesday, 2017-09-06

gitlab-br-botbuildstream: issue #88 ("Update the install doc with OSTree >= v2017.8") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/8800:10
gitlab-br-botbuildstream: issue #88 ("Update the install doc with OSTree >= v2017.8") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/8800:39
gitlab-br-botpush on buildstream@master (by Tristan Van Berkom): 2 commits (last: tests/frontend/buildcheckout.py: Run tests in strict/non-strict mode) https://gitlab.com/BuildStream/buildstream/commit/787fb42f962951756268afa5b6bebc0030e68d8900:39
*** tristan has quit IRC00:54
*** bochecha has quit IRC01:18
*** bochecha has joined #buildstream01:19
*** bochecha has quit IRC01:23
*** bochecha has joined #buildstream01:51
*** bochecha has quit IRC02:23
*** bochecha has joined #buildstream03:51
*** bochecha has quit IRC04:23
*** bochecha has joined #buildstream05:51
*** bochecha has quit IRC06:23
*** bochecha has joined #buildstream06:28
*** bochecha has quit IRC07:01
*** bochecha has joined #buildstream08:04
*** jonathanmaw has joined #buildstream08:43
*** tlater has joined #buildstream10:31
tlaterHeh, I can't reproduce the blessings failure when running the tests directly, without setup.py10:36
tlaterThis is a fun one10:37
tlaterIt's definitely the redirection through setup.py - presumably it closes something before running pytest.11:53
tlater... No, maybe not11:54
tlaterWhen forcing the tests to run through subprocess.call() it works fine. It's the way in which pytest is called by default.11:55
tlaterI guess that makes sense since subprocess.call() would open a new stdin/stdout11:55
tlaterYup, this is how pytest-runner runs pytest:12:08
tlaterwith _save_argv(_sys.argv[:1] + self.addopts):12:08
tlaterself.result_code = __import__('pytest').main()12:08
*** tlater has quit IRC12:24
*** tlater has joined #buildstream13:10
*** tlater has quit IRC13:23
*** tlater has joined #buildstream13:23
*** xjuan has joined #buildstream13:26
*** xjuan has quit IRC13:35
*** xjuan has joined #buildstream13:38
tlaterCalling `sys.__stdout__ = open('/dev/stdout', 'w')` in the function that executes bst fixes the problem... Not sure if this hack is alright, but it's the best I've got.13:49
gitlab-br-botpush on buildstream@cross_platform (by Tristan Maat): 1 commit (last: Fix tests for other platforms) https://gitlab.com/BuildStream/buildstream/commit/9ac88535b370ce7e69145e1417dfa2059ed2bd7c14:19
gitlab-br-botbuildstream: merge request (cross_platform->master: WIP: Cross platform) #81 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/8114:19
*** tlater has quit IRC14:22
gitlab-br-botpush on buildstream@cross_platform (by Tristan Maat): 1 commit (last: Fix tests for other platforms) https://gitlab.com/BuildStream/buildstream/commit/d0169e205ac32d6ad618071979032e61fa02937414:29
gitlab-br-botbuildstream: merge request (cross_platform->master: WIP: Cross platform) #81 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/8114:29
*** bochecha has quit IRC14:40
gitlab-br-botpush on buildstream@cross_platform (by Tristan Maat): 1 commit (last: Fix tests for other platforms) https://gitlab.com/BuildStream/buildstream/commit/8fa354b40d41ecc480e4d412bd032aab7d8f646f14:42
gitlab-br-botbuildstream: merge request (cross_platform->master: WIP: Cross platform) #81 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/8114:42
*** tlater has joined #buildstream15:13
*** bochecha has joined #buildstream16:05
*** bochecha_ has joined #buildstream16:37
*** bochecha has quit IRC16:37
gitlab-br-botpush on buildstream@validation (by Tristan Maat): 2 commits (last: Add _yaml.validate_node) https://gitlab.com/BuildStream/buildstream/commit/5948904371d31d09674271b3e5975188775168a716:51
*** bochecha_ has quit IRC17:03
*** bochecha_ has joined #buildstream17:03
*** bochecha_ has quit IRC17:08
*** jonathanmaw has quit IRC17:09
*** bochecha_ has joined #buildstream17:32
*** tlater has quit IRC17:46
*** tristan has joined #buildstream18:11
*** ChanServ sets mode: +o tristan18:11
jjardon[m]Hi, is there a way to tell buildstream to not push artifacts? Im trying to avoid https://gitlab.com/baserock/definitions/-/jobs/3155093918:19
jjardon[m]https://buildstream.gitlab.io/ give a 404, is that expected?18:20
gitlab-br-botbuildstream: issue #89 ("Absolute build directories should be unique per element") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/8918:22
jjardon[m]doesnt seem there is an option here: https://buildstream.gitlab.io/buildstream/config.html#config18:22
tristanjjardon[m], mmyeah currently I either configure it or not18:23
tristanjjardon[m], obviously if you're configuring the buildstream.conf already, you can comment out the push url18:24
tristanbut there is nothing to do that on the CLI18:24
tristanjjardon[m], I am thinking of changing the configuration of artifact caches, though18:25
tristanSo that there can be more than one, and I think I like the idea of having some kind of `bst remote add/remove` kind of api for it18:25
jjardon[m]Im currently trying to think of the best solution for https://gitlab.com/baserock/definitions/issues/1318:27
tristanjjardon[m], I'm having a hard time to see what that issue is about18:27
tristanonly a title18:27
jjardon[m]if you are not using a protected branch, the CI will fail. so normal contributors can not submit anything18:28
tristanUmm18:29
jjardon[m]it will fail because bst will return failure when trying to push the artifacts18:29
jjardon[m]as those branches doesnt have the key18:29
tristanSo basically buildstream will fail to build if it is configured to push but cannot18:29
tristanis it ?18:29
jjardon[m]exactly18:29
tristanHmmmyeah I see, currently it's coded to bail out18:31
tristanwhich is strange, because we have this litter of 'can_push()' all over the place18:31
tristanwhich I would think we wouldnt need if we were just going to bail out anyway18:31
tristanjjardon[m], well ok so a work around right now is to sed the .gitlab-ci.yml differently based on the branch right ?18:34
tristanjjardon[m], other than that I think it's a bug18:34
tristanbut18:34
jjardon[m]maybe an option to "only warn instead fail" when push fail would work18:34
jjardon[m]tristan: yeah, but for that I have to duplicate all the jobs18:34
jjardon[m]one for "protected" branches, another for the rest18:34
gitlab-br-botbuildstream: issue #90 ("bst build bails if it cannot push to the artifact cache") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/9018:36
tristanjjardon[m], really ? that sucks18:37
tristanjjardon[m], you cant have the branch as a variable and conditionally generate the buildstream.conf depending on the branch name ?18:37
tristaneven without substing the .gitlab-ci.yml, just in the launch script18:38
jjardon[m]mmm, probably18:38
jjardon[m]time for dinner; I will try that later! thanks18:38
tristananyway I filed a bug, I think by default we should just work18:38
* jjardon[m] wants to spend some time with buildstream anyway :)18:38
*** xjuan has quit IRC18:41
tristanjjardon[m], happen to still be around ?19:12
tristanI could use someone with gitlab talents19:12
* tristan wants to cache the .coverage files and read them back in a later stage19:13
jjardon[m]tristan: add them to artifacts:19:26
jjardon[m]AFAIK, all the artifacts are passed by default between stages19:26
tristanjjardon[m], can I read the artifact generated from a previous stage in the next ?19:26
tristanWhere do I read it from ?19:26
tristanjust expect it to be there ?19:27
tristanjjardon[m], I was taking an approach of dumping them in the cache directories19:27
jjardon[m]tristan: yes https://docs.gitlab.com/ee/ci/yaml/#artifacts19:27
jjardon[m]and yes they are passed by default: https://docs.gitlab.com/ee/ci/yaml/#dependencies19:28
tristanjjardon[m], freaking perfect19:28
tristan:)19:28
*** bochecha_ has quit IRC21:40
*** tristan has quit IRC23:08

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