Thursday, 2017-01-12

*** saneax is now known as saneax-_-|AFK00:00
*** saneax-_-|AFK is now known as saneax00:16
SpamapSjeblair: so.. the thing is, none of these changes are "dependent on one another", so all 3 merge builds do not contain the others.00:37
SpamapSjeblair: we could test that history has all 3 in it once everything is released I suppose.00:38
* SpamapS isn't sure why he used the scare quotes there :-P00:38
*** saneax is now known as saneax-_-|AFK00:46
*** jamielennox is now known as jamielennox|away00:50
*** Cibo_ has joined #zuul01:00
*** jamielennox|away is now known as jamielennox01:21
timrchavez_jeblair: Wish this was pure Python :) https://github.com/yaronn/blessed-contrib. Could add https://github.com/lovehandle/blessed-graphite and have a decent replacement for grafana :-)01:57
timrchavez_Maybe it's time to start chartty?01:57
*** Shuo has quit IRC02:05
mordredchartty++03:40
*** saneax-_-|AFK is now known as saneax05:01
*** Cibo_ has quit IRC05:08
*** bhavik1 has joined #zuul06:36
*** bhavik1 has quit IRC06:47
*** hashar has joined #zuul08:36
*** saneax is now known as saneax-_-|AFK10:55
*** saneax-_-|AFK is now known as saneax11:58
*** bhavik1 has joined #zuul12:45
*** bhavik1 has quit IRC13:17
*** yolanda has quit IRC13:18
*** yolanda has joined #zuul13:18
*** timrchavez_ is now known as timrc_13:55
*** saneax is now known as saneax-_-|AFK14:16
*** gundalow is now known as docbot15:42
*** docbot is now known as searchy15:43
*** searchy is now known as answerible15:43
*** answerible is now known as answerable15:43
*** answerable is now known as cowsay15:44
*** cowsay is now known as OptimusBot15:44
*** OptimusBot is now known as gundalow15:44
*** gundalow is now known as answerbot15:49
*** answerbot is now known as gundalow15:50
*** rbergeron has quit IRC16:14
*** dag_ has joined #zuul16:20
*** dag_ is now known as rbergeron16:21
*** rbergeron has quit IRC16:21
*** rbergeron has joined #zuul16:21
*** Cibo_ has joined #zuul16:38
*** Cibo_ has quit IRC16:43
jeblairSpamapS: yeah, it's (incidentally) testing sequencing.  they should share a change queue, and therefore, in the repo state produced by the launcher-merger, builds for C should contain A and B.  if they don't, then it's probably because we need to update the test configuration to manually specify that they share a change queue.16:48
jeblairSpamapS: (we automatically created shared change queues based on job names in v2, but need to manually specify them in v3)16:48
jeblairtimrc, timrc_: chartty :)16:48
*** hashar has quit IRC16:51
openstackgerritPaul Belanger proposed openstack-infra/nodepool: DNM: prepare for fedora-25  https://review.openstack.org/41955716:52
*** rmoe has quit IRC17:08
*** rmoe has joined #zuul17:09
SpamapSjeblair: hm. I feel like every time we talk I get schooled on something I don't understand about the data model.17:25
SpamapSjeblair: I thought gate automatically implied a shared change queue?17:26
jeblairSpamapS: this is where the terms 'pipeline' and 'queue' take on real importance -- the gate pipeline ('gate' isn't the special thing here, but the fact that we tell it to use the 'dependent' pipeline manager is) observes and automatically creates (based on the order equeued) dependencies between changes, but only within a shared change queue (that's model.ChangeQueue).17:29
jeblairSpamapS: in zuulv2, any projects which share a job get combined into a shared change queue.  (eg. "gate-tempest-dsvm" running on both nova and tempest cause both projects to share a change queue in the gate pipeline)17:30
jeblairSpamapS: in v3, since we're dropping unique job names per-project (in v2 we have gate-nova-python27, gate-zuul-python27 -- they are different, so no shared change queue; in v3, they will both run "python27") so we have to manually say "nova and tempest share a change queue"17:31
jeblairthat's done with the "queue:" attribute of the project-pipeline configuration;  git grep "queue: integrated" for examples17:32
SpamapSjeblair: ok, the bulb is getting les dim in my head. ;)17:34
SpamapSless17:34
jeblairfiat lux!17:34
SpamapSjeblair: thanks, I think that is, in fact, it.17:34
SpamapSles dim bulb, c'est bon.17:35
*** pabelanger has quit IRC17:41
*** pabelanger has joined #zuul17:41
*** Shuo has joined #zuul17:51
SpamapSjeblair: funny enough, a few other tests seem to have been re-worked to deal with the lack of ordering for org/project's gate17:54
jeblairSpamapS: hrm :( i think it might be good to correct this to match the old behavior (and if we want to change it, do that separately later, possibly while inventorying which tests cover things like shared change queue ordering)18:01
SpamapSjeblair: well, org/project2 is actually just org/project with the integrated gate18:03
SpamapSso changing the test to project2 works18:03
*** Shuo_ has joined #zuul18:10
*** bhavik1 has joined #zuul18:22
*** bhavik1 has quit IRC18:26
SpamapShm.. there may be a race or some kind of system dependence in test_success_url18:31
SpamapSit just failed for me.. then passed on retry18:31
*** openstackgerrit has quit IRC18:33
*** openstackgerrit has joined #zuul18:59
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/zuul: Re-enable test_build_configuration_branch  https://review.openstack.org/41960918:59
SpamapSjeblair: ^ the final product.18:59
*** pabelanger has quit IRC19:39
*** pabelanger has joined #zuul20:02
pabelangerjeblair: mind reviewing 412160 again? allows nodepool-builder to only build diskimages, if configured for that21:10
*** openstackgerrit has quit IRC21:18
*** Shuo has quit IRC22:18
*** Shuo_ is now known as Shuo22:18
SpamapSadam_g: argh! I duplicated your work23:01
*** openstackgerrit has joined #zuul23:02
openstackgerritJesse Keating proposed openstack-infra/zuul: Add trigger capability on github pr review  https://review.openstack.org/41968423:02
SpamapSadam_g: https://review.openstack.org/409376 revamps test_build_configuration_branch but you never claimed it :-/23:02
* SpamapS abandons https://review.openstack.org/41960923:03
adam_gSpamapS: i thought i had a task for it in sb, one sec23:03
adam_gSpamapS: yea, #344323:05
SpamapSnope23:05
SpamapS Clint Byrum created the task "test_build_configuration_branch" on 2017-01-10 at 19:56:4123:05
SpamapSadam_g: 3443 is only for the first one23:05
SpamapSerr no23:05
SpamapSadam_g: sorry you confused me23:06
SpamapSadam_g: I just reassigned 3443 to you23:06
SpamapSadam_g: the one you have is 3424, which is just for _conflict23:06
SpamapSadam_g: anyway, it's fine to touch 3 in one task, but if the name of the test isn't in the story's task list.. this happens. :-P23:07
adam_gSpamapS: ohh i see. my baddy.23:07
SpamapSit's ok I probably learned something23:07
SpamapSadam_g: I made a 3445 for _multi_branch too23:07
adam_gSpamapS: if you have a simple fix to get that one going, tho, maybe we can merge that instead? digging into that group of tests kinda got me stuck in the mud with other things23:08
SpamapSadam_g: see 419609 ... it does pass here locally. Not sure why it didn't pass on the check test23:12
*** saneax-_-|AFK is now known as saneax23:19
adam_gSpamapS: sure gimme a few. need to kick off an os install on desktop after a hard drive blew up23:23
adam_gSpamapS: so on my version of the test i was looking at the repos in the each builds JobDir23:50

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