IRC logs for #baserock for Wednesday, 2016-12-07

*** locallycompact has quit IRC00:54
*** gtristan has quit IRC04:54
*** lchlan has quit IRC05:15
*** thrace_ has quit IRC05:16
*** thrace has joined #baserock05:17
*** lchlan has joined #baserock05:19
*** gtristan has joined #baserock05:45
jjardonlocallycompact nice. Time to add some ci to the spec repo as well?06:14
*** franred has joined #baserock07:13
*** gtristan_ has joined #baserock07:28
*** gtristan has quit IRC07:32
*** gtristan_ is now known as gtristan07:37
*** gtristan has quit IRC08:56
*** fay has joined #baserock08:57
*** fay is now known as Guest3600108:58
*** toscalix has joined #baserock09:01
*** jude_ has joined #baserock09:02
*** rdale has joined #baserock09:11
*** gtristan has joined #baserock09:28
*** jonathanmaw has joined #baserock09:44
*** locallycompact has joined #baserock09:49
*** locallycompact has quit IRC09:55
*** locallycompact has joined #baserock10:26
*** thinkl33t has left #baserock11:19
*** lachlan has quit IRC11:41
*** lachlan has joined #baserock11:43
*** lachlan has quit IRC11:52
jjardonlocallycompact paulsherwood can you review https://gitlab.com/baserock/ybd/merge_requests/278 ?11:55
locallycompactI approved that already11:56
jjardon:) thanks!11:56
* jjardon should have refreshed the browser tab before asking11:56
locallycompactjjardon, can you help me move some of this in https://gitlab.com/baserock/ybd/merge_requests/27711:56
*** lachlan_ has joined #baserock11:58
*** lachlan has joined #baserock11:59
gtristanThis relatively simple patch fixes sandbox.py's composition of the PATH env variable: https://gitlab.com/baserock/ybd/merge_requests/27912:32
*** franred has quit IRC13:06
paulsherwoodeek... looks like something's broken idempotence on the runners https://gitlab.com/baserock/ybd/builds/712589013:22
paulsherwoodcould that have been https://gitlab.com/baserock/ybd/merge_requests/278 by any chance?13:24
locallycompactThat's been happening anyway13:27
locallycompactSince at least a week13:27
locallycompactgit clone isn't idempotent anyway for some reason it isn't using clean dockers or something13:28
paulsherwoodjjardon: any idea what's changed to cause this?13:32
jjardonpaulsherwood: no idea13:38
jjardonpaulsherwood: can you aprove https://gitlab.com/baserock/ybd/merge_requests/270 ?13:40
jjardonpaulsherwood: nothing has changed in the runners configuration since at least 3 weeks13:42
jjardon(well, at least not by me)13:43
paulsherwoodjjardon: i've approved it, rebased13:53
jjardonlocallycompact: do you know what happen with this pipeline? https://gitlab.com/baserock/ybd/pipelines/525884613:57
jjardonseems its being running for the last 2 days?13:58
locallycompactjust ui bug13:59
locallycompactbuilds are finished14:00
paulsherwoodjjardon: https://gitlab.com/baserock/ybd/merge_requests/278 definitely broke things. shall we revert it, or fix the breakage?14:03
locallycompactgit clone definitions error is nothing to do with that commit, it's been happening for over a week14:06
paulsherwoodlocallycompact: can you point to an example? the fails i looked at were not this14:08
locallycompacttwo days ago in !275 https://gitlab.com/baserock/ybd/builds/702769514:09
locallycompactand probably every other merge of mine has at least one cache_keys_v1 failure14:09
locallycompactI am pressing this retry button until it goes away normally14:09
locallycompactbut it seems to be getting more frequent14:10
locallycompactthe docker image there already has everything from requirements.txt too so it's definitely being reused14:10
paulsherwoodhmmm. we must've changed *something* to cause this breakage14:10
locallycompactor maybe gitlab started reusing dockers from previous steps and wasn't previously14:12
locallycompactor we did that14:12
locallycompactbut I don't think we did anything like that14:12
pedroalvarezmaybe is reusing the state, but not the docker container14:13
pedroalvarez(you can mount folders inside the docker containers, and they will persist outside them once they are killed)14:13
locallycompactif it has site-packages then it might as well be mounting the whole system14:14
pedroalvarezyou have started using a different docker container, maybe that one  has some of the dependencies needed?14:15
*** gtristan has quit IRC14:16
locallycompactno normally looks like this https://gitlab.com/baserock/ybd/builds/712946314:16
jjardonpaulsherwood: we are not using the new docker image yet14:20
paulsherwoododd, then. i'm about to try a patch, anyway14:23
*** gtristan has joined #baserock14:43
paulsherwoodhttps://gitlab.com/baserock/ybd/commit/bc7482c1935fd21e611401431d1d88361606cb8414:44
paulsherwoodhttps://gitlab.com/baserock/ybd/merge_requests/28014:45
*** jude_ has quit IRC15:27
*** lachlan has quit IRC16:16
*** lachlan has joined #baserock16:27
jjardonpaulsherwood: seems that MR failed?16:38
*** rdale has quit IRC16:39
*** rdale has joined #baserock16:42
*** Guest36001 has quit IRC17:26
*** jonathanmaw has quit IRC18:00
*** toscalix has quit IRC18:19
*** locallycompact has quit IRC18:21
*** lachlan has quit IRC18:27
jjardonpaulsherwood: I think I found the problem: MaxBuilds should be 1 so only one build is done in a runner18:29
jjardonmaybe they have changed the default as we do not specify anything at the moment18:29
*** jude_ has joined #baserock19:00
*** lachlan has joined #baserock19:11
*** jude_ has quit IRC19:41
*** toscalix has joined #baserock21:22
*** toscalix has quit IRC22:07

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