Thursday, 2014-02-13

*** dkehn__ is now known as dkehn_00:00
*** mrodden has quit IRC00:01
*** dkranz has quit IRC00:03
*** banix has joined #openstack-infra00:03
*** gaelL has joined #openstack-infra00:04
*** chadlung has quit IRC00:06
*** dkehn has quit IRC00:06
*** dkehn has joined #openstack-infra00:07
*** dkehn_ has quit IRC00:09
*** dkehn is now known as dkehn_00:09
*** harlowja is now known as harlowja_away00:09
*** dkehn_ has quit IRC00:09
*** lyle has quit IRC00:10
*** dkehn has joined #openstack-infra00:10
*** dkehn_ has joined #openstack-infra00:10
*** virmitio has quit IRC00:12
*** lyle has joined #openstack-infra00:12
openstackgerritTim Landscheidt proposed a change to openstack-infra/git-review: Retrieve remote pushurl independently of user's locale  https://review.openstack.org/6430700:14
*** banix has quit IRC00:22
*** dkehn__ has joined #openstack-infra00:24
*** mrodden has joined #openstack-infra00:25
*** krtaylor has joined #openstack-infra00:25
*** dkehn__ has quit IRC00:26
*** dkehn_ has quit IRC00:26
*** ryanpetrello has quit IRC00:26
*** dkehn has quit IRC00:26
*** dkehn__ has joined #openstack-infra00:27
*** dkehn has joined #openstack-infra00:28
clarkbdown to 16 remaining shards. bumping the concurrent repplication number seems to have helped a bit00:28
clarkbstarting to dig into logstash.o.o swap slightly00:30
wenlockfungi, that problem i had earlier was resolved by adjusting our config for https://github.com/openstack-infra/zuul/commit/01c2e63057a77f1c58bdcc6b1321de346f759c2300:30
clarkband down to 1400:31
fungiwenlock: oh, are you pushing your zuul refs into your gerrit repositories?00:32
wenlockyeah, we've been using the git plugin section00:32
wenlockwith ZUUL_REF as the refspec00:32
clarkbyou really don't want to though :( it made everyting so slow00:32
fungii had forgotten that was even an option, we've been serving them from zuul for so long00:33
fungispeaking of which, i want time to focus on the zuul merger/git server scale-out changes00:33
jeblairwenlock: that option won't be in the next version of zuul either.00:33
wenlockahh so something we might need to revisit00:33
wenlockjeblair, good to know!   so that means scm section is depricated?00:34
jeblairwenlock: no, just that zuul will need to serve the refs itself; you should be able to use the git plugin to fetch it from zuul instead of gerrit00:34
openstackgerritA change was merged to openstack/requirements: Oslo program has adopted cliff, pycadf, stevedore, and taskflow  https://review.openstack.org/7233000:36
wenlockjeblair, because zuul is always pulled as latest with vcsrepo, it means we have to make sure to keep up with changes like that quickly , right?00:36
openstackgerritA change was merged to openstack/requirements: Use OS's pypi mirror for Marconi  https://review.openstack.org/7264600:36
jeblairwenlock: if you run it like that, yes; we occasionally merge potentially breaking changes though we try not to00:38
wenlockjeblair, makes sense... thinking maybe it should be parameter to offer an option to use a specific tag00:38
jeblairwenlock: safest thing if you want to do that is to review upcoming changes in gerrit.00:38
jeblairwenlock: not sure i understand that suggestion00:38
*** banix has joined #openstack-infra00:39
wenlockjeblair, making a parameter to the zuul class in config/modules/zuul/manifests/init.pp that implements an argument for pulling some tag instead of latest as an option, while defaulting to latest.00:39
clarkb12 shards now00:40
*** gokrokve_ has quit IRC00:41
jeblairwenlock: ah yes, that's a good idea.  i'd accept that patch.  :)00:41
*** gokrokve has joined #openstack-infra00:41
jeblairwenlock: also, fyi i try to make sure that major or breaking changes are noted in the NEWS file, so you can check that on new releases.00:42
wenlockcool00:42
wenlockjeblair, awesome00:42
clarkb27MB into swap on logstash.o.o00:43
jeblairwenlock: if you run it in the same configuration we do, it's much safer to run CD off of master00:43
wenlockjeblair, agreed... we were following the wiki setup instructions...i think we might default to something closer to a devstack gate in our side00:44
jeblairwenlock: we very rarely break that configuration; and i actually don't expect us to in the forseable future00:44
wenlockjeblair, im not even sure we understood what value refspecs were giving us00:44
*** dcramer_ has joined #openstack-infra00:45
*** gokrokve has quit IRC00:45
wenlockkinda learning it as we go00:45
jeblairwenlock: well, zuul wants to tell a worker to "test something" where something is specified by the ZUUL_REF.  how zuul comes up with that is a long story and best ignored for now.00:46
jeblairwenlock: so the issue at hand is where that ZUUL_REF (which is just a git ref) is located; on the zuul server itself or on gerrit00:47
jeblairhaving gerrit serve it is convenient and easy to set up, but we found in the long run, a bad idea00:47
jeblairbecause your gerrit server gets full of silly useless one-time refs00:47
jeblairso instead, it's better to just have zuul serve that to the workers directly00:47
wenlockjeblair... ahh that makes sense.... so there is no clean up for them?00:47
jeblairnope.  we have a cron on our zuul server that packs them at least, but the only way to get rid of them is actually quite intrusive...00:48
*** senthil has joined #openstack-infra00:48
jeblairwenlock: so we occasionally just delete all the repos on our zuul server to clean them up.00:48
jeblair(it reclones them automatically)00:49
clarkbnow 42MB of swap, I guess in a real emergency I can stop apache and kibana for a short while00:49
*** yamahata has joined #openstack-infra00:49
jeblairhopefully in the future, we can have zuul take care of this for us.00:49
jeblairclarkb: kibana looked to be using a lot of ram00:49
clarkbjeblair: yeah the ruby vm like python and java isn't cheap00:50
wenlockjeblair.... i was wrong, second glance, there is already a revision option on that git clone for zuul... my bad00:53
*** talluri has quit IRC00:56
*** david-lyle has joined #openstack-infra00:58
*** rcleere has quit IRC00:59
*** ryanpetrello has joined #openstack-infra01:00
*** mgagne has quit IRC01:02
*** senthil has quit IRC01:04
*** jhesketh_ has quit IRC01:04
*** bhuvan has joined #openstack-infra01:05
*** cody-somerville has quit IRC01:06
*** ryanpetrello has quit IRC01:06
*** senthil has joined #openstack-infra01:11
*** bhuvan_ has joined #openstack-infra01:11
*** bhuvan has quit IRC01:11
*** david-lyle has quit IRC01:11
*** VijayT has joined #openstack-infra01:13
clarkbjeblair: I am leaning towards stopping kibana on logstash.o.o so that I can walk home without watching that host01:13
jeblairclarkb: sounds good01:13
clarkbjeblair: thoughts? then from home I will spin up the workers as we are getting closish to full recovery01:13
clarkb9 shards in process now01:14
clarkbkibana stopped, heading home now back on in about half an hour01:16
openstackgerritJames E. Blair proposed a change to openstack-infra/gear: Use client_id as part of the logger name  https://review.openstack.org/7311801:16
jeblairthat ^ might seem out of left field, but i think i need it to help track down one of the nondeterministic zuul test bugs01:16
*** dkehn_ has joined #openstack-infra01:17
*** prad__ has quit IRC01:18
*** alexpilotti has quit IRC01:18
*** cody-somerville has joined #openstack-infra01:18
fungithe bare-centos6 image build in rax-ord is running for close to 90 minutes now, but it's still doing things, not hung01:19
openstackgerritA change was merged to openstack/requirements: Add pbr to projects.txt  https://review.openstack.org/7183401:19
jeblairfungi: yeah, i just had the same thought and checked.01:19
*** dkehn_ has quit IRC01:19
openstackgerritA change was merged to openstack-infra/devstack-gate: add dstat to our list of statistics collectors  https://review.openstack.org/7278601:19
*** dkehn has quit IRC01:19
*** dkehn_ has joined #openstack-infra01:20
fungihoping once it finishes i can shoot one of the rax-iad nodes in the head and watch one successfully build in ord01:20
*** dkehn__ has quit IRC01:20
fungii would say then we're probably okay to merge the bare-centos6 job config switch, but it's getting late and if my power goes out i won't be around to look after it01:20
openstackgerritJenkins proposed a change to openstack-dev/pbr: Updated from global requirements  https://review.openstack.org/7311901:21
jeblairfungi: yeah, i have about an hour left before i'm supposed to be elsewhere, so we may want to wait.01:21
jeblairoh look at that.  ^ :)01:21
*** dkehn has joined #openstack-infra01:21
openstackgerritDevananda van der Veen proposed a change to openstack-infra/reviewstats: Update the Ironic core team list  https://review.openstack.org/7306801:22
jeblairmordred: can you confirm that we really do want automatic updates for pbr?  i could see how we might not due to it's location at the bottom of the dependency stack01:23
jeblairmordred: https://review.openstack.org/#/c/71834/ and https://review.openstack.org/7311901:23
openstackgerritMatthew Oliver proposed a change to openstack-infra/config: Nodepool image logs accessible via HTTP  https://review.openstack.org/7119001:26
*** senthil has quit IRC01:28
*** yaguang has joined #openstack-infra01:31
*** oubiwann_ has joined #openstack-infra01:34
*** atiwari has quit IRC01:34
openstackgerritJames E. Blair proposed a change to openstack-infra/reviewstats: Add Sergey Lukjanov to infra-core  https://review.openstack.org/7312201:35
openstackgerritJames E. Blair proposed a change to openstack-infra/reviewstats: Add new infra projects  https://review.openstack.org/7312301:35
*** mgagne has joined #openstack-infra01:38
*** david-lyle has joined #openstack-infra01:40
*** harlowja_away is now known as harlowja01:43
*** w_ has joined #openstack-infra01:48
openstackgerritKhai Do proposed a change to openstack-infra/gerrit: replace submodule url reference from relative to absolute  https://review.openstack.org/7312501:49
*** alexpilotti has joined #openstack-infra01:50
*** olaph has quit IRC01:50
*** zul has joined #openstack-infra01:51
openstackgerritKhai Do proposed a change to openstack-infra/gerrit: add a .gitreview file  https://review.openstack.org/7312601:52
*** pcrews has quit IRC01:53
zarofungi: i figured out the submodule situation.  need to make a few changes to openstack/2.8 branch. wondering if you have time to help me push thru?01:55
fungizaro: i can try01:55
zarofungi: https://review.openstack.org/#/c/73126 and https://review.openstack.org/#/c/7312501:56
zarofungi: those failed, but tests are invalid.01:56
*** nosnos has joined #openstack-infra02:00
*** morganfainberg has quit IRC02:02
zarofungi: what i mean is that changes to openstack/2.8 branch triggers tests meant to for master branch,  master is on gerrit 2.402:04
fungizaro: what needs to be done to get check-gerrit-unittests passing?02:04
*** zehicle_at_dell has joined #openstack-infra02:04
*** bradm has quit IRC02:05
fungiwe should probably add a gerrit-specific noop job which is voting, and then make check-gerrit-unittests nonvoting on everything except the '^openstack/2\.4\..*$' branch pattern02:06
clarkbdown to 4 shards now02:06
fungiawesome02:06
zarofungi: this change sets up correctly testing on the right branches https://review.openstack.org/#/c/6034802:07
*** bradm has joined #openstack-infra02:07
fungizaro: do we have a unit test job for gerrit 2.8?02:07
fungior at least the information we need to write one?02:07
zarofungi: yes, it's in 6034802:07
*** salv-orlando_ has joined #openstack-infra02:07
zarofungi: but 60348 depends on 7312602:08
fungiahh, yes, i had reviewed a previous patchset for the job addition02:08
zarofungi: sorry i mean 60348 depends on 7312502:08
*** sarob has quit IRC02:08
zarofungi: we need to setup the git submodules correctly before merging 6034802:09
*** nicedice_ has joined #openstack-infra02:09
fungigot it, so we need 73125 force-merged to be able to run the correct jobs02:09
*** sarob has joined #openstack-infra02:09
zarofungi: yup02:09
fungithough... i think we should be able to merge 60348 first02:10
*** mkerrin1 has joined #openstack-infra02:10
*** homeless_ has joined #openstack-infra02:10
*** lttrl has quit IRC02:10
fungiit will leave us unable to merge anything new for gerrit 2.8 until we merge 7312502:10
zarofungi: 60348 run 'buck build release' that type of build requires the plugins02:10
fungibut we ought to be able to recheck 73125 once 60348 is applied on jenkins/zuul02:10
*** amotoki__ has joined #openstack-infra02:10
openstackgerritA change was merged to openstack-dev/pbr: Remove tox locale overrides  https://review.openstack.org/7220702:11
zarofungi: ok if this is confusing.  i can roll 73125 into 60348.02:11
zarofungi: how's that?02:11
fungiyou can't roll them together--they're for separate projects02:11
zarofungi: ohh yeah, your right.02:12
fungithe config change implementing the job can go in, so long as we don't run that job nothing breaks, right?02:12
fungiand then the first change which would run that job is 73125, which adds the submodule corrections02:12
clarkb3 shards now02:12
fungiwe ought to be able to recheck 73125 and see the job run and hopefully pass02:12
*** mtaylor has joined #openstack-infra02:13
zarofungi: hmm, but that change sets up the job and it will run.02:13
*** harlowja_ has joined #openstack-infra02:13
*** locke105 has quit IRC02:13
*** SergeyLukjanov has quit IRC02:13
*** orionNebula has joined #openstack-infra02:13
*** sc68cal has quit IRC02:13
zarofungi: are you suggesting maybe adding noop job to that change and running that initially instead?02:13
*** sarob has quit IRC02:14
fungizaro: those jobs being added in 60348 only run in check, gate and post pipelines for changes to gerrit02:14
*** mikal_ has joined #openstack-infra02:14
*** DinaBelova_ has quit IRC02:14
*** mikal has quit IRC02:14
*** ogelbukh1 has joined #openstack-infra02:14
*** tristanC_ has joined #openstack-infra02:14
*** nico_ has joined #openstack-infra02:14
zarofungi: yes, 60348 will fail unless 73125 is merged.02:15
fungiit's a "if a tree falls in the forest and there's noone round to hear it" question02:15
*** bnemec_ has joined #openstack-infra02:15
*** SergeyLukjanov has joined #openstack-infra02:15
fungidoes it matter if that jobs broken, if the first change it runs on is the one which fixes it?02:15
*** mkerrin has quit IRC02:15
*** bhuvan_ has quit IRC02:15
*** amotoki_ has quit IRC02:15
*** NikitaKonovalov has quit IRC02:15
*** yaguang has quit IRC02:15
*** jhesketh__ has quit IRC02:15
*** mordred has quit IRC02:15
*** SpamapS has quit IRC02:15
*** salv-orlando has quit IRC02:15
*** changbl has quit IRC02:15
*** homeless has quit IRC02:15
*** bnemec has quit IRC02:15
*** Clabbe has quit IRC02:15
*** ruhe has quit IRC02:15
*** nkinder has quit IRC02:15
*** kevinbenton has quit IRC02:15
*** gaelL has quit IRC02:15
*** vladan has quit IRC02:15
*** rockstar has quit IRC02:15
*** salv-orlando_ is now known as salv-orlando02:15
*** nosnos_ has joined #openstack-infra02:15
*** plomakin_ has joined #openstack-infra02:15
zarofungi: ohh, i get what you mean.  sorry.  i'm brain dead.02:15
*** changbl has joined #openstack-infra02:15
*** davidlenwell_ has joined #openstack-infra02:15
*** max_lobur has joined #openstack-infra02:16
*** dkehn__ has joined #openstack-infra02:16
*** sc68cal has joined #openstack-infra02:16
zarofungi: ok.  so 60348 can go in before 73125.02:16
*** jhesketh has joined #openstack-infra02:16
*** bodepd_ has joined #openstack-infra02:16
*** mkerrin has joined #openstack-infra02:16
fungiso we approve the job change, which will merge to the config repo just fine. then we recheck the gerrit change which corrects the submodules, which ends up self-testing on the new job02:16
*** kevinbenton has joined #openstack-infra02:17
fungiyep02:17
*** gaelL has joined #openstack-infra02:17
fungiand if the recheck fails, we analyze the reason, and either adjust it or land another config patch correcting the job further02:17
openstackgerritCyril Roelandt proposed a change to openstack-infra/config: python-savannaclient: enable the py33 gates  https://review.openstack.org/7312802:17
zaroyeah, ok.  i'm heading out now, but will 60348 take it off WIP status.02:17
fungisounds great02:18
*** Clabbe has joined #openstack-infra02:18
*** b3nt_pin has joined #openstack-infra02:18
*** mrodden has quit IRC02:18
*** CaptTofu has quit IRC02:18
*** max_lobur_afk has quit IRC02:18
*** ProfFalken has quit IRC02:18
*** plomakin has quit IRC02:18
*** branen_ has joined #openstack-infra02:18
*** david-lyle has quit IRC02:18
*** bodepd has quit IRC02:18
*** dkehn_ has quit IRC02:18
*** ProfFalken has joined #openstack-infra02:19
*** mtreinish has quit IRC02:19
*** esmute has quit IRC02:19
*** bradm has quit IRC02:19
*** zehicle_at_dell has quit IRC02:19
*** alexpilotti has quit IRC02:19
*** VijayT has quit IRC02:19
*** yamahata has quit IRC02:19
*** lyle has quit IRC02:19
*** beagles has quit IRC02:19
*** yamamoto has quit IRC02:19
*** gyee has quit IRC02:19
*** ogelbukh has quit IRC02:19
*** nicedice has quit IRC02:19
*** freyes has quit IRC02:19
*** harlowja has quit IRC02:19
*** exz has quit IRC02:19
*** tristanC has quit IRC02:19
*** lcestari has quit IRC02:19
*** gema has quit IRC02:19
*** dkliban has quit IRC02:19
*** yamahata_ has quit IRC02:19
*** mattymo has quit IRC02:19
*** cp16net has quit IRC02:19
*** kruskakli has quit IRC02:19
fungiyay! we have bare-centos6 nodepool images in both rax-iad and rax-ord now02:19
*** zehicle_at_dell has joined #openstack-infra02:19
openstackgerritYongli He proposed a change to openstack-infra/storyboard: Use six.moves.urllib.parse instead of urlparse  https://review.openstack.org/7288702:19
*** ogelbukh1 has quit IRC02:20
*** asalkeld has joined #openstack-infra02:20
*** gema has joined #openstack-infra02:20
fungideleted one of the 14 ready rax-iad nodes, waiting for one to hopefully start building in rax-ord02:21
*** esmute has joined #openstack-infra02:22
*** mkerrin1 has quit IRC02:22
*** nosnos has quit IRC02:22
*** zul has quit IRC02:22
*** dcramer_ has quit IRC02:22
*** banix has quit IRC02:22
*** branen has quit IRC02:22
*** kmartin has quit IRC02:22
*** davidlenwell has quit IRC02:22
*** bogdando has quit IRC02:22
*** medieval1_ has quit IRC02:22
*** uvirtbot has quit IRC02:22
clarkbwoot02:22
asalkeldhi folks where is the best place to look for setting up a full genkins/zuul gating system?02:22
clarkbare they pvhvm?02:23
asalkeldjenkin/gerrit02:23
*** cody-somerville has quit IRC02:23
asalkeld(so install/configuration instructions)02:23
openstackgerritCyril Roelandt proposed a change to openstack-infra/config: python-tuskarclient: enable the py33 gate  https://review.openstack.org/7313002:23
clarkbasalkeld: there is a running your own doc at http://ci.openstack.org which is probably a good place to start02:23
*** nkinder has joined #openstack-infra02:23
*** NikitaKonovalov_ has joined #openstack-infra02:24
*** kevinbenton_ has joined #openstack-infra02:24
*** SpamapS has joined #openstack-infra02:24
*** SpamapS has quit IRC02:24
*** SpamapS has joined #openstack-infra02:24
*** lttrl has joined #openstack-infra02:25
*** gaelL_ has joined #openstack-infra02:25
*** esmute has quit IRC02:25
*** ProfFalken has quit IRC02:25
fungiasalkeld: http://ci.openstack.org/running-your-own.html02:25
*** CaptTofu has joined #openstack-infra02:25
*** tru_tru has quit IRC02:25
*** lcestari_ has joined #openstack-infra02:25
*** NikitaKonovalov_ is now known as NikitaKonovalov02:26
*** yamahata__ has joined #openstack-infra02:26
*** nico_ is now known as exz02:26
*** nicedice_ has quit IRC02:26
asalkeldthanks clarkb fungi02:26
*** DinaBelova_ has joined #openstack-infra02:26
*** DinaBelova_ is now known as DinaBelova02:27
*** VijayT has joined #openstack-infra02:27
*** eharney has quit IRC02:27
*** ruhe has joined #openstack-infra02:27
*** cody-somerville has joined #openstack-infra02:27
*** davidlenwell has joined #openstack-infra02:28
*** max_lobur has quit IRC02:28
*** esmute has joined #openstack-infra02:28
*** david-lyle has joined #openstack-infra02:28
*** nicedice has joined #openstack-infra02:28
*** yamahata has joined #openstack-infra02:28
fungiclarkb: yeah, the bare-centos slaves use base-image: 'CentOS 6.5 (PVHVM)'02:28
fungiand of course name-filter: 'Performance'02:28
*** yaguang has joined #openstack-infra02:29
*** freyes has joined #openstack-infra02:29
*** SpamapS has quit IRC02:29
*** eharney has joined #openstack-infra02:30
*** lttrl has quit IRC02:30
*** julim has quit IRC02:30
*** SpamapS has joined #openstack-infra02:30
*** VijayT has quit IRC02:30
*** coolsvap has quit IRC02:30
*** kmartin has joined #openstack-infra02:30
*** zul has joined #openstack-infra02:30
*** dkliban has joined #openstack-infra02:30
*** dripton_ has joined #openstack-infra02:30
*** max_lobur has joined #openstack-infra02:30
*** bhuvan has joined #openstack-infra02:30
*** dripton has quit IRC02:30
*** gaelL_ has quit IRC02:31
*** lttrl has joined #openstack-infra02:31
*** ruhe has quit IRC02:32
*** julim_ has joined #openstack-infra02:32
*** julim_ has quit IRC02:32
*** NikitaKonovalov has quit IRC02:32
*** tristanC_ has quit IRC02:32
*** nosnos has joined #openstack-infra02:32
*** zehicle_at_dell has quit IRC02:33
*** b3nt_pin has quit IRC02:33
*** Clabbe has quit IRC02:33
*** kevinbenton has quit IRC02:33
*** bodepd_ has quit IRC02:33
*** jhesketh has quit IRC02:33
*** gaelL has quit IRC02:33
*** dkehn__ has quit IRC02:33
*** davidlenwell_ has quit IRC02:33
*** nosnos_ has quit IRC02:33
*** oubiwann_ has quit IRC02:33
*** kevinbenton_ is now known as kevinbenton02:33
*** andre__ has quit IRC02:34
*** locke105 has joined #openstack-infra02:34
*** dkehn has quit IRC02:35
*** lcestari_ has quit IRC02:35
*** lttrl has quit IRC02:35
*** DinaBelova has quit IRC02:35
*** bhuvan has quit IRC02:36
*** SpamapS has quit IRC02:36
*** cp16net has joined #openstack-infra02:36
*** SpamapS has joined #openstack-infra02:36
*** SpamapS has quit IRC02:36
*** SpamapS has joined #openstack-infra02:36
*** Clabbe has joined #openstack-infra02:37
*** vladan has joined #openstack-infra02:37
*** mgagne has quit IRC02:37
*** dripton__ has joined #openstack-infra02:37
*** NikitaKonovalov has joined #openstack-infra02:38
*** bhuvan has joined #openstack-infra02:38
*** jhesketh has joined #openstack-infra02:38
*** oubiwann_ has joined #openstack-infra02:38
*** tristanC has joined #openstack-infra02:38
*** gaelL_ has joined #openstack-infra02:38
*** jhesketh has quit IRC02:38
*** asalkeld has quit IRC02:38
*** dripton_ has quit IRC02:38
*** exz has quit IRC02:38
*** andre__ has joined #openstack-infra02:38
*** ruhe has joined #openstack-infra02:38
*** jhesketh has joined #openstack-infra02:38
*** mtreinish has joined #openstack-infra02:38
*** julim has joined #openstack-infra02:38
*** weshay has quit IRC02:38
*** weshay has joined #openstack-infra02:39
*** VijayT has joined #openstack-infra02:40
*** DinaBelova_ has joined #openstack-infra02:40
*** andre__ has quit IRC02:40
*** DinaBelova_ is now known as DinaBelova02:41
*** thedodd has joined #openstack-infra02:41
*** amotoki_ has joined #openstack-infra02:41
*** dkehn_ has joined #openstack-infra02:42
*** esker has joined #openstack-infra02:42
*** tru_tru has joined #openstack-infra02:42
*** gaelL has joined #openstack-infra02:43
*** gaelL_ has quit IRC02:43
*** lttrl has joined #openstack-infra02:43
*** mkerrin1 has joined #openstack-infra02:43
*** bnemec has joined #openstack-infra02:43
*** lcestari_ has joined #openstack-infra02:43
*** bodepd has joined #openstack-infra02:44
*** mgagne has joined #openstack-infra02:44
*** ilyashakhat_ has joined #openstack-infra02:45
*** SergeyLukjanov2 has joined #openstack-infra02:45
*** DinaBelova has quit IRC02:45
*** saper_ has joined #openstack-infra02:45
*** plomakin has joined #openstack-infra02:45
*** russellb_ has joined #openstack-infra02:45
*** b3nt_pin has joined #openstack-infra02:47
*** zul has quit IRC02:47
*** DinaBelova_ has joined #openstack-infra02:47
*** DinaBelova_ is now known as DinaBelova02:47
*** dhellmann_ has joined #openstack-infra02:48
*** motoki has joined #openstack-infra02:48
*** Ryan_Lane2 has joined #openstack-infra02:48
*** mikal has joined #openstack-infra02:49
*** bhuvan has quit IRC02:49
*** lyle has joined #openstack-infra02:49
*** bhuvan has joined #openstack-infra02:49
*** dstanek has quit IRC02:49
*** vladan has quit IRC02:49
*** nosnos has quit IRC02:49
*** david-lyle has quit IRC02:49
*** kevinbenton has quit IRC02:49
*** mkerrin has quit IRC02:49
*** sc68cal has quit IRC02:49
*** plomakin_ has quit IRC02:49
*** SergeyLukjanov has quit IRC02:49
*** bnemec_ has quit IRC02:49
*** mikal_ has quit IRC02:49
*** amotoki__ has quit IRC02:50
*** krtaylor has quit IRC02:50
*** dims has quit IRC02:50
*** rwsu has quit IRC02:50
*** ilyashakhat has quit IRC02:50
*** lifeless has quit IRC02:50
*** afazekas|sick has quit IRC02:50
*** amotoki has quit IRC02:50
*** gothicmindfood has quit IRC02:50
*** ianw has quit IRC02:50
*** saper has quit IRC02:50
*** belliott has quit IRC02:50
*** Ryan_Lane has quit IRC02:50
*** jpeeler has quit IRC02:50
*** dhellmann has quit IRC02:50
*** anteaya has quit IRC02:50
*** mburned has quit IRC02:50
*** russellb has quit IRC02:50
*** SergeyLukjanov2 is now known as SergeyLukjanov02:50
*** asalkeld has joined #openstack-infra02:50
*** dhellmann_ is now known as dhellmann02:50
*** russellb_ is now known as russellb02:50
*** dstanek has joined #openstack-infra02:50
*** vladan has joined #openstack-infra02:50
*** sc68cal has joined #openstack-infra02:51
*** kevinbenton has joined #openstack-infra02:51
*** NikitaKonovalov_ has joined #openstack-infra02:52
*** zul has joined #openstack-infra02:52
*** gothicmindfood has joined #openstack-infra02:54
*** mtreinish_ has joined #openstack-infra02:54
*** nosnos has joined #openstack-infra02:56
*** anteaya has joined #openstack-infra02:56
*** NikitaKonovalov has quit IRC02:57
*** mtreinish has quit IRC02:57
*** jhesketh has quit IRC02:57
*** NikitaKonovalov_ is now known as NikitaKonovalov02:57
*** mtreinish_ is now known as mtreinish02:57
*** jpeeler1 has joined #openstack-infra02:57
*** dims has joined #openstack-infra02:57
*** andre__ has joined #openstack-infra02:57
*** ianw has joined #openstack-infra02:57
*** lifeless has joined #openstack-infra02:58
*** gothicmi1dfood has joined #openstack-infra02:58
*** jhesketh has joined #openstack-infra02:59
*** thomasem has joined #openstack-infra02:59
*** thomasem has quit IRC02:59
*** krtaylor has joined #openstack-infra03:00
*** sarob has joined #openstack-infra03:01
*** rwsu has joined #openstack-infra03:01
*** whayutin_ has joined #openstack-infra03:02
*** dklyle has joined #openstack-infra03:03
*** pleia2 has quit IRC03:03
*** sarob has quit IRC03:03
*** pleia2 has joined #openstack-infra03:03
*** HenryG has quit IRC03:04
*** sarob has joined #openstack-infra03:04
*** gothicmindfood has quit IRC03:04
*** lttrl has quit IRC03:04
*** sc68cal has quit IRC03:04
*** belliott has joined #openstack-infra03:04
*** weshay has quit IRC03:04
*** yaguang has quit IRC03:04
*** lyle has quit IRC03:04
*** afazekas|sick has joined #openstack-infra03:04
*** bhuvan_ has joined #openstack-infra03:06
*** bhuvan has quit IRC03:06
*** belliott has quit IRC03:06
clarkbwoot cluster is green starting workers now03:07
*** branen_ has quit IRC03:08
*** yaguang has joined #openstack-infra03:08
*** sarob has quit IRC03:08
*** branen_ has joined #openstack-infra03:09
anteayago clarkb03:09
*** yamahata__ has quit IRC03:09
*** sc68cal has joined #openstack-infra03:10
*** yamahata__ has joined #openstack-infra03:10
*** belliott has joined #openstack-infra03:11
*** esker has quit IRC03:17
*** lttrl has joined #openstack-infra03:18
*** mburned has joined #openstack-infra03:20
*** dkehn_ has quit IRC03:21
*** pcrews has joined #openstack-infra03:21
clarkb64 gearman workers are running through 16 lgostash indexers03:21
clarkband the backlog is falling03:21
clarkbleaving kibana off for now, will start that later thsi evening when memory use on logstash.o.o shouldn't be too bad03:22
*** dkehn has joined #openstack-infra03:22
*** lttrl has quit IRC03:24
*** esker has joined #openstack-infra03:24
*** lttrl has joined #openstack-infra03:25
*** CaptTofu has quit IRC03:26
*** mgagne has quit IRC03:27
*** mgagne has joined #openstack-infra03:28
*** exz has joined #openstack-infra03:29
*** ProfFalken has joined #openstack-infra03:29
*** gyee has joined #openstack-infra03:29
*** bradm has joined #openstack-infra03:29
*** bogdando has joined #openstack-infra03:29
*** flaper87|afk has quit IRC03:29
*** flaper87|afk has joined #openstack-infra03:29
*** homeless_ has quit IRC03:30
*** homeless_ has joined #openstack-infra03:30
*** harlowja_ has quit IRC03:30
*** harlowja_ has joined #openstack-infra03:30
*** kmartin has quit IRC03:30
*** kmartin has joined #openstack-infra03:30
*** mkerrin1 has quit IRC03:30
*** mkerrin1 has joined #openstack-infra03:30
*** ianw has quit IRC03:30
*** ianw has joined #openstack-infra03:30
*** dklyle has quit IRC03:30
*** dklyle has joined #openstack-infra03:30
*** branen_ has quit IRC03:30
*** branen_ has joined #openstack-infra03:30
*** yamahata__ has quit IRC03:30
*** yamahata__ has joined #openstack-infra03:30
*** mburned has quit IRC03:30
*** mburned has joined #openstack-infra03:30
*** zul has quit IRC03:31
*** lttrl has quit IRC03:33
clarkbgoing to step away for a bit now while it chews through the backlog03:34
clarkbits down from 390k to about 370k already03:34
*** lttrl has joined #openstack-infra03:34
*** zul has joined #openstack-infra03:36
*** mgagne has quit IRC03:37
*** jishaom has joined #openstack-infra03:38
*** pcrews has quit IRC03:42
*** bhuvan_ has quit IRC03:42
*** bhuvan has joined #openstack-infra03:44
*** rcleere has joined #openstack-infra03:44
*** jhesketh_ has joined #openstack-infra03:47
*** mgagne has joined #openstack-infra03:48
*** senthil has joined #openstack-infra03:51
*** zul has quit IRC03:52
*** gokrokve has joined #openstack-infra03:55
*** chandan_kumar has joined #openstack-infra03:58
*** chandan_kumar has quit IRC04:09
*** chandan_kumar has joined #openstack-infra04:10
*** sarob has joined #openstack-infra04:14
*** asalkeld has left #openstack-infra04:16
*** bhuvan has quit IRC04:18
*** sarob has quit IRC04:19
clarkbdown to 337k, this is going to take some time04:20
*** ogelbukh1 has joined #openstack-infra04:20
*** hdd has joined #openstack-infra04:21
*** mikal has quit IRC04:26
*** zehicle_at_dell has joined #openstack-infra04:26
*** mikal has joined #openstack-infra04:27
*** senthil has quit IRC04:32
*** amcrn has quit IRC04:42
*** dcramer_ has joined #openstack-infra04:49
clarkbnow 317k, so about 1k every couple minutes05:00
clarkb~10 hours?05:00
clarkbmy maths are fuzzy05:01
jesusaurushow much swap were you eating up?05:01
jesusaurusand did you bump your es flush size?05:01
StevenKclarkb: 316/30 == 10.53, so roughly05:01
jesusaurusclarkb: ?05:01
clarkbjesusaurus: yes bumped es flush queue, es is doing over 10k events per second right now05:02
clarkbjesusaurus: right now I am down to 30MB of swap05:02
jesusaurusnice05:02
jesusaurus10k/s is pretty good05:03
*** cadenzajon has quit IRC05:05
*** krtaylor has quit IRC05:06
clarkbya I am happy with it, once it is caught up it should have no problem keeping up05:06
clarkbmemory use for logstash looks a lot better than before05:08
*** zul has joined #openstack-infra05:10
*** krtaylor has joined #openstack-infra05:11
*** krtaylor has quit IRC05:11
*** thedodd has quit IRC05:12
*** DinaBelova is now known as DinaBelova_05:15
*** mriedem has quit IRC05:18
*** hdd has quit IRC05:18
*** krtaylor has joined #openstack-infra05:19
*** Guest46806 has quit IRC05:20
openstackgerritA change was merged to openstack-infra/reviewstats: Add Sergey Lukjanov to infra-core  https://review.openstack.org/7312205:20
openstackgerritA change was merged to openstack-infra/reviewstats: Add new infra projects  https://review.openstack.org/7312305:21
openstackgerritA change was merged to openstack-infra/reviewstats: Update the Ironic core team list  https://review.openstack.org/7306805:21
*** lillie has joined #openstack-infra05:21
*** lillie is now known as Guest7531705:21
*** masayukig has joined #openstack-infra05:22
*** nicedice has quit IRC05:24
*** CaptTofu has joined #openstack-infra05:27
*** CaptTofu has quit IRC05:31
*** dstanek has quit IRC05:32
*** UtahDave has joined #openstack-infra05:32
*** masayukig has quit IRC05:32
*** masayukig has joined #openstack-infra05:32
openstackgerritKhai Do proposed a change to openstack-infra/gerrit: add a .gitreview file  https://review.openstack.org/7312605:34
*** jamielennox is now known as jamielennox|away05:34
*** oubiwann_ has quit IRC05:35
*** gyee has quit IRC05:40
*** zehicle_at_dell has quit IRC05:40
*** amcrn has joined #openstack-infra05:42
*** chandankumar_ has joined #openstack-infra05:49
*** chandankumar_ has joined #openstack-infra05:50
openstackgerritKhai Do proposed a change to openstack-infra/config: update comment filter for gerrit 2.8  https://review.openstack.org/7086405:50
*** fifieldt has joined #openstack-infra05:51
*** chandankumar_ has quit IRC05:52
*** dkehn has quit IRC05:52
*** tomhe has quit IRC05:52
*** chandan_kumar has quit IRC05:53
*** chandan_kumar has joined #openstack-infra05:54
*** dkehn has joined #openstack-infra05:54
*** gokrokve has quit IRC05:54
*** gokrokve has joined #openstack-infra05:55
*** UtahDave has quit IRC05:57
openstackgerritYongli He proposed a change to openstack-infra/storyboard: Remove extraneous vim configuration comments  https://review.openstack.org/7315605:57
*** gokrokve has quit IRC05:58
*** Ryan_Lane2 is now known as Ryan_Lane06:01
*** Ryan_Lane has joined #openstack-infra06:01
openstackgerritNathan Kinder proposed a change to openstack-infra/config: Add openstack-security-notes stackforge project  https://review.openstack.org/7315706:02
openstackgerritRussell Bryant proposed a change to openstack-infra/reviewstats: Fix Launchpad ID for Nikolay Starodubtsev  https://review.openstack.org/7315806:05
*** dkehn_ has joined #openstack-infra06:05
*** zul has quit IRC06:08
*** amotoki_ has quit IRC06:09
SergeyLukjanovjeblair, fungi, clarkb, good evening06:13
SergeyLukjanovhm, or night already06:13
fungiSergeyLukjanov: good morning!06:13
clarkbfungi: you are still awake?06:14
clarkbwe are down to 291k06:14
clarkbabout 100k less than where we started06:14
SergeyLukjanovis there anything with that I can help while you sleeping? :)06:14
fungiit's only a little after 1am local time where i am06:14
openstackgerritA change was merged to openstack-infra/reviewstats: Fix Launchpad ID for Nikolay Starodubtsev  https://review.openstack.org/7315806:14
fungicurrently trying to get caught up on e-mail06:15
*** afazekas|sick has quit IRC06:16
*** yolanda has joined #openstack-infra06:21
*** e0ne has joined #openstack-infra06:22
*** e0ne has quit IRC06:23
*** masayuki_ has joined #openstack-infra06:23
*** masayuki_ is now known as masayukig_06:23
*** masayukig has quit IRC06:24
*** masayukig_ is now known as masayukig06:24
*** w_ is now known as olaph06:26
*** sarob has joined #openstack-infra06:31
openstackgerritKhai Do proposed a change to openstack-infra/config: update All-Project configuration for gerrit 2.8  https://review.openstack.org/7081806:33
*** marun has quit IRC06:40
*** harlowja_ is now known as harlowja_away06:44
openstackgerritKhai Do proposed a change to openstack-infra/config: disable the jeepyb trivial rebase hook on review-dev.o.o  https://review.openstack.org/6980006:45
clarkbfungi: re the opendaylight third party tests06:46
clarkbfungi: I thought they were open and could just be tested upstream06:47
fungiwas there a separate thread on that? i've been behind on e-mail06:48
clarkbfungi: no,06:48
clarkbit just occurs to me that we went from the extreme of having no testing to the other where none of the interesting testing happens upstream...06:48
clarkbdocker is in that boat too06:48
fungiwonder why they're requesting a third party account06:48
clarkbfungi: I think because it is "required"06:48
fungiah06:49
clarkbthere is a disconnect between what we actually want which is testing, and what people think they need to do that06:49
clarkbgranted trying to spin up docker or opendaylight tests upstream just before i3 probably isn't going to happen :)06:49
clarkbstarting to get worried that the third party tests will just make it more difficult to upstream the tests06:50
clarkbbecause each of them will do things differently06:50
clarkbbut in the case of open source testable things we want consistency06:50
*** pblaho has joined #openstack-infra06:50
clarkbbut having tests is a good thing so I should just stop worrying and poke markmcclain06:51
fungiprobably the same for any other plugin which could be tested upstream but waited until they had a couple weeks to go to look into it06:51
zaroclarkb, fungi : does change id link work for you? https://review.openstack.org/#/c/6980006:52
zaroone in the last comment06:52
fungizaro i'm not where i can pull up gerrit easily but i'll try to have a look tomorrow06:53
*** fifieldt_ has joined #openstack-infra06:53
*** fifieldt has quit IRC06:53
clarkbzaro: no, look at the url it is using, the $1 isn't being expanded06:54
zaroclarkb: yeah, is that new problem?06:54
clarkbI think so06:54
clarkbdid we restart gerrit to pick up more changes?06:54
*** masayukig has quit IRC06:55
zaroi didn't if you are asking me.06:56
clarkbzaro: I know :) it may have been picekd up when gerrit restarted for project renames last week06:57
clarkbbut I wasn't aware of us merging any additional changes to the comment links06:57
zaroanyways it might be good to restart or something. i've noticed that it's been a little flaky.  sometimes it fails to submit a review.06:57
clarkbzaro: we need to fix the config06:57
clarkbrestarting won't help otherwis06:57
zaroi really noticed flakeness today, but previously it's been ok.06:58
zaroor maybe i'm just making more commits today than usual. getting all the gerrit upgrade patches in shape.06:59
*** motoki has quit IRC07:00
*** garyk has joined #openstack-infra07:02
*** denis_makogon has joined #openstack-infra07:04
*** wenlock has quit IRC07:12
*** dkehn__ has joined #openstack-infra07:19
*** sarob has quit IRC07:21
*** dkehn__ has quit IRC07:21
*** sarob has joined #openstack-infra07:21
*** dkehn__ has joined #openstack-infra07:21
*** dkehn_ has quit IRC07:22
*** dkehn has quit IRC07:22
*** dkehn has joined #openstack-infra07:24
*** sarob has quit IRC07:26
*** CaptTofu has joined #openstack-infra07:27
*** CaptTofu has quit IRC07:32
*** flaper87|afk is now known as flaper8707:33
fungiso...much...e-mail07:38
*** UtahDave has joined #openstack-infra07:38
fungithe rest will just have to wait for tomorrow07:39
fungi<- zzz07:39
openstackgerritLin Tan proposed a change to openstack-infra/jeepyb: Rename Openstack to OpenStack  https://review.openstack.org/7318007:42
openstackgerritA change was merged to openstack-infra/git-review: Add 2m timeout to tests  https://review.openstack.org/7122307:43
*** e0ne has joined #openstack-infra07:47
openstackgerritYuriy Taraday proposed a change to openstack-infra/git-review: Make Gerrit port and dir selection deterministic  https://review.openstack.org/7068507:51
*** VijayT has quit IRC07:56
*** rlandy has joined #openstack-infra07:58
*** mkoderer has quit IRC07:59
*** denis_makogon_ has joined #openstack-infra07:59
*** afazekas has joined #openstack-infra08:00
*** mkoderer has joined #openstack-infra08:00
*** denis_makogon has quit IRC08:02
openstackgerritAndreas Jaeger proposed a change to openstack-infra/config: Publish link to docs-drafts for successful builds  https://review.openstack.org/7318508:06
*** cyeoh has quit IRC08:12
*** cyeoh has joined #openstack-infra08:12
*** fifieldt_ has quit IRC08:13
*** locke105 has quit IRC08:13
*** bodepd has quit IRC08:13
*** SpamapS has quit IRC08:14
*** greghaynes has quit IRC08:14
*** bodepd has joined #openstack-infra08:14
*** fifieldt_ has joined #openstack-infra08:14
*** locke105 has joined #openstack-infra08:15
*** SpamapS has joined #openstack-infra08:15
*** SpamapS has quit IRC08:15
*** SpamapS has joined #openstack-infra08:15
*** openstackgerrit has quit IRC08:16
*** persia has quit IRC08:16
*** branen_ has quit IRC08:16
*** branen_ has joined #openstack-infra08:17
*** leifmadsen has quit IRC08:17
*** lifeless has quit IRC08:17
*** persia has joined #openstack-infra08:18
*** persia is now known as Guest4378108:18
*** lifeless has joined #openstack-infra08:19
*** Guest43781 has quit IRC08:20
*** Guest43781 has joined #openstack-infra08:20
*** Guest43781 is now known as persia08:20
*** orionNebula has quit IRC08:20
*** orionNebula has joined #openstack-infra08:21
*** sarob has joined #openstack-infra08:22
*** mattymo has joined #openstack-infra08:23
*** mrmartin has joined #openstack-infra08:23
*** che-arne has joined #openstack-infra08:24
AJaegerIs anybody around to tell me why Zuul/Jenkins are still using for openstack-operations-guide-ja the old job? This was changed 12 hours ago08:24
*** cp16net has quit IRC08:25
*** cp16net has joined #openstack-infra08:25
*** leifmadsen has joined #openstack-infra08:27
*** sarob has quit IRC08:28
*** mattymo has quit IRC08:28
*** DeeJay1 has joined #openstack-infra08:32
*** jcoufal has joined #openstack-infra08:35
*** shardy_afk is now known as shardy08:35
*** jpich has joined #openstack-infra08:36
*** bogdando has quit IRC08:38
*** bogdando has joined #openstack-infra08:39
*** dizquierdo has joined #openstack-infra08:44
*** saju_m has joined #openstack-infra08:50
*** fifieldt_ has quit IRC08:57
*** hashar has joined #openstack-infra09:01
*** llu has joined #openstack-infra09:03
*** llu has left #openstack-infra09:03
*** jgallard has joined #openstack-infra09:03
*** derekh has joined #openstack-infra09:09
*** yassine has joined #openstack-infra09:11
*** mkerrin1 has quit IRC09:13
*** dkliban has quit IRC09:21
*** sarob has joined #openstack-infra09:25
*** rossella_s has joined #openstack-infra09:25
*** markmc has joined #openstack-infra09:26
*** denis_makogon_ has quit IRC09:28
*** CaptTofu has joined #openstack-infra09:28
*** sneelakantan has joined #openstack-infra09:30
*** DinaBelova_ is now known as DinaBelova09:32
*** CaptTofu has quit IRC09:33
*** IvanBerezovskiy has joined #openstack-infra09:36
*** jooools has joined #openstack-infra09:47
*** jishaom has quit IRC09:56
SergeyLukjanovAJaeger, hey, looking on it09:56
AJaegerSergeyLukjanov, thanks!09:56
AJaegerhere's the logfile https://jenkins01.openstack.org/job/openstack-operations-guide-ja/81/09:56
*** sarob has quit IRC09:57
AJaegerAnd this is the patch you approved last night: https://review.openstack.org/#/c/71056/09:57
AJaegerI see updates for the other patch I pinged  active - just not this one.09:58
* AJaeger is completely puzzled09:58
SergeyLukjanovAJaeger, hm zuul  Last reconfigured: Thu Feb 13 2014 02:39:08 GMT+0400 (VOLT)09:59
AJaegerso, that looks fine10:00
AJaegerit wsa approved 4 hours or so before10:00
SergeyLukjanovAJaeger, oh, it's not about zuul, you only changed corresponding job10:00
AJaegerSergeyLukjanov, you're right. So, this is jenkins problem?10:00
AJaegerSorry for misleading you10:00
SergeyLukjanovAJaeger, np, it looks like job hasn't been updated10:02
SergeyLukjanovAJaeger, thinking about how to check it10:02
*** bookwar has quit IRC10:03
*** bookwar has joined #openstack-infra10:03
*** mattymo has joined #openstack-infra10:03
mattymoI noticed that in the last 8 hours openstackgerrit died10:03
mattymoanyone monitoring it? maybe our dearest SergeyLukjanov ?10:03
mrmartinSergeyLukjanov: hi, I have the same problem here: http://logs.openstack.org/94/72294/2/check/gate-groups-unittests/a20b42b/console.html#_2014-02-13_08_46_04_53010:03
mrmartinI made same modification yesterday on related gating job, but the previous one was executed.10:04
SergeyLukjanovmattymo, hey, I have no root access to this server to reboot it10:04
mattymoSergeyLukjanov, how about some puppet triggers :)10:05
SergeyLukjanovmrmartin, could you please share the link to your config change?10:06
mrmartinof course: https://review.openstack.org/#/c/68912/10:06
AJaegerSergeyLukjanov, my issue is not urgent that it needs fixing *now*, if you want to wait for others to show up, no problem. Just wanted to bring it up here...10:08
SergeyLukjanovAJaeger, ok10:09
*** sneelakantan has left #openstack-infra10:13
*** greghaynes has joined #openstack-infra10:14
SergeyLukjanovAJaeger, the only idea is that jenkins job update wasn't applied on the moment of job execution, but it's really strange, I have no access to check the job correctness10:15
SergeyLukjanovAJaeger, but probably, you could try to submit another change to check how it works now10:16
AJaegerSergeyLukjanov, my job was submitted two hours ago which should have been enough time. The first job after the merge was after 40 mins and also failed10:16
SergeyLukjanovAJaeger, oh10:17
*** sarob has joined #openstack-infra10:22
SergeyLukjanovAJaeger, I confirm that "./tools/generatedocbook -l ja -b openstack-ops" is still in job "openstack-operations-guide*" on all jenkins servers :(10:23
AJaegerSergeyLukjanov, so no need to test - this is wrong.10:24
SergeyLukjanovAJaeger, yup10:24
AJaegerSergeyLukjanov, thanks for checking!10:24
SergeyLukjanovAJaeger, I have no access to retrigger jjb but I'll think about how to workaround it :)10:24
SergeyLukjanovmrmartin, looking on your issue now10:24
mrmartinthanks10:25
*** CaptTofu has joined #openstack-infra10:25
AJaegerSergeyLukjanov, I'm still fine to wait for other that teach you how to do it properly later ;)10:26
SergeyLukjanovAJaeger, I think that it requires just puppet-agent --test but I have no root access :)10:26
AJaegerSergeyLukjanov, neither have I ;)10:26
SergeyLukjanovAJaeger, heh, looks like I need to ask access to some servers to be able to repair some stuff like this10:27
*** jp_at_hp has joined #openstack-infra10:27
AJaegerSergeyLukjanov, you're still new in the job - so I expect some more of these missing permissions ;(10:28
*** che-arne has quit IRC10:29
SergeyLukjanovAJaeger, yeah, but I'm not the admin guy, so, I'd like to keep amount of servers that I have access to as small as possible :)10:30
AJaegerAh, ok.10:30
SergeyLukjanovAJaeger, heh, don't want to break the world while everyone else in team sleeping ;)10:31
AJaegerIf you rather review, here's one think I noticed that could help my last two changes: https://review.openstack.org/7318510:31
SergeyLukjanovAJaeger, anyway, I hope we'll find the solution to grant me enough permissions to help :)10:31
AJaegerSergeyLukjanov, I thought the expectation is that you fix all the problems of the world while the rest of the team is sleeping ;)10:32
mrmartinI think we need to wait until the US infra team wake up.10:33
SergeyLukjanovmrmartin, I confirm that you have correct jobs on all jenkins servers now for groups, both unittests and release10:33
*** e0ne has quit IRC10:33
mrmartinSergeyLukjanov, had you made any change, or just checked the files?10:33
*** andreaf has joined #openstack-infra10:33
*** mrfatstrat has joined #openstack-infra10:34
SergeyLukjanovmrmartin, I've check it directly in jenkins10:34
mrmartinso it is a mysterious bug10:34
SergeyLukjanovmrmartin, oh, it looks like you're calling apt-get update under the bootstrap.sh10:36
mrmartinI now, but it was the previous script, and yesterday's patch eliminated the bootstrap.sh call :)10:37
SergeyLukjanovmrmartin, and it fails due to permission denied10:37
SergeyLukjanovmrmartin, oops, I was looking on https://review.openstack.org/#/c/68912/5/modules/openstack_project/files/jenkins_job_builder/config/groups.yaml :(10:37
mrmartinmy question is why the bootstrap.sh was called, when we removed it10:37
SergeyLukjanovmrmartin, it means that jenkins jobs configs wasn't updated for some time10:38
SergeyLukjanovmrmartin, but not for your case, https://review.openstack.org/#/c/72921/2 isn't yet merged :)10:39
mrmartinok, Sorry10:39
mrmartinhttps://review.openstack.org/#/c/72921/10:39
SergeyLukjanovmrmartin, so, you should wait until it'll be merged and re-check after that10:39
SergeyLukjanovI'm adding it to my baklog10:40
SergeyLukjanovbacklog*10:40
mrmartinI see now, I thought it was merged yesterday, I just see that jeblair just review it :)10:40
SergeyLukjanovAJaeger, anyway, your changes to jobs wasn't applied and it's bas10:40
mrmartinoh thanks10:40
SergeyLukjanovbad*10:40
AJaegerSergeyLukjanov, yeah, agreed.10:41
*** mkerrin has joined #openstack-infra10:42
*** sarob has quit IRC10:55
*** UtahDave has quit IRC10:57
*** hashar is now known as hasharCodingDojo11:01
*** ArxCruz has joined #openstack-infra11:02
*** yaguang has quit IRC11:03
*** w_ has joined #openstack-infra11:08
*** jgallard has quit IRC11:10
*** olaph has quit IRC11:10
*** CaptTofu has quit IRC11:13
*** hasharCodingDojo has quit IRC11:21
*** sarob has joined #openstack-infra11:22
*** jooools has quit IRC11:24
*** jooools has joined #openstack-infra11:24
*** ArxCruz has quit IRC11:29
*** notmyname has quit IRC11:29
*** ArxCruz has joined #openstack-infra11:30
*** CaptTofu has joined #openstack-infra11:31
*** pleia2 has quit IRC11:31
*** dragondm has quit IRC11:31
*** dhellmann has quit IRC11:31
*** pleia2 has joined #openstack-infra11:32
*** dhellmann has joined #openstack-infra11:32
*** notmyname has joined #openstack-infra11:32
*** sileht has quit IRC11:32
*** dkehn_ has joined #openstack-infra11:32
*** dragondm has joined #openstack-infra11:32
*** antonym has quit IRC11:32
*** apevec has joined #openstack-infra11:33
*** krtaylor has quit IRC11:33
*** sileht has joined #openstack-infra11:33
apevecDaviey, ping11:33
*** tteggel has joined #openstack-infra11:33
*** tteggel_ has joined #openstack-infra11:33
*** sc68cal has quit IRC11:34
apevecDaviey, last time adam_g reported Ubuntu CI results running against stable branch, I'd like to see those now11:34
*** tteggel_ has quit IRC11:34
apeveclast patch exception was merged to neutron havana, and I'm ready to tag11:34
*** dkehn_ has quit IRC11:35
*** tteggel_ has joined #openstack-infra11:35
*** dkehn__ has quit IRC11:35
*** dkehn has quit IRC11:35
*** dkehn_ has joined #openstack-infra11:35
*** sc68cal has joined #openstack-infra11:37
*** dkehn has joined #openstack-infra11:37
*** krtaylor has joined #openstack-infra11:38
*** tteggel_ has quit IRC11:40
*** tteggel_ has joined #openstack-infra11:41
*** tteggel_ has joined #openstack-infra11:42
*** tteggel_ has quit IRC11:42
*** tteggel has quit IRC11:42
*** tteggel has joined #openstack-infra11:42
*** mrmartin has quit IRC11:43
*** e0ne has joined #openstack-infra11:44
*** e0ne has quit IRC11:48
*** e0ne has joined #openstack-infra11:51
*** aude has quit IRC11:55
*** sarob has quit IRC11:55
*** mrfatstrat has quit IRC11:57
Davieyapevec: Ah, good call. I presume Adam hasn't offered the results already?11:57
apevecno, he just mentioned on irc last time, but w/o public url11:58
apevecis there any?11:58
DavieyGood question...11:58
Daviey(looking)11:58
*** vkozhukalov has joined #openstack-infra11:59
*** dstanek has joined #openstack-infra12:00
*** aude has joined #openstack-infra12:04
*** ociuhandu has quit IRC12:10
*** jgallard has joined #openstack-infra12:16
*** jcooley has quit IRC12:17
*** EntropyWorks has quit IRC12:18
*** mrmartin has joined #openstack-infra12:20
*** jcooley has joined #openstack-infra12:20
*** yamahata has quit IRC12:22
*** EntropyWorks has joined #openstack-infra12:22
*** sarob has joined #openstack-infra12:22
*** rfolco has joined #openstack-infra12:23
*** talluri has joined #openstack-infra12:26
*** hashar has joined #openstack-infra12:28
*** CaptTofu has quit IRC12:39
*** jooools has quit IRC12:40
*** jooools has joined #openstack-infra12:41
*** jhesketh has quit IRC12:42
isviridovHi infra team. Hi SergeyLukjanov. I would like to ask about https://review.openstack.org/#/c/71308/, should i do anything to make it merged?12:42
*** NikitaKonovalov is now known as NikitaKonovalov_12:43
isviridovSergeyLukjanov, around?12:48
dolphmcan anyone push this one over the edge to gate keystoneclient on py33? https://review.openstack.org/#/c/72831/12:48
dolphmi'd like to cut a release gated against py3312:49
*** xBsd has joined #openstack-infra12:52
*** sarob has quit IRC12:55
*** smarcet has joined #openstack-infra12:59
*** dstanek has quit IRC13:01
SergeyLukjanovdolphm, sure, I'll do it13:02
*** b3nt_pin is now known as beagles13:02
dolphmSergeyLukjanov: thanks!13:02
SergeyLukjanovisviridov, hey, I'll take a look13:02
*** xBsd has quit IRC13:02
SergeyLukjanovdolphm, np, done13:03
SergeyLukjanovisviridov, we currently have some problems with manage-projects script that is used for new projects creation, so, you should wait for the fix13:03
*** jcoufal has quit IRC13:04
SergeyLukjanovisviridov, no estimates, here is an issue - https://bugs.launchpad.net/openstack-ci/+bug/124256913:05
*** pdmars has joined #openstack-infra13:05
*** jcoufal has joined #openstack-infra13:05
isviridovSergeyLukjanov, thx for update13:05
*** NikitaKonovalov_ is now known as NikitaKonovalov13:05
*** jhesketh_ has quit IRC13:07
*** nosnos has quit IRC13:10
*** dprince has joined #openstack-infra13:10
*** heyongli has joined #openstack-infra13:12
*** zul has joined #openstack-infra13:14
SergeyLukjanovfungi, clarkb, jeblair, short update - looks like gerrit bot is down, job configs from https://review.openstack.org/#/c/71056/ doesn't applied to jenkins jobs :(13:20
*** senthil has joined #openstack-infra13:21
*** sarob has joined #openstack-infra13:22
*** esker has quit IRC13:23
*** esker has joined #openstack-infra13:23
*** lcostantino has joined #openstack-infra13:25
*** dcramer_ has quit IRC13:27
*** dstanek has joined #openstack-infra13:27
*** mrmartin has quit IRC13:28
*** esker has quit IRC13:28
ewindischI'm inclined to remove this section from 3rd-party testing: http://ci.openstack.org/third_party.html#the-jenkins-gerrit-trigger-plugin-way13:28
ewindischthe jenkins-gerrit-trigger doesn't support rechecks.13:29
ewindischunless the 'support rechecks' requirement isn't (a requirement).13:30
*** mrmartin has joined #openstack-infra13:30
*** antonym has joined #openstack-infra13:31
*** jamespage has joined #openstack-infra13:32
jamespageapevec, hi - you where after adam_g to run testing against stable branch packages right?13:33
*** daniil has joined #openstack-infra13:38
*** w_ is now known as olaph13:43
*** prad has joined #openstack-infra13:44
*** oubiwann_ has joined #openstack-infra13:46
*** yamahata has joined #openstack-infra13:49
*** prad_ has joined #openstack-infra13:50
*** prad has quit IRC13:51
*** prad_ is now known as prad13:51
*** mrfatstrat has joined #openstack-infra13:52
*** che-arne has joined #openstack-infra13:52
*** markwash has joined #openstack-infra13:55
*** oanufriev has joined #openstack-infra13:55
*** sarob has quit IRC13:55
*** gokrokve has joined #openstack-infra13:57
*** heyongli has quit IRC14:03
*** che-arne has quit IRC14:06
*** dkranz has joined #openstack-infra14:11
*** mrmartin has quit IRC14:13
*** mrmartin has joined #openstack-infra14:13
*** ociuhandu has joined #openstack-infra14:14
*** oubiwann_ has quit IRC14:16
*** nkinder has quit IRC14:17
*** sarob has joined #openstack-infra14:22
*** saju_m has quit IRC14:28
*** ok_delta has joined #openstack-infra14:30
*** esker has joined #openstack-infra14:34
*** esker has quit IRC14:34
*** esker has joined #openstack-infra14:35
*** max_lobur is now known as max_lobur_afk14:36
*** homeless_ has quit IRC14:38
*** homeless has joined #openstack-infra14:38
*** dims has quit IRC14:38
*** CaptTofu has joined #openstack-infra14:40
*** fifieldt has joined #openstack-infra14:40
*** sarob has quit IRC14:40
*** jhesketh has joined #openstack-infra14:41
sdaguegerrit bot dead?14:41
anteayaaccording to SergeyLukjanov, yes14:42
*** fifieldt has quit IRC14:43
*** dcramer_ has joined #openstack-infra14:43
SergeyLukjanovsdague, anteaya, yup14:43
sdagueok, cool14:44
sdagueon the up side, git review was extra fast for me this morning (faster than I can remember it being ever)14:44
sdagueso something got better on the gerrit side14:44
SergeyLukjanovsdague, we could upload/merge some dirty stuff ;)14:44
*** CaptTofu has quit IRC14:44
*** olaph has quit IRC14:45
anteayaSergeyLukjanov: bad code would be removed14:45
anteayapoor taste would be shamed14:45
*** ryanpetrello has joined #openstack-infra14:45
SergeyLukjanovanteaya, ;(14:45
anteayaand good morning14:47
*** oubiwann_ has joined #openstack-infra14:47
*** oubiwann_ has quit IRC14:48
*** oubiwann_ has joined #openstack-infra14:48
SergeyLukjanovyeah, morning ;)14:48
*** oubiwann_ has quit IRC14:49
*** mrfatstrat has quit IRC14:50
*** oubiwann_ has joined #openstack-infra14:50
*** dims has joined #openstack-infra14:51
*** wenlock has joined #openstack-infra14:52
ttxfungi: are we still using tools/atc/email-stats.sh to generate ATC lists ? Or are you consuming governance/programs.yaml instead ?14:53
SergeyLukjanovttx, does governance/programs contain all the repos from email-stats?14:57
SergeyLukjanovttx, and non-code atcs?14:57
ttxemail-stats is not the reference of anything.14:57
SergeyLukjanovoh, rollback, it does14:57
SergeyLukjanovttx, ok14:57
ttxit duplicates information, so it should die14:58
ttxand just read info from where it lives14:58
SergeyLukjanovttx, agreed14:58
*** olaph has joined #openstack-infra14:58
*** AlexF has joined #openstack-infra15:00
apevecjamespage, yes, I'd like to get ack from adam_g before pushing tags15:01
apevecjamespage, do you have CI running against latest stable/havana?15:01
jamespageapevec, just kicked off a run now15:01
apevecthanks!15:02
jamespage(needed to fixup a few package build failures first - now done)15:02
mrmartinSergeyLukjanov, this patch 72294,3 was posted into check more than 1 hour ago and the gate-groups-unittests job is queued. Could we check somewhere what happens?15:02
*** nkinder has joined #openstack-infra15:03
anteayaapevec: you understand the purpose for my comment on 72754?15:03
*** mfer has joined #openstack-infra15:03
apevecanteaya, "This patch has failed in the gate 4 times in a row." - not quite :)15:04
anteayawould you like to know?15:04
apevecyes please15:04
anteayaI was letting you know I was watching the patch15:04
apevecbtw it merged, looks like stable gate likes EU morning15:04
anteayaand that it had failed in the gate 4 times in a row15:04
SergeyLukjanovmrmartin, it's misconfigured now, looks like by the latest patch15:04
dimslogstash seems to be down15:04
apevecanteaya, but poor patch is innocent :)15:04
anteayaand that if it would be sent back in and block other patches, I would remove it15:05
anteayaapevec: I think we have a failure of understanding between us15:05
fungiSergeyLukjanov: i'll get gerritbot restarted--this is not an uncommon occurrence (it has a tendency not to recover from netslipts due to a bug in the underlying framework)15:05
anteayaall code is innocent15:05
*** oanufriev has quit IRC15:05
apevecanteaya, ihrachys is now looking what Tempest patches should be backported15:05
*** markwash has quit IRC15:05
anteayaall is made up of ones and zeros which alone and by themselves are innocent15:05
anteayait is only in combination that problems occur15:05
fungittx: yes, using that but i modified it to parse the governance repo. i'll see if i forgot to submit a review with that improvement15:05
SergeyLukjanovfungi, morning, great15:06
ttxfungi: cool. Just want people to stop proposig patch to that list as if it was the canonical list15:06
anteayaapevec: great, well that should improve stablility for testing for stable/havana15:06
mrmartinSergeyLukjanov, what is misconfigured?15:06
anteayaand decrease failures in the gate, decreasing my need to remove them15:06
SergeyLukjanovmrmartin, looks like we've missed that https://github.com/openstack-infra/config/commit/15761d181a8110afa1cfdcb15922de88bcb047d8 going to use {node} but it's not set for project15:07
anteayaapevec: do you have a better sense of my motivation for my actions?15:07
mrmartinSergeyLukjanov, ok, what we need to change?15:07
apevecanteaya, also separate gate queue for stable would help, since master and stable and not equal re. stability15:07
*** mgagne has quit IRC15:07
anteayathat is an interesting idea15:07
apevecanteaya, I understand your actions completely15:07
SergeyLukjanovmrmartin, looking on it atm15:07
anteayaa separate gate queue for stable15:08
anteayaapevec: great, thanks I am glad we are there15:08
*** wenlock has quit IRC15:08
anteayaapevec: if you would like to propose a separate gate for stable, I would encourage you to add an item to the infra meeting agenda so that it can be discussed15:08
mrmartinSergeyLukjanov, I cloned those parts from storyboard-webclient, seems to me something was missed15:08
mtreinishanteaya: we can't do a separate gate queue for stable because stable has implications for grenade tests15:08
mtreinishit needs to be integrated with everything because of that15:09
anteayamtreinish: thank you15:09
anteayaah15:09
mtreinishotherwise we'll wedge15:09
*** nicedice has joined #openstack-infra15:09
anteayano weding15:09
SergeyLukjanovmrmartin, I see in job config label: {node}15:09
anteayawedging even15:09
SergeyLukjanovmrmartin, I mean I see it in jenkins, so, {node} should be replaced with actual value15:09
apevecanteaya, iirc fungi & co entertained the idea already, I'll add as an item (I'm not sure what it would take)15:09
anteayaneither do I15:10
anteayahopefully mtreinish will also attend and bring his perspective to the discussion15:10
mtreinishanteaya: I'm normally around during the infra meeting15:11
anteayamtreinish: great15:11
anteayashould make for a good discussion15:11
SergeyLukjanovmrmartin, IMO it should be taken from the node definition here - https://review.openstack.org/#/c/72921/2/modules/openstack_project/files/jenkins_job_builder/config/projects.yaml15:11
fungiapevec: the problem is that stable can't be gated separately, because we have upgrade testing15:11
mrmartinyeap, so, do I need to change anything? I thought the same.15:12
apevecfungi, why is that a problem? Master tests h->i, stable/havana g->h ?15:13
SergeyLukjanovmrmartin, I'm trying to understand why it's not working, no specific ideas atm15:13
*** AlexF has quit IRC15:13
mrmartinSergeyLukjanov, ok thanks15:13
fungiapevec: we test that patches to stable don't prevent upgrading to the next release, and that patches to the next release don't prevent you from being unable to upgrade from stable15:13
SergeyLukjanovfungi, could you please clarify that in this case https://review.openstack.org/#/c/72921/2/modules/openstack_project/files/jenkins_job_builder/config/projects.yaml "node=precise" will be passed to gate-groups-unittests job template?15:14
*** oanufriev has joined #openstack-infra15:14
fungiapevec: but regardless, separatin the gate pipelines for stable and master is just a workaround. the proper solution is to get enough people who care about stable to backport fixes to it and keep it in shape. with just a handful of us it's fairly insurmountable15:14
apevectrue that15:15
fungibut i'm afraid unless we just declare it bankrupt, not many other entities are likely to step up and help15:15
apevecyeah, I tried making noise on the lists, I'll keep trying :)15:15
fungiSergeyLukjanov: having a look15:16
*** jnoller has joined #openstack-infra15:16
anteayafungi apevec right now how many people work on stable maintenance?15:16
SergeyLukjanovfungi, thx15:16
apevecanteaya, in theory it's stable-maint group https://review.openstack.org/#/admin/groups/120,members15:16
*** jnoller_ has joined #openstack-infra15:16
fungianteaya: depends on how you count them15:16
*** jooools has quit IRC15:16
*** mfer has quit IRC15:16
*** jooools1 has joined #openstack-infra15:16
apevecbut not all are active15:16
*** jnoller_ has quit IRC15:16
apevecand we have active backporters who aren't core15:17
*** jnoller_ has joined #openstack-infra15:17
*** CaptTofu has joined #openstack-infra15:17
anteayafungi: how do you count them?15:17
apevecalso stable-maint doesn't have +2 on Tempest15:17
*** jnoller_ has quit IRC15:17
apevecstable/* branches of Tempest that is15:17
anteayaapevec: so you need someone who has +2 on Tempest to be in the group?15:18
anteayawould that help?15:18
*** jergerber has joined #openstack-infra15:18
apevecI hope so15:18
mtreinishapevec, anteaya: the tempest core team has +2 on tempest stable15:18
*** banix has joined #openstack-infra15:18
mtreinishwhen we went to do our first backport no one had permission to do it :)15:18
apevecmtreinish, would you like to join stable-maint ?15:18
apevecwe need some intersection15:18
anteayawould seem to fill a need for both15:18
mtreinishI knew I shouldn't have said anything...15:19
mtreinishapevec: sure I guess15:19
*** jnoller_ has joined #openstack-infra15:19
*** jnoller has quit IRC15:19
anteayaapevec: how does he join?15:19
apevecfor all other projects each PTL is stable-maint member by default15:19
anteayaah15:19
apevecanteaya, I'll fast track him right now :)15:19
fungianteaya: i think he just did ;)15:19
anteayahence the length of the list15:19
anteayaapevec fungi well it helps15:19
anteayaI seen neither jeblair or fungi on the stable-maint list15:20
fungimuch like core security reviewers, each project ideally would have at least a couple people taking care to propose or at least shepherd in backports and watch the nightly bitrot job results and so on15:20
anteayathought you were on it, fungi15:21
fungianteaya: i don't want to be in the position of approving changes to stable branches15:21
anteayaah I am thinking of core security15:21
anteayafair enough15:21
anteayaapevec: who is active for stable-maint15:21
fungii mostly notice stable breakage when trying to get security fixes tested15:21
anteayaah15:21
*** mkerrin has quit IRC15:21
apevecttx, ^^^ do you have your stats for stable-maint?15:21
fungiwhich is fairly often, and then i do my best to track down and fix whatever's gone awry15:21
anteayafungi: ah so you offer fixes but are not core15:22
apevecttx, it's perhaps time to review membership15:22
apevecor we do it at I release?15:22
*** sarob has joined #openstack-infra15:22
SergeyLukjanovfungi, additionally, https://review.openstack.org/#/c/71056/ still wasn't applied (>15h) due to the https://jenkins04.openstack.org/view/All/job/openstack-operations-guide-ja/configure15:22
sdagueapevec: so we could add stable-maint to approve on tempest stable branches15:22
sdaguethe reality is that when we did our first backport, the stable-maint team at the time had no interest in being part of that group15:23
ttxapevec: well, the trick is the original script isn't stable/* compatible15:23
ttxI remember tweaking it though, let me see if I kept that around15:24
*** prad has quit IRC15:25
*** ArxCruz has quit IRC15:25
*** prad has joined #openstack-infra15:25
*** AlexF has joined #openstack-infra15:25
dhellmannsdague: do you know anything about the history of https://bugs.launchpad.net/devstack/+bug/1260723 ? I seem to be hitting it even after a fix has merged.15:25
apevecsdague, looks like that need to change and stable-maint should learn Tempest better15:25
*** zhiyan has joined #openstack-infra15:25
apevecsdague, I'd like we have intersection, so I've added mtreinish to stable-maint15:26
apevecmtreinish, welcome to stable-maint https://review.openstack.org/#/admin/groups/120,members !15:26
sdaguesure15:26
mtreinishapevec: thanks15:26
anteayayay15:26
*** zhiyan is now known as zhiyan_15:26
anteayadhellmann: you are on the stable-maint list15:27
dhellmannanteaya: yes15:27
fungianteaya: but yeah, i'm a release cycle management project core team member by way of being a vulnerability management team member, though we have some separation-of-concerns in place to keep release management and vulnerability management and stable branch management siloed, aside from our illustrious ptl of course15:27
anteayadhellmann: is this an accurate reflection of your interest?15:27
*** sarob has quit IRC15:27
*** ArxCruz has joined #openstack-infra15:27
fungis/project/program/15:27
dhellmannanteaya: I am interested in helping, yes15:27
anteayafungi: fair enough15:27
anteayadhellmann: great, how do I support that?15:27
sdaguedhellmann: yes, though that's not an actual failure15:27
anteayasince apevec is needing some stable-maint help15:28
sdaguesahid was working on that yesterday15:28
dhellmannanteaya: I'm not sure I understand the question15:28
dhellmannsdague: http://logs.openstack.org/37/72437/1/check/check-devstack-dsvm-cells/6272c5f/console.html15:28
apevecanteaya, ttx will do periodic membership cleanups (was already done in the past)15:28
apevecbased on (in)activity15:28
dhellmannsdague: am I misinterpreting that log?15:28
apevecas in reviews15:28
anteayadhellmann: apevec is feeling that stable-maint is not getting enough attention, and slow backports are affecting release timing15:28
sdaguedhellmann: I think so15:28
sdaguethe actually failure is a server 500 on volumes create15:29
sdaguehttp://logs.openstack.org/37/72437/1/check/check-devstack-dsvm-cells/6272c5f/console.html#_2014-02-13_13_49_57_79315:29
ttxapevec: will pastebin in a few15:29
*** chandan_kumar has quit IRC15:29
*** chandankumar_ has joined #openstack-infra15:29
anteayadhellmann: I am helping to acertain who can help to improve the speed of stable-maint backports to help releases15:29
dhellmannsdague: ok, I thought that might have been caused by the credentials thing15:29
dhellmannsdague: I'll look for another bug to recheck against15:29
anteayadhellmann: so I am asking people on the list if their presence is an accurate reflection of interest15:29
dhellmannanteaya: my time is limited, but I do try to help15:30
*** mgagne has joined #openstack-infra15:30
sdaguedhellmann: http://logs.openstack.org/37/72437/1/check/check-devstack-dsvm-cells/6272c5f/logs/screen-c-api.txt.gz15:30
anteayaso apevec knows whom he can talk to regarding getting stable-maint work done15:30
sdaguetaskflow / stevedore thing15:30
*** mkerrin has joined #openstack-infra15:30
anteayadhellmann: you are a helping kind of person, which is why I asked you15:30
anteayaapevec is there something dhellmann can help with?15:30
sdaguedhellmann: the fact that it fails all the devstack jobs is usually a good indication that it's a pretty core issue :)15:31
*** tomhe has joined #openstack-infra15:31
*** jcoufal has quit IRC15:31
sdaguebasically I wonder if taskflow git actually works15:32
dhellmannsdague: hrm, I'll have to ask harlowja_away what that means15:32
dhellmannsdague: thanks15:32
apevecanteaya, dunno, up to him, but latest failures are nova/neutron area afaict15:32
*** rcleere has quit IRC15:33
anteayaapevec: okay, well going to be difficult for me to support getting some people to help with stable-maint if they don't have something to help with15:33
*** mgagne1 has joined #openstack-infra15:34
dhellmannapevec: were any of those neutron fixes for the isolated gate jobs backported? is that what needs review?15:34
*** marun has joined #openstack-infra15:34
apevecihrachys has dug out some master Tempest changes which could help15:34
fungianteaya: i think probably one of the biggest helps to stable branch management would be people subscribing to openstack-stable-maint@lists.openstack.org and looking at the bitrot job failure reports ("Stable check of <fooproject> failed" from jenkins)15:34
*** mrodden has joined #openstack-infra15:34
dhellmannapevec: I saw mtreinish's email to the list, but haven't looked at the reviews he linked15:34
fungianteaya: and then trying to stay on top of fixing them as they crop up15:34
mtreinishdhellmann: there were a lot of them15:35
*** mgagne has quit IRC15:35
mtreinishimproving the isolated job was one of the big pushes at montreal15:35
dhellmannmtreinish: right, I'm trying to understand if we're trying to backport them or if they are only going into master15:35
dhellmannmtreinish: because if we're backporting then it would be good to have a list to review and if we're not then I agree we should talk about turning that gate job off15:35
* anteaya joins openstack-stable-maint@lists.openstack.org15:36
*** wenlock has joined #openstack-infra15:36
*** thomasem has joined #openstack-infra15:37
mtreinishdhellmann: I really think it'll be better to just turn it off. We mostly added the job so neutron would feel the pain and try to prioritize fixing it.15:37
*** AlexF has quit IRC15:37
mtreinishthe job has been broken for master on and off the whole cycle15:38
dhellmannmtreinish: yeah, if we know it doesn't work it's just pissing everyone off to keep it15:38
*** rockstar has joined #openstack-infra15:38
*** rockstar has quit IRC15:38
*** rockstar has joined #openstack-infra15:38
fungianteaya: but also, these branches are supported for the benefit of distributions and deployers who use them, so the hope is that they step forward with sufficient prople to keep it working. if they don't, then i'm not entirely sure why the project as a whole puts forth the effort to coordinate it15:38
fungis/prople/people/15:38
anteayahmmmm15:39
fungiand to that end, we've scaled back the supported duration and release count multiple times15:39
* anteaya feels a summit session brewing15:39
apevecfungi, yes that was the idea, to share the pain of keep old branch working15:39
*** atiwari has joined #openstack-infra15:40
*** AlexF has joined #openstack-infra15:40
fungianteaya: there is a long-standing session at every summit which might as well be entitled "we're reducing stable support even more this cycle"15:40
dhellmannfungi: heh15:40
anteayaah I have missed prior sessions, obviously15:40
*** dripton__ is now known as dripton_shovelin15:41
* anteaya just wishes everything would just work in the gate15:41
fungianteaya: they're some of my favorite sessions, just remember to bring your anti-depressants15:41
anteayano no no15:41
mtreinishanteaya: I've done that before it kind of defeats the purpose15:41
anteayabrewer's yeast and soy lecithin15:41
*** hashar has quit IRC15:42
* fungi drinks plenty of brewer's yeast15:42
*** vkozhukalov has quit IRC15:42
anteayayou don't have bad effects like from anti-depressents15:42
anteayafungi: I keep trying to convince myself it tastes like chocolate milk15:42
*** pcrews has joined #openstack-infra15:42
anteayaso far I have been unsuccessful15:42
*** pblaho has quit IRC15:43
anteayamtreinish: rather, yes15:43
anteayabut I can dream15:43
*** hashar has joined #openstack-infra15:43
sdagueI think a piece of this whole problem is that if you surveyed core reviewers across the projects, I suspect < 50% have ever run tempest15:43
anteayasdague: I would agree with you15:43
anteayaa running tempest focused effort?15:44
anteayaan educational push?15:44
sdagueor figuring out what needs to be done to make that easier15:44
mtreinishsdague: to be fair I rarely run it15:44
anteayasdague: interesting15:44
*** hdd has joined #openstack-infra15:44
mtreinishsdague: that was going to be summit session I was going to submit, that combined with tempest doc gaps15:45
fungisdague: i would say, from what i've seen from my vantage point on the vmt, way fewer than 50% actually bother to run their patches through tempest locally on embargoed fixes too15:45
sdaguemtreinish: so lets start there, what sort of setup would make it simpler for you to run it locally? Would a good vagrant guest be helpful15:45
mtreinishsdague: no it's just a matter of me being lazy or killing python deps on a node15:45
mtreinishI've got a cloud snapshot that I can spin up and rerun devstack and get there fairly quickly15:46
mtreinishit's just a matter of doing that vs pushing and waiting for the check queue to come back15:46
sdaguemtreinish: right, so if it was easier to pull a good known revision, sort of conceptually a tox for tempest, would that help?15:46
fungii can still try to get my devstack-gate manual control scripts into a publishable condition if it will help15:46
fungibut the barrier to entry there is having a cloud provider15:47
fungiaccess to one, at any rate15:47
*** openstackgerrit has joined #openstack-infra15:47
mtreinishsdague: like having one command that spins up a node and has everything setup?15:47
anteayathat is becoming less of a barrier all the time15:47
sdaguefungi: so, I actually bet we could adapt them to also work on local vagrant very easily15:47
jeblairor a workstation that can run a vm.15:47
*** AlexF has quit IRC15:47
mtreinishthat's basically what I have now15:47
fungijeblair: sdague: true enough15:47
jeblairopenstackgerrit: welcome back15:47
fungii just meant in the current shape i have them in, they assume novaclient15:47
jeblairi think gerritbot was netsplit15:48
fungiyeah, i was looking at its logs15:48
sdaguefungi: sure, if you want to stackforge all that, I can play with adapting it to other envs as well15:48
sdaguebecause I think a coherent launch script to do this for local testing would bring more folks into the fold15:48
*** AlexF has joined #openstack-infra15:48
sdagueand I think that would be a huge benefit to the project as a whole15:49
openstackgerritDiane Fleming proposed a change to openstack-infra/config: OpenStack Image Service API v2 Reference link broken  https://review.openstack.org/7333115:49
*** jasond` has joined #openstack-infra15:49
jeblairfungi, SergeyLukjanov: would you like to review https://review.openstack.org/#/c/68817/ before i self-approve?15:49
fungisdague: sure thing. it's been on my to do list, just not high enough to get any traction. also it's broken again with recent devstack-gate changes so i need to track it down. we don't really have an api for d-g so anything like this which tries to call bits of it has a tendency to break15:50
jasond`can anybody tell me why this isn't merging? https://review.openstack.org/#/c/71082/15:50
SergeyLukjanovjeblair, morning, looking on it15:50
sdaguebasically I have a lot of lesser hacks, that are hard to motivate on making generic, but if I had someone to work with, like yourself, I think I could stay more motivated15:50
jasond`i don't see it in the zuul status page15:50
fungijeblair: ahh, yes, i had looked at that one before, but not since it was un-wip'd and rebased15:50
fungilgtm15:50
*** mrmartin has quit IRC15:51
*** atiwari has quit IRC15:51
anteayajasond`: the gate jobs failed then you have been running recheck which sends it to the check queue15:51
anteayato get it back into the gate queue you have to run reverify bug <bug #> or reapprove15:51
jeblairfungi, SergeyLukjanov: thanks, it's the kind of change i want to merge early in my day15:52
jasond`anteaya: i see.  at this point, what should i run to get it back to the gate queue?15:52
jasond`reverify no bug?15:52
anteayano that command has been removed15:52
fungijasond`: right, it's not currently failing, so just get shardy to remove and readd his approval, or get another core reviewer on that project to add an approve vote15:52
anteayayou are core? you can reapprove15:52
anteayaor15:52
anteaya`reverify bug <bug #>`15:52
jasond`anteaya: i am. i'll try that15:52
jasond`thanks for the help15:52
anteayak15:52
anteayanp15:52
anteayagood luck15:53
fungijeblair: makes sense. i'd like to go ahead and do the bare-centos6 one now too if possible, since there are several of us around to quickly revert it if needed15:53
*** gothicmi1dfood is now known as gothicmindfood15:53
openstackgerritAlexander Jones proposed a change to openstack-infra/git-review: Fix parsing of SCP-style URLs, as these are valid in Git itself  https://review.openstack.org/7275115:53
anteayajasond`: this is your patch, so this is the only case where your reapproval is okay, just to send it back to the gate15:53
anteayanormally self approval is verboten15:54
jeblairfungi: yep15:54
jasond`anteaya: i made a note to that effect.  thanks15:54
anteayanp15:54
*** DeeJay1 has quit IRC15:54
*** IvanBerezovskiy has left #openstack-infra15:54
fungiSergeyLukjanov: did you want to take a look at https://review.openstack.org/73107 real quick?15:54
*** dcramer_ has quit IRC15:55
SergeyLukjanovfungi, looking15:55
SergeyLukjanovfungi, I'm ok with it15:56
jeblairfungi: we have iad and ord ready now?15:56
SergeyLukjanovfungi, just find open tab with non-submitted +2 :)15:56
fungijeblair: yeah, we had nodes in both before i went to sleep last nighth15:56
jeblaircool.  both those are approved now15:57
funginight15:57
jeblairi'm also merging this potentially breaking change: https://review.openstack.org/#/c/68998/15:58
openstackgerritA change was merged to openstack-infra/config: Require a recent (48h) positive check result  https://review.openstack.org/6881715:58
fungijeblair: for completing the bare-precise transition, i'm thinking we may want to refactor the way we're applying the single-use parameter function, yeah? since only a handful of jobs will now go to long-running trusted nodes, single-use becomes the common case. or should we just do a negative pattern match on the proposal/upload jobs and be done with it?15:59
mtaylorjeblair: I'm behind - are you not passing in the url anymore because you're pushing the merges to git.o.o ?15:59
fungi68998 lgtm too, and i'll keep an eye out for all jobs failing to get any commits16:00
openstackgerritMate Lakat proposed a change to openstack-infra/config: XenServer installer script  https://review.openstack.org/6058116:00
fungimtaylor: zuul passing the git url in a parameter16:00
jeblairmtaylor: no, it's part of a series; if you walk up to the first change, you'll see the rationale: https://review.openstack.org/#/c/68997/116:00
openstackgerritA change was merged to openstack-infra/config: Switch to bare-centos6 single-use nodes  https://review.openstack.org/7310716:00
jeblairmtaylor: it's to support the zuul merger split16:00
mtaylorjeblair: ok. cool. I'll go read16:01
*** rcleere has joined #openstack-infra16:01
jeblairfungi: i think we should reverse the pattern at some point16:01
mtaylorjeblair: got it. thanks16:02
*** andreaf has quit IRC16:02
fungijeblair: would you rather two-step that? first do the negative regex, then add a new parameter function (or two) and swap it around in a subsequent change?16:03
*** mgagne1 is now known as mgagne16:03
openstackgerritA change was merged to openstack-infra/config: Stop supplying the zuul url to ggp  https://review.openstack.org/6899816:04
jeblairfungi: i think it can be in one change.16:04
ttxapevec: http://paste.openstack.org/show/65076/16:04
ttxapevec: hand-made a bit16:05
ttx(my script did not add the stable-maint core ** so I adedd them manually)16:05
ttxremoving anyone** under 9 sounds fair, as would adding anyone above 3016:06
*** mtaylor is now known as mordred16:07
*** mordred has quit IRC16:07
*** mordred has joined #openstack-infra16:07
ttxalthough we could review the quality of their reviews and their knowledge of the stable policy first16:07
*** saju_m has joined #openstack-infra16:08
apevecyeah, also this should be an offer, not forced :)16:08
mtreinishttx: where are those numbers pulled from because I -2d like 7 tempest backports last week.16:09
*** virmitio has joined #openstack-infra16:09
ttxmtreinish: I quickly hacked a script because reviewstats doesn't support stable/* -- might be inaccurate. Only stable/havana considered16:11
*** AlexF has quit IRC16:11
*** mrfatstrat has joined #openstack-infra16:11
*** AlexF has joined #openstack-infra16:11
anteayathere are 20 people listed with 0 reviews16:12
openstackgerritJames E. Blair proposed a change to openstack-infra/config: OpenstackID Documentation  https://review.openstack.org/6962016:12
mtreinishttx: ok16:12
ttxanteaya: yeah. Weird huh. I should probably take the time to properly hack that script.16:12
mtreinishttx: just for reference: https://review.openstack.org/#/q/status:abandoned+project:openstack/tempest+branch:stable/havana+topic:bug/1252620,n,z16:13
anteayaor we can just admire the anomolies16:13
anteayaseems faster16:13
mtreinishit might be because he abandonded them16:13
ttxmtreinish: yes.. only status:merged is taken into account16:13
ttxbecause of some strange query limitation iirc16:13
anteayaah so -2'ing be definition is not counted as a review16:14
ttxlet me see if running without status:merged looks better16:14
*** dizquierdo has quit IRC16:14
*** jpeeler1 has quit IRC16:14
ttxrunning...16:15
*** chandankumar__ has joined #openstack-infra16:16
*** dripton_shovelin is now known as dripton16:17
ttxapevec, mtreinish: http://paste.openstack.org/show/65084/16:17
*** garyk has quit IRC16:18
*** VijayT has joined #openstack-infra16:18
*** chandankumar_ has quit IRC16:19
openstackgerritDoug Hellmann proposed a change to openstack/requirements: Add requirements for taskflow  https://review.openstack.org/7334216:19
openstackgerritAndreas Rehn proposed a change to openstack-infra/jenkins-job-builder: Added support for Delivery Pipeline Plugin  https://review.openstack.org/7165816:20
anteayaapevec: who are tamazur and jsbryant?16:21
mtreinishttx: well now I think you might be overcounting a bit. :) I only counted 10 -2's from me.16:21
apevecthey're now stable-maint members, just people doing reviews on stable patches16:22
apevecs/now/not/16:22
ttxmtreinish: right, I think that's why I ended up adding status:merged to the query16:22
ttxbecause otherwise ther e was some doublecounts16:22
anteayabecause if they have reliable knowledge of stable policy and good quality reivews, I would support their consideration for membership16:22
anteayasame with zhaoqin16:23
apevecanteaya, yes, as ttx mentioned: we need to review their reviews then ask them16:23
*** mrfatstrat has quit IRC16:23
anteayaapevec: are you in a position to review their reviews?16:23
anteayaor do you need some additional people for that?16:23
anteayaplus whichever list you go by, it appears some folks are not interested in maintaining membership16:24
apevecyeah, ttx/me/adam_g should go over the list in the next few days16:24
openstackgerritJeremy Stanley proposed a change to openstack-infra/config: Modernize ATC list format  https://review.openstack.org/7334716:25
openstackgerritJeremy Stanley proposed a change to openstack-infra/config: Retrieve ATC project list from governance repo  https://review.openstack.org/7334816:25
fungittx: ^16:25
fungijeblair: ^16:25
ttxapevec: i should probably fix the stats generation properly then16:25
jeblairfungi: neato!16:25
fungithat's been rotting in a topic branch on my machine for too long16:26
jeblairbtw, it's halfway through storyboard meeting time.  i should get better at remembering to announce that here16:26
fungigah16:26
anteayaapevec: that sounds great16:26
*** AlexF has quit IRC16:27
*** mfer has joined #openstack-infra16:28
*** AlexF has joined #openstack-infra16:29
*** david-lyle has joined #openstack-infra16:31
*** pafuent has joined #openstack-infra16:31
fungijenkins02 and 04 have some bare-centos6 nodes running jobs now16:33
*** AlexF has quit IRC16:34
*** pafuent has left #openstack-infra16:34
*** pafuent has joined #openstack-infra16:35
fungithis one succeeded... https://jenkins02.openstack.org/job/gate-cinder-python26/6160/console16:35
jeblairyaay!16:35
fungias did https://jenkins02.openstack.org/job/gate-ceilometer-python26/4105/console16:35
fungiso two different jobs16:35
fungithis one looks sort of bound up though... https://jenkins02.openstack.org/job/gate-ceilometer-python26/4103/console16:36
*** gyee has joined #openstack-infra16:37
*** afazekas has quit IRC16:37
jeblaira bunch of heat and horizon changes in the gate just turned into a christmas tree, but the failures don't look like they could be related to any of our changes16:38
*** chandankumar__ has quit IRC16:38
*** AlexF has joined #openstack-infra16:38
sdaguejeblair: all failures with mock. Did we take a new version?16:40
sdagueor fail to have one in the mirror at the right level16:40
*** chandan_kumar has joined #openstack-infra16:41
fungithe bare-centos6 fail i saw looks like zuul retried it on a different node a minute later... not entirely certain why16:41
fungicheck pipeline, so not a gate reset16:42
fungicompare https://jenkins02.openstack.org/job/gate-ceilometer-python26/4103/ and https://jenkins02.openstack.org/job/gate-ceilometer-python26/4105/16:42
*** AlexF has quit IRC16:43
fungisame change and patchset number16:44
*** AlexF has joined #openstack-infra16:44
*** andreaf has joined #openstack-infra16:46
jeblairsdague: the failing job and last successful job both had mock 1.0.116:46
sdaguejeblair: odd16:46
*** dcramer_ has joined #openstack-infra16:47
sdaguejeblair: so is your sufficiency check through now?16:47
sdaguebecause, honestly, once that's through, I want to purge all the non devstack jobs on the gate side, as then unit test fails like this will only affect the project in question, and not everyone else16:48
anteayasdague: https://review.openstack.org/#/c/68817/16:48
jeblairsdague: i believe it is based on the zuul reload time16:48
sdaguejeblair: ok, so then step 1 job trimming, remove all the unit test, pep8, and docs jobs from the gate?16:49
*** mrodden has quit IRC16:49
sdagueI want a head nod before I work up the patch :)16:49
sdagueso I'm not wasting my time16:49
sdagueI'm also testing that it all works as expected with an older devstack change16:51
jamespageapevec, apologies - this is taking a bit longer than expected16:52
jog0I'm getting a 503 from from elasticSearch16:52
jamespageapevec, the havana/iscsi topology tested OK16:52
apevecjamespage, np, just let me know when you have results16:52
sdague503 is typically either over capacity (cold cache) or bad params16:52
jamespageapevec, unfortunately the lab ran out of IP's during the havana/ceph test16:52
jamespageapevec, re-running that now16:52
sdaguethe proxy isn't great at distingusihing them16:52
apevecjamespage, cool, is there maybe public url with results?16:53
jog0sdague:  I am concerned about dropping pep8 and unit tests from gate16:53
jamespageapevec, apparently not (I was surprised)16:54
jog0http://status.openstack.org/elastic-recheck/data/uncategorized.html shows we have a good number of valid bugs that are cought on those jobs in gate http://status.openstack.org/elastic-recheck/data/uncategorized.html16:54
jamespageapevec, not going to fix that now - will do for next16:54
*** geekinutah has joined #openstack-infra16:54
apevecok16:54
apevecjamespage, re. ceph - there's one patch which is Ceph related but came too late: https://review.openstack.org/7257516:55
apevecso I'll wait for you ceph results before tagging nova16:55
*** DinaBelova is now known as DinaBelova_16:55
jog0sdague: I am afraid that if we drop pep8 and unit from gate we will constantly be fixing them in check queue16:55
jamespageapevec, the ceph scenarios does not do boot from rbd via nova16:56
jamespagebut lets check it out anyway16:56
sdaguejog0: then they are fixable on the check queue in the project in question16:56
sdaguebecause right now, it's the same issue, except it kills changes from unrelated projects16:57
jog0sdague: then each project will be blockeda nd playing fix up16:57
sdaguejog0: instead of each project blocking all the other projects?16:57
jog0sdague: what about this:16:57
sdaguejog0: they have to have clean check score to make it into the gate16:57
fungi(as of a few minutes ago)16:58
jog0if the gate has TOP -> nova -> cinder.16:58
*** e0ne has quit IRC16:58
jog0and nova unit tests fail, we don't invalidate cinder16:58
jog0but just all nova patches16:58
sdaguejog0: ok, so go and rewrite zuul to do that in the next week16:59
jog0so if nova-1 -> cinder-1 -> nova-2. nova-2 gets invalidated16:59
sdagueright now the dependency setup is one dimensional16:59
jog0sdague: I am afraid your proposal will make each project play catch up all the time16:59
*** garyk has joined #openstack-infra17:00
sdaguejog0: well the alternative, given the failure rate of unit tests, is a 60 hr merge queue again on icehouse-317:00
*** DinaBelova_ is now known as DinaBelova17:00
sdaguethis was all outlined in that change proposal17:00
*** NikitaKonovalov is now known as NikitaKonovalov_17:00
jog0if we get there we can take drastic action  but I don't think we don't need to yet17:00
sdaguejog0: we really do17:00
jog0sdague: right now a failed nova unit test doesn't 'block' a cinder patch it just slows it down17:01
*** mrodden has joined #openstack-infra17:01
*** gokrokve has quit IRC17:01
sdaguejog0: so then people will fix the unit tests on the check side17:01
jog0sdague: with the gate queue flushing out every night, why do you think we need this now?17:01
sdaguejog0: because we're at really low inbound17:01
*** gokrokve has joined #openstack-infra17:02
*** AlexF has quit IRC17:02
jog0sdague: that violates the major idea behind the gate, keep trunk working so we don't block eachother17:02
sdagueand doing changes like this when everything is backed up 60 hrs and thrashing, is the wrong time to do it17:02
sdaguejog0: you can't get in without working check17:02
*** afazekas has joined #openstack-infra17:02
mordredwhat if we remove bad tests from the unittest set?17:02
sdagueand you are cogating on all the tempest17:02
anteayajog0: https://review.openstack.org/#/c/68817/17:02
mordredif there are flaky unit tests - that means they are bad tests17:02
sdaguemordred: or bad code17:03
mordredno17:03
anteayado you mean remove or fix?17:03
anteayaI'm all for fixing, I'm against removing17:03
mordredif they are hitting race conditions, then they are functional tests17:03
jog0sdague: even with the run check >2 days before gate we are seeing the gate catch valid pep8 and py27 issues17:03
mordredand they shoudl not be in the unittests17:03
sdaguemordred: ok, fine semantics17:03
sdaguejog0: the gate should never be catching valid pep8 / py27 errors17:03
sdagueever17:03
jog0sdague: but it is17:03
jog0I agree with that statement17:03
mordredsdague: so, I'm just saying, if we have tests in the unittests that are not 1005 consistent, remove them17:03
jog0but its invalid today17:04
sdaguebecause people are being lazy and not fixing it in check17:04
jog0mordred: this is things that fail 100% of the time17:04
anteayamordred: remove, as in delete, or move as in put them where they belong?17:04
jog0mordred: see status.openstack.org/elastic-recheck/data/uncategorized.html17:04
sdaguemordred: so glance, swift, nova, heat, neutron all have races in their unit tests17:04
jog0skim the pep8 issues for trivial examples17:04
*** mfer has quit IRC17:04
sdaguehave for 6 months17:04
mordredsdague: great. delete the unittests17:05
mordredI mean, delete the racey tests. full stop. if someone wants to move the thing it's testing to tempest, that's awesome17:05
fungijog0: as of today, it's impossible for changes to get approved into the gate when they're not getting a +1 from check jobs first. and shortly we're looking at upping that from 48 hours to 24 hours to narrow the eye of the needle a bit more17:05
mordredbut they are, by all measures, bad tests adn in the wrong place17:05
*** gokrokve has quit IRC17:06
sdaguemordred: that's fine, that's not addressable by icehouse17:06
*** andre__ has quit IRC17:06
*** AlexF has joined #openstack-infra17:06
jog0fungi: but we are still seeing things being catched in gate today17:06
jog0see pepe817:06
anteayafungi: from jenkins check jobs, or any check jobs?17:07
mordredgotta step away for a sec17:07
sdaguebasically it's a ton of grunt work that no one is willing to do. I think having projects wedge themselves on check, in a way that's completely fixable by the projects independently, is the right call17:07
sdaguebecause that will put the incentive in the right place17:07
anteayasdague: I agree17:07
fungijog0: and rerunning those in the check queue within the same 24-hour period would have succeeded?17:07
anteayaif they won't fix their tests, keep them in check17:07
jeblairsqlalchemy migration numbers are a concise example of the kind of thing that the gate is designed to prevent landing17:08
sdaguebecause the answer to failed unit test in gate is just reverify, I've never seen anyone actually work to fix a race there17:08
anteayafungi: would a +1 verified from a third party testing system enough to get a patch into the gate, even if jenkins hasn't come back with results yet?17:08
jeblairit's a perfectly legitimate problem, and i know we ran into it all the time17:08
sdaguejeblair: and devstack / tempest wouldn't catch that?17:08
jog0how about this:17:08
jeblairsdague: i'm citing it as an example of the process17:08
fungianteaya: no, the filter is on the specific gerrit account17:08
sdaguejeblair: sure17:08
anteayafungi: yay!17:08
jog0wait a 2 weeks since https://review.openstack.org/#/c/68817/ was landed17:09
jog0and see what http://status.openstack.org/elastic-recheck/data/uncategorized.html looks like17:09
*** HenryG has joined #openstack-infra17:09
anteayajog0: that puts us into feature freeze ramp up17:09
anteayajust for timing17:09
jog0then we will have real data on this17:09
anteayaI agree we need to access the effects of 6881717:09
jog0anteaya: if we make a bad choice now things will get worse IMHO17:10
anteayaI agree17:10
sdaguejog0: why not make the change now, see how bad it is in 2 weeks instead?17:10
anteayajust letting you know what will be happening in two weeks from now17:10
*** markmc has quit IRC17:10
jeblairsdague: to be honest, this is the part of the process i'm most hesitant about.  i do think that lifeless made some good points.  but i also said that i was willing to try it and i'll stick to that.17:10
sdagueyou make this assumption that the alternative is going to be terrible, but we don't actually know17:10
*** andre__ has joined #openstack-infra17:10
jog0sdague: because if you are wrong you move the burdon off of a computer onto people17:10
*** markmcclain has joined #openstack-infra17:11
sdaguejog0: no... we move it from impacting unrelated projects, to only impacting single projects17:11
jeblairsdague: (if you change your mind, i will be somewhat relieved)17:11
sdagueso here's where I stand17:11
jog0sdague: if we let in a blocking patch (something that breaks pep8 or unit tests 100% of time) then we have a blocked repo17:11
sdagueduring the month of january, I spent about 3 weeks straight, including weekends, trying to unwedge the gate17:11
jog0vs a slighly slower gate17:11
jeblairsdague: my personal preference would be, if it's possible, to see what impact the freshness requirement alone has first, and preferably at a 24h interval, before we remove the unit test jobs17:12
sdaguethat meant manually pulling changes out of the gate (hundreds) that were failing unit tests17:12
jog0jeblair: ++17:12
sdagueand manually getting 40ish changes promoted17:12
sdagueand I think the biggest issue is that stuff would fail the gate that had no reason being there, like pep8 changes failing on a git name resolve issue17:13
sdagueor unit tests failing 8% of the time in swift17:13
jog0sdague: so the freshness logic will catch all that17:13
sdagueit won't catch the races17:13
jog0no, but we can do a better job of classififying and dealing with those17:13
jog0(disable them, yell at people etc.)17:13
ttxapevec: how is the release going ? Need me around ?17:13
sdagueand I brought them up numerous times with people, and the general reaction was *schrug* don't know17:14
anteayajog0: can you define a better job?17:14
sdagueby the core team members17:14
anteayaI think sdague is saying he doesn't want to do what he did in January again in March17:14
sdagueand people typing reverify 12345678917:14
jog0anteaya: yes, a better job at tracking gate failruse17:14
anteayajog0: how?17:14
jog0anteaya: aka stairing at http://status.openstack.org/elastic-recheck/data/uncategorized.html17:14
sdagueanteaya: I think what I'm saying is I *will not* do it again, period17:14
anteayasdague: okay, and I agree with you17:15
anteayathe same flavour of frustration drove me into Neutron17:15
sdagueso my intent is to provide systematic changes to push the pain point back to the right place17:15
sdaguebecause the issue right now, is the pain point is in the wrong place17:15
anteayaand now I will snipe a Neutron patch from the gate without mercy17:15
sdagueso it's not getting fixed17:15
anteayaso I feel better coming into this ff17:15
apevecttx, started tagging, will wait for CI result from jamespage before pushing17:15
apevecttx, I should be all set17:15
jog0sdague: btw docs failed once in gate with an unknown error17:16
anteayasdague: I agree, if the pain point is in the correct place, action either happens or it doesn't, but the rest of the folks carry on17:16
sdagueand to try this experiment before the queue is already jammed17:16
sdaguebecause we can always revert17:16
anteayasdague: how long would you like to try it?17:16
jog0sdague: so this is a two part experiment17:16
anteayawhat length of time to experiment?17:16
jog01) the freshness aspect. 2) removing things from gate17:17
notmynamesdague: I'm in a meeting now, but I want to follow up with you a little later on this17:17
jog0more info on 1) gives us more confidence on the impact on 217:17
anteayaI'm in agreement with you17:17
jog0sdague: if you are wrong on 2, you cause a lot more pain for a lot more people that we can avoid.17:17
anteayahowever given our timing to ff, we need to come to agreement soon17:17
anteayaas in this week-ish17:18
sdaguejog0: no, it causes different pain17:18
anteayaso we are not testing things during traffic17:18
sdagueit's not actually more pain17:18
jog0anteaya: we should have decent Preliminary  results ian week17:18
sdagueand it's pain back at the point of fixing it17:18
anteayajog0: how much time does that give us to evaluate the second part?17:18
jog0sdague: the pain is we wedge pep8 on repo x and everything stops17:18
sdaguejog0: yep17:19
anteayaand enough time to revert before we are overrun?17:19
sdagueand you know what, nova core will fix that pep8 issue really fast17:19
jog0anteaya: well two weeks of data gets us clean ES data, but a week is decent17:19
jog0sdague: if we are all asleep? if its a unit test issue?17:19
jog0its a big distraction that we would rather avoid17:19
pafuentHi. I'm trying to get working a devstack-dsvm gate for Climate, and after some tries I get it working, except that I can't see any screen logs uploaded in the gate results. What is the meaning of this? The screens failed for some reason or I need to do something special in order to get those logs uploaded?17:19
anteayajog0: but that gives us 0 time to test sdague's proposal before the traffic hits17:19
*** mfer has joined #openstack-infra17:19
jog0anteaya: so I see two possible impacts of removing pep8/unit tests fro mgate17:20
sdaguejog0: so instead it's preferable to crater heat landing changes?17:20
sdagueand make heat core devs have to babysit gate queue for days17:20
jog01) we wedge more repos more often. 2) make gate faster by moving the burdon of racey unit tests17:21
sdaguebecause basically what I'm hearing is - we should put more burden on all the other 9 integrated projects instead of fixing our unit tests17:21
*** atiwari has joined #openstack-infra17:21
jog0without implimenting removing thigns from gate we will get a clear idea of 1) and 2 I am fairly confident about17:21
anteayajog0: yes, those are the impacts17:21
jog0sdague: the burdon is only a slower gate17:21
anteayajog0: you have yet to give a time frame for evaluating 1 that leaves enough time to evaluate 217:22
sdaguejog0: that's a real burden17:22
zaromorning17:22
sdaguewhen you need to do things like land dependent changes across projects17:22
jog0sdague: which we can live with for a bi17:22
sdaguejog0: we really can't17:22
jog0and the gate is in the best shape it has been in  a long long time17:22
jog0we landed 435 patches two weeks ago17:22
*** AlexF has quit IRC17:22
sdaguejog0: because we're at a lull17:22
sdaguethat's super low17:22
jog0sdague: that is a record for us17:23
anteayajog0: yes take a look at traffic17:23
anteayavolume changes everything17:23
jog0and the gate flushes out over the weekend ... and every night17:23
anteayaright now yes17:23
zarofungi: i'm can't seem to push new PS in gerrit due to https://review.openstack.org/#/c/7312617:23
sdaguejog0: we're at a lull17:23
jog0anteaya: yes, if that changes we can re-evalute the timeline17:23
zarofungi: can you help?17:23
anteayathat is not fair17:23
jog0sdague: I don't think landing 141 patches this Tuesday is a lul17:23
anteayathe only change will be more traffic, which is coming, which elimnates the ability test anything17:24
sdaguewell, you have my point of view.17:24
*** chandan_kumar has quit IRC17:24
jog0sdague: at the very least I think we should wait a week17:24
sdagueI'm basically at the point where I don't think we'll make it through icehouse 317:24
anteayaI still think it is possible to set a time frame to test 1 which allows enough time to test 2 within the next week17:24
jog0I think we have a better solution which is much  more invasive but safer (zuul changes)17:24
jeblairjog0: what solution is that?17:25
sdagueand honestly, if that crap hits the fan, I have no interest in helping to unwind this during i317:25
sdaguebecause, I completely burnt out on diagnosing and hotfixing during i217:25
anteayasdague: I hear you17:25
jog0jeblair: given TOP_OF_GATE -> nova-1 -> cinder-1 -> nova-217:25
jeblairsdague: i will +2 the change to remove unit tests, etc from the gate but i think we should set the interval to 24h first (i think that was the plan)17:26
sdaguejeblair: yep, I'm fine with that17:26
jog0jeblair: if pep8 or unit tests fail in nova 1, don't invalidate cinder-1, only nova-217:26
sdagueI think 24hr freshness is good17:26
jeblairjog0: that won't work; the nova change still affects the system that is being tested with the cinder change17:26
sdaguejeblair: can you seperate the auto freshness from the required freshness?17:26
*** AlexF has joined #openstack-infra17:26
jog0jeblair: oh,  how so?17:26
sdaguebecause I think we can actually live with required freshness at 3 or 4 days17:26
sdaguesorry, auto freshness at 3 or 4 days17:27
sdagueand required at 24h17:27
jeblairsdague: i don't think so; we're counting on side effects of their being the same17:27
*** yassine has quit IRC17:27
jeblairsdague: particularly, the fact that if you approve something and it doesn't meet required freshness, it auto-triggers auto-freshness17:27
sdaguejeblair: ok, that just means more check queue traffic, but we can probably deal with that for now17:27
sdagueok17:27
sdaguejeblair: where do we change the 24hr bump?17:27
jeblairsdague: i mean, if we wanted to give that up, we could, but basically we'll spend all day telling people "that wasn't enqueued because the check result is too old, type 'recheck no bug'"17:28
sdaguejeblair: no, I think this is good17:28
fungizaro: why is 73126 preventing you from pushing patches into gerrit?17:28
jeblairsdague: just s/48/24/ in zayout.yaml17:28
sdagueI think there could be an optimization in the future17:28
jog0jeblair: what is the issue  with my idea above ^17:29
sdaguejeblair: oh... could we trigger off of different votes?17:29
zarofungi: i'm trying to push new PS for https://review.openstack.org/#/c/65133  but when i gitreview it says no .gitreview file.17:29
sdagueso +2 / +A triggered at a different interval than -1,0,+117:29
jeblairjog0: you might land a change to cinder that only works if the change in nova is present17:30
sdagueanyway, future optimization17:30
zarofungi: when i add .gitreview it won't work either because it's not untracke.17:30
jog0jeblair: that isn't true for unit tests17:30
jog0err pep817:30
jog0not sure how that is possible for unit tests17:30
zarofungi: *it's untracked17:30
jog0unless you are ceilometer and pull in nova repo during unit tests17:31
jeblairjog0: this would be a failed devstack test17:31
sdaguejog0: but you aren't kicking out tests, you are kicking out changes17:31
sdagueso you need to reset the whole queue17:31
jeblairjog0: except it's not failing because it's still being tested with the nova change because your proposal was that we keep the cinder tests running17:31
fungizaro: why would an untracked .gitreview file in your repo be a problem?17:31
jog0jeblair: ohh I see now17:31
sdaguejog0: believe me, a ton of brain power has been spent on the possible interactions17:32
clarkbmorning17:32
jog0sdague: the 'just trust me' answer is never a good one17:32
jog0clarkb: I can't hit http://logstash.openstack.org/17:33
fungizaro: we should land the change to update the gerrit unit test job, then the change which fixes the submodules, then with working tests for that branch we can land the .gitreview file addition17:33
jog0clarkb: and good morning17:33
fungizaro: if we do it right, those should be able to merge properly in that sequence17:33
clarkbgearman logstash queue is better than yesterday (under 90k) but we did lose 3 indexers (last night we lost all 15 on old code so an improvement) going to scale back to two workers per indexer on new indexers to see if that helps more17:33
clarkbjog0 yup I turned off kibana17:33
jog0clarkb: oh?17:34
clarkbwill correct when everything is caught up17:34
openstackgerritSean Dague proposed a change to openstack-infra/config: narrow the freshness window to 24h  https://review.openstack.org/7336617:34
jog0clarkb: ack17:34
jog0jeblair sdague: what about  having a pre-gate pipeline that runs pep8 and unit tests only17:34
zarofungi: let me check again.  maybe typed the wrong git cmd17:34
jog0right before gate17:35
sdaguejog0: ok, don't trust me. I'll stop looking at any gate things, because apparently the aggregate knowledge gained there isn't useful in convincing you of things :)17:35
jog0sdague: all I am saying its good to share knoweldge and reasoning17:35
openstackgerritKhai Do proposed a change to openstack-infra/gerrit: Add a Change Owner group  https://review.openstack.org/6513317:35
zarofungi: sorry about that.  i forgot to specify the branch when doing the git review17:36
fungizaro: also, note my comment on that change. you can solve that by adding defaultbranch to the .gitreview file17:37
jog0clarkb: any ETA when things will be back (no rush, just wondering so I can plan accordingly)17:37
clarkbI need to run a few errands this morning and will finish es/logstash cleanup when I return17:37
jog0clarkb: thanks17:37
clarkbjog0 ~3 hours? depends on how much new stuff rolls in this morning17:37
jog0clarkb: awesome17:38
*** hashar has quit IRC17:38
openstackgerritKhai Do proposed a change to openstack-infra/gerrit: Fix Schema_86 to use correct UUID for Change Owner group  https://review.openstack.org/6513417:39
*** NikitaKonovalov_ is now known as NikitaKonovalov17:39
jog0sdague: so I unit tests can have race conditions. but docs and pep8?17:40
jog0I don't know of any pep8 races17:41
jog0and docs appear to never fail due to docs issues17:41
*** bhuvan has joined #openstack-infra17:41
jog0(but that is neither here nor there)17:42
openstackgerritKhai Do proposed a change to openstack-infra/gerrit: Add a Change Owner group  https://review.openstack.org/6513317:42
openstackgerritKhai Do proposed a change to openstack-infra/gerrit: Fix Schema_86 to use correct UUID for Change Owner group  https://review.openstack.org/6513417:42
*** hashar has joined #openstack-infra17:43
*** vkozhukalov has joined #openstack-infra17:43
fungijog0: i would count doc tooling issues as ways those jobs can break17:44
*** jpich has quit IRC17:44
zarofungi: thanks.  i have added defaultbranch.17:44
jog0fungi: sure, so what about pep8?17:45
jog0if there is a pep8 issue that I don't know about?17:45
jog0(gate related that is)17:45
ewindischjog0: btw, in regard to rechecks, the 3rd-party requirements page is actually kind of broken.17:45
fungithere are plenty of ways to break pep8... are you asking about nondeterministic results from pep8 or just general failures in pep8 jobs?17:46
jog0ewindisch: oh?17:46
jog0fungi: non-deterministic17:46
jog0ewindisch: why aren't you skiing?17:46
ewindischjog0: at the top it says you must support rechecks, but then suggests using the jenkins gerrit-trigger (and a configuration for it), but it doesn't support rechecks.17:46
ewindischjog0: because I have to work. I'm going to hit the slopes after lunch17:46
jog0ewindisch: ahh yes that is an issue17:46
jog0ewindisch: ahh enjoy17:47
ewindischjog0: I thought you said you were staying today, too?17:47
jog0jeblair: ^ w/ regard to third_party  tseting docs17:47
fungiewindisch: i agree, we've had mailing list threads about that too. alternatives are to either expect third-parties to clean up/manually inspect all failures reported, or run something more capable than the gerrit trigger plugin17:47
jog0ewindisch: something came up so have go back today17:47
fungii'm not really comfortable even continuing to recommend the gerrit trigger plugin, because even though i've never myself used or even configured it, people constantly ask me to troubleshoot their installations of it17:48
lifelessjeblair: sdague: oh hi17:48
ewindischfungi: I'm using it without problems, other than it doesn't do rechecks.17:48
ewindischfungi: my personal preference would be some leniency in getting it implemented in a timely fashion IRT the hypervisor deprecation plan17:49
lifelessI'm not against trying it; what I'd ask is that we be willing to revert if it is worse - but how do we assess worse?17:49
*** senthil has quit IRC17:49
ewindischfungi: but I did start on a solution for long-term: https://github.com/ewindisch/gherkin-trigger17:49
fungiewindisch: love the name too!17:49
jog0lifeless: w/ respeect to pulling unit tests out of gate?17:49
jeblairewindisch, fungi: yeah, we should probably just recommend zuul.17:50
lifelessjog0: that broad discussion yes17:50
lifelessjog0: it was AIUI more than just unit tests that were going to be pulled17:50
ewindischjeblair: that's another option, although  I didn't really want to go down that road. It's probably an irrational fear of the unknown17:50
jeblairewindisch: why don't you patch the gerrit-trigger-plugin?17:50
ewindischjeblair: java.17:51
jog0lifeless: yes, sdague wants all !devstack jobs to be  pulled17:51
fungizuul is lightweight enough it could run on a jenkins master and talk over loopback17:51
fungidepending on your scaling needs17:51
jeblairewindisch: do you realize you just re-wrote commit 1 of zuul?17:51
jog0lifeless: so we can assess worse by seeing if we wedge unit and pep8 tests for individual repos more17:51
ewindischbut yeah, there are countless options. My main problem is that I have an impossible deadline17:51
jog0lifeless: not sure best way to collect that data though17:52
fungigerrit trigger plugin being unable to parse gerrit comments -> zuul17:52
* jog0 gets ready to board his flight17:53
anteayaewindisch: gherkin is a tool used with cucumber testing framework, if you care17:53
ewindischanteaya: bleh. It's also an alternative pickle library.17:53
jeblairewindisch: zuul can actually be pretty lightweight for just a check queue; we had a lot of problems with that approach that we have solved along the way.17:54
anteayaewindisch: as long as you know17:54
ewindischanteaya: I didn't, actually :(17:54
anteayak17:54
ewindischjeblair: it isn't very clear how to run zuul without gearman17:54
*** hashar has quit IRC17:54
jeblairewindisch: if you are into writing python-based test tools, i'd encourage you to contribute to the ones we're working on17:55
anteayaewindisch: the biggest problem is how to convey this information to people standing up these systems will zero knowledge of openstack development17:55
fungizuul has a gearman server built in. attach any gearman workers you like, including the jenkins-gearman plugin17:55
jeblairewindisch: you can't; you'll need the gearman plugin for jenkins.17:55
ewindischjeblair: my problem is that I have negative time. I'm taking every shortcut I can.17:55
lifelessjog0: so, I want to add tripleo-gate jobs into the gate :)17:55
ewindischjeblair: my interest is in gating the docker code so Russell doesn't rip it out of Nova.17:55
HenryGWhen submitting a "git review", how is the 'topic' determined?17:55
anteayathey also seem to have little knowledge of opensource, how to use mailing lists or how to work on anything other than windows apps17:55
anteayajust some general observations17:56
fungiHenryG: it's guessed from your local branch name, or specified with the -t command-line option17:56
lifelessjog0: that is actually my biggest concern, but there's a path to doing that - we need the check-on-+2-only queue.17:56
ewindischjeblair: I have deadline-insanity for that, so no, things won't get done the best way. Throwaway work will get done.17:56
anteayaHenryG: it is the topic branch you are on in your local git repo17:56
fungiHenryG: and if your local branch name is the same as the remote branch and you don't use -t, then it empties the topic field on the assumption you didn't want one (because a topic named the same as the remote branch is pretty useless usually)17:57
anteayaHenryG: before editing, run git checkout -b <branch name>17:57
anteayafor a new branch17:57
ewindischjeblair: because spending an hour writing my own python script is faster than learning anyone else's tool, even if it's a whole lot better17:57
anteayaor git checkout <branch name> for an existing branch17:57
ewindisch(and yes, that sucks)17:57
jeblairewindisch: okay, well when things calm down, a lot of people are setting up zuul servers for 3rd party testing, so the setup process is going to get a lot better and there will be a lot of support for it.17:57
HenryGanteaya: that's what I did, but my local branch and the topic in gerrit do not seem in any way related.17:58
jeblairewindisch: and we'd love your contributions to that.  :)17:58
ewindischjeblair: cool. and I wish I was using zuul/gearman. I do plan to move to it, once there are cycles.17:58
fungiewindisch: some of the people installing zuul for third-party test integration are also writing howto documentation17:58
anteayaHenryG: did you create the branch before editing and commiting?17:58
sdaguelifeless: I think we want to consider if a smaller set of configurations for integration points work in the gate. Phase 1, just get rid of the non devstack jobs, as those are clearly not cross project.17:58
HenryGanteaya: yes17:58
anteayaHenryG: what url for the patch17:59
fungiHenryG: oh, also, we have a hook which will change your topic to bug/NNNNNNN if you include a bug header line in your commit message17:59
anteayaHenryG: and paste the output of git status17:59
HenryGanteaya: https://review.openstack.org/7335517:59
*** senthil has joined #openstack-infra17:59
ewindischer.. anyway, the whole point was that the 3rd-party testing page is wrong to suggest using gerrit-trigger17:59
ewindisch;-)17:59
lifelesssdague: so the conceptual problem I have is that pre-merge gating is valuable for projects on their own, in it's own right.17:59
anteayafungi: looks like the hook hits bp for as well18:00
ewindisch(unless 3rd-party rechecks aren't really necessary)18:00
lifelesssdague: and this feels like we're taking that away from projects and saying 'you choose - pre-merge gating, or integrated pre-merge gating'18:00
sdaguelifeless: so if the functionality is caught in a tempest jobs, it will be fine.18:00
lifelesssdague: I think we actually need to do more of a pipeline arrangement in zuul18:00
anteayaHenryG: seems the hook has picked up the blueprint for in the commit message18:01
sdaguebecause most of the real issues I've seen crop up were because projects, like heat, weren't actually testing in tempest18:01
openstackgerritAndreas Jaeger proposed a change to openstack-infra/config: OpenStack Image Service API v2 Reference link broken  https://review.openstack.org/7333118:01
lifelesssdague: where we do the isolated project gating - avoiding races within the project, and then it goes into an integrated queue for cross project testing, and then finally lands at the end18:01
anteayaewindisch: the problem is that rechecks are necessary18:01
*** hashar has joined #openstack-infra18:01
*** Ajaeger1 has joined #openstack-infra18:01
lifelesssdague: this doesn't capture cross-project breakage of internal tests - but we make no commitment to catching that today anyway18:01
HenryGanteaya: ah, ok, I have the text '... see the blueprint for more info...' and it picked that up.18:02
jeblairlifeless: that's essentially what sdaque is proposing18:02
sdagueand projects should not be running cross project internal tests18:02
anteayaewindisch: and the knowledge base of the folks implementing is far away from the knowledge required to implement the ideal system18:02
fungianteaya: HenryG: i see that, it apparently had trouble identifying the blueprint name or something18:02
anteayaewindisch: you have correctly identified a gap18:02
sdaguelifeless: so I realize there is an exposure, especially if the merge queue gets long18:02
lifelesssdague: projects that are built on top of the very core layers use the core layer clients in their test code18:02
jeblairlifeless: the requirement for a recent positive check result is very similar to that.18:02
anteayabut getting rid of the requirement is not a winning approach18:02
HenryGfungi, anteaya: I assume I can use -t to change the topic in a follow-up patch?18:02
anteayaewindisch: the ideal approach would be, you must be this tall to ride18:03
*** loquacities has quit IRC18:03
anteayabut we can't do that either18:03
fungiHenryG: i believe so. no guarantees it won't change it back but give that a try on your next patchset18:03
lifelesssdague: e.g. everyone depends on pbr, and we see issues with unit tests there that tempest liekly won't catch18:03
sdaguehowever this is a good approximation of that, and maybe zuul could be extended to cover it18:03
lifelesssdague: and heat depends on neutronclient and novaclient18:03
*** AlexF has quit IRC18:03
lifelesssdague: and its unit tests - rightly or wrongly - involve codepaths within those libraries18:03
sdaguelifeless: pbr doesn't currently gate on everyone's unit tests18:04
jamespageapevec, cinder/iscsi and cinder/ceph scenarios test OK - just waiting for the neutron scenario to complete18:04
*** morganfainberg_Z has joined #openstack-infra18:04
sdagueso that's not an arguement18:04
*** loquacities has joined #openstack-infra18:04
*** morganfainberg_Z is now known as morganfainberg18:04
jeblairsdague, lifeless: i'm not sure how extending zuul to cover it would be different than either sdague's approach on the one hand, or the current system on the other.  the fundamental difference is whether a unit test occupies a position in the gate queue.  one approach says no, the other yes.18:04
*** jgallard has quit IRC18:04
*** morganfainberg has quit IRC18:05
*** morganfainberg has joined #openstack-infra18:05
*** morganfainberg is now known as Guest8657618:05
*** Guest86576 is now known as adskfgjsdkl18:05
HenryGfungi, anteaya: thanks for the insights. No big deal, and happy just to understand things a little better.18:05
sdaguejeblair: I think the item would be another tier of queues, so the nova patches that all enter final gate in an order where unit tests are known to work in that order18:05
*** hashar is now known as hasharMeeting18:05
sdaguebut a reset would kill that anyway18:05
*** adskfgjsdkl is now known as needscoffee18:05
lifelesssdague: I know pbr doesn't - but AIUI there is interest in having it do so18:05
anteayaHenryG: thanks for asking, I didn't know the reason for it either, glad to learn something new myself18:05
sdaguelifeless: well we have real issues today18:05
fungiHenryG: we might ought to consider putting more standardization around blueprint header lines in commit messages and then restricting the actions of that hook to that pattern18:05
sdagueand lets try to address those18:05
Ajaeger1SergeyLukjanov, did you solve the mystery of the non-updated Jenkins jobs?18:05
lifelesssure18:05
sdagueand solve the future when we get there18:06
*** needscoffee is now known as morganfainberg18:06
*** morganfainberg is now known as morganfainberg_z18:06
sdaguebecause the alternative is kind of give up, and let the gate collapse in i3, imo18:06
*** morganfainberg_z is now known as morganfainberg18:06
* anteaya just wants to ensure sdague knows his work is valued18:06
anteayabecause we need you18:07
sdaguethanks anteaya :)18:07
anteaya:D18:07
*** mriedem has joined #openstack-infra18:08
*** e0ne has joined #openstack-infra18:09
sdaguejeblair / clarkb: https://review.openstack.org/#/c/73366/ the freshness change passed if anyone wants to approve it18:09
*** pafuent has left #openstack-infra18:10
* anteaya +1's the freshness change and heads out for a walk18:10
jeblairsdague: let me think a moment about whether we could do the split you were talking about18:10
sdagueok, no worries18:11
sdagueI need to go shovel some snow anyway18:11
*** e0ne has quit IRC18:12
*** e0ne has joined #openstack-infra18:12
*** esker has quit IRC18:12
*** esker has joined #openstack-infra18:13
*** thomasbiege has joined #openstack-infra18:13
*** jooools1 has quit IRC18:13
*** derekh has quit IRC18:13
SergeyLukjanovAjaeger1, nope :( we need to ask someone from the root team to check logs18:14
*** jooools has joined #openstack-infra18:15
*** harlowja_away is now known as harlowja_18:15
*** pafuent1 has joined #openstack-infra18:15
*** talluri has quit IRC18:17
*** esker has quit IRC18:18
*** talluri has joined #openstack-infra18:18
salv-orlandoHi! We would like to enable filtering logs by severity when they're displayed through HTTP in our CI (vmware minesweeper). Do you have any pointers?18:18
salv-orlandoin other words we would like them to appear as the do on logs.openstack.org18:19
fungisalv-orlando: sdague wrote most of that. the project is openstack-infra/os-loganalyze i believe18:19
sdagueyep, that's the project18:19
sdaguesalv-orlando: it's a pretty simple wsgi filter18:20
Ajaeger1Who's around from the root team and can tell me why Jenkins is still using a more than 20hours old job? The patch was merged yesterday...18:20
fungisalv-orlando: and if you look in openstack-infra/config in the vhost template for logs.openstack.org you can see how we apply it18:20
fungiAjaeger1: i am. just a sec let me get back off my lunch computer to a proper system18:20
*** nkinder has left #openstack-infra18:20
salv-orlandofungi: thanks18:21
Ajaeger1fungi: sure! Just tell me once you've got time...18:21
geekinutahclarkb: hey I had jog0 in my office the other day explaining how we use Rackspace and HP cloud to do gating18:22
*** asomya has joined #openstack-infra18:22
*** talluri has quit IRC18:22
geekinutahclarkb: I'd like to donate some capacity, but not sure if I can support all the API calls that you need (we broke the scheduler for example, so I have to jimmy rig that)18:23
geekinutahclarkb: is there some code you could point me at so I can figure out what else I need to fix?18:23
*** hasharMeeting has quit IRC18:23
fungigeekinutah: we use python-novaclient18:24
jaypipesSergeyLukjanov, fungi, jeblair: apologies for bugging you again on this :( I'm trying to figure out the use of the 'ssh_key' parameter to the jenkins::slave puppet manifest. I had previously thought that this was intended for a private SSH key for that Jenkins slave to register with the master Jenkins, but I am questioning that assumption now, since I see ssh_key = '' in the openstack_proejct::bare_slave class inclu18:24
jaypipession.18:24
openstackgerritJames E. Blair proposed a change to openstack-infra/config: narrow the freshness window to 24h  https://review.openstack.org/7336618:24
jeblairsdague: ^ i _think_ that might do it...18:24
geekinutahfungi: okay, but what drives the client? is that nodepool?18:24
fungigeekinutah: so having a functional nova endpoint is the big need. also a usable glance and working cinder are a big plus18:24
fungigeekinutah: yes, nodepool uses novaclient to create and delete nodes18:25
jeblairSergeyLukjanov, fungi: can you double check that you think https://review.openstack.org/73366 will do what i think it will.  :)18:25
SergeyLukjanovjeblair, looking on it18:25
geekinutahfungi: glance shouldn't be a problem and we don't run cinder yet18:26
*** shardy is now known as shardy_afk18:26
geekinutahfungi: besides nodepool are there other things that use the nova endpoint?18:27
jeblairfungi, geekinutah: i don't think we need cinder for testing at the moment; only for our long-running servers18:27
fungigeekinutah: we mostly currently just use cinder for infrastructure servers18:27
fungier, what jeblair said18:27
geekinutahoh, what do the long-running servers do?18:28
fungijeblair: i believe 73366 will do what it says, but i'm having trouble holding the full state in my head to think it through18:28
jeblairgeekinutah: not gating jobs, but public services like gerrit, etc.18:28
fungigeekinutah: we run our gerrit code review, jenkins masters, zuul server(s), nodepool server, wiki, logs, and so on, and so on... on long-running systems18:29
jeblairgeekinutah: i think nodepool is the only thing that would interact with the endpoint.18:30
SergeyLukjanovfungi, jeblair, agreed, it looks like the commit message describes it but quite difficult to collect the whole context in head :)18:30
openstackgerritBhuvan Arumugam proposed a change to openstack-infra/jeepyb: Load project settings from ini file, if present.  https://review.openstack.org/7132318:31
jeblairsdague: if you're around can you review my update to https://review.openstack.org/7336618:31
geekinutahjeblair, fungi: cool, I'll check it out, I'm assuming you use nodepool for long running servers also?18:31
*** pafuent1 is now known as pafuent18:31
fungigeekinutah: no, only for single-use systems18:31
geekinutahLike, I couldn't just give you guys some baremetal with IPMI or something?18:31
fungigeekinutah: for bare metal, you may want to talk to lifeless about federating with the tripleo cloud as another region18:32
geekinutahokay, thanks much :-)18:32
lifelesshi18:32
lifelessyes18:32
fungigeekinutah: they're organizing multi-region bare-metal cloud for project use18:32
geekinutahyeah, lifeless and I have chatted previously, I'm still moving that through company politics, budget and such...18:33
geekinutah:-(18:33
lifelesswe'd love more contributed machines, we're running checks now, waiting for the second region (RH's) to come on line before we start voting18:33
fungiAjaeger1: just for context, you're saying that https://review.openstack.org/70594 doesn't seem to have taken effect, or am i looking at the wrong change?18:33
lifeless'checks' == zuul check jobs upstream18:33
Ajaeger1fungi, patch: https://review.openstack.org/#/c/71056/ but Jenkins uses old config: https://jenkins04.openstack.org/view/All/job/openstack-operations-guide-ja/configure18:34
jaypipesfungi, jeblair, SergeyLukjanov: do you all have some time today to chat with me about some Jenkins-related questions? maybe 20 minutes or so?18:34
sdaguejeblair: lgtm18:34
Ajaeger1fungi: so, yes, you looked at the wrong patch18:34
fungiAjaeger1: aha. makes slightly more sense now18:35
clarkbjog0: down to 67k events now if that gives you a feel for how quickly we are keeping up18:35
Ajaeger170594 works just fine, I'm really happy - will be perfect  once this is approved: https://review.openstack.org/#/c/73185/18:35
*** andrew_plunk has joined #openstack-infra18:35
bhuvanSergeyLukjanov: i've rebased https://review.openstack.org/71323, due to conflict. can you review/approve it again?18:36
openstackgerritPablo Andres Fuente proposed a change to openstack-infra/config: Added climate-a/climate-m to ENABLED_SERVICES  https://review.openstack.org/7337518:36
SergeyLukjanovbhuvan, adding to the backlog18:36
SergeyLukjanovjaypipes, I'm on a meeting now, sorry18:37
jaypipesSergeyLukjanov: no, I was asking if you had some time today...18:37
SergeyLukjanovfungi, Ajaeger1, I've checked all jenkins node and all of them contain obsolete job config18:37
*** thomasbiege has quit IRC18:37
fungiSergeyLukjanov: yeah, i'm going to dig deeper and see whether jjb is breaking or something18:38
fungiSergeyLukjanov: Ajaeger1: i agree that config change did not seem to get applied to the jobs in question18:39
SergeyLukjanovjaypipes, should have some time today18:40
jaypipesthx. I've been struggling to get this working for a couple days now and I've exhausted my levels of knowledge on Jenkins-related things :(18:40
clarkbSergeyLukjanov must be on a polyphasic sleep cycle18:41
*** mriedem has quit IRC18:41
jaypipesclarkb: do you know why the jenkins master manifest doesn't install a jenkins user, but the jenkins slaves do?18:43
bhuvanjeblair or any other volunteers to review https://review.openstack.org/#/c/72445/, log analyzer for openstack irc logs, using super serious stats ...18:43
clarkbjaypipes: because the jenkins package should install the user for us18:44
jeblairjaypipes: i think there was a problem with that recently the last time we spun up a master.  i'm not sure at this point whether we expect puppet to or the jenkins .deb.  i think we may have decided that we need puppet to do it for us before installing the deb.18:44
fungijaypipes: i believe the jenkins user on the master gets created by the jenkins software package installation scripts (and it may be doing it in a way which doesn't jive with our puppetry too, based on findings during the last couple we built). we puppe a jenkins user onto the slaves so we can configure it to accept ssh connections from the masters18:44
clarkbbut that assumption may not be true18:44
clarkbre 3rd party requirements and showing people how to do it with gerrit trigger plugin. I think those things are orthogonal18:44
*** cadenzajon has joined #openstack-infra18:44
jeblairfungi: we're like the same person today.  :)18:45
clarkbjust because gerrit trigger plugin doesn't meet the requirements for rechecking on openstack/* projects doesn't mean it can't be useful for say stackforge projects18:45
fungijeblair: clearly we should deduplicate our efforts--we'd be much more efficiene that way18:45
jaypipesok. when trying to figure out what was going on, I just noticed that there was no /home/jenkins on the master, but there was on the slave, so I wondered if that was problematic..18:45
fungiefficient18:45
* jeblair goes to do something else18:45
clarkbjaypipes: jenkins master homedir is /var/lib/jenkins18:45
jaypipesclarkb: ah.. that makes sense, of course.18:46
jaypipeswell.. as much sense as jenkins installations go...18:46
*** ywu has quit IRC18:46
fungiabout this ][ much sense18:46
jaypipeshehe18:46
* fungi does maxwell smart gesture18:47
jaypipesclarkb: I gather from reading the nodepool code that the upstream jenkins slaves are set up as JNPL slaves and not SSH-driven slaves.18:47
jaypipesclarkb: how exactly does that work? sorry, I'm not understanding how the masters communicate with the slaves if there are no keys exchanged or anything..18:47
jaypipesclarkb: does nodepool register each jenkins slave with one of the masters when it launches a node?18:48
jaypipes(me trying to figure out how to set up this simple external testing platform without needing nodepool...)18:48
clarkbjaypipes: they are ssh slaves18:49
clarkbso nothing fancy, nodepool via puppet installs public key into known_hosts for jenkins user, once node is ready nodepool adds it to jenkins as an ssh slave, and the connection is made from maste rto slave18:49
jaypipesclarkb: is that the nodepool_ssh_key variable?18:50
jaypipesused in the install_jenkins_slave.sh script?18:50
clarkbjaypipes: yes it can be, that variable allows you to override our default key easily18:50
clarkbour default key is baked into the puppet18:50
jaypipesoh...18:51
fungibtw, power flickering has started in earnest here... if i disappear longer than now through the weekend, send in the national guard i guess18:51
jaypipesperhaps that is the magic I am missing...18:51
*** rossella_s has quit IRC18:51
clarkbmordred: sdague: a lot of flaky unittest fails aren't due to races in bad code, but instead due to test ordering issues which won't be fixed by disabling one test, it will just affect the next vulnerable test and so on18:51
jeblairfungi: good luck!18:51
clarkbfungi: gl, I saw pictures of burning cars on the freeway outside of raleigh18:52
clarkbso I pretty much assume the zombies have taken over18:52
*** marun has quit IRC18:52
apevechow long should it take for tagged tarball to appear at tarballs.o.o ?18:52
fungiclarkb: i have my shotgun and trusty zombie bat within reach. should be fine through the worst of it18:53
apevecI've pushed https://github.com/openstack/nova/releases/tag/2013.2.218:53
*** mrfatstrat has joined #openstack-infra18:53
*** jooools has quit IRC18:53
apevecstill not there http://tarballs.openstack.org/nova/?C=M;O=D18:53
jaypipesclarkb: so I created an SSH key on my jenkins slave, supplied that key as the ssh_key parameter to the jenkins::slave manifest, and puppet applied it. I saw the message in puppet output about authorized_keys being added to...  I then manually added a node in the master jenkins web UI with that private key and the IP of the slave host... but get this on the jenkins master: http://paste.openstack.org/show/64787/18:53
nibalizerfungi: locate ham radio18:53
jaypipesclarkb: there are no iptables rules on any of the hosts18:53
funginibalizer: already got one18:53
*** jooools has joined #openstack-infra18:54
jaypipesclarkb: and I'm wondering if I am adding the ssh keys in the right place... it's quite confusing to me, sorry :(18:54
fungiapevec: the tarball job may have failed. looking in a moment18:54
apevecfungi, thanks18:54
jaypipesclarkb: perhaps I just need to create *one* key pair and automatically inject that somewhere on all hosts in the system?18:54
clarkbjaypipes: np, the jenkins master will fall back on using the ssh key in /var/lib/jenkins/.ssh18:54
clarkbjaypipes: is that hte key you supplied to the slave?18:55
jaypipeslemme check.18:55
clarkbjaypipes: yes one key, with public and private in /var/lib/jenkins/.ssh on master, then give that public key to all slave known_hosts18:55
*** VijayT has quit IRC18:55
clarkbyou can do fancier stuff than that, but that is sort of simple and useable18:55
*** asomya has left #openstack-infra18:55
apevecfungi, looks like it's just nova, other projects are ok18:56
openstackgerritBhuvan Arumugam proposed a change to openstack-infra/jenkins-job-builder: allow missing reports for HTML publisher  https://review.openstack.org/7337818:59
jaypipesclarkb: so there is only a single key  in /var/lib/jenkins/.ssh/ and it is id_rsa[.pub] that is the value of the $jenkins_ssh_private_key and $jenkins_ssh_public_key parameters to the jenkins::master puppet manifest. The SSH key I added manually in the web interface is not there. Must be stored somewhere else.18:59
fungiapevec: just so you can follow along, git show-ref 2013.2.2 will give you the sha of the tag object (rather than the commit it references). log url for that object's associated jobs is http://logs.openstack.org/c6/c671c2dc66fb983c19193c7b6af7b757d3a1c32d/18:59
jaypipesclarkb: I'm going to rework things to use a single key. I'll check back with you later. thanks for talking with me.19:00
fungiapevec: tarball job log does indeed indicate failure http://logs.openstack.org/c6/c671c2dc66fb983c19193c7b6af7b757d3a1c32d/release/nova-tarball/3e5ac77/console.html19:00
lifelesswhats up with https://review.openstack.org/#/c/71954/ not merging? it has no deps, though it does have something abandonded depending on it19:00
apevecfungi, hmm what did I do?19:01
fungiapevec: for some reason, https://review.openstack.org/p/openstack/nova did not have the tag (that /p/ goes to a local git replica on the same server as gerrit, but it still might be out of sync somehow)19:01
clarkbjaypipes: np19:02
*** mriedem has joined #openstack-infra19:02
fungiapevec: it looks properly signed... checking gerrit now to see if it was just behind for some reason19:02
clarkblifeless: were its tests lost when the tripleo cloud was down?19:03
apevecfungi, tag is on github so yeah looks like something wasn't in sync in Jenkins?19:03
fungiclarkb: jeblair: mordred: SergeyLukjanov: ^ btw, worth thinking about, maybe release jobs should look directly to gerrit's git server for tags rather than a replica19:03
fungiso as to reduce racyness19:03
openstackgerritKhai Do proposed a change to openstack-infra/config: install gerrit core plugins  https://review.openstack.org/6536819:03
SergeyLukjanovfungi, sounds reasonable IMO19:03
lifelessclarkb: they are check-non-vote only19:03
clarkbfungi: ++19:03
clarkblifeless: right, but if they never ran zuul would never +219:04
clarkbwhich is why it hasn't merged19:04
lifelessclarkb: Jenkins +119:04
clarkbbut you need +2 to merge19:04
lifelessclarkb: they aren't set for gate at all, so no, its not that19:04
lifelessclarkb: I know19:04
lifelessclarkb:     gate-noop SUCCESS in 0s19:04
lifelesswas the only comment until the recheck19:05
lifelesscan I reverify no bug perhaps?19:05
clarkblifeless: not no bug, but you can apply a +1 approve19:05
lifelessok19:05
lifelessdone19:05
lifelessmerged19:05
lifelessweird19:05
fungiapevec: review.openstack.org:/var/lib/git/openstack/nova.git/refs/tags/2013.2.2 definitely exists now, so i have to assume some sort of replication delay there. we may see more of these if zuul/jenkins start to get more efficient faster than gerrit does :/19:06
zarofungi, jeblair, clarkb : ok, now officially bugging infra-core to review needed changes for gerrit upgrade.  first one that needs to go in is https://review.openstack.org/#/c/6034819:06
fungiapevec: i'll retrigger the tarball job now19:06
jeblairlifeless: the tripelo failure exposed a bug in zuul and so we had to do a hard-stop and lost its queue.  if that approval happened around that time, it's possible that was the cause.19:06
apevecfungi, thanks, just wanted to ask how could I retrigger19:06
lifelessjeblair: ah, ack - thanks19:06
*** mrfatstrat has quit IRC19:06
jeblairfor the record, that bug is fixed in https://review.openstack.org/#/c/72042/19:07
fungiapevec: we don't have an easy way. at the moment it's a python utility which is run locally on the zuul server to retrigger post/pre-release/release pipeline jobs manually19:07
*** lcestari_ is now known as lcestari19:07
*** mriedem has quit IRC19:08
jeblairfungi: yes, either gerrit directly, or after the new merger is in place, perhaps via zuul.  i think it should dtrt there.19:09
*** oanufriev has quit IRC19:09
*** NikitaKonovalov is now known as NikitaKonovalov_19:09
openstackgerritBhuvan Arumugam proposed a change to openstack-infra/config: log analyzer for openstack IRC logs  https://review.openstack.org/7244519:09
zaroSergeyLukjanov: forgot to put you on the bugging list. look a few lines above.19:09
SergeyLukjanovzaro, see it19:10
*** jgrimm has joined #openstack-infra19:10
jeblairzaro: that changed a lot; why did we go from building buck in the job to installing the vcsrepo on the slave?19:10
*** mrmartin has joined #openstack-infra19:11
*** NikitaKonovalov_ is now known as NikitaKonovalov19:11
notmynamesdague: you mentioned earlier something about swift unit tests failing nearly 10% of the time. can you give me some more info there? I don't see that in what I'm seeing, so I want to make sure that we fix a problem if it's there19:11
fungijeblair: does trigger-job.py need an update to start passing ZUUL_URL?19:12
jeblairfungi: probably so19:12
fungijeblair: cool, i'll fix it then19:12
fungijust making sure i hadn't missed something obvious19:12
jeblairzaro: oh, it always did that19:13
fungiapevec: slight delay while i fix our tools to support our tools ;)19:13
jeblairzaro, fungi: i'm sorry i didn't catch this earlier, but do we really want to build and install buck on all the slaves?19:14
apevecfungi, np, I'm uploading other projects19:14
*** jomara has joined #openstack-infra19:14
fungijeblair: i believe the reasoning is that buck doesn't provide binary releases we can consume, so we have to build buck before building gerrit19:15
fungizaro: was that right?19:15
zarofungi, jeblair : yes, fungi is correct.19:15
jeblairfungi, zaro: but can we just do that in the gerrit build job?19:15
fungijeblair: i assume so. consider it part of the gerrit build process19:15
zarojeblair: i guess that's possible.  clarkb suggested the current implementation.19:16
jeblairfungi, zaro: we don't build and install any other job-specific software in the slave manifest, i feel weird doing it for buck.19:16
jeblairclarkb: thoughts ^?19:17
clarkbI agree it is weird, but I think it is a reasonable option19:17
zarojeblair: ohh that would me we would need to build buck for any of the 3rd party plugins jobs as well.19:17
clarkbsince it doesn't require root and keeps us up to date19:17
mrmartinfungi, jeblair: re, could you check me what was the exact reason behind the failure of this test: https://review.openstack.org/#/c/72294/ this patch also related, because it executes the gate-groups-unittests: https://review.openstack.org/#/c/72921/19:18
fungizaro: jeblair: agreed. i overlooked it in reviewing, but that ant call should just live in the (check|gate)-gerrit-buck job, maybe as a builder macro both can include19:18
jeblairclarkb: presumably building buck doesn't require root19:18
zaroso like this change for building javamelody https://review.openstack.org/#/c/7001419:18
*** NikitaKonovalov is now known as NikitaKonovalov_19:18
zarono building buck does not require root.19:18
Ajaeger1mrmartin: lost job - some kind of hickup of gerrit ;819:19
jeblairclarkb: we tend to take a pretty hard line on what gets included on openstack's unit test slaves; we've said no to installs of other random software before19:19
jeblairmrmartin: it's possible that job wasn't automatically created...19:20
clarkbjeblair: oh I misread, are we installing this at a slave level and not a job level?19:20
clarkbI was thinking we could install it at the job level19:20
clarkblocal to the workspace19:20
fungiclarkb: yes, i misread it the same way. glad i'm not the only one19:20
jeblairfungi: have you gotten to looking into the wrong job for Ajaeger1?19:20
fungijeblair: i'm looking at the syslog on jenkins0119:21
jeblairfungi: i wonder if Ajaeger1 and mrmartin are seeing manifestations of the same problem19:21
mrmartinjeblair: ok, so it means, that something wrong with the gating scripts, or I can fix it with a simple commit --amend19:21
fungiwe had one puppet timeout on the jenkins-jobs exec, but others in the timeframe since that change merged which succeeded. however i don't see puppet noticing that particular yaml file having a change, which is odd19:21
fungistill digging19:21
jeblairmrmartin: when it's fixed you can leave a comment with "recheck no bug" and it will re-run it19:22
*** sarob has joined #openstack-infra19:22
pleia2Mithrandir: oh! nice meeting you last night :)19:23
*** VijayT has joined #openstack-infra19:23
*** alexpilotti has joined #openstack-infra19:23
fungijeblair: the file is definitely updated in /etc though19:23
jeblairgate-groups-unittests exists on jenkins01 and 0219:24
jeblairoops19:24
jeblairmrmartin: the jjb config for that job is broken, it wants to run on "{node}"19:25
jeblairfungi: so there doesn't seem to be a problem with jjb execution related to groups19:25
mrmartinjeblair, yeap, I copy this part from storyboard-webclient, where this {node} value come from?19:25
fungimrmartin: they specify theirs in the projects.yaml file in that same directory19:26
clarkbjog0: down to 54k now, so slowly chewing through the backlog19:27
jeblairmrmartin: substitunions are only valid for job-templates, but gate-groups-unittests is a job (not a template)19:27
mrmartinfungi: as I do, https://review.openstack.org/#/c/72921/2/modules/openstack_project/files/jenkins_job_builder/config/projects.yaml19:27
fungijeblair: i found in syslog where puppet saw manuals.yaml receive the update, and then it triggered the jenkins-jobs run which seemed to complete error-free19:27
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Fix groups unit tests job  https://review.openstack.org/7338119:28
jeblairmrmartin: ^19:28
mrmartinjeblair: there's the point. so I need to replace {node} with precise?19:28
jeblairyep19:28
SergeyLukjanovjeblair, heh, I was really sure that it works for jobs too :(19:28
*** talluri has joined #openstack-infra19:28
*** sarob has quit IRC19:29
mrmartinSergeyLukjanov: no prob, part of the continous learning process19:29
fungijeblair: SergeyLukjanov: mrmartin: oh! i always miss that too. should be job-template19:29
fungijob doesn't expect variable substitution, job-template does19:30
mrmartinfungi: a typical hard to find typo19:30
*** tomhe has quit IRC19:30
SergeyLukjanovmrmartin, fungi, jeblair, probably we'd like it to use bare-precise?19:30
devanandalogstash down?19:30
clarkbdevananda: yes19:30
clarkbbut not really19:30
devananda?19:30
fungiSergeyLukjanov: well, we have a ton of them we still need to transition to the new node label19:30
jeblairSergeyLukjanov: probably, but it would need a change to zuul's layout.yaml to set the single use flag, so we can take care of it later19:30
mrmartinSergeyLukjanov: we need the php5-cli package19:30
devanandatrying to track down why i'm getting py26 failures in ironic's gate recently19:31
clarkbI killed the UI to free up memory so that the gearman server wouldn't OOM. But all of the data is currently be processed and you can query ES directly if you want19:31
fungidevananda: how recently?19:31
SergeyLukjanovjeblair, got it19:31
devanandafungi: this morning.19:31
fungidevananda: just in the past few hours19:31
devanandafungi: we also landed like 10 changes :)19:31
devanandayes19:31
SergeyLukjanovjeblair, so +2 on it19:31
fungidevananda: oh, we also switched the node type for py26 jobs19:31
devanandafungi: i was going to assume it was something we did ... but that seems more likely19:31
fungidevananda: let me have a link to a failure19:31
jamespageapevec, neutron tests passed as well - all good from my perpsective19:31
clarkbunfortunately the gearman server doesn't GC as well as I had hoped so I haven't been able to start the UI now that we are down to 53k gearman jobs19:31
mrmartinjeblair: I thing it won't work with bare-precise because I added the php5-cli package and it was not included in bare19:31
devanandafungi: http://logs.openstack.org/15/67415/15/gate/gate-ironic-python26/399c460/19:31
jeblairclarkb: is log-gearman-client leaking?19:32
clarkbI think I will need to restart the gearman serer when it hits 019:32
clarkbjeblair: yes19:32
apevecjamespage, excellent, thanks for jumping in!19:32
jeblairclarkb: is log-geraman-client also the server?19:32
fungiclarkb: jeblair: also i noticed that nodepool is swapping very heavily again19:32
clarkbjeblair: yup19:32
clarkbthey are bundled together :/19:32
apevecjamespage, release is on its way19:32
jamespageapevec, \o/19:32
*** talluri has quit IRC19:32
jeblairfungi: not anymore.  19:33:11 up  1:06,  1 user,  load average: 0.21, 0.18, 0.1519:33
fungigah19:33
jeblairfungi: that probably means we have some zombie nodes.19:33
* fungi shakes his fist at the sky in futility and gets back to debugging things19:34
*** dcramer_ has quit IRC19:34
jeblairfungi: i will remove the nodes that have been building for > 1 hour19:35
fungijeblair: thanks!19:35
*** jooools has quit IRC19:37
fungijeblair: so, looking at the openstack-operations-guide-ja job on jenkins01, the /var/lib/jenkins/jobs/openstack-operations-guide-ja/config.xml file reflects the updated job configuration, but the jenkins webui for the same job shows the old configuration19:37
*** jooools has joined #openstack-infra19:37
jeblairfungi: wow!19:37
jeblairfungi: oh!19:37
fungiso for some reason the running jenkins server doesn't know the xml updated?19:37
jeblairfungi: this is a maven->freestyle change isn't it19:37
fungiYES19:38
jeblairfungi: i just remembered that is broken in jenkins and can't be handled by jjb19:38
fungiwhat dark magic do i not know about there?19:38
jeblairfungi: so we need to delete that job and let jjb recreate it19:38
fungijeblair: can do19:38
jeblairAjaeger1: and we'll need to do that for other maven->freestyle changes19:38
fungiAjaeger1: ^ mystery solved. blame jenkins19:38
Ajaeger1fungi, jeblair thanks for solving this!19:38
devanandafungi: any leads on the py26 issue? should I just retry for now?19:38
fungiAjaeger1: i'll get this one fixed asap19:39
jeblair(but it's definetly worth doing; we'll just have to try to remember to do that when we approve thes changes)19:39
Ajaeger1jeblair, fungi: If I do another maven->freestyl change, should I mark this somehow special so that you know to delete the job first?19:39
fungidevananda: you're 3rd in line. i need to fix one of our tools next so i can help apevec work around a race condition we encountered for the nova stable release19:39
*** e0ne has quit IRC19:39
*** e0ne has joined #openstack-infra19:40
fungiAjaeger1: or just bug one of us with root access after it merges, because it's possible we'll forget between approving and running19:40
clarkbjeblair: fungi https://review.openstack.org/#/c/73037/ any idea what was causing jobs to be lost this morning? also that change was approved again but not queued19:40
Ajaeger1fungi: will try to remember. Thanks!19:40
clarkbjeblair: wondering if that is related to the stuff you and sdague were talking about earlier19:40
jeblairclarkb: let's investigate those in the opposite order19:40
clarkbsure19:40
devanandafungi: ack, np :)19:41
jeblairclarkb: i think the enqueue problem is a hole in the 24->72 hour freshness window change19:42
*** e0ne_ has joined #openstack-infra19:42
*** mrmartin has quit IRC19:42
*** NikitaKonovalov_ is now known as NikitaKonovalov19:42
fungijeblair: we just need to wait another 48 hours and everything will be working as intended, right? ;)19:43
*** gyee has quit IRC19:43
jeblairoh, actually, i don't think it's related to the 24-72 hour change, i think this was an expected behavior19:43
jeblairthat -2'd changes would need to have a recheck applied manually, you couldn't just aprv them19:44
*** e0ne has quit IRC19:45
jeblairclarkb: so i think the current state with the new gate configuration is that if jenkins leaves a -2 vote on a patch, you need to do 'recheck bug xxx' to get a clean check result.19:46
*** mfer has quit IRC19:46
notmynamejeblair: when did that change? and in this case, what bug number do I use?19:47
jeblairclarkb: i think that means that reverify is broken19:47
boris-42_jeblair clarkb could you pls take a look at this patch (adding requirements that is actually already presented) https://review.openstack.org/#/c/73025/ (already have 2x+2)19:47
fungijeblair: i think recheck followed by reverify might work?19:47
jeblairfungi: yes19:47
*** NikitaKonovalov is now known as NikitaKonovalov_19:47
*** fbo is now known as fbo_away19:47
fungiassuming the recheck passes anyway19:48
jeblairnotmyname: let's hold 'what bug to use' until i figure out why it was LOST19:48
jeblairsdague: ping19:48
notmynamejeblair: k19:48
*** mfer has joined #openstack-infra19:48
*** NikitaKonovalov_ is now known as NikitaKonovalov19:49
notmynamejeblair: thanks for looking into it. FWIW, I'm (initially) not a fan of removing core reviewer's ability to approve something and have is re-enqueued. but this is the first I've heard of it, so it's still new to me19:49
jeblairsdague, clarkb, fungi: when sdague is back from shoveling snow, let's see about being more clear about the expected process when jenkins leaves a -2; cause i think we broke reverify.19:49
clarkbok19:50
jeblairnotmyname: i think we have a rough edge here that i bet we can work out, but i don't want to go into details without sdague19:51
jeblairso moving on to LOST...19:51
jeblair2014-02-13 18:15:42,489 INFO zuul.Gearman: Gearman job <gear.Job 0x7f0dde0c9890 handle: H:127.0.0.1:1626291 name: build:gate-tempest-dsvm-full unique: d36b728b64c141e6a7fc2ba3d2f1e2cd> lost due to disconnect19:51
Ajaeger1infra team, could you quickly approve this one as well, please? This would be soo nice for the docs-drafts: https://review.openstack.org/#/c/73185/19:51
fungiAjaeger1: okay, openstack-operations-guide-ja has been deleted and recreated on all the jenkins masters. that's the only job affected by that template so far, right?19:52
jeblairit looks like we lost 3063 jobs around 1815 due to disconnects19:53
*** lcheng_ has joined #openstack-infra19:53
Ajaeger1fungi: yes, that's the only job you had to touch.19:53
Ajaeger1I wanted to start the maven migration very slowly...19:53
Ajaeger1... with some baby steps ;)19:54
*** wenlock_ has joined #openstack-infra19:54
fungijeblair: that's *suspiciously* close to when nodepool.o.o got rebooted too19:54
fungireboot   system boot  Thu Feb 13 18:2619:55
jeblairfungi: interesting; zuul has been up that whole time19:55
fungii don't see any way it could be related, unless they rebooted a bunch of our vms rather than just nodepool19:56
fungii don't suppose we know if those disconnects were only to rackspace nodes19:56
jeblairfungi: well, because of the way the gearman plugin for jenkins works, there are really only connections to zuul's gearman from the jenkins masters19:57
fungioh! those are jenkins masters disconnecting from zuul?19:58
jeblairfungi: (there's still a connection for each worker, but they'll all be from the master hosts)19:58
jeblairfungi: or zuul disconnecting from geard19:58
funginothing obvious related on status.rackspace.com19:58
*** NikitaKonovalov is now known as NikitaKonovalov_19:59
jeblairfungi: and actually my gear log patch would really help in this case19:59
clarkbdoes cacti show a spike in tcp errors?19:59
fungirackspace ticket saying "The server 23.253.76.129 has been rate-limited due to excessive queries against our caching nameservers."20:00
clarkbno apparent errors caught by cacti20:00
*** NikitaKonovalov_ is now known as NikitaKonovalov20:00
fungino reverse dns on 23.253.76.129 so probably a nodepool node20:00
*** pafuent has quit IRC20:01
fungiand the usual friendly "1st Notice: Server nodepool.openstack.org, Out of Memory/Swapping and Rebooted"20:01
*** e0ne has joined #openstack-infra20:03
fungizuul says "Last reconfigured: Thu Feb 13 2014 16:25:04 GMT+0000 (UTC)" so this doesn't coincide with a config reload either20:03
jeblair2014-02-13 18:15:41,797 ERROR gear.Client: Connection <gear.Connection 0x25b7190 host: 127.0.0.1 port: 4730> timed out waiting for a response to a submit job request: <gear.Job 0x7f0e8029dd10 handle: None name: build:gate-horizon-pep8 unique: 1ca1dfc3b17e408f9bad8aad74af7be4>20:03
jeblairso i think zuul abandoned its own connection to its own gearman server20:04
*** ArxCruz has quit IRC20:04
*** rfolco has quit IRC20:04
*** mrodden1 has joined #openstack-infra20:05
jeblairthat's a 30 second timeout20:05
*** andreaf2 has joined #openstack-infra20:05
*** mrodden has quit IRC20:05
*** DinaBelova is now known as DinaBelova_20:05
*** andreaf has quit IRC20:06
*** e0ne_ has quit IRC20:06
apevecfungi, drop me an email when you retrigger tarball job - I'll check later after dinner20:06
apevecall other tarballs are uploaded to LP20:06
*** boris-42_ has quit IRC20:07
*** DinaBelova_ is now known as DinaBelova20:08
fungiapevec: it's running now--i'll let you know once it's there20:08
apevecok, thanks20:08
apevecfungi, actually I'll see it when it shows up on tarballs.o.o :)20:09
sdaguejeblair: back20:09
sdaguewhat's up?20:09
*** mrodden1 has quit IRC20:10
fungiapevec: well, i should say, it's in the gearman queue, but i think it may still be waiting to get picked up by a worker20:10
*** dcramer_ has joined #openstack-infra20:13
*** CaptTofu has quit IRC20:14
fungiapevec: it worked. http://logs.openstack.org/c6/c671c2dc66fb983c19193c7b6af7b757d3a1c32d/release/nova-tarball/3e5ac77,2/console.html http://tarballs.openstack.org/nova/nova-2013.2.2.tar.gz20:14
apevecfungi, thank you!20:14
* apevec uploads20:14
jeblairnotmyname: https://bugs.launchpad.net/openstack-ci/+bug/127996320:15
jeblairnotmyname: use that bug for rechecks please20:15
openstackgerritJeremy Stanley proposed a change to openstack-infra/zuul: Update trigger script for new zuul url parameter  https://review.openstack.org/7339220:15
jeblairfungi: nodepool does talk to that gear server; i wonder if gear got stuck trying to reply to nodepool and blocked processing of other clients (eg zuul)20:16
notmynamejeblair: thanks20:16
fungijeblair: quite possible. that could be around the time it ran out of swap20:16
* fungi checks cacti20:16
fungihttp://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=979&rra_id=all20:17
funginot really20:17
fungithough there is a nice gap in all nodepool's cacti graphs a little while prior20:18
jeblairfungi: it was certainly within the last 15 minutes of the server's life20:18
jeblairfungi: the iowait and swap activity metrics were as high as they were going to get20:19
*** jooools has quit IRC20:19
fungidefinitely20:19
*** pafuent has joined #openstack-infra20:19
*** saju_m has quit IRC20:19
fungii mean, the server was probably thrashing hard for a while there, so it's anybody's guess what could go on under thiose conditions20:20
jeblairfungi: so i think i should fix the nodepool leak; it may be worth a look in gear to see if there's something i need to do to prevent clients from blocking each other.20:20
* clarkb is busy trying to spin up a second laptop sorry I haven't been more helpful20:20
sdaguejeblair: so what was the -2 issue that showed up?20:20
clarkbam still watching logstash though under 40k gearman jobs20:20
fungidevananda: okay, looking at your failure now to see if maybe there's something different about our bare-centos6 nodes than the previous static centos6 slaves20:21
*** jooools has joined #openstack-infra20:21
sdaguecan we get another requirements core to +2 this - https://review.openstack.org/#/c/72776/ ?20:21
jeblairsdague: https://review.openstack.org/#/c/73037/20:21
jeblairsdague: we have some rough edges around jenkins leaving a -220:22
sdagueok20:22
sdagueso can we trigger it all over again on a +A toggle?20:22
jeblairsdague: currently you have to "recheck bug xxx" then "reverify bug xxx".20:23
jeblairsdague: possibly.  but now i need to go to lunch20:23
sdagueor recheck bug, and +A toggle20:23
sdagueright?20:23
openstackgerritAndreas Jaeger proposed a change to openstack-infra/config: Replace install-deploy-guide-$LANG jobs  https://review.openstack.org/7339720:23
sdagueok, enjoy lunch20:23
jeblairsdague: thx, let's pick this up when i get back20:24
sdagueyep20:24
*** jooools has quit IRC20:26
*** cadenzajon has quit IRC20:26
*** melwitt has joined #openstack-infra20:27
*** Sukhdev has joined #openstack-infra20:27
*** jooools has joined #openstack-infra20:28
devanandafungi: i can get you a few more links to py26 failures if it helps20:28
dstanekbknudson: in https://review.openstack.org/#/c/73026 why is it --log_config instead of --log-config?20:29
fungidevananda: having the earliest one you know of would also be great20:30
fungidevananda: because if it ran on a slave named like centos6-somenumber instead of bare-centos6.stuff then we know it's not related to the node type change20:31
bknudsondstanek: what is --log_config?20:31
*** jooools has quit IRC20:31
dstanekbknudson: that's what i was wondering :-) https://review.openstack.org/#/c/73026/1/doc/source/man/keystone-all.rst20:31
bknudsondstanek: it's probably a bug in oslo.config?20:32
*** jooools has joined #openstack-infra20:32
dstanekbknudson: both of those see to work, but you change the docs to use the _ version; i didn't know if that was intentional or not20:33
bknudsondstanek: not sure why you want to discuss this on -infra?20:33
devanandak, looking20:33
devanandafwiw, having logstash would help ;)20:33
*** boris-42_ has joined #openstack-infra20:34
dstanekbknudson: oops wrong channel20:34
openstackgerritTerry Howe proposed a change to openstack-infra/gitdm: Terry Howe added to other files  https://review.openstack.org/7165120:34
fungidevananda: also, if test__get_node_vif_ids_no_ports is likely to use particular system libraries, that would be good to know too. we at least discovered that rackspace didn't necessarily install all the same packages by default on the pvhvm centos images as on the traditional non-pvhvm ones, so it could be some unpuppeted system dependency we're suddenly missing20:34
*** senthil has quit IRC20:35
dtroyerjeblair, fungi, clarkb: When you get a minute for a not-fix-broken-gate review this one is hopefully simple: https://review.openstack.org/#/c/71318/ to add devstack top openstackclient's gate…we now have 2 devstack reviews that use OSC, I don't want the approve them without this change20:37
devanandafungi: "DriverNotFound: Failed to load driver fake_pxe" seems to be the error, which is coming from stevedore20:38
*** jooools has quit IRC20:38
devanandawe wrap the stevedore error and raise that20:38
*** jooools has joined #openstack-infra20:40
*** jooools has quit IRC20:40
apevecIf anyone has a second, please proof-read https://wiki.openstack.org/wiki/ReleaseNotes/2013.2.220:40
*** jooools has joined #openstack-infra20:42
*** oanufriev has joined #openstack-infra20:46
*** CaptTofu has joined #openstack-infra20:48
annegentleapevec: nothing comes up in spellcheck nor read-through... I am not the best copy editor but wanted to take a look :)20:48
apevecannegentle, thanks, much appreciated!20:49
annegentleapevec: you might say "low risk" rather than "risk free" as nothing is free of risk :)20:49
annegentleapevec: otherwise, lgtm20:49
apevecgood point, I'll update our template20:49
*** bhuvan has quit IRC20:50
*** Sukhdev has quit IRC20:50
openstackgerritDiane Fleming proposed a change to openstack-infra/config: OpenStack Image Service API v2 Reference link broken  https://review.openstack.org/7333120:50
*** gokrokve has joined #openstack-infra20:51
*** yolanda has quit IRC20:51
*** NikitaKonovalov is now known as NikitaKonovalov_20:51
*** bhuvan has joined #openstack-infra20:52
*** oanufriev has quit IRC20:53
*** NikitaKonovalov_ is now known as NikitaKonovalov20:54
*** jooools has quit IRC20:54
fungidevananda: if it helps, here's the difference in system packages between the old static centos6 slaves and the new bare-centos6 nodepool nodes... http://paste.openstack.org/show/65171/20:54
apevecannegentle, although, we do say "relatively risk free" which is rather strange wording, now that I read it20:55
*** jooools has joined #openstack-infra20:55
fungidevananda: i'd pay particular attention to any of those which are python libraries, if ironic is using system site packages in tox/virtualenv like nova does20:55
fungiapevec: you can blame the english language. it's full of horrible self-contradictory phrases like that one20:56
apevecannegentle, "low risk" is better20:56
apevecfungi, that's what makes it fun for me as a non-native speaker20:56
fungiapevec: i'll grant you it's fun enough as a native speaker20:57
Ajaeger1infra team, could you review and approve Diane's patch https://review.openstack.org/#/c/73331/, please? This is needed since she reworked the image-api repo a bit21:00
*** andre__ has quit IRC21:00
Mithrandirmaybe "no known regressions"?21:01
*** andreaf2 has quit IRC21:01
lifelessgot another approved-but-not-merging https://review.openstack.org/#/c/73078/21:03
lifelesssuggestions? ANd this is new, so definitely not the preior disucssion21:03
*** denis_makogon_ has joined #openstack-infra21:05
fungilifeless: it's got a recent successful check run. i wonder if another side effect of the recent change is that approvals which are applied before the check completes get ignored by zuul...21:05
fungijeblair: ^ (when you're back)21:05
pafuentHi. Can someone take a look at https://review.openstack.org/#/c/73375/? This will help Climate for incubation. BTW the change is only one line21:05
*** smarcet has left #openstack-infra21:05
devanandahm, after a gate failure, does re-ticking +A no longer trigger jenkins to retry?21:06
fungidevananda: you need to recheck first at the moment21:06
fungirecheck bug somenumber21:06
devanandafungi: some patches are coming back fine now. are you running a mix of py26 hosts?21:06
lifelessfungi: so, I'll hit +A and see what happens?21:06
lifeless/?21:06
fungilifeless: right, see if that kicks it21:06
*** rlandy has quit IRC21:06
fungidevananda: no, at this point they should all be bare-precise nodepool nodes, but i'll make sure jenkins isn't still doing something wierd like handing some jobs to the old nodes21:07
devanandafungi: is there a bug for the work ya'll did this morning?21:07
*** ociuhandu has quit IRC21:07
devanandaotherwise, there's just a few left, and i'll recheck no bug21:07
lifelessfungi: yes, that tickled it. I like your theory.21:07
fungidevananda: looks like jenkins isn't assigning any new jobs to the old static centos6 slaves at this point. your py26 errors aren't happening any longer?21:08
fungioof, power blackouts are worsening21:08
*** apevec has quit IRC21:08
devanandafungi: a few tests that finished in the last 15 min were fine. i'm retrying the ones that failed21:08
devanandamay have jsut been transitory21:09
*** thedodd has joined #openstack-infra21:09
fungidevananda: might have been something in one of your requirements then, which got released broken and then a quick brown bag release to fix. you could compare pip freeze results between runs21:09
*** denis_makogon_ is now known as denis_makogon21:10
*** DinaBelova is now known as DinaBelova_21:10
jeblairdtroyer: wow, uh, is openstackclient an official project?21:11
*** NikitaKonovalov is now known as NikitaKonovalov_21:11
mgagneIs check gate run on -1? They look to have been triggered by my comment: https://review.openstack.org/#/c/69511/21:13
mgagneSame here with -2: https://review.openstack.org/#/c/71108/21:14
jeblairfungi: i think your theory is correct21:14
fungimgagne: it's triggered on any comment if it's been a while since the last check run21:14
mgagnefungi: oh, funny. thanks!21:14
*** Ajaeger1 has quit IRC21:14
fungimgagne: currently 48 hours, soon to probably be 2421:14
mgagnefungi: a while means? 48h ?21:15
*** lcostantino has quit IRC21:15
* fungi nods21:15
jeblairfungi: i think that merged21:15
mgagnety21:15
jeblairfungi: except it should be 72 if not approved21:15
jeblair24 if approved21:15
fungioh, right21:15
fungii keep forgetting that stretched the unapproved rechecks back to 7221:16
jeblairfungi: but back to your theory... actually it should have triggered the gate pipeline; that's supposed to trigger on jenkins +121:16
* SergeyLukjanov going to write the note about it somewhere21:16
fungigood point... so something about that didn't match up... looking at the zuul debug log to see if i can tell what21:16
jeblairsdague: i'm back, you still here?21:17
jeblairdtroyer: i really like the idea of openstackclient; i'm just thinking that if we're going to start using it everywhere, including in the gate, it ought to be an official project21:18
*** pafuent has left #openstack-infra21:18
jeblairdtroyer: hrm, we put it in openstack/; it's just not listed in openstack/governance21:19
jeblairdtroyer: who's running it?21:19
openstackgerritDoug Hellmann proposed a change to openstack/requirements: Add cmd2 for cliff  https://review.openstack.org/7341221:19
jeblairfungi: oh, the 24h change hasn't merged due to the same thing you're looking into.21:21
fungirelevant line for the start of zuul seeing its own comment is i think at...21:22
fungi2014-02-13 20:15:57,499 DEBUG zuul.Gerrit: Change <Change 0x7f0ddc187a50 73078,4> status: NEW21:22
dtroyerjeblair: I run it… it isn't official, and was created before any of the guidelines were in place for repos, etc (almost 2 years ago)21:22
*** derekh has joined #openstack-infra21:23
openstackgerritA change was merged to openstack-infra/config: narrow the freshness window to 24h  https://review.openstack.org/7336621:23
jeblairdtroyer: it's all coming back to me now.  :)  we should find a program for it...21:23
*** marun has joined #openstack-infra21:24
dtroyerjeblair: we've talked about sticking it under the Oslo program, dhellmann was kind enough to give me some slots in HK21:24
dtroyerjeblair: but I think longer-term it makes sense with the SDK stuff that should be coming up soon21:24
fungii mean, it's called "openstack/python-openstackclient" so how much more official can it get? says openstack twice right there ;)21:24
jeblairdtroyer: yes, i agree21:24
jeblairfungi: they cancel out21:24
jeblairdtroyer: openstackclient + sdk may make a good fit21:25
*** CaptTofu has quit IRC21:25
dtroyerjeblair: I'm not sure about the timing of that though21:25
jeblairdtroyer: devstack-gate change lgtm; aprvd21:26
fungiyeah, it does match up with the other general service-agnostic openstack sdks for various languages21:26
dtroyerjeblair: thanks21:26
*** cadenzajon has joined #openstack-infra21:26
dtroyerfungi: which is why I'm sticking my nose into that work too  :)21:27
sdaguejeblair: yes21:27
*** jooools has quit IRC21:27
*** Clabbe has quit IRC21:28
*** jooools has joined #openstack-infra21:28
*** mrmartin has joined #openstack-infra21:28
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Restore expected reverify/reapprove behavior  https://review.openstack.org/7341821:28
*** mrodden has joined #openstack-infra21:29
jeblairsdague: okay, while i was waiting, i went ahead and threw that together ^ but we can talk about whether that's desirable21:29
openstackgerritA change was merged to openstack-infra/config: Gate openstackclient on DevStack  https://review.openstack.org/7131821:29
*** amcrn has quit IRC21:29
*** mriedem has joined #openstack-infra21:29
jeblairsdague: so i think the current state is that we broke reverify and reapprove because they will never satisfy the freshness critera (which is all about having a +1; there's no current way to get to a +1 with those actions)21:30
*** alexpilotti has quit IRC21:30
jeblairsdague: so that change makes it so that reverify now triggers the _check_ pipeline iff jenkins has left a -221:30
devanandafungi: what's the difference between http://logs.openstack.org/82/70382/4/check/gate-ironic-python26/ca57a41 and http://logs.openstack.org/21/66621/6/check/gate-ironic-python26/b37dde321:30
jeblairsdague: it also makes it so that an approved vote triggers the _check_ pipeline iff jenkins has left a -221:31
devanandafungi: first one passed, second one failed, within a minute of each other.21:31
openstackgerritA change was merged to openstack-infra/config: Added climate-a/climate-m to ENABLED_SERVICES  https://review.openstack.org/7337521:31
jeblairsdague: reverify has no effect on the gate now.  but the +1 result from check runs should trigger gate21:31
*** cody-somerville has quit IRC21:31
fungijeblair: so, looking at the debug log, zuul saw its vrif:1 event on the stream, and then again at 20:19 it saw when its parent change (71725,3) got enqueued to the gate pipeline, but didn't enqueue it then either21:31
fungii'm going to stare at the event config patch again for a bit and see if the answer just comes to me21:33
*** cody-somerville has joined #openstack-infra21:33
*** cody-somerville has quit IRC21:33
*** cody-somerville has joined #openstack-infra21:33
jeblairfungi: ugh, what if there's a race condition and gerrit wasn't reporting the verified vote when zuul queried it21:33
jeblairfungi: (like gerrit sends out events, and then, a few ms later it updates its cache/db so that queries report the same data)21:34
fungizuul gets the vrif:1 comment-added event on the stream. does it then go recheck gerrit to find out whether that's legit?21:34
*** mrmartin has quit IRC21:35
jeblairfungi: yeah, it's not too bright that way.  basically each time there is an event from gerrit, zuul requeries the associated change so it has all of the latest data21:35
jeblairfungi: but it doesn't try to match what gerrit is doing though.21:36
fungithen that sounds like a potential design flaw in our assumptions21:36
jeblairfungi: we'd need to verify that's the case.  i'm just brainstorming.21:36
fungii agree it's a distinct possibility21:36
jeblairfungi: i think we can find out easily by enabling zuul's gerrit debug logging21:38
jeblair(which will require a restart i think)21:38
*** derekh is now known as derekh_afk21:39
*** gokrokve has quit IRC21:39
*** gokrokve has joined #openstack-infra21:40
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Enable zuul gerrit debug logging  https://review.openstack.org/7342321:40
jeblairfungi: ^21:40
*** oanufriev has joined #openstack-infra21:40
*** vkozhukalov has quit IRC21:41
fungidevananda: same type of node on the same provider built from the same image, consistent pip freeze output (eyeballing it, so it's possible i missed something). my guess is either a nondeterministic problem in the code, or one of those patches is at fault?21:42
devanandafungi: neither patch changes code near the error. yay nondeterminism21:43
openstackgerritA change was merged to openstack-infra/config: Fix groups unit tests job  https://review.openstack.org/7338121:43
devanandafungi: thanks for checking on the node types to confirm it is in fact ironic21:44
*** gokrokve has quit IRC21:44
fungidevananda: well, i'm still not 100% sure, but sure enough that i'm comfortable it's not likely impacting any other projects, and so have other things i need to look at now21:44
*** hemna has joined #openstack-infra21:45
devanandafungi: ack :)21:45
openstackgerritA change was merged to openstack/requirements: Add requirements for taskflow  https://review.openstack.org/7334221:47
*** ok_delta has quit IRC21:47
jheskethHey21:49
jeblairjhesketh: hello21:49
*** jhesketh__ has joined #openstack-infra21:50
*** jooools has quit IRC21:52
*** jooools has joined #openstack-infra21:53
jheskethhey jeblair. If you have some time to take a look at my swift changes to make sure I'm on the right track before I write some tests that'd be awesome https://review.openstack.org/#/c/68297/21:53
*** CaptTofu has joined #openstack-infra21:55
hemnaany chance I can get a +3 on my requirements change ? https://review.openstack.org/#/c/72776/21:56
openstackgerritDoug Hellmann proposed a change to openstack-dev/pbr: Updated from global requirements  https://review.openstack.org/7311921:58
*** MIDENN_ has joined #openstack-infra21:59
fungihemna: i think vishy is the only person who wields the transcendent +3 vote22:00
*** gokrokve has joined #openstack-infra22:01
fungihemna: anyway, the people who regularly +2 requirements changes are listed at https://review.openstack.org/#/admin/groups/131,members22:01
*** dcramer_ has quit IRC22:03
clarkbdown to 8k events22:03
clarkbwhen it hits 0 I will restart kibana and puppet on that node after restarting the leaky process22:04
hemnafungi, ok thanks22:04
fungijeblair: a race is feeling more likely... look at the sequence of check result, approval and gating on your 73423 change22:05
fungijeblair: oh, or did you use zuul enqueue from the command line on that one?22:05
*** arborism has joined #openstack-infra22:05
anteayahemna: markmcclain is looking at it22:06
fungijeblair: particularly since that change should (in theory) have emitted no events for ~6 minutes before it was spontaneously enqueued22:06
hemnathank you22:06
openstackgerritA change was merged to openstack-infra/config: Enable zuul gerrit debug logging  https://review.openstack.org/7342322:07
markmcclainhemna: approved22:08
anteayasdague: who is in charge of culling the requirements-core list? I am wondering having Dave Walker, Joe Heck and Gabriel Hurley listed is still accurate.22:08
anteayaalso should anyone else be added?22:08
*** arborism is now known as amcrn22:09
anteayahemna: you are +A'd22:09
lifelessredeploying the testenvironments for the tripleo ci cloud22:09
*** skraynev is now known as skraynev_afk22:09
lifelessthis will stall jobs, but they should recover shortly22:09
*** dcramer_ has joined #openstack-infra22:11
openstackgerritSergey Lukjanov proposed a change to openstack/requirements: Don't update pbr's setup.py  https://review.openstack.org/7343122:11
SergeyLukjanovdhellmann, crazy 2am patch :) ^^22:12
*** mfer has quit IRC22:13
*** pdmars has quit IRC22:13
dhellmannSergeyLukjanov: do you think it's better to look at the contents of setup.py or the path to detect pbr?22:13
hemnaanteaya, thank you22:13
jeblairfungi: oh, i think i reapproved it but i guess gerrit didn't record that22:14
SergeyLukjanovdhellmann, 50-50, it could be used manually by users, so, it's hard to rely on path22:14
fungiweiiiird22:14
dhellmannSergeyLukjanov: good point22:14
jeblairfungi: i think it's the thing clarkb is fixing?  did you see 2 aprv events for it?22:15
dhellmannSergeyLukjanov: how about looking at the name in setup.cfg? I'm worried if we ever do change the way that setup.py works we'll break this and not realize it22:15
clarkbjeblair: fungi sounds like it22:15
SergeyLukjanovdhellmann, that's a good point too :)22:15
dhellmannSergeyLukjanov: crazy 5 PM thinking ;-)22:15
SergeyLukjanovdhellmann, good option to look into the setup.cfg, will upload a new patch22:16
SergeyLukjanovdhellmann, like this idea much more than relying on content of setup.py22:16
*** thomasem has quit IRC22:17
*** senthil has joined #openstack-infra22:17
*** thomasem has joined #openstack-infra22:17
*** thomasem has quit IRC22:17
mordred++22:17
mordrednice solution22:17
mordreddhellmann: so - I had this crazy idea ...22:18
* dhellmann reaches for the bourbon22:18
mordreddhellmann: which is that, as part of sdist in a pbr project, we generate a setup.py that is self-contained and doesn't use pbr22:18
mordreddhellmann: so that a tarball on pypi behaves as a tarball on pypi does with no crazy22:18
* dhellmann thinks22:18
mordreddhellmann: and then write a pbr command-line tool that you can install on your system if you're going to be developing from git repos on projects that use pbr22:19
dhellmanncould pbr just generate all of the metadata and put it in the package with a generic setup.py?22:19
mordredand not have a setup.py in the git repo at all22:19
mordreddhellmann: it already does that22:19
dhellmannexcept that the setup.py we have now also wants pbr22:19
mordred(except we might be talking about different thigns)22:19
dhellmannis all of the metadata already in the package? or is some being generated at install time?22:20
dhellmannmordred: either way, I like the idea of dropping the install time use of pbr22:20
*** dstanek has quit IRC22:20
openstackgerritSergey Lukjanov proposed a change to openstack/requirements: Don't update pbr's setup.py  https://review.openstack.org/7343122:20
mordreddhellmann: all the metadata is in the sdist tarball - it's self-contaiend - EXCEPT, it setup_requires pbr22:21
dhellmannmordred: we would need to do something for the tox case, right?22:21
SergeyLukjanovdhellmann, mordred, ^^22:21
fungijeblair: it saw the approval event reflected in the gerrit comment histry at 21:45 and then one more at 21:59 coinciding with when it started enqueuing22:21
fungibased on my reading of the debug log22:21
fungiclarkb: is that what you're tracking?22:21
SergeyLukjanovmordred, dhellmann, do we really need to increasy complexity of pbr's usage?22:21
SergeyLukjanov(crazy question)22:22
dhellmannSergeyLukjanov: I'd rather parse that file but that looks close to right22:22
clarkbfungi: I am fixing https://gerrit-review.googlesource.com/#/c/54490/1 upstream22:22
dhellmannSergeyLukjanov: the idea is to find a way to reduce it, by making it something that distributors of packages use, but installers do not need to know about22:22
clarkbfungi: tl;dr is that sometimes when you don't change the info going into a comment it is never recorded in the DB but events are generated22:22
clarkblike a phantom comment22:22
mordreddhellmann: nope. tox is easy - tox already has requirements list - and it processes them before it processes the local repo22:22
dhellmannmordred: so the requirements would include pbr, and then what?22:23
mordredso we could tell it to requirements = pbr - and then we could have the install command be "pbr install ." or some shit22:23
*** e0ne has quit IRC22:23
fungiclarkb: aha, that sounds right then22:23
dhellmannmordred: is that install command used just for the current package, or for all things?22:23
jeblairfungi: so yeah, it still missed enqueuing based on the verify+1, and only enqueued after i gave it a second aprv which is not in the ui because i left no text both times22:23
mordredSergeyLukjanov: I'm actually trying to make it less complex22:23
mordreddhellmann: jsut for current package22:23
mordredSergeyLukjanov: but I might be failing :)22:23
dhellmannmordred: ok, that was never clear to me22:23
*** e0ne has joined #openstack-infra22:23
SergeyLukjanovdhellmann, I understand you :)22:24
* fungi needs to heat up some dinner... bbiab22:24
jeblairfungi: this is going to explode zuul's logs22:24
jeblairfungi: so i'm going to delete some old ones to make room22:24
dhellmannmordred: I like that direction22:24
jeblairfungi: root fs is at 80&22:24
dhellmannmordred: I wonder if we could get a GSoC project to work on some of this?22:24
*** gokrokve has quit IRC22:25
mordredclarkb: are the upstream people ok with you fixing that?22:25
mordreddhellmann: zomg. then I'd actually have to clearly write down thoughts22:25
*** gokrokve has joined #openstack-infra22:25
dhellmannmordred: life is all about trade-offs22:25
jeblairdhellmann: ooh, you know what, we never applied to gsoc for infra because we were working on other peoples tools... we have enough of our own now i bet we could do something.22:25
mordreddhellmann: in a form other than "dhellman: I had a crazy idea about pbr"22:25
mordredjeblair: ++22:25
anteayamordred: or ensure dhellmann and GSoC had lots of bourbon22:25
dhellmannjeblair: quick! deadline is tomorrow22:25
clarkbmordred: they seemed to be, dborowitz indicated it should be fixed and I think my second patchset meets his criteria, still need to figure out buck and test it though22:26
*** dcramer_ has quit IRC22:26
jeblairdhellmann: maybe next year then.  i don't want to do a rush job (i think it merits more time and consideration than i can give it by tomorrow)22:26
mordredjeblair: I just forwarded you an email about mentoring org applications22:26
mordredkk22:26
dhellmannjeblair: definitely22:26
mordredanteaya: :)22:26
dhellmannanteaya: :-)22:27
*** gokrokve_ has joined #openstack-infra22:27
sdaguejeblair: ok back22:28
sdaguesorry about that22:28
*** e0ne has quit IRC22:28
jeblairsdague: np22:28
*** marun has quit IRC22:29
clarkbdown to 2.3k gearman jobs22:29
*** gokrokve has quit IRC22:29
jeblairclarkb: i'm going to hard-stop zuul; remove git repos, then start it again22:30
jeblairclarkb: this is mostly to pick up the logging change22:30
dhellmanncan I ask where things stand with that bug that means importing new repos is a manual task? I know there was some debugging work going on22:30
jeblairclarkb: but i figure we'd clean the repos up while it's down22:31
anteayadhellmann: so far we have logging22:31
jaypipesclarkb: I hate to bother you again, but I'm really struggling to get stuff to work in my testing platform. Do you have some time to do a hangout with me?22:31
jeblairclarkb: since we're about to get busy22:31
anteayaand the script sometimes works automatically22:31
dhellmannanteaya: ok, thanks22:31
anteayaI have it on my list of things to do to stand up a mock gerrit and test the manage-projects script22:31
jeblairclarkb: does that work for you?22:31
anteayabut so far have been distracted by expense reports, which are now done22:32
* anteaya hopes mordred has checked his inbox22:32
jeblairdhellmann: do you have openstack projects to import?  we'll just do those; don't let the lack of automation slow you down22:32
anteayadhellmann: so my obstacles are decreasing22:32
dhellmannoslo has 2 libraries we're working on setting up, so if we could schedule some time to take care of those, maybe early next week, it would help22:32
dhellmannjeblair: sure, I'm trying to be careful with your time, too22:32
jeblairdhellmann: are they stackforge renames?22:32
dhellmannjeblair: there are changes up for oslo.vmware and oslo.test22:32
dhellmannjeblair: no, brand new ones22:32
dhellmannwell, importing from github22:33
openstackgerritBoris Pavlovic proposed a change to openstack/requirements: Adding Mako>=0.4.0 to global requirements  https://review.openstack.org/7302522:33
dhellmannjeblair: https://review.openstack.org/#/c/70761/ and https://review.openstack.org/#/c/73036/22:33
jeblairdhellmann: ok.  scheduling can be pretty loose then, no downtime required, you'll just want to know when the switch actually happens22:33
dhellmannjeblair: yes, we're not committing to those upstream repos, just providing them as the source for importing stuff22:34
jeblairclarkb: i'm taking your silence as assent.  :)22:34
*** jnoller_ has quit IRC22:34
jeblair#static alert restarting zuul for a configuration change22:34
jeblair#status alert restarting zuul for a configuration change22:34
openstackstatusNOTICE: restarting zuul for a configuration change22:34
*** ChanServ changes topic to "restarting zuul for a configuration change"22:35
*** tru_tru has quit IRC22:35
*** dstanek has joined #openstack-infra22:36
fungijeblair: sorry, popping back in while stuff heats up. sounds good to me22:36
jeblairfungi: cool, in progress22:36
jeblairit's cloning now22:36
fungiwe blew away old zuul logs the last time we rebuilt the server too (well, i think we haven't deleted the old server yet, but anyway...)22:37
jaypipesWARNING zuul.Scheduler: Project openstack-dev/sandbox not found <-- how does one track down why that would be...22:38
jeblairjaypipes: it means the project isn't defined in the layout22:39
openstackgerritBoris Pavlovic proposed a change to openstack/requirements: Adding Mako>=0.4.0 to global requirements  https://review.openstack.org/7302522:39
jaypipesjeblair: *sigh*... debugged that yesterday, forgot to change my source repo to put projects: instead of - projects: in the layout.yaml.  Ty sir.22:40
* vishy votes for hemna22:42
*** dstanek has quit IRC22:42
zaropleia2: how do you determine which python version is on which ubuntu release?22:42
pleia2zaro: http://packages.ubuntu.com/search?keywords=python22:43
pleia2that will show you defaults22:43
anteayavishy: should hemna be on requirements-core?22:43
jeblair#status ok22:43
*** ChanServ changes topic to "Discussion of OpenStack Project Infrastructure | Docs http://ci.openstack.org/ | Bugs https://launchpad.net/openstack-ci | Code https://git.openstack.org/cgit/openstack-infra/"22:43
zaropleia2: thanks.22:43
pleia2zaro: can search for python3 to see what python3 is has: http://packages.ubuntu.com/search?keywords=python322:43
anteayavishy: was that your meaning?22:43
pleia2(you can install python3 instead of just python)22:44
vishyanteaya: no he asked for a +3 and fungi said I weilded the transcendant +3 vote22:44
anteayaah, sorry I mis-understood22:44
vishy(fairly far back in scrollback) :)22:44
zaropleia2: i'm using precise vm, can i just upgrade to 2.7.5?22:44
pleia2zaro: there might be a ppa, but the easy answer is "not really"22:44
jaypipesok, now back to the job not registered with Gearman problem...22:44
anteayasince you wield the +3, can you also update the requirements-core list?22:45
anteayayes I remember fungi saying that, was getting my wires crossed22:45
hemna:)22:45
anteayahemna: I'm trying buddy22:45
hemnaheh thanks for that :)22:45
anteayavishy: since at least 3 members or req-core appear stale to my eyes22:46
anteayas/or/of22:46
hemnafwiw, I'll have a follow up on that patch to update the lower number in that requirements.22:46
hemnathe current one was just updating the upper number to make the mirror happy22:47
harlowja_hi guys, do u have a sec to help the taskflow figure out a requirement that it wants to use but which isn't on the pypi openstack mirror22:47
harlowja_https://bugs.launchpad.net/openstack-ci/+bug/128000822:47
harlowja_is there a way to selectively use the global pypi, and only for certain cases use the openstack pypi/requirements22:47
openstackgerritIvan Melnikov proposed a change to openstack/requirements: Add pyMySQL to global requirements  https://review.openstack.org/7343722:47
harlowja_instead of it being all or nothing22:47
harlowja_the all or nothing approach, makes it hard for taskflow to be compatiable with dependencies that do exist in the public realm, but which are not direct openstack requirements22:48
harlowja_*since certain pieces of openstack are not that compatible yet22:48
jeblairharlowja_: so the purpose of the openstack/requirements project is to ensure that all parts of openstack are using a compatible set of requirements, and that change to that go through review22:49
dhellmannjeblair: in this case, we want to test with those requirements, and with newer versions of the lib -- sqlalchemy in particular22:49
dhellmannso we want *both* sets of tests22:49
dhellmannif that's not readily possible, we can live with it, but if there's a way to make it work that would be nice22:50
jeblairdhellmann: i do not believe that is possible with the current design22:51
*** geekinutah has left #openstack-infra22:51
dhellmannjeblair: that's what I thought22:51
*** iv_m has joined #openstack-infra22:51
harlowja_jeblair i can understand setting a minimum set of requirements, but also bounding the upper limit seems odd :-/22:52
*** med_ has quit IRC22:52
*** med_ has joined #openstack-infra22:52
harlowja_since in taskflows case at least it is compatible with 0.7,0.8 and 0.922:52
*** tru_tru has joined #openstack-infra22:52
dhellmannharlowja_: we've had far far too many cases of libraries outside our control releasing breaking versions22:52
jeblairdhellmann: a change to the openstack/requirements repo will run jobs with new versions, so you can test whether a new version breaks by proposing such a change22:52
clarkbjog0: kibana has been restored22:52
dhellmannjeblair: we want to test taskflow against many versions of sqlalchemy in different jobs22:52
anteayaclarkb: \o/22:53
dhellmannjeblair: we were doing this when it was on stackforge, but there it was not restricted to the openstack requirements list22:53
harlowja_jeblair another option is to possibly have a way to mark jobs to use global, or openstack pypi22:53
jeblairdhellmann: i understand; just making sure you knew what capabilities were there.22:53
dhellmannjeblair: sure, it's just not quite the same thing22:53
ryanpetrelloso, infra review/zuul question22:54
*** e0ne has joined #openstack-infra22:54
jaypipesanybody know how I can debug why my jenkins jobs aren't being registered with gearman? there's no information in jenkins logs and nothing in zuul logs other than "job not regsistered" messages. or perhaps I should be asking *how* jobs get registered with gearman? on jenkins start? on zuul start?22:54
*** beagles has quit IRC22:54
ryanpetrelloif I open a review, and then issue a ps2 ~3 minutes later22:54
ryanpetrellois zuul smart enough to cancel the first patchset?22:54
jeblairharlowja_: this system is focused on making sure that there is a set of requirements that all of openstack can work with.  it's really designed to make openstack work, not to make projects work outside of openstack.22:54
StevenKryanpetrello: Yes.22:55
ryanpetrello\o/22:55
jeblairjaypipes: did you enable the gearman plugin?22:55
dhellmannjeblair: yeah, we can work with that, we just wanted to see if there was a way around22:55
harlowja_jeblair with more oslo libraries does that start to make sense anymore :(22:55
jaypipesjeblair: yep, it's enabled and running fine, according to jenkins logs.22:55
*** e0ne has quit IRC22:55
*** e0ne has joined #openstack-infra22:56
dhellmannharlowja_: we can address that in the next cycle. I think moving to libraries instead of the incubator is going to make managing those requirements easier across the board22:56
harlowja_dhellmann sure22:56
*** thedodd has quit IRC22:56
jeblairharlowja_: most of this comes from the fact that pip doesn't have a true dependency resolver, so there isn't a good way to say "this project can have different minimum/maximum versions than /requirements" in a reasonable way22:56
jeblairharlowja_: dstufft will fix all that though.  :)22:57
harlowja_jeblair sure, understandable, but could there be a way to allow certain zuul jobs to use/not use the global-requirements?22:57
harlowja_like for py26, py27, py3, use the restricted requirements, for anything else, dont22:57
harlowja_a flag possibly in yaml22:58
harlowja_?22:58
jeblairharlowja_: not easily.22:58
harlowja_hmm22:58
harlowja_durn22:58
*** esker has joined #openstack-infra22:59
jeblairharlowja_: i think the changes would be to the select-mirror script; but note that you would also need to have a different requirements.txt for the different tox envs...22:59
jeblairharlowja_: and then, how do you publish an artifact that works with the other versions?22:59
jeblairharlowja_: i think that's the real problem.  testing it is tractable, but what do you do with the results?22:59
harlowja_keep as much compatiblity as we can23:00
jeblairthe thing published to pypi needs to use the openstack requirements23:00
harlowja_hmmm23:00
*** ryanpetrello has quit IRC23:00
harlowja_interesting, but if it works with a superset of the openstack requirements, then it should still work with openstack right?23:01
*** e0ne has quit IRC23:01
jeblairharlowja_: you're back at "pip doesn't have a dependency resolver".23:01
harlowja_durn :-P23:01
jaypipesjeblair: does the registration happen when zuul starts?23:02
jeblairhonestly, if that gets fixed, we can do a lot more interesting things.  dstufft really is working on that23:02
harlowja_jeblair k, thx, guess we might have to adjust our jobs then for now23:02
jeblairjaypipes: and on certain other events23:02
fungi'pip install a b c' does not necessarily get you the same versions of packages installed as 'pip install a c b' and you also have to keep in mind that it's *very* possible for pip to install versions of packages which don't actually satisfy later declarations of required versions23:02
fungiso it's all extremely fiddly23:03
jaypipesjeblair: is there a way to manually trigger the event of registering a job? or some other way to try to debug the process of registration?23:03
*** amcrn_ has joined #openstack-infra23:03
jeblairjaypipes: start jenkins, change a job config, change a node config23:03
jeblairjaypipes: did you enable the gearman plugin?23:03
dstufftfungi: to be fair, we can atleast promise that pip abc will give you the same thing as pip a b c :V23:03
dstuffter23:03
dstufftpip a b c and pip a b c23:03
dstufftjoke ruined by bad typing :(23:04
jaypipesjeblair: yes, the gearman plugin is definitely enabled.23:04
jeblairjaypipes: does your jenkins show up in the status command to the gearman server?23:04
fungidstufft: repeatability is certainly a virtue23:04
fungidstufft: as long as those two install commands are soon enough together than stuff on pypi doesn't change ;)23:04
*** amcrn has quit IRC23:05
jaypipesjeblair: all I get from the status command is this:23:05
jaypipeszuul:promote00123:05
jaypipeszuul:enqueue00123:05
jaypipes.23:05
dstufftactually techincally even if you have the exact same packages we can't guarentee that :(23:05
jeblairjaypipes: jenkins isn't connected to gearman.23:05
jeblairjaypipes: oh, sorry, i meant 'workers'.23:05
dstufftbecuase insall_requires=random.sample(["a", "b", c"], 2)23:05
fungidstufft: ha!23:06
*** ryanpetrello has joined #openstack-infra23:06
*** maurosr has quit IRC23:06
*** gyee has joined #openstack-infra23:07
fungime conceives a chaos package that declares completely random pip requirements from pypi on every run23:07
*** hdd_ has quit IRC23:07
dstufftfungi: I tried to register requirements.txt becuase I was going to do basically that :323:08
funginice23:08
dstufftor have it depend on everything on PyPI23:08
dstufftbut jannis beat me too it :D23:08
fungigreat for when someone forgets a -r23:09
jaypipesjeblair: so in the jenkins UI, under manage plugins -> enabled, gearman plugin is showing up. I don't see any way to configure the plugin (and it was installed with the openstack-infra/config jenkins::plugin manifest in the same way as upstream jenkins masters. Sorry for asking so many questions. I'm just not entirely sure where to look for the connection between jenkins and gearman.23:09
*** sarob has joined #openstack-infra23:09
fungijaypipes: go to configure jenkins and it will have a section there23:09
fungijaypipes: when the plugin is enabled, it is configured in the main jenkins configuration screen. that's typical for jenkins. silly, but typical23:10
*** mrodden has quit IRC23:10
jeblairjaypipes: it defaults to disabled.  it needs to be configured to talk to the zuul server and it needs to be enabled in the screen fungi mentioned.  there's a connection test there too.23:10
*** mrodden1 has joined #openstack-infra23:10
jaypipesfungi: OK, that was really weird. Yeah, the plugin was enabled, but in the Configure System -> Gearman Plugin Config , the Enable Gearman box was unchecked :(23:10
fungithe plugin and gearman enabling are separate, so you can disable communication without unconfiguring23:11
jaypipesI see..23:11
fungiat least i assume that was the reason23:12
jaypipesfungi: so do you guys manually check "Enable Gearman" then?23:12
jaypipesfungi: or is there some puppet thing I'm missing?23:12
*** ryanpetrello has quit IRC23:12
fungijaypipes: we manually...lots of things for jenkins23:12
*** e0ne has joined #openstack-infra23:12
fungijaypipes: because doing otherwise mostly means guessing at xml23:12
jaypipesfungi: ok, thx23:12
fungijenkins-job-builder is a primary exception. it has raised guessing at jenkins xml to a high art23:13
jaypipesson of a... I restart Jenkins and it unchecks the damn Enable Gearman box again :(23:13
fungithat sounds wrong23:13
fungiwe definitely start jenkins fairly often without it disabling23:14
lifelessfungi: 7037623:14
*** e0ne has quit IRC23:14
lifelessfungi: yes,we certainly do23:14
fungithough i think it may require a successful connection test to actually enable?23:14
jeblairjaypipes: did you click save?23:14
*** e0ne has joined #openstack-infra23:14
fungior that, right23:14
jeblairjaypipes, fungi: we can't configure that part of jenkins with puppet.  that is one of the many reasons we're getting rid of jenkins.23:15
jeblairjaypipes: and the reason it doesn't default to enabled is because there is no sensible default for the host it should connect to.23:15
*** e0ne has quit IRC23:15
jeblairjaypipes: so defaulting it to enabled would mean defaulting to on-and-broken.23:15
fungilifeless: restoring context...failed23:15
*** e0ne has joined #openstack-infra23:16
jeblairfungi: so the first vrfy+1 i examined in the log does have the correct data in the query which is returned 0.185s after zuul got the event.23:17
lifelessfungi: review 7037623:17
jaypipesjeblair: lol. had no idea there was a save button below :( when I clicked Enable Gearman, it did something and said "Test successful!" so I figured it was saved :/ Anyway, saved now, and after restarting zuul, BOOM I have my jobs in gearman registered. w00t. on to the next obstacle. thx again you guys for your patience with me.23:17
fungijeblair: but it still did not get enqueued?23:17
lifelessharlowja_: ping23:17
lifelessharlowja_: space & confirmation etc23:18
jeblairfungi: it wasn't supposed to be; i wasn't looking for a gate one specifically23:18
harlowja_lifeless sup23:18
harlowja_lifeless u see seans email?23:18
fungijeblair: oh, okay23:18
jog0clarkb: awesome, btw what happend?23:19
jeblairjog0: we upgraded23:19
jaypipeshttps://review.openstack.org/#/c/72790/ <-- \o/ it all worked.23:19
jog0jeblair: ohh nice I thought so23:19
jaypipesnow... to quickly write this down before I forget :)23:19
anteayajaypipes: I am looking forward to reading your notes23:21
*** e0ne has quit IRC23:21
jeblairsdague: had a chance to look at https://review.openstack.org/#/c/73418/ ?23:21
*** dims has quit IRC23:22
sdaguejeblair: just did23:22
sdaguethat looks fine to me. Honestly, I'm not 100% sure I parse the matrix correctly in my head. This is one of those places where my zuul grokking remains low23:22
sdagueconceptually, I think it's the right behavior23:23
clarkbjeblair: you use xfce right? are you using the ssh-agent that it starts automagically for you? I was doing that with xfce on precise and it worked iwth ssh-add -c no problem. xfce + ssh agent on saucy xfce will accept the -c option but when I try to load the key the agent socket is removed23:23
clarkbI wonder if I need to set SSH_ASKPASS23:24
lifelessharlowja_: ah, yes I have23:24
jeblairsdague: it's new to me too.  :)23:24
lifelesscody-somerville: you were copied on it23:24
lifelesscody-somerville: about 30m ago23:24
harlowja_k23:25
jeblairclarkb: yeah i'm using the default agent: /usr/bin/ssh-agent /usr/bin/dbus-launch --exit-with-session startxfce423:26
jeblairclarkb: but this was never a gnome machine, so it doesn't have the gnome agent installed; if it did, i think it might use it instead23:26
jeblairclarkb: also, it's precise.23:27
*** oubiwann_ has quit IRC23:27
clarkbjeblair: yeah that is what I have on my precise box23:27
clarkband it works. on saucy it is just ssh-agent without the extra stuff23:27
clarkbI guess I can try it with those options if I can find the magical place ssh-agent is invoked by xfce23:27
jeblairfungi: i'm testing with this change: https://review.openstack.org/#/c/69800/23:27
fungik23:28
openstackgerritCedric Brandily proposed a change to openstack-infra/git-review: Add http(s) protocol support to set_hooks_commit_msg  https://review.openstack.org/7074623:28
openstackgerritCedric Brandily proposed a change to openstack-infra/git-review: Retrieve remote pushurl independently of user's locale  https://review.openstack.org/6430723:28
fungiclarkb: i just use keychain and invoke it in my ~/.xinitrc23:29
fungibut maybe i'm weird23:29
jeblairfungi: oh, i think nodepool's connection to zuul's gearman broke.23:29
fungioh, though i use startx via an alias and have the local command-line agent fir for that, so i get prompted for that in the vtty23:30
jeblairrestarted nodepool23:30
fungijeblair: you mean when zuul restarted?23:30
jeblairfungi: probably; i think there's still a gear bug that could cause that23:30
mordredclarkb: let me know if you find how to do that - I would like to change the options too23:30
fungiclarkb: anyway, caching your key prior to starting your x session isn't hard as long as you don't use a login manager23:31
jeblair2014-02-13 23:31:34,258 DEBUG nodepool.NodePool:   Deficit: devstack-precise: 97 (start: 123 min: 123 ready: 26)23:31
jeblairthat's more like it23:31
*** sarob has quit IRC23:31
*** sarob has joined #openstack-infra23:32
clarkbfungi: yeah there are alternatives, but the reason I bother withxfce is so that it does this stuff for me :)23:35
*** dims_ has joined #openstack-infra23:35
fungixfce uses xdm?23:36
clarkbmordred: figured it out, need to install ssh-askpass23:36
fungioh, yeah you definitely need that regardless23:37
StevenKHuh23:37
StevenKI have a working agent setup without ssh-askpass23:37
clarkbStevenK: we use ssh-add -c23:38
clarkbit works fine without the -c23:38
StevenKAh23:38
*** gokrokve_ has quit IRC23:39
*** gokrokve has joined #openstack-infra23:39
clarkbfungi: I think on my old box the gnome ask pass was installed because unity, but this has no unity therefore needed to isntall23:39
mordredclarkb: wait- so where are you doing your ssh-add -c ?23:39
clarkbmordred: I just do that by hand when I start a new session23:40
mordredoh. weird23:40
mordredmine happens automatically. I guess I'll investigate23:40
jeblairclarkb: i have a startup task that runs ssh-add -c23:41
fungiclarkb: oh, you don't use keychain for that?23:41
jeblairmordred: if it's happening automatically, it's probably not doing "-c".23:42
mordredjeblair: right. that's what I'm investigating right now23:42
clarkbfungi: no, it is just starting ssh-agent which is all I really want23:42
clarkbthen I can manually add keys as appropriate23:42
fungigot it23:42
openstackgerritJoshua Harlow proposed a change to openstack-infra/config: Remove taskflow sqlalchemy 0.9 testing  https://review.openstack.org/7344523:43
zaroclarkb: questions about available packages on jenkins slaves23:43
clarkbzaro: sure, there is a list in the slave.pp manifset23:43
zaroclarkb: i don't see git defined in pp files so is it available?23:43
clarkbgit is available, it gets in there as part of the image install iirc23:43
clarkband probably as part of a lower level puppet manifest23:44
boris-42_somebody could pls take a look at this patch https://review.openstack.org/#/c/73025/  ?)23:44
zaroclarkb: how would i know that?23:44
*** gokrokve has quit IRC23:44
fungiclarkb: well, early on, our install shell scripts install git so that they can get a copy of the puppet manifest23:45
fungiso we install git before we apply puppetry23:45
clarkbright23:45
zaroclarkb: so i spend most of my day troubleshooting gerrit build and it turned out that the problem was that zip package was not on the vm i was using.23:45
clarkbzaro: see the install puppet script23:45
clarkbzaro: what are we using zip files for?23:45
zaroclarkb: not me it's the buck buildding gerrit23:45
*** dhellmann is now known as dhellmann_23:45
clarkbah23:46
fungioh, and they don't assume you might build buck on a system with no windows-centric zip tool? funny23:46
zarofungi: they are all mac lovers23:46
fungii guess it helps that java jarfiles are actually zip with a different extension23:46
*** rcleere has quit IRC23:46
fungiso java devs most likely assume you install zip everywhere, since it's baked into their toolchain anyway23:47
lifelessfungi: s/assume/know that/23:47
fungiheh23:47
zarozip installs unzip, but not the other way around.23:47
zaroshould both be explicitly stated in pp file? or just zip since it install unzip.23:48
lifelessbe explicit IMO23:48
lifelesszip installing unzip is as weird as unzip not installing zip23:48
lifelessso I wouldn't bet on it never changing23:48
fungiyeah, there's no guarantee that in $future it might change23:48
fungiwhat lifeless said23:49
zaroso why not be explicit with git too?23:49
mordredare we not?23:49
zaroi don't see it in slave.pp23:49
zaroor param.pp23:49
fungizaro: if we don't have it in our puppet manifest somewhere there or at a higher level then i would consider that a latent bug23:50
sdaguehonestly, jar is a better zip than zip23:50
mordredoh - I know why23:50
sdagueas it takes the same flags as tar, and works on archives over 4G23:50
mordredit's because our "bootstrap this node" script installs it I think23:50
*** bhuvan has quit IRC23:50
fungimordred: yep, we already discussed that. we should still be explicit and have puppet insist on it anyway23:51
mordrednope. wrong23:51
mordredit's in base.pp23:51
mordreds/wrong/me was wrong/23:51
fungiyeah, so sounds like we already do, in which case no bug23:51
*** denis_makogon has quit IRC23:52
*** bhuvan has joined #openstack-infra23:52
*** gokrokve has joined #openstack-infra23:54
zarook, i see it now.  would it be safe to assume that the slave will also have gcc?23:54
fungizaro: the build-essential package depends on the gnu libc toolchain among other things23:55
zarofungi: ohh yeah, i found that.  thanks.23:56
fungiexplicitly listing all packages brought in by build-essential might count as overly pedantic23:56
*** derekh_afk is now known as derekh23:56
jeblairfungi: same behavior observed with https://review.openstack.org/#/c/69800/23:56
jeblairfungi: it looks like it has the correct data.  i'll try to reproduce locally23:56
*** sarob_ has joined #openstack-infra23:56
jog0fungi: I want to follow up in bug 127038223:57
fungijeblair: i see that. looking at the debug log23:57
*** andrew_plunk has quit IRC23:57
jog0or jeblair ^, its marked as incomplete, is there any other data you want etc23:57
jog0as we e-r thinks thats thh top gate bug23:57
dstufftsdague: zip64 $$23:58
fungijog0: so it's still happening?23:58
*** jgrimm has quit IRC23:58
jeblairfungi: i wonder if that's related to that ticket you mentioned23:58
sdaguedstufft: but then I have to remember zip flags :)23:58
sdaguejar xvf foo.zip :)23:58
*** bhuvan has quit IRC23:59
jeblairjog0: i think based on a ticket rackspace openened on our account, it looks like their resolvers may be rate limiting us23:59
fungijeblair: it's possible, but they only ever opened one ticket about one slave, and it was long after that bug got reported23:59
mtreinishsdague: jar or tar? :)23:59

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