Friday, 2015-01-30

jheskethclarkb: so, the ones it fails on appear to reach the retry limit00:00
clarkbmtreinish: so really probably what it needs is a non boolean flag that determines how far along you get00:00
*** abhirc has quit IRC00:00
clarkbmtreinish: no tempest is a service you have to enable00:00
*** abhirc has joined #openstack-infra00:00
jheskethclarkb: and during that time continues to fetch the same number of bytes each time00:00
clarkbjhesketh: yup00:00
jhesketh(rather than -c continuing the download)00:00
jheskethso I'm dubious this will fix it, but it's worth a try00:00
clarkbjhesketh: gotcha00:01
mtreinishclarkb: sure, but it's default on iirc. Where in d-g.sh does it set that in the localrc for devstack if it's needed?00:01
armaxmtreinish: posting a patch now to unveil the failure mode I was observing00:01
clarkbmtreinish: via features.yaml00:01
*** dimtruck is now known as zz_dimtruck00:01
clarkbwhich dumps tempest in enable_services00:01
fungiso so much teh scrollbax00:02
mtreinishclarkb: ah, that's what I was forgetting sry.00:03
*** dangers is now known as dangers_away00:03
mtreinishclarkb: was the comment about tempest-lib in response to my question regarding spinning up functional testing in projects?00:03
clarkbmtreinish: yes the only reason we have this problem is tempest is being misuesd00:04
mtreinisharmax: nah, I got what was missing now, clarkb showed me what I was missing00:04
clarkbthe black box integration tests are being used to also run functional tests00:04
clarkbwhich is probably expedient but I thought tempest-lib exists to solve this more properly00:04
armaxclarkb: my udnerstanding is that tempestlib is not quite there yet00:05
*** ddieterly has quit IRC00:05
*** oomichi has joined #openstack-infra00:05
fungimorganfainberg: you are single-handedly restoring my faith in humanity00:06
armaxmtreinish: ya, I think that was what I found out while playing with d-g, but I must have blanked out00:06
armax:)00:06
mtreinishclarkb: yeah tempest-lib is there for breaking out the common bits so people can write there own functional tests using the bits from tempest they need00:06
mtreinishbut it's still getting rolling, we haven't broken out all the common pieces yet. The only thing people can really do with it now is the cli testing00:07
clarkbI see00:07
mtreinish(which jogo and I started porting novaclient to use the other day)00:07
clarkbjogo: neutron aiopcpu does pass so its likely my networking changes that broke it00:07
clarkb(my networking changes are there just for nova network)00:07
fungiRyan_Lane: long time no ping! hope things are well00:08
Ryan_Lane:)00:08
Ryan_Lanethey are00:08
Ryan_LaneI got tired of waiting for barbican, so I'm writing a credential manager00:08
*** jgrimm is now known as zz_jgrimm00:09
*** fandi has quit IRC00:09
Ryan_Lanealas, it's pretty heavily based on AWS services00:09
*** mriedem has joined #openstack-infra00:10
*** amotoki has joined #openstack-infra00:10
fungieek00:10
Ryan_Lane:D00:10
Ryan_Lanethat said, it's all python! :D00:10
openstackgerritArmando Migliaccio proposed openstack-infra/devstack-gate: Add variable to allow Tempest to disable the test execution  https://review.openstack.org/14805200:10
openstackgerritKhai Do proposed openstack-infra/jenkins-job-builder: Allow JJB to set connection timeout to jenkins server  https://review.openstack.org/15147200:11
mtreinishclarkb: but tempest-lib probably won't solve the setup issue. People will probably still need to setup separate devstack envs to run the tests with00:11
fungiRyan_Lane: i hear barbican is coming along, at least. if you have done similar things now, they might appreciate reviews/patches ;)00:11
clarkbmtreinish: well tempest isn't solving that either00:11
clarkbmtreinish: the issue is literally need a working tmepest00:11
clarkbmtreinish: whcih tempest-lib will solve00:11
Ryan_Lanewell, I'm only a week into mine. the scope of mine is also much, much smaller00:11
*** craigbr has quit IRC00:11
*** ashleighfarnham has quit IRC00:11
openstackgerritMerged openstack-infra/puppet-project_config: Add missing LICENSE file  https://review.openstack.org/15144800:11
*** tteggel_ is now known as tteggel00:14
*** tteggel has quit IRC00:14
openstackgerritMerged openstack-infra/puppet-openstackid: Add missing LICENSE file  https://review.openstack.org/15144700:14
*** tteggel has joined #openstack-infra00:14
*** amotoki has quit IRC00:15
openstackgerritJeremy Stanley proposed openstack-infra/puppet-httpd: Add missing LICENSE file  https://review.openstack.org/15143200:16
*** jaypipes has joined #openstack-infra00:17
*** shivharis has joined #openstack-infra00:18
clarkbmtreinish: fungi I just approved the subunit worker revert. Once that gets in I will restart the workers00:18
*** tjones1 has quit IRC00:18
mtreinishclarkb: ok cool, thanks00:19
mtreinishIIRC we were only running one process before00:19
clarkbmtreinish: ok00:20
mtreinishI probably should update the policy to indicate that with 4 it seems to eat all the rams00:20
openstackgerritJeremy Stanley proposed openstack-infra/puppet-vcsrepo: Add missing LICENSE file  https://review.openstack.org/15145400:20
fungii'm not sure that ^ one actually makes sense to approve00:20
openstackgerritMerged openstack-infra/puppet-yum: Add missing LICENSE file  https://review.openstack.org/15145600:21
fungithe original puppet-vcsrepo seems to be gpl v2+ not apl200:21
clarkbfungi: there was a conversation that happened where puppet said "oh ya thats the wrong license"00:21
* mtreinish flight's landing, is out for the night00:21
clarkbfungi: I think it was just missed when they did their mass relicensing00:21
openstackgerritMerged openstack-infra/puppet-storyboard: Add missing LICENSE file  https://review.openstack.org/15145200:22
openstackgerritMerged openstack-infra/puppet-redis: Add missing LICENSE file  https://review.openstack.org/15144900:22
*** mwagner_lap has joined #openstack-infra00:23
openstackgerritMerged openstack-infra/puppet-subunit2sql: Revert "Add support to the subunit workers to reuse zuul uuids"  https://review.openstack.org/15113300:25
*** fandi has joined #openstack-infra00:25
*** sabeen has quit IRC00:27
*** otter768 has joined #openstack-infra00:27
fungiclarkb: 151455 isn't going to pass without a rakefile. i'll add one00:28
openstackgerritMerged openstack-infra/puppet-ssh: Add missing LICENSE file  https://review.openstack.org/15145100:28
openstackgerritJeremy Stanley proposed openstack-infra/puppet-vinz: Add missing LICENSE file  https://review.openstack.org/15145500:29
clarkbfungi: sorry I am mostly mindlessly going "license file? recheck"00:29
fungiheh00:29
clarkbI thought I could get away with that00:29
jeblairclarkb, fungi, jhesketh: http://paste.openstack.org/show/163891/00:29
openstackgerritIlya Alekseyev (irc:adiantum) proposed openstack-infra/system-config: Add list for discussions in russian openstack-ru  https://review.openstack.org/15148300:29
*** crc32 has quit IRC00:30
jeblairclarkb, fungi, jhesketh: given that git tree structure, if zuul enqueued the * change, which order do you think is correct?00:30
fungiclarkb: keep in mind that i added them to all openstack-infra/puppet-.* projects even if they predated the module split, so some (like puppet-vinz) might be plain broken00:30
*** david-lyle is now known as david-lyle_afk00:31
jheskethjeblair: by git tree structure do you mean dependencies?00:31
jeblairjhesketh: yep00:31
jheskethie either through gerrit or the depends-on commit msg00:31
jeblairjhesketh: yeah, either one, though i don't have more than one parent here, so it's safe to assume just git structure for now00:31
fungijeblair: 1 and 2 are in a dependent pipeline together? if so, how?00:31
jheskethjeblair: surely the two scenarios there are the same except with different names?00:32
jeblairclarkb, fungi, jhesketh: (2) is the shortest path to getting C morged, but (1) keeps the children closest to their parents00:32
jeblairfungi: 1 and 2 are alternate scenarios00:32
fungiaha00:32
clarkbjeblair: I would do a breadth first search00:32
*** otter768 has quit IRC00:32
*** dims has quit IRC00:32
clarkbjeblair: because that should drop out merge conflicts earlier?00:32
clarkbor maybe it will make more merge conflicts00:32
fungii still not understanding the question. in 1 the * change is D and in 2 the * change is C. each has its own parentage and so depending on whether you enqueued D or C you would use the history associated with that commit00:33
jheskethjeblair: I'm confused by the two scenarios... were you meant to * C in both of them?00:33
jeblairclarkb: i _think_ 2 reduces potential for merge conflicts for the actual change added.  of course, this is actually a bit weird because it's very hard to get into that state since you have to ignore events in order to do it00:34
*** jaypipes has quit IRC00:34
jeblairjhesketh: nope, enqueuing the commit in the same position regardless -- only the _outcome_ of the linearization changes in the two scenarios00:34
jeblairfungi: yeah ^ the same commit is being enqueued both times -- i'm just asking which ordered outcome is correct00:35
jeblairfungi: (the zuul queue is ABCDEFG in both cases, so you see which commits those point to changes)00:35
openstackgerritMerged openstack-infra/puppet-tmpreaper: Add puppet module files for tmpreaper  https://review.openstack.org/15146900:35
*** waynr has quit IRC00:36
*** fandi has quit IRC00:36
fungijeblair: i assumed each of a, b, c, d, e, f and g would just be cherry-picked onto the branch tip in series in a dependent pipeline in that case00:36
jeblairi'm starting to think that (1) is better because it matches what you would get if you enqueued A, then B, then D00:36
fungijeblair: so the history would look like neither of those and would just be linear00:36
*** zz_dimtruck is now known as dimtruck00:37
openstackgerritMerged openstack-infra/puppet-subunit2sql: Add missing LICENSE file  https://review.openstack.org/15145300:37
jeblairfungi: yes it would, but the order of the commits is different00:37
fungii'm very clearly missing a key part of the problem statement00:37
jeblairfungi: let me redraw00:37
jheskethyeah I'm not sure I follow either, but I just thought that was me being silly00:37
fungii thought zuul enqueued them chronologically in trigger event order00:37
fungioh, though i guess it does skip events and pull changes in immediately for gerrit dependencies00:38
marunwho to talk to about storyboard?00:38
jeblairfungi: yep.  easiest way to imagine this is a cold start with approvals on everything.  then someone adds C or D00:38
jeblairfungi, clarkb, jhesketh: http://paste.openstack.org/show/163892/00:38
*** derekh has joined #openstack-infra00:39
*** salv-orlando has quit IRC00:39
*** salv-orlando has joined #openstack-infra00:39
jeblairmarun: i'm happy to talk about it, or may further direct your call as needed :)00:39
marunjhesketh: thanks for the review on skip-if. I'll have a patch up shortly with scenario testing working.00:39
clarkbjeblair: ya so its basically breadth first or depth first after B00:39
*** tjones1 has joined #openstack-infra00:39
*** markmcclain has joined #openstack-infra00:39
jheskethmarun: no trouble, it looks cool. I'd also like to see it as an equeue-if scenario too, (replacing the file match thingy) but we can figure that out later00:40
clarkbjeblair: my initial reaction is still to go with C to cut off long patch chains early rather than late00:40
*** wenlock has quit IRC00:40
fungijeblair: ahh, got it, so green field no events zuul constructing a test order out of the blue00:40
*** markmcclain has quit IRC00:40
jeblairclarkb: you mean (1)00:40
clarkbjeblair: yes00:40
jeblairfungi: yep, but starting with D as the triggering change00:40
marunjeblair: I think there is a real need to be able to manage reviews in a similar way to managing the bugs and blueprints they relate to.00:40
marunjeblair: i.e. assignment, priority, etc.00:41
fungijeblair: this is to work out the enqueueing order for cross-repo dependencies i guess?00:41
marunjeblair: I've always thought of this as its own thing, but I think that's me accepting the status quo that is launchpad.00:41
jheskethjeblair: if D is the one to be queued, I would expect the test order to be ABD (ie #2 scenario)00:41
jheskethbecause if the right side of the tree has more commits, where would you stop?00:42
jheskethas far as gerrit is concerned, dependencies are rather linear00:42
marunjeblair: I'd like to take a look at where storyboard is at with an eye to seeing if adding this sort of capability makes sense.00:42
jeblairfungi: actually i'm just trying to correct the current situation with single repos00:42
*** tmcpeak has quit IRC00:42
jeblairjhesketh: i thought so too at first, but then when i think about the fact that if you re-enqueued everything, you might get a different order, i am being swayed toward (1)00:43
fungijeblair: i think i'm with clark on 1 mainly because that increases the chances that changes closest to the branch tip will make it in earlier, regardless of the fact that it lengthens the change series ahead of the triggering change00:43
openstackgerritMerged openstack-infra/puppet-elastic_recheck: Add puppet module files for elastic_recheck  https://review.openstack.org/15146000:43
clarkbjhesketh: (1) should avoid the infinite depth problem too00:43
jeblairmarun: https://wiki.openstack.org/wiki/StoryBoard https://wiki.openstack.org/wiki/StoryBoard/Vision https://wiki.openstack.org/wiki/StoryBoard/Roadmap00:44
clarkbsince in my head its breadth first00:44
marunjeblair: awesome, thank you!00:44
jheskethjeblair: I'm not sure how you'd get a different order on re-enqueue unless something has merged in the mean time. If that's the case then it'll be a different scenario anyway00:44
jeblairmarun: so i think we want very good tie in with storyboard and gerrit; i think we are expecting a story task for every commit in gerrit that links to a story00:44
*** signed8bit is now known as signed8bit_ZZZzz00:44
openstackgerritShiv Haris proposed openstack-infra/project-config: As a next step: Adding <project>.config file  https://review.openstack.org/15148700:44
jheskethfungi: I do think that point is valid though (re easier to get changes up the branch)00:45
marunjeblair: neat00:45
jeblairmarun: i don't think that storyboard will take on being an alternate interface to gerrit, but i think there will probably be enough data and cross-linking to help with that00:45
jheskethbut what about where we have a long set of patches relating to the one issue. Wouldn't a depth first make more sense from the patch submitters point of view?00:45
openstackgerritMerged openstack-infra/puppet-zanata: Add missing LICENSE file  https://review.openstack.org/15145700:45
*** annegent_ has joined #openstack-infra00:45
shivharisfungi: hi00:45
*** spandhe has quit IRC00:46
openstackgerritMerged openstack-infra/puppet-asterisk: Add missing LICENSE file  https://review.openstack.org/15142000:46
openstackgerritMerged openstack-infra/puppet-nodepool: Add missing LICENSE file  https://review.openstack.org/15144500:46
jeblairmarun: krotscheck and ttx are other big-picture thinkers on storyboard; the infra or dev lists are good for discussion, and there's an irc meeting00:47
fungishivharis: what's broken?00:48
marunjeblair: cool00:48
shivharisfungi: nothing, just that I was asked to contact infra team when I am ready to add core members to the project00:49
marunjeblair: You've given me what I'll need to engage, thank you.00:49
jeblairjhesketh: if you had (2), and then cleared the queue and re-enqueued everything in queue order, you'd end up with (1)00:49
jeblairjhesketh: (because you would enqueue A to start with instead of D)00:49
jheskethjeblair: so maybe we need to maintain the order when re-equeuing?00:50
jeblairjhesketh: i think it's easier to just make it stable and go with the algorithm that produces (1)00:50
shivharisfungi: i need to add a group networking-brocade-core group, i have the corresponding .config file ready00:50
jheskethjeblair: sure, that's probably true00:51
jeblair(which is, incidentally, what we would usually see anyway since the only way to get (2) is to miss events)00:51
shivharisfungi: patchset https://review.openstack.org/15148700:51
jheskethI don't think I fully understand the consequences of re-enqueuing, but I think there are enough reasons to do #200:51
jhesketh*#100:51
*** bdpayne has quit IRC00:52
*** wuhg has joined #openstack-infra00:52
fungishivharis: i've added you as the initial member of the two groups created by that change. you should be able to make further membership edits to those as needed00:52
shivharisfungi: great, thanks much, doing this for the first time ever so treading very carefully00:52
shivharisfungi: bug thanks00:53
shivharisfungi: s/bug/big/00:53
jeblairclarkb, fungi, jhesketh: thanks!00:53
fungishivharis: for some reason, networking-brocade-core and networking-brocade-release groups already exist00:54
fungishivharis: even though that change you linked hasn't merged yet00:54
fungilooking into it00:54
shivharisfungi: surprising, who happens to be in the group00:54
fungishivharis: they were empty00:55
jeblairmarun, jhesketh: i haven't had a chance to do a real review yet, but just letting it rattle around in my brain, i have a small concern that we may find it hard to control the behavior of skip-if across a large number of projects...(i will elaborate)00:55
fungishivharis: oh, there's an existing stackforge/networking-brocade project00:55
marunjeblair: maybe we need to mandate the project field?00:55
jeblairmarun, jhesketh: since it applies to a job, and we use, say, 'gate-tempest-dsvm' on many projects, if we applied skip-if to that, then it might skip the run if someone made changes only to files in the tests/ directory in tempest00:56
openstackgerritMerged openstack-infra/puppet-bup: Add puppet module files for bup  https://review.openstack.org/15145900:56
*** bradjones has quit IRC00:56
shivharisfungi: that project was merged three days ago, I am adding a new file to it, gerrit/acls/...00:56
*** carl_baldwin has quit IRC00:56
fungishivharis: did you mean to edit the acl for that project rather than adding one for openstack/networking-brocade (which doesn't seem to be a project)?00:56
jeblair(i don't know offhand what the dir structure of tempest is, but one might imagine that the intention to not run jobs on neutron unit test changes might conflict with the tempest dir structure)00:57
marunjeblair: my intent was that the scope be limited based on how dangerous the potential filter could be.00:57
jheskethjeblair: interesting thought... I might have to rethink on the re-review, but I think it's avoidable by tightening the filter params... will look at it later00:57
marunjeblair: so anything that wasn't universal (and enforced) would have to be project-specific00:57
marunjeblair: e.g. x/tests/.* is always safe to skip tempest jobs on00:58
shivharisi meant to add the file to the existing project at stackforge/networking-brocade00:58
jeblairmarun: yeah, and i don't have fully-formed thoughts around this yet -- after all, the existing files filter has the same problem, though it's a bit easier if you're saying "only run this on a requirements.txt change"; it's inverting that where it gets a little more vague00:58
shivharisfungi: i meant to add the file to the existing project at stackforge/networking-brocade00:58
clarkbfyi I am going to rerun the aipcpu experimental jobs and hold the nodes running the nova networking test. Grepping through logs I don't see why my shift of the IPs would break so going to manually investigate.00:58
jeblairmarun: why is "x/tests/.*" always safe?00:58
*** Sukhdev has quit IRC00:58
marunjeblair: I made the feature generic because it made for a cleaner implementation, but I didn't intend for its use to be unlimited.00:58
jeblair(is x standing for something important?)00:59
fungishivharis: you'll want to update your change to adjust the correct acl then00:59
marunjeblair: x -> project under test00:59
*** annegent_ has quit IRC00:59
*** dannywilson has quit IRC00:59
*** bradjones has joined #openstack-infra00:59
marunjeblair: I just mean that unless a certain path means the same thing in every project targeted by a job, we can't skip on it.00:59
jeblairmarun: ok, yeah, makes sense.00:59
marunjeblair: without adding a project specifier, anyway01:00
*** fifieldt has quit IRC01:00
jeblairmarun: indeed -- that was the thought i was having.  i'm glad we share it.01:00
jeblairmarun: i don't have anything else to add right now, but we've achieved a shared context, so that will be helpful.  :)01:00
marunjeblair: I'm happy to incorporate that kind of limitation into the feature if you don't think that a convention could be safely enforced.01:00
shivharisfungi: ok will fix it01:00
marunjeblair: cool :)01:01
*** avozza is now known as zz_avozza01:05
openstackgerritMerged openstack-infra/puppet-meetbot: Add missing LICENSE file  https://review.openstack.org/15144301:07
openstackgerritMerged openstack-infra/puppet-statusbot: Add puppet module files for Statusbot  https://review.openstack.org/15146501:08
openstackgerritMerged openstack-infra/puppet-sudoers: Add puppet module files for sudoers  https://review.openstack.org/15146701:09
openstackgerritShiv Haris proposed openstack-infra/project-config: File updated gerrit/acls/stackforge/networking-brocade.config  https://review.openstack.org/15148701:10
*** derekh has quit IRC01:10
*** armax has quit IRC01:12
*** zhiwei has joined #openstack-infra01:13
fungiso as asselin_ said, we01:16
*** ddieterly has joined #openstack-infra01:16
fungi're now just missing polish commits on puppet-ulimit, unattended_upgrades, unbound, user and zuul01:17
*** amitgandhinz has joined #openstack-infra01:17
*** adalbas has quit IRC01:18
*** hdd has quit IRC01:18
*** SumitNaiksatam has quit IRC01:19
*** Sukhdev has joined #openstack-infra01:19
*** achanda_ has joined #openstack-infra01:19
*** ddieterly has quit IRC01:21
*** achanda has quit IRC01:23
clarkbok I can pingthe 172.24.5.1 floating ip given to a vm01:24
*** achanda_ has quit IRC01:24
clarkbwill let tempest get further along before I much with it too much01:24
*** zz_avozza is now known as avozza01:26
clarkbfungi: watch it will be a security group thing where tempest assumes 172.24.4.0/24 is the range to open01:26
fungihah01:26
clarkbhrm shelve instance just failed we shall wait to see why01:26
clarkbit may be less work overall to just rewrite my comments from scratch if thats the case01:27
clarkbsince 172.24.4.0/24 seems fairly embedded01:27
openstackgerritMerged openstack-infra/puppet-snmpd: Add missing LICENSE file  https://review.openstack.org/15145001:27
fungiit's unfortunate that there's not just one variable somewhere all other uses consult01:27
*** amitgandhinz has quit IRC01:28
clarkbya another ssh time out so at least its consistent now I can work on debugging it01:28
fungii think i'm going to knock off for the night, but enjoy your evening west-coasters01:31
*** tiswanso has joined #openstack-infra01:32
*** tiswanso has quit IRC01:34
*** markvoelker_ has quit IRC01:34
*** tiswanso has joined #openstack-infra01:35
*** markvoelker has joined #openstack-infra01:35
clarkbhrm there are no rules in the default security group01:36
*** mtanino has quit IRC01:39
*** hdd has joined #openstack-infra01:39
*** yamahata has joined #openstack-infra01:39
clarkbhrm I can't even ping the addr now01:40
clarkbjogo: ^ possibly tempest tests regressed?01:40
*** markvoelker has quit IRC01:40
*** ayoung has joined #openstack-infra01:41
*** tjones1 has quit IRC01:42
clarkbok tempest cleans up all of its security groups (probably a good thing) which makes it slightly hard to review them after the fact01:44
*** whayutin_ has quit IRC01:44
*** annegent_ has joined #openstack-infra01:45
*** ayoung has quit IRC01:46
jogoclarkb: very possible something regressed01:47
jogoclarkb: which job?01:47
clarkbjogo: nova net aiopcpu01:47
clarkbjogo: the shelve server test fails due to an ssh client timeout01:47
clarkbjogo: I am on the two nodes post tempest fail. I have booted a node, added a floating ip to it, updated security group rules, and both ping + ssh work01:47
clarkbso I have at least convinced my self that the base set of netowrking is sound. Going to try replicating what the test does now01:48
*** ChuckC_ has quit IRC01:48
jogoclarkb: not a regression, its a known issue01:48
jogoclarkb: the shelve issue is:01:48
clarkbjogo: ok? does that mean we cannot merge aiopcpu right now?01:48
jogoshelve instance on machine A, if unshelve moves instance to machine B it fails01:49
jogoelse success01:49
*** ChuckC_ has joined #openstack-infra01:49
jogoclarkb: so its a race condition. We cannot make this job voting anywhere01:49
clarkbI see01:49
clarkbwhy don't we stop testing shelve if its broken01:49
jogobut we can get the basic bits in place to make this experimental on tempest and nova so we can better debug01:49
clarkbalso why can't we make it voting?01:50
jogoclarkb: that is an option, I was hoping to try fixing it first01:50
clarkbthe whole point of this exercise is to have the test actually run and provide feedback01:50
jogoclarkb: well we haven't run this job enough to see how stable it is01:50
jogoso first step non-voting experimental on a few repos01:50
jogoand go from there01:50
clarkbsure but to flat out say we can't make it voting anywhere is premature01:50
clarkbits already experimental now01:51
clarkbI want to change the status qu01:51
jogoclarkb: well its only experimental on devstack-gate right?01:51
*** annegent_ has quit IRC01:51
marunjhesketh: can you please help me interpret the following?01:51
marunjhesketh: <Completed build, result: SUCCESS name: project-test-skip-if #21 changes: 5,2 6,2>01:51
clarkbjogo: I think it may also be on devstack01:51
clarkbjogo: I would have to check since most of the owrk so far has been on d-g01:51
jogoclarkb: actually come to think of it, enough patches have landed in the d-g patch series to get this to pass most of the time on master d-g01:52
clarkbjogo: but the stability won't change if you move it to devstack/nova/etc experimental...01:52
jogoit wouldn't have multihost though01:52
clarkbcorrect01:52
marunjhesketh: I'm not expecting that job to run on 5,2, only on 6,2, and I'm confused as to why 5,2 is listed in that message.01:52
clarkbbut ya everything but multihost should work (that and things like migration testing which come after multihost)01:52
jogoright, but we can see how specific patches work in multi-node (fixing unshelve)01:52
jogoclarkb: but I think you are right, lets disable the shelve test until its fixed and move on01:52
clarkbjogo: is shelve broken because the host moves but doesn't rearp?01:53
marunjeblair: a question regarding zuul job status messages? ^01:53
clarkbjogo: or maybe the host moves and the floating ip stays on the old hypervisor01:53
jogoclarkb: not sure yet, it only happens on nova-net and not neutron01:53
clarkbjogo: let me try to reproduce on this setup. just shelve unshelve and ping/ssh until it breaks?01:53
jogoclarkb: yup, that should do it01:54
clarkbjogo: can do01:54
jogoclarkb: thanks01:54
*** yamahata has quit IRC01:54
jogoclarkb: on a unrelated note - if we pin stable/juno to the pip-freeze of dsvm-tempest grenade fails, if we pin stable/juno to the pip-freeze of grenade, dsvm-tempest fails01:54
jogobecause the two pip-freezes are different01:55
clarkbjogo: wow01:55
jogoclarkb: isn't that fun :)01:55
clarkbjogo: ok reproduced. However its a little different than you describe. it unshelved onto the same host01:57
*** amotoki has joined #openstack-infra01:59
*** mohitsharma has joined #openstack-infra01:59
clarkbso now I am trying to see if the NAT rules are wrong (maybe it unshelved with a different kvm assigned ip?)01:59
*** bradjones has quit IRC02:00
clarkbjogo: ya ok, hitting the private ip works but not the floating ip however iptables shows nat rules that look correct to me02:01
clarkbtime for tcpdump02:01
*** bradjones has joined #openstack-infra02:01
*** ayoung has joined #openstack-infra02:01
*** amotoki has quit IRC02:03
*** packet has quit IRC02:07
*** shivharis has quit IRC02:08
clarkbjogo: OHHH I see maybe02:09
clarkbjogo: the floating ip is being added to the other node (the one doing the pinging) but the floating ip is still on the other node where the vm is running02:10
*** hdd has quit IRC02:10
jogoso floating ip is on both nodes?02:10
clarkbjogo: ya that is what I was seeing02:11
*** abhirc has quit IRC02:11
*** salv-orlando has quit IRC02:12
jogoand where is the actual VM02:12
clarkblooks like the nat rule ends up on both nodes too02:13
*** jerryz has joined #openstack-infra02:13
clarkbbut since we are multihost the routing needs to happen on the compute node with the VM I think02:13
*** signed8bit_ZZZzz is now known as signed8bit02:13
clarkbjogo: two node install A and B. VM is running on B. shelve unshelve a couple times, VM continues to run on B but A gets the floating ip and nat rules02:13
clarkbjogo: pings from A to floating ip fail. pings from B to floating ip succeed02:14
jogoclarkb: here is the bug I filed https://bugs.launchpad.net/nova/+bug/140406302:14
*** fandi has joined #openstack-infra02:14
clarkbjogo: I will update with what I have found02:14
jogoclarkb: thanks02:14
marunfungi: I'm having trouble interpreting a zuul build status, can you help?02:15
jogoso it sounds like we aren't cleaning something up correctly02:15
marunfungi: <Completed build, result: SUCCESS name: project-test-skip-if #21 changes: 5,2 6,2>02:15
*** homeless1 has quit IRC02:15
jogoclarkb: hmm https://bugs.launchpad.net/nova/+bug/129063502:16
*** asettle has quit IRC02:16
marunfungi: what does the inclusion of both 5,2 and 6,2 mean?  In the configuration I'm working with, the 5,2 change should not have run the project-test-skip-if job02:16
*** asettle has joined #openstack-infra02:16
jogoclarkb: or this https://bugs.launchpad.net/nova/+bug/135070402:16
clarkbjogo: that second one appears to be the issue02:18
jogoclarkb: yeah and there is a patch up too :)02:18
clarkbjogo: let me get what I have into your first bug then you can link/merge/mark duplicate whatever is most appropriate02:18
jogosounds good02:18
jogowant to try the fix as well?02:18
clarkbjogo: maybe? might be interesting to dig into nova but not sure where to start02:18
jogoclarkb: here is the fix https://review.openstack.org/#/c/110869/2402:19
*** _amrith_ is now known as amrith02:19
clarkbjogo: your first bug is updated02:21
jogoclarkb: thanks02:22
*** dmsimard_away is now known as dmsimard02:22
jogoclarkb: ahh that does sound like a duplicate of https://bugs.launchpad.net/nova/+bug/1350704 which has a proposed fix: https://review.openstack.org/#/c/110869/24/nova/compute/manager.py,cm02:23
*** dimtruck is now known as zz_dimtruck02:23
jheskethmarun: sorry was out for lunch... the two changes are possibly because one is dependant on the other02:24
marunjhesketh: it's race-y though02:25
marunjhesketh: or do you mean there is a merge dependency?02:25
marunjhesketh: I can trigger the problem every 5 test runs or so02:25
jheskethmerge dependency... not sure what is race0y02:25
marunjhesketh: I'll upload the change and you can see what I mean.02:25
jheskethmarun: is this as a unit test?02:25
jheskethokay :-)02:25
marunjhesketh: scenario test, as requested.02:26
*** ddieterly has joined #openstack-infra02:28
*** otter768 has joined #openstack-infra02:29
clarkbjogo: looks like Alex Xu is still working the fix? updated pretty recently02:29
*** mohitsharma has quit IRC02:30
*** ddieterly has quit IRC02:32
*** ddieterly has joined #openstack-infra02:33
*** otter768 has quit IRC02:33
*** baoli has joined #openstack-infra02:34
*** dmsimard is now known as dmsimard_away02:34
*** achuprin_ has quit IRC02:35
openstackgerritYAMAMOTO Takashi proposed openstack-infra/project-config: Add networking-ofagent-ci group  https://review.openstack.org/15151202:35
openstackgerritMaru Newby proposed openstack-infra/zuul: Add support for a skip-if filter on jobs  https://review.openstack.org/14676202:35
marunjhesketh: sorry for delay: https://review.openstack.org/#/c/146762/02:36
openstackgerritYAMAMOTO Takashi proposed openstack-infra/project-config: Add networking-ofagent-ci group  https://review.openstack.org/15151202:36
*** yamamot__ has joined #openstack-infra02:36
*** Kerralissa has quit IRC02:36
marunjhesketh: line 1847 in https://review.openstack.org/#/c/146762/11/tests/test_scheduler.py02:36
marunjhesketh: I was seeing periodic test failures previous to implementing the workaround described there.02:37
*** Rockyg has joined #openstack-infra02:38
*** baoli has quit IRC02:39
jogoclarkb: I ran a recheck02:40
*** mohitsharma has joined #openstack-infra02:40
*** melwitt has quit IRC02:41
*** amotoki has joined #openstack-infra02:42
*** amotoki has quit IRC02:42
jheskethmarun: as in the failures happen in master?02:43
marunjhesketh: master in zuul?02:44
marunjhesketh: no, this is local to my test02:44
marunjhesketh: As per the comment, I need a way to detect whether the skip-configured job ran for a given change.02:44
jheskethoh, I mis-read your explaination02:45
marunjhesketh: The only example of this that I could see was looking at the changes attached to the job status message02:45
marunjhesketh: job status -> build status (I guess?)02:45
marunjhesketh: and sometimes the build status references both a job that should have been skipped and one that should have ran it.02:46
*** annegent_ has joined #openstack-infra02:46
marunjhesketh: hence my question of merge dependency, since I am assuming that the inclusion of the preceding change id is because of something related to merge order rather than an indication that the job ran for both changes02:47
*** amrith is now known as _amrith_02:48
*** rwsu is now known as rwsu-afk02:48
jheskethmarun: hmm, so basically they are entering the gate pipeline in an inconsistent order?02:50
marunjhesketh: I'm adding them in the same order every time.  I can only assume that there is some kind of concurrent processing going on that introduces the variability.02:50
marunjhesketh: certainly the skip-if matching has no choice but to be deterministic.02:51
*** annegent_ has quit IRC02:51
*** achuprin_ has joined #openstack-infra02:52
jheskethmarun: I mean the scheduler is queuing them inconsistently ?02:53
marunjhesketh: Maybe?  I'm pretty sure I know enough to answer that definitively.02:53
marunjhesketh: Do you see anything strange in the test?02:54
marunjhesketh: I cribbed from other similar tests.02:54
jheskethnope, it looks good to me02:55
marunjhesketh: oops, I meant that I *don't* know enough to answer that definitively02:55
jheskethI suspect the race is in the queue processing... if you put the queue in hold mode, queue them all and then release them all at once it might be more deterministic02:56
jheskethlook for a test where  self.worker.hold_jobs_in_build is set02:57
marunjhesketh: but what does it mean when 2 changes are included for a built status message?02:57
marunjhesketh: I don't really care about the ordering, I just need to figure out which changes a job ran for.02:57
marunbuilt -> build02:58
*** harlowja_ is now known as harlowja_away02:58
jheskethmarun: I think it's because it's a dependant pipeline. You're queuing 6 changes (ABCDEF) so for change C the tests are ran with master -> A -> B then C applied02:59
jheskethit's built into the change so that cross project jobs can pull the correct order03:00
marunjhesketh: so does that imply that the only change that was actually tested was the last one in the list of changes on a build status?03:00
*** bradjones has quit IRC03:00
marunjhesketh: and that preceding changes are simply dependencies of the change the job ran against?03:00
marunjhesketh: that's what the code assumes right now, because that seemed the most likely conclusion03:01
jheskethyes03:01
jheskethalthough each other change will also have gone through... so E would have been tested with master -> ABCDE applied03:01
*** bradjones has joined #openstack-infra03:02
marunjhesketh: so why is the previous change included in some build statuses and not others?03:03
openstackgerritMaru Newby proposed openstack-infra/zuul: Add support for a skip-if filter on jobs  https://review.openstack.org/14676203:04
jheskethmarun: it's possible that the build is finishing and being merged into master03:04
jheskethso if A and B have merged, when testing E it'd look like master -> CDE03:04
jheskethwhich is why if you put it in hold when queuing the changes it should be more consistent03:05
marunjhesketh: I'll take a look at how to do that03:05
jheskethkeep in mind I may be wrong ;p03:05
*** unicell has quit IRC03:06
marunjhesketh: worth a try, at least :)03:08
marunjhesketh: I'm not a huge fan of this kind of scenario testing, tbh03:08
openstackgerritRamy Asselin proposed openstack-infra/puppet-ulimit: Add puppet module files for puppet-ulimit  https://review.openstack.org/15151403:08
marunjhesketh: too many moving pieces to learn and to go wrong, makes it hard to isolate the failure domain03:08
marunjhesketh: but I'll do what's asked :)03:09
*** SumitNaiksatam has joined #openstack-infra03:09
*** SumitNaiksatam has quit IRC03:09
*** Rockyg has quit IRC03:09
*** SumitNaiksatam has joined #openstack-infra03:10
jheskethmarun: intresting, because I am a fan.. when zuul is used in the wild all these moving pieces can go wrong so it's good to test them03:11
jheskethgranted, it's a learning curve to write the tests, but I think it's worth it03:11
marunjhesketh: it's good to be able to do this kind of testing, don't get me wrong03:11
marunjhesketh: but I don't think it should be necessary to test more that the change matchers work in concert with everything else03:12
openstackgerritRamy Asselin proposed openstack-infra/puppet-unbound: Add puppet module files for unbound  https://review.openstack.org/15151603:12
*** hdd has joined #openstack-infra03:12
marunjhesketh: it should be possible to isolate all specific testing of the change matchers, without involving other machinery03:12
jheskethmarun: I think this is how you uncover edge and race conditions to a certain extent though.. In an ideal world I would agree with you03:13
marunjhesketh: I had validated that job.changeMatches interacted correctly with the skip-if matcher, that the matcher could be constructed from configuration by the scheduler, and that the matchers worked.03:13
marunjhesketh: all of that in unit tests03:13
marunjhesketh: knowing that the existing tests are calling the job.changeMatches function, I know that everything below that would work properly.03:13
marunjhesketh: without that kind of separability for testing, the effort required for full coverage is exponential03:14
jheskethwell in this case you're doing both so it's even better ;-)03:14
jheskethbut I hear you, it's a burden03:14
marunjhesketh: I think the term 'test pyramid' accurately describes my philosophy of testing.03:15
marunjhesketh: validate everything at as low a level as feasible, because the cost is less.  low -> unit, high-> integration, and everything in between.03:16
* marun digresses03:16
jheskethI think your points are fair and I don't necessarily disagree03:17
openstackgerritRamy Asselin proposed openstack-infra/puppet-user: Add puppet module files for puppet-user  https://review.openstack.org/15151703:17
jhesketh:-)03:17
*** KanagarajM has joined #openstack-infra03:17
asselin_one more!03:18
marunjhesketh: :)03:18
*** Sukhdev has quit IRC03:19
marunjhesketh: your suggestion to use hold_jobs_in_build seems to be a good approach.03:19
marunjhesketh: I'm seeing consistent behavior at last.03:19
*** xyang0 has joined #openstack-infra03:19
marunjhesketh: ah crap, spoke too soon03:19
marunjhesketh: I guess the work-around stays for now03:20
marunjhesketh: maybe jeblair can figure it out.03:20
clarkbwhy not run tests in an independent pipeline?03:20
jhesketh:-(03:20
*** otter768 has joined #openstack-infra03:20
marunclarkb: https://review.openstack.org/#/c/146762/12/tests/test_scheduler.py03:21
openstackgerritRamy Asselin proposed openstack-infra/puppet-unattended_upgrades: Add puppet module files for unattended_upgrades  https://review.openstack.org/15151903:22
marunclarkb: test_skip_if_jobs is just trying to verify that jobs are skipped as expected.03:22
clarkbya instead of approving the changes just do patchset created events03:22
*** annegent_ has joined #openstack-infra03:22
clarkbthen each change is tested independently03:22
asselin_^^ Last one! :)03:22
marunclarkb: do you have a pointer to an example?03:23
* asselin_ going to eat03:23
marunclarkb: oh wait03:23
marunclarkb:  you mean run them in the check queue?03:23
clarkbyes03:23
marunclarkb: because I don't care about merging, just the job status03:23
marunclarkb: gotcha.  serves me right for cribbing to hard03:23
*** Ryan_Lane has quit IRC03:23
marunclarkb: so what command triggers the check queue?03:24
clarkbits a patchset created event should be part of create change if you just drop the approval events03:24
marunclarkb: ah, ok03:25
*** rkukura has quit IRC03:26
*** annegent_ has quit IRC03:27
*** sdake has joined #openstack-infra03:27
openstackgerritMaru Newby proposed openstack-infra/zuul: Add support for a skip-if filter on jobs  https://review.openstack.org/14676203:28
marunclarkb: that did it, thank you.03:28
*** signed8bit has quit IRC03:28
*** bdpayne has joined #openstack-infra03:28
*** xyang0 has quit IRC03:29
*** adam_g_out is now known as adam_g03:29
*** bdpayne has quit IRC03:30
*** bradjones has quit IRC03:30
openstackgerritMaru Newby proposed openstack-infra/zuul: Add support for a skip-if filter on jobs  https://review.openstack.org/14676203:32
*** bradjones has joined #openstack-infra03:36
openstackgerritMaru Newby proposed openstack-infra/zuul: Add support for a skip-if filter on jobs  https://review.openstack.org/14676203:40
*** Sukhdev has joined #openstack-infra03:41
*** annegent_ has joined #openstack-infra03:46
*** mtreinish_ has joined #openstack-infra03:47
*** annegent_ has quit IRC03:51
*** dmsimard_away is now known as dmsimard03:53
*** dannywilson has joined #openstack-infra03:56
*** nelsnelson has quit IRC03:56
*** dmsimard is now known as dmsimard_away03:57
*** ChuckC_ has quit IRC03:58
*** ChuckC_ has joined #openstack-infra03:59
*** tiswanso has quit IRC04:00
*** dannywilson has quit IRC04:00
*** mohitsha_ has joined #openstack-infra04:02
openstackgerritMaru Newby proposed openstack-infra/zuul: Add support for a skip-if filter on jobs  https://review.openstack.org/14676204:03
*** coolsvap|afk is now known as coolsvap04:03
*** sdake has quit IRC04:03
*** doug-fish has left #openstack-infra04:03
*** otter768 has quit IRC04:04
*** mtreinish has quit IRC04:04
*** mtreinish_ is now known as mtreinish04:04
*** mohitsharma has quit IRC04:05
*** hdd has quit IRC04:05
*** hdd has joined #openstack-infra04:06
marunjhesketh: I've tried to implement Clark's suggestion but I appear to have muffed it: https://review.openstack.org/#/c/146762/04:06
marunjhesketh: The jobs are no longer triggered. Am I doing anything obviously wrong in https://review.openstack.org/#/c/146762/16/tests/test_scheduler.py ?04:06
*** rkukura has joined #openstack-infra04:06
marunclarkb: ^^04:08
*** mohitsha_ has quit IRC04:08
jheskethmarun: looks correct04:09
jheskethjust in the middle of a meeting or else I'd take a closer look04:09
marunjhesketh: no worries ;)04:09
marun:)04:09
jheskethmarun: it doesn't work locally?04:09
marunjhesketh: nope04:09
*** spzala has quit IRC04:12
marunjhesketh: this late in my workday it's likely pebkac and I should try tomorrow :)04:12
jheskethheh, no worries04:13
jheskethhave a good one04:13
*** woodster_ has quit IRC04:13
marunjhesketh: you too!04:14
*** nelsnelson has joined #openstack-infra04:15
*** amotoki has joined #openstack-infra04:16
*** mohitsharma has joined #openstack-infra04:19
marunjhesketh: stubborn I am.  Looks like the job is legitimately being skipped due to the change not correctly adding the files that would trigger a skip.04:28
marunjhesketh: or rather, the reverse. not adding files to the default set (commit message, readme) means the skip-if matcher always succeeds and the job never runs.04:30
marunclarkb: ^^ I've done as you suggested and tried to triggered job execution with a 'patch added' event, but now the changes don't seem to include the extra files I provide to addPatchset.04:32
*** koolhead17 has joined #openstack-infra04:36
*** achanda has joined #openstack-infra04:36
*** ivar-laz_ has joined #openstack-infra04:39
*** achanda has quit IRC04:40
*** ivar-lazzaro has quit IRC04:43
*** marun has quit IRC04:44
*** ivar-laz_ has quit IRC04:44
*** annegent_ has joined #openstack-infra04:46
*** achanda has joined #openstack-infra04:49
*** nelsnelson has quit IRC04:50
*** annegent_ has quit IRC04:51
*** sdake has joined #openstack-infra04:51
*** unicell has joined #openstack-infra04:51
*** hdd has quit IRC04:54
*** pcrews has quit IRC04:56
*** mohitsharma has quit IRC05:03
*** zz_dimtruck is now known as dimtruck05:07
*** garyh has quit IRC05:09
*** sigmavirus24 is now known as sigmavirus24_awa05:11
*** fandi has quit IRC05:12
*** asettle has quit IRC05:17
*** achanda has quit IRC05:20
*** BharatK_ has joined #openstack-infra05:27
*** melwitt has joined #openstack-infra05:31
*** fandi has joined #openstack-infra05:31
*** achanda has joined #openstack-infra05:31
*** woodster_ has joined #openstack-infra05:32
*** garyh has joined #openstack-infra05:40
*** achanda has quit IRC05:41
*** Sukhdev has quit IRC05:41
*** afazekas_ has joined #openstack-infra05:43
*** jyuso1 has quit IRC05:43
*** vigneshvar has joined #openstack-infra05:45
*** koolhead_ has joined #openstack-infra05:45
*** annegent_ has joined #openstack-infra05:46
*** koolhead17 has quit IRC05:47
*** hdd has joined #openstack-infra05:48
*** mohitsharma has joined #openstack-infra05:49
*** ildikov has quit IRC05:49
*** annegent_ has quit IRC05:51
*** koolhead_ has quit IRC05:53
*** dimtruck is now known as zz_dimtruck05:53
*** achanda has joined #openstack-infra05:58
*** otter768 has joined #openstack-infra06:04
*** jyuso1 has joined #openstack-infra06:07
*** otter768 has quit IRC06:09
*** BharatK_ has quit IRC06:13
*** fandi has quit IRC06:18
*** BharatK has joined #openstack-infra06:23
*** achanda has quit IRC06:24
*** achanda has joined #openstack-infra06:25
*** harlowja_at_home has joined #openstack-infra06:27
*** EmilienM is now known as EmilienM|afk06:29
*** bhunter71 has quit IRC06:29
*** harlowja_at_home has quit IRC06:31
*** markmcclain has joined #openstack-infra06:33
*** fandi has joined #openstack-infra06:37
*** markmcclain has quit IRC06:40
*** markmcclain has joined #openstack-infra06:40
*** stevemar2 is now known as stevemar06:41
*** markmcclain has quit IRC06:44
*** fandi has quit IRC06:44
*** markmcclain has joined #openstack-infra06:44
*** Sukhdev has joined #openstack-infra06:45
*** annegent_ has joined #openstack-infra06:46
*** annegent_ has quit IRC06:51
*** achanda has quit IRC06:54
*** yolanda has joined #openstack-infra06:54
*** camunoz has quit IRC07:01
*** andreykurilin_ has joined #openstack-infra07:03
*** SumitNaiksatam has quit IRC07:03
*** ihrachyshka has joined #openstack-infra07:07
*** SumitNaiksatam has joined #openstack-infra07:08
*** jkraj has joined #openstack-infra07:08
*** fandi has joined #openstack-infra07:09
*** andreykurilin|ho has joined #openstack-infra07:11
*** andreykurilin_ has quit IRC07:12
*** mohitsharma has quit IRC07:13
*** afazekas_ has quit IRC07:16
*** mohitsharma has joined #openstack-infra07:19
*** mase_home has joined #openstack-infra07:20
*** ihrachyshka has quit IRC07:20
*** stevemar has quit IRC07:24
*** arxcruz has joined #openstack-infra07:24
*** jichen has joined #openstack-infra07:24
*** afazekas has quit IRC07:25
*** Sukhdev has quit IRC07:28
*** sdake has quit IRC07:29
*** koolhead17 has joined #openstack-infra07:37
*** avozza is now known as zz_avozza07:37
*** yamahata has joined #openstack-infra07:39
*** scheuran has joined #openstack-infra07:39
*** hdd has quit IRC07:45
*** annegent_ has joined #openstack-infra07:46
*** belmoreira has joined #openstack-infra07:46
*** annegent_ has quit IRC07:51
*** zz_avozza is now known as avozza07:52
*** woodster_ has quit IRC07:53
*** ildikov has joined #openstack-infra07:59
*** dvorak has quit IRC08:01
*** HeOS has quit IRC08:03
openstackgerritGhanshyam Mann proposed openstack-infra/project-config: Add non-voting job for Nova V2.1 API with neutron  https://review.openstack.org/15156408:03
openstackgerritMerged openstack-infra/puppet-vinz: Add missing LICENSE file  https://review.openstack.org/15145508:05
*** otter768 has joined #openstack-infra08:05
*** doude has joined #openstack-infra08:06
*** mpaolino has joined #openstack-infra08:07
*** vigneshvar has quit IRC08:07
*** chlong has quit IRC08:08
*** dvorak has joined #openstack-infra08:09
*** otter768 has quit IRC08:11
*** unicell1 has joined #openstack-infra08:13
openstackgerritMerged openstack-infra/puppet-httpd: Add missing LICENSE file  https://review.openstack.org/15143208:13
*** unicell has quit IRC08:14
*** Ala has joined #openstack-infra08:15
*** unicell has joined #openstack-infra08:23
*** unicell has quit IRC08:23
*** unicell has joined #openstack-infra08:23
*** unicell1 has quit IRC08:23
*** dtantsur|afk is now known as dtantsur08:24
*** yamahata has quit IRC08:24
*** melwitt1 has joined #openstack-infra08:27
*** melwitt has quit IRC08:27
*** MaxV has joined #openstack-infra08:28
*** jkraj has quit IRC08:31
*** dvorak has quit IRC08:32
*** jkraj has joined #openstack-infra08:33
*** avozza is now known as zz_avozza08:35
*** KanagarajM has quit IRC08:35
*** salv-orlando has joined #openstack-infra08:38
*** oomichi has quit IRC08:39
*** achuprin_ has quit IRC08:41
*** markus_z has joined #openstack-infra08:41
*** salv-orlando has quit IRC08:45
*** annegent_ has joined #openstack-infra08:46
*** cebruns has quit IRC08:47
*** wznoinsk_ has quit IRC08:47
*** yjiang5 has quit IRC08:47
*** wznoinsk has joined #openstack-infra08:47
*** yjiang5 has joined #openstack-infra08:48
*** cebruns has joined #openstack-infra08:48
*** dvorak has joined #openstack-infra08:50
*** annegent_ has quit IRC08:52
*** dtantsur is now known as dtantsur|brb08:52
*** achuprin_ has joined #openstack-infra08:54
*** dhritishikhar has joined #openstack-infra08:55
*** yamamot__ has quit IRC08:56
*** jcoufal has joined #openstack-infra08:57
*** tnovacik has joined #openstack-infra09:01
*** jpich has joined #openstack-infra09:02
*** melwitt1 has quit IRC09:02
*** mase_home has quit IRC09:02
*** KanagarajM has joined #openstack-infra09:03
openstackgerritMehdi Abaakouk proposed openstack-infra/project-config: oslo.messaging: uses devstack-f21 instead f20  https://review.openstack.org/15132209:06
openstackgerritMehdi Abaakouk proposed openstack-infra/project-config: oslo.messaging: add zeromq funcionnal job  https://review.openstack.org/15157809:06
*** vigneshvar has joined #openstack-infra09:10
*** Guest84671 has joined #openstack-infra09:13
*** afazekas has joined #openstack-infra09:17
*** derekh has joined #openstack-infra09:19
*** jlibosva has joined #openstack-infra09:22
*** yamamoto_ has joined #openstack-infra09:25
*** yamamoto_ has quit IRC09:25
*** jistr has joined #openstack-infra09:31
*** jp_at_hp has joined #openstack-infra09:32
*** pblaho has joined #openstack-infra09:35
*** HeOS has joined #openstack-infra09:36
*** yamamoto_ has joined #openstack-infra09:39
*** mrmartin has joined #openstack-infra09:39
*** yamamoto_ has quit IRC09:43
*** johnson_ has joined #openstack-infra09:45
*** ssam2 has joined #openstack-infra09:45
*** annegent_ has joined #openstack-infra09:46
*** annegent_ has quit IRC09:51
*** BharatK has quit IRC09:53
*** dhritishikhar has quit IRC09:53
*** jedimike has joined #openstack-infra09:54
*** hichihara has quit IRC09:57
*** johnson_ has quit IRC09:57
*** amotoki has quit IRC09:58
*** coolsvap is now known as coolsvap|afk09:58
*** mpaolino has quit IRC10:00
*** changbl has quit IRC10:01
*** sdake has joined #openstack-infra10:02
*** mrmartin has quit IRC10:04
*** fandi has quit IRC10:05
*** isviridov_away is now known as isviridov10:05
*** adalbas has joined #openstack-infra10:05
*** adalbas has quit IRC10:06
*** otter768 has joined #openstack-infra10:06
*** BharatK has joined #openstack-infra10:07
*** zhiwei has quit IRC10:08
*** e0ne has joined #openstack-infra10:10
*** dalgaaf has quit IRC10:10
*** dalgaaf has joined #openstack-infra10:10
*** otter768 has quit IRC10:11
*** teran has quit IRC10:11
*** mpaolino has joined #openstack-infra10:13
*** dhritishikhar has joined #openstack-infra10:19
*** pelix has joined #openstack-infra10:20
openstackgerritMehdi Abaakouk proposed openstack-infra/project-config: oslo.messaging: uses devstack-f21 instead f20  https://review.openstack.org/15132210:22
openstackgerritMehdi Abaakouk proposed openstack-infra/project-config: oslo.messaging: add zeromq funcionnal job  https://review.openstack.org/15157810:22
openstackgerritMehdi Abaakouk proposed openstack-infra/project-config: oslo.messaging: enable functionnal jobs  https://review.openstack.org/15159310:22
*** tsufiev_ is now known as tsufiev10:26
openstackgerritPetrut Lucian proposed openstack-dev/pbr: Fixes Windows generated scripts headers  https://review.openstack.org/15159510:29
*** CIBuilds has quit IRC10:33
*** mohitsharma has quit IRC10:34
*** jamielennox is now known as jamielennox|away10:36
*** erlon has joined #openstack-infra10:36
*** erlon has quit IRC10:36
*** erlon has joined #openstack-infra10:37
openstackgerritPetrut Lucian proposed openstack-dev/pbr: Fixes Windows generated scripts headers  https://review.openstack.org/15159510:39
*** pblaho has quit IRC10:42
*** chlong has joined #openstack-infra10:43
*** zz_johnthetubagu is now known as johnthetubaguy10:43
*** BharatK has quit IRC10:43
*** yamamoto_ has joined #openstack-infra10:44
*** jamielennox|away is now known as jamielennox10:44
*** teran has joined #openstack-infra10:46
*** annegent_ has joined #openstack-infra10:46
*** ldnunes has joined #openstack-infra10:47
*** vigneshvar has quit IRC10:49
*** teran has quit IRC10:50
*** yamamoto_ has quit IRC10:51
*** annegent_ has quit IRC10:51
*** BharatK has joined #openstack-infra10:55
*** sdake has quit IRC10:56
*** jamielennox is now known as jamielennox|away10:56
*** andreaf has joined #openstack-infra11:01
*** dhritishikhar has quit IRC11:02
openstackgerritPetrut Lucian proposed openstack-dev/pbr: Fixes Windows generated scripts headers  https://review.openstack.org/15159511:02
*** HeOS has quit IRC11:04
*** rlandy has joined #openstack-infra11:05
*** HeOS has joined #openstack-infra11:07
*** BharatK has quit IRC11:13
*** rfolco has joined #openstack-infra11:14
*** sdake has joined #openstack-infra11:17
*** teran has joined #openstack-infra11:23
openstackgerritNikita Konovalov proposed openstack-infra/storyboard: Fix status code on delete  https://review.openstack.org/15160311:24
*** dtantsur|brb is now known as dtantsur11:25
*** KanagarajM has quit IRC11:28
*** baoli has joined #openstack-infra11:28
*** Guest84671 has quit IRC11:29
*** sdake has quit IRC11:30
nibalizerfungi: nice patchbomb11:32
nibalizeri hope that was scripted :P11:32
*** adalbas has joined #openstack-infra11:33
*** cdent has joined #openstack-infra11:40
*** zz_avozza is now known as avozza11:41
*** ildikov has quit IRC11:42
*** yamamoto has joined #openstack-infra11:45
*** yamamoto has quit IRC11:45
openstackgerrityolanda.robla proposed openstack-infra/infra-specs: StoryBoard integration tests specification  https://review.openstack.org/15074311:48
*** yolanda_ has joined #openstack-infra11:49
yolandajeblair, krotscheck, amended spec for storyboard testing ^11:50
*** pblaho_ has joined #openstack-infra11:51
*** pblaho_ is now known as pblaho11:55
*** mpaolino has quit IRC11:57
*** aysyd has joined #openstack-infra11:57
*** e0ne is now known as e0ne_11:58
*** andreykurilin|ho has quit IRC12:00
*** e0ne_ is now known as e0ne12:00
*** yamamoto has joined #openstack-infra12:04
*** pc_m has joined #openstack-infra12:05
*** dhritishikhar has joined #openstack-infra12:05
*** otter768 has joined #openstack-infra12:07
odyssey4meHi there - it would appear that zuul is stuck with one or more jobs which have succeeded, but they're not finishing as they usually do by reporting back to gerrit - and they just sit in the check queue. This is definitely happening for https://review.openstack.org/15139012:08
odyssey4mein fact, then entire check queue is sitting...12:08
odyssey4meand the gate queue12:09
odyssey4meI think all the queues, actually. It would appear that perhaps zuul is dead. Long live zuul.12:09
*** otter768 has quit IRC12:12
*** yolanda_ has quit IRC12:12
*** andreaf has quit IRC12:13
*** _amrith_ is now known as amrith12:13
*** andreaf has joined #openstack-infra12:14
*** HeOS has quit IRC12:20
*** dalgaaf has quit IRC12:20
*** HeOS has joined #openstack-infra12:21
*** baoli has quit IRC12:29
*** wuhg has quit IRC12:29
*** wuhg has joined #openstack-infra12:29
*** baoli has joined #openstack-infra12:29
*** rfolco has quit IRC12:29
*** jpich has quit IRC12:37
*** jpich has joined #openstack-infra12:38
*** woodster_ has joined #openstack-infra12:44
*** rfolco has joined #openstack-infra12:46
*** _nadya_ has joined #openstack-infra12:47
*** isviridov is now known as isviridov_na12:47
*** tnovacik has quit IRC12:48
*** liam_ has joined #openstack-infra12:50
*** mase_x200 has joined #openstack-infra12:50
*** liam_ is now known as Guest5170312:50
*** Guest51703 has quit IRC12:56
*** berendt has joined #openstack-infra13:02
*** rfolco has quit IRC13:05
*** rfolco_ has joined #openstack-infra13:05
*** dkliban_afk is now known as dkliban13:13
BobBallWhy is the stackforge/xenapi-os-testing not merge changes? e.g. https://review.openstack.org/#/c/151602/13:17
BobBallHas the project config needed changed?13:17
openstackgerrityolanda.robla proposed openstack-infra/storyboard: Remove duplicate register_options call  https://review.openstack.org/15163013:18
*** bhunter71 has joined #openstack-infra13:22
andreafodyssey4me, yes indeed everything seems to be dead - no tests are started on new commit13:22
odyssey4meBobBall as andreaf has confirmed, zuul has stalled for all jobs13:23
odyssey4meeverything in the queue has stayed exactly as it was for the last two hours or so13:23
openstackgerritJeremy Stanley proposed openstack-infra/puppet-user: Add puppet module files for puppet-user  https://review.openstack.org/15151713:26
*** EmilienM|afk is now known as EmilienM13:28
*** radez_g0n3 is now known as radez13:28
*** mpaolino has joined #openstack-infra13:29
*** Guest84671 has joined #openstack-infra13:30
funginibalizer: yes, scripted. i had held off until we had more of the pending boilerplate changes merged13:30
fungilooking into zuul status issues now13:31
*** dprince has joined #openstack-infra13:32
*** mbacchi has joined #openstack-infra13:35
*** mbacchi has quit IRC13:35
fungilooks like the scheduler is just queuing events and results, but not acting on them13:36
*** markvoelker has joined #openstack-infra13:37
odyssey4mefungi that queue must be looking quite long by now13:38
*** tnurlygayanov has quit IRC13:41
*** changbl has joined #openstack-infra13:41
fungifor some reason its signal handler didn't act on my attempt to generate a thread dump either13:41
*** mbacchi has joined #openstack-infra13:41
*** doug-fish has joined #openstack-infra13:42
fungioh, wait it did13:42
BobBallthanks odyssey4me13:42
fungii was looking in the wrong log13:42
*** deva__ has joined #openstack-infra13:45
fungithis traceback in the log may coincide with when things stopped moving http://paste.openstack.org/show/164159/13:47
fungitrying to match it up to the graphs now13:47
fungiyeah, looks likely13:49
fungiseeing what the gerrit logs have to say about that... the change itself exists and is merged according to the webui13:50
*** sigmavirus24_awa is now known as sigmavirus2413:50
*** jgallard_ has joined #openstack-infra13:51
*** markvoelker has quit IRC13:51
*** markvoelker has joined #openstack-infra13:52
openstackgerritAleksey Ripinen proposed openstack-infra/storyboard: Delete created_at and updated_at fields from requests  https://review.openstack.org/15164113:53
*** AJaeger has joined #openstack-infra13:56
*** HeOS has quit IRC13:56
*** markvoelker has quit IRC13:57
*** annegent_ has joined #openstack-infra13:57
funginothing that looks at all related13:57
*** scheuran has quit IRC14:00
fungithe only thing anomalous i can find in gerrit or zuul's cacti graphs in that window is a spike in established tcp sessions for gerrit http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=30&rra_id=all14:01
openstackgerritAleksey Ripinen proposed openstack-infra/storyboard: Fix stories put method  https://review.openstack.org/15164314:02
fungia few git uploads hanging around in the gerrit show-queue output for several hours, since that time14:02
*** yamamoto has quit IRC14:03
*** tiswanso has joined #openstack-infra14:05
*** e0ne is now known as e0ne_14:06
*** otter768 has joined #openstack-infra14:08
fungikilled the stale git upload-pack threads in gerrit but it doesn't seem like that has started zuul moving14:08
*** mattfarina has joined #openstack-infra14:10
openstackgerritRicardo Carrillo Cruz proposed openstack-infra/storyboard: Pass the old entity values in the NotificationHook  https://review.openstack.org/15164514:12
*** e0ne_ is now known as e0ne14:12
*** mriedem1 has joined #openstack-infra14:12
fungii wonder if zuul lost contact with its local gearman service14:13
*** otter768 has quit IRC14:13
*** mriedem has quit IRC14:14
HenryGsdague: Morning. This https://review.openstack.org/145793 could do with a +A to unblock a lot of dependent work.14:14
*** spzala has joined #openstack-infra14:15
fungiugh. the gearman log doesn't seem to have timestamps14:15
*** mfink_ has joined #openstack-infra14:16
openstackgerritRicardo Carrillo Cruz proposed openstack-infra/storyboard: Pass the old entity values in the NotificationHook  https://review.openstack.org/15164514:17
fungi#status alert zuul isn't running jobs since ~10:30 utc, investigation underway14:17
fungigah, and statusbot is missing?14:17
*** dhritishikhar has quit IRC14:17
*** _nadya_ has quit IRC14:18
*** dustins has joined #openstack-infra14:18
BobBallstatusbot is always missing when there is something important to say :P14:19
*** openstackstatus has joined #openstack-infra14:20
*** ChanServ sets mode: +v openstackstatus14:20
rcarrillocruzhmm14:22
rcarrillocruzhas zuul gone postal?14:22
BobBallyes14:22
fungi#status alert zuul isn't running jobs since ~10:30 utc, investigation underway14:22
openstackstatusfungi: sending alert14:22
rcarrillocruzk, thx fungi14:23
annegent_happy attic day!14:24
*** ddieterly has quit IRC14:24
-openstackstatus- NOTICE: zuul isn't running jobs since ~10:30 utc, investigation underway14:24
*** ChanServ changes topic to "zuul isn't running jobs since ~10:30 utc, investigation underway"14:24
fungii think the "exception moving gerrit event" is a red herring. i find plenty of them in zuul's log from previous days14:25
openstackstatusfungi: finished sending alert14:26
*** markmcclain has quit IRC14:27
openstackgerritAleksey Ripinen proposed openstack-infra/storyboard: Reject false creator_id from requests  https://review.openstack.org/15165014:28
fungibased on merger debug logs it looks like they were getting merge requests up until ~10:50 utc14:29
fungiwhich is very closely correlated to the spike in established tcp connections on gerrit14:30
*** markmcclain has joined #openstack-infra14:31
fungiright at that time fifieldt remarked on another channel about "gerrit downtime"14:31
fungibut he's not around now so i don't quite know what he meant by that14:32
*** dmsimard_away is now known as dmsimard14:32
*** changbl has quit IRC14:33
openstackgerritRicardo Carrillo Cruz proposed openstack-infra/puppet-etherpad_lite: Change /sbin/nologin to /usr/sbin/nologin for Ubuntu  https://review.openstack.org/15165114:33
*** _nadya_ has joined #openstack-infra14:33
*** EmilienM is now known as EmilienM|afl14:35
*** EmilienM|afl is now known as EmilienM|afk14:35
fungijeblair: if you're waking up, zuul has gotten itself into a state where it's not processing its queue. it looks like it may have corresponded to some sort of network traffic anomaly for gerrit. it's not obvious to me from zuul's logs or thread dumps what it's waiting on and i don't want to restart it for fear we might lose a critical debugging opportunity14:36
*** dims has joined #openstack-infra14:36
openstackgerritRicardo Carrillo Cruz proposed openstack-infra/puppet-etherpad_lite: Enable websockets transport on Etherpad  https://review.openstack.org/15165214:36
* fungi will brb14:36
fungimade the mistake of looking at irc before i got my morning coffee14:36
* rcarrillocruz redoing puppet changes that were by the time on system-config and now are unnmergable due to puppet modules split14:36
*** mriedem1 has quit IRC14:38
*** mrodden has quit IRC14:38
*** bknudson has joined #openstack-infra14:38
fungircarrillocruz: thanks, it was on my agenda to -2/abandon those today anyway14:39
*** wuhg has quit IRC14:39
rcarrillocruzno worries, i can do, i guess you have a loooong list of changes to abandon14:39
*** miqui_ has joined #openstack-infra14:39
*** mriedem has joined #openstack-infra14:40
*** mrodden has joined #openstack-infra14:41
*** mrmartin has joined #openstack-infra14:44
*** rlandy has quit IRC14:46
*** mase_x200 has quit IRC14:46
*** kgiusti has joined #openstack-infra14:47
openstackgerritRicardo Carrillo Cruz proposed openstack-infra/puppet-etherpad_lite: Add connection-tuning file to new Apache 2.4 conf folder structure  https://review.openstack.org/15165614:48
*** aysyd has quit IRC14:48
bknudsonis this supposed to work? https://lists.openstack.org/cgi-bin/mailman/listinfo/openstack14:51
bknudsonit's the link from https://launchpad.net/openstack14:51
*** HeOS has joined #openstack-infra14:53
*** Guest84671 has quit IRC14:54
*** jichen_ has joined #openstack-infra14:56
*** signed8bit has joined #openstack-infra14:57
*** ddieterly has joined #openstack-infra14:57
*** sabeen has joined #openstack-infra14:57
*** bogdando has quit IRC14:59
*** zz_jgrimm is now known as jgrimm14:59
*** jichen has quit IRC15:00
*** jichen_ is now known as jichen15:00
*** armax has joined #openstack-infra15:00
fungibknudson: it's _supposed_ to work15:03
fungibut looks like the webserver is kaput on it15:03
*** yamamoto has joined #openstack-infra15:03
fungibknudson: ohhhhh, http not https15:04
fungisorry15:04
fungicaffeine is just starting to kick in finally15:04
*** sdake has joined #openstack-infra15:04
* mordred is once again not here - but would like to annoucne that today is the last day of his two weeks of nearly constant meetings15:04
*** tonytan4ever has joined #openstack-infra15:04
*** amitgandhinz has joined #openstack-infra15:04
bknudsoncan https://launchpad.net/openstack be updated?15:05
fungimordred: too bad--we have a zuul outage i'm having little luck debugging. you get to miss the excitement15:05
fungibknudson: yeah, i'm working on that right now15:05
bknudsonor can https be enabled?15:05
bknudsonfungi: thanks!15:05
bknudsonsomeday that launchpad page config will be in gerrit.15:06
fungibknudson: launchpad is updated now15:06
*** mtanino has joined #openstack-infra15:06
bknudsonfungi: tried it and it works for me. Thanks15:06
fungibknudson: someday that launchpad page config will be unnecessary because we won't be using launchpad15:06
*** yamamoto has quit IRC15:08
*** ryanpetrello_ has joined #openstack-infra15:09
*** bogdando has joined #openstack-infra15:09
*** tmcpeak has joined #openstack-infra15:10
*** ryanpetrello has quit IRC15:11
*** ryanpetrello_ is now known as ryanpetrello15:11
fungii think i've about exhausted my options with zuul's situation and have collected any debugging info i know how, so i'm going to grab a snapshot of the current pipeline states and then restart it and try to figure out from the logs what else needs to get reenqueued which is hiding in the event queue currently15:13
*** avozza is now known as zz_avozza15:14
*** prad has joined #openstack-infra15:18
fungiit's restarted, reenqueuing previous changes now15:18
fungigah, i didn't wait long enough for all the node types to register15:19
*** ZZelle has quit IRC15:20
fungirestarted again, and i'll toggle the gearman plugin on one of the jenkins masters once it's into the main scheduler loop15:20
*** ZZelle has joined #openstack-infra15:20
tchaypo is it possible to find how many bugs times a bug has been mentioned in a “recheck bug xxxx” message?15:22
*** che-arne has joined #openstack-infra15:22
*** r-daneel has joined #openstack-infra15:22
*** markvoelker has joined #openstack-infra15:22
*** stevemar has joined #openstack-infra15:24
*** dhritishikhar has joined #openstack-infra15:24
*** markvoelker has quit IRC15:25
*** pradk has joined #openstack-infra15:25
*** markvoelker has joined #openstack-infra15:25
*** wenlock has joined #openstack-infra15:26
fungiohhhh! i think i know what may have precipitated this15:28
*** zz_avozza is now known as avozza15:29
clarkbtchaypo yes but we fpund the data to be mostly useless. which is why we dropped the bug requirement and focused on elastic recheck15:29
fungirackspace was restarting trove database instances in dfw (and are still in progress doing so according to https://status.rackspace.com/ updates)15:29
clarkbfungi oh good I was going t have to start looking at logs15:29
jeblairfungi: morning15:29
fungiso that at least explains why gerrit's open tcp connections would have spiked up over about a 10-minute period15:30
*** timcline has joined #openstack-infra15:30
jrollinteresting15:30
fungii'm still trying to get bare-precise and py3k-precise workers registered in zuul's gearman before i reenqueue the lists i saved15:30
*** zz_dimtruck is now known as dimtruck15:30
*** sdake has quit IRC15:31
clarkbtchaypo I think sdague may have the script that combs comments for recheck bug #### if you want a way to do it. uses gerrit rest api iirc15:32
jeblairdisabling the registration check is next on my list of zuul features to add15:32
jeblairyou could probably use a gerrit search now, right?15:32
jeblair"comment:'recheck bug'"15:33
*** packet has joined #openstack-infra15:33
fungiunfortunately we have some stuck in a building state in nodepool for a similar timeframe so possible something went south with the (looks like rax-iad) nova api endoint15:33
fungisomething tells me the rackspace "reboot all the things" scramble is not over and we'll probably be fighting fallout from it well into the day15:33
jeblairmaybe we should just go ahead and move our maint window to now :)15:34
fungican't hurt i guess15:34
*** dansmith is now known as superdan15:34
fungii'm going to restart nodepool so i can clear these hung building nodes and try to get some workers registered15:35
* clarkb notes he isnt quite awake yet but dont let that stop you15:35
jeblairneither am i; if we wanted to move it, we'd probably still need a little while to get ready15:35
tchaypothanks jeblair15:36
*** achanda has joined #openstack-infra15:37
*** dims has quit IRC15:39
*** mrmartin has quit IRC15:39
*** roaet- has quit IRC15:43
*** achanda has quit IRC15:43
*** avozza is now known as zz_avozza15:44
*** jpich has quit IRC15:46
*** dims has joined #openstack-infra15:46
tchaypohttps://review.openstack.org/#/q/comment:%22recheck+bug%22,n,z isn’t finding me anything.15:46
fungiokay, bare-precise and py3k-precise are registered so reenqueuing saved changes now15:47
fungiand then i'll try to see what's in the event history in the zuul log and get those manually enqueued15:47
*** salv-orlando has joined #openstack-infra15:47
*** carl_baldwin has joined #openstack-infra15:48
*** koolhead17 has quit IRC15:48
*** waynr has joined #openstack-infra15:49
fungijeblair: once you're more awake, there are a couple of thread dumps in my homedir on zuul (threaddump1 and threaddump2) taken a couple minutes apart while zuul was not processing its queues15:49
*** rainya_ is now known as rainya15:50
*** dims has quit IRC15:50
jeblairfungi: cool, will take a look15:51
*** kragniz is now known as zingark15:53
jeblairfungi: it was stuck trying to report back to gerrit15:54
*** _nadya_ has quit IRC15:54
fungihuh, so the paramiko thread?15:54
jeblairfungi: well, the reporting happens in the main thread15:54
*** Ala has quit IRC15:54
fungioh, got it. paramiko is the stream events listener i guess15:54
*** dangers_away is now known as dangers15:55
jeblairfungi: Thread: 14034406262144015:55
*** _nadya_ has joined #openstack-infra15:55
jeblairhttp://paste.openstack.org/show/164203/15:55
*** abhirc has joined #openstack-infra15:55
jeblairso either gerrit went non-responsive, or the connection went bad15:56
*** carl_baldwin has quit IRC15:56
*** timcline has quit IRC15:56
fungii _think_ gerrit's trove database was offline15:56
*** carl_baldwin has joined #openstack-infra15:56
fungi(being fanatically rebooted by rackspace)15:57
*** timcline has joined #openstack-infra15:57
jeblairi wonder what gerrit's behavior was in that case?  did you restart gerrit?15:57
funginope15:57
fungiit continued on its merry way15:57
jeblairhrm15:57
fungii need to dig deeper into the gerrit logs once i'm done getting the straggler changes reenqueued from the triggers in the zuul log15:57
*** mpaolino has quit IRC15:58
clarkbmaybe a bug in the ssh command gerrit side?15:58
jeblairso figuring out how to time out an ssh command in zuul might be a good thing15:58
*** jcoufal_ has joined #openstack-infra16:00
clarkbzaro did you see replication in gerrit 2.9 requires an explicit flag to force push? we will nees that16:00
jeblairhttp://docs.paramiko.org/en/1.15/api/channel.html#paramiko.channel.Channel.settimeout16:02
jeblairwe can probably work something out with that ^16:02
fungieverything that got approved while zuul was offline is getting reenqueued into check now (and will propagate into the gate once it passes if it's ready)16:02
*** ddieterly has quit IRC16:02
*** pcrews has joined #openstack-infra16:03
*** nelsnelson has joined #openstack-infra16:03
*** nelsnelson has quit IRC16:04
*** jcoufal has quit IRC16:04
superdando we need to recheck things after this morning's outage? (either now or at some point)16:04
jeblairhttps://storyboard.openstack.org/#!/story/200014116:04
*** nelsnelson has joined #openstack-infra16:05
*** zz_avozza is now known as avozza16:05
*** EmilienM|afk is now known as EmilienM16:07
fungieverything that got a patchset uploaded or was restored from an abandoned state while zuul was offline is being enqueued into the check pipeline now too16:07
clarkbsuperdan I think fungi has you covered16:07
*** ddieterly has joined #openstack-infra16:07
superdanrock on16:07
fungisuperdan: i don't think so, except possibly for a few minutes between when zuul was restarted i've mined events out of the logs (since zuul was still listening to gerrit while it was stuck)16:07
superdanokay16:08
fungisuperdan: in a few minutes everything will be done getting reenqueued and then you should check http://status.openstack.org/zuul/ for changes if in doubt as to whether or not zuul is planning to test them already16:08
superdansure, thanks16:09
fungiyou bet16:09
fungiand apologies for the disruption. clouds16:09
*** mpaolino has joined #openstack-infra16:10
*** rfolco_ has quit IRC16:10
fungi#status ok zuul is running again and changes have been reenqueud. seehttp://status.openstack.org/zuul/ before rechecking if in doubt16:11
openstackstatusfungi: sending ok16:11
*** berendt has quit IRC16:11
*** ashleighfarnham has joined #openstack-infra16:11
*** belmoreira has quit IRC16:11
clarkblast night I mostly figured out the multinode test failure. when you shelve then unshelve a node you can get floating ip and nat rules on both routers16:11
*** salv-orlando has quit IRC16:12
clarkbthis creates brokeness with multi router nova net16:12
*** ChanServ changes topic to "Discussion of OpenStack Developer and Community Infrastructure | docs http://docs.openstack.org/infra/manual/ http://ci.openstack.org/ | bugs https://storyboard.openstack.org/ | source https://git.openstack.org/cgit/openstack-infra/"16:12
-openstackstatus- NOTICE: zuul is running again and changes have been reenqueud. seehttp://status.openstack.org/zuul/ before rechecking if in doubt16:12
clarkbthere is a proposed fix I need to test though so hopefully not an issue for long16:12
mtreinishclarkb: we can just add skip for the bug to tempest if you want to unblock faster16:14
openstackstatusfungi: finished sending ok16:15
mtreinishI don't think the loss of coverage on the shelve api for small time is really a big deal compared to getting multinode testing16:15
clarkbmtreinish if the nova fix works we can probably go with it16:15
*** rfolco has joined #openstack-infra16:16
*** dannywilson has joined #openstack-infra16:16
openstackgerritAleksey Ripinen proposed openstack-infra/storyboard: Reject false creator_id from requests  https://review.openstack.org/15165016:17
*** sdake_ is now known as sdake16:18
fungiokay, i'm going to catch up on e-mail and eat some breakfast now that there's a bit of a lull16:18
*** david-lyle_afk is now known as david-lyle16:19
*** jgallard_ has quit IRC16:19
*** funzo has quit IRC16:21
*** jerryz has quit IRC16:22
*** funzo has joined #openstack-infra16:22
*** rfolco has quit IRC16:22
*** rfolco has joined #openstack-infra16:23
*** tonytan4ever has quit IRC16:25
*** apmelton_ is now known as apmelton16:25
*** tonytan4ever has joined #openstack-infra16:25
*** _nadya_ has joined #openstack-infra16:26
*** asselin_ has quit IRC16:26
*** carl_baldwin has quit IRC16:28
*** carl_baldwin has joined #openstack-infra16:29
openstackgerritClark Boylan proposed openstack-infra/devstack-gate: DO NOT MERGE testing nova multinode fix  https://review.openstack.org/15169016:30
clarkbmtreinish: ^ let us see how that does16:30
*** tonytan4ever has quit IRC16:30
*** avozza is now known as zz_avozza16:30
*** abhirc has quit IRC16:30
*** dims has joined #openstack-infra16:31
clarkband now to turn on subunit worker16:31
*** crc32 has joined #openstack-infra16:31
*** dims has quit IRC16:31
*** dims has joined #openstack-infra16:32
*** zz_avozza is now known as avozza16:32
*** dims is now known as dimsum__16:33
*** _nadya_ has quit IRC16:34
*** asselin_ has joined #openstack-infra16:35
*** asselin has joined #openstack-infra16:35
*** tonytan4ever has joined #openstack-infra16:37
*** andreaf has quit IRC16:37
*** andreaf has joined #openstack-infra16:38
*** craigbr has joined #openstack-infra16:42
*** jaypipes has joined #openstack-infra16:42
*** ashleighfarnham has quit IRC16:44
*** deva__ has quit IRC16:45
*** homeless has joined #openstack-infra16:45
*** _nadya_ has joined #openstack-infra16:46
*** cdent has quit IRC16:46
*** cdent has joined #openstack-infra16:47
openstackgerritK Jonathan Harker proposed openstack-infra/zuul: Add a remote url override location  https://review.openstack.org/7605716:47
*** alexpilotti has joined #openstack-infra16:48
alexpilottimordred: hello16:48
alexpilottimordred: we have a small blocking issues on pbr: https://review.openstack.org/#/c/151595/16:48
*** rwsu-afk is now known as rwsu16:49
alexpilottimordred: wonder if you could please take a quick look, it’s missing the second +2 blessing :-)16:49
*** AJaeger has quit IRC16:49
*** jkraj has quit IRC16:49
*** vigneshvar has joined #openstack-infra16:49
clarkbalexpilotti: by brackets do you mean quotes? I am trying to understand the code change given the commit message16:50
*** salv-orlando has joined #openstack-infra16:51
alexpilotticlarkb: quotes, possibly lpetruts wrote brackets instead of quotes :-)16:51
*** marun has joined #openstack-infra16:51
*** e0ne is now known as e0ne_16:51
*** mrmartin has joined #openstack-infra16:51
*** e0ne_ is now known as e0ne16:52
clarkbalexpilotti: +A16:52
alexpilotticlarkb: tx a lot!16:52
*** packet has quit IRC16:54
*** craigbr has quit IRC16:55
*** packet has joined #openstack-infra16:56
*** sabeen2 has joined #openstack-infra16:56
*** unicell has quit IRC16:56
*** woodster_ has quit IRC16:56
*** woodster_ has joined #openstack-infra16:56
*** ssam2 has quit IRC16:57
*** hj-hp has joined #openstack-infra16:58
*** che-arne has quit IRC16:58
*** sabeen has quit IRC16:58
*** ssam2 has joined #openstack-infra16:59
*** KanagarajM2 has joined #openstack-infra16:59
*** ChuckC has joined #openstack-infra17:01
openstackgerritMerged openstack-infra/project-config: Add networking-cisco to stackforge  https://review.openstack.org/14579317:03
*** annegent_ has quit IRC17:05
*** ChuckC_ has quit IRC17:05
clarkbmtreinish: I ran into a thing yesterday when debugging the shelving test. The default security group after tempest ran had no rules in it. Does tempest remove all rules so that it can test specific things?17:05
clarkbor is the default security group completely locked down by default?17:05
*** yjiang5 is now known as yginger517:06
*** denis_makogon has quit IRC17:06
*** ChuckC_ has joined #openstack-infra17:06
zaromorning17:06
zaroclarkb: that's will look into that replication flag.17:07
*** zingark is now known as krangiz17:07
openstackgerritJames E. Blair proposed openstack-infra/zuul: Add test for a enqueuing complex dependencies  https://review.openstack.org/15170517:07
*** krangiz is now known as kragniz17:07
zarothat/thanks17:07
clarkbzaro: good morning. sounds good. I noticed it in a recent gerrit group thread17:07
jeblairclarkb, fungi, jhesketh: ^ okay that's the result of my tour down the rabbit hole from yesterday17:07
jeblairclarkb, fungi, jhesketh: the result is that basically, i think zuul is working correctly, and i made one tiny change that shouldn't really make a difference.17:08
jeblairbut at least it's explained, documented, and tested now :)17:08
*** craigbr has joined #openstack-infra17:08
jeblairclarkb, fungi, jhesketh: and that tree ordering is actually a third option, different from the two yesterday (it's a true depth-first order)17:09
*** ChuckC has quit IRC17:10
*** ChuckC has joined #openstack-infra17:10
*** jcoufal_ has quit IRC17:10
mtreinishclarkb: it's that way by default. We create a new tenant for each test class and the default sg for a new tenant doesn't have any rules in it IIRC17:10
clarkbmtreinish: huh that is not how public clouds work :)17:11
*** gyee has joined #openstack-infra17:11
clarkbmtreinish: well shelve instance test failed with the supposed nova fix so we may want to skip it after all17:12
clarkbI am going to double check that the right versino of nova was installed17:12
*** ChuckC_ has quit IRC17:14
clarkboh cool my checkout nova fix may be buggy let me fix that before we skip any tests17:14
*** Sukhdev has joined #openstack-infra17:15
*** hdd has joined #openstack-infra17:15
openstackgerritClark Boylan proposed openstack-infra/devstack-gate: DO NOT MERGE testing nova multinode fix  https://review.openstack.org/15169017:17
mtreinishclarkb: ok17:17
jeblairclarkb, fungi, mordred, jhesketh, SergeyLukjanov: http://lists.openstack.org/pipermail/openstack-dev/2015-January/055661.html17:17
clarkbthe zuul mergers are not replicated to so cannot be treated as mirrors17:17
*** dtantsur is now known as dtantsur|afk17:17
dougwighi infra, what's the right copyright header to put in source files? i've been using company name, which I assume then gets assigned via the CCLA, but have some review feedback to use openstack foundation.17:18
*** spzala has quit IRC17:18
*** spzala_ has joined #openstack-infra17:18
jeblairdougwig: if you do not work for the foundation, you should not indicate that your copyright is owned by it.  the right answer is _probably_ your company (i can't say that for certain of course)17:18
jeblairdougwig: can you point me at the review?17:18
*** spzala_ is now known as spzala17:18
dougwighttps://review.openstack.org/#/c/151375/17:19
clarkbthe CLA does not require copyright assignment so it basically boils down to who owns the work you did. Often your employer but not necessarily17:19
clarkbjeblair: !17:19
dougwigclarkb: that's what i thought.17:20
jeblairi really love it when i get to write an email like that :)17:20
*** alexpilotti has quit IRC17:20
* mtreinish is still happy to have a personal copyright in tempest from a week of unemployment17:20
*** salv-orlando has quit IRC17:21
jeblairmtreinish: i don't work on apache licensed projects on my own time :)  i spent my week of unemployment in the mountains17:21
clarkbjeblair: where you wrote GPL licensed smoke signal drivers?17:23
*** afazekas has quit IRC17:23
mtreinishjeblair: I didn't really have that luxury then. It was the week before I departed on my road trip to Atl so I had a bunch of summit prep stuff to do. I would have prefered not to be working that week17:24
*** ddieterly has quit IRC17:25
*** craigbr has quit IRC17:26
openstackgerritMerged openstack-dev/hacking: Fix H238 to allow parent classes to be returned by a function  https://review.openstack.org/14856417:26
mtreinishclarkb: heh, well it would have to be gpl if it was for the linux kernel right? :)17:26
*** ddieterly has joined #openstack-infra17:27
*** ddieterly has quit IRC17:27
clarkbmtreinish: it could toggle a usb device from userspace I dunno17:27
*** ddieterly has joined #openstack-infra17:27
*** craigbr has joined #openstack-infra17:28
mtreinishclarkb: hahaha, fair enough17:28
*** hj-hp has quit IRC17:28
*** ccrouch has joined #openstack-infra17:28
*** derekh has quit IRC17:29
*** hj-hp has joined #openstack-infra17:29
*** ddieterly has quit IRC17:30
*** mtanino has quit IRC17:31
*** patrickeast has joined #openstack-infra17:32
clarkboh hey the ceph job seems to be working now17:32
openstackgerritJeremy Stanley proposed openstack-infra/project-config: Suspend Py3K jobs for oslo.messaging oslo.rootwrap  https://review.openstack.org/15171217:32
openstackgerritJeremy Stanley proposed openstack-infra/project-config: Switch from Python 3.3 to 3.4 for Py3K jobs  https://review.openstack.org/15171317:32
openstackgerritJeremy Stanley proposed openstack-infra/system-config: Stop building py3k-precise nodes  https://review.openstack.org/15171417:33
ccrouchclarkb: quick update on my "changing my openstack related email addresses" saga ..17:34
ccrouchI'm think I'm ready to flip them over from @redhat to @gmail17:34
ccrouchI can see my primary contact email address in Launchpad is @redhat, but my @gmail is confirmed there too17:34
ccrouchon review.openstack.org I can see @redhat is preferred email, but @gmail is also registered now17:34
ccrouchfinally www.openstack.org has my primary as @redhat and secondary @gmail17:34
ccrouchclarkb: I think the next steps are...17:34
openstackgerritJeremy Stanley proposed openstack-infra/project-config: Stop using py3k-precise nodes  https://review.openstack.org/15171517:35
clarkbccrouch: in your foundation profile flip your primary to gmail then do the same for gerrit17:35
ccrouchswitch @gmail to preferred email in review.openstack.org17:36
ccrouchthen switch Launchpad so primary is @gmail, and I'll need to reset the password17:36
ccrouchthen finally set www.openstack.org so primary is @gmail17:36
openstackgerritAdam Gandelman proposed openstack-infra/shade: Adds get_flavor() and get_network() functions  https://review.openstack.org/15064917:36
openstackgerritAdam Gandelman proposed openstack-infra/shade: Fixes broken image/object hashing  https://review.openstack.org/15065117:36
openstackgerritAdam Gandelman proposed openstack-infra/shade: Adds some more swift operations  https://review.openstack.org/15065017:36
clarkbccrouch: ya that sounds about right17:36
clarkbfungi: ^17:36
*** melwitt has joined #openstack-infra17:36
clarkbmy only concern is when the launchpad change happens you may still get a new openid?17:36
clarkbbut iirc fungi had a workaround for that so probably just a step missing if at all17:37
ccrouchI'm a little concerned on the openid side17:37
ccrouchin launchpad i have https://launchpad.net/~ccrouch17:37
fungiccrouch: yeah, check the openid lp reports for you currently and make a note of it17:37
*** unicell has joined #openstack-infra17:37
ccrouchin review.openstack.org > Identities > @redhat has an entry17:37
ccrouchbut @gmail is blank17:37
fungiccrouch: stay logged into gerrit while you switch launchpad17:38
fungiccrouch: then check the openid launchpad is reporting after the switch17:38
clarkbccrouch: to check what launchpad reports view the source of your use page17:38
fungiccrouch: if it's different, add it at https://review.openstack.org/#/settings/web-identities (maybe. for some reason i don't see the ability to add to that list though)17:38
fungiccrouch: so it might end up with a new gerrit account the next time you log out of lp and back in anyway17:39
fungiccrouch: if it does, we'll have to go poking in the gerrit database to disable one of the duplicate accounts, just give us a heads up17:39
*** changbl has joined #openstack-infra17:39
*** salv-orlando has joined #openstack-infra17:40
mtreinishclarkb: I don't think it will I switched mine from an ibm address to my domain a while ago and didn't have any issues17:40
mtreinishit did take me forever to figure out how to switch it in lp though17:40
ccrouchfungi: launchpad > OpenID login: and17:40
ccrouchreview.openstack.org > Identities17:40
ccrouchare definitely showing different urls for the same @redhat email right now17:40
*** dimsum__ has quit IRC17:40
fungiyeah, i've never quite been able to pin down what triggers lp to create a new openid. definitely seemed to be tied to lp account modifications, just not sure what exactly17:40
*** bdpayne has joined #openstack-infra17:40
fungiccrouch: does it show more than one there?17:41
*** spzala has quit IRC17:41
zarozuul-dev.o.o will be up and down for testing17:41
mtreinishclarkb, fungi: qq can we change the permissions on the openstack-qa list to only allow the emails from the periodic jobs and reject all others?17:41
jeblairfungi, clarkb: i'm going to send an email about our 61 new repos; sound good?17:41
fungijeblair: sounds great!17:41
clarkbjeblair: yup, I assume with some directions on making puppet changes too?17:41
jeblair(granted, probably only 57 of them are actually new, but who's counting)17:42
*** hj-hp has quit IRC17:42
jeblairclarkb: can you elaborate on that?17:42
clarkbmtreinish: I am no mailman expert but I believe you can give certain subscribers extra permissions17:42
ccrouchfungi: on review.openstack.org > Identities17:42
ccrouchthere are two rows one for @redhat and one for @gmail17:42
ccrouchbut only the @redhat has anything in the identity column17:42
clarkbjeblair: like "you will now need to make puppet changes to specific modules to modify their behavior" that seems to general17:42
jeblairclarkb: here's what i have: https://etherpad.openstack.org/p/ZRYtKd4OpG17:42
clarkbjeblair: just a heads up that system-config is no longer the place to update eg jenkins17:42
*** dhritishikhar has quit IRC17:42
*** mpaolino has quit IRC17:43
jeblairclarkb: okay, i grok.  lemme add some stuff...17:43
ccrouchfungi: fyi url for @redhat email in review.openstack.org > Identities is  https://login.launchpad.net/+id/n6xFnFx17:43
fungijeblair: we're up to 109 infra projects, 255 active (not -attic) official openstack projects, 308 active stackforge projects and 569 total projects in gerrit17:44
clarkbspeaking of attic. Do we need any prep changes for the 1900 renames?17:44
jeblairfungi: drat, i was really hoping we'd be a higher % of openstack by now; at least half.  :)17:45
ccrouchfungi: again fwiw in LP the openid login is just         https://launchpad.net/~ccrouch17:45
clarkbccrouch: https://login.launchpad.net/+id/n6xFnFx is what I see for you in LP as well17:45
clarkbccrouch: so those match up17:45
fungijeblair: announcement lgtm17:46
clarkbccrouch: if you view the source of your user page you will see them listed17:46
*** ildikov has joined #openstack-infra17:46
ccrouchah ok17:46
clarkbccrouch: so what happens is you are still ~ccrouch in lp but sometimeswhen you make changes to mail ou get a new +id/$hash string thing17:46
ccrouchclarkb:  got it <link rel="openid.delegate"17:46
clarkbccrouch: it doesn't remove the old string either you just end up with two of them, then order comes into play and it causes gerrit to get confused17:47
ccrouchso just to be crystal clear..17:47
*** armax has quit IRC17:47
ccrouchi should be swapping emails in review.openstack.org *before* LP right?17:47
ccrouchand remaining logged into review.o.o while making the LP change17:48
*** e0ne has quit IRC17:48
clarkbccrouch: yes17:48
ccrouchack, thanks17:48
* ccrouch dons his openid retardant suit17:48
*** melwitt_ has joined #openstack-infra17:49
*** harlowja_away is now known as harlowja_17:50
clarkbmtreinish: jogo http://logs.openstack.org/90/151690/2/experimental/check-tempest-dsvm-aiopcpu/95baf08/logs/devstack-gate-setup-workspace-new.txt.gz#_2015-01-30_17_22_13_780 I got the shelve nova fix installed properly this time but the test still failed17:50
jeblairclarkb, fungi: puppet announcement email updated17:50
*** yginger5 has quit IRC17:51
*** ashleighfarnham has joined #openstack-infra17:51
clarkbjeblair: looks good, thank you17:51
clarkbmtreinish: jogo: so maybe a skip is best for now? can we skip based on multinode vs single node? eg could tempest ask nova for the number of computes and select that test only if one compute exists? makes that test seem somewhat artificial though17:52
clarkb"here is a feature for the cloud, it only works with a one node cloud"17:53
jeblairclarkb: ha!17:53
fungiebadditions are great17:53
fungier17:53
jeblairfungi: i hope they are not ebad17:53
fungijeblair: additions are great17:53
*** yjiang5 has joined #openstack-infra17:54
*** yjiang5 is now known as yginger517:54
fungifor some reason my terminal ate the initial j in jeb and so my tab complete failed and i didn't notice17:54
*** MaxV has quit IRC17:54
jeblairfungi: "oh! The badditions you've made are great!"17:54
fungi_e_badditions17:54
*** ddieterly has joined #openstack-infra17:54
*** bnemec is now known as beekneemech17:55
mtreinishclarkb: well we can skip on the bug and a config flag17:55
*** melwitt has quit IRC17:55
fungithey're all terrigreat17:55
*** marun has quit IRC17:55
mtreinishbut we don't allow skip decisions to be made by api responses17:55
*** abhirc has joined #openstack-infra17:55
jeblairfungi: sounds flawsome!17:55
*** melwitt_ is now known as melwitt17:55
clarkbmtreinish ok17:55
fungicraptacular17:55
mtreinishbecause the last time we did that we accidently ended up skipping all of the neutron testing and didn't even realize it17:56
fungii'll whip up a change to the governance repo for the new e-badditions while i'm thinking about it17:56
*** _nadya_ has quit IRC17:56
*** marun has joined #openstack-infra17:56
*** teran has quit IRC17:56
jeblairfungi: already done17:56
clarkbmtreinish nice17:56
jeblairfungi: 15110017:56
zarozuul question: anybody know where the openstack_functions.py is located?  that's the file that gets included in layout.yaml17:56
fungijeblair: yep, i checked the proposed list just now when i started17:56
fungizaro: project-config17:56
fungizaro: it's possible the manifest for zuul-dev is lacking a file resource for that17:57
fungizaro: alternatively, we could simply strip out the include directive from its layout.yaml17:57
*** ashleighfarnham has quit IRC17:57
*** dhritishikhar has joined #openstack-infra17:57
jeblairfungi, zaro: i think we changed it to be a directory, so the manifest shouldn't need changing17:57
jeblair(i mean, maybe double check that, but that's my recollection)17:58
*** annegent_ has joined #openstack-infra17:58
*** ChuckC has quit IRC17:58
zaroit's not on review-dev and i think that might be the problem.  testing it now17:58
*** jp_at_hp has quit IRC17:59
*** shivharis has joined #openstack-infra17:59
*** ashleighfarnham has joined #openstack-infra17:59
zaro*review-dev/zuul-dev17:59
*** _nadya_ has joined #openstack-infra18:00
*** ChuckC has joined #openstack-infra18:00
*** sarob has joined #openstack-infra18:00
fungihas anybody started the maintenance plan for the project renames yet? if not i'll get the etherpad put together since we're t-minus one hour and counting18:00
jeblairfungi: nak18:01
*** ChuckC_ has joined #openstack-infra18:01
*** crc32 has quit IRC18:02
*** spzala has joined #openstack-infra18:03
clarkbI havent18:03
*** salv-orlando has quit IRC18:04
*** harlowja_ has quit IRC18:04
*** salv-orlando has joined #openstack-infra18:05
*** julim has quit IRC18:05
*** ChuckC has quit IRC18:05
*** mestery has quit IRC18:05
*** julim has joined #openstack-infra18:06
*** mestery has joined #openstack-infra18:06
*** yamamoto has joined #openstack-infra18:06
*** charz has quit IRC18:07
*** harlowja has joined #openstack-infra18:07
*** dimsum__ has joined #openstack-infra18:07
*** mkoderer has quit IRC18:07
*** crc32 has joined #openstack-infra18:08
*** annegent_ has quit IRC18:08
*** begal-sop_ has joined #openstack-infra18:08
*** Guest74239 has quit IRC18:08
*** vladan has quit IRC18:08
*** loquacities has quit IRC18:08
*** morgabra has quit IRC18:08
*** ddieterly has quit IRC18:09
*** dkliban has quit IRC18:09
*** BobBall has quit IRC18:09
*** alaski has quit IRC18:09
*** alazarev has quit IRC18:09
*** yginger5 has quit IRC18:09
*** Vivek_ has joined #openstack-infra18:09
*** med_ has quit IRC18:10
*** radez is now known as radez_g0n318:10
*** BobBall has joined #openstack-infra18:10
*** BobBall is now known as Guest1135218:10
*** yamamoto has quit IRC18:11
*** yjiang5 has joined #openstack-infra18:11
*** spzala has quit IRC18:12
*** begal-sop has quit IRC18:12
*** radez_g0n3 is now known as radez18:12
*** tonytan4ever has quit IRC18:13
*** alaski has joined #openstack-infra18:13
*** mkoderer has joined #openstack-infra18:13
*** alazarev has joined #openstack-infra18:13
ccrouchfungi: clarkb: so the results for switching my primary email in review.o.o and LP were I think mixed :-/18:13
ccrouchthe good news:18:13
ccrouch1) i was able to successfully switch both accounts to have @gmail as primary email18:13
ccrouch2) I can then log into both accounts with the @gmail address18:13
ccrouch3) the openid in launchpad remained the same (according to openid.delegate in the src for https://launchpad.net/~ccrouch, its still https://login.launchpad.net/+id/n6xFnFx)18:13
ccrouchnow the bad news18:13
ccroucha) when I'm logged into review.o.o I can no longer see any Starred Changes ( I could have sworn I had some)18:13
ccrouchI presume this is due to the fact that now when I go to https://review.openstack.org/#/settings/web-identities18:13
ccrouchI only see one row with my @gmail address but a new OpenId url: https://login.launchpad.net/+id/4wwPefA18:13
*** morgabra has joined #openstack-infra18:13
*** vladan has joined #openstack-infra18:13
*** med_ has joined #openstack-infra18:13
*** charz has joined #openstack-infra18:14
*** med_ has quit IRC18:14
*** med_ has joined #openstack-infra18:14
*** yjiang5 has left #openstack-infra18:14
*** armax has joined #openstack-infra18:14
*** loquacities has joined #openstack-infra18:14
*** ssam2 has quit IRC18:15
clarkbhuh I would expect the starred changes table to use the internal account id as a foreign key not anything openid related18:16
openstackgerritRicardo Carrillo Cruz proposed openstack-infra/storyboard: Pass the old entity values in the NotificationHook  https://review.openstack.org/15164518:16
*** achanda has joined #openstack-infra18:16
openstackgerritMatthew Treinish proposed openstack-infra/system-config: Only start 1 subunit worker per node  https://review.openstack.org/15172518:16
*** markus_z has quit IRC18:17
fungiclarkb: nope, if you log into gerrit via openid, gerrit seems to first consult its table of external identities looking for a match for the openid url. if it finds none, it creates a new account and associates that openid url with it18:18
*** jlibosva has quit IRC18:18
*** dhritishikhar has quit IRC18:18
*** _nadya_ has quit IRC18:18
*** ChuckC_ has quit IRC18:18
ccrouchclarkb: hmm I *thought* I had some watched changes. /me can't be sure18:18
*** ChuckC has joined #openstack-infra18:18
clarkbccrouch: you have two accounts \o/ :)18:19
ccrouchlol18:19
fungiccrouch: i'll deactivate the new account and associate that new openid url with the old account if so. what's your new account id number shown at https://review.openstack.org/#/settings/ ?18:19
* ccrouch bets fungi has a simple and elegant fix to the above situation18:19
clarkbaccount id 9352 is the old one and 14911 is the new one18:19
fungik18:19
ccrouchthanks fungi clarkb18:20
clarkbfungi: is that all doable through the command line yet? or does it still require manual db edits?18:20
clarkbset-account should be able to set external ids but maybe only email addrs?18:20
*** rcarrillocruz has quit IRC18:21
clarkbya just emails ... boo18:21
fungiyep, looks like http://paste.openstack.org/show/164265/ right now18:21
fungii'll give it a shot via the api18:21
*** dkliban has joined #openstack-infra18:21
clarkbfungi: do we have an etherpad yet? I can etherpad while you fix account stuff18:22
*** rcarrillocruz has joined #openstack-infra18:22
fungiset-account --help doesn't mention anything for changing arbitrary external ids, just e-mail18:22
clarkbfungi: ya I think set account is limited in its abilities18:23
clarkbfungi: but set account 14911 inactive may be sufficient in this case?18:23
*** craigbr has quit IRC18:23
*** tchaypo has quit IRC18:23
*** crc32 has quit IRC18:23
*** tchaypo has joined #openstack-infra18:24
fungiccrouch: log out of gerrit and back in again18:24
fungiclarkb: nah, because it'll just create yet another account with that new openid url if we only deactivate18:24
clarkbfungi: but ~ccrouch in LP only has the one openid18:25
fungiyeah, the new one, which doesn't match what gerrit had for the old one18:25
*** markvoelker has quit IRC18:25
fungii did `update account_external_ids set account_id=9352 where account_id=14911;`18:25
clarkbI see18:25
fungiand then `gerrit set-account --inactive 14911` via the api18:25
clarkba couple of the rename changes need rebases so I am doing that now18:26
clarkbI have not started on the etherpad18:26
fungithanks. i'm nearly done wit the etherpad. should be ready for review shortly18:26
*** e0ne has joined #openstack-infra18:26
*** yjiang5 has joined #openstack-infra18:26
*** yjiang5 is now known as yginger518:27
ccrouchfungi: clarkb: \o/ old account number is back at review.o.o and I can see my Starred Changes :-)18:27
ccrouchthank you very much guys18:27
*** ddieterly has joined #openstack-infra18:27
clarkbhrm maybe they were just caught in the zuul merger blip. rechecking now18:27
*** signed8bit is now known as signed8bit_ZZZzz18:27
yginger5Hi, when grenade.sh upgrade the services, will it recreate the config files? I checked the code and seems they will not create the config file, am I right?18:27
jeblairclarkb, fungi: i set their topics to 'project-rename'18:28
asselin_clarkb, fungi  I think this patch has issues: https://review.openstack.org/#/c/147984/18:28
fungiyginger5: it uses the old config files. the new services have to be capable of working with the previous configs18:28
*** dimtruck is now known as zz_dimtruck18:28
fungijeblair: thanks18:28
*** weshay has joined #openstack-infra18:29
asselin_still need to confirm18:29
ccrouchhttps://review.openstack.org/#/settings/web-identities is overflowing with identities :-), but everything looks good18:29
openstackgerritClark Boylan proposed openstack-infra/project-config: Moves the <service>-api repos to the attic  https://review.openstack.org/14528918:29
clarkbfungi: jeblair ^ that one did have aconflict18:29
*** dhritishikhar has joined #openstack-infra18:30
openstackgerritAlexander Tivelkov proposed openstack/requirements: Add semantic_version to global-requirements  https://review.openstack.org/15146618:30
clarkband after rechecking the other one it claims to still conflict so rebase will be pushed18:30
yginger5fungi: thanks for answer, so if a new config item added, the default value has to make sure it will not break anything, right? It really make sense.18:30
*** Ryan_Lane has joined #openstack-infra18:30
openstackgerritClark Boylan proposed openstack-infra/project-config: move oslo.version to attic  https://review.openstack.org/14181218:31
*** koolhead17 has joined #openstack-infra18:31
fungiyginger5: correct--that's one of the things grenade is there to confirm. operators need to be able to upgrade services in-place with the config files from the previous release, then update their configs later (before the next release)18:31
*** jpeeler1 has quit IRC18:32
openstackgerritMatthew Treinish proposed openstack-infra/project-config: Just store preseed subunit stream  https://review.openstack.org/15173218:32
*** shivharis has quit IRC18:33
*** ZZelle_ has joined #openstack-infra18:33
yginger5fungi: thanks for your answer.18:34
*** ivar-lazzaro has joined #openstack-infra18:34
*** Ryan_Lane has quit IRC18:35
*** johnthetubaguy is now known as zz_johnthetubagu18:35
asselin_clarkb, fung, so reverting that patch resolves this issue: http://paste.openstack.org/show/164268/ I'll see if there's a fix18:35
openstackgerritMaru Newby proposed openstack-infra/zuul: Add support for a skip-if filter on jobs  https://review.openstack.org/14676218:36
clarkbasselin_: oh hey at least the error is useful, what does your nodepool.yaml look like?18:37
*** shivharis has joined #openstack-infra18:37
clarkbasselin_: I did test that it builds both snapshot and dib images so I wonder if maybe the nodepool.yamls differ in ways that break it?18:37
*** KanagarajM2 has quit IRC18:37
asselin_clarkb, it's possible. top half: http://paste.openstack.org/show/164269/18:38
clarkbasselin_: ya your label name has to match the image name18:39
clarkbiirc that is necessary due to dib18:39
clarkbor rather how the dib machinery works18:39
clarkbdib itself knows nothing about labels in nodepool18:39
*** annegent_ has joined #openstack-infra18:39
*** jpeeler has joined #openstack-infra18:39
*** jpeeler has quit IRC18:39
*** jpeeler has joined #openstack-infra18:39
clarkbwould need to spend time to see if we can remove that requirement18:40
asselin_clarkb, ok I see. I was taking advantage of the ability for them to be different.18:40
jeblairstatus notice Gerrit and Zuul will be offline from 1900 to 1930 UTC for project renames18:41
jeblairclarkb, fungi: ^ lgty?18:41
clarkbjeblair: yup18:41
jeblair#status notice Gerrit and Zuul will be offline from 1900 to 1930 UTC for project renames18:41
openstackstatusjeblair: sending notice18:41
clarkbasselin_: from memory, the reason its that way is label providers provide labels they do not provide images18:41
clarkbasselin_: so the image key is redundant? its defnitely fixable it just wasn't the existing behavior18:42
fungijeblair: lgtm18:42
*** DuncanT has quit IRC18:42
shivharisfungi: hi, how do i look up the group member list of "networking-brocade-core" and "networking-brocade-release"18:42
*** DuncanT has joined #openstack-infra18:42
-openstackstatus- NOTICE: Gerrit and Zuul will be offline from 1900 to 1930 UTC for project renames18:42
*** pblaho has quit IRC18:43
*** jedimike has quit IRC18:43
asselin_clarkb, ok, I had a need to shorten the names of the nodes, but maybe there's another way.18:43
openstackstatusjeblair: finished sending notice18:43
openstackgerritRicardo Carrillo Cruz proposed openstack-infra/puppet-etherpad_lite: Add connection-tuning file to new Apache 2.4 conf folder structure  https://review.openstack.org/15165618:44
*** achanda has quit IRC18:44
*** achanda has joined #openstack-infra18:45
*** jaypipes has quit IRC18:45
fungijeblair: clarkb: i think https://etherpad.openstack.org/p/repo-renaming-2015-01-30 is ready to go18:47
asselin_clarkb, fyi, docs would need to be updated: http://ci.openstack.org/nodepool/configuration.html#labels18:47
*** baoli has quit IRC18:47
clarkbasselin_: no I think we fix it to do the expected behavior18:48
jeblairfungi, clarkb: i'll do step218:48
openstackgerritKhai Do proposed openstack-infra/project-config: Fix invalid spacing in yaml file  https://review.openstack.org/15173518:48
zarofungi ^18:48
fungishivharis: https://review.openstack.org/#/admin/groups/ (browsing people -> list groups in the top navigation bar of the webui gets you there)18:48
*** craigbr has joined #openstack-infra18:49
shivharisfungi: thanks18:49
clarkbasselin_: we will just need to untangle places where label.name is used instead of label.image and swap them out properly18:49
*** jaypipes has joined #openstack-infra18:49
*** craigbr has quit IRC18:50
*** jaypipes is now known as leakypipes18:50
fungii've deleted the -2 or wip votes from https://review.openstack.org/#/q/status:open+topic:project-rename,n,z in preparation for the maintenance18:50
*** mrmartin has quit IRC18:51
fungii'll go ahead and grab the gerrit cli steps18:51
*** Sukhdev has quit IRC18:52
clarkbasselin_: I can look into it mroe deeply once we rename things. This may actually be simpler than I thought18:52
asselin_clarkb, ok. I can help test then :)18:53
clarkbasselin_: as we do seem to use label.image during disk image building so not sure why I thought that wasn't the case18:53
*** sarob has quit IRC18:54
clarkbI can do the github and mirror renames18:55
*** elliottk has left #openstack-infra18:55
fungitry your hand at the transfer ownership feature in github?18:55
fungiworked well for me last time around18:55
clarkbya18:55
*** baoli has joined #openstack-infra18:56
jeblairfungi, clarkb: still waiting on puppet/ansible; will keep you updated18:56
fungiclarkb: might want to make sure to do that before manage-projects runs via puppet18:56
fungiotherwise we'll end up with new repos in the destination org18:56
clarkbfungi: yup18:56
clarkbasselin_: oh I remember, I think it is because image-update completely ignored the concept of a label which is where we map clouds to images18:58
*** Ryan_Lane has joined #openstack-infra18:58
clarkbasselin_: so we need to get the label and do that mapping properly but I seem to have done it wrongly18:58
clarkbasselin_: the old code just shotgunned images which wasn't quite right either. Maybe new change will be better than both18:58
asselin_clarkb, +118:59
*** mriedem has quit IRC18:59
clarkbasselin_: like image-update would allow you to build an image on a cloud that it didn't have a label for18:59
clarkbasselin_: which should be an error18:59
*** mtanino has joined #openstack-infra19:00
clarkbbecause there is no way that nodepool would boot slaves off of that image so its just a waste of effort19:00
jeblairoh how interesting...19:00
jeblaira new ansible run has started even though the old one was still going19:01
jeblairi didn't think that was supposed to happen19:01
clarkbthere should be a lock, however do we run two different ansible "jobs" now?19:01
clarkbmaybe that is how it happened19:01
fungihuh. with flock around them? or were we just relying on puppet agent locks to prevent concurrent runs on a specific node?19:01
asselin_clarkb, it seems you should be able to map a label to multiple images19:01
clarkbasselin_: possibly. that would be a new feature though19:02
asselin_clarkb, and when bootstrapping, I would like to do a manual image-update, even though it's not on a label19:02
clarkbasselin_: like instead of a single image key allow for each provider in the provider list to specify a distinct image19:02
asselin_clarkb, yes that would be new, but a bit implied by the way it's configured19:02
clarkbasselin_: so I really think that should be an error, because there is no way for nodepool to use that image if there is no label19:02
clarkbit will just make an image that is never booted off of19:03
*** teran has joined #openstack-infra19:03
* jeblair is killing ansible processes19:03
fungiwith a vengeance?19:03
clarkb*shuts down puppetmaster19:04
asselin_clarkb, yes, but the intent is to 'temporarity' not boot off of.19:04
*** shardy is now known as shardy_afk19:04
clarkbasselin_: in that case just add a label...19:04
asselin_clarkb, ok I see, that can work19:04
jeblairclarkb, fungi: okay, ansible is stopped.  i'm ready to proceed.  shall i stop zuul now?19:05
asselin_clarkb, that's actually a really good idea. it also solve the next issue I have: creating the image & testing it. thanks!19:05
asselin_s/image/instance/19:05
jeblair#status alert Gerrit and Zuul are offline until 1930 UTC for project renames19:06
openstackstatusjeblair: sending alert19:06
*** adalbas has quit IRC19:06
fungiyep19:06
fungii'm ready to kill gerrit as soon as you're ready19:06
jeblairfungi: zuul is stopped19:06
fungistopping gerrit now19:06
ekarlsowhat projects are renaming ?19:06
jeblair(i'm going to do nodepool cleanup now)19:06
clarkbfungi: jeblair: I am going to start renaming things on the mirrors19:06
clarkbthen do github19:07
zarofungi: so what should we do about the openstack_functions.py file?19:07
fungiekarlso: http://lists.openstack.org/pipermail/openstack-dev/2015-January/055612.html19:07
mroddenever thought about an ansible playbook to do the renaming?19:07
zarois it even needed by zuul-dev?19:07
zaroshould we keep it in the zuul-dev layout.yaml?19:07
-openstackstatus- NOTICE: Gerrit and Zuul are offline until 1930 UTC for project renames19:07
*** baoli has quit IRC19:07
*** ChanServ changes topic to "Gerrit and Zuul are offline until 1930 UTC for project renames"19:07
clarkbmrodden: yes, my only concern with it is I would still have to github because their api never works19:07
fungigerrit is now offline. proceeding with database edits19:07
mroddenclarkb, yeah that would be an issue19:08
clarkbmrodden: but renaming on things we do control would be a simplish playbook if we can feed in a mapping19:08
*** jistr has quit IRC19:08
clarkbI just don't know how ansible expects that info19:08
*** baoli has joined #openstack-infra19:08
* clarkb <- ansible noob19:08
*** pblaho has joined #openstack-infra19:08
mroddenclarkb, yeah i just started with it, but they have like a ridiculous amount of ways to feed in variable data19:09
*** pblaho has quit IRC19:09
fungigerrit database edits completed without error, proceeding with gerrit filesystem changes19:10
openstackstatusjeblair: finished sending alert19:10
clarkbnow to do github this will be new and exciting19:10
*** ashleighfarnham has quit IRC19:11
fungigerrit filesystem changes completed without incident, lucene search reindex running now19:12
*** bknudson has quit IRC19:14
*** ZZelle has quit IRC19:14
*** ashleighfarnham has joined #openstack-infra19:15
*** Sukhdev has joined #openstack-infra19:15
*** salv-orlando has quit IRC19:16
fungizaro: we can either adjust the manifest for zuul-dev so that the file gets installed, or adjust the layout.yaml to not include it. depends on whether we feel it necessary to test functions in it (maybe keep it around so we can still set the offline node option for ephemeral nodes we might want to test on jenkins-dev?)19:16
clarkbok github is done19:16
*** salv-orlando has joined #openstack-infra19:16
*** mriedem has joined #openstack-infra19:16
fungilucene reindex has reached ~50%19:17
*** tqtran_afk has joined #openstack-infra19:17
*** armax has quit IRC19:17
*** tqtran_afk is now known as tqtran19:18
*** ashleighfarnham has quit IRC19:18
*** mtanino_ has joined #openstack-infra19:19
zarofungi: ok.  i'll make an attempt to adjust manifest to include it so it can be tested.19:19
*** craigbr has joined #openstack-infra19:20
*** mtanino has quit IRC19:20
*** tqtran is now known as tqtran_afk19:20
fungilucene reindex just completed19:21
*** marun has quit IRC19:21
fungieveryone ready for me to start gerrit again?19:21
jeblairfungi: yep19:21
fungiclarkb?19:21
zarorcarrillocruz: ping19:22
clarkbfungi: yes gitXX and github things have been renamed19:22
*** annegent_ has quit IRC19:22
fungihere goes19:22
fungiseems to be up and working19:22
*** annegent_ has joined #openstack-infra19:22
jeblairi'll start zuul now19:23
jeblairre-enqueueing19:24
*** zul has quit IRC19:24
jeblairdone19:25
*** carl_baldwin has quit IRC19:25
jeblairshould be okay to merge the project-config changes now19:25
fungifor some reason gerrit's not giving me the option to select verified votes on 141812 but does on the other two19:25
fungigonna try flushing caches real quick19:25
fungiwhich of course makes gerrit very unresponsive for a minute :/19:26
clarkbfungi: if you have previous votes it doesn't let you add categories19:26
clarkbfungi: so the fix is vote 019:26
clarkbthen vote +2 all the things19:26
clarkbbecause gerrit19:26
fungithanks clarkb--that did it19:27
*** pelix has quit IRC19:27
fungiokay, approving them now19:27
*** zul has joined #openstack-infra19:27
jeblair#status ok Gerrit is back online19:28
openstackstatusjeblair: sending ok19:28
fungidawned on me i didn't need to set verified on these anyway19:28
fungiso just did normal +2 and approve19:28
*** cdent has quit IRC19:29
zarocan someone tell me where things like '$::project_config::gerrit_acls_dir' in system-config repo get defined? i thought it might be in project-config repo but can't find it there either.19:29
*** ChanServ changes topic to "Discussion of OpenStack Developer and Community Infrastructure | docs http://docs.openstack.org/infra/manual/ http://ci.openstack.org/ | bugs https://storyboard.openstack.org/ | source https://git.openstack.org/cgit/openstack-infra/"19:30
clarkbzaro: its part of the project_config module which now lives at openstack-infra/puppet-project_config19:30
-openstackstatus- NOTICE: Gerrit is back online19:30
zaroclarkb: ahh, thanks!19:30
*** marun has joined #openstack-infra19:30
*** vhoward has left #openstack-infra19:30
*** signed8bit_ZZZzz is now known as signed8bit19:31
openstackstatusjeblair: finished sending ok19:32
openstackgerritJames E. Blair proposed openstack-infra/system-config: Update gerrit rename instructions  https://review.openstack.org/15174219:32
jeblairwow, so many cloud errors19:37
*** armax has joined #openstack-infra19:37
*** salv-orlando has quit IRC19:37
*** salv-orlando has joined #openstack-infra19:37
*** pradk has quit IRC19:38
fungicloud errors?19:39
openstackgerritMerged openstack-infra/project-config: stackforge cookbook-ceph is being removed  https://review.openstack.org/14725019:40
jeblairfungi: nodepool -- rax-dfw nodes are going straight into error; novaclient 500s; timeouts waiting for servers in hpcloud to go active19:40
jeblairthat's actually probably why our 'delete' ratio seems so high -- nodes going straing from build to delete :(19:40
fungifun!19:41
jeblairclarkb, fungi: i cleaned up the alien list19:42
fungiawesome--i was just about to do that. now i can do something else ;)19:42
clarkb:/19:44
clarkbjeblair: fungi: could we be leaking floating ips again? though timeouts probably aren't leaked floating IPs19:44
*** spzala has joined #openstack-infra19:45
fungiif it's hpcloud maybe19:45
jrolljeblair: what sort of errors on rax?19:45
jeblairjroll: nova error state19:45
jrolljeblair: any details, though?19:45
*** vhoward has joined #openstack-infra19:45
jeblairjroll: probably not at this point -- we should have nodepool query nova and log info there :/19:46
fungii'm going to start going through and marking system-config changes wip or abandoning depending on whether they're split between openstack_infra and other modules or don't modify openstack_infra and the remaining files in that repo at all19:46
openstackgerritMerged openstack-infra/project-config: Moves the <service>-api repos to the attic  https://review.openstack.org/14528919:46
openstackgerritMerged openstack-infra/project-config: move oslo.version to attic  https://review.openstack.org/14181219:46
jrolljeblair: ok, poking around to see if there's known things19:47
jeblairjroll: (because it immediately starts deleting servers in that case)19:47
*** EmilienM is now known as EmilienM|afk19:47
openstackgerritClark Boylan proposed openstack-infra/nodepool: Better image checking in update_image command  https://review.openstack.org/15174919:48
jeblairfungi: all 3 changes are merged -- should i re-enable ansible cron?19:48
*** dimsum__ has quit IRC19:48
clarkbasselin_: ^ that is what I have whipped up after a few minutes of tackling the problem19:48
fungijeblair: yep, go for it19:48
jeblairdone19:48
* asselin_ looking19:48
jeblairi think that's everything then19:48
clarkbasselin_: I think it should fix the case you ran into and another case where the old code wasn't quite right19:48
clarkbjeblair: woot19:49
fungijeblair: thanks--i'll check back in on the acls in a bit and make sure they're appropriately updated19:49
clarkbjeblair: do we need to debug the locking around ansible runs?19:49
*** spzala has quit IRC19:49
jeblairclarkb: looking at /opt/system-config/production/run_all.sh, i don't see any locking19:51
jeblairi have to run, back in a bit19:51
*** hdd has quit IRC19:51
clarkbI think the cron it self has the lock. I will check19:51
asselin_clarkb, +119:52
*** vigneshvar has quit IRC19:52
clarkbjeblair: fungi hrm doesn't look like there is locking. I will write a patch for that19:53
*** thedodd has joined #openstack-infra19:53
*** shivharis has quit IRC19:54
*** ddieterly has quit IRC19:54
fungik, awesome19:55
clarkbasselin_: so you ran it and it fixed your error?19:55
asselin_clarkb, yes it did!19:55
*** ddieterly has joined #openstack-infra19:55
asselin_clarkb, thank you :)19:56
jrolljeblair: still seeing rax issues?19:56
asselin_clarkb, installed it here: http://paste.openstack.org/show/164309/ and ran the image-update as before as nodepool user19:58
*** rfolco has quit IRC19:58
*** jedimike has joined #openstack-infra19:59
jrolljeblair: at any rate, not seeing issues on our end20:01
openstackgerritClark Boylan proposed openstack-infra/system-config: Use flock around puppet_run_all script  https://review.openstack.org/15175120:02
clarkbfungi: jeblair that should do it20:03
clarkbjroll: I can look in a secnd20:03
asselin_clarkb, the error message could be improved however in the case that there is no label-image mapping: http://paste.openstack.org/show/164310/20:03
jrollclarkb: ok, tenant id would be helpful too if you are seeing problems20:03
clarkbasselin_: hrm I thought I covered all errors with explicit errors20:04
clarkbasselin_: so that failure happened because no label has image dpcfc?20:04
*** salv-orlando has quit IRC20:05
*** SumitNaiksatam has quit IRC20:05
clarkbasselin_: line 185 of my change is what guards against that then line 225 should report that error20:05
clarkbasselin_: oh I see the bug derp20:05
openstackgerritClark Boylan proposed openstack-infra/nodepool: Better image checking in update_image command  https://review.openstack.org/15174920:06
clarkbasselin_: ^ I used the wrong variable name in one location20:06
*** openstackgerrit has quit IRC20:06
*** openstackgerrit has joined #openstack-infra20:07
*** carl_baldwin has joined #openstack-infra20:11
*** spzala has joined #openstack-infra20:11
*** ldnunes has quit IRC20:15
*** jedimike has quit IRC20:17
*** e0ne has quit IRC20:20
pleia2thanks guys, I think now I should drink more beer at fosdem :)20:20
clarkbpleia2: yes!20:21
*** teran_ has joined #openstack-infra20:21
*** teran has quit IRC20:23
*** teran has joined #openstack-infra20:23
asselin_clarkb, looking now20:25
*** david-lyle has quit IRC20:25
*** teran_ has quit IRC20:25
*** shivharis has joined #openstack-infra20:25
*** ddieterl_ has joined #openstack-infra20:26
*** ddieterly has quit IRC20:26
*** markmcclain has quit IRC20:26
fungii checked the 8 renamed projects in gerrit and all of them have the new read-only acls applied correctly, so manage-projects seems to have been successful once puppet was restored to order20:26
fungiand i've skimmed all outstanding system-config changes, either abandoning (if they only applied to other puppet modules than openstack_project) or marked wip (if they applied to openstack_project and one or more other modules). i'll follow up to the mailing lists letting everyone know to check their abandoned changes20:28
*** markmcclain has joined #openstack-infra20:28
*** harlowja is now known as harlowja_away20:29
*** tqtran_afk is now known as tqtran20:29
*** hdd has joined #openstack-infra20:30
openstackgerritAnne Gentle proposed openstack-infra/project-config: Adds a Sphinx theme for use for docs.openstack.org  https://review.openstack.org/15176620:32
*** harlowja_away is now known as harlowja20:34
*** alexpilotti has joined #openstack-infra20:34
*** alexpilotti has quit IRC20:34
Mithrandirpleia2: ooh, at fosdem?  Slightly envious.20:36
*** Ryan_Lane has quit IRC20:36
*** avozza is now known as zz_avozza20:36
fungilooks like we have 6 more module-split topic reviews waiting for a second +2 and approval. the end is in sight!20:37
*** crc32 has joined #openstack-infra20:38
*** dimsum__ has joined #openstack-infra20:41
*** carl_baldwin has quit IRC20:43
asselin_clarkb, -1, error message shows but is misleading. see my comment on the patch https://review.openstack.org/#/c/151749/20:43
*** pradk has joined #openstack-infra20:44
*** aysyd has joined #openstack-infra20:44
*** zz_dimtruck is now known as dimtruck20:46
*** markmcclain has quit IRC20:46
openstackgerritAnne Gentle proposed openstack-infra/project-config: Adds a Sphinx theme for use for docs.openstack.org  https://review.openstack.org/15176620:47
*** bknudson has joined #openstack-infra20:47
*** koolhead17 has quit IRC20:48
*** jedimike has joined #openstack-infra20:48
fungiclarkb: jhesketh: this change looks like it had a couple of log upload failures today https://review.openstack.org/15132220:48
*** aysyd has quit IRC20:49
*** markmcclain has joined #openstack-infra20:49
*** hdd has quit IRC20:51
*** radez is now known as radez_g0n320:51
*** _nadya_ has joined #openstack-infra20:51
clarkbasselin_ thanks will fix20:52
openstackgerritMerged openstack-infra/system-config: Fix module order  https://review.openstack.org/15110320:53
clarkbasselin_ not sure how it gets there though since the first loop should only add providers with labels that have that image20:53
openstackgerritMerged openstack-infra/puppet-ulimit: Add puppet module files for puppet-ulimit  https://review.openstack.org/15151420:54
asselin_clarkb, yea, I'm looking at that too...after re-reading your previous comment 12:0520:54
openstackgerritMerged openstack-infra/puppet-unbound: Add puppet module files for unbound  https://review.openstack.org/15151620:54
openstackgerritMerged openstack-infra/puppet-user: Add puppet module files for puppet-user  https://review.openstack.org/15151720:54
clarkboh i see because if you specify the specific provider then it goes there20:54
openstackgerritMerged openstack-infra/puppet-unattended_upgrades: Add puppet module files for unattended_upgrades  https://review.openstack.org/15151920:54
clarkbI can fix that20:55
*** hj-hp has joined #openstack-infra20:55
*** MaxV has joined #openstack-infra20:55
*** ashleighfarnham has joined #openstack-infra20:55
*** abhirc has quit IRC20:56
*** MaxV has quit IRC20:57
*** zz_avozza is now known as avozza20:57
*** craigbr has quit IRC20:58
*** shardy_afk has quit IRC20:59
*** e0ne has joined #openstack-infra21:00
*** hdd has joined #openstack-infra21:00
*** andreaf has quit IRC21:02
*** Ryan_Lane has joined #openstack-infra21:03
*** shivharis has quit IRC21:06
*** dhritishikhar has quit IRC21:06
*** abhirc has joined #openstack-infra21:08
*** david-lyle has joined #openstack-infra21:08
*** yamamoto has joined #openstack-infra21:09
*** sigmavirus24 is now known as sigmavirus24_awa21:13
*** yamamoto has quit IRC21:13
*** dimsum__ has quit IRC21:15
*** Sukhdev has quit IRC21:15
*** tiswanso has quit IRC21:17
*** shivharis has joined #openstack-infra21:17
*** sigmavirus24_awa is now known as sigmavirus2421:18
*** dprince has quit IRC21:18
*** leakypipes has quit IRC21:20
*** mjturek has quit IRC21:21
*** _nadya_ has quit IRC21:21
openstackgerritClark Boylan proposed openstack-infra/nodepool: Better image checking in update_image command  https://review.openstack.org/15174921:25
clarkbasselin_: ^ how is that?21:25
*** mrmartin has joined #openstack-infra21:25
* asselin_ looking21:26
*** erlon has quit IRC21:26
*** e0ne has quit IRC21:27
*** Duarte-DVR has joined #openstack-infra21:28
asselin_clarkb, looks good: Exception: No label with image dpc is provided by provider p21921:28
asselin_clarkb, will check the other cases now make sure they still work :)21:28
*** annegent_ has quit IRC21:28
clarkbasselin_: cool -1 away if necessary :)21:29
*** unicell has quit IRC21:29
*** unicell has joined #openstack-infra21:30
openstackgerritJeremy Stanley proposed openstack-infra/puppet-etherpad_lite: Add missing LICENSE file  https://review.openstack.org/15178221:31
fungithat ^ one got missed21:31
*** ddieterl_ has quit IRC21:31
fungii probably raced with a boilerplate approval on it and so didn't pick it up until my clean-up pass just now21:31
clarkbfungi: should we rm the py33 stuff now?21:32
clarkbfungi: eg have we reached enough consensus on the thread to do that?21:32
fungiclarkb: if you're ready21:32
*** annegent_ has joined #openstack-infra21:32
fungiclarkb: yeah, dhellmann confirmed21:32
fungii'll send an announcement to the ml later this evening once it switches over21:33
clarkbfungi: did you see AJaeger's -1 at https://review.openstack.org/#/c/151713/ ?21:34
clarkbI approved the first patch in the chain though21:34
openstackgerritJeremy Stanley proposed openstack-infra/puppet-gerrit: Update Gerrit Puppet module for Trusty  https://review.openstack.org/15178321:34
*** radez_g0n3 is now known as radez21:34
clarkband the -1 may explain why the test failed too21:34
fungiunfortunately the log is truncated21:35
clarkbfungi: ya but I am pretty sure that the thing ajaeger points out is why it failed21:35
clarkbI have just approved https://review.openstack.org/#/c/151475/ and can rebuild images to see if that makes jenkins happier once that merges21:36
*** david-lyle_ has joined #openstack-infra21:37
*** david-lyle has quit IRC21:37
fungiclarkb: i'm not sure what AJaeger suggests i'm missing in jenkins/jobs/python-jobs.yaml21:38
*** annegen__ has joined #openstack-infra21:38
*** annegen__ has quit IRC21:39
*** achanda has quit IRC21:39
*** annegent_ has quit IRC21:39
fungithough i do see that i missed deleting the template from the python-jobs group in the later wip change 15171521:39
*** mtanino_ has quit IRC21:39
*** achanda has joined #openstack-infra21:39
*** annegent_ has joined #openstack-infra21:39
clarkbfungi: I think the jjb job group for python jobs must do 33 and not 3421:40
clarkbso the layout check breaks when looking for 34 jos21:40
fungiclarkb: it has both in it already21:41
clarkboh nevermind then21:41
* clarkb runs the tox command locally to see the error21:41
fungii have it running right now21:41
fungishould know something momentarily21:41
openstackgerritMerged openstack-infra/project-config: Suspend Py3K jobs for oslo.messaging oslo.rootwrap  https://review.openstack.org/15171221:42
fungiJob gate-ironic-discoverd-python34 not defined21:42
fungithere we go21:42
fungifixing21:42
*** amitgandhinz has quit IRC21:43
*** achanda has quit IRC21:43
clarkbwhat is an ironic-discoverd?21:44
fungia stackforge thing21:44
*** mtanino has joined #openstack-infra21:44
clarkbI can't help but think about ironic discoveries let us see what google has to say about that21:44
*** mrmartin has quit IRC21:44
clarkbwow ironic-discoverd comes up when I search ironic discovery gj21:45
*** ashleighfarnham has quit IRC21:45
openstackgerritJeremy Stanley proposed openstack-infra/project-config: Switch from Python 3.3 to 3.4 for Py3K jobs  https://review.openstack.org/15171321:45
openstackgerritJeremy Stanley proposed openstack-infra/project-config: Stop using py3k-precise nodes  https://review.openstack.org/15171521:45
fungianyway, fixed those couple changes ^21:45
openstackgerritMerged openstack-infra/project-config: Do not use cache while fetching console logs  https://review.openstack.org/15147521:46
openstackgerritKhai Do proposed openstack-infra/system-config: Add openstack_functions.py to zuul-dev.o.o  https://review.openstack.org/15179021:46
*** achanda has joined #openstack-infra21:48
*** Kerralissa has joined #openstack-infra21:49
*** ddieterly has joined #openstack-infra21:49
openstackgerritJeremy Stanley proposed openstack-infra/system-config: Declare a non-overlapping dynamic UID/GID range  https://review.openstack.org/14764321:49
openstackgerritJeremy Stanley proposed openstack-infra/system-config: Run review-dev on Ubuntu Trusty  https://review.openstack.org/14765521:49
clarkbsweet rebuilding bare-trusty nodes now21:49
*** achanda has quit IRC21:49
openstackgerritDoug Hellmann proposed openstack-infra/project-config: Add openstack/oslo.versionedobjects  https://review.openstack.org/15179221:51
*** david-lyle_ has quit IRC21:51
*** jgrimm is now known as zz_jgrimm21:52
openstackgerritJeremy Stanley proposed openstack-infra/system-config: Run review-dev on Ubuntu Trusty  https://review.openstack.org/14765521:53
*** amitgandhinz has joined #openstack-infra21:56
zarofungi: you know what happened here? https://review.openstack.org/#/c/151735/21:58
*** packet has quit IRC21:58
fungizaro: i'm wondering if we have a problem with the swift upload script on centos6 workers21:59
fungiclarkb: jhesketh: ^21:59
clarkbmtreinish: still around? what do I need to do to skip the shelve test?21:59
clarkbmtreinish: I bet there are docs I should read huh?21:59
mtreinishclarkb: hmm, I'm not sure we have docs for that one22:00
mtreinishbut basically just push a patch to tempest with a skip_because decorator on the test22:00
clarkbok I can grep for examples22:00
mtreinishI can find an example one sec22:00
*** achanda has joined #openstack-infra22:00
*** doug-fish has left #openstack-infra22:01
mtreinishclarkb: oh look at that there is a doc on it :) http://docs.openstack.org/developer/tempest/HACKING.html#test-skips-because-of-known-bugs22:01
openstackgerritSpencer Krum proposed openstack-infra/puppet-httpd: Rename apache2 to httpd globally  https://review.openstack.org/15109822:02
*** craigbr has joined #openstack-infra22:02
nibalizerafternoon y'all22:02
nibalizerawesome to see pleia2 getting nominated for core22:02
*** ashleighfarnham has joined #openstack-infra22:03
*** SumitNaiksatam has joined #openstack-infra22:03
*** abhirc has quit IRC22:03
fungiokay, heading out to grab some dinner, then i'll be back to send the couple of announcements i owe the ml22:03
*** carl_baldwin has joined #openstack-infra22:04
*** craigbr has quit IRC22:04
openstackgerritJames E. Blair proposed openstack-infra/zuul: Add a test for enqueuing complex dependencies  https://review.openstack.org/15170522:05
openstackgerritJames E. Blair proposed openstack-infra/zuul: Support cross-repo-dependencies in dependent pipelines  https://review.openstack.org/14455522:05
openstackgerritJames E. Blair proposed openstack-infra/zuul: Support cross-repo-dependencies in independent pipelines  https://review.openstack.org/14455622:05
zaronibalizer: afternoon22:05
*** craigbr has joined #openstack-infra22:05
zaronibalizer: hey how do you check that puppet is even running? is there a status command?22:06
zaronibalizer: i meant the agent22:06
jeblairzaro: we don't run the agent, we run puppet apply from the puppetmaster22:07
*** miqui_ is now known as miqui_away22:07
jeblairzaro: you can check for an individual host on http://puppetboard.openstack.org/22:07
jeblairzaro: review-dev, storyboard-dev, and zuul-dev are all having problems right now22:08
jeblairwhere problems == not able to run puppet22:08
clarkbfungi: we should ping ryanpetrello about running a pecan scaffolds 34 job22:08
clarkbryanpetrello: ^ you have been pung22:08
ryanpetrelloo/22:09
jeblairthat's some good pingin22:09
clarkbryanpetrello: see https://review.openstack.org/#/c/151713/2/zuul/layout.yaml22:09
*** abhirc has joined #openstack-infra22:09
clarkbryanpetrello: we are trying to move away from py33 since the test nodes for that are a bit crazy. But you get 34 now which is more awesomer :)22:09
clarkbryanpetrello: just pointing out that we should switch your pecan job there to 3422:09
*** crc32 has quit IRC22:09
*** hj-hp has quit IRC22:10
ryanpetrellofungi: you mind just changing that s/3/4 for me?22:10
*** achanda has quit IRC22:10
ryanpetrellothere already exists a scaffolds-34 in pecan's tox22:11
*** mattfarina has quit IRC22:11
jeblairzaro: there were mismatched ssh keys because of host replacement.  i have fixed it for review-dev and zuul-dev22:12
jeblairzaro: puppet should run on those hosts at 22:20 utc22:12
*** achanda has joined #openstack-infra22:12
jeblairzaro: or rather, between 22:20 and 22:3022:12
*** stevemar has quit IRC22:16
jogoclarkb: thanks for https://review.openstack.org/#/c/151798/122:16
clarkbjogo: ya I tested the proposed nova fix and it idn't help so moving onward!22:17
*** thedodd has quit IRC22:17
*** morganfainberg is now known as NotLurkingHere22:17
*** NotLurkingHere is now known as morganfainberg22:19
jogoclarkb: yeah I saw22:19
*** crc32 has joined #openstack-infra22:20
*** armax has quit IRC22:21
mtreinishclarkb: I take it the centos 6 apply job is fubar? Or should I jsut recheck my patches22:23
clarkbwe should have new bare-trusty images shortly across all providers22:23
clarkbmtreinish: in some cases yes because git22:23
clarkbmtreinish: did it timeout?22:23
zaroclarkb: no log file22:24
mtreinishclarkb: err yeah I can't check because I get a 404 on the log link22:24
*** abhirc has quit IRC22:25
zaromtreinish: i've already recheck, same thing again.22:25
clarkbhrm you know hpcloud centos6 images may be old enough that they don't have the fix for the virtualenv stuff22:25
clarkblet me try rebuilding those. Likely an image was renamed22:25
jeblairmtreinish: link to change or 404 log link?22:26
*** craigbr has quit IRC22:26
*** carl_baldwin has quit IRC22:26
mtreinishjeblair: https://review.openstack.org/#/c/151725/ and https://review.openstack.org/#/c/151732/22:26
mtreinishjeblair: the 404's are http://logs.openstack.org/32/151732/1/check/gate-infra-puppet-apply-centos6/6110001/ and http://logs.openstack.org/25/151725/1/check/gate-infra-puppet-apply-centos6/8290f49/22:26
*** dmsimard is now known as dmsimard_away22:27
*** ddieterly has quit IRC22:27
openstackgerritMatthew Treinish proposed openstack-infra/project-config: Just store preseed subunit stream  https://review.openstack.org/15173222:27
clarkbya I think that when we are under more node contention hpcloud gets used for those images and they are out of date22:28
clarkband the reason they are out of date is epel22:28
*** armax has joined #openstack-infra22:28
clarkbianw: are you weekending?22:28
jeblairhttps://jenkins06.openstack.org/job/gate-infra-puppet-apply-centos6/884/consoleFull22:29
jeblairmtreinish, clarkb: there's one of the failures22:29
jeblair2015-01-30 20:28:05.706 | /tmp/hudson5916613440066750934.sh: line 2: /usr/zuul-swift-logs-env/bin/python: No such file or directory22:29
clarkbya the venv is missing22:29
clarkbwe need new images but Error: Cannot retrieve metalink for repository: epel. Please verify its path and try again22:29
jeblairwhy would that only affect hpcloud?22:29
*** Sukhdev has joined #openstack-infra22:29
jeblairhttps://community.hpcloud.com/article/centos-63-instance-giving-cannot-retrieve-metalink-repository-epel-error22:30
jeblairi have not read that yet, but i'm kind of dreading what i'm about to read22:30
mtreinishjeblair: yeah, I just read it, you're not going to like it...22:31
mtreinishtheir fix to the problem is to run: sudo sed -i "s/mirrorlist=https/mirrorlist=http/" /etc/yum.repos.d/epel.repo && sudo yum update22:31
*** annegent_ has quit IRC22:31
clarkbuhm22:31
Sukhdevclarkb: can I ask you a quick question?22:32
jeblairwhere are these endpoints set?22:32
*** dimsum__ has joined #openstack-infra22:32
clarkbI guess no one is interested in fixing the base image. that is from november22:32
jeblairwhy is the solution not to remove the endpoints that use ssl3.0 (and then nuke them from orbit)?22:32
jogomtreinish: wow is tempest's experimental list up to date22:32
clarkbSukhdev: sure22:32
jogomtreinish: are those all 'in progress'22:32
*** carl_baldwin has joined #openstack-infra22:32
*** dimsum__ has quit IRC22:33
mtreinishjogo: dunno, I haven't looked in a while, there's probably some pruning that could be done22:33
*** dimsum__ has joined #openstack-infra22:33
Sukhdevclarkb: How do I find out why has this not merged yet - https://review.openstack.org/#/c/148749/22:33
SukhdevI looked at zuul status dashboard - I can not see it there22:33
jeblairclarkb, mtreinish: "sudo yum upgrade ca-certificates --disablerepo=epel" from http://serverfault.com/questions/637549/epel-repo-for-centos-6-causing-error22:34
jeblairclarkb, mtreinish: so i'm guessing that what they actually need to do is to install a never ca-certs package on the base image?22:34
mtreinishjeblair: yeah that's what it sounds like22:34
clarkbSukhdev: thats a good question. I suppose it is possible that change got caught up in our gerrit downtime today22:35
clarkbSukhdev: you can safely recheck and it should go straight to the gate22:35
Sukhdevclarkb: "recheck no bug", right?22:35
mtreinishjogo: what were you looking at?22:35
jogoadding aiopcpu22:36
clarkbSukhdev: just `recheck` will work22:36
Sukhdevclarkb: cool - thanks22:36
clarkbjeblair: hrm we have that in prepare_node.sh22:36
clarkbjeblair: I am looking at the log more closely to see if it needs to happen earlier22:37
openstackgerritJoe Gordon proposed openstack-infra/project-config: Run aiopcpu experimental job in a several places  https://review.openstack.org/15180622:37
jogomtreinish: tempest has a very long list22:37
mtreinishjogo: yeah I just looked at the list, all of those are either in progress efforts or voting somewhere but not important enough to be voting on tempest22:37
jogowow22:37
Sukhdevclarkb: I did the "recheck" it did not go to gate queue, instead it went to check queue - is this normal?22:38
*** jedimike has quit IRC22:38
* jeblair quietly discards flaming commit he was authoring22:38
jogomtreinish: what about keystonev3 full?22:38
mtreinishjogo: the only one I'm not sure about is why there is a ceilo mongo job which is smoke and full we probably don't need one of those22:38
mtreinishjogo: that's tempest using v3 auth only22:39
clarkbjogo: oh lol22:39
clarkber jeblair ^22:39
*** tmcpeak has quit IRC22:39
clarkbjeblair: I will paste code and logs so we can see what happens22:39
jeblairclarkb: ++construcive approach22:39
jogomtreinish: that isn't our default yet?22:39
mtreinishjogo: I'd have to check with andreaf_ on the status of that, my guess is it would probably work fine though22:40
clarkbjeblair: http://paste.openstack.org/show/164355/ the problem is the if yum repolist check fails due to the same error so then we skip the update ca-certs with the ignore epel option22:40
*** leakypipes has joined #openstack-infra22:40
jogomtreinish: well added aiopcpu so if that lands I will run check experimental on clarkb's patch to turn off the bad test22:40
clarkbjeblair: I don't know enough about yum to know exactly what we want here, but what if we just remove that if else check and always update ca-certificates without epel then update with epel22:40
mtreinishjogo: the rest of your d-g stack landed?22:41
clarkbjeblair: that is my naive suggestions22:41
*** kgiusti has quit IRC22:41
jogomtreinish: enough to get smoke working22:42
jogonext patch in series is turn on nova net multihost22:42
jeblairclarkb: i wonder if we can just check for a file's existence22:42
jogobut things should work without that too22:42
jeblairclarkb: i will spin up some hpcloud centos test nodes22:43
clarkbjeblair: sounds good22:43
*** marun has quit IRC22:43
* jeblair learns nova boot22:44
*** armax has quit IRC22:44
*** mfink_ has quit IRC22:45
asselin_jeblair, nibalizer looks like module-split is basically done. last change that needs +A https://review.openstack.org/#/c/151782/22:45
*** achanda has quit IRC22:45
*** annegent_ has joined #openstack-infra22:45
clarkbI was going to review that change but I already reviewed it22:45
*** achanda has joined #openstack-infra22:46
asselin_jeblair, nibalizer I updated https://storyboard.openstack.org/#!/story/302, added a clean-up task to remove the extra pre-merged changes.22:46
*** morganfainberg is now known as outforteaorcoffe22:47
jeblairasselin_, sweston: i still think we should merge 137991 -- it's super cool, handy institutional knowledge we shouldn't forget22:48
jeblairsweston: do you have time to address the comments there or maybe replace it with the latest version of what you actually ran?22:49
jeblairasselin_: thanks!22:49
asselin_jeblair, sweston did an awesome job. that saved a ton of time!22:49
*** achanda has quit IRC22:50
swestonjeblair: yes, I will address the comments now.22:50
swestonasselin_: thank you :-)22:50
Sukhdevclarkb: so, the recheck did not take https://review.openstack.org/#/c/148749/ to the gate - it is executing in the check queue - is there a way to get it into the gate - so that it can merge?22:52
clarkbSukhdev: hrm, it should've gone to the gate unless it cannot merge for some reason. I may need to look closer22:53
Sukhdevclarkb: can you please?22:54
*** baoli has quit IRC22:54
*** ChuckC has quit IRC22:54
*** pradk has quit IRC22:56
jeblairclarkb: so we can either test for /etc/yum.repos.d/epel.repo22:57
jeblairclarkb: or run  sudo yum update -y ca-certificates || sudo yum --disablerepo=epel update -y ca-certificates22:58
clarkbjeblair: I was thinking to double tap the yum update in order to get ca-certificates updated ptentially from epel after we do the first update?22:58
clarkbsudo yum --disablerepo=epel update -y ca-certificates ; sudo yum update -y ca-certificates22:59
jeblairclarkb: i don't think ca-certificates is expected to be in epel...22:59
clarkbjeblair: oh then why don't we just run it with the disablerepo always?22:59
jeblairclarkb: comment says "--disablerepo will fail"23:00
jeblairif we don't have epel configured23:00
jeblairyay, just corrupted my yum database; moving on to a new host :)23:00
clarkbha23:01
*** hdd has quit IRC23:01
jeblairclarkb: i _think_ disablerepo=epel* might work23:02
jeblair(this is coming from finger memory from a past life; testing)23:02
clarkbif not your suggestion with the || sounds fine to me23:02
*** amitgandhinz has quit IRC23:02
*** yamamoto has joined #openstack-infra23:03
*** harlowja is now known as harlowja_away23:05
clarkbSukhdev: I am not seeing anythin readily apparently in the zuul logs23:05
openstackgerritJames E. Blair proposed openstack-infra/project-config: Fix centos6 image updates on hpcloud  https://review.openstack.org/15181423:05
jeblairclarkb: i think epel* will work for us23:05
jeblairi will now detabify it.23:06
Sukhdevclarkb: interesting!! So, what would be the next step for me once it passes the check (it seems half way through) to get it to the gate?23:06
clarkbSukhdev: well hopefully it goes to the gate, if not then need to figure out what is keeping it out23:07
openstackgerritJames E. Blair proposed openstack-infra/project-config: Fix centos6 image updates on hpcloud  https://review.openstack.org/15181423:07
Sukhdevclarkb: OK I  will wait and see what happens next once it passes the check. Will ping you once the check is done23:07
clarkboh hrm recheck isn't a thing on the gate23:08
clarkbjeblair: ^ when did that change?23:08
clarkbso that probably explains it. needs to go to check then gate23:08
*** abhirc has joined #openstack-infra23:08
openstackgerritMatthew Treinish proposed openstack-infra/project-config: Move missing periodic qa jobs to tempest experimental  https://review.openstack.org/15181523:09
mtreinishanteaya, clarkb, jeblair: ^^^ should be a quick review23:09
jeblairclarkb: my guess is that is since it's only useful if zuul misses an event (since all genuine failures should end up with negative votes and need to go to check for clean-check)23:09
openstackgerritRamy Asselin proposed openstack-infra/project-config: Clean-up deleted/refactored puppet module repo not  https://review.openstack.org/15181623:10
*** ddieterly has joined #openstack-infra23:10
clarkbjeblair: do you need to escape the * to avoid shell globbing?23:10
*** leakypipes has quit IRC23:10
clarkbI guess that is only a problem if something glob matches epel* in the current dir23:10
jeblairclarkb: or glob matches "--disablerepo=epel"?23:11
*** arxcruz has quit IRC23:11
clarkbright :) its probably fine as is23:11
clarkb+2 I can rebuild images as soon as that merges23:11
*** andreykurilin_ has joined #openstack-infra23:12
*** ashleighfarnham has quit IRC23:12
*** dustins has quit IRC23:12
*** shivharis has quit IRC23:14
*** dimtruck is now known as zz_dimtruck23:15
*** wenlock has quit IRC23:17
clarkbon the incomplete console log front: I can reproduce locally by fetching the same urls as the job23:17
clarkband the --no-cache option doesn't seem to help23:17
clarkbalso my browser exhibits the same behavior so I don't think wget and/or wget -c are at fault23:18
jeblairclarkb: oh interesting; how long after the job finished?23:18
clarkbhowever as soon as the job finishes I seem to get the complete log23:18
jeblairah ok23:19
clarkbjeblair: so it seems to do that while the job is running but works after the job completes23:19
jeblairclarkb: are you seeing the whole log in the browser?23:19
jeblairclarkb: or does the browser look the same?23:19
clarkbjeblair: when the job was running browser was truncated, after job finished log was complete in browser23:20
jeblairclarkb: so it seems like buffering, yeah?23:20
clarkbya23:20
clarkbmaybe the slave process itself needs to flush23:20
jeblairclarkb: but we see it move one line at a time on slow jobs23:20
clarkboh right so in the browser on the normal cosole page that worked23:20
*** imcsk8 has quit IRC23:21
*** imcsk8 has joined #openstack-infra23:21
clarkbso probably not the slave. sorry when I was talking about truncated things in the browser above I meanted looking at the urls that the swift uploader is grabbing23:21
jeblairah23:21
clarkbhttps://jenkins01.openstack.org/job/gate-project-config-layout/420/console vs https://jenkins01.openstack.org/job/gate-project-config-layout/420/logText/progressiveHtml first one is complete second one was truncated until job end23:21
*** ashleighfarnham has joined #openstack-infra23:23
jeblairclarkb: jenkins actually using a POST with start=x parameter23:23
*** timcline has quit IRC23:23
clarkbjeblair: I am just going to approve https://review.openstack.org/#/c/151814/ now. I want those images rebuilt23:23
*** timcline has joined #openstack-infra23:23
jeblairclarkb: ++23:23
*** prad has quit IRC23:24
*** dkliban is now known as dkliban_afk23:26
*** yamamoto has quit IRC23:26
*** achanda has joined #openstack-infra23:26
clarkbjeblair: wait the 420/console path is doing a POST to get the data back?23:26
jeblairclarkb: a POST to progressiveHtml23:27
* clarkb turns on some chromium developer tools23:27
*** harlowja_away is now known as harlowja23:27
*** chuckC_ has joined #openstack-infra23:28
*** timcline has quit IRC23:28
*** sabeen2 has quit IRC23:29
*** harlowja has quit IRC23:29
*** ddieterly has quit IRC23:29
*** harlowja has joined #openstack-infra23:29
openstackgerritMerged openstack-infra/project-config: Fix centos6 image updates on hpcloud  https://review.openstack.org/15181423:30
clarkbunfortunately that means we have to calculate our start: X offset manually since wget -c probably won't23:31
clarkbbut it seems to work23:31
jeblairclarkb: i'm not understanding this yet23:31
jeblairclarkb: this isn't a problem in all cases -- do we know if there's a pattern?23:32
jeblairclarkb: i just watched https://jenkins04.openstack.org/job/gate-infra-puppet-apply-centos6/901/console and it worked normally23:32
*** leakypipes has joined #openstack-infra23:32
clarkbjeblair: I think it has to do with file length. the layout job console is much longer/larger than the other jobs23:33
*** ddieterly has joined #openstack-infra23:33
clarkbjeblair: so the other jobs get the entire log the first go around whereas the layout console is longer and needs to try again (assuming buffering is happening)23:33
*** dangers is now known as dangers_away23:33
jeblairclarkb: and you're thinking progressiveHTML has a limit to how much it will serve?23:34
zarozuul question: noop jobs should get return success back to gerrit immediately correct?  for some reason noop jobs are getting stuck in the zuul queue. http://zuul-dev.openstack.org/23:34
zarocan anybody help?23:34
clarkbjeblair: ya23:34
*** hdd has joined #openstack-infra23:36
*** Duarte-DVR has quit IRC23:36
jeblairzaro: no merger running23:36
jeblairzaro: 2015-01-30 22:51:48,069 DEBUG zuul.MergeClient: Submitting job merger:merge with data {'items': [{'oldrev': None, 'newrev': None, 'refspec': u'refs/changes/72/5272/3', 'merge_mode': 2, 'number': u'5272', 'project': 'gtest-org/gtest', 'url': 'ssh://zuul-dev@review-dev.openstack.org:29418/gtest-org/gtest', 'branch': u'master', 'patchset': u'3', 'ref': 'Z5a2e180e51ae467faad89dcca5f8d13b'}]}23:36
clarkbI am rebuilding centos6 images now23:36
jeblairzaro: that's the last action from the log file23:36
*** ZZelle_ has quit IRC23:37
zarojeblair: thanks23:37
*** achanda has quit IRC23:39
fungiclarkb: the scaffords33 job wasn't being run anyway, which is why i removed that section23:40
clarkbjeblair: I think we could probably switch over to `curl -XPOST --data="$(wc -c /tmp/console.html || echo 0)" $URLTOJENKINSPROGHTML` >> /tmp/console.html23:40
fungiryanpetrello: ^23:41
fungier, scaffolds23:41
fungioh, actually i do see it in the layout on master now23:42
clarkbthough maybe stat would be faster than wc23:43
fungiahh, i see my confusion. updating 151713 accordingly23:43
jeblairclarkb: sounds reasonable23:44
*** achanda has joined #openstack-infra23:44
*** timcline has joined #openstack-infra23:44
fungiryanpetrello: i haven't tried that job on python 3.4 since it wasn't a standard py34 tox env, but if you suspect it works then fine by me23:44
*** chuckC_ has quit IRC23:45
*** timcline_ has joined #openstack-infra23:46
asselinjeblair, fungi: last one, really: https://review.openstack.org/15181623:47
openstackgerritJeremy Stanley proposed openstack-infra/project-config: Switch from Python 3.3 to 3.4 for Py3K jobs  https://review.openstack.org/15171323:47
openstackgerritJeremy Stanley proposed openstack-infra/project-config: Stop using py3k-precise nodes  https://review.openstack.org/15171523:47
*** hdd_ has joined #openstack-infra23:47
fungiclarkb: ryanpetrello: ^23:47
*** michchap has joined #openstack-infra23:47
*** hdd has quit IRC23:48
jogoclarkb: I just noticed tempest-full version of aiopcpu takes 20 minutes or so23:48
jogowhile regular tempest-full takes 33 minutes23:48
*** timcline has quit IRC23:49
jogohmm maybe not23:49
*** michchap_ has quit IRC23:49
jogooh that was when devstack was broken23:49
*** annegent_ has quit IRC23:50
*** ddieterly has quit IRC23:50
*** timcline_ has quit IRC23:51
clarkbjeblair: centos6 iamge builds are progressing now.23:51
fungiclarkb: did you manato spot what was broken with swift log uploads for centos6?23:53
*** shivharis has joined #openstack-infra23:53
*** bswartz has joined #openstack-infra23:53
* fungi skipped some of the copious scrollback which accumulated while he was dining23:53
clarkbfungi: ya hpcloud images were ancient due to epel cert fun. jeblair whipped up a fix23:53
*** timcline has joined #openstack-infra23:54
*** timcline has quit IRC23:58
*** changbl has quit IRC23:58
*** achanda has quit IRC23:59
*** signed8bit is now known as signed8bit_ZZZzz23:59

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