*** unicell1 is now known as unicell | 00:00 | |
*** unicell has joined #openstack-infra | 00:00 | |
*** tsg has joined #openstack-infra | 00:00 | |
*** oomichi has joined #openstack-infra | 00:00 | |
openstackgerrit | A change was merged to openstack-infra/config: Check and gate PEP8 for stackforge/gerrit-dash-creator https://review.openstack.org/112767 | 00:01 |
---|---|---|
*** signed8bit_ZZZzz is now known as signed8bit | 00:05 | |
*** ZZelle_ has quit IRC | 00:06 | |
*** ramashri has quit IRC | 00:08 | |
*** Alexandra_ has joined #openstack-infra | 00:11 | |
*** asettle has quit IRC | 00:12 | |
*** Alexandra_ is now known as a | 00:13 | |
*** a is now known as asettle | 00:13 | |
jogo | clarkb: ping | 00:13 |
*** Ish__ has quit IRC | 00:14 | |
jeblair | jogo: he mentioned he would be mostly afk today, back tomorrow | 00:14 |
jogo | oh, I will fl a bug then | 00:14 |
jogo | file* | 00:14 |
jogo | looks like we are missing some !console logs in logstash | 00:14 |
jeblair | jogo: feel free to flog one too | 00:15 |
jeblair | jogo: tripleo or otherwise? (i saw a change in review for tripleo i think) | 00:15 |
jogo | otherwise | 00:16 |
jogo | jeblair: still digging in | 00:16 |
*** Ish__ has joined #openstack-infra | 00:16 | |
openstackgerrit | A change was merged to openstack-infra/config: Add python33 job for python-rallyclient https://review.openstack.org/112814 | 00:16 |
*** yamahata has joined #openstack-infra | 00:18 | |
*** otherwiseguy has joined #openstack-infra | 00:22 | |
jogo | here is the bug for clarkb to look at https://bugs.launchpad.net/openstack-ci/+bug/1358550 | 00:24 |
uvirtbot | Launchpad bug 1358550 in openstack-ci "some files are not getting added to logstash.openstack.org" [Undecided,New] | 00:24 |
*** signed8bit has quit IRC | 00:26 | |
*** aysyd has quit IRC | 00:26 | |
*** signed8bit has joined #openstack-infra | 00:29 | |
*** harlowja is now known as harlowja_away | 00:29 | |
*** homeless has quit IRC | 00:31 | |
*** homeless has joined #openstack-infra | 00:32 | |
*** _buzz_ has quit IRC | 00:34 | |
*** Ish__ has quit IRC | 00:35 | |
*** marcoemorais has quit IRC | 00:35 | |
*** _buzz has joined #openstack-infra | 00:35 | |
*** rwsu has quit IRC | 00:37 | |
*** signed8bit has quit IRC | 00:37 | |
openstackgerrit | James E. Blair proposed a change to stackforge/gertty: Fix immediate sync of change by change ID https://review.openstack.org/115156 | 00:38 |
*** crc32 has quit IRC | 00:40 | |
openstackgerrit | A change was merged to openstack-dev/pbr: Teach pbr about post versioned dev versions. https://review.openstack.org/80449 | 00:44 |
*** homeless has quit IRC | 00:44 | |
*** armax has quit IRC | 00:50 | |
*** rwsu has joined #openstack-infra | 00:50 | |
*** terryw has joined #openstack-infra | 00:51 | |
*** Ish__ has joined #openstack-infra | 00:52 | |
*** ramashri has joined #openstack-infra | 00:52 | |
*** otherwiseguy has quit IRC | 00:54 | |
*** andreaf_ has joined #openstack-infra | 00:54 | |
*** dims has joined #openstack-infra | 00:54 | |
*** marcoemorais has joined #openstack-infra | 00:55 | |
*** harlowja_away is now known as harlowja | 00:56 | |
*** andreaf has quit IRC | 00:56 | |
*** otherwiseguy has joined #openstack-infra | 00:59 | |
*** ramashri has quit IRC | 01:01 | |
*** SumitNaiksatam has joined #openstack-infra | 01:01 | |
*** ramashri has joined #openstack-infra | 01:01 | |
*** terryw has quit IRC | 01:02 | |
openstackgerrit | A change was merged to openstack-infra/config: Enable profiling in rally-jobs https://review.openstack.org/112816 | 01:02 |
openstackgerrit | A change was merged to openstack-infra/config: Change selenium and jshint to voting for tuskar-ui https://review.openstack.org/113510 | 01:03 |
*** rick__ has joined #openstack-infra | 01:04 | |
*** bhuvan has quit IRC | 01:05 | |
*** Ish__ has quit IRC | 01:06 | |
*** praneshp has quit IRC | 01:09 | |
*** gokrokve has joined #openstack-infra | 01:10 | |
openstackgerrit | A change was merged to openstack-infra/config: Make Ceilometer MongoDB jobs check instead of experimental https://review.openstack.org/114208 | 01:12 |
*** mmaglana_ has quit IRC | 01:17 | |
*** gokrokve has quit IRC | 01:17 | |
*** gokrokve has joined #openstack-infra | 01:18 | |
*** gokrokve has quit IRC | 01:22 | |
*** bknudson has quit IRC | 01:23 | |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/config: Add in job to run pip missing requirements https://review.openstack.org/114137 | 01:24 |
*** mbacchi has quit IRC | 01:25 | |
*** gokrokve has joined #openstack-infra | 01:25 | |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/config: Send python-job logs to swift https://review.openstack.org/109485 | 01:28 |
*** yaguang has joined #openstack-infra | 01:28 | |
*** yaguang has quit IRC | 01:29 | |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/config: Add in job to run pip missing requirements https://review.openstack.org/114137 | 01:30 |
*** yaguang has joined #openstack-infra | 01:32 | |
*** hdd has joined #openstack-infra | 01:32 | |
*** trinathS has quit IRC | 01:32 | |
*** signed8bit has joined #openstack-infra | 01:32 | |
*** changbl has joined #openstack-infra | 01:36 | |
*** hdd has quit IRC | 01:39 | |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/jenkins-job-builder: Start libification of jjb https://review.openstack.org/107724 | 01:42 |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/config: Send python-job logs to swift https://review.openstack.org/109485 | 01:43 |
*** otherwiseguy has quit IRC | 01:44 | |
*** Ish__ has joined #openstack-infra | 01:47 | |
*** nosnos has joined #openstack-infra | 01:47 | |
*** Ish__ has quit IRC | 01:47 | |
*** changbl has quit IRC | 01:48 | |
*** signed8bit has quit IRC | 01:48 | |
*** oomichi has quit IRC | 01:48 | |
*** tsg has quit IRC | 01:48 | |
*** jamespag` has quit IRC | 01:48 | |
*** che-arne|2 has quit IRC | 01:48 | |
*** dmsimard_away has quit IRC | 01:48 | |
*** stevemar has joined #openstack-infra | 01:52 | |
*** gokrokve_ has joined #openstack-infra | 01:56 | |
*** xchu has joined #openstack-infra | 01:56 | |
*** changbl has joined #openstack-infra | 01:57 | |
*** signed8bit has joined #openstack-infra | 01:57 | |
*** oomichi has joined #openstack-infra | 01:57 | |
*** tsg has joined #openstack-infra | 01:57 | |
*** jamespag` has joined #openstack-infra | 01:57 | |
*** che-arne|2 has joined #openstack-infra | 01:57 | |
*** dmsimard_away has joined #openstack-infra | 01:57 | |
*** gokrokve_ has quit IRC | 01:58 | |
*** Daisy has joined #openstack-infra | 01:58 | |
*** ChuckC has joined #openstack-infra | 01:58 | |
*** gokrokve_ has joined #openstack-infra | 01:59 | |
*** gokrokve has quit IRC | 01:59 | |
*** otherwiseguy has joined #openstack-infra | 01:59 | |
*** arnaud__ has quit IRC | 02:01 | |
*** gokrokve has joined #openstack-infra | 02:08 | |
*** alexpilotti has quit IRC | 02:09 | |
*** gokrokve_ has quit IRC | 02:09 | |
*** trinathS has joined #openstack-infra | 02:10 | |
*** RicoLin has joined #openstack-infra | 02:10 | |
*** mwagner_lap has joined #openstack-infra | 02:10 | |
*** directxman12 has joined #openstack-infra | 02:12 | |
*** morganfainberg is now known as morganfainberg_Z | 02:14 | |
*** Ryan_Lane has quit IRC | 02:15 | |
*** hdd has joined #openstack-infra | 02:25 | |
*** Krast_ has quit IRC | 02:26 | |
*** Krast has joined #openstack-infra | 02:26 | |
*** otter768 has joined #openstack-infra | 02:29 | |
*** marcoemorais has quit IRC | 02:31 | |
*** hdd has quit IRC | 02:32 | |
*** gokrokve has quit IRC | 02:34 | |
*** annegentle has joined #openstack-infra | 02:34 | |
*** gokrokve has joined #openstack-infra | 02:34 | |
*** arnaud has quit IRC | 02:35 | |
*** dims has quit IRC | 02:35 | |
*** amotoki_ has quit IRC | 02:37 | |
*** gokrokve has quit IRC | 02:39 | |
*** trinathS has quit IRC | 02:40 | |
*** otherwiseguy has quit IRC | 02:41 | |
*** arnaud has joined #openstack-infra | 02:41 | |
*** dims has joined #openstack-infra | 02:42 | |
*** dims has quit IRC | 02:47 | |
*** daya_k has joined #openstack-infra | 02:48 | |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: Allow for grenade upgrades within a release https://review.openstack.org/111094 | 02:49 |
*** ramashri has quit IRC | 02:53 | |
*** ramashri has joined #openstack-infra | 02:55 | |
*** adalbas has quit IRC | 03:01 | |
*** _buzz has quit IRC | 03:02 | |
*** bdpayne has joined #openstack-infra | 03:03 | |
*** _buzz has joined #openstack-infra | 03:03 | |
*** dims has joined #openstack-infra | 03:12 | |
*** sdake_ has joined #openstack-infra | 03:13 | |
mikal | So... tempest is failing for a nova change, and logs that there are non-whitelisted errors in the logs | 03:14 |
mikal | Is that what's causing the failure? | 03:14 |
clarkb | mikal: it could be. If tempest said X tests ran passed but the job still fails then the non whitelisted errors would be at fault | 03:15 |
mikal | Yah, it says all the tests passed | 03:15 |
mikal | So... Ceilometer hates me then | 03:15 |
mikal | https://review.openstack.org/#/c/115158/1 | 03:15 |
mikal | Doh, not that one | 03:15 |
mikal | http://logs.openstack.org/07/114007/4/check/check-tempest-dsvm-full/1f74423/console.html | 03:15 |
mikal | That one | 03:15 |
mikal | Does tempest only fail on logs related to the thing under test? | 03:16 |
mikal | Or any non-whitelisted error on an integrated project? | 03:16 |
*** KanagarajM has joined #openstack-infra | 03:17 | |
*** dims has quit IRC | 03:17 | |
*** bdpayne has quit IRC | 03:17 | |
*** AlexF has joined #openstack-infra | 03:18 | |
clarkb | any non whitelisted error | 03:19 |
mikal | Hmmm | 03:20 |
*** markmcclain has quit IRC | 03:24 | |
*** Sukhdev has joined #openstack-infra | 03:27 | |
*** bdpayne has joined #openstack-infra | 03:30 | |
*** gyee_ has quit IRC | 03:32 | |
*** amotoki has joined #openstack-infra | 03:33 | |
openstackgerrit | James E. Blair proposed a change to stackforge/gertty: Add unified diff view https://review.openstack.org/115166 | 03:37 |
jeblair | fungi: ^ let me know what you think | 03:37 |
*** bdpayne has quit IRC | 03:41 | |
*** Azher has joined #openstack-infra | 03:42 | |
*** SumitNaiksatam has quit IRC | 03:47 | |
*** tsg has quit IRC | 03:47 | |
*** SumitNaiksatam has joined #openstack-infra | 03:48 | |
*** annegentle has quit IRC | 03:51 | |
*** shashankhegde has joined #openstack-infra | 03:53 | |
*** hogepodge has quit IRC | 03:53 | |
*** lttrl has quit IRC | 03:55 | |
*** sdake_ has quit IRC | 03:57 | |
*** Ish__ has joined #openstack-infra | 03:58 | |
*** imeerovi has quit IRC | 04:00 | |
*** hdd has joined #openstack-infra | 04:00 | |
*** stevemar has quit IRC | 04:00 | |
*** xianghuihui has quit IRC | 04:03 | |
*** xianghuihui has joined #openstack-infra | 04:04 | |
*** AlexF has quit IRC | 04:07 | |
*** harlowja is now known as harlowja_away | 04:08 | |
*** garyh has quit IRC | 04:09 | |
*** asettle has quit IRC | 04:10 | |
*** alexandra has joined #openstack-infra | 04:10 | |
*** alexandra is now known as asettle | 04:10 | |
*** Ish__ has quit IRC | 04:11 | |
*** otherwiseguy has joined #openstack-infra | 04:13 | |
*** shashankhegde has quit IRC | 04:16 | |
*** xianghuihui has quit IRC | 04:18 | |
*** xianghui has joined #openstack-infra | 04:21 | |
*** otherwiseguy has quit IRC | 04:24 | |
*** Daisy has quit IRC | 04:26 | |
*** arosen1 has joined #openstack-infra | 04:27 | |
*** flaviof_zzz is now known as flaviof | 04:30 | |
*** otter768 has quit IRC | 04:32 | |
*** camunoz has joined #openstack-infra | 04:35 | |
openstackgerrit | Ian Wienand proposed a change to openstack-infra/nodepool: Clean up 'images' documentation https://review.openstack.org/115168 | 04:41 |
*** hdd has quit IRC | 04:43 | |
openstackgerrit | Kanagaraj Manickam proposed a change to openstack-infra/config: namos - Device and service manager https://review.openstack.org/109494 | 04:44 |
*** achanda has joined #openstack-infra | 04:51 | |
*** bradm has quit IRC | 04:52 | |
*** tsg has joined #openstack-infra | 04:52 | |
*** garyh has joined #openstack-infra | 04:52 | |
*** Ish__ has joined #openstack-infra | 04:53 | |
*** harlowja_at_home has joined #openstack-infra | 04:57 | |
*** hdd has joined #openstack-infra | 04:58 | |
*** harlowja_at_home has quit IRC | 04:59 | |
*** arnaud has quit IRC | 05:00 | |
*** dims has joined #openstack-infra | 05:01 | |
*** bdpayne has joined #openstack-infra | 05:02 | |
*** Ish__ has quit IRC | 05:02 | |
*** signed8bit has quit IRC | 05:03 | |
*** harlowja_at_home has joined #openstack-infra | 05:04 | |
*** hdd has quit IRC | 05:05 | |
*** dims has quit IRC | 05:06 | |
*** Ish__ has joined #openstack-infra | 05:06 | |
*** Krast has quit IRC | 05:08 | |
*** pcrews has quit IRC | 05:08 | |
*** ramashri has quit IRC | 05:09 | |
*** directxman12 has quit IRC | 05:15 | |
*** ctracey has quit IRC | 05:16 | |
*** chuckC_ has joined #openstack-infra | 05:16 | |
*** afazekas has quit IRC | 05:17 | |
*** ctracey has joined #openstack-infra | 05:20 | |
*** e0ne has joined #openstack-infra | 05:21 | |
*** loki184 has joined #openstack-infra | 05:21 | |
*** e0ne has quit IRC | 05:23 | |
*** hdd has joined #openstack-infra | 05:24 | |
*** Daisy has joined #openstack-infra | 05:25 | |
*** arnaud has joined #openstack-infra | 05:26 | |
*** Sukhdev has quit IRC | 05:27 | |
*** e0ne has joined #openstack-infra | 05:27 | |
*** e0ne has quit IRC | 05:30 | |
*** arnaud has quit IRC | 05:31 | |
*** simonmcc_ has quit IRC | 05:31 | |
*** chuckC_ has quit IRC | 05:32 | |
*** simonmcc__ has joined #openstack-infra | 05:34 | |
*** e0ne has joined #openstack-infra | 05:34 | |
*** shashankhegde has joined #openstack-infra | 05:34 | |
*** e0ne has quit IRC | 05:35 | |
*** k4n0 has joined #openstack-infra | 05:36 | |
*** Ish__ has left #openstack-infra | 05:36 | |
*** Alex_Gaynor has quit IRC | 05:37 | |
*** frank_zhou has joined #openstack-infra | 05:38 | |
*** e0ne has joined #openstack-infra | 05:40 | |
*** frank_zhou has quit IRC | 05:40 | |
*** willroberts has quit IRC | 05:40 | |
*** morganfainberg_Z is now known as morganfainberg | 05:41 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Add check-requirements to devstack-gate https://review.openstack.org/114139 | 05:41 |
*** Alex_Gaynor_ has joined #openstack-infra | 05:41 | |
*** reed has quit IRC | 05:42 | |
*** willroberts has joined #openstack-infra | 05:42 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Add new template stackforge-specs-jobs https://review.openstack.org/113211 | 05:44 |
*** hdd has quit IRC | 05:45 | |
*** e0ne has quit IRC | 05:45 | |
*** e0ne has joined #openstack-infra | 05:50 | |
*** rwsu has quit IRC | 05:51 | |
*** e0ne has quit IRC | 05:53 | |
*** Alex_Gaynor_ is now known as Alex_Gaynor | 05:55 | |
*** Alex_Gaynor has quit IRC | 05:55 | |
*** Alex_Gaynor has joined #openstack-infra | 05:55 | |
*** e0ne has joined #openstack-infra | 05:58 | |
*** ildikov has quit IRC | 05:58 | |
*** ildikov has joined #openstack-infra | 05:58 | |
*** e0ne has quit IRC | 06:01 | |
*** arnaud has joined #openstack-infra | 06:04 | |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/os-loganalyze: Add support for non-text files https://review.openstack.org/107267 | 06:04 |
*** ildikov has quit IRC | 06:04 | |
*** baohua has joined #openstack-infra | 06:07 | |
*** rwsu has joined #openstack-infra | 06:07 | |
*** bdpayne has quit IRC | 06:08 | |
*** achanda has quit IRC | 06:08 | |
*** mmaglana has joined #openstack-infra | 06:11 | |
*** shashankhegde has quit IRC | 06:13 | |
*** fabo has quit IRC | 06:13 | |
*** fabo has joined #openstack-infra | 06:13 | |
*** Krast has joined #openstack-infra | 06:14 | |
*** arnaud__ has joined #openstack-infra | 06:16 | |
*** bdpayne has joined #openstack-infra | 06:20 | |
openstackgerrit | Steve Kowalik proposed a change to openstack-infra/config: Add python3-jobs to diskimage-builder https://review.openstack.org/115183 | 06:20 |
*** harlowja_at_home has quit IRC | 06:23 | |
*** sdake has quit IRC | 06:26 | |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/zuul: Fix pep8 issues https://review.openstack.org/115187 | 06:27 |
*** Krast has quit IRC | 06:28 | |
*** baohua_ has joined #openstack-infra | 06:29 | |
*** mrda has quit IRC | 06:30 | |
*** baohua has quit IRC | 06:32 | |
*** mrda has joined #openstack-infra | 06:33 | |
*** rdopieralski has joined #openstack-infra | 06:35 | |
openstackgerrit | Radomir Dopieralski proposed a change to openstack-infra/config: Create a repository for additional Tuskar-UI plugins https://review.openstack.org/108102 | 06:40 |
*** arnaud has quit IRC | 06:40 | |
*** arnaud__ has quit IRC | 06:40 | |
*** maxbit has joined #openstack-infra | 06:44 | |
*** flaper87|afk is now known as flaper87 | 06:46 | |
*** lukego has joined #openstack-infra | 06:46 | |
*** bradm has joined #openstack-infra | 06:49 | |
*** e0ne has joined #openstack-infra | 06:50 | |
*** frank_zhou has joined #openstack-infra | 06:52 | |
*** rdopieralski has joined #openstack-infra | 06:53 | |
*** ildikov has joined #openstack-infra | 06:54 | |
*** MaxV has joined #openstack-infra | 06:54 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Add check-requirements to devstack-gate https://review.openstack.org/114139 | 06:55 |
*** ihrachyshka has joined #openstack-infra | 06:59 | |
*** denis_makogon has joined #openstack-infra | 06:59 | |
*** chandankumar has joined #openstack-infra | 07:04 | |
*** e0ne has quit IRC | 07:04 | |
*** dtantsur|afk is now known as dtantsur | 07:06 | |
*** camunoz is now known as camunoz_gone | 07:08 | |
*** e0ne has joined #openstack-infra | 07:09 | |
*** ihrachyshka has quit IRC | 07:10 | |
*** melwitt1 has quit IRC | 07:11 | |
*** MaxV has quit IRC | 07:11 | |
*** afazekas has joined #openstack-infra | 07:12 | |
*** mwagner_lap has quit IRC | 07:13 | |
*** hashar has joined #openstack-infra | 07:13 | |
*** ildikov_ has joined #openstack-infra | 07:14 | |
*** ildikov has quit IRC | 07:15 | |
*** bdpayne has quit IRC | 07:15 | |
*** jlibosva has joined #openstack-infra | 07:18 | |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/zuul: Add gerrit reviews into patchset approvals https://review.openstack.org/97390 | 07:21 |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/zuul: Allow a pipeline to specify alternative gerrit acc https://review.openstack.org/97391 | 07:21 |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/zuul: Add support for negative requirements https://review.openstack.org/102726 | 07:21 |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/os-loganalyze: Add support for non-text files https://review.openstack.org/107267 | 07:22 |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/config: Remove Jenkins EnvInject plugin from production https://review.openstack.org/101101 | 07:23 |
*** yolanda_ has joined #openstack-infra | 07:23 | |
*** yolanda has quit IRC | 07:24 | |
*** Longgeek has joined #openstack-infra | 07:26 | |
*** dizquierdo has joined #openstack-infra | 07:26 | |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/nodepool: Remove the need for zmq in proper demand calcs. https://review.openstack.org/98306 | 07:26 |
*** jcoufal has joined #openstack-infra | 07:26 | |
yolanda_ | hi, were you able to merge nodepool dib change? | 07:28 |
*** _nadya_ has joined #openstack-infra | 07:28 | |
AJaeger | flaper87, are you looking into zuul-specs as well? | 07:32 |
flaper87 | AJaeger: will do that in a bit, I just got back from vacations. Sorry :D | 07:33 |
AJaeger | flaper87, ah, you're catching up ;) Thanks! | 07:33 |
flaper87 | AJaeger: going through tons of emails right now :( | 07:33 |
flaper87 | yeah | 07:33 |
flaper87 | AJaeger: btw, thanks for following up on all those rename changes. I appreciate it | 07:34 |
AJaeger | flaper87, thanks to to SergeyLukjanov and jeblair as well. | 07:34 |
flaper87 | AJaeger: agreed. SergeyLukjanov and jeblair, thanks guys. :) | 07:34 |
AJaeger | zaqar-specs is the only repo AFAIK that needs some more handholding - I wrote you an email as well. So, go ahead and dig into your inbox ;) | 07:35 |
*** e0ne has quit IRC | 07:40 | |
*** pblaho has joined #openstack-infra | 07:42 | |
*** hashar has quit IRC | 07:42 | |
*** MaxV has joined #openstack-infra | 07:44 | |
*** jcoufal has quit IRC | 07:51 | |
*** che-arne|2 has quit IRC | 07:52 | |
*** fifieldt_ has joined #openstack-infra | 07:58 | |
*** arosen1 has quit IRC | 07:59 | |
*** Guest36489 has joined #openstack-infra | 07:59 | |
*** imeerovi has joined #openstack-infra | 07:59 | |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/zuul: Fix pep8 issues https://review.openstack.org/115187 | 08:01 |
*** jpich has joined #openstack-infra | 08:02 | |
*** jcoufal has joined #openstack-infra | 08:03 | |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/zuul: Fix pep8 issues https://review.openstack.org/115187 | 08:03 |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/zuul: Add gerrit reviews into patchset approvals https://review.openstack.org/97390 | 08:04 |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/zuul: Allow a pipeline to specify alternative gerrit acc https://review.openstack.org/97391 | 08:04 |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/zuul: Add support for negative requirements https://review.openstack.org/102726 | 08:04 |
*** Longgeek has quit IRC | 08:04 | |
*** Longgeek has joined #openstack-infra | 08:05 | |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/zuul: Fix up status page dependencies https://review.openstack.org/98028 | 08:05 |
*** Longgeek has quit IRC | 08:06 | |
*** Longgeek has joined #openstack-infra | 08:06 | |
*** ihrachyshka has joined #openstack-infra | 08:07 | |
openstackgerrit | Maxime Vidori proposed a change to openstack-infra/config: Add XStatic-angular-* packages for Horizon https://review.openstack.org/114947 | 08:08 |
*** ihrachyshka has quit IRC | 08:13 | |
*** ihrachyshka has joined #openstack-infra | 08:13 | |
*** trinaths has joined #openstack-infra | 08:17 | |
MaxV | AJaeger: For https://review.openstack.org/#/c/114947/ I just have to add | 08:19 |
MaxV | template: | 08:19 |
MaxV | - name: merge-check | 08:19 |
MaxV | on each project | 08:20 |
AJaeger | MaxV: Yes. | 08:20 |
MaxV | ok | 08:20 |
*** [1]trinaths has joined #openstack-infra | 08:20 | |
AJaeger | This was done yesterday on all projects, so yours should have it now as well. | 08:20 |
AJaeger | But first rebase your change to head - then you see it ;) | 08:20 |
*** trinaths has quit IRC | 08:21 | |
*** [1]trinaths is now known as trinaths | 08:21 | |
*** Krast has joined #openstack-infra | 08:22 | |
*** jp_at_hp has joined #openstack-infra | 08:23 | |
openstackgerrit | Julien Danjou proposed a change to openstack-infra/config: Use python3-jobs template where possible https://review.openstack.org/110256 | 08:23 |
*** derekh has joined #openstack-infra | 08:23 | |
openstackgerrit | Maxime Vidori proposed a change to openstack-infra/config: Add XStatic-angular-* packages for Horizon https://review.openstack.org/114947 | 08:25 |
openstackgerrit | Julien Danjou proposed a change to openstack-infra/config: Enable Python 3 job for oslo.messaging https://review.openstack.org/115208 | 08:25 |
loki184 | AJaeger: hi, is the merge-check template mandatory https://review.openstack.org/#/c/110562/ | 08:26 |
*** gpocente1 is now known as gpocentek | 08:27 | |
*** gpocentek has joined #openstack-infra | 08:27 | |
openstackgerrit | Radomir Dopieralski proposed a change to openstack-infra/config: Create a repository for stackforge/xstatic-font-awesome https://review.openstack.org/115209 | 08:28 |
*** doude has joined #openstack-infra | 08:31 | |
*** tsg has quit IRC | 08:32 | |
*** amuller has joined #openstack-infra | 08:32 | |
openstackgerrit | Radomir Dopieralski proposed a change to openstack/requirements: Add xstatic-font-awesome for Horizon https://review.openstack.org/115211 | 08:33 |
*** loki184 has quit IRC | 08:35 | |
*** hashar has joined #openstack-infra | 08:35 | |
*** loki184 has joined #openstack-infra | 08:35 | |
*** xchu has joined #openstack-infra | 08:36 | |
*** Longgeek has quit IRC | 08:42 | |
*** Longgeek has joined #openstack-infra | 08:43 | |
*** xchu has quit IRC | 08:45 | |
*** frank_zhou has left #openstack-infra | 08:47 | |
*** Longgeek has quit IRC | 08:47 | |
openstackgerrit | A change was merged to openstack-infra/config: Updated storyboard build to use new test modules https://review.openstack.org/106193 | 08:48 |
openstackgerrit | A change was merged to openstack-infra/config: Updated vinz build to use new test modules https://review.openstack.org/112128 | 08:49 |
*** Longgeek has joined #openstack-infra | 08:49 | |
AJaeger | loki184, yes, it's mandatory. See also http://ci.openstack.org/stackforge.html | 08:52 |
*** p5ntangle has joined #openstack-infra | 08:52 | |
AJaeger | loki184, rebase to head first... | 08:53 |
loki184 | Ajaeger: i have to trouble lifeless again to give +1 for the new patch i upload so asked you whether mandatory | 08:54 |
*** denis_makogon has quit IRC | 08:54 | |
loki184 | Ajaeger: i will put a new patch | 08:54 |
AJaeger | loki184, I'm sure fungi will scroll up in the history and notice that lifeless agreed with the plan... If not ask him friendly | 08:55 |
loki184 | Ajaeger: ok ty :) | 08:56 |
*** omrim has joined #openstack-infra | 08:56 | |
omrim | Hello | 08:57 |
*** yamahata has quit IRC | 08:57 | |
*** ihrachyshka_ has joined #openstack-infra | 08:59 | |
*** hashar has quit IRC | 09:00 | |
*** amuller has quit IRC | 09:01 | |
*** ihrachyshka has quit IRC | 09:02 | |
*** proffalken has quit IRC | 09:04 | |
*** jedimike has joined #openstack-infra | 09:05 | |
*** amuller has joined #openstack-infra | 09:06 | |
*** rfolco has quit IRC | 09:07 | |
omrim | I have a question..Can someone help please?:) | 09:08 |
*** andreaf_ has quit IRC | 09:10 | |
*** pelix has joined #openstack-infra | 09:10 | |
*** proffalken has joined #openstack-infra | 09:11 | |
*** skolekonov has joined #openstack-infra | 09:13 | |
openstackgerrit | lokesh s proposed a change to openstack-infra/config: Adding windows disk-image builder project https://review.openstack.org/110562 | 09:14 |
openstackgerrit | François Magimel proposed a change to openstack-infra/config: Added python-cloudkittyclient to stackforge projects https://review.openstack.org/112507 | 09:14 |
rdopieralski | omrim: the rule is that you have to ask the question first, and then if anybody knows the answer, they will answer | 09:15 |
omrim | rdopieralski: Thank you | 09:17 |
pleia2 | clarkb: will need to chat about some pootle issues we ran into tonight, currently have a spinning progress thing stuck on the main page following attempting to upload a po file | 09:17 |
pleia2 | but for now I'm going to bed :) | 09:17 |
*** e0ne has joined #openstack-infra | 09:17 | |
*** ociuhandu has quit IRC | 09:17 | |
openstackgerrit | François Magimel proposed a change to openstack-infra/config: Added cloudkitty-dashboard to stackforge projects https://review.openstack.org/112789 | 09:17 |
*** fifieldt_ has quit IRC | 09:18 | |
AJaeger | loki184, did you rebase? | 09:18 |
omrim | I have an issue with my Zuul-Gearman protocol. | 09:19 |
*** marekd has joined #openstack-infra | 09:19 | |
omrim | When a job start to run I notice that the job flagged as NOT_REGISTERED.. | 09:20 |
omrim | Is someone familiar with that? | 09:21 |
*** yolanda__ has joined #openstack-infra | 09:21 | |
loki184 | Ajaeger: Yes | 09:21 |
AJaeger | loki184, looking at your file, it looks old. Did you "git pull origin/master" first? | 09:22 |
omrim | I assume that the issue came from the following src code (gearman.py): if not self.isJobRegistered(gearman_job.name): self.log.error("Job %s is not registered with Gearman" % gearman_job) self.onBuildCompleted(gearman_job, 'NOT_REGISTERED') return build | 09:23 |
*** amuller has quit IRC | 09:23 | |
AJaeger | loki184, looking at https://review.openstack.org/#/c/110562/14/modules/openstack_project/files/zuul/layout.yaml I would expect to see merge-check with each project | 09:23 |
*** amuller has joined #openstack-infra | 09:23 | |
AJaeger | but it'S not in those that are shown - that's why I think you did not rebase properly against head | 09:23 |
*** Mithrandir_ has joined #openstack-infra | 09:24 | |
loki184 | Ajaeger: let me check | 09:24 |
*** Daisy has quit IRC | 09:24 | |
*** proffalken has quit IRC | 09:24 | |
*** yolanda_ has quit IRC | 09:25 | |
*** Mithrandir has quit IRC | 09:26 | |
*** Mithrandir_ is now known as Mithrandir | 09:26 | |
*** cnesa7 has joined #openstack-infra | 09:27 | |
*** proffalken has joined #openstack-infra | 09:27 | |
loki184 | Ajaeger: i have rebased it and i dont see merge-check again all the projects | 09:28 |
*** proffalken has quit IRC | 09:29 | |
openstackgerrit | Nadya Privalova proposed a change to openstack-infra/reviewstats: Add nprivalova and ildikov to ceilometer-core https://review.openstack.org/115223 | 09:29 |
*** proffalken has joined #openstack-infra | 09:30 | |
AJaeger | loki184, just check http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/zuul/layout.yaml#n2584 | 09:30 |
*** baohua_ has quit IRC | 09:31 | |
AJaeger | That's the current version of the file, line 2587 contains the merge-check for tuskar-ui but it's not in your patch | 09:31 |
AJaeger | loki184, try: | 09:33 |
AJaeger | git fetch origin | 09:33 |
AJaeger | does this download files? | 09:33 |
openstackgerrit | François Magimel proposed a change to openstack-infra/config: Added python-cloudkittyclient to stackforge projects https://review.openstack.org/112507 | 09:33 |
*** kashyap is now known as kashyapc | 09:34 | |
openstackgerrit | François Magimel proposed a change to openstack-infra/config: Added cloudkitty-dashboard to stackforge projects https://review.openstack.org/112789 | 09:34 |
*** dims has joined #openstack-infra | 09:35 | |
loki184 | AJaeger: i think something wrong with my branch i will do it again | 09:35 |
AJaeger | loki184, always work on a branch ... | 09:36 |
AJaeger | loki184, you could do the following: | 09:36 |
AJaeger | git checkout master | 09:36 |
AJaeger | git reset --hard origin/master | 09:37 |
AJaeger | git review -d 110562 # this downloads your change | 09:37 |
AJaeger | git rebase origin/master # this rebases your change | 09:37 |
AJaeger | resolve the merge conflict... | 09:37 |
AJaeger | and send for review again | 09:38 |
loki184 | Ajaeger: Ty, i was doing the same let me recheck | 09:38 |
AJaeger | loki184, if you want, I could also update the patch for you... | 09:38 |
loki184 | AJaeger: if you wish you can do it | 09:39 |
*** dims has quit IRC | 09:39 | |
*** andreaf has joined #openstack-infra | 09:42 | |
loki184 | Ajaeger: i have change ready i will commit it | 09:42 |
*** che-arne has joined #openstack-infra | 09:43 | |
openstackgerrit | A change was merged to openstack-infra/config: Do not fail the install_puppet.sh on 2th run https://review.openstack.org/105002 | 09:43 |
*** Daviey has quit IRC | 09:44 | |
openstackgerrit | lokesh s proposed a change to openstack-infra/config: Adding windows disk-image builder project https://review.openstack.org/110562 | 09:44 |
*** alexpilotti has joined #openstack-infra | 09:45 | |
loki184 | AJaeger: i submitted again please review | 09:45 |
openstackgerrit | Stéphane Albert proposed a change to openstack-infra/config: Added Read the Docs hook to cloudkitty job https://review.openstack.org/113194 | 09:46 |
AJaeger | loki184, let me check | 09:47 |
AJaeger | loki184, yeah, looks fine now. | 09:48 |
loki184 | AJaeger: Thanks for the help :) | 09:49 |
*** unicell has quit IRC | 09:51 | |
SergeyLukjanov | anteaya, I'm ok with Sahara Community CI or Sahara Mirantis CI if you like | 09:51 |
SergeyLukjanov | anteaya, Sahara Cluster CI is mostly fair too | 09:51 |
*** hashar has joined #openstack-infra | 09:52 | |
*** ociuhandu has joined #openstack-infra | 09:58 | |
*** mmaglana has quit IRC | 09:59 | |
*** pblaho is now known as pblaho|afk | 10:00 | |
*** bookwar has joined #openstack-infra | 10:00 | |
*** AJaeger has quit IRC | 10:00 | |
*** rick__ has quit IRC | 10:01 | |
*** linkid has joined #openstack-infra | 10:01 | |
*** MaxV has quit IRC | 10:03 | |
*** doude has quit IRC | 10:10 | |
*** Daviey has joined #openstack-infra | 10:11 | |
*** doude has joined #openstack-infra | 10:12 | |
*** amuller has quit IRC | 10:14 | |
*** Guest36489 has quit IRC | 10:17 | |
*** rlandy has joined #openstack-infra | 10:18 | |
*** Longgeek has quit IRC | 10:24 | |
omrim | Hi, What is the reson that my jobs are not registered with Gearman? | 10:26 |
*** _nadya_ has quit IRC | 10:27 | |
*** _nadya_ has joined #openstack-infra | 10:27 | |
*** e0ne has quit IRC | 10:29 | |
*** pblaho|afk is now known as pblaho | 10:29 | |
*** pcm_ has joined #openstack-infra | 10:30 | |
*** andreaf has quit IRC | 10:33 | |
*** zul has quit IRC | 10:36 | |
*** AlexF has joined #openstack-infra | 10:42 | |
*** hashar has quit IRC | 10:42 | |
*** MaxV has joined #openstack-infra | 10:44 | |
*** Longgeek has joined #openstack-infra | 10:53 | |
openstackgerrit | Julie Pichon proposed a change to openstack-infra/config: Add integration tests to the Horizon gate https://review.openstack.org/115004 | 10:59 |
*** Azher has quit IRC | 10:59 | |
*** yaguang_ has joined #openstack-infra | 11:00 | |
*** amuller has joined #openstack-infra | 11:01 | |
*** yaguang has quit IRC | 11:02 | |
*** ihrachyshka_ is now known as ihrachyshka | 11:04 | |
*** ppai has joined #openstack-infra | 11:07 | |
openstackgerrit | A change was merged to openstack-infra/os-loganalyze: Support Designate's log for severity filtering https://review.openstack.org/107701 | 11:07 |
*** KanagarajM has quit IRC | 11:11 | |
*** e0ne has joined #openstack-infra | 11:15 | |
*** dims has joined #openstack-infra | 11:17 | |
*** dalgaaf_ has joined #openstack-infra | 11:18 | |
openstackgerrit | A change was merged to openstack-infra/config: Add instructions to rename gerrit groups https://review.openstack.org/114765 | 11:18 |
*** andreaf has joined #openstack-infra | 11:19 | |
*** RicoLin has quit IRC | 11:23 | |
*** Longgeek has quit IRC | 11:26 | |
*** Longgeek has joined #openstack-infra | 11:26 | |
openstackgerrit | Russell Bryant proposed a change to openstack-infra/config: Disable ironic job on stable branches https://review.openstack.org/115248 | 11:28 |
jedimike | omrim, hi, I had this problem this morning too, it was because my jenkins jobs were tagged to only build on certain slaves, but I didn't have any slaves online with those tags | 11:30 |
jedimike | omrim, so when zuul tried to run the jobs, gearman returns NOT_REGISTERED, which means it doesn't have any suitable nodes registered for the job to run on | 11:30 |
openstackgerrit | Prashanth Pai proposed a change to openstack-infra/config: Add swiftonfile project to stackforge https://review.openstack.org/115250 | 11:31 |
omrim | jedimike: Unfortunately It's not the issue in my case | 11:32 |
omrim | jedimike: I do have machines that are indeed ready for each job | 11:33 |
omrim | jedimike: When some job start I notice that in the gerrit there is the label: NOT_REGISTERED | 11:34 |
jedimike | omrim, I wish I could help more, I've only just started looking at our problem so that's all I've discovered so far. I'm not sure what else it might be, sorry | 11:34 |
*** loki184 has quit IRC | 11:37 | |
*** dmsimard_away is now known as dmsimard | 11:39 | |
*** oanufriev has quit IRC | 11:39 | |
omrim | jedimike: Thank you anyway :) | 11:40 |
*** yaguang_ has quit IRC | 11:41 | |
*** salv-orlando_ has joined #openstack-infra | 11:43 | |
omrim | Is there someone that know why my jobs are not registered with Gearman? | 11:43 |
*** salv-orlando has quit IRC | 11:43 | |
*** salv-orlando_ is now known as salv-orlando | 11:43 | |
openstackgerrit | Russell Bryant proposed a change to openstack-infra/config: Remove an unused node type https://review.openstack.org/113280 | 11:46 |
openstackgerrit | Russell Bryant proposed a change to openstack-infra/config: Set min-ready=0 for dsvm-precise-krnl https://review.openstack.org/115253 | 11:46 |
*** SumitNaiksatam_ has joined #openstack-infra | 11:48 | |
*** baohua has joined #openstack-infra | 11:49 | |
*** SumitNaiksatam has quit IRC | 11:49 | |
*** SumitNaiksatam_ has quit IRC | 11:49 | |
rcarrillocruz | FYI, puppet-lint 1.0.0 has been released a couple of days ago. It checks newer things that it didn't before, just in case you see all of the sudden puppet-lint check errors when pushing changes | 11:49 |
*** salv-orlando has quit IRC | 11:49 | |
openstackgerrit | Prashanth Pai proposed a change to openstack-infra/config: Add swiftonfile project to stackforge https://review.openstack.org/115250 | 11:49 |
*** SumitNaiksatam has joined #openstack-infra | 11:49 | |
*** pblaho has quit IRC | 11:50 | |
openstackgerrit | A change was merged to openstack-infra/reviewstats: Add nprivalova and ildikov to ceilometer-core https://review.openstack.org/115223 | 11:54 |
*** salv-orlando has joined #openstack-infra | 11:55 | |
*** dkranz has joined #openstack-infra | 11:56 | |
*** dims has quit IRC | 11:56 | |
*** weshay has joined #openstack-infra | 11:57 | |
*** ppai has quit IRC | 11:58 | |
*** dprince has joined #openstack-infra | 12:00 | |
*** weshay is now known as weshay_mtg | 12:00 | |
marcusvrn | Hi guys...my job dsvm-tempest-full works well the first time that I start it.....if I start the job again, it always returns FAILURE....I found only one error: "ERROR: InvocationError: '/bin/bash tools/pretty_tox.sh..." | 12:02 |
openstackgerrit | A change was merged to openstack-infra/reviewstats: Update documentation setup https://review.openstack.org/114319 | 12:02 |
*** Krast has quit IRC | 12:02 | |
*** sdague has joined #openstack-infra | 12:04 | |
jpich | russellb: Hello! Thank you for your comments on https://review.openstack.org/#/c/115004/ , I really appreciate it! I am wondering about your last comment: does this mean that this should actually be sufficient to get an experimental job set up, or more simply that you can't see anything egregiously wrong with it at this point? | 12:04 |
*** hdd has joined #openstack-infra | 12:04 | |
russellb | jpich: yeah, i think it's OK, as far as i can tell | 12:06 |
russellb | jpich: assuming you have a py27integration env set up | 12:06 |
russellb | jpich: the only other thing I thought about suggesting is a change that would make it so "gate" doesn't have to be hard coded, and could be check or gate depending on which pipeline it's in | 12:07 |
russellb | but that's really just beatification, not important IMO | 12:07 |
jpich | russellb: Oh cool, I thought there'd me more trickstery to it all. The tox environment does exist | 12:08 |
russellb | jpich: yeah, it's kind of amazing, really | 12:08 |
jpich | russellb: Noted for the rest as well, I'll see what I can do. Cheers! | 12:08 |
russellb | up to you, it's not a big deal | 12:09 |
jpich | Got it | 12:09 |
russellb | if you really want to mess with it, it's basically changing "job" to "job-template" and then changing the name to something like "{pipeline}-horizon-integration{branch-designator}' | 12:09 |
russellb | swift.yaml would be a good example of that | 12:09 |
*** dims has joined #openstack-infra | 12:09 | |
russellb | it's close to what you're doing | 12:09 |
*** Guest36489 has joined #openstack-infra | 12:10 | |
*** kashyapc is now known as kashyap | 12:10 | |
jpich | What could possibly go wrong | 12:11 |
russellb | i'd probably just leave it as is and see if someone complains :) | 12:11 |
jpich | I just might :-) Thanks for the hints either way! | 12:12 |
russellb | sure np | 12:12 |
jpich | Maybe I'll do that when it's promoted to a non-experimental job later on | 12:12 |
*** pblaho has joined #openstack-infra | 12:14 | |
*** kashyap is now known as kashyapc | 12:16 | |
*** ZZelle has joined #openstack-infra | 12:20 | |
*** pcm__ has joined #openstack-infra | 12:22 | |
*** rfolco has joined #openstack-infra | 12:22 | |
*** pblaho has quit IRC | 12:24 | |
*** jab has joined #openstack-infra | 12:25 | |
*** pcm_ has quit IRC | 12:26 | |
*** pdmars has joined #openstack-infra | 12:26 | |
*** aysyd has joined #openstack-infra | 12:26 | |
*** kgiusti has joined #openstack-infra | 12:26 | |
*** pblaho has joined #openstack-infra | 12:27 | |
*** jab is now known as bradjones | 12:27 | |
*** annegentle has joined #openstack-infra | 12:30 | |
*** gargola has joined #openstack-infra | 12:30 | |
*** bswartz has joined #openstack-infra | 12:30 | |
*** adalbas has joined #openstack-infra | 12:32 | |
*** mbacchi has joined #openstack-infra | 12:35 | |
*** dims has quit IRC | 12:37 | |
openstackgerrit | Julien Danjou proposed a change to openstack-infra/config: Use python3-jobs template where possible https://review.openstack.org/110256 | 12:38 |
*** dims has joined #openstack-infra | 12:38 | |
*** AlexF has quit IRC | 12:39 | |
*** julim has joined #openstack-infra | 12:42 | |
*** julim has quit IRC | 12:43 | |
*** unicell has joined #openstack-infra | 12:45 | |
*** hdd has quit IRC | 12:45 | |
*** hdd has joined #openstack-infra | 12:46 | |
*** hashar has joined #openstack-infra | 12:46 | |
*** trinaths has quit IRC | 12:46 | |
*** pblaho has quit IRC | 12:48 | |
*** dizquierdo has quit IRC | 12:49 | |
*** markmc has joined #openstack-infra | 12:49 | |
*** maxbit has quit IRC | 12:50 | |
*** maxbit has joined #openstack-infra | 12:50 | |
*** annegentle has quit IRC | 12:52 | |
fungi | jeblair: awesome--thanks! i'll try out the whole patch series here shortly | 12:53 |
*** dkranz has quit IRC | 12:54 | |
*** pblaho has joined #openstack-infra | 12:56 | |
openstackgerrit | Timur Sufiev proposed a change to openstack-infra/config: Create merlin repo https://review.openstack.org/110325 | 12:56 |
*** nosnos has quit IRC | 12:56 | |
*** dkliban_afk is now known as dkliban | 12:57 | |
*** dkranz has joined #openstack-infra | 12:57 | |
*** heyongli has joined #openstack-infra | 13:00 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Add new template stackforge-specs-jobs https://review.openstack.org/113211 | 13:00 |
*** jgrimm has joined #openstack-infra | 13:00 | |
*** pblaho is now known as pblaho|meeting | 13:00 | |
*** jpeeler has joined #openstack-infra | 13:01 | |
*** jpeeler has quit IRC | 13:01 | |
*** jpeeler has joined #openstack-infra | 13:01 | |
*** amotoki has quit IRC | 13:03 | |
*** julim has joined #openstack-infra | 13:03 | |
rcarrillocruz | fungi: hi there, how often do you update nodepool images? | 13:05 |
Mmike | rcarrillocruz: daily, if the docs are fine | 13:05 |
fungi | rcarrillocruz: daily starting at 1400utc (usually finishes by 1530) | 13:05 |
rcarrillocruz | just as a FYI, http://bombasticmonkey.com/2014/08/19/puppet-lint-1.0.0/ | 13:06 |
rcarrillocruz | our bare precise images were updated, and so did the puppet-lint | 13:06 |
rcarrillocruz | which broke a lot of manifests | 13:06 |
fungi | rcarrillocruz: oh, neat! | 13:06 |
rcarrillocruz | just a FYI in case you guys see puppet-lint job failures as i did | 13:06 |
rcarrillocruz | it seems it's pickier than it was before | 13:06 |
fungi | rcarrillocruz: noted. if we need to, we can pin it or something i guess | 13:07 |
rcarrillocruz | for example, it requires ONLY one space between the longest attribute of a class and the '=>' | 13:07 |
rcarrillocruz | otherwise, you get arrow alignment check failures | 13:07 |
rcarrillocruz | yep, just in case | 13:07 |
rcarrillocruz | bbl, lunch | 13:07 |
* fungi loves tools which are picky about non-semantic whitespace. such a great waste of everyone's time | 13:08 | |
mattymo | can an abandoned patchset be restored by anyone except the owner? | 13:08 |
*** dustins has joined #openstack-infra | 13:11 | |
fungi | mattymo: any core reviewer for the same project too | 13:11 |
mattymo | fungi, thanks! | 13:11 |
*** bknudson has joined #openstack-infra | 13:12 | |
*** pradk has joined #openstack-infra | 13:12 | |
flaper87 | is there a good way to debug "git clone" issues in the gate? I'm hitting this http://logs.openstack.org/65/115265/3/check/gate-tempest-dsvm-large-ops/ba64713/logs/devstacklog.txt.gz#_2014-08-19_12_24_19_150 | 13:13 |
flaper87 | but the same git_clone command seem to work locally | 13:13 |
flaper87 | and I don't see any git error in the logs | 13:13 |
*** p5ntangle has quit IRC | 13:14 | |
*** mriedem has joined #openstack-infra | 13:15 | |
*** mfer has joined #openstack-infra | 13:16 | |
*** ominakov has joined #openstack-infra | 13:17 | |
flaper87 | ah nvm... (facepalm) | 13:19 |
*** miqui has joined #openstack-infra | 13:19 | |
*** paulrad has joined #openstack-infra | 13:20 | |
*** ilyashakhat has joined #openstack-infra | 13:21 | |
*** gfidente has joined #openstack-infra | 13:21 | |
gfidente | guys, I'm not sure where gerrit is picking up the list of uses when adding someone to a review but today my name/email appears twice in that list and neither of two work, as I get a message saying it doesn't not map to a user/identity | 13:22 |
gfidente | any idea what is going on? maybe needs some cleanup? | 13:22 |
*** dizquierdo has joined #openstack-infra | 13:23 | |
*** pdmars has quit IRC | 13:23 | |
*** ildikov_ has quit IRC | 13:23 | |
*** pdmars has joined #openstack-infra | 13:23 | |
*** radez_g0n3 is now known as radez | 13:25 | |
fungi | flaper87: it looks like you're running afoul of the reclone restrictions in devstack--you probably need to dig into how to go about that. also i doubt we want to be cloning from git directly there... instead we should be using the same precloned git sources we use for all the other projects in our integration tests, right? | 13:27 |
*** pdmars has quit IRC | 13:28 | |
fungi | flaper87: anyway, probably #openstack-qa will be a better venue to discuss that sort of design requirement | 13:28 |
gfidente | so I cannot add my user as reviewer to the submissions; any idea what is gerrit doing there? something I can look into by myself? | 13:28 |
fungi | gfidente: there may be multiple gerrit accounts for you | 13:28 |
openstackgerrit | A change was merged to openstack-infra/config: Create a repository for additional Tuskar-UI plugins https://review.openstack.org/108102 | 13:28 |
*** penguinRaider has quit IRC | 13:29 | |
gfidente | fungi, yeah that seems to be but both have same name/email and the thing is that neither seems to work | 13:29 |
*** hdd has quit IRC | 13:29 | |
*** jaypipes has joined #openstack-infra | 13:29 | |
flaper87 | fungi: I'll ask there. I was not aware of the precloned git sources. Following what glance does there, I thought that was the desired thing. I also looked into what oslo.messaging does since the case is similar | 13:29 |
flaper87 | fungi: thanks for the feedback, I'll dig a bit more | 13:29 |
fungi | gfidente: i've seen that when users change something on lp, which causes lp to decide to switch them to a new openid url, then gerrit reimports their e-mail address for the new account and starts breaking because of the e-mail address conflicting (gerrit assumes e-mail addresses will be unique, but does not enforce that when adding them from an openid provider) | 13:30 |
gfidente | fungi, AAAh, I see that now | 13:30 |
fungi | gfidente: did you switch lp accounts, rename one or merge it with another recently? | 13:31 |
*** ildikov has joined #openstack-infra | 13:31 | |
gfidente | fungi, I played a bit with the groups memberships | 13:31 |
*** pblaho|meeting is now known as pblaho | 13:31 | |
fungi | hrm, usually i wouldn't expect changing lp team membership to affect that | 13:31 |
*** penguinRaider has joined #openstack-infra | 13:32 | |
*** pdmars has joined #openstack-infra | 13:32 | |
gfidente | uhm and we have no control over LP anyway | 13:32 |
*** pcrews has joined #openstack-infra | 13:33 | |
fungi | gfidente: yeah, basically i need to ferret out whatever various accounts it ended up creating for you and try to disable the ones you're not using to eliminate the conflict. just a moment | 13:33 |
*** ildikov_ has joined #openstack-infra | 13:34 | |
*** ildikov has quit IRC | 13:34 | |
*** pcrews_ has joined #openstack-infra | 13:34 | |
openstackgerrit | Julien Danjou proposed a change to openstack-infra/config: Import pylockfile https://review.openstack.org/101911 | 13:34 |
*** pblaho has quit IRC | 13:34 | |
*** pdmars has quit IRC | 13:35 | |
openstackgerrit | A change was merged to openstack-infra/config: Create refs permission isn't needed for sahara now https://review.openstack.org/113834 | 13:35 |
openstackgerrit | Julien Danjou proposed a change to openstack-infra/config: Import pylockfile https://review.openstack.org/101911 | 13:35 |
*** hashar_ has joined #openstack-infra | 13:35 | |
*** pdmars has joined #openstack-infra | 13:35 | |
*** pcrews__ has joined #openstack-infra | 13:36 | |
*** heyongli has quit IRC | 13:37 | |
*** eharney has joined #openstack-infra | 13:37 | |
*** pblaho has joined #openstack-infra | 13:37 | |
*** pcrews has quit IRC | 13:38 | |
*** hashar has quit IRC | 13:38 | |
*** penguinRaider has quit IRC | 13:39 | |
*** homeless has joined #openstack-infra | 13:39 | |
*** pcrews_ has quit IRC | 13:39 | |
*** ominakov_ has joined #openstack-infra | 13:40 | |
*** dmellado has joined #openstack-infra | 13:41 | |
*** pcrews__ has quit IRC | 13:41 | |
gfidente | fungi, if there is anything I can do myself, please let me know | 13:41 |
gfidente | I did not intend to bother with this, thought was more of a generic problem | 13:41 |
*** ominakov has quit IRC | 13:41 | |
*** penguinRaider has joined #openstack-infra | 13:41 | |
fungi | gfidente: yep, two accounts for you in gerrit, both with the same full name and preferred e-mail address. creation date on the more recent one is about two weeks ago | 13:41 |
*** dizquierdo has quit IRC | 13:42 | |
*** oomichi has quit IRC | 13:42 | |
gfidente | two weeks ago I suppose can be deleted, after I'm logged in gerrit, I can see all submissions both older and newer than two weeks | 13:43 |
*** pcrews has joined #openstack-infra | 13:43 | |
gfidente | so I suppose something happened 2 weeks ago, but lucky me never logged in with that account? | 13:43 |
fungi | gfidente: well, you did log in with that account, which is how it ended up being created in gerrit (accounts are autocreated based on the openid url passed back from external authentication, in this case launchpad) | 13:44 |
fungi | i've moved that openid to be an additional one on your old account and then deactivated the newer account | 13:44 |
gfidente | wasn't login.ubuntu.com updated few weeks back? | 13:45 |
fungi | gfidente: log out of gerrit and back in, and then see if things are working any better | 13:45 |
fungi | gfidente: yeah, they did some new theming at least to match ubuntu one's interface better, but not sure if that had any backend implications | 13:45 |
*** jcoufal has quit IRC | 13:45 | |
*** jcoufal has joined #openstack-infra | 13:46 | |
*** HenryG_ has joined #openstack-infra | 13:46 | |
gfidente | I logged out and back in | 13:46 |
*** zz_dimtruck is now known as dimtruck | 13:46 | |
*** HenryG has quit IRC | 13:47 | |
gfidente | I see two identities, one from login.ubuntu.com and one from login.launchpad.net in gerrit settings | 13:47 |
gfidente | maybe that was it? | 13:47 |
fungi | gfidente: quite possibly | 13:48 |
gfidente | fungi, but I was able to add my accunt to a review now so I'm fine with current setup | 13:49 |
gfidente | thanks a lot | 13:49 |
fungi | gfidente: and yeah, i wouldn't be surprised if they (very briefly) while updating login.launchpad.net configured it to report as login.ubuntu.com instead, then switched it back as soon as they realized it broke everyone using it, but you logged into gerrit during that window and ended up with a second account. if so, there are possibly more users in the same boat | 13:50 |
gfidente | I see what you mean | 13:51 |
openstackgerrit | Dan Prince proposed a change to openstack-infra/config: Import os-net-config project under TripleO https://review.openstack.org/112331 | 13:52 |
*** RicoLin has joined #openstack-infra | 13:54 | |
openstackgerrit | Malini Kamalambal proposed a change to openstack-infra/config: Update Marconi to Zaqar https://review.openstack.org/115291 | 13:55 |
*** gargola has quit IRC | 13:57 | |
*** gargola has joined #openstack-infra | 13:58 | |
*** hashar_ is now known as hashar | 13:58 | |
*** stevemar has joined #openstack-infra | 14:03 | |
*** marun has joined #openstack-infra | 14:04 | |
*** marun is now known as marun_afk | 14:05 | |
flaper87 | fungi: I think I need to add glance.store to the list of the pre-cloned repos. Any hint? I've no idea how to do that. | 14:05 |
fungi | flaper87: yeah, let me find it for you | 14:06 |
fungi | flaper87: probably at least in lib/glance | 14:07 |
*** HenryG_ is now known as HenryG | 14:07 | |
fungi | which you already tried in your change | 14:07 |
*** markmcclain has joined #openstack-infra | 14:07 | |
flaper87 | fungi: right :/ | 14:08 |
*** dmakogon_ is now known as denis_makogon | 14:08 | |
fungi | oh, i know where to find it | 14:08 |
*** enikanorov__ has joined #openstack-infra | 14:09 | |
fungi | hrm, you also already added it to stackrc | 14:10 |
*** enikanorov has quit IRC | 14:10 | |
*** stevemar has quit IRC | 14:11 | |
*** tnurlygayanov has quit IRC | 14:11 | |
fungi | flaper87: oh, you may *first* need to add it to the PROJECTS list in openstack-infra/devstack-gate:devstack-vm-gate-wrap.sh | 14:11 |
*** stevemar has joined #openstack-infra | 14:11 | |
flaper87 | fungi: ah there you go, I think that's what I was looking for | 14:12 |
flaper87 | fungi: thanks D | 14:12 |
flaper87 | :D | 14:12 |
*** amotoki has joined #openstack-infra | 14:12 | |
*** oomichi has joined #openstack-infra | 14:12 | |
*** dizquierdo has joined #openstack-infra | 14:13 | |
openstackgerrit | Flavio Percoco proposed a change to openstack-infra/devstack-gate: Add glance.store to the list of projects https://review.openstack.org/115299 | 14:14 |
openstackgerrit | Flavio Percoco proposed a change to openstack-infra/devstack-gate: Add glance.store to the list of projects https://review.openstack.org/115299 | 14:14 |
flaper87 | fungi: ^ | 14:14 |
flaper87 | thanks for your help here, fungi | 14:14 |
fungi | flaper87: yw! | 14:15 |
*** nelsnelson has joined #openstack-infra | 14:16 | |
*** abhishekk has joined #openstack-infra | 14:16 | |
*** oomichi has quit IRC | 14:17 | |
abhishekk | hi, I have submitted this patch https://review.openstack.org/#/c/102215/ for updating the version of websockify library | 14:17 |
abhishekk | which is merged | 14:17 |
*** chandankumar has quit IRC | 14:17 | |
abhishekk | but still I am not able to see the latest websockify in http://pypi.openstack.org/openstack/websockify/ | 14:17 |
abhishekk | can anybody help me to understand how this latest websockify will be uploaded to http://pypi.openstack.org/openstack/websockify/ | 14:18 |
*** lttrl has joined #openstack-infra | 14:18 | |
*** mfer has quit IRC | 14:23 | |
*** david-lyle has joined #openstack-infra | 14:24 | |
*** linkid has quit IRC | 14:24 | |
*** malini has joined #openstack-infra | 14:25 | |
*** maxbit_ has joined #openstack-infra | 14:25 | |
malini | Hello..Can I get a couple of reviews for https://review.openstack.org/#/c/115291/ ? We need this to unblock Zaqar (Marconi) gate | 14:25 |
*** reed has joined #openstack-infra | 14:26 | |
*** ramashri has joined #openstack-infra | 14:26 | |
*** reed has quit IRC | 14:26 | |
*** reed has joined #openstack-infra | 14:26 | |
*** marun_afk has quit IRC | 14:27 | |
*** maxbit has quit IRC | 14:28 | |
*** hdd has joined #openstack-infra | 14:30 | |
*** ramashri has quit IRC | 14:30 | |
*** tonytan4ever has joined #openstack-infra | 14:34 | |
*** markvoelker has joined #openstack-infra | 14:34 | |
openstackgerrit | Longgeek proposed a change to openstack-infra/puppet-apparmor: Add Rakefile file and update puppet coding style https://review.openstack.org/102794 | 14:35 |
*** tsg has joined #openstack-infra | 14:35 | |
*** ramashri has joined #openstack-infra | 14:35 | |
openstackgerrit | Longgeek proposed a change to openstack-infra/puppet-dashboard: Add Rakefile file and dd variables and document https://review.openstack.org/102796 | 14:35 |
markvoelker | Anybody around to have a look at https://review.openstack.org/#/c/100632/? Seems that review got lost in the gate. It is Workflow +1 but never seems to have gotten picked up. | 14:36 |
openstackgerrit | A change was merged to openstack-infra/config: Add tripleo to specs.o.o https://review.openstack.org/111588 | 14:36 |
*** maxbit_ has quit IRC | 14:36 | |
*** dangers_away is now known as dangers | 14:37 | |
*** rdopieralski has quit IRC | 14:37 | |
*** ildikov_ has quit IRC | 14:38 | |
*** mestery_afk is now known as mestery | 14:39 | |
abhishekk | hi, can any one help me to understand how to upload latest library in http://pypi.openstack.org/openstack/websockify/ | 14:40 |
fungi | abhishekk: look in http://pypi.openstack.org/simple/websockify/ instead | 14:40 |
fungi | abhishekk: the other url is our old mirror which we don't use/update any longer | 14:40 |
abhishekk | fungi: thank you | 14:41 |
fungi | abhishekk: also, our mirror now updates basically as soon as something is uploaded to pypi.python.org. we no longer base it on the global-requirements.txt contents | 14:41 |
abhishekk | fungi:thank you for the update | 14:42 |
*** datsun180b has joined #openstack-infra | 14:43 | |
*** Guest36489 has quit IRC | 14:43 | |
*** dmellado has quit IRC | 14:44 | |
*** kashyapc is now known as kashyap | 14:45 | |
*** dizquierdo has quit IRC | 14:46 | |
*** pblaho is now known as pblaho|afk | 14:46 | |
*** atiwari has joined #openstack-infra | 14:46 | |
yolanda__ | anteaya, fungi, if i want to update documentation on http://ci.openstack.org/nodepool/, where should i clone from? | 14:47 |
*** wenlock has joined #openstack-infra | 14:47 | |
*** pblaho|afk is now known as pblaho | 14:47 | |
openstackgerrit | Felipe Reyes proposed a change to openstack-infra/elastic-recheck: Add fingerprint for bug 1358779 https://review.openstack.org/115306 | 14:48 |
uvirtbot | Launchpad bug 1358779 in openstack-ci "swift: no space left on device" [Undecided,New] https://launchpad.net/bugs/1358779 | 14:48 |
fungi | yolanda__: that's the documentation in the nodepool repository | 14:48 |
*** kashyap is now known as kashyapc | 14:48 | |
fungi | yolanda__: http://git.openstack.org/cgit/openstack-infra/nodepool/tree/doc/source | 14:49 |
*** kashyapc is now known as kashyap | 14:49 | |
*** tonytan4ever has quit IRC | 14:51 | |
*** markmcclain has quit IRC | 14:52 | |
*** hdd has quit IRC | 14:53 | |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add fingerprint for nova unit test bug 1357578 https://review.openstack.org/114680 | 14:53 |
uvirtbot | Launchpad bug 1357578 in nova "Unit test: nova.tests.integrated.test_multiprocess_api.MultiprocessWSGITest.test_terminate_sigterm timing out in gate" [Undecided,New] https://launchpad.net/bugs/1357578 | 14:53 |
*** tnurlygayanov has joined #openstack-infra | 14:53 | |
*** markmcclain has joined #openstack-infra | 14:53 | |
*** mfer has joined #openstack-infra | 14:54 | |
omrim | Is there someone that know why my jobs are not registered with Gearman? | 14:57 |
*** hdd has joined #openstack-infra | 14:58 | |
fungi | omrim: perhaps you have a firewall or some local packet filtering restricting traffic between your zuul server's gearman service and the gearman plugin on your jenkins master? | 14:58 |
*** afazekas has quit IRC | 14:58 | |
fungi | omrim: or you haven't enabled the plugin? | 14:59 |
*** julim has quit IRC | 14:59 | |
*** dkranz has quit IRC | 15:00 | |
fungi | omrim: have you done a 'echo status|nc -q 3 localhost 4730' on your zuul server and looked through the output to see if your jobs are registered incorrectly (rather than not at all)? | 15:00 |
*** mestery is now known as mestery_afk | 15:01 | |
yolanda__ | fungi, thx, i was thinking in some manuals repo | 15:02 |
*** imeerovi has quit IRC | 15:02 | |
yolanda__ | were you able to merge nodepool dib changes? | 15:02 |
*** arosen1 has joined #openstack-infra | 15:03 | |
omrim | fungi: First Yes I indeed enable the Gearman Jenkins Plugin Second I run this command and I saw that all of the jobs have a worker and they do run correctly | 15:03 |
*** annegentle has joined #openstack-infra | 15:03 | |
*** arosen1 has quit IRC | 15:03 | |
*** skolekonov has quit IRC | 15:03 | |
*** gokrokve has joined #openstack-infra | 15:03 | |
*** annegentle has quit IRC | 15:03 | |
*** AlexF has joined #openstack-infra | 15:03 | |
fungi | yolanda__: they merged but we haven't restarted nodepool yet to implement them | 15:03 |
*** arosen1 has joined #openstack-infra | 15:03 | |
*** arosen1 has quit IRC | 15:04 | |
*** annegentle has joined #openstack-infra | 15:04 | |
fungi | omrim: have you looked in zuul's debug log to find out what job-name:node-label combination it's seeking for those jobs? when it determines a job is not registered it logs what it thought should be registered which wasn't | 15:04 |
phschwartz | Morning infra | 15:05 |
fungi | yolanda__: perhaps later this evening once the test volume drops off we can try to remember to do that | 15:06 |
*** tonytan4ever has joined #openstack-infra | 15:06 | |
fungi | phschwartz: hey there. did you see my comment yesterday about those mystery issues in dfw? | 15:06 |
*** arosen1 has joined #openstack-infra | 15:06 | |
*** arosen1 has quit IRC | 15:06 | |
*** mmaglana has joined #openstack-infra | 15:06 | |
*** tsg has quit IRC | 15:07 | |
omrim | Yes I take a look into the logs | 15:07 |
*** Guest36489 has joined #openstack-infra | 15:07 | |
fungi | phschwartz: to recap, since our daily image update on friday went into effect, we've seen no recurrence of any of those three symptoms we were tracking | 15:08 |
*** marun has joined #openstack-infra | 15:08 | |
omrim | fungi: Now I will take a look again to sfor you suggestion | 15:08 |
fungi | phschwartz: so just wondering if anything was perhaps found/fixed in dfw on friday... if so i'll go ahead and close out the bugs we have open on our end | 15:08 |
*** marun has quit IRC | 15:09 | |
*** marun has joined #openstack-infra | 15:09 | |
*** marun has quit IRC | 15:09 | |
*** abhishekk has quit IRC | 15:10 | |
*** marun has joined #openstack-infra | 15:10 | |
*** marun has quit IRC | 15:10 | |
*** directxman12 has joined #openstack-infra | 15:12 | |
*** markmcclain has quit IRC | 15:12 | |
*** marun has joined #openstack-infra | 15:14 | |
*** dkranz has joined #openstack-infra | 15:14 | |
*** maxbit has joined #openstack-infra | 15:15 | |
*** ramashri has quit IRC | 15:15 | |
*** directxman12 has quit IRC | 15:15 | |
*** Guest36489 has quit IRC | 15:16 | |
*** primeministerp has joined #openstack-infra | 15:17 | |
*** primemin1sterp has joined #openstack-infra | 15:17 | |
*** primemin2sterp has joined #openstack-infra | 15:18 | |
*** linkid has joined #openstack-infra | 15:18 | |
*** alexpilotti has quit IRC | 15:18 | |
*** amuller_ has joined #openstack-infra | 15:18 | |
*** flaper87 is now known as flaper87|afk | 15:19 | |
*** hogepodge has joined #openstack-infra | 15:20 | |
*** amuller has quit IRC | 15:20 | |
lxsli | fungi: https://review.openstack.org/#/c/108737/ has 2x +2 now, please would you review and maybe approve? | 15:20 |
lxsli | or any other infra core :) | 15:20 |
*** marun has quit IRC | 15:22 | |
*** ihrachyshka has quit IRC | 15:22 | |
*** zz_gondoi is now known as gondoi | 15:23 | |
fungi | lxsli: i suppose my concerns about applying iptables rules peculiar to tripleo's infrastructure network to all of our job workers regardless of provider were not shared by other infra cores... | 15:23 |
jeblair | fungi: oh, i didn't read that as a very strong objection | 15:25 |
*** isaacb_ has joined #openstack-infra | 15:26 | |
jeblair | fungi, lxsli: but now that you mention it, why doesn't devtest just do this? | 15:27 |
isaacb_ | Hi | 15:27 |
*** che-arne has quit IRC | 15:28 | |
*** roaet has left #openstack-infra | 15:28 | |
omrim | fungi: I found somtinh like that: 2014-08-19 16:05:16,227 INFO zuul.Gearman: Launch job Check-MLNX-Neutron-ML2-Sriov-driver for change <Change 0x7f0b08239b50 74134,9> with dependent changes [] 2014-08-19 16:05:16,227 ERROR zuul.Gearman: Job <gear.Job 0x7f0b081699d0 handle: None name: build:Check-MLNX-Neutron-ML2-Sriov-driver unique: 8bea9eb4c36c4144b3c662bc2201043c> is not registered with Gearman 2014-08-19 16:05:16,227 INFO | 15:28 |
*** lukego has quit IRC | 15:28 | |
omrim | fungi: what is the reason for the error: "is not registered with Gearman" | 15:29 |
isaacb_ | anyone has experience with dsvm-tempest-full? | 15:29 |
jeblair | isaacb_: lots of us do, but you might want to ask a more specific question. | 15:30 |
*** baohua has quit IRC | 15:30 | |
*** e0ne has quit IRC | 15:31 | |
*** lcheng has joined #openstack-infra | 15:32 | |
*** atiwari has quit IRC | 15:32 | |
fungi | jeblair: lxsli: they weren't strong concerns, but this is a case of something else in another part of tripleo's cloud communicating with the worker over the backend network, applicable to a very limited set of jobs | 15:32 |
fungi | so it seemed preferable to only apply that to the specialized workers which run them | 15:32 |
lxsli | fungi jeblair: I think other cores felt the alternative (new kind of node) was less appealing than this wafer-thin change | 15:33 |
*** e0ne has joined #openstack-infra | 15:33 | |
fungi | but i agree our puppet (if that's the best place to address it) would require some fairly complex added layering to accomplish it | 15:33 |
lxsli | fungi: the devtest script runs on the Jenkins node whereas the hole needs to be added from the nodepool node | 15:33 |
jeblair | lxsli: the jenkins node is the nodepool node | 15:34 |
fungi | lxsli: the jenkins slave and the nodepool node are the same machine | 15:34 |
*** hashar has quit IRC | 15:34 | |
jeblair | lxsli: so it seems like devtest could just run an iptables command to add the hole | 15:34 |
*** Ish__ has joined #openstack-infra | 15:34 | |
jeblair | much like devstack does similar things | 15:35 |
lxsli | two secs have fetched help, I think I may have used incorrect terminology | 15:36 |
lxsli | by the nodepool node, I meant the libvirt pool node | 15:37 |
*** praneshp has joined #openstack-infra | 15:39 | |
*** mestery_afk has quit IRC | 15:39 | |
phschwartz | fungi: Awesome for the follow up. We are watching though just in case | 15:40 |
*** isaacb_ has quit IRC | 15:41 | |
*** gyee has joined #openstack-infra | 15:41 | |
fungi | phschwartz: so no clue what may have been fixed? i always worry about problems which quietly vanish :/ | 15:41 |
*** che-arne has joined #openstack-infra | 15:42 | |
*** afazekas has joined #openstack-infra | 15:42 | |
phschwartz | fungi: No changes at all on our end as we had not located an issue yet. | 15:42 |
fungi | spooky | 15:42 |
phschwartz | Worries me too | 15:42 |
lxsli | fungi jeblair: does that make more sense? | 15:43 |
*** dimtruck is now known as zz_dimtruck | 15:44 | |
*** juzuluag has joined #openstack-infra | 15:45 | |
fungi | lxsli: so... the script called directly by the jenkins job (which runs in the same context as our nodepool prep scripts have puppeted the original firewall rules into the boot snapshot) would be a place to make it happen. maybe that's not actually devtest itself... i need to review the job definition | 15:45 |
*** dtantsur is now known as dtantsur|brb | 15:45 | |
*** e0ne has quit IRC | 15:46 | |
fungi | lxsli: the commit message doesn't mention any particular job... what job(s) are going to need that? | 15:48 |
lxsli | The whole check-tripleo pipeline I believe | 15:48 |
lxsli | It's to do with seed standup so it affects everything | 15:48 |
fungi | lxsli: most of the jobs in jenkins_job_builder/config/tripleo.yaml seem to call a custom devstack-gate gate_hook of toci_gate_test.sh | 15:49 |
openstackgerrit | Aishwarya Thangappa proposed a change to openstack-infra/storyboard: Added a subscription_events table & subscription hander https://review.openstack.org/113016 | 15:49 |
fungi | seed standup? | 15:49 |
fungi | i'm not sure i follow. what's the seed? | 15:49 |
fungi | and how does it stand up? | 15:49 |
lxsli | OK, stop me if I'm overexplaining | 15:50 |
lxsli | TripleO is the Openstack-on-Openstack program | 15:50 |
lxsli | designed to allow an openstack system (the overcloud) to be deployed using openstack tooling | 15:50 |
*** IvanBerezovskiy has left #openstack-infra | 15:50 | |
* fungi knows most of the background, just not familiar with the internal implementation details for toci | 15:51 | |
lxsli | to do this it deploys an undercloud, the compute nodes of which run instances which provide the overcloud compute nodes and backplace | 15:51 |
lxsli | *backplane | 15:51 |
lxsli | the seed does the same job for the undercloud, for bootstrap reasons - eventually we hope it'll not be needed | 15:52 |
*** derekh has quit IRC | 15:52 | |
fungi | lxsli: the single_use_slave.pp you're trying to modify in 108737 is (intended to be) used exclusively to configure the context in which the jenkins slave agent is running | 15:52 |
lxsli | so if the seed doesn't stand up due to the race condition the fix dependent on this firewall hole fixes, the undercloud and overcloud can't stand up | 15:52 |
*** loki184 has joined #openstack-infra | 15:53 | |
*** shashankhegde has joined #openstack-infra | 15:54 | |
lxsli | How is the libvirt pool configured then? | 15:54 |
fungi | lxsli: i have no idea what a libvirt pool is | 15:54 |
fungi | lxsli: explanation from the other end... | 15:55 |
*** armax has joined #openstack-infra | 15:55 | |
fungi | the nodepool server connects to the tripleo test cloud api endpoint | 15:55 |
lxsli | that's what lifeless calls it :x | 15:55 |
*** wenlock has quit IRC | 15:55 | |
*** mriedem has quit IRC | 15:55 | |
*** sdague has quit IRC | 15:55 | |
loki184 | fungi: hi, lifeless gave +1 for the patch https://review.openstack.org/#/c/110562/ can you please review the patch | 15:55 |
fungi | the nodepool server then asks that cloud to nova boot an instance from a base image | 15:56 |
fungi | in glance | 15:56 |
*** sdake has joined #openstack-infra | 15:56 | |
*** sdake has quit IRC | 15:56 | |
*** sdake has joined #openstack-infra | 15:56 | |
lxsli | That sounds like it - the "tripleo test cloud" | 15:56 |
*** zz_dimtruck is now known as dimtruck | 15:57 | |
fungi | once that "template" instance is booted, the nodepool server logs into it via ssh and runs a "prep" script which calls other scripts, applies that openstack_project::single_use_slave puppet class and does some other things | 15:57 |
*** chandankumar has joined #openstack-infra | 15:57 | |
lxsli | Nick is agreeing with this | 15:57 |
fungi | then the nodepool server asks the tripleo test cloud to stop that template instance and take a snapshot | 15:57 |
fungi | later, the nodepool server asks the tripleo test cloud to nova boot some arbitrary number of instances from that template snapshot image | 15:58 |
fungi | each of those instances runs a jenkins slave agent, and is registered into a jenkins master as a new slave | 15:58 |
*** Guest36489 has joined #openstack-infra | 15:58 | |
openstackgerrit | Adrian Otto proposed a change to openstack-infra/config: OpenStack Containers Service https://review.openstack.org/115328 | 15:59 |
fungi | these nodepool nodes, which are also jenkins slaves, are assigned a group label which marks them as suitable for running tripleo-specific jenkins jobs | 15:59 |
*** emagana has joined #openstack-infra | 15:59 | |
*** dims has quit IRC | 15:59 | |
*** dims has joined #openstack-infra | 15:59 | |
fungi | when zuul decides a job should be run, the configuration for those jobs (as templated via jenkins-job-builder in the tripleo.yaml) are executed on an available nodepool node | 16:00 |
*** otherwiseguy has joined #openstack-infra | 16:00 | |
*** MarkAtwood has joined #openstack-infra | 16:00 | |
fungi | lxsli: so basically, things can either be preconfigured and baked into the snapshot image for those nodes (via puppet/prep script) or applied at job run time by shell called from the jenkins job | 16:01 |
fungi | but either way, they're both configuring behavior within that same context (the system which is the nodepool node, which is also where the jenkins slave agent is running) | 16:02 |
lxsli | So you're suggesting instead of affecting all nodepool nodes, we could run a shell step in tripleo.yaml to punch the hole? | 16:02 |
*** primemin1sterp has quit IRC | 16:02 | |
*** primeministerp has quit IRC | 16:03 | |
*** ildikov has joined #openstack-infra | 16:03 | |
*** primemin2sterp has quit IRC | 16:03 | |
fungi | possibly, if it's acceptable to do that at job run time rather than having it baked into the snapshot image so that it's already in place as each nodepool node is booted | 16:03 |
*** primeministerp has joined #openstack-infra | 16:03 | |
*** wenlock has joined #openstack-infra | 16:03 | |
*** mriedem has joined #openstack-infra | 16:03 | |
*** sdague has joined #openstack-infra | 16:03 | |
*** dims has quit IRC | 16:04 | |
*** alexpilotti has joined #openstack-infra | 16:04 | |
*** mika has quit IRC | 16:04 | |
openstackgerrit | Matt Kovacs proposed a change to openstack-infra/config: Adding project saml-generator https://review.openstack.org/110378 | 16:05 |
*** arnaud has joined #openstack-infra | 16:05 | |
fungi | the current complication with modifying the snapshot as it's built is that all our nodepool nodes derive from a single puppet class, so if you try to modify that behavior in puppet it's going to end up applying to pretty much all nodepool nodes unless we add further conditional indirection in puppet | 16:05 |
*** AlexF has quit IRC | 16:05 | |
*** alexpilotti has quit IRC | 16:06 | |
fungi | because of this we mostly differentiate our nodepool node snapshot image configurations using the various node-label-specific nodepool prep scripts | 16:06 |
*** alexpilotti has joined #openstack-infra | 16:06 | |
fungi | (though differences peculiar to particular operating system distributions are handled deeper in the puppet manifest through platform-specific conditional blocks) | 16:07 |
*** timrc is now known as timrc-afk | 16:08 | |
*** dkranz has quit IRC | 16:08 | |
*** rdopieralski has joined #openstack-infra | 16:09 | |
*** atiwari has joined #openstack-infra | 16:09 | |
fungi | label-specific nodepool prep scripts are specified in http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/templates/nodepool/nodepool.yaml.erb via provider.images.setup parameters | 16:09 |
fungi | and the contents of those scripts are found at http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/nodepool/scripts/ | 16:10 |
*** hashar has joined #openstack-infra | 16:10 | |
*** emagana has quit IRC | 16:11 | |
*** ramashri has joined #openstack-infra | 16:11 | |
*** emagana has joined #openstack-infra | 16:12 | |
*** mfer has quit IRC | 16:12 | |
fungi | so for example the "tripleo-precise" nodes are configured by running prepare_node_tripleo.sh (which is a wrapper around both prepare_node.sh and prepare_tripleo.sh) | 16:12 |
*** emagana has quit IRC | 16:12 | |
*** emagana has joined #openstack-infra | 16:12 | |
*** tomhe has joined #openstack-infra | 16:13 | |
fungi | if the firewall rule needs to be added before the node is booted rather than at the start of the jenkins job, it might make sense to implement it within prepare_tripleo.sh where the other internal network interface configuration is happening | 16:14 |
lxsli | So it looks like my patch https://review.openstack.org/#/c/110653/ is what you're suggesting? | 16:14 |
lxsli | modulo several errors | 16:14 |
*** Guest36489 has quit IRC | 16:14 | |
*** jlibosva has quit IRC | 16:14 | |
*** afazekas_ has joined #openstack-infra | 16:14 | |
lxsli | I can fix the errors and unabandon if that's the correct solution? | 16:14 |
lxsli | We don't care which lands, you're the infra guy, just need this hole :) | 16:15 |
*** pblaho has quit IRC | 16:15 | |
*** tsg has joined #openstack-infra | 16:16 | |
fungi | lxsli: i probably should debate this with clarkb once he's around. i don't think he directly rejected that design any more than i'm directly rejecting the other design, we should just discuss the tradeoffs for each and come to a decision | 16:16 |
jeblair | fungi, lxsli: what about the run-time design? | 16:16 |
fungi | right, we also need to know if both these other designs are actually unneeded (if it can just be done as teh job runs rather than baked into configuration in advance) | 16:17 |
*** sdake has quit IRC | 16:17 | |
*** sdake has joined #openstack-infra | 16:17 | |
*** MaxV has quit IRC | 16:18 | |
zaro | pelix: do you have any suggestions for testing this change? https://review.openstack.org/#/c/85106 | 16:18 |
*** MaxV has joined #openstack-infra | 16:18 | |
*** omrim has quit IRC | 16:19 | |
*** amuller__ has joined #openstack-infra | 16:19 | |
*** sdake has quit IRC | 16:20 | |
*** sdake has joined #openstack-infra | 16:20 | |
*** marun has joined #openstack-infra | 16:21 | |
*** marcoemorais has joined #openstack-infra | 16:21 | |
tomhe | jeblair: I upgraded our internal (non-openstack) Zuul this weekend (coming from March 14 Fix ZUUL_COMMIT in certain conditions) and since the upgrade Zuul fails to detect patchset-created for patch set #1 on changes. Following patch sets (#2, #3, ...) gets detected and put in pipelines. Any hints on how to debug? I don't see any obvious errors in the log files. | 16:22 |
pelix | zaro: thanks for the reminder on that, since with your changes in for testing the cmdline it should be easy to add something to test | 16:22 |
pelix | zaro: could try mocking the recurse_path or the os.walk function to return a predefined set of values and then mock the builder.update_job method to check the final data passed through | 16:22 |
*** MaxV has quit IRC | 16:23 | |
*** amuller_ has quit IRC | 16:23 | |
*** marun has quit IRC | 16:23 | |
zaro | pelix: maybe suggest that to the author? | 16:23 |
*** arnaud has quit IRC | 16:24 | |
*** melwitt has joined #openstack-infra | 16:24 | |
*** marun has joined #openstack-infra | 16:24 | |
pelix | zaro: I will, or if he's busy I'll look at uploading a new version since the current patch is a combination of both our approaches | 16:24 |
*** marun has quit IRC | 16:25 | |
zaro | pelix: sounds good, thanks! | 16:25 |
*** shashankhegde has quit IRC | 16:26 | |
*** julim has joined #openstack-infra | 16:27 | |
zaro | hashar, pelix : this one is confusing me. https://review.openstack.org/#/c/100020 | 16:27 |
*** Guest36489 has joined #openstack-infra | 16:27 | |
*** cody-somerville has quit IRC | 16:28 | |
zaro | seems like it's a dup of https://review.openstack.org/#/c/85336 and this one merged. | 16:28 |
zaro | i'm confused as to what we are doing there. | 16:28 |
pelix | substitution happening in slight different places | 16:29 |
*** Ish__ has quit IRC | 16:30 | |
*** Ish__ has joined #openstack-infra | 16:31 | |
hashar | zaro: https://review.openstack.org/#/c/85336 has been reverted by https://review.openstack.org/#/c/105156/ and this is being reproposed | 16:31 |
pelix | https://review.openstack.org/#/c/103134/ probably gives a better description, but https://review.openstack.org/#/c/85336 introduced a regression which causes problems that weren't caught at the time | 16:31 |
hashar | yeah that caused a regression with my jobs definitions :/ | 16:31 |
hashar | darragh eventually added some hack to make it work. | 16:32 |
pelix | Think we should get https://review.openstack.org/#/c/105155/ in first as that tests the existing behaviour and then go through the original change and see if my hack is really the best solution to the problem | 16:32 |
*** marun has joined #openstack-infra | 16:32 | |
pelix | Sorry wrong change | 16:33 |
*** AlexF has joined #openstack-infra | 16:33 | |
*** markmcclain has joined #openstack-infra | 16:34 | |
openstackgerrit | Alexis Lee proposed a change to openstack-infra/config: Add firewall traffic on port 27410 https://review.openstack.org/110653 | 16:34 |
openstackgerrit | Felipe Reyes proposed a change to openstack-infra/elastic-recheck: Add fingerprint for bug 1358779 https://review.openstack.org/115306 | 16:35 |
uvirtbot | Launchpad bug 1358779 in openstack-ci "swift: no space left on device" [Undecided,New] https://launchpad.net/bugs/1358779 | 16:35 |
lxsli | fungi: updated ^^ please have a look | 16:35 |
lxsli | thanks very much for your help | 16:35 |
lxsli | fungi: https://review.openstack.org/110653 | 16:35 |
zaro | hashar, pelix : should we go ahead and merge https://review.openstack.org/#/c/103134 then? i've already gave it a +2. | 16:36 |
*** SumitNaiksatam has quit IRC | 16:36 | |
hashar | zaro: yeah I think both are fine | 16:37 |
*** SumitNaiksatam has joined #openstack-infra | 16:37 | |
hashar | 103134 is an integration test I wrote to highlight the regression | 16:37 |
*** datsun180b has quit IRC | 16:37 | |
fungi | lxsli: have you inquired as to why this needs to be done during the image build rather than when the jobs are run? | 16:37 |
*** datsun180b has joined #openstack-infra | 16:37 | |
hashar | zaro: so if integration test from 103134 pass that means parent change https://review.openstack.org/#/c/100020/ is fine :D | 16:37 |
pelix | hashar: I think you meant https://review.openstack.org/#/c/105157/ | 16:37 |
pelix | that has the regression test without the hack ;-) | 16:38 |
hashar | oh | 16:38 |
hashar | see I am confused | 16:38 |
lxsli | fungi: tripleo sees the node as a black box so it's hard to do at runtime | 16:38 |
pelix | I think we need to cherry-pick https://review.openstack.org/#/c/100020 and https://review.openstack.org/#/c/103134 | 16:39 |
openstackgerrit | Ladislav Smola proposed a change to openstack/requirements: Update xstatic-rickshaw to >= 1.5.0 https://review.openstack.org/110948 | 16:39 |
*** jedimike has quit IRC | 16:39 | |
fungi | lxsli: i'm still lacking sufficient detail there... you're saying that the socket needs to be reachable early during the node boot process rather than when the job starts? | 16:39 |
jeblair | tomhe: do you see any activity in the zuul debug log related to the first patchset created? if not, you might enable debug logging for the gerrit zuul module to see exactly what zuul is receiving from gerrit | 16:40 |
jeblair | tomhe: [logger_gerrit] | 16:40 |
jeblair | tomhe: level=DEBUG | 16:40 |
lxsli | fungi: from the devtest script, we don't know where the libvirt node is so we don't know that we need to punch a hole | 16:40 |
jeblair | tomhe: in the zuul logging config (we leave ours set to INFO normally because it's verbose) | 16:40 |
lxsli | fungi: usually the script runs on the same node as the libvirt instance but in CI it doesn't | 16:41 |
*** Ish__ has quit IRC | 16:41 | |
tomhe | jeblair: Thanks, I'll enable more debugging and see what I can find. | 16:41 |
fungi | lxsli: right, which is why i was suggesting to run it in the job rather than wherever devtest script is running | 16:41 |
openstackgerrit | Alexis Lee proposed a change to openstack-infra/config: Add firewall traffic on port 27410 https://review.openstack.org/110653 | 16:42 |
*** timrc-afk is now known as timrc | 16:42 | |
fungi | lxsli: the jenkins slave is the same instance you're configuring by changing the puppet manifest | 16:42 |
*** RicoLin has quit IRC | 16:42 | |
lxsli | so for dummies, in the tripleo.yaml? | 16:42 |
pelix | zaro hashar: https://review.openstack.org/#/c/100020 could go in now, may need rebasing to apply (doc changes), https://review.openstack.org/#/c/103134 should either be added after a new submission of https://review.openstack.org/#/c/85336 including the hack from https://review.openstack.org/#/c/105155/ (or a alternative approach), or combined with the previous two. | 16:42 |
fungi | lxsli: right, or in the gate_hook script being called from there | 16:43 |
*** amuller__ is now known as amuller | 16:43 | |
*** gokrokve_ has joined #openstack-infra | 16:43 | |
jeblair | or whatever the gate_hook calls | 16:43 |
*** mmaglana has quit IRC | 16:43 | |
hashar | pelix: will attempt to look at it later in the evening. In meeting right now then dinner time :D | 16:43 |
fungi | lxsli: or possibly implemented in a pre_test_hook if needed to happen before devstack setup | 16:43 |
pelix | hashar: no worries, I'm still not back up to speed since my holiday ;-) | 16:43 |
*** RicoLin has joined #openstack-infra | 16:44 | |
hashar | pelix: I have been busy upgrading our Zuul version :D | 16:44 |
jeblair | hashar: did you see the 'merge-check' stuff i added? | 16:44 |
hashar | jeblair: havent see. Mind adding me to reviewers ? :) | 16:45 |
*** tomhe has quit IRC | 16:45 | |
*** SumitNaiksatam has quit IRC | 16:45 | |
*** e0ne has joined #openstack-infra | 16:46 | |
*** e0ne has quit IRC | 16:46 | |
*** pballand has joined #openstack-infra | 16:46 | |
*** SumitNaiksatam has joined #openstack-infra | 16:46 | |
*** Ish__ has joined #openstack-infra | 16:47 | |
*** gokrokve has quit IRC | 16:47 | |
*** RicoLin has quit IRC | 16:48 | |
jeblair | hashar: already merged | 16:49 |
zaro | pelix, hashar : for clarification, the sequence is.. 100020 -> 85336 and 105155 -> 103134 ? | 16:49 |
lxsli | fungi: ok put up yet another alternative :) https://review.openstack.org/#/c/115341/ | 16:49 |
jeblair | hashar: i'll point you to it in just a sec | 16:49 |
lxsli | is that what you were thinking? | 16:49 |
*** RicoLin has joined #openstack-infra | 16:49 | |
*** markmcclain has quit IRC | 16:49 | |
jeblair | hashar: http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/zuul/layout.yaml#n234 | 16:50 |
jeblair | hashar: it performs a merge-check of every open change in a project each time a change merges | 16:50 |
*** tonytan4ever has quit IRC | 16:50 | |
*** _nadya_ has quit IRC | 16:51 | |
*** markvoelker has quit IRC | 16:51 | |
openstackgerrit | Alexis Lee proposed a change to openstack-infra/config: Add firewall traffic on port 27410 https://review.openstack.org/110653 | 16:51 |
homeless | hi guys, can anyone review this -> https://review.openstack.org/#/c/113598/ | 16:52 |
openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Fix typo in merge-check pipeline description https://review.openstack.org/115344 | 16:53 |
*** e0ne has joined #openstack-infra | 16:53 | |
*** arnaud has joined #openstack-infra | 16:54 | |
*** sarob_ has joined #openstack-infra | 16:54 | |
fungi | lxsli: yes, that was what i think jeblair and i were suggesting. that way it applies at job run-time specifically where it's needed rather than globally to all nodepool nodes for any provider environment | 16:54 |
clarkb | so much sb and mail to catch up on | 16:55 |
pelix | zaro: yes - took a sec to parse :) | 16:55 |
*** markmcclain has joined #openstack-infra | 16:55 | |
lxsli | fungi: ok fantastic, thanks again for your help, please could you +1? | 16:55 |
*** emagana has quit IRC | 16:56 | |
*** amotoki has quit IRC | 16:56 | |
jeblair | rcarrillocruz: are you re-working 109816? | 16:56 |
hashar | jeblair: thanks for the pointer, will have to look at that merge-check carefully :] For now commute, dinner time | 16:56 |
*** amotoki has joined #openstack-infra | 16:56 | |
*** emagana has joined #openstack-infra | 16:56 | |
hashar | zaro: pelix: send myself a reminder. Will attempt to look at the JJB patches tonight. | 16:56 |
*** harlowja_away is now known as harlowja | 16:56 | |
hashar | for now sorry gotta rush | 16:56 |
*** hashar has quit IRC | 16:57 | |
*** rdopieralski has quit IRC | 16:57 | |
clarkb | fungi lxsli my concern with that one change is that it was complicated. diff of 45 lines compared to 3 | 16:57 |
clarkb | doing the change at job runtime is fine too | 16:57 |
*** ihrachyshka has joined #openstack-infra | 16:58 | |
lxsli | clarkb: ok thanks :) | 16:58 |
krtaylor | does anyone know media wiki templates? I am really close to being able to set color in a table cell created by a template (CI system status) | 16:58 |
openstackgerrit | James E. Blair proposed a change to stackforge/gertty: Add unified diff view https://review.openstack.org/115166 | 16:58 |
fungi | lxsli: except for the syntax error | 16:59 |
krtaylor | AJaeger isn't on - anyone else? | 16:59 |
*** maxbit has quit IRC | 17:00 | |
lxsli | fungi: yep fixed already :x | 17:00 |
*** emagana has quit IRC | 17:01 | |
*** paulrad has quit IRC | 17:02 | |
*** maxbit has joined #openstack-infra | 17:02 | |
*** mriedem1 has joined #openstack-infra | 17:02 | |
*** arosen1 has joined #openstack-infra | 17:02 | |
*** arosen1 has quit IRC | 17:02 | |
*** arnaud has quit IRC | 17:02 | |
jeblair | fungi: do you know how 110562 (windows-diskimage-builder) is going to be tested? | 17:02 |
fungi | lxsli: closer, but i think it may still be wrong. can you see my latest comment and confirm? | 17:02 |
*** bswartz has left #openstack-infra | 17:02 | |
jeblair | fungi: afaik, windows is not one of our supported platforms | 17:03 |
*** mriedem has quit IRC | 17:03 | |
fungi | jeblair: i haven't the faintest idea... hopefully the tripleo spec indicates what they plan for testability | 17:03 |
*** SumitNaiksatam has quit IRC | 17:03 | |
fungi | checking | 17:03 |
jeblair | fungi: nope, it just says "it runs on windows and uses native windows commands" | 17:03 |
*** e0ne has quit IRC | 17:03 | |
jeblair | https://blueprints.launchpad.net/tripleo/+spec/build-windows-images | 17:03 |
clarkb | Its powershell iirc | 17:04 |
fungi | fun! | 17:04 |
clarkb | so not tested at all by us | 17:04 |
fungi | then i too have the same question as jeblair | 17:04 |
alexpilotti | jeblair fungi: we can maybe take care of it in the Hyper-V CI | 17:04 |
jeblair | then i'm not sure it should be in openstack | 17:04 |
fungi | clarkb: if it's not tested, it's broken! ;) | 17:04 |
alexpilotti | primeministerp ociuhandu: ^^ | 17:04 |
*** paulrad has joined #openstack-infra | 17:04 | |
lxsli | fungi: thanks, Nick thinks we need it both ways so I've pushed a new patchset | 17:05 |
*** dkranz has joined #openstack-infra | 17:05 | |
fungi | lxsli: we probably already allow all traffic on the default output chain | 17:05 |
fungi | lxsli: checking now to confirm | 17:05 |
clarkb | we do | 17:05 |
fungi | oh, clarkb confirms faster than my fingers | 17:05 |
lxsli | Right you are, thanks for that, will change | 17:06 |
*** hdd has quit IRC | 17:07 | |
zaro | hashar, pelix: thanks. will review soon, hopefully we can just move it thru. | 17:07 |
zaro | clarkb: you in office anymore? | 17:07 |
*** shashankhegde has joined #openstack-infra | 17:07 | |
*** e0ne has joined #openstack-infra | 17:07 | |
clarkb | zaro I am. I should be in today | 17:08 |
*** hogepodge has quit IRC | 17:08 | |
clarkb | https://review.openstack.org/#/c/100632/ didnt have gate jobs run /me togglesci | 17:09 |
fungi | jeblair: so if it's a new development effort outside of openstack, it doesn't seem like it's directly addressing an openstack spec. but i guess it's up to tripleo if they want to have specs which are just "this should be possible, but it's not something we can do officially" | 17:09 |
*** paulrad_ has joined #openstack-infra | 17:09 | |
jeblair | fungi: yeah, i think the latest ps puts it in 'openstack/' though, under the tripleo program | 17:10 |
clarkb | looks like check was still in progress and it didnt promote it because? | 17:10 |
*** johnthetubaguy is now known as zz_johnthetubagu | 17:10 | |
jeblair | fungi: if it were in stackforge, i'd butt-out :) | 17:10 |
fungi | jeblair: i butted in when it was a project proposed to stackforge claiming to implement an official openstack spec | 17:10 |
fungi | dilemma | 17:10 |
*** paulrad has quit IRC | 17:10 | |
jeblair | fungi: heh, yeah. i agree that's also awkward and should be discussed. but i think "tripleo should support this use case via external tooling" is an answer that doesn't run afoul of any governance issues. | 17:12 |
fungi | clearly we just need microsoft to set all the windows source code free. then this would cease to be an issue | 17:12 |
lxsli | https://review.openstack.org/#/c/115341 is looking good in CI, it's run the iptables command :) | 17:12 |
jeblair | (if that were in fact to be the answer) | 17:12 |
jeblair | fungi: that would work | 17:12 |
*** loki184 has quit IRC | 17:12 | |
*** e0ne has quit IRC | 17:12 | |
jeblair | clarkb: it says 'started gate'; did it not actually end up in the queue? | 17:12 |
clarkb | jeblair that is after the reverify | 17:13 |
clarkb | it didnt start gate after the approval earlier | 17:13 |
jeblair | clarkb: the one from ihar? | 17:13 |
fungi | clarkb: that started shortly before the zuul restart and was not reenqueued into check after | 17:13 |
jeblair | there was a zuul restart yesterday? | 17:14 |
*** dtantsur|brb is now known as dtantsur | 17:14 | |
fungi | jeblair: no | 17:14 |
alexpilotti | fungi: I cannot provide you the source code for WIndows, but we can do those tests and build the images in tehe Microsoft CI :-) | 17:14 |
fungi | 08-12 22:11 was when it reacted to the vmware recheck comment, and then | 17:14 |
fungi | we restarted zuul just after that | 17:14 |
fungi | and so it had no jenkins verify vote on it when it got approved | 17:15 |
clarkb | jeblair yes ihar | 17:15 |
*** jpich has quit IRC | 17:15 | |
fungi | thus jobs didn't trigger again until the reverify | 17:15 |
jeblair | fungi: oh, gotcha | 17:15 |
*** annegentle has quit IRC | 17:15 | |
fungi | so that behavior is easily explained | 17:15 |
clarkb | because it was verify 0? | 17:16 |
jeblair | i don't know what a cloudkitty is, but i feel like i'd be much happier if i could use python-cloudkittyclient for something | 17:16 |
*** mmaglana has joined #openstack-infra | 17:16 | |
*** AlexF has quit IRC | 17:17 | |
fungi | clarkb: yep | 17:17 |
*** dmsimard is now known as dmsimard_away | 17:17 | |
clarkb | thanks | 17:17 |
*** bdpayne has joined #openstack-infra | 17:17 | |
fungi | clarkb: at least the way we have things configured now (and ever since the clean check implementation) is that approve only triggers jobs if there is a jenkins vertify vote value other than 0 | 17:17 |
fungi | verify | 17:18 |
fungi | actually probably only if there is a +1 in fact | 17:18 |
fungi | checking | 17:18 |
clarkb | ya and removing clean check will fix that right? | 17:18 |
fungi | ahh, right, if there's any nonzero jenkins verify vote >24 hours, it will enqueue into check | 17:19 |
*** tomhe has joined #openstack-infra | 17:20 | |
jeblair | yeah, not sure we can do "on workflow+1 with no vote from jenkins, enqueue into check" | 17:20 |
jeblair | oh right | 17:20 |
*** bdpayne has quit IRC | 17:20 | |
jeblair | jhesketh wrote negative requirements, they might be able to do that | 17:21 |
*** marekd has left #openstack-infra | 17:21 | |
jeblair | (but last i checked that change depended on the multi-user change which was still in review) | 17:21 |
*** _nadya_ has joined #openstack-infra | 17:21 | |
tomhe | jeblair: http://pastebin.com/rudfJZvm ERROR zuul.GerritEventConnector: Exception moving Gerrit event: Exception: Change 12471,1 not found | 17:21 |
jeblair | tomhe: what gerrit version are you using? | 17:21 |
tomhe | 2.9 | 17:22 |
*** bdpayne has joined #openstack-infra | 17:22 | |
fungi | ooh, some new breakage we can look forward to i guess | 17:22 |
jeblair | tomhe: what happens if you run "gerrit query --format json --all-approvals --comments --commit-message --current-patch-set --dependencies --files --patch-sets --submit-records 12471" | 17:22 |
jeblair | tomhe: i'm wondering if there's a timing issue or something | 17:23 |
primeministerp | alexpilotti: ping | 17:24 |
*** emagana has joined #openstack-infra | 17:24 | |
linkid | jeblair: oh, thank you for your review for cloudkitty's repos :) | 17:24 |
*** Guest36489 has quit IRC | 17:24 | |
jeblair | fungi, tomhe: that's also the same behavior we saw a couple times when querying some old changes in openstack's gerrit | 17:24 |
alexpilotti | primeministerp: jeblair and fungi were discussing about the WIndows images build script | 17:25 |
tomhe | jeblair: looks good to me: http://pastebin.com/mrKsLNmR | 17:25 |
*** markmcclain has quit IRC | 17:25 | |
*** markmcclain has joined #openstack-infra | 17:25 | |
primeministerp | alexpilotti: the one we already have on the cbs github repos? | 17:25 |
alexpilotti | primeministerp: I thought it was a good topic for Microsoft :-) | 17:25 |
primeministerp | alexpilotti: definately | 17:25 |
alexpilotti | primeministerp: this is the stuff that HP contributed | 17:25 |
primeministerp | alexpilotti: do we need licenses | 17:25 |
alexpilotti | primeministerp: it’s using cloudbase-init | 17:25 |
fungi | primeministerp: alexpilotti: no, from an official upstream perspective we can't (and won't) test on proprietary platforms | 17:26 |
jeblair | tomhe: do you have a secondary search index configured? | 17:26 |
*** AlexF has joined #openstack-infra | 17:26 | |
*** loki184 has joined #openstack-infra | 17:26 | |
primeministerp | fungi: fair enough | 17:26 |
alexpilotti | fungi: how do you plan to build those images? | 17:27 |
fungi | primeministerp: alexpilotti: though a third-party ci could perform testing on such a thing, we can't probably have something which can only be run/tested on a proprietary platform as an official openstack project | 17:27 |
*** AlexF has quit IRC | 17:27 | |
fungi | alexpilotti: i don't think we did plan to. tripleo had a blueprint approved to add windows disk image building support to disk-image-builder | 17:27 |
tomhe | jeblair: Not sure. Probably not. | 17:27 |
fungi | alexpilotti: so they might have some input on how they expected to test it | 17:28 |
jeblair | tomhe: so it's starting to seem like a race condition, where zuul got an event, did a query, and got no results, but now, much later, you can do the query and get results | 17:28 |
jeblair | tomhe: does this happen with other changes? | 17:28 |
primeministerp | fungi: I was aware of the tripleo folks interest in windows/ironic support | 17:28 |
jeblair | tomhe: oh, you already said so earlier... nm. | 17:29 |
primeministerp | fungi: as far as I knew the hp folks were the one looking into doing some testing as well | 17:29 |
primeministerp | fungi: but I have to admit, those discussions happen sporatically over hk->atl | 17:29 |
fungi | alexpilotti: primeministerp: jeblair: one question i have though, is would the argument be somehow different if there was a feature within the openstack/disk-image-builder project which was not testable on officially-supported platforms (and got tested via third-party ci). if so, this is not especially dissimilar to nova's vmware hypervisor, or the slew of proprietary drivers in neutron/cinder | 17:29 |
openstackgerrit | François Magimel proposed a change to openstack-infra/config: Added python-cloudkittyclient to stackforge projects https://review.openstack.org/112507 | 17:29 |
tomhe | jeblair: Happens to most changes. I've seen some that Zuul picked up, but most gets lost on patch set #1 | 17:29 |
fungi | alexpilotti: primeministerp: jeblair: and so does having it implemented as disjoint code within an official openstack project, some of which can be tested in upstream ci, differ from having it split into a separate git repository and why (philosophy time!) | 17:30 |
jeblair | fungi: i suspect that would be less of an issue because at least the whole project would not require the proprietary platform | 17:30 |
jeblair | tomhe: what zuul commit did you say you were on? | 17:31 |
jeblair | (before you upgraded to master) | 17:31 |
tomhe | jeblair: Just checked and we are using LUCENE as secondary index | 17:31 |
*** hogepodge has joined #openstack-infra | 17:33 | |
tomhe | jeblair: before upgrade we were on bb1fe50c6f32042625cc602dd5b55f76c9d1d384 | 17:33 |
*** adalbas has quit IRC | 17:33 | |
openstackgerrit | François Magimel proposed a change to openstack-infra/config: Added cloudkitty-dashboard to stackforge projects https://review.openstack.org/112789 | 17:33 |
*** emagana has quit IRC | 17:34 | |
*** emagana has joined #openstack-infra | 17:35 | |
*** loki184 has quit IRC | 17:37 | |
*** ChuckC has quit IRC | 17:37 | |
openstackgerrit | James E. Blair proposed a change to openstack-infra/zuul: DO NOT MERGE - test race condition in gerrit query https://review.openstack.org/115357 | 17:38 |
jeblair | tomhe: would you be interested in running that change locally to collect some more data? ^ | 17:39 |
*** chandankumar has quit IRC | 17:39 | |
tomhe | jeblair: sure, will do | 17:39 |
*** emagana has quit IRC | 17:39 | |
tomhe | jeblair: also thought to try what happens with a sleep() right before self.updateChange(change). Will test your change first though. | 17:40 |
*** paulrad_ has quit IRC | 17:41 | |
*** pcrews has quit IRC | 17:41 | |
jeblair | tomhe: yeah, that's pretty equivalent; i mostly did it this way to try to characterize if it is a race, how slow is it. :) | 17:41 |
mtreinish | clarkb: have a few secs to push some buttons in the gerrit gui and perform the manual steps from: http://git.openstack.org/cgit/openstack-infra/config/commit/modules?id=fe5f398d02f758b17958da1c07f8feb1f0418f56 | 17:41 |
jeblair | tomhe: (also doing the query first should mean at least one failure, so we can still be sure we're seeing the problem) | 17:41 |
*** pcrews has joined #openstack-infra | 17:41 | |
*** tonytan4ever has joined #openstack-infra | 17:42 | |
*** lukego has joined #openstack-infra | 17:44 | |
*** loki184 has joined #openstack-infra | 17:44 | |
*** emagana has joined #openstack-infra | 17:45 | |
clarkb | mtreinish as soon as I get to the office sure | 17:45 |
clarkb | mtreinish is tempest ready for tox upgrade? | 17:45 |
mtreinish | clarkb: ok cool, thanks | 17:45 |
mtreinish | clarkb: should be we landed the hash seed = 0 patch | 17:46 |
clarkb | perfect | 17:46 |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: git: Support timeout option https://review.openstack.org/111280 | 17:46 |
*** sirushti has joined #openstack-infra | 17:47 | |
*** _nadya_ has quit IRC | 17:47 | |
*** crc32 has joined #openstack-infra | 17:47 | |
*** _nadya_ has joined #openstack-infra | 17:48 | |
*** otherwiseguy has quit IRC | 17:50 | |
jeblair | fungi: i think puppet-lint may be failing everything | 17:50 |
jeblair | http://logs.openstack.org/94/113194/3/gate/gate-config-puppet-lint/96d10b0/console.html | 17:51 |
*** adalbas has joined #openstack-infra | 17:51 | |
*** otherwiseguy has joined #openstack-infra | 17:52 | |
jeblair | do we get anything useful out of puppet lint? | 17:52 |
jeblair | clarkb, fungi: ^? | 17:52 |
linkid | jeblair: I had this error too and I did a rebase to see it was ok, then. But it seems that files which had failed were not modify, I think | 17:52 |
*** dims has joined #openstack-infra | 17:53 | |
openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Remove puppet lint https://review.openstack.org/115361 | 17:53 |
clarkb | jeblair rcarrilocruz mentioned that. there is a new version | 17:53 |
jeblair | clarkb: yes, i understand. i'm confirming that we can merge nothing because of it. | 17:53 |
jeblair | also, i think we need the puppet apply jobs for the new puppet modules | 17:53 |
jeblair | because otherwise we'll lose our best puppet testing as we move them out | 17:53 |
clarkb | we get consistency in our format other than that the other tests help more | 17:54 |
clarkb | ++ | 17:54 |
*** atiwari has quit IRC | 17:55 | |
*** mfink has joined #openstack-infra | 17:55 | |
jeblair | clarkb, fungi: ^ well you have a choice in front of you: hit +2 on my change or go fix a bunch of whitespace ;). i'm not actually partial to which solution we do. | 17:55 |
*** loki184 has quit IRC | 17:55 | |
jeblair | biab | 17:55 |
tomhe | jeblair: Should I expect Zuul to always detect the patch set with this? Tried a couple of times and I see no error. | 17:55 |
jeblair | tomhe: no, it should still fail the first time | 17:55 |
jeblair | tomhe: i mean, it should log a message at debug level | 17:56 |
tomhe | Ok, will try some more. | 17:56 |
jeblair | tomhe: you should see a msg like "Change 12471,1 not found" in the debug logs at least once, then it should succeed | 17:56 |
tomhe | jeblair: Yes, looked for "retrying" | 17:57 |
jeblair | yeah, that one :) | 17:57 |
*** mfink has quit IRC | 17:57 | |
*** _nadya_ has quit IRC | 17:58 | |
*** radez is now known as radez_g0n3 | 17:58 | |
yolanda__ | clarkb, jeblair, the dib change got finally merged? | 17:59 |
tomhe | jeblair: Nothing yet. Time for a break to put my kids to bed. Will debug more later. | 17:59 |
openstackgerrit | A change was merged to openstack/requirements: Fix ordering of requests_mock package https://review.openstack.org/115104 | 18:00 |
*** mfer has joined #openstack-infra | 18:00 | |
*** ChuckC has joined #openstack-infra | 18:00 | |
fungi | jeblair: yes, new puppet-lint today, and according to rcarrillocruz it now is even more picky about rocket alignment whitespace than before, so we probably want to pin it | 18:01 |
fungi | oh, clarkb also just said that | 18:02 |
*** chuckC_ has joined #openstack-infra | 18:02 | |
*** bdpayne has quit IRC | 18:03 | |
*** sarob_ has quit IRC | 18:04 | |
*** doude has quit IRC | 18:04 | |
fungi | yeah, i think i'm in favor of eliminating puppet-lint for now. if they relax it again upstream, make it more flexible in what it enforces, or someone wants to go through and make a huge whitespace-only change to get it working again, we can always add it back later | 18:04 |
*** radez_g0n3 is now known as radez | 18:05 | |
*** _nadya_ has joined #openstack-infra | 18:05 | |
fungi | it's worth expecting that we'll backslide on our current conformance between now and then as well | 18:05 |
clarkb | can we just pin it? | 18:06 |
clarkb | keep what we had | 18:06 |
*** _nadya_ has quit IRC | 18:07 | |
clarkb | I think the current zuul backslide is not a good thibg | 18:07 |
*** lcheng has quit IRC | 18:07 | |
fungi | clarkb: want to propose a pin in thick_slave.pp? (assuming the gem package provider supports that)? | 18:08 |
*** sirushti has left #openstack-infra | 18:08 | |
fungi | looks like it must, since we already pin rake that way | 18:09 |
clarkb | ya I can do that | 18:09 |
*** pradk has quit IRC | 18:09 | |
fungi | either way, we'll probably have to manually merge it, since the change will be deadlocked | 18:09 |
*** prad has joined #openstack-infra | 18:10 | |
fungi | and with the pin, we have to rebuild all our bare-{precise?,trusty?} images | 18:10 |
fungi | so i think i'm in favor of removing the job until tomorrow | 18:10 |
fungi | at least | 18:10 |
*** hdd has joined #openstack-infra | 18:10 | |
*** annegentle has joined #openstack-infra | 18:10 | |
*** paulrad has joined #openstack-infra | 18:13 | |
*** abhishekk has joined #openstack-infra | 18:16 | |
*** che-arne has quit IRC | 18:16 | |
*** cnesa7 has quit IRC | 18:16 | |
abhishekk | fungi:hi, you around? | 18:17 |
*** emagana has quit IRC | 18:17 | |
fungi | abhishekk: what's up? | 18:17 |
abhishekk | fungi:having one doubt | 18:17 |
*** emagana has joined #openstack-infra | 18:17 | |
jeblair | clarkb: can you -1/+3 https://review.openstack.org/#/c/115361/ ? | 18:18 |
abhishekk | fungi:does devstack use http://pypi.openstack.org/openstack/ mirror to install libraries? | 18:18 |
*** paulrad has quit IRC | 18:18 | |
fungi | abhishekk: no, it uses http://pypi.openstack.org/simple/ | 18:18 |
clarkb | jeblair: I am going to propose a pin for it now | 18:19 |
*** dmsimard_away is now known as dmsimard | 18:19 | |
fungi | abhishekk: at least in the official openstack ci. if you're asking in regards to your own ci system or test environment then you probably want to use something else, like https://pypi.python.org/ | 18:20 |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/config: Move rally-dsvm-fakevirt-heat to experimental https://review.openstack.org/114387 | 18:20 |
jeblair | clarkb: regardless, can you please either -1 or +3 https://review.openstack.org/#/c/115361/ ? | 18:20 |
fungi | clarkb: we still have to wait for image rebuilds if we pin it | 18:20 |
mgagne | for whoever asked to be able to disable some puppet lint checks with a per-line flag: http://puppet-lint.com/controlcomments/ | 18:21 |
abhishekk | fungi:so in nova's requirement.txt websockify version is mentioned as >=0.5.1,<0.7.0 then will devstack install webscockify 6.0? | 18:21 |
fungi | clarkb: so in the meantime, being able to get work done would be cool | 18:21 |
clarkb | fungi: right and that change won't actually pass until after the pin | 18:21 |
*** emagana_ has joined #openstack-infra | 18:21 | |
clarkb | I guess I will go ahead and approve it then work the pin back in | 18:21 |
*** weshay_mtg has quit IRC | 18:21 | |
fungi | clarkb: the pin change won't pass either | 18:21 |
clarkb | right | 18:21 |
*** bdpayne has joined #openstack-infra | 18:21 | |
*** weshay has joined #openstack-infra | 18:21 | |
fungi | unless we stop running the job for now | 18:21 |
jeblair | clarkb: if i had some idea from you as to whether you were okay with https://review.openstack.org/#/c/115361/ i would be happy to force merge it. :) | 18:21 |
*** kmartin has joined #openstack-infra | 18:21 | |
*** emagana has quit IRC | 18:21 | |
clarkb | fungi: I am reading your comment and agreeing | 18:21 |
clarkb | jeblair: +2 force merge sounds good to me | 18:22 |
*** jp_at_hp has quit IRC | 18:22 | |
fungi | clarkb: got it. conversational lag ;) | 18:22 |
*** dtantsur is now known as dtantsur|afk | 18:22 | |
fungi | abhishekk: presumably, yes. did it not? | 18:22 |
openstackgerrit | A change was merged to openstack-infra/config: Remove puppet lint https://review.openstack.org/115361 | 18:22 |
abhishekk | fungi:it's still installing minor version | 18:23 |
*** mriedem has joined #openstack-infra | 18:23 | |
abhishekk | fungi:i.e websockify 0.5.1 | 18:23 |
*** dmsimard is now known as dmsimard_away | 18:24 | |
fungi | abhishekk: have a link to a log you're looking at? | 18:24 |
*** mriedem1 has quit IRC | 18:24 | |
*** lukego has quit IRC | 18:25 | |
abhishekk | fungi:I am installing it locally with latest code | 18:25 |
fungi | mgagne: is it possible to disable the more restrictive rocket alignment checking in today's new release? | 18:25 |
mgagne | fungi: checking | 18:25 |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Pin puppet lint to 0.3.2. https://review.openstack.org/115370 | 18:25 |
fungi | abhishekk: oh, so not in our ci. then you probably configured your system to use a mirror, possibly our old one. you should unconfigure that | 18:25 |
jeblair | ttx: can you take a look at 113833 | 18:26 |
mgagne | fungi: PuppetLint.configuration.send('disable_arrow_alignment') | 18:26 |
mgagne | fungi: http://puppet-lint.com/checks/arrow_alignment/ | 18:26 |
fungi | abhishekk: look in your .pip/pip.conf and ~/.pydistutils.cfg | 18:26 |
abhishekk | fungi:can you help me little bit on how to change the mirror | 18:26 |
abhishekk | fungi:ok, I have a look on the same | 18:26 |
mgagne | that's in the Rakefile, I'm sure you can create a .puppet-lint.rc file somewhere | 18:26 |
*** bdpayne has quit IRC | 18:27 | |
fungi | abhishekk: if you remove any index_url/index-url lines in those, pip should go back to using normal pypi.python.org | 18:27 |
mgagne | they removed the class_parameter_defaults check too | 18:27 |
abhishekk | fungi:thank you, i will check it, brb | 18:27 |
*** radez is now known as radez_g0n3 | 18:28 | |
*** bdpayne has joined #openstack-infra | 18:28 | |
fungi | mgagne: interesting. anyway we noticed that the new version they just released decided to complain about additional space now before => so if your longest parameter has more than one space after it before the rocket, fail | 18:28 |
*** chandankumar has joined #openstack-infra | 18:28 | |
*** pelix has quit IRC | 18:28 | |
mgagne | fungi: lot of fun ^^' | 18:29 |
fungi | mgagne: i guess if you want to stop it from complaining about that, you have to remove the alignment check entirely now | 18:29 |
jeblair | i have no idea why we think this sort of thing is so critically important we are willing to waste time with it instead of doing real work | 18:29 |
clarkb | mgagne: fungi we want arrow alignment though | 18:29 |
*** bdpayne has quit IRC | 18:29 | |
clarkb | problem is the rule changed rather than adding a new rulke | 18:30 |
* fungi wonders who "we" is in this case, but okay ;) | 18:30 | |
jeblair | clarkb: you may be using the "royal we" :) | 18:30 |
mgagne | lets not open a pandora's box =) | 18:30 |
clarkb | in any case linting doesn't actually catch many actual issues | 18:30 |
clarkb | so I am fine with it disabled | 18:30 |
jeblair | pretty sure mordred opened that one years ago with pep8, and i'll never forget him for it :) | 18:30 |
clarkb | in fairness flake8 does catch some real issues :) | 18:32 |
mgagne | anyone still affected by bug #1277538 ? it's damn annoying | 18:32 |
jeblair | clarkb: pyflakes does catch real issues. | 18:33 |
uvirtbot | Launchpad bug 1277538 in tempest "Non-zero exit code (2) from test listing" [Undecided,In progress] https://launchpad.net/bugs/1277538 | 18:33 |
*** bdpayne has joined #openstack-infra | 18:33 | |
*** hdd has quit IRC | 18:35 | |
*** dims has quit IRC | 18:37 | |
linkid | do you think I have to recheck my commit, due to the puppet-lint, or is it ok ? | 18:38 |
jeblair | linkid: if it got a -1 because of lint, you should 'recheck' | 18:38 |
linkid | I rechecked it after a -1 but before lint was out | 18:39 |
linkid | I will recheck it, then | 18:39 |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Tox fixed the openstack breaking bug use latest https://review.openstack.org/108515 | 18:39 |
clarkb | fungi: jeblair ^ is the first half of tomorrow's upgrade fun | 18:39 |
* clarkb preps precise to trusty default change now | 18:39 | |
*** hdd has joined #openstack-infra | 18:40 | |
*** gokrokve has joined #openstack-infra | 18:40 | |
openstackgerrit | François Magimel proposed a change to openstack-infra/config: Added cloudkitty-dashboard to stackforge projects https://review.openstack.org/112789 | 18:42 |
jeblair | stevebaker: can you review https://review.openstack.org/#/c/114387/ please? | 18:43 |
jeblair | stevebaker: (it's a heat related infra change) | 18:43 |
*** gokrokve_ has quit IRC | 18:44 | |
lifeless | lxsli: libvirt pool, whats that? | 18:44 |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Set default test node to trusty for all projects. https://review.openstack.org/115376 | 18:45 |
tomhe | jeblair: Uploaded somewhere around 20 changes with no misses yet. I'll keep monitoring tomorrow when under normal system load when people are back to work (Europe time here) and I'll let you know what I find. | 18:45 |
clarkb | fungi: jeblair ^ and that is the other | 18:45 |
*** e0ne has joined #openstack-infra | 18:45 | |
*** gokrokve has quit IRC | 18:45 | |
jeblair | tomhe: okay, make sure you are still logging at debug level in the gerrit logger (or maybe change that log line to .info) | 18:46 |
jeblair | tomhe: to make sure you're not accidentally missing it | 18:46 |
ttx | jeblair: done | 18:47 |
tomhe | jeblair: didn't change the logging. But I'll change to info and run some more changes. | 18:48 |
*** Sukhdev has joined #openstack-infra | 18:48 | |
*** lyxus has joined #openstack-infra | 18:49 | |
*** emagana_ has quit IRC | 18:50 | |
*** david-ly_ has joined #openstack-infra | 18:50 | |
lyxus | Hi, I am trying to do a gerrit query to find the the latest patch for the last X hours. Does anyone has this thing handy ? | 18:51 |
*** david-lyle has quit IRC | 18:51 | |
jeblair | lyxus: "-age:Xh" | 18:52 |
lyxus | I came up with this age:6h AND project:openstack/neutron AND branch:master but it will show every single patchset | 18:52 |
*** SumitNaiksatam has joined #openstack-infra | 18:52 | |
*** lukego has joined #openstack-infra | 18:52 | |
jeblair | lyxus: i think that's older than 6h, '-age' should be newer than 6h | 18:53 |
*** emagana has joined #openstack-infra | 18:53 | |
*** erlon has joined #openstack-infra | 18:55 | |
lyxus | jeblair, interesting on the jenkins plugin the -age doesn't work but it work on the webinterface | 18:57 |
*** atiwari has joined #openstack-infra | 18:58 | |
*** amuller has quit IRC | 18:59 | |
*** dims has joined #openstack-infra | 19:00 | |
*** dims has quit IRC | 19:00 | |
fungi | meeting time, once keystone wraps up | 19:00 |
*** dims has joined #openstack-infra | 19:01 | |
*** _nadya_ has joined #openstack-infra | 19:02 | |
jeblair | meeting time for realz | 19:02 |
fungi | really for reals | 19:02 |
tomhe | jeblair: Switched back to "pre your patch" and got the problem right away. Reset to your patch and no Exceptions, no errors. Ran serveral new changes. So it appears that the for loop you added changes the timings enough to always suceed with the query. | 19:03 |
jeblair | tomhe: wow, that suggests an extremely tight race | 19:03 |
*** abhishekk has quit IRC | 19:03 | |
*** homeless has quit IRC | 19:03 | |
tomhe | jeblair: I'll run with your code tomorrow, but maybe with 30 s instead of 300 and see how that goes. | 19:04 |
*** homeless has joined #openstack-infra | 19:04 | |
*** hdd has quit IRC | 19:05 | |
*** dims has quit IRC | 19:05 | |
*** Longgeek has quit IRC | 19:06 | |
*** gyee has quit IRC | 19:08 | |
*** Ish__ has quit IRC | 19:10 | |
malini | jeblair: Can you please review https://review.openstack.org/#/c/115291/ when you have a minute? will help us unblock zaqar gate | 19:10 |
clarkb | malini: can you make sure that my change to devstack-gate is complete as well? | 19:11 |
*** ramashri has quit IRC | 19:11 | |
malini | clarkb: sure! | 19:11 |
clarkb | malini: I wasn't entirely sure if features.yaml's update was completely correct depending on how you go about it | 19:11 |
* clarkb digs up link | 19:11 | |
clarkb | malini: https://git.openstack.org/cgit/openstack-infra/devstack-gate/commit/?id=340d2388d2e0a9d63ed1e0fcc113c0561132968e | 19:12 |
malini | thanks clarkb! checking now | 19:13 |
*** harlowja has quit IRC | 19:13 | |
*** harlowja has joined #openstack-infra | 19:13 | |
*** david-ly_ is now known as david-lyle | 19:13 | |
*** tonytan4ever has quit IRC | 19:14 | |
openstackgerrit | Adam Gandelman proposed a change to openstack-infra/config: Consolidate and rename Ironic jobs https://review.openstack.org/112134 | 19:14 |
*** Guest64178 is now known as med_ | 19:14 | |
*** med_ has joined #openstack-infra | 19:14 | |
*** med_ is now known as medberry | 19:14 | |
yolanda__ | clarkb, jeblair, sorry to push so much. Can you give me status of nodepool-dib? working on documentation updates right now, but wanted to be sure that change was merged and working | 19:15 |
clarkb | yolanda__: we need to restart nodepool but your latest change merged | 19:15 |
malini | clarkb: We have renamed the marconi-server to zaqar-server. So https://review.openstack.org/#/c/114771/1/features.yaml will need to be updated..I'll submit a patch | 19:16 |
clarkb | malini: thanks | 19:16 |
yolanda__ | clarkb, but not tested yet? i saw comments about that yesterday, about some problems with UpdateImage but wasn't sure what was about | 19:16 |
malini | thank you clarkb! | 19:16 |
*** ihrachyshka has quit IRC | 19:16 | |
clarkb | yolanda__: we are over in openstack-meeting right now, but I am sure jeblair or fungi can fill in about yesterday's stuff | 19:16 |
*** hashar has joined #openstack-infra | 19:17 | |
*** Ish__ has joined #openstack-infra | 19:17 | |
*** tonytan4ever has joined #openstack-infra | 19:17 | |
*** markmcclain has quit IRC | 19:17 | |
yolanda__ | ok, jeblair, fungi, please let me know if there is some issue with the changes. I'll be sending a patch for documentation tomorrow | 19:18 |
jroll | is there a way to change the owner on a review? | 19:23 |
*** ildikov_ has joined #openstack-infra | 19:23 | |
clarkb | jroll: I don't think so. initial uploader is the owner | 19:23 |
jroll | clarkb: boo, thanks | 19:23 |
fungi | yolanda__: other than an oversight in the image-update cli subcommand, nothing else is wrong that i'm aware of... but we haven't restarted nodepool since merging that either, so i expect to know more once we do (maybe this afternoon?) | 19:23 |
*** emagana has quit IRC | 19:24 | |
clarkb | fungi: yolanda__ I am happy to give it a go again this afternoon | 19:24 |
asselin_ | yolanda__, there was an issue with nodepool image-update. I submitted a patch that fixed it: https://review.openstack.org/#/c/115125/ | 19:24 |
*** ildikov has quit IRC | 19:24 | |
*** gokrokve has joined #openstack-infra | 19:24 | |
*** emagana has joined #openstack-infra | 19:24 | |
fungi | thanks again for spotting (and fixing) that asselin_! | 19:24 |
yolanda__ | oh, let me check | 19:24 |
*** emagana has quit IRC | 19:25 | |
asselin_ | fungi, glad to "pay back" you're all very helpful. | 19:25 |
*** paulrad has joined #openstack-infra | 19:25 | |
yolanda__ | asselin_ , thx for fixing that silly mistake | 19:26 |
yolanda__ | that way | 19:26 |
yolanda__ | you see that i only tested with the "all" keyword | 19:26 |
*** rcarrillocruz has quit IRC | 19:26 | |
openstackgerrit | Malini Kamalambal proposed a change to openstack-infra/devstack-gate: Update Marconi to Zaqar https://review.openstack.org/115384 | 19:26 |
asselin_ | yeah, that was my workaround :) | 19:26 |
*** hdd has joined #openstack-infra | 19:27 | |
*** rcarrillocruz has joined #openstack-infra | 19:28 | |
*** _nadya_ has quit IRC | 19:28 | |
*** shashankhegde has quit IRC | 19:30 | |
*** tsg has quit IRC | 19:30 | |
*** malini has quit IRC | 19:32 | |
*** lukego has quit IRC | 19:32 | |
*** emagana has joined #openstack-infra | 19:34 | |
*** RicoLin has quit IRC | 19:34 | |
*** ihrachyshka has joined #openstack-infra | 19:35 | |
*** tonytan4ever has quit IRC | 19:37 | |
*** dmsimard_away is now known as dmsimard | 19:38 | |
*** dmsimard is now known as dmsimard_away | 19:39 | |
openstackgerrit | A change was merged to openstack-infra/config: Bump postgres module version https://review.openstack.org/113561 | 19:42 |
*** tonytan4ever has joined #openstack-infra | 19:45 | |
*** dhellman_ has joined #openstack-infra | 19:46 | |
*** signed8bit has joined #openstack-infra | 19:49 | |
*** adalbas has quit IRC | 19:49 | |
notmyname | what version of gerrit is being used? | 19:49 |
*** marun has quit IRC | 19:49 | |
clarkb | notmyname: 2.8 + a backported patch of a change in 2.9 | 19:49 |
notmyname | clarkb: thanks | 19:49 |
*** markmcclain has joined #openstack-infra | 19:50 | |
*** dhellman_ has left #openstack-infra | 19:51 | |
phschwartz | Is there a way to do the opposite of jjb and pull the jobs out to yaml files for an existing jenkins server? | 19:52 |
clarkb | phschwartz: no there isn't | 19:52 |
phschwartz | :( | 19:52 |
phschwartz | darn | 19:52 |
*** lttrl has quit IRC | 19:52 | |
phschwartz | That would be a great tool. lol | 19:52 |
clarkb | it would be hard to write | 19:53 |
*** unicell has quit IRC | 19:53 | |
phschwartz | Yeah, I know. I was hoping though | 19:53 |
notmyname | clarkb: do you have any idea why our gerrit doesn't work with the "is:mergeable" search? | 19:53 |
clarkb | notmyname: I think all of that stuff is new in 2.9 | 19:53 |
*** dmsimard_away is now known as dmsimard | 19:54 | |
*** Ryan_Lane has joined #openstack-infra | 19:54 | |
clarkb | notmyname: jeblair is working on zuul based merge checks though | 19:54 |
clarkb | which should be in now | 19:54 |
openstackgerrit | A change was merged to openstack-infra/config: Add jshint job for horizon as non-voting https://review.openstack.org/113825 | 19:54 |
notmyname | clarkb: ah, yes. it would seem that a search for "gerrit 2.8 docs" did indeed take me to the 2.9 docs | 19:54 |
*** tsg has joined #openstack-infra | 19:54 | |
*** shashankhegde has joined #openstack-infra | 19:55 | |
*** daya_k has quit IRC | 19:55 | |
*** dmsimard is now known as dmsimard_away | 19:56 | |
*** chandankumar has quit IRC | 19:56 | |
*** dmsimard_away is now known as dmsimard | 19:58 | |
*** RicoLin has joined #openstack-infra | 19:58 | |
*** rlandy has quit IRC | 19:59 | |
*** stevebaker has quit IRC | 20:00 | |
*** stevebaker has joined #openstack-infra | 20:00 | |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Use new d-g GRENADE flags in jenkins jobs. https://review.openstack.org/111853 | 20:01 |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Add ironic and neutron sideways upgrade jobs https://review.openstack.org/112431 | 20:01 |
clarkb | jeblair: SergeyLukjanov ^ thank you for catching the derp in 111853 I have no idea where that came from | 20:01 |
clarkb | but those should be good to go now | 20:01 |
pleia2 | clarkb: when you have some time today, there are some issues with the pootle install, if you want to add my ssh key to the server I can poke around, but I think we might have some permissions issues - if you nagivate to the site you get a spinning progress thing, that started last night during our demo when we tried to upload .po files | 20:01 |
nibalizer | if you 'check experimental' does zuul just do it and not respond? | 20:02 |
fungi | notmyname: to be on the safe side, i usually read the docs bundled in our gerrit... https://review.openstack.org/Documentation/index.html | 20:02 |
nibalizer | i.e. im kind of expecting a 'ya im on it, boss' | 20:02 |
clarkb | nibalizer: correct you can check the zuul status | 20:02 |
nibalizer | okay cool | 20:02 |
*** kgiusti has left #openstack-infra | 20:02 | |
nibalizer | just making sure | 20:02 |
clarkb | pleia2: hrm let me look | 20:02 |
pleia2 | so uploading didn't work, and had to do the demo over on another server dwayne had | 20:02 |
pleia2 | all worked out ok, just oops broken :) | 20:02 |
anteaya | I welcome input in the third party scaling problem | 20:03 |
pleia2 | I'm working with the zanata folks now on getting a wildfly instance up on one of my HPCloud VMs | 20:03 |
openstackgerrit | Elizabeth K. Joseph proposed a change to openstack-infra/config: Add third party announce and request mailing lists https://review.openstack.org/115392 | 20:03 |
fungi | nibalizer: the *only* reason zuul currently responds with a comment when enqueuing into the check pipeline is to support freshness and clean-check use cases (since gerrit didn't consider simply removing a vote to be a significant enough event to update a timestamp) | 20:04 |
anteaya | just as a data point, I'm guessing now is not the time | 20:04 |
anteaya | and i should get back to slides | 20:04 |
fungi | nibalizer: we're looking forward to ripping that back out soon, i believe | 20:04 |
pleia2 | anteaya: I put your email address in, I can show you how to do list admin (happy to be another admin too, but didn't want to put my email in there since you're the leader here) | 20:04 |
nibalizer | fungi: oh wow | 20:04 |
*** penguinRaider has quit IRC | 20:04 | |
*** amalagon has joined #openstack-infra | 20:04 | |
clarkb | pleia2: glad it worked out and sorry it didn't work on our server | 20:05 |
nibalizer | im not sure i reccomend that, i think zuul telling everybody that its on it is a good feature | 20:05 |
anteaya | pleia2: fair enough | 20:05 |
clarkb | fungi: I think we can rip it out today | 20:05 |
pleia2 | clarkb: no worries, I should have poked around our server a bit more before the demo | 20:05 |
anteaya | yes, I will need to do the list admin tutorial? | 20:05 |
clarkb | fungi: gerrit 2.8 mostly fixed that issue and fixed it for the zuul case | 20:05 |
fungi | clarkb: i believe so too. looking back over the change now | 20:05 |
fungi | clarkb: oh, that part | 20:05 |
clarkb | fungi: if you specify a string in the comment then that is enough to update the timestamp | 20:05 |
anteaya | can we do it friday, can you be list admin until then? I present on thursday | 20:05 |
clarkb | and zuul adds a comment | 20:05 |
clarkb | pleia2: there are tracebacks | 20:08 |
clarkb | pleia2: http://paste.openstack.org/show/97470/ I don't think it is related to permissions issues | 20:09 |
clarkb | unless submissions is on disk? | 20:09 |
pleia2 | hrm :\ | 20:09 |
clarkb | a quick check of the po/ perms shows pootle:nogroup owns everything | 20:10 |
pleia2 | ok | 20:10 |
pleia2 | and we do need the user profile pages working, Daisy wants to look into how stats and things are displayed and those require working profile pages | 20:11 |
*** andreaf_ has joined #openstack-infra | 20:11 | |
pleia2 | maybe I should come up with a list, we need the API enabled too if it's not :) | 20:11 |
*** andreaf_ has quit IRC | 20:12 | |
*** bdpayne has quit IRC | 20:12 | |
clarkb | user profiles work right? (other than the openid issues and this new thing) | 20:12 |
ianw | anteaya: this is just a vague feeling, but i feel like 3rd party CI is really for the 3rd party, not the developer. that was/is my opinion of my external testing -- the things that are core we care about get pushed upstream (centos7 and current fedora, in red hat case). sometimes i feel like that distinction i've made is not universal | 20:12 |
*** andreaf_ has joined #openstack-infra | 20:12 | |
pleia2 | clarkb: http://23.253.231.198/profiles/lyz | 20:12 |
clarkb | if the api is enabled by default we hsould have it running | 20:12 |
*** andreaf_ has quit IRC | 20:12 | |
anteaya | ianw: you are correct | 20:12 |
anteaya | ianw: and that was the conclusion we reached in germany | 20:13 |
*** andreaf_ has joined #openstack-infra | 20:13 | |
pleia2 | clarkb: and the API is disabled by default, need to install django-tastypie and add POOTLE_ENABLE_API = True to config | 20:13 |
anteaya | ianw: the problem is getting these folks from the perspective which they have when they enter openstack to the perspective you just outlined | 20:13 |
anteaya | ianw: *I'm* in agreement with you | 20:13 |
pleia2 | clarkb: http://pootle.readthedocs.org/en/latest/api/ has some details | 20:13 |
anteaya | that doesn't change the fact that there is a lot of work to be done with this group | 20:14 |
anteaya | ianw: some in this space share that perspective _now_ | 20:14 |
anteaya | ianw: but when they come in? | 20:14 |
anteaya | ianw: they usually enter blind and hungry and chased by wolves | 20:14 |
*** andreaf has quit IRC | 20:14 | |
krtaylor | but, I believe, it will be useful to the developer someday | 20:14 |
*** Ish__ has quit IRC | 20:15 | |
krtaylor | it is just painful now | 20:15 |
anteaya | krtaylor: it is useful to the devloper when the operator of the ci comments on their patch | 20:15 |
krtaylor | well, when it works | 20:15 |
anteaya | with the line that is breaking that operators ci | 20:15 |
*** e0ne has quit IRC | 20:15 | |
anteaya | and hopefully with a fix that won't break that operators ci | 20:15 |
anteaya | then is it useful to the developer | 20:16 |
anteaya | we have a long way to go | 20:16 |
krtaylor | but if it does break their ci, then it worked too | 20:16 |
krtaylor | as long as it is reliable | 20:16 |
anteaya | you missed what I said | 20:16 |
krtaylor | that is the key | 20:16 |
anteaya | that is where we are now, we have to take that further for it to be useful to the developer | 20:16 |
krtaylor | agreed | 20:16 |
anteaya | and like ianw for the operator to own their ci | 20:16 |
anteaya | the operator needs to be the one commenting on the developer's patch | 20:17 |
anteaya | the operator has to be the consumer of the results of their ci | 20:17 |
anteaya | then it is useful for the developer | 20:17 |
krtaylor | it is a MUCH bigger commitment than most operators realize | 20:17 |
anteaya | yes | 20:17 |
anteaya | yes it is | 20:18 |
anteaya | but it is still the goal | 20:18 |
krtaylor | I think eventually it can be automated | 20:18 |
krtaylor | no operator in the middle | 20:18 |
anteaya | and we are working towards it as a group (both devs and ci) in incremental steps | 20:18 |
anteaya | krtaylor: then the automation just breaks | 20:18 |
krtaylor | agreed, baby steps | 20:18 |
*** Ish__ has joined #openstack-infra | 20:18 | |
anteaya | but anyway that is incidental since we are a long way from the goal | 20:18 |
krtaylor | I believe it can be made reliable | 20:19 |
anteaya | but moving in that direction | 20:19 |
krtaylor | so true | 20:19 |
anteaya | krtaylor: I'm glad | 20:19 |
anteaya | I don't share your belief but I will support it | 20:19 |
asselin_ | is there a way to see the review status of a particular ci inside gerrit? | 20:20 |
*** markmc has quit IRC | 20:21 | |
openstackgerrit | A change was merged to openstack-infra/config: Update Marconi to Zaqar https://review.openstack.org/115291 | 20:21 |
anteaya | asselin_: which ci, I can show you a query | 20:21 |
asselin_ | any. how abot hp storage ci :) | 20:21 |
asselin_ | *about | 20:21 |
anteaya | here is a start: https://review.openstack.org/#/q/reviewer:%22HP+Storage+CI+%253Chp.cinder.ci%2540external.groups.hp.com%253E%22,n,z | 20:22 |
*** MaxV has joined #openstack-infra | 20:23 | |
clarkb | pleia2: cool I can make that change and restart it | 20:23 |
clarkb | pleia2: I really have no idea what those tracebacks mean | 20:23 |
anteaya | asselin_: and then use this url: http://paste.openstack.org/show/97475/ and replace the ci name in the url with yours | 20:25 |
anteaya | asselin_: if you are happy with the results do share with others | 20:25 |
anteaya | asselin_: and thanks | 20:25 |
pleia2 | clarkb: thanks, and yeah, I don't know either, after it's restarted I can see about trying to upload again and see if we get any new info | 20:27 |
clarkb | pleia2: it is restarted | 20:27 |
pleia2 | hmm, and still showing that progress spinner | 20:28 |
*** cmart13 has joined #openstack-infra | 20:28 | |
clarkb | pleia2: yup I see the same traceback | 20:28 |
*** atiwari has quit IRC | 20:28 | |
pleia2 | ok, I'll poke around their bug tracker | 20:29 |
clarkb | it almost looks like we have a db model and internal model mismatch | 20:29 |
clarkb | maybe their migrations aren't quite working on mysql? /me pokes in the db | 20:29 |
pleia2 | clarkb: oh, profile-wise I see my user is just broken probably due to the openid thing, can see Daisy's page http://23.253.231.198/profiles/daisy-ycguo | 20:30 |
clarkb | pleia2: I can see yours too | 20:31 |
clarkb | pleia2: maybe you can't see yourself when logged in? I can't see myself | 20:31 |
cmart13 | hello (and sorry to bother). Is there any jslint/jshint build check implemented that I could look for? We are trying to add a js check in our builds | 20:31 |
clarkb | cmart13: there is, the storyboard jobs use it iirc | 20:31 |
pleia2 | clarkb: ah, interesting | 20:31 |
clarkb | cmart13: in openstack-infra/config/modules/openstack_project/files/jenkins_job_builder/config you should be able to find it | 20:31 |
*** maxbit has quit IRC | 20:32 | |
cmart13 | clarkb: great! I'll check that. Thanks | 20:33 |
openstackgerrit | Colleen Murphy proposed a change to openstack-infra/config: Update version of puppetlabs-stdlib in install_modules.sh https://review.openstack.org/109351 | 20:33 |
*** dprince has quit IRC | 20:34 | |
clarkb | pleia2: http://paste.openstack.org/show/97478/ I think unit_id being NULL there is why that traceback happens | 20:34 |
*** pdmars has quit IRC | 20:34 | |
*** maxbit has joined #openstack-infra | 20:35 | |
pleia2 | clarkb: odd, I don't know what that should be | 20:35 |
*** gyee has joined #openstack-infra | 20:36 | |
linkid | jeblair: if you have some time, could you take a look to my commits about cloudkitty, please ? (https://review.openstack.org/#/c/112507/ & https://review.openstack.org/#/c/112789/) | 20:36 |
pleia2 | I think I still have my local test vm around somewhere, let me see what it looks like there (not running 2.6, but hopefully it's similar) | 20:37 |
*** dmsimard is now known as dmsimard_away | 20:37 | |
*** dims has joined #openstack-infra | 20:37 | |
pleia2 | but after lunch, the 1AM translations meeting last night is catching up with me :) | 20:37 |
cmart13 | clarkb: I found this: gate-horizon-jshint at horizon. Should I look for horizon code in order to know how it's configured? | 20:39 |
cmart13 | clarkb: *horizon.yaml, in the path you said | 20:39 |
clarkb | pleia2: I think it is a foreign key for pootle_store_unit | 20:40 |
clarkb | pleia2: so you do a submission with a pointer to where the submission is stored? | 20:40 |
clarkb | cmart13: I would look at storyboard instead | 20:40 |
clarkb | cmart13: I think horizon stopped running that test? | 20:40 |
pleia2 | clarkb: not a clue | 20:41 |
*** stevebaker has quit IRC | 20:41 | |
clarkb | pleia2: I double checked and there are no other errors in the log output from pootle | 20:41 |
*** stevebaker has joined #openstack-infra | 20:41 | |
cmart13 | clarkb: I really don´t know.. Where can I find the storyboard jobs? | 20:41 |
pleia2 | clarkb: ok, good to know | 20:41 |
clarkb | cmart13: looks like javascript.yaml? | 20:42 |
*** emagana has quit IRC | 20:44 | |
*** dims has quit IRC | 20:45 | |
*** emagana has joined #openstack-infra | 20:45 | |
jeblair | mordred, SergeyLukjanov, jhesketh, clarkb, fungi: public service announcement: before approving new projects that go into openstack/ make sure that the relevant ptl has signed off on them joining the program. | 20:46 |
jeblair | mordred, SergeyLukjanov, jhesketh, clarkb, fungi: you might want to make sure there is also an openstack/governance change to add it | 20:46 |
fungi | thanks, and agreed--good idea | 20:46 |
jeblair | mordred, SergeyLukjanov, jhesketh, clarkb, fungi: https://review.openstack.org/#/c/106361/8 was approved without ptl approval, and currently the ptl doesn't know if it should be in the program or not | 20:46 |
jeblair | so we may have to yank it back _out_ of openstack/ | 20:47 |
clarkb | jeblair: :/ noted | 20:47 |
fungi | oh, ick | 20:47 |
*** hashar has quit IRC | 20:47 | |
*** Sukhdev has quit IRC | 20:48 | |
cmart13 | clarkb: great! Now I have to find the .jshintrc that is being used, right? | 20:49 |
*** mestery has joined #openstack-infra | 20:49 | |
clarkb | cmart13: maybe? https://git.openstack.org/cgit/openstack-infra/storyboard/tree is where I would start | 20:49 |
jeblair | relevant governance review: https://review.openstack.org/#/c/107585/1 | 20:49 |
*** emagana has quit IRC | 20:49 | |
*** miqui has quit IRC | 20:49 | |
*** mestery has quit IRC | 20:49 | |
*** mestery has joined #openstack-infra | 20:50 | |
*** atiwari has joined #openstack-infra | 20:50 | |
clarkb | pleia2: I wonder if it is trying to write to someplace else on disk and not only where I told it to put po files? | 20:51 |
clarkb | pleia2: then silently failing when that happens? | 20:51 |
clarkb | pleia2: I can drop the db and try again if you like | 20:51 |
pleia2 | clarkb: that might be the best option | 20:51 |
clarkb | I can figure out how to get more verbose logs too | 20:52 |
clarkb | let me go ahead and do that. I assume there isn't anything in that install you need or want to keep? | 20:52 |
pleia2 | nope | 20:52 |
clarkb | hrm looks like I already log at debug or that was the attempt anyways | 20:53 |
*** jgrimm has quit IRC | 20:53 | |
*** denis_makogon_ has joined #openstack-infra | 20:54 | |
*** andreaf_ has quit IRC | 20:54 | |
*** andreaf_ has joined #openstack-infra | 20:54 | |
*** andreaf_ has quit IRC | 20:54 | |
*** sdake has quit IRC | 20:55 | |
*** cmart13 has quit IRC | 20:55 | |
*** bdpayne has joined #openstack-infra | 20:56 | |
*** ramashri has joined #openstack-infra | 20:58 | |
clarkb | pleia2: ok go ahead and register again | 20:59 |
clarkb | I can make you superuser | 21:00 |
*** bdpayne has quit IRC | 21:00 | |
pleia2 | clarkb: lyz user created | 21:00 |
clarkb | you are super user now | 21:00 |
pleia2 | looks good, thanks | 21:01 |
clarkb | I think I made changes to get more debugging now. I see a giant traceback when trying to go to my profile while logged in | 21:01 |
clarkb | ya I can hit our profiles while logged out | 21:01 |
*** weshay has quit IRC | 21:01 | |
clarkb | I am guessing that particular thing is a real pootle bug? | 21:01 |
*** emagana has joined #openstack-infra | 21:02 | |
pleia2 | seems like | 21:02 |
*** bdpayne has joined #openstack-infra | 21:03 | |
*** mbacchi has quit IRC | 21:04 | |
SergeyLukjanov | jeblair, /me noted too, I'm always trying to make sure that it's a legal changee, but was sure about glance.store because of familiar author/committer | 21:04 |
*** IAX has joined #openstack-infra | 21:04 | |
*** juzuluag_ has joined #openstack-infra | 21:05 | |
jeblair | fungi: did you get a chance to try unified diff? | 21:05 |
*** rfolco has quit IRC | 21:06 | |
fungi | jeblair: no, still trying to catch up on e-mail from this morning, and need to go get dinner now. hopefully in a couple hours | 21:06 |
*** juzuluag has quit IRC | 21:07 | |
*** cody-somerville has joined #openstack-infra | 21:07 | |
*** nuritv has joined #openstack-infra | 21:09 | |
SergeyLukjanov | fungi, clarkb, folks, please, could you take a look on https://review.openstack.org/#/c/113833/ ? | 21:09 |
*** lttrl has joined #openstack-infra | 21:10 | |
*** e0ne has joined #openstack-infra | 21:10 | |
*** cody-somerville_ has joined #openstack-infra | 21:11 | |
openstackgerrit | Felipe Reyes proposed a change to openstack-infra/elastic-recheck: Add fingerprint for bug 1358779 https://review.openstack.org/115306 | 21:11 |
uvirtbot | Launchpad bug 1358779 in openstack-ci "swift: no space left on device" [Undecided,New] https://launchpad.net/bugs/1358779 | 21:11 |
*** Sukhdev has joined #openstack-infra | 21:11 | |
*** julim has quit IRC | 21:11 | |
*** aysyd has quit IRC | 21:12 | |
*** cody-somerville has quit IRC | 21:14 | |
*** e0ne has quit IRC | 21:16 | |
*** cody-somerville_ has quit IRC | 21:16 | |
nuritv | Hi, i have a question - i would like to post a question / help request over the mailing list regarding our CI system. is there a specific subject i need to add (i mean the subjects in []) | 21:16 |
*** mfer has quit IRC | 21:20 | |
clarkb | nuritv: I don't think there is a specific answer. In general I would use the project name of the thing you are testing and maybe [infra' | 21:24 |
clarkb | er [infra] | 21:24 |
*** dustins has quit IRC | 21:25 | |
clarkb | pleia2: were you going to try and reproduce? I can give it a go too if you like (I know you are tired) | 21:26 |
clarkb | pleia2: was it just upload a .pot file to a project? | 21:26 |
* clarkb gives it a whirl | 21:26 | |
nuritv | clarkb: thanks you clarkb. and also, should i post it on dev list or infra list? | 21:26 |
*** dmellado has joined #openstack-infra | 21:26 | |
fungi | heading out to grab a quick bite, back shortly | 21:27 |
*** fifieldt has quit IRC | 21:27 | |
clarkb | nuritv: again I think it depends. If your questions are about running the infrastructure definitely do the infra list. If you are looking from feedback from devs on what to test and so on I would use the -dev list | 21:27 |
*** gokrokve has quit IRC | 21:28 | |
nuritv | clarkb: thanks. i just don't want to trash the lists... will start on infra list. thanks again | 21:29 |
*** crc32 has quit IRC | 21:31 | |
*** dkranz has quit IRC | 21:34 | |
*** crc32 has joined #openstack-infra | 21:35 | |
*** crc32 has quit IRC | 21:35 | |
clarkb | wow I cannot figure out how to give pootle a .pot file for a project | 21:36 |
*** mestery_ has joined #openstack-infra | 21:36 | |
clarkb | do I have to do this on disk? | 21:37 |
*** mestery has quit IRC | 21:37 | |
*** malini has joined #openstack-infra | 21:38 | |
*** ominakov_ has quit IRC | 21:40 | |
*** mestery_ is now known as mestery | 21:40 | |
*** fifieldt has joined #openstack-infra | 21:41 | |
clarkb | pleia2: ok I reproduced. my extra logging didn't log anything for this thing | 21:43 |
clarkb | there is a fly out to upload template files after you add templates to a project as a "language" | 21:43 |
*** emagana has quit IRC | 21:43 | |
*** emagana has joined #openstack-infra | 21:44 | |
*** gokrokve has joined #openstack-infra | 21:45 | |
*** crc32 has joined #openstack-infra | 21:45 | |
*** mestery has quit IRC | 21:45 | |
*** signed8bit has quit IRC | 21:46 | |
*** mestery has joined #openstack-infra | 21:46 | |
*** emagana_ has joined #openstack-infra | 21:46 | |
*** emagana has quit IRC | 21:46 | |
clarkb | pleia2: when I try to delete the pot via web ui I get ERROR Error while trying to delete /templates/clarktest//templates/clarktest/nova.pot | 21:49 |
clarkb | pleia2: so there appears to be path things that are broken | 21:49 |
clarkb | the .pot file ends up in the correct place though | 21:49 |
*** gondoi is now known as zz_gondoi | 21:53 | |
*** dimtruck is now known as zz_dimtruck | 21:57 | |
*** bdpayne has quit IRC | 21:58 | |
clarkb | pleia2: I am going to do another round of db drop and retry I think I found a db option that may be partially to blame for some of this | 22:00 |
*** gokrokve has quit IRC | 22:00 | |
*** bdpayne has joined #openstack-infra | 22:00 | |
*** mestery has quit IRC | 22:01 | |
*** annegentle has quit IRC | 22:01 | |
*** zz_dimtruck is now known as dimtruck | 22:02 | |
*** otherwiseguy has quit IRC | 22:03 | |
clarkb | pleia2: well that didn't fix the import via web ui issue | 22:03 |
*** denis_makogon_ has quit IRC | 22:04 | |
mattoliverau | Morning | 22:06 |
*** malini has quit IRC | 22:06 | |
*** malini has joined #openstack-infra | 22:06 | |
clarkb | hi there | 22:06 |
*** sdake has joined #openstack-infra | 22:07 | |
*** sdake has quit IRC | 22:07 | |
*** sdake has joined #openstack-infra | 22:07 | |
*** malini1 has joined #openstack-infra | 22:08 | |
*** malini has quit IRC | 22:08 | |
*** hogepodge has quit IRC | 22:08 | |
clarkb | pleia2: I managed to get stuff working if I manually put the file in place | 22:10 |
clarkb | pleia2: but deletions are still broken too | 22:10 |
*** MaxV has quit IRC | 22:12 | |
*** gokrokve has joined #openstack-infra | 22:12 | |
*** ihrachyshka has quit IRC | 22:14 | |
jeblair | sdague: welcome back :) while you were gone, we got the post-merge mergability-check thing going (it's running now on all projects). when you are settled and have a minute, i'd like to talk about dropping the freshness check: https://review.openstack.org/#/c/114626/ | 22:15 |
clarkb | fungi: jeblair I am completely confused by https://review.openstack.org/#/c/113598 | 22:16 |
clarkb | er sorry jhesketh ^ | 22:16 |
sdague | ok, cool. I'm going to be sort of intermittent this week with linuxcon, at least until later in the week. | 22:16 |
clarkb | why did we change names? | 22:16 |
*** malini1 has quit IRC | 22:16 | |
*** malini has joined #openstack-infra | 22:16 | |
clarkb | that is a backward incompatible change, is there a change in config to deal with that? | 22:16 |
clarkb | sdague: the d-g change for sideways upgrade merged, changes to use it are up for review | 22:17 |
sdague | clarkb: cool. Honestly I just made it through the mailing list backlog on the plane. Haven't even openned gerrit yet :) | 22:17 |
*** HenryG_ has joined #openstack-infra | 22:17 | |
pleia2 | clarkb: sigh, I wonder if this is all due to it being an RC? I didn't imagine it would be this bad, I'll browse bug tracker and see what else I can dig up | 22:17 |
clarkb | fungi: jhesketh I am not a fan of the current setup. What am I missing that patchset 4 didn't solve? | 22:18 |
jeblair | sdague: okay cool; mondays and tuesdays are our biggest days for freshness-check derived load (>3000 nodes/day), so if we have the conversation before the weekend, that would be great | 22:18 |
clarkb | pleia2: its possible, I got the feeling not much was tested after hitting the openid stuff | 22:18 |
clarkb | pleia2: but also possible the docs/config is wrong | 22:18 |
*** gokrokve has quit IRC | 22:19 | |
*** HenryG has quit IRC | 22:19 | |
*** malini has quit IRC | 22:19 | |
*** gokrokve has joined #openstack-infra | 22:19 | |
*** malini has joined #openstack-infra | 22:19 | |
clarkb | fungi: jhesketh: is that inversion with the flag needed because otherwise we can't tell it to remove them? | 22:19 |
clarkb | arg | 22:19 |
pleia2 | clarkb: yeah | 22:19 |
*** otherwiseguy has joined #openstack-infra | 22:20 | |
*** malini has quit IRC | 22:20 | |
*** malini has joined #openstack-infra | 22:21 | |
jeblair | the current merge-check queue is 274 with a 20 minute wait time (a nova change merged); it's not terrible, but we might consider spinning up a zm03 | 22:21 |
*** malini has quit IRC | 22:21 | |
*** malini has joined #openstack-infra | 22:21 | |
jeblair | 261. it took me a while to type that. | 22:21 |
jeblair | but this is a bursty thing, so it's not putting us over the edge in terms of queue processing or anything | 22:22 |
* jeblair thinks it's pretty cool you can watch it on the zuul status page | 22:23 | |
*** juzuluag_ has quit IRC | 22:23 | |
*** gokrokve has quit IRC | 22:24 | |
clarkb | jeblair: ya we may want zm03 especially for feature freeze time period | 22:24 |
clarkb | since we tend to merge a ton of code that week | 22:24 |
mtreinish | jeblair: only 261, I would have assumed it would be far higher than that... | 22:25 |
*** nuritv has quit IRC | 22:25 | |
mtreinish | but I guess it is a slower week | 22:25 |
*** tonytan4ever has quit IRC | 22:26 | |
jeblair | http://graphite.openstack.org/render/?from=-8hours&width=800&height=600&target=color%28stats.gauges.zuul.pipeline.merge-check.current_changes,%20%276b8182%27%29&_t=0.2328968844721383#1408487258930 | 22:26 |
clarkb | nibalizer: https://review.openstack.org/#/c/100363/4/specs/public_hiera.rst what is the symlink for? | 22:26 |
clarkb | nibalizer: at first glance that seems like a bug | 22:26 |
jeblair | looks like it peaked at 560 | 22:26 |
clarkb | nibalizer: because it will make puppet apply clunky | 22:26 |
*** markmcclain has quit IRC | 22:26 | |
jeblair | mtreinish: the theoretical max for that is the set of all open changes in all projects | 22:26 |
*** kmartin has quit IRC | 22:27 | |
nibalizer | clarkb: not sure why that will make puppet clunky, but we symlink them together becuase puppet uses /etc/hiera/hiera.yaml and the 'hirea' command line tool uses /etc/hiera.yaml | 22:27 |
nibalizer | if they are linked there can be no confusion | 22:27 |
clarkb | nibalizer: thats, I don't even | 22:27 |
nibalizer | the hiera command line tool is good for debugging the hierarchy, and can be used with ansible to set the right parameters to the commands it will run on the agents | 22:27 |
clarkb | nibalizer: clunky because you would have to do a symlink then run puppet apply but apparently that is only for the hiera command | 22:28 |
nibalizer | ya puppet doesn't require /etc/hiera.yaml | 22:28 |
clarkb | nibalizer: can puppet be told to use the /etc/hiera.yaml file? | 22:28 |
clarkb | since that is where hiera wants the info? | 22:28 |
nibalizer | maybe? | 22:28 |
openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Add zm03 https://review.openstack.org/115417 | 22:28 |
openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Use zm03 https://review.openstack.org/115418 | 22:28 |
nibalizer | i think in puppet 2.7 'hiera config file' literally is $confdir/hiera.yaml which would imply no | 22:28 |
nibalizer | but im happy to be wrong about that | 22:28 |
mtreinish | jeblair: yeah that makes sense, I just would thought the numbers would have been bigger, but I guess I just didn't have a realistic idea of how many open changes there were | 22:29 |
clarkb | its a separate issue anyways | 22:29 |
*** otter768 has joined #openstack-infra | 22:29 | |
clarkb | nibalizer: also is it not weird that hiera can't really speak to different DBs? | 22:29 |
clarkb | nibalizer: because that is extremely weird to me (its so reliant on a default /etc/hiera.yaml file) | 22:29 |
jeblair | mtreinish: heh, based on gerrit's ui, my internal understanding is that there are more than 25 changes open. ;) | 22:29 |
clarkb | nibalizer: can we tell hiera to use /etc/puppet/hiera.yaml? | 22:29 |
mtreinish | jeblair: heh, I switched it to 100/page on my account | 22:30 |
nibalizer | clarkb: 'use different dbs' what do you mean? | 22:30 |
clarkb | nibalizer: I mean when using the hiera tool I should be able to tell it to talk to /etc/puppet/hiera.yaml without a symlink | 22:31 |
clarkb | nibalizer: the hiera tool shouldn't assume a single source of data | 22:31 |
nibalizer | well hold on | 22:31 |
nibalizer | so hiera.yaml is just a config file | 22:31 |
nibalizer | pointing to where the data is | 22:31 |
clarkb | right | 22:32 |
nibalizer | and you can pick one at runtime with hiera -c myfile.yaml | 22:32 |
clarkb | oh well then can we not make that symlink? | 22:32 |
nibalizer | i dont follow? what do you propose? | 22:32 |
clarkb | nibalizer: put hiera.yaml in /etc/puppet/hiera.yaml. Don't make a symlink. Anything that wants to use the hiera cli tool can use the -c flag | 22:33 |
clarkb | nibalizer: its extremely confusing otherwise | 22:33 |
nibalizer | its extremely confusing otherwise | 22:33 |
clarkb | ya, why would you have two config files that in theory do two different things be the same thing | 22:33 |
nibalizer | you dont want to be debugging 'WHY PUPPET BROKE' and then facepalm 'i was supposed to be passing -c !!!!!?!??!!?!' | 22:33 |
clarkb | nibalizer: when you use -c it is explicit | 22:33 |
greghaynes | nibalizer: needs a few more !'s | 22:33 |
clarkb | and says please talk to my puppet hiera data | 22:34 |
clarkb | whcih is what I want to talk to | 22:34 |
*** sdake has quit IRC | 22:34 | |
nibalizer | the only possible reason to use the hiera command line utility is to debug hiera or MAYBE to look up data to pass to ansible or puppet apply | 22:34 |
nibalizer | clarkb: i dont know what to tell you, puppetlabs should have made them the same file | 22:34 |
nibalizer | they didn't | 22:34 |
nibalizer | everyone just symlinks it | 22:34 |
*** paulrad has quit IRC | 22:34 | |
clarkb | nibalizer: ok if that is the expectation I can live with it | 22:35 |
clarkb | nibalizer: usually when you have a distinction like that its important | 22:35 |
nibalizer | okay | 22:37 |
*** kmartin has joined #openstack-infra | 22:38 | |
*** harlowja has quit IRC | 22:39 | |
*** harlowja has joined #openstack-infra | 22:39 | |
*** prad has quit IRC | 22:39 | |
*** hogepodge has joined #openstack-infra | 22:41 | |
*** AlexF has joined #openstack-infra | 22:42 | |
*** lttrl has quit IRC | 22:42 | |
*** tomhe has quit IRC | 22:43 | |
*** bdpayne has quit IRC | 22:44 | |
*** linkid has left #openstack-infra | 22:47 | |
clarkb | jeblair: fungi jesusaurus nibalizer https://review.openstack.org/#/c/99990/ see comments there | 22:48 |
clarkb | that is a very light -1. Mostly since others have indicated confusion on the why of the thing I htink we should make it more clear | 22:48 |
*** AlexF has quit IRC | 22:48 | |
clarkb | also I think it is worth calling out that we will continuously deploy these modules (unless there is strong opposition to doing it that way) | 22:48 |
*** mriedem has quit IRC | 22:49 | |
jeblair | clarkb: good comments | 22:49 |
*** melwitt has quit IRC | 22:51 | |
*** Ish__ has quit IRC | 22:52 | |
*** mestery has joined #openstack-infra | 22:53 | |
clarkb | jeblair: and commented onyour related spec too | 22:54 |
*** hdd has quit IRC | 22:55 | |
fungi | clarkb: added some clarification to my -1 for 113598. i thought i had updated the review after discussing with homeless in irc, but i seem to have forgotten to do so | 22:55 |
lifeless | ahahaha | 22:55 |
lifeless | so my next pbr patch fails | 22:55 |
lifeless | because it works | 22:55 |
lifeless | http://logs.openstack.org/69/108269/16/check/check-grenade-dsvm/058a91b/logs/old/devstacklog.txt.gz | 22:55 |
lifeless | 2014-08-18 03:02:55.051 | ValueError: git history requires a target version of pbr.version.SemanticVersion(2014.1.3), but target version is pbr.version.SemanticVersion(2014.1.2) | 22:55 |
fungi | lifeless: you win at pbr | 22:56 |
lifeless | 2014-08-18 03:02:55.051 | error in setup command: Error parsing /opt/stack/old/nova/setup.cfg: ValueError: git history requires a target version of pbr.version.SemanticVersion(2014.1.3), but target version is pbr.version.SemanticVersion(2014.1.2) | 22:56 |
lifeless | oops, doublepaste | 22:56 |
clarkb | fungi: oh good points | 22:56 |
lifeless | I wonder how many projects will need fixing | 22:57 |
clarkb | lifeless: why would that fail? shouldn't it update the version to that version? | 22:57 |
clarkb | lifeless: I thought the whole point was that it would be automagic | 22:57 |
lifeless | clarkb: its preversioned | 22:57 |
lifeless | clarkb: preversioned is defined as winning | 22:58 |
clarkb | oh I see | 22:58 |
lifeless | clarkb: perhaps we should change that | 22:58 |
fungi | buncha preverts | 22:58 |
lifeless | clarkb: but it would be - AIUI - quite a conceptual change | 22:58 |
clarkb | lifeless: ya | 22:58 |
clarkb | I grok now | 22:58 |
clarkb | so for preversioned stuff can we turn off semantic versioning? | 22:58 |
clarkb | then remove preversioning | 22:58 |
*** arnaud has joined #openstack-infra | 22:59 | |
lifeless | 2014.1.2-7-g27eea59 <- nova's stable/icehouse | 22:59 |
lifeless | from git describe | 22:59 |
lifeless | clarkb: what do you mean ? | 23:00 |
lifeless | clarkb: I was thinking to just go fix the versions - e.g. stable/icehouse should now say 2014.1.3 in its setup.cfg | 23:00 |
lifeless | since thats the version its working towards | 23:00 |
lifeless | by the preversioning rules | 23:00 |
*** dimtruck is now known as zz_dimtruck | 23:00 | |
clarkb | jeblair: and responded to your comments again. thanks | 23:01 |
clarkb | lifeless: I mean stop supporting pre versioning | 23:01 |
*** bknudson has quit IRC | 23:01 | |
lifeless | clarkb: I see little need for it, but maybe others still want it. | 23:01 |
* fungi still thinks pre-versioned release candidates are a release process albatross that we, as a society, have realized are suboptimal in modern practice (complicated to compare against other relative version numbers, thus a pain for downstream consumers). ttx and i will have to agree to disagree on that point, i think | 23:01 | |
clarkb | lifeless: to get there we have to make pbr forward and backward compat, but then we can make openstack use sane versioning (via pbr semver), then we can remove preversioning | 23:01 |
*** dkliban is now known as dkliban_afk | 23:02 | |
*** mestery_ has joined #openstack-infra | 23:02 | |
*** mestery has quit IRC | 23:02 | |
*** hdd has joined #openstack-infra | 23:03 | |
lifeless | clarkb: you're proposing not enforcing this check if the tree is preversioned | 23:03 |
lifeless | clarkb: I can clearly do that, I'm not sure if its a good idea. | 23:03 |
clarkb | lifeless: correct, but only long enough for us to stop using preversioned | 23:03 |
clarkb | granted as fungi points out ttx will need to buy in | 23:03 |
lifeless | clarkb: we could just fix the projects it impacts - which will be primarily the API servers plus a few other projects that cargo culted preversioning when they didn't want or need it | 23:04 |
lifeless | see e.g. https://review.openstack.org/115426 | 23:04 |
fungi | wow... just received via e-mail from a service provider (the benefits provider my employer contracts with, but irrelevant)... "[foo] will be performing an annual disaster recovery test next weekend to ensure that our products and services remain available to our clients in the event of a natural or man-made disaster. [foo services] will be unavailable at the following times during this annual | 23:04 |
fungi | disaster recovery test. [list of affected services]" | 23:04 |
lifeless | do they get the irony? | 23:05 |
fungi | indeed | 23:05 |
clarkb | lifeless: but won't this be a persistent issue? any time the preversion and the actual version get out of sync it will break. or will this be self gated? | 23:05 |
lifeless | clarkb: so grenade is blocking the pbr change to enforce this. | 23:06 |
lifeless | clarkb: once we get the change into pbr, projects will be told and we won't end up broken again. | 23:06 |
*** mestery_ has quit IRC | 23:07 | |
*** mestery has joined #openstack-infra | 23:07 | |
jhesketh | Morning | 23:08 |
pleia2 | morning jhesketh | 23:08 |
*** otherwiseguy has quit IRC | 23:09 | |
clarkb | lifeless: gothca | 23:09 |
*** otherwiseguy has joined #openstack-infra | 23:09 | |
lifeless | irrespective of the way forward, projects that need a patch like https://review.openstack.org/115427 are already broken | 23:09 |
lifeless | so I'm going to send a note to -dev now | 23:10 |
lifeless | and then look at scripting a bulk check | 23:10 |
lifeless | where's the script to mirror all of openstack | 23:10 |
*** Sukhdev has quit IRC | 23:11 | |
lifeless | clarkb: ^ | 23:12 |
jeblair | lifeless: cache_git_repos.py from the nodepool scripts could do it | 23:12 |
lifeless | jeblair: thanks | 23:12 |
clarkb | lifeless: for project in `ssh -p 29418 review.o.o ls-projects` ; do git clone review.o.o/$project ; done | 23:12 |
*** mestery has quit IRC | 23:12 | |
fungi | lifeless: well, as long as we're relying on prerelease version numbering, we'll have to store the "next expected version" somewhere pbr can find it (since it can't just guess without a lot of additional mechanics describing the numbering plan) and setup.cfg seems like as good a place as any. to change that paradigm, we need release manager buy-in | 23:12 |
*** mestery has joined #openstack-infra | 23:13 | |
lifeless | fungi: its in setup.cfg | 23:13 |
jeblair | clarkb: even shorter :) | 23:13 |
jeblair | clarkb, lifeless: but nicer if you clone from git.o.o | 23:13 |
lifeless | fungi: semver can do roughly equivalent as soon as you tag just one alpha | 23:13 |
fungi | lifeless: hrm, good point | 23:14 |
*** tsg has quit IRC | 23:14 | |
lifeless | fungi: e.g. tag 2014.7.0.0a1 and after that semver knows where you're going. | 23:14 |
*** camunoz_gone is now known as camunoz | 23:14 | |
lifeless | fungi: if we've got it right, that is :) | 23:14 |
lifeless | clarkb: ssh -p 29418 review.openstack.org ls-projects | 23:14 |
lifeless | Gerrit Code Review: ls-projects: not found | 23:14 |
fungi | clarkb: that's basically what i do, except i test for the .git subdir of each and then either remote update+reset --hard or rm -rf and clone fresh | 23:14 |
jeblair | lifeless: 'gerrit ls-projects' | 23:15 |
lifeless | ah thanks | 23:15 |
clarkb | jeblair: can you see my comments on https://review.openstack.org/#/c/110793/ I don't quite think they need a -1 but figured specs review is more conversational so let me know what you think | 23:15 |
fungi | but i'm also religious about using git stash and/or commit to a topic branch if i have work in progress not pushed to gerrit. maybe destructive for anyone not doing that | 23:15 |
*** maxbit has quit IRC | 23:15 | |
*** maxbit has joined #openstack-infra | 23:16 | |
lifeless | I have a separate tree for working in, just gertty has this set of trees :) | 23:16 |
sdague | jeblair: https://review.openstack.org/#/c/114626/ lgtm. I figure if the unpassable gate changes show up later, we can address it then | 23:17 |
fungi | lifeless: yeah, i didn't think about inferring target release from prior prerelease versions--that's an awesome optimization and gets rid of annoying version numbers in setup.cfg entirely | 23:17 |
clarkb | sdague: jeblair: does this mean I can review and approve? | 23:18 |
fungi | lifeless: so basically your dev versioning is a postrelease snapshot until the first prerelease tag in your history, and then you use prerelease versions | 23:18 |
jeblair | sdague: okay; our toolbox is getting pretty big now too -- we could also do things like drop the 72 hour check but keep the 24h one, etc | 23:19 |
clarkb | jeblair: can we do verified: [-2, -1, 0] for check test require approvals? | 23:19 |
sdague | jeblair: yeh | 23:19 |
clarkb | jeblair: to fix the approve on verified = 0 issue? | 23:19 |
sdague | jeblair: that's what I assume we can bring back if it's actually an issue | 23:20 |
sdague | but maybe the merge check will flush most of those | 23:20 |
lifeless | fungi: its a prerelease snapshot of the smallest semver increment | 23:20 |
fungi | clarkb: do we still rely on removing the vote as a locking semaphore to indicate that the change is being tested? | 23:20 |
lifeless | fungi: its never a postrelease snapshot | 23:20 |
jeblair | clarkb: i don't think we can require verified=0; i think with jhesketh's change we might be able to require not veried!=0, which would let us do that. i think i mentioned that earlier. | 23:20 |
fungi | lifeless: oh, also a very good point | 23:20 |
sdague | at any rate, given that we're in node crunch, I'm happy to give it a shot | 23:20 |
clarkb | fungi: ya I think we do | 23:21 |
lifeless | fungi: first commit after 2014.1.0 will be 2014.1.1.dev1.g1235678 | 23:21 |
clarkb | jeblair: oh I see | 23:21 |
lifeless | fungi: so the first tag is to say 'I want a much higher version, thanks' | 23:21 |
clarkb | jeblair: is that because verified: 0 isn't a thing in the gerrit event? | 23:21 |
* clarkb streams events | 23:21 | |
fungi | lifeless: yep. makes total sense now | 23:21 |
jeblair | clarkb: i think in this case, it's not the stream, but rather the query | 23:22 |
jeblair | clarkb: i don't think the lack of an approval shows up as an approval :) | 23:22 |
fungi | lifeless: and we rely on semver to define what the smallest divisible subcomponent is... so keystone 2014.1.2.1 would not really have been allowed | 23:22 |
lifeless | fungi: right | 23:24 |
fungi | lifeless: i guess that means we need to socialize that we should never use an explicit fourth version component as that's a reserved automagic space? | 23:24 |
clarkb | jeblair: it isn't in the stream either | 23:24 |
clarkb | seems like that is a gerrit bug. the comment shows -Verified but there is no Verified: 0 | 23:24 |
*** emagana_ has quit IRC | 23:25 | |
jeblair | clarkb: i think it might be a matter of semantics :) | 23:25 |
*** emagana has joined #openstack-infra | 23:25 | |
fungi | clarkb: gerrit seems to treat setting a vote to 0 as removal of the vote. 0 and null/none are synonymous for it | 23:26 |
clarkb | fungi: ya | 23:26 |
lifeless | fungi: also pbr will just ignore it :) | 23:26 |
clarkb | jeblair: cool looks like you will update the docs publishing spe | 23:26 |
clarkb | *spec | 23:26 |
lifeless | fungi: with +version = 3.4.5.6 | 23:26 |
lifeless | fungi: in setup.cfg | 23:26 |
*** datsun180b has quit IRC | 23:26 | |
lifeless | sdist builds | 23:26 |
fungi | lifeless: i ignore all sorts of things all the time... i'm starting to feel a kinship with pbr now | 23:27 |
lifeless | aking hard links in os-apply-config-3.4.5.dev1.g58760d2... | 23:27 |
*** emagana has quit IRC | 23:27 | |
clarkb | did we restart nodepool today? | 23:27 |
fungi | clarkb: not yet | 23:27 |
clarkb | the node graph looks funny | 23:27 |
fungi | clarkb: it does indeed | 23:27 |
clarkb | fungi: still want to do that? | 23:27 |
fungi | maybe someone else started it? or maybe we emptied very briefly... or maybe we rebooted | 23:27 |
jhesketh | so I'm pretty sure this has been discussed before, but why do we delete and boot new nodes instead of rebuilding them? | 23:27 |
fungi | it wasn't rebooted at least | 23:28 |
*** emagana has joined #openstack-infra | 23:28 | |
clarkb | jhesketh: mordred hasn't finished the nodepool feature to do rebuilds | 23:28 |
fungi | process started on the 15th | 23:28 |
*** gargola has quit IRC | 23:28 | |
jeblair | fungi, clarkb: we should see if it lines up with a zuul merge-check spike | 23:28 |
fungi | clarkb: jeblair: is that gap in the graph... | 23:28 |
fungi | yeah, that's what i was about to surmise | 23:28 |
lifeless | or are you pleased to see me? | 23:28 |
lifeless | boom, tish | 23:28 |
clarkb | wait but merge-check is independent of nodepool | 23:29 |
lifeless | hmm, I might be a little tired still :) | 23:29 |
fungi | clarkb: but it's not necessarily independent of zuul handing out more work | 23:29 |
jhesketh | clarkb: oh, so it's in progress? | 23:29 |
jeblair | we can probably overlay the jobs launched on top of that too... i'll see if i can construct a graph | 23:29 |
clarkb | jhesketh: it is something that is being worked on, but I think mordred ran into some corner cases, like races between deletes and rebuilds (because you eventually do want to delete if you don't need the node) | 23:29 |
fungi | clarkb: if zuul got busy with mergability checks, it may have given jobs a chance to wrap up and nodepool an opportunity to recover a ready pool | 23:30 |
*** hdd has quit IRC | 23:30 | |
clarkb | oh | 23:30 |
*** jpeeler has quit IRC | 23:30 | |
fungi | clarkb: the slope tail leading into the gap is basically the same as what we see when we put zuul into shutdown | 23:31 |
clarkb | jhesketh: I don't see a change pushed for it though, maybe mordred never pushed it | 23:31 |
clarkb | jhesketh: he was talking about it at one time though | 23:31 |
clarkb | mordred: can you update us on your nodepool using nova rebuild stuff? | 23:31 |
jhesketh | okay | 23:31 |
*** atiwari has quit IRC | 23:31 | |
jeblair | clarkb: i don't think mordred is going to be updating us on anything for a few weeks | 23:32 |
*** ociuhandu has quit IRC | 23:32 | |
clarkb | jeblair: ya he was around earlier today but apparently that is that | 23:32 |
* fungi did not know there was a nova rebuild... should learn about this openstack stuff | 23:32 | |
jeblair | ha nice | 23:32 |
jeblair | also, it's kind of an implementation detail that we expect that to be faster | 23:33 |
fungi | clarkb: did he construct a network access terminal in the desert out of sand and human excrement? | 23:33 |
jeblair | (i also think that some minor nova enhancements may be required for the rebuild stuff) | 23:33 |
*** maxbit has quit IRC | 23:33 | |
jeblair | (to be truly and reliably faster) | 23:33 |
*** KurtMartin has joined #openstack-infra | 23:33 | |
clarkb | fungi: apparently | 23:33 |
jeblair | at least, i think phschwartz mentioned something to that effect a while ago | 23:34 |
clarkb | fungi: jeblair how do you guys feel about restarting nodepool now? | 23:34 |
*** ZZelle_ has joined #openstack-infra | 23:34 | |
clarkb | we seem caught up with demand for the current running tests | 23:34 |
fungi | clarkb: we're surprisingly not gobsmacked with node demand to the point we're flatlining at our quota, so probably okay? | 23:35 |
clarkb | I am happy to do it if others feel it is ok | 23:35 |
*** ZZelle has quit IRC | 23:36 | |
*** kmartin has quit IRC | 23:36 | |
clarkb | jhesketh: jeblair: this and the reuse of floating IPs makes we want to rewrite bits of nodepool to have an explicit state machine for each node and each provider | 23:36 |
clarkb | to make it a little easier to reason about where in an allocation we are for a provider or where in the life of a node we are | 23:36 |
jeblair | https://etherpad.openstack.org/p/EyLv8op0Wo | 23:36 |
jeblair | clarkb, fungi: ^ graph | 23:37 |
jeblair | clarkb: it has a state machine; it's stored in the state column of the node table | 23:37 |
openstackgerrit | A change was merged to openstack-infra/config: Remove freshness check https://review.openstack.org/114626 | 23:37 |
clarkb | jeblair: right, but the actual dispatch of actions based on that isn't always clear | 23:38 |
clarkb | jeblair: so I don't think the details of the machine change, but we can make the transitions cleaner | 23:38 |
clarkb | but maybe I am overthinking it | 23:38 |
jeblair | clarkb, fungi: i'm interpreting that graph as supporting evidence the merge check slowed the launching of new jobs sufficiently to affect node use | 23:38 |
jeblair | so we should probably approve that zm03 change :) | 23:38 |
clarkb | jeblair: yup the grey line matches up with the blue one pretty well | 23:39 |
*** KurtMartin has quit IRC | 23:39 | |
clarkb | jeblair: you ok with nodepool restart? | 23:39 |
fungi | jeblair: i agree with your analysis | 23:39 |
jeblair | clarkb: i'm totally cool adding more states, or adding substates (eg, DELETE/jenkins; DELETE/removeip, DELETE/delete, ....) | 23:40 |
jeblair | clarkb: nodepoor restart ok | 23:40 |
jeblair | then we will be noderich | 23:40 |
* clarkb restarts nodepool | 23:41 | |
jeblair | (the red line in the graph is zuul jobs launched per 10 minutes) | 23:41 |
clarkb | afte rconfirming the install | 23:41 |
fungi | nodepoor in spirit, noderich in life | 23:41 |
clarkb | oh right stopped puppet there, /me gets that going again | 23:41 |
*** mestery has quit IRC | 23:41 | |
*** HenryG_ is now known as HenryG | 23:42 | |
jeblair | i except the rate of merge-check falloff depends on the repos in question. ie, slower for nova because it takes almost 30 seconds to do a merge check there | 23:42 |
jeblair | expect even | 23:43 |
*** mestery has joined #openstack-infra | 23:44 | |
fungi | seems probable | 23:44 |
clarkb | arg pip | 23:44 |
*** david-lyle has quit IRC | 23:44 | |
clarkb | it says nodepool's installed version is ancient | 23:44 |
*** david-lyle has joined #openstack-infra | 23:45 | |
*** david-lyle has quit IRC | 23:45 | |
* clarkb does surgery | 23:45 | |
*** mestery has quit IRC | 23:45 | |
jeblair | btw, check out the relationship between build/available during that window, it's almost pure sin/cos | 23:45 |
fungi | i was hoping for kicked pendulum chaos, but not every system is a strange attractor i guess | 23:46 |
openstackgerrit | A change was merged to openstack-infra/config: Add zm03 https://review.openstack.org/115417 | 23:46 |
fungi | anyway, there ^ | 23:47 |
*** kmartin has joined #openstack-infra | 23:47 | |
fungi | who wants to launch it and find out what we've broken in our launch scripts lately? | 23:47 |
clarkb | somehow now we have a version installed that isn't in git | 23:47 |
clarkb | I don't even | 23:47 |
fungi | clarkb: that may have been the local revert jeblair put in plave? | 23:47 |
fungi | place? | 23:47 |
clarkb | was there a local revert? | 23:48 |
clarkb | what I did last week was check out HEAD~1 and isntall that | 23:48 |
clarkb | jeblair: ^ | 23:48 |
fungi | clarkb: yes. we didn't merge a revert of the dib bits, just reverted locally on the nodepool server and stopped puppet | 23:48 |
fungi | (i think?) | 23:48 |
clarkb | fungi: no that wasn't a revert. that was just git checkout HEAD~1 | 23:48 |
clarkb | and pip install that | 23:48 |
fungi | oh, huh | 23:48 |
clarkb | I think the issue here is that we don't have a tag on nodepool | 23:49 |
*** crc32 has quit IRC | 23:49 | |
clarkb | so the versions are hash based and not necessarily increasing | 23:49 |
fungi | ooh, does pip see many of them as older version numbers as a result? | 23:49 |
fungi | we should consider releasing 0.0.1 as our initial commit of any pip-installable project in the future | 23:50 |
fungi | yeah, not much is going to top 9661501 for a random 7-digit hex value | 23:51 |
fungi | (currently you have a 4% chance of constructing a commit pip will view as a higher version number) | 23:52 |
lifeless | interesting nova/stable/havana is setup correctly :) | 23:52 |
fungi | (actually less than that because i think it's only considering hex values with digits in the decimal range) | 23:53 |
lifeless | clarkb: I have a fix for that in pbr | 23:53 |
*** yamahata has joined #openstack-infra | 23:53 | |
*** ociuhandu has joined #openstack-infra | 23:54 | |
lifeless | clarkb: 1758998881ef5e3c81a76910755ce1e33688b9e0 - | 23:54 |
lifeless | clarkb: which is landed but not in a release yet. | 23:54 |
lifeless | fungi: ^ | 23:54 |
*** wenlock has quit IRC | 23:55 | |
fungi | lifeless: oh, neat. glad to see that will eventually cease being an issue | 23:56 |
*** marun has joined #openstack-infra | 23:56 | |
clarkb | fungi: lifeless: I am going to pip uninstall nodepool, remove stuff in dist-packages then reinstall | 23:57 |
clarkb | then we can restart | 23:57 |
clarkb | jeblair: fungi: but ya a 0.0.1 tag would be nice | 23:57 |
fungi | clarkb: seems reasonable. you could probably force it to "downgrade" | 23:57 |
fungi | but however is easier for you | 23:57 |
*** markmc has joined #openstack-infra | 23:57 | |
clarkb | ya that worked starting restart now | 23:58 |
*** sdake has joined #openstack-infra | 23:58 | |
clarkb | it is doing stuff post restart | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!