| *** Ryan_Lane has quit IRC | 00:03 | |
| *** dina_belova has joined #openstack-infra | 00:04 | |
| *** dina_belova has quit IRC | 00:09 | |
| *** annegentle has quit IRC | 00:09 | |
| *** dkliban has quit IRC | 00:13 | |
| *** sarob has joined #openstack-infra | 00:21 | |
| *** sarob has quit IRC | 00:26 | |
| *** changbl has quit IRC | 00:40 | |
| *** changbl has joined #openstack-infra | 00:43 | |
| *** dkliban has joined #openstack-infra | 00:44 | |
| openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/zuul: Allow build info to be updated https://review.openstack.org/41020 | 00:47 |
|---|---|---|
| *** changbl has quit IRC | 00:47 | |
| *** mriedem has quit IRC | 01:02 | |
| *** mberwanger has joined #openstack-infra | 01:02 | |
| *** dina_belova has joined #openstack-infra | 01:05 | |
| *** dina_belova has quit IRC | 01:10 | |
| *** erfanian has quit IRC | 01:12 | |
| *** erfanian has joined #openstack-infra | 01:16 | |
| *** changbl has joined #openstack-infra | 01:20 | |
| *** sarob has joined #openstack-infra | 01:22 | |
| *** sarob has quit IRC | 01:27 | |
| *** jhesketh has quit IRC | 01:30 | |
| *** dkliban has quit IRC | 01:39 | |
| *** jhesketh has joined #openstack-infra | 01:51 | |
| *** dina_belova has joined #openstack-infra | 02:05 | |
| *** dguitarbite has joined #openstack-infra | 02:07 | |
| *** dina_belova has quit IRC | 02:10 | |
| *** yaguang has joined #openstack-infra | 02:11 | |
| *** tian has joined #openstack-infra | 02:21 | |
| *** erfanian has quit IRC | 02:22 | |
| *** sarob has joined #openstack-infra | 02:22 | |
| *** sdake_ has quit IRC | 02:25 | |
| *** sdake_ has joined #openstack-infra | 02:25 | |
| *** sdake_ has quit IRC | 02:25 | |
| *** sdake_ has joined #openstack-infra | 02:25 | |
| *** dguitarbite has quit IRC | 02:26 | |
| *** sarob has quit IRC | 02:27 | |
| *** SergeyLukjanov has joined #openstack-infra | 02:34 | |
| *** cody-somerville_ has joined #openstack-infra | 02:36 | |
| *** cody-somerville has quit IRC | 02:37 | |
| *** erfanian has joined #openstack-infra | 02:38 | |
| jeblair | clarkb, Alex_Gaynor: something went wrong there; I've pulled the jenkins logs i need to figure out what. not sure if it's new plugin behavior -- it may be related to stopping/starting jenkins instead. | 02:41 |
| jeblair | clarkb, Alex_Gaynor: i canceled one of the jobs from jenkins' queue; that unstuck one of the changes; the other one will either need to be unstuck with a new patchset or a zuul restart. | 02:43 |
| *** sgviking has quit IRC | 02:47 | |
| *** sgviking has joined #openstack-infra | 02:56 | |
| *** dina_belova has joined #openstack-infra | 03:06 | |
| jeblair | restarted zuul | 03:07 |
| *** dina_belova has quit IRC | 03:10 | |
| *** cody-somerville_ is now known as cody-somerville | 03:19 | |
| *** annegentle has joined #openstack-infra | 03:20 | |
| *** SergeyLukjanov has quit IRC | 03:21 | |
| *** sarob has joined #openstack-infra | 03:23 | |
| *** sarob has quit IRC | 03:28 | |
| *** dkliban has joined #openstack-infra | 03:46 | |
| *** dkliban has quit IRC | 03:57 | |
| *** dkliban has joined #openstack-infra | 03:58 | |
| *** erfanian has quit IRC | 03:59 | |
| *** rcleere has joined #openstack-infra | 04:03 | |
| *** rcleere has quit IRC | 04:09 | |
| *** dguitarbite has joined #openstack-infra | 04:23 | |
| *** pcrews has joined #openstack-infra | 04:23 | |
| *** sarob has joined #openstack-infra | 04:24 | |
| *** sarob has quit IRC | 04:29 | |
| *** vogxn has joined #openstack-infra | 04:44 | |
| *** mberwanger has quit IRC | 04:51 | |
| *** boris-42 has joined #openstack-infra | 04:55 | |
| *** dina_belova has joined #openstack-infra | 05:09 | |
| *** dina_bel_ has joined #openstack-infra | 05:14 | |
| *** dina_belova has quit IRC | 05:14 | |
| *** dina_belova has joined #openstack-infra | 05:21 | |
| *** boris-42 has quit IRC | 05:23 | |
| *** dina_be__ has joined #openstack-infra | 05:23 | |
| *** dina_bel_ has quit IRC | 05:23 | |
| *** sarob has joined #openstack-infra | 05:24 | |
| *** boris-42 has joined #openstack-infra | 05:25 | |
| *** dina_belova has quit IRC | 05:27 | |
| *** SergeyLukjanov has joined #openstack-infra | 05:28 | |
| *** sarob has quit IRC | 05:29 | |
| *** SergeyLukjanov has quit IRC | 05:37 | |
| *** SergeyLukjanov has joined #openstack-infra | 05:42 | |
| *** nicedice_ has quit IRC | 05:46 | |
| *** nayward has joined #openstack-infra | 05:48 | |
| *** SergeyLukjanov has quit IRC | 06:05 | |
| *** dguitarbite has quit IRC | 06:22 | |
| *** dina_be__ has quit IRC | 06:23 | |
| *** sarob has joined #openstack-infra | 06:25 | |
| *** SergeyLukjanov has joined #openstack-infra | 06:27 | |
| *** dina_belova has joined #openstack-infra | 06:28 | |
| *** sarob has quit IRC | 06:29 | |
| *** SergeyLukjanov has quit IRC | 06:48 | |
| *** afazekas_ has joined #openstack-infra | 06:52 | |
| *** SergeyLukjanov has joined #openstack-infra | 06:56 | |
| *** SergeyLukjanov has quit IRC | 07:01 | |
| *** dina_belova has quit IRC | 07:05 | |
| *** vogxn has quit IRC | 07:06 | |
| *** odyssey4me has joined #openstack-infra | 07:06 | |
| *** dina_belova has joined #openstack-infra | 07:13 | |
| *** SergeyLukjanov has joined #openstack-infra | 07:20 | |
| *** sarob has joined #openstack-infra | 07:25 | |
| *** sarob has quit IRC | 07:31 | |
| *** SergeyLukjanov has quit IRC | 07:35 | |
| *** nayward_ has joined #openstack-infra | 07:36 | |
| *** nayward has quit IRC | 07:39 | |
| *** jpich has joined #openstack-infra | 07:39 | |
| *** koolhead11 has joined #openstack-infra | 07:42 | |
| *** Ryan_Lane has joined #openstack-infra | 07:44 | |
| *** xchu has joined #openstack-infra | 07:47 | |
| *** SpamapS has quit IRC | 07:48 | |
| *** dina_belova has quit IRC | 07:49 | |
| *** dina_belova has joined #openstack-infra | 07:49 | |
| *** derekh has joined #openstack-infra | 07:55 | |
| *** Mikhail has joined #openstack-infra | 07:56 | |
| *** Mikhail is now known as Guest71730 | 07:56 | |
| *** fbo_away is now known as fbo | 07:58 | |
| openstackgerrit | Kui Shi proposed a change to openstack-dev/hacking: H233 regex can not match some print operator https://review.openstack.org/41355 | 08:01 |
| *** morganfainberg is now known as morganfainberg_a | 08:02 | |
| *** BobBall_AFK is now known as Bobba | 08:02 | |
| *** morganfainberg_a is now known as morganfainberg | 08:02 | |
| *** SpamapS has joined #openstack-infra | 08:10 | |
| *** SpamapS has quit IRC | 08:15 | |
| *** SpamapS has joined #openstack-infra | 08:19 | |
| openstackgerrit | Michael Basnight proposed a change to openstack-infra/reviewstats: Fixing minor misspelling https://review.openstack.org/41357 | 08:22 |
| *** sarob has joined #openstack-infra | 08:26 | |
| *** sarob has quit IRC | 08:31 | |
| *** dina_belova has quit IRC | 08:32 | |
| *** dina_belova has joined #openstack-infra | 08:33 | |
| *** CliMz has joined #openstack-infra | 08:37 | |
| *** SergeyLukjanov has joined #openstack-infra | 08:44 | |
| *** vogxn has joined #openstack-infra | 08:46 | |
| *** SergeyLukjanov has quit IRC | 08:55 | |
| *** SergeyLukjanov has joined #openstack-infra | 08:55 | |
| *** ruhe has joined #openstack-infra | 09:08 | |
| *** Ryan_Lane has quit IRC | 09:08 | |
| EmilienM | Hi, could we have reviews from core on https://review.openstack.org/#/c/41128/ ? thank's :) | 09:11 |
| *** SpamapS has quit IRC | 09:13 | |
| *** SpamapS has joined #openstack-infra | 09:19 | |
| *** SpamapS has quit IRC | 09:19 | |
| *** SpamapS has joined #openstack-infra | 09:19 | |
| *** markmc has joined #openstack-infra | 09:20 | |
| openstackgerrit | Tom Fifield proposed a change to openstack-infra/config: Add build job for Japanese Operations Guide https://review.openstack.org/39846 | 09:21 |
| *** nayward_ has quit IRC | 09:22 | |
| *** SpamapS has quit IRC | 09:24 | |
| *** ruhe has quit IRC | 09:24 | |
| *** sarob has joined #openstack-infra | 09:27 | |
| *** sarob has quit IRC | 09:31 | |
| *** SpamapS has joined #openstack-infra | 09:35 | |
| *** vogxn has quit IRC | 09:35 | |
| *** nayward has joined #openstack-infra | 09:36 | |
| *** dina_belova has quit IRC | 09:37 | |
| *** nayward has quit IRC | 09:40 | |
| *** SpamapS has quit IRC | 09:47 | |
| *** SpamapS has joined #openstack-infra | 09:48 | |
| *** ruhe has joined #openstack-infra | 09:49 | |
| *** psedlak has joined #openstack-infra | 09:56 | |
| *** dina_belova has joined #openstack-infra | 09:56 | |
| *** xchu has quit IRC | 10:02 | |
| *** afazekas has quit IRC | 10:07 | |
| *** afazekas_ is now known as afazekas | 10:07 | |
| *** ruhe has quit IRC | 10:09 | |
| *** vipul-away has quit IRC | 10:14 | |
| *** SergeyLukjanov has quit IRC | 10:16 | |
| *** ruhe has joined #openstack-infra | 10:18 | |
| *** SergeyLukjanov has joined #openstack-infra | 10:18 | |
| openstackgerrit | Sergey Kolekonov proposed a change to openstack-infra/jenkins-job-builder: Existing plugins improvements https://review.openstack.org/41375 | 10:20 |
| *** vipul has joined #openstack-infra | 10:23 | |
| *** vogxn has joined #openstack-infra | 10:24 | |
| *** pcm_ has joined #openstack-infra | 10:27 | |
| *** sarob has joined #openstack-infra | 10:27 | |
| *** sarob has quit IRC | 10:32 | |
| *** fbo is now known as fbo_away | 10:39 | |
| *** enikanorov-w has joined #openstack-infra | 10:45 | |
| *** enikanorov-w_ has quit IRC | 10:45 | |
| *** CliMz has quit IRC | 10:50 | |
| *** mjfork has joined #openstack-infra | 10:53 | |
| *** w_ is now known as olaph | 10:54 | |
| *** vogxn has quit IRC | 11:17 | |
| openstackgerrit | Mark McLoughlin proposed a change to openstack/requirements: Allow use of oslo.messaging 1.2.0a1 https://review.openstack.org/41381 | 11:17 |
| *** SpamapS has quit IRC | 11:20 | |
| *** Ryan_Lane has joined #openstack-infra | 11:27 | |
| *** sarob has joined #openstack-infra | 11:28 | |
| *** sarob has quit IRC | 11:32 | |
| *** yaguang has quit IRC | 11:36 | |
| *** woodspa has joined #openstack-infra | 11:40 | |
| *** fbo_away is now known as fbo | 11:41 | |
| *** pabelanger_ has joined #openstack-infra | 11:42 | |
| *** lcestari has joined #openstack-infra | 11:46 | |
| *** pabelanger_ has quit IRC | 11:47 | |
| *** pabelanger_ has joined #openstack-infra | 11:47 | |
| *** pabelanger has quit IRC | 11:47 | |
| *** pabelanger_ is now known as pabelanger | 11:47 | |
| *** pabelanger_ has joined #openstack-infra | 11:48 | |
| *** pabelanger has quit IRC | 11:49 | |
| *** pabelanger has joined #openstack-infra | 11:49 | |
| *** weshay has joined #openstack-infra | 11:51 | |
| *** luis_ has quit IRC | 11:52 | |
| *** rscottcoyle has joined #openstack-infra | 12:01 | |
| sdague | does something else need to happen to get the status.html to flush? | 12:02 |
| *** thomasm has joined #openstack-infra | 12:02 | |
| *** rfolco has joined #openstack-infra | 12:04 | |
| *** ArxCruz has joined #openstack-infra | 12:08 | |
| *** Ryan_Lane has quit IRC | 12:09 | |
| *** dprince has joined #openstack-infra | 12:16 | |
| *** pabelanger has quit IRC | 12:25 | |
| *** sandywalsh has quit IRC | 12:33 | |
| *** pabelanger has joined #openstack-infra | 12:35 | |
| *** dina_belova has quit IRC | 12:42 | |
| *** dina_belova has joined #openstack-infra | 12:44 | |
| *** beagles has joined #openstack-infra | 12:45 | |
| *** dina_belova has quit IRC | 12:45 | |
| *** sandywalsh has joined #openstack-infra | 12:46 | |
| pabelanger | morning | 12:52 |
| pabelanger | So, I see -infra no longer cherry-picks commits with gerrit: https://github.com/openstack-infra/config/commit/87bd7eb02655f66acb149fcb11a6ce4830a9d2d1 | 12:52 |
| pabelanger | was curious if the plan was to try and move back to it or not? | 12:53 |
| *** dkliban has quit IRC | 12:53 | |
| pabelanger | eg: This was a good test and we learned a lot by doing it, but I think what we learned is that using merges and sacrificing some clean git history avoids more problems than it causes. | 12:53 |
| mordred | pabelanger: no, I think we decided that we don't think the workflow headache is worth it - especially when thinking about how zuul does things | 12:54 |
| mordred | yes. what you siad | 12:54 |
| mordred | :) | 12:54 |
| pabelanger | mordred: Ya, looked at the review notes, and clarkb said that in a review | 12:54 |
| openstackgerrit | will soula proposed a change to openstack-infra/jenkins-job-builder: Adding support for the Warnings plugin https://review.openstack.org/40621 | 12:57 |
| *** sarob has joined #openstack-infra | 12:59 | |
| *** mriedem has joined #openstack-infra | 13:00 | |
| *** vogxn has joined #openstack-infra | 13:03 | |
| *** sarob has quit IRC | 13:03 | |
| *** ewindisch has joined #openstack-infra | 13:04 | |
| *** jaypipes has joined #openstack-infra | 13:05 | |
| openstackgerrit | will soula proposed a change to openstack-infra/jenkins-job-builder: Adding support for the Warnings plugin https://review.openstack.org/40621 | 13:08 |
| fungi | just a heads up, i'm mostly awol for a good chunk of the day (christine threw her back out over the weekend, so i'm connecting by wireless modem from medical office lobbies when waiting around) | 13:09 |
| mordred | fungi: lovely. hope her back becomes less suck | 13:11 |
| *** anteaya has joined #openstack-infra | 13:16 | |
| *** ewindisch has quit IRC | 13:17 | |
| mordred | fungi: https://review.openstack.org/#/c/41199/1 and https://review.openstack.org/#/c/40856/2 should be easy to do in a doctor's office | 13:20 |
| *** adalbas has quit IRC | 13:23 | |
| *** dhellmann is now known as dhellmann_ | 13:24 | |
| *** dhellmann_ is now known as dhellmann | 13:24 | |
| *** mgagne has joined #openstack-infra | 13:26 | |
| mordred | jog0: https://review.openstack.org/#/c/40686/ needs a rebase | 13:29 |
| *** jpeeler has joined #openstack-infra | 13:36 | |
| *** adalbas has joined #openstack-infra | 13:39 | |
| *** cppcabrera has joined #openstack-infra | 13:39 | |
| *** jergerber has joined #openstack-infra | 13:39 | |
| *** cppcabrera has left #openstack-infra | 13:39 | |
| EmilienM | mordred: Hi, any chance to review https://review.openstack.org/#/c/41128/ today ? | 13:42 |
| *** ruhe has quit IRC | 13:45 | |
| mordred | EmilienM: done | 13:46 |
| EmilienM | mordred: wow, fast. Thank's ;) | 13:46 |
| openstackgerrit | A change was merged to openstack-infra/jeepyb: Update Launchpad projects for puppet modules https://review.openstack.org/41128 | 13:47 |
| *** dkliban has joined #openstack-infra | 13:47 | |
| *** prad_ has joined #openstack-infra | 13:47 | |
| *** beagles has quit IRC | 13:48 | |
| *** beagles has joined #openstack-infra | 13:49 | |
| *** fifieldt has quit IRC | 13:50 | |
| *** vijendar has joined #openstack-infra | 13:55 | |
| *** ruhe has joined #openstack-infra | 13:58 | |
| *** sarob has joined #openstack-infra | 13:59 | |
| *** changbl has quit IRC | 14:01 | |
| *** dina_belova has joined #openstack-infra | 14:01 | |
| *** sdake has quit IRC | 14:01 | |
| *** sdake has joined #openstack-infra | 14:02 | |
| *** sdake has quit IRC | 14:02 | |
| *** sdake has joined #openstack-infra | 14:02 | |
| openstackgerrit | Sergey Kolekonov proposed a change to openstack-infra/jenkins-job-builder: Existing plugins improvements https://review.openstack.org/41375 | 14:03 |
| *** sarob has quit IRC | 14:04 | |
| openstackgerrit | A change was merged to openstack-infra/config: Add tuskar and tuskarclient to stackforge https://review.openstack.org/40856 | 14:04 |
| openstackgerrit | Russell Bryant proposed a change to openstack-infra/reviewstats: Tweak disagreement ratio calcuation https://review.openstack.org/41425 | 14:05 |
| *** prad_ has quit IRC | 14:05 | |
| *** mgagne1 has joined #openstack-infra | 14:05 | |
| *** mgagne1 has quit IRC | 14:05 | |
| *** mgagne1 has joined #openstack-infra | 14:05 | |
| openstackgerrit | A change was merged to openstack-infra/reviewstats: Tweak disagreement ratio calcuation https://review.openstack.org/41425 | 14:05 |
| *** vijendar1 has joined #openstack-infra | 14:05 | |
| *** dina_belova has quit IRC | 14:06 | |
| *** dkehn has quit IRC | 14:08 | |
| *** prad_ has joined #openstack-infra | 14:09 | |
| *** prad_ has quit IRC | 14:10 | |
| *** mgagne has quit IRC | 14:10 | |
| *** vijendar has quit IRC | 14:10 | |
| *** mgagne1 is now known as mgagne | 14:10 | |
| *** dina_belova has joined #openstack-infra | 14:11 | |
| *** burt has joined #openstack-infra | 14:12 | |
| openstackgerrit | Jordan OMara proposed a change to openstack-infra/config: Add tuskar-ui to stackforge https://review.openstack.org/41199 | 14:12 |
| *** luisg has joined #openstack-infra | 14:13 | |
| *** dina_belova has quit IRC | 14:16 | |
| *** ruhe has quit IRC | 14:18 | |
| openstackgerrit | Jordan OMara proposed a change to openstack-infra/config: Add tuskar-ui to stackforge https://review.openstack.org/41199 | 14:21 |
| *** pcrews has quit IRC | 14:22 | |
| *** openstackgerrit has quit IRC | 14:24 | |
| *** openstackgerrit has joined #openstack-infra | 14:24 | |
| openstackgerrit | A change was merged to openstack-infra/reviewstats: Fixing minor misspelling https://review.openstack.org/41357 | 14:24 |
| *** wayneeseguin has quit IRC | 14:24 | |
| *** wayneeseguin has joined #openstack-infra | 14:24 | |
| *** markmcclain has joined #openstack-infra | 14:26 | |
| *** krtaylor has joined #openstack-infra | 14:26 | |
| openstackgerrit | A change was merged to openstack-infra/config: Add tuskar-ui to stackforge https://review.openstack.org/41199 | 14:26 |
| *** datsun180b has joined #openstack-infra | 14:27 | |
| openstackgerrit | Mark McLoughlin proposed a change to openstack/requirements: Allow use of oslo.messaging 1.2.0a1 https://review.openstack.org/41381 | 14:27 |
| openstackgerrit | Mark McLoughlin proposed a change to openstack/requirements: Allow use of oslo.messaging 1.2.0a2 https://review.openstack.org/41381 | 14:28 |
| *** markmcclain has quit IRC | 14:30 | |
| *** markmcclain has joined #openstack-infra | 14:30 | |
| sdague | mordred: you know why http://status.openstack.org/zuul/ hasn't refreshed yet? | 14:31 |
| fungi | sdague: what do you mean by refreshed? it looks like it's refreshing to me | 14:32 |
| sdague | fungi: the style changes that merged this weekend | 14:33 |
| sdague | to status.html | 14:33 |
| fungi | oh, that. i thought you meant the content of the status page had gone stale | 14:33 |
| fungi | i'll see if i can tell what's up. could be puppet's hung on static.o.o | 14:33 |
| fungi | sdague: what files should have changed? | 14:34 |
| *** dina_belova has joined #openstack-infra | 14:34 | |
| sdague | status.html | 14:34 |
| fungi | ahh | 14:34 |
| sdague | all the content should be there | 14:35 |
| fungi | -rw-r--r-- 1 root root 4950 Aug 1 21:54 /srv/static/status/zuul/index.html | 14:36 |
| *** ruhe has joined #openstack-infra | 14:36 | |
| fungi | so not updated, agreed | 14:36 |
| openstackgerrit | Monty Taylor proposed a change to openstack-dev/hacking: Add check for deprecated method assertEquals https://review.openstack.org/35962 | 14:36 |
| fungi | digging in puppet logs now | 14:36 |
| *** dkehn has joined #openstack-infra | 14:37 | |
| *** pentameter has joined #openstack-infra | 14:38 | |
| *** openstackgerrit has quit IRC | 14:40 | |
| *** openstackgerrit has joined #openstack-infra | 14:40 | |
| fungi | weird, puppet is still running and has no child processes but stopped updating (both according to the dashboard and its local syslog) at midnight utc friday morning | 14:41 |
| fungi | strace shows it quietly idling in a select() call | 14:41 |
| mordred | fungi: I've been seeing puppet get hung a lot recently | 14:42 |
| fungi | mordred: yeah, wish it actually provided some indication of why it's hung | 14:43 |
| fungi | anyway, i've restarted the agent cleanly so it should update shortly now sdague | 14:43 |
| fungi | after restarting it returned a failure log for that midnight run, but all it included was "Caught TERM; calling stop" so unfortunately not a lot to go on there either | 14:45 |
| *** dina_belova has quit IRC | 14:47 | |
| *** SpamapS has joined #openstack-infra | 14:47 | |
| *** SpamapS has quit IRC | 14:47 | |
| *** SpamapS has joined #openstack-infra | 14:47 | |
| *** dina_belova has joined #openstack-infra | 14:48 | |
| *** boris-42 has quit IRC | 14:48 | |
| *** pcrews has joined #openstack-infra | 14:51 | |
| *** lascii is now known as alaski | 14:56 | |
| *** vandervenv is now known as devananda | 14:57 | |
| * fungi is afk again for a bit | 14:59 | |
| *** rnirmal has joined #openstack-infra | 15:00 | |
| *** sarob has joined #openstack-infra | 15:00 | |
| *** rnirmal has quit IRC | 15:00 | |
| *** rnirmal has joined #openstack-infra | 15:00 | |
| *** sarob has quit IRC | 15:05 | |
| annegentle | jeblair: clarkb: or other infra-ers, up for Docs Boot Camp? The registration form is on https://wiki.openstack.org/wiki/Docs_Bootcamp_2013 | 15:11 |
| *** UtahDave has joined #openstack-infra | 15:11 | |
| *** changbl has joined #openstack-infra | 15:11 | |
| *** sarob has joined #openstack-infra | 15:12 | |
| jeblair | annegentle: done! based on the schedule, i put myself down for day 1, but if you want me there both days i can do that too. | 15:13 |
| annegentle | jeblair: awesome thank you! I am hoping you can do a "hands on lab" -- like an example yaml file? | 15:13 |
| annegentle | jeblair: definitely join us for dinner Monday evening, we're working on something fun | 15:14 |
| jeblair | annegentle: yeah, i'll try to prepare something in advance; i'll probably talk to you about it more when we're a little closer. | 15:14 |
| annegentle | jeblair: cool, sounds good | 15:15 |
| annegentle | jeblair: we should have 20 people, maybe 25 | 15:15 |
| jeblair | annegentle: excellent! | 15:15 |
| annegentle | jeblair: yeah! | 15:16 |
| mordred | jeblair: morning! | 15:16 |
| mordred | jeblair: I don't know if you saw in the scrollback - but we're narrowed the devstack-on-pristine-ubuntu problem down to "works with our mirror, does not work with not our mirror" | 15:17 |
| mordred | for reasons thus-far surpassing understanding | 15:17 |
| jeblair | mordred: i did see that. that is wacky. | 15:17 |
| *** sarob has quit IRC | 15:17 | |
| mordred | my favorite part is that for some reason our mirror prevents the prettytable permissions problem | 15:18 |
| *** psedlak has quit IRC | 15:18 | |
| *** ruhe has quit IRC | 15:21 | |
| dtroyer | mordred: peek inside the prettytable package in the mirror to see if it has the same broken permissions and the one on pypi | 15:22 |
| dtroyer | I don't know why it would change, but I don't know why this wasn't a problem before pip 1.4 either | 15:23 |
| *** prad_ has joined #openstack-infra | 15:24 | |
| * anteaya fires up an ubuntu devstack | 15:24 | |
| mordred | dtroyer: all three copies have the same md5sum - tar.gz, tar.bz2 and .zip | 15:26 |
| mordred | dtroyer: I'm wondering if perhaps the logic changed and is getting zip by default over .tar.gz or something like that | 15:26 |
| dtroyer | could be, 1.4 is using the zip file | 15:27 |
| mordred | but is different on our mirror beause we're using autoindex | 15:27 |
| sdague | fungi: man, takes a lot of refreshing to get all that stuff to clear cache | 15:27 |
| sdague | fungi: maybe we should default mod_expires everything after 24 hrs | 15:28 |
| mordred | dtroyer: there it is | 15:28 |
| mordred | dtroyer: I get the tarball from the openstack mirror | 15:28 |
| mordred | dtroyer: and I get the zipfile from pypi.o.o | 15:28 |
| jeblair | sdague: ? i just hit reload. anyway, looks great! | 15:28 |
| mordred | gah. pypi.python.org | 15:28 |
| *** ruhe has joined #openstack-infra | 15:29 | |
| sdague | jeblair: I had to go to the status.js and refresh it directly | 15:29 |
| *** dina_belova has quit IRC | 15:29 | |
| jeblair | ttx: ^ sdague granted your wish about right-alignment on zuul-status | 15:29 |
| sdague | maybe chrome likes to cache too much :) | 15:29 |
| mordred | dstufft: any way to get pip to prefer tarballs over zip files? | 15:29 |
| jeblair | sdague: maybe chrome is only compatible with the way google caches things. :P | 15:30 |
| mordred | fungi: ^^ problem figured out | 15:30 |
| *** vijendar1 has quit IRC | 15:30 | |
| jeblair | mordred, fungi, clarkb: a couple random thoughts i had: | 15:30 |
| jeblair | mordred, fungi, clarkb: 1) i think we'll be able to do a smooth transition between current devstack-gate -> nodepool | 15:31 |
| jeblair | mordred, fungi, clarkb: 2) the combined dib-kexec system makes using nodepool for standard unit test slaves much more compelling | 15:31 |
| jeblair | mordred: so when i wrap up moving to the new system, i think i'll want to work with you on incorporating that next. | 15:32 |
| mordred | jeblair: cool! | 15:33 |
| mordred | jeblair: I think it's basically functional, but needs more testing (clearly) and more brainspace looking at what I might have missed | 15:34 |
| sdague | jeblair: probably :) | 15:34 |
| mordred | jeblair: I also want to work out a smarter way to re-use/re-copy caching between how dib is doing it and how we're doing it | 15:34 |
| mordred | sdague: did you see above that we figured out one of the reasons why we're not seeing prettytable issues in the gate, but are seeing them elsehwere? | 15:36 |
| dtroyer | mordred: the permissions in all three archives are the same, 0600. looks like they're handled differently though somewhere | 15:36 |
| *** dina_belova has joined #openstack-infra | 15:36 | |
| *** dina_bel_ has joined #openstack-infra | 15:37 | |
| *** dina_belova has quit IRC | 15:37 | |
| sdague | nope, but that's good | 15:37 |
| mordred | dtroyer: but do unzip and untar copy the permissions in the same way is what I'm thinking | 15:37 |
| dtroyer | i'm checking on that now | 15:38 |
| mordred | sdague, dtroyer: also, I got MUCH further with the global venv patch. swift refuses to start properly now, and I'm not sure why | 15:38 |
| notmyname | mordred: take out the "is_monty_trying_to_run_swift" config variable. or set the "just_DWIM" env var | 15:39 |
| mordred | dtroyer: also, I learned from dstufft over the weekend that we should be able to stop mucking with setuptools: https://review.openstack.org/#/c/41281/ | 15:40 |
| mordred | notmyname: AH. I forgot those | 15:40 |
| dtroyer | mordred: I saw that…I wasn't sure if I'm ready to nuke that yet, but maybe | 15:40 |
| dtroyer | it isn't being used now though | 15:40 |
| openstackgerrit | Sergey Kolekonov proposed a change to openstack-infra/jenkins-job-builder: Existing plugins improvements https://review.openstack.org/41375 | 15:41 |
| mordred | dtroyer: yeah.not important to nuke - but potentially good to know | 15:41 |
| openstackgerrit | A change was merged to openstack/requirements: Allow use of oslo.messaging 1.2.0a2 https://review.openstack.org/41381 | 15:41 |
| dtroyer | man unzip | 15:42 |
| dtroyer | that looks bad out of context! | 15:42 |
| mordred | sdague, lifeless: also, pip has protections against instaling binary wheels from pypi - so we are safe to use them for now | 15:43 |
| mordred | sdague, lifeless: and the protection is specific to pypi.python.org - so we _can_ upload binary wheels to pypi.o.o and make use of them | 15:44 |
| *** rcleere has joined #openstack-infra | 15:45 | |
| mordred | notmyname: btw, I got the docs updated per our last conversation: http://docs.openstack.org/developer/pbr/ | 15:47 |
| mordred | notmyname: with a new page for packagers | 15:47 |
| dstufft | mordred: Uhhh, I'm not sure offhand | 15:48 |
| dstufft | there's not a way exposed | 15:48 |
| notmyname | mordred: great, thanks! | 15:49 |
| mordred | dstufft: ok. I think dtroyer is still verifying, but it seems like the difference between installing from zip and installing from tar.gz may be what's causing the bad permissions problems we're seeing when we install prettytable | 15:49 |
| dstufft | mordred: interesting, does the zip and tar.gz have different perms inside the archives? | 15:51 |
| mordred | dstufft: nope. same perms | 15:51 |
| mordred | dstufft: but we verified that if you use our mirror, you get the tarball, and if you use pypi, you get the zip | 15:52 |
| devananda | mordred: g'morning! | 15:52 |
| mordred | dstufft: and that running devstack against our mirror with nothing else changed works | 15:52 |
| mordred | and runing it without our mirror set does not work | 15:52 |
| mordred | so - I'm making some logical jumps now | 15:52 |
| mordred | devananda: morning! | 15:52 |
| *** Ryan_Lane has joined #openstack-infra | 15:53 | |
| devananda | mordred: when you have a moment -- thoughts on https://review.openstack.org/#/c/41086/ would be appreciated | 15:53 |
| devananda | hopefully it's trivial to approve | 15:53 |
| dstufft | mordred: yea pip respects the version of the files listed on the index in absence of something else pushing one link higher or lower | 15:54 |
| mordred | devananda: it's pretty trivial - are we assuming the pyghmi is going to not break things with new releases? | 15:54 |
| dstufft | so if you look I bet the .zip appears first on the simple page on pypi, bu the tar.gz on the openstac mirror | 15:54 |
| dtroyer | I think I found the difference from 1.3.1 to 1.4…pip-1.4.1/pip/util.py line 500: | 15:55 |
| dtroyer | if unix_attributes: | 15:55 |
| dtroyer | os.chmod(fn, unix_attributes) | 15:55 |
| *** ruhe has quit IRC | 15:55 | |
| devananda | mordred: we should assume nothing of pyghmi at this point. i'm slowly pulling it more into stackforge's purvue, but for the most part, jbjohnson is developing/maintaining it | 15:55 |
| dtroyer | not present before 1.4. looks to be .zip only | 15:55 |
| mordred | dtroyer: awesome. so - pip is setting the files to the permissions in the zip archive nly if it's a zip archive, and it's not performing that 'help' on tarballs | 15:57 |
| dtroyer | mordred: exactly. so what is the bug? the addition of the perm preservation or the bad perms in existing packages? | 15:57 |
| devananda | mordred: IMHO, pyghmi is kinda like pecan in that it's depended on by an openstack project, written by folks involved in openstack, and developed partly for openstack but partly not. | 15:58 |
| *** lcestari_ has joined #openstack-infra | 15:58 | |
| mordred | dtroyer: I'm going to say the combination of those things. I think if it's goingto chmod things, it should do so in a way that makes them sensible | 15:58 |
| dtroyer | but who is to say that the package contents aren't sensible? in our use case the aren't for sure | 15:59 |
| mordred | but we seem to be getting the perfect storm of downloading-zip+pip 1.4+bad-perms-in-package - remove any of those things and it works | 15:59 |
| mordred | dtroyer: I'd say any package contents that are only readable by root are probably rpetty weird :) | 16:00 |
| dtroyer | bit I'd expect pip to behave more like tar(1) or unzip(1) does…that's what threw me on Friday, tar and unzip did what I expected | 16:00 |
| mordred | dtroyer: what do they do different? | 16:00 |
| dtroyer | the preserve the permissions inside the archive by default | 16:01 |
| *** lcestari has quit IRC | 16:01 | |
| dtroyer | so honestly, it's pip's tar extract that is now doing-what-i-don't-expect | 16:01 |
| dtroyer | except that it's exposed a bunck of semi-borken packages | 16:02 |
| *** mrodden has quit IRC | 16:02 | |
| dtroyer | and o course, in a venv you don't notice as it's all readable by the user | 16:02 |
| mordred | cry | 16:02 |
| * anteaya admires her "Keystone did not start" error in a fresh devstack on ubuntu - https://gist.github.com/anteaya/18c24018181e1131da40 | 16:04 | |
| mordred | anteaya: yup! | 16:05 |
| mordred | that would be the prettytale issue | 16:05 |
| mordred | prettytable | 16:05 |
| anteaya | yay, I got it right | 16:05 |
| anteaya | any I can do now to help? | 16:05 |
| openstackgerrit | will soula proposed a change to openstack-infra/jenkins-job-builder: Adding support for the Warnings plugin https://review.openstack.org/40621 | 16:06 |
| anteaya | s/any/anything | 16:06 |
| mordred | anteaya: there is a workaroud for it here: https://review.openstack.org/#/c/41209/ | 16:07 |
| *** marun has joined #openstack-infra | 16:07 | |
| clarkb | morning | 16:08 |
| anteaya | morning clarkb | 16:09 |
| *** pcrews has quit IRC | 16:09 | |
| dtroyer | dstufft, mordred: looks like the chmod during unzip came in the wheel support, explaining why it is zip only? | 16:10 |
| dtroyer | so I'm not sure what the 'correct' fix is here…other than fixing a bunch of packages or forcing tar-only | 16:11 |
| *** afazekas has quit IRC | 16:11 | |
| *** lcestari_ has quit IRC | 16:12 | |
| dstufft | heh | 16:12 |
| *** mrodden has joined #openstack-infra | 16:12 | |
| dstufft | that's an interesting question | 16:12 |
| *** pentameter has quit IRC | 16:12 | |
| *** lcestari has joined #openstack-infra | 16:13 | |
| mordred | dtroyer: maybe just a recursive chmod on the site-packages location to make it ugo+r (and ugo+rx for dirs) once we're done installing things? | 16:13 |
| dstufft | probably what makes the most sense is for pip to keep the executable bit as it exists in the package, but reset the permissions otherwise to what a new file would be created as with the umask | 16:13 |
| dtroyer | I hate to do the whole thing, it would have to be done in every call to pip_install() | 16:13 |
| dtroyer | or after anything similar | 16:14 |
| mordred | dtroyer: yuck | 16:14 |
| *** vogxn has quit IRC | 16:14 | |
| *** pentameter has joined #openstack-infra | 16:14 | |
| *** vogxn has joined #openstack-infra | 16:14 | |
| mordred | dstufft: wouldnt' that still leave us unable to read files in the package though? (prettytable is installing mode 600 things, iirc) | 16:14 |
| dtroyer | I'm more of a mind to keep the hack-around in fixup_stuff.sh and make a list of the packages with the problem and start filing bugs. | 16:15 |
| dstufft | dtroyer: well pip should at the very least be consistent, either apply the permissions in the package to tar and zip, or don't do it in either | 16:16 |
| mordred | dtroyer: I'm fine with that | 16:16 |
| mordred | dstufft: ++ | 16:16 |
| mordred | dstufft: it seems that not doing it breaks less things, although it's also possibly less correct | 16:16 |
| dstufft | mordred: my proposal would install them to whatever the umask would cause a new file to be created as | 16:17 |
| mordred | AH. I grok what you were saying earlier better now | 16:17 |
| pleia2 | all settled in back at home, good morning :) | 16:18 |
| mordred | dstufft: I think that sounds more righter - because these aren't general zip/tar files - they are specific with a specific purpose in mind | 16:19 |
| anteaya | morning pleia2 | 16:19 |
| anteaya | glad you are home safe | 16:19 |
| dstufft | I think as to which one to do depends mostly on who you think should control the file mode of the newly created files. The packager or the installer. I lean towards giving control to the installer as I think people generally suck at packaging things (and it's also hard to know what file permissions are reasonable in every installation environment) | 16:19 |
| dstufft | and fixing it in pip is easier than getting a bunch of random people fix their packages | 16:20 |
| anteaya | pleia2: highlights so far, zuul has a fresher look thanks to jeblair and sdague also devstack on ubuntu fails right now due to permissions on files (tar, zip) being set differently on install | 16:21 |
| dtroyer | dstufft: [puts on SA hat] I'll buy that… | 16:21 |
| pleia2 | anteaya: thanks! | 16:21 |
| anteaya | np | 16:21 |
| anteaya | also fungi is here intermittently, he is being supportive in hospital waiting rooms | 16:22 |
| *** jpich has quit IRC | 16:23 | |
| *** derekh has quit IRC | 16:25 | |
| *** markmc has quit IRC | 16:25 | |
| zaro | anteaya: nice highlig | 16:25 |
| zaro | mordred: thanks. Wow! builds have become super snappy since the additional jenkins masters.~ | 16:25 |
| dstufft | dtroyer: can you post an issue with details? | 16:25 |
| dtroyer | dstufft: yup, witing it now | 16:26 |
| dstufft | dtroyer: thanks | 16:26 |
| anteaya | zaro: thanks and morning | 16:26 |
| zaro | sorry, network hiccup. but yeah, you got my gist. | 16:27 |
| mordred | pleia2: woot. welcome back | 16:27 |
| mordred | pleia2: I have not tested that jeepyb change at all, btw | 16:27 |
| *** ruhe has joined #openstack-infra | 16:28 | |
| pleia2 | mordred: it's first up on my list, I also am thinking of adding a .backup rotation of the file for in case things go sideways during generation (it doesn't exactly have syntax checking) | 16:28 |
| pleia2 | well, after catching up on email :) | 16:29 |
| mordred | dtroyer: I'm going to write a note to the -dev list teling people what's up | 16:29 |
| mordred | pleia2: good idea | 16:29 |
| *** annegentle_ has joined #openstack-infra | 16:32 | |
| *** annegentle has quit IRC | 16:32 | |
| *** pentameter has quit IRC | 16:33 | |
| *** dkehn has quit IRC | 16:33 | |
| dtroyer | dstufft: https://github.com/pypa/pip/issues/1133 I didn't spell out a solution, just tried to describe what we're seeing. | 16:34 |
| dstufft | dtroyer: thanks | 16:35 |
| mordred | dtroyer: http://paste.openstack.org/show/43956/ look ok to you? | 16:36 |
| annegentle_ | hey infra, just fyi, once https://review.openstack.org/#/c/41071/ merges, we'll only validate/build modified files in the openstack-manuals directory for our "gate" testing. | 16:37 |
| mordred | annegentle_: neat | 16:37 |
| *** dkehn has joined #openstack-infra | 16:38 | |
| dtroyer | mordred: I think I'd add a link to either the devstack review with fixup_stuff.sh or a pointer to the commit if it goes in befoe you send this. otherwise its good | 16:38 |
| *** afazekas has joined #openstack-infra | 16:41 | |
| mordred | dtroyer: good call. added. | 16:42 |
| *** dina_bel_ has quit IRC | 16:42 | |
| *** ruhe has quit IRC | 16:44 | |
| openstackgerrit | Sergey Kolekonov proposed a change to openstack-infra/jenkins-job-builder: Existing plugins improvements https://review.openstack.org/41375 | 16:45 |
| *** UtahDave has quit IRC | 16:46 | |
| dtroyer | sdague, mordred: is https://review.openstack.org/41209 ready to approve? I'm ok with Maru's addition of httplib2 | 16:47 |
| *** vogxn has quit IRC | 16:47 | |
| *** UtahDave has joined #openstack-infra | 16:47 | |
| *** pentameter has joined #openstack-infra | 16:47 | |
| anteaya | 41209,4 works for me | 16:50 |
| anteaya | as a side note sourcing openrc gets me an odd response: $ source openrc | 16:50 |
| anteaya | No packages found matching xenserver-core. | 16:50 |
| clarkb | I was going to suggest attempting to test pbx again but cacti is currently out of commision | 16:50 |
| anteaya | that is new and could be upsetting to a newcomer | 16:51 |
| clarkb | fungi: ^ any update on your request to get that fixed? | 16:51 |
| Shrews | mordred: Happy birthday old man | 16:52 |
| *** ^d has joined #openstack-infra | 16:52 | |
| *** ^d has joined #openstack-infra | 16:52 | |
| *** odyssey4me has quit IRC | 16:52 | |
| *** pentameter has quit IRC | 16:52 | |
| *** dina_belova has joined #openstack-infra | 16:53 | |
| clarkb | mordred: thank you for the devstack update. very helpful in my attempt to catch up after the weekend | 16:53 |
| clarkb | also happy birthday! | 16:53 |
| mordred | dtroyer: I'm good with landing that patch | 16:53 |
| mordred | Shrews, clarkb thanks! | 16:53 |
| *** sdake__ has joined #openstack-infra | 16:54 | |
| *** pentameter has joined #openstack-infra | 16:54 | |
| fungi | sdague: jeblair: ttx: likey the new result/progress alignment on the zuul status page, for sure | 16:54 |
| * fungi is scrollbacking for a bit | 16:54 | |
| *** sdake has quit IRC | 16:55 | |
| *** fbo is now known as fbo_away | 16:55 | |
| fungi | also, dib/kexec to turn minimal vm into devstack slave or unit test slave or whatever is very interesting | 16:56 |
| anteaya | filed it as a bug: https://bugs.launchpad.net/devstack/+bug/1211413 | 16:57 |
| uvirtbot | Launchpad bug 1211413 in devstack "No packages found matching xenserver-core. response when sourcing openrc" [Undecided,New] | 16:57 |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/config: Remove oslo gerritbot posts from openstack-nova https://review.openstack.org/40686 | 16:58 |
| jog0 | mordred: just pressed the magical 'rebase' button on gerrit hopefully that workd as expected | 16:59 |
| fungi | and yeah, pypi.o.o uses apache mod_index which does alpha order by file name, so .tar.somethng would sort before .zip | 16:59 |
| jog0 | ahh that is how the commiter becomes gerrit | 16:59 |
| dstufft | PyPI orders by upload date I think | 17:00 |
| *** sarob has joined #openstack-infra | 17:00 | |
| dstufft | or possibly by whatever order they are in the DB | 17:00 |
| *** rscottcoyle has quit IRC | 17:02 | |
| fungi | clarkb: bug 1210822 seems to be waiting for someone to debuild patched packages and request a security sponsor. i was actually going to give it a shot yesterday until i suddenly had to deal with real life more than anticipated. the package maintainer's interpretation seems to suggest that these vulnerabilities are only only post-authentication though, so i'm on the fence about just turning it back on | 17:05 |
| uvirtbot | Launchpad bug 1210822 in cacti "Please import 0.8.8b+dfsg-2 from Debian and backport security fixes to 12.04 LTS" [Undecided,Triaged] https://launchpad.net/bugs/1210822 | 17:05 |
| clarkb | fungi: good to know, thanks | 17:06 |
| *** jjmb has joined #openstack-infra | 17:06 | |
| *** dina_belova has quit IRC | 17:08 | |
| *** SergeyLukjanov has quit IRC | 17:09 | |
| jeblair | clarkb, zaro: i believe the gearman-plugin error we saw over the weekend (where two jobs were stuck in the jenkins queue) is related to race conditions around onlining and offlining nodes | 17:11 |
| jeblair | clarkb, zaro: the node in question was flapping, and i'm pretty sure it ended up with at least two gearman executor threads. | 17:11 |
| jeblair | clarkb, zaro: i'm working on a fix | 17:12 |
| *** sarob has quit IRC | 17:12 | |
| clarkb | jeblair: ok, thank you for the update | 17:12 |
| mordred | thx | 17:12 |
| *** sarob has joined #openstack-infra | 17:12 | |
| *** jjmb1 has joined #openstack-infra | 17:14 | |
| *** sarob_ has joined #openstack-infra | 17:14 | |
| *** sarob has quit IRC | 17:15 | |
| *** pcrews has joined #openstack-infra | 17:15 | |
| *** jjmb has quit IRC | 17:17 | |
| openstackgerrit | will soula proposed a change to openstack-infra/jenkins-job-builder: Adding support for the Warnings plugin https://review.openstack.org/40621 | 17:17 |
| *** dirk has joined #openstack-infra | 17:17 | |
| dirk | is there a way to change the owner of an abandoned review request? | 17:18 |
| *** rscottcoyle has joined #openstack-infra | 17:18 | |
| clarkb | dirk: I don't think we would want to do that (for CLA related reasosn and stuff). Are you trying to unabandon it? | 17:20 |
| dirk | clarkb: yes | 17:20 |
| clarkb | dirk: members of the project's core group should be able to do that | 17:20 |
| clarkb | dirk: or I can do it if you provide a link to the change | 17:20 |
| dirk | https://review.openstack.org/#/c/28819/ | 17:20 |
| dirk | clarkb: ^^ | 17:22 |
| clarkb | dirk: done | 17:23 |
| *** ladquin has joined #openstack-infra | 17:23 | |
| dirk | clarkb: thank you! | 17:25 |
| *** nicedice_ has joined #openstack-infra | 17:25 | |
| clarkb | no problem | 17:26 |
| *** nati_ueno has joined #openstack-infra | 17:28 | |
| *** jjmb1 has quit IRC | 17:29 | |
| *** jjmb has joined #openstack-infra | 17:30 | |
| *** gyee has joined #openstack-infra | 17:30 | |
| sdague | clarkb: ok, I figured out why the 404 catching I did was no good | 17:31 |
| sdague | because it's in a generator, so we actually throw the exception too late | 17:31 |
| *** fungi has quit IRC | 17:31 | |
| *** fungi has joined #openstack-infra | 17:32 | |
| *** dirk has quit IRC | 17:33 | |
| *** afazekas has quit IRC | 17:34 | |
| *** jfriedly has joined #openstack-infra | 17:35 | |
| *** rscottcoyle has quit IRC | 17:40 | |
| nati_ueno | hi guys how do you think this bug? | 17:44 |
| nati_ueno | https://bugs.launchpad.net/neutron/+bug/1210391 | 17:44 |
| uvirtbot | Launchpad bug 1210391 in neutron "Intermittent neutron-adv-test related failure in gate" [High,Confirmed] | 17:44 |
| nati_ueno | It looks like we are missing SERVICE_TIMEOUT | 17:45 |
| *** ruhe has joined #openstack-infra | 17:52 | |
| *** fbo_away is now known as fbo | 17:55 | |
| *** pentameter has quit IRC | 17:56 | |
| *** jjmb has quit IRC | 17:57 | |
| mordred | dude. I cannot drive a truck this well: http://humortrain.com/post/58008670217 | 18:01 |
| *** SergeyLukjanov has joined #openstack-infra | 18:03 | |
| *** ruhe has quit IRC | 18:04 | |
| clarkb | nati_ueno: does that command have $SERVICE_TIMEOUT in it and we are not setting the value? | 18:05 |
| *** sarob_ has quit IRC | 18:05 | |
| nati_ueno | clarkb: Thanks. I figued out the issue. TIMEOUT value was ok but it fails getting ip from nova show command | 18:05 |
| *** sarob has joined #openstack-infra | 18:05 | |
| nati_ueno | clarkb: I'll continue investigation | 18:05 |
| clarkb | nati_ueno: well I don't see a timeout value in the command | 18:06 |
| clarkb | nati_ueno: 2013-08-09 02:59:55.883 | timeout: invalid time interval `sh' is a result of that | 18:06 |
| nati_ueno | clarkb: yes. so the func using $1(name) $2(ip) $2(timeout) | 18:07 |
| nati_ueno | clarkb: but in previous command we faild to get the $2(ip) | 18:07 |
| nati_ueno | clarkb: so $1(name) $3(timeout) passed to the command | 18:07 |
| nati_ueno | clarkb: so it looks we are lacking timeout value | 18:07 |
| nati_ueno | clarkb: but the cause was getting ip address | 18:07 |
| *** sarob has quit IRC | 18:10 | |
| clarkb | I see | 18:10 |
| *** pentameter has joined #openstack-infra | 18:11 | |
| jeblair | why doesn't a java List, which is a generic, parameterize its "contains" method so the compiler checks that it's even possible for a list to contain the kind of object you're asking about? | 18:11 |
| *** jpeeler1 has joined #openstack-infra | 18:11 | |
| clarkb | nati_ueno: should there be a more aggressive check of the arguments so that we can give better error messages? | 18:12 |
| nati_ueno | clarkb: yes I agree. also we should add error handing for getting ip | 18:13 |
| fungi | jeblair: i take it that was the bug which resulted in the flapping slave getting multiple workers? | 18:13 |
| nati_ueno | clarkb: currently there is no error if we can't get ip address. and the result of nova show is missing | 18:13 |
| fungi | jeblair: wrong type passed in a contains() method call? | 18:13 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/config: robustify htmlifier for logs https://review.openstack.org/41486 | 18:13 |
| sdague | clarkb: so, I think that's the fix | 18:13 |
| clarkb | sdague: looking | 18:13 |
| clarkb | sdague: also "robustify" is an excellent | 18:14 |
| *** pentameter has quit IRC | 18:14 | |
| clarkb | *excellent word | 18:14 |
| sdague | I also put a couple of other things in that patch which I think we need | 18:14 |
| mordred | jeblair: late binding? maybe it's expecting that ... no, nevermind. I've got nothing | 18:14 |
| fungi | ha! cloud relevant... zooko/leastauthority.com has a tahoe/lafs-backed cloud storage service now called "s4" (simple secure storage service) | 18:14 |
| *** jpeeler has quit IRC | 18:14 | |
| * fungi lol'd | 18:15 | |
| *** mrmartin has joined #openstack-infra | 18:15 | |
| fungi | i'm tempted to pay the $50/mo rate for 350gb storage with them, just on principle | 18:15 |
| *** sarob has joined #openstack-infra | 18:15 | |
| *** sarob has quit IRC | 18:16 | |
| fungi | "discounts are available for [...] open source projects" | 18:16 |
| *** sarob has joined #openstack-infra | 18:17 | |
| fungi | too bad they're putting it on top of s3 for the moment, but they claim to be targeting hpcloud as a backend with a future iteration | 18:18 |
| *** fbo is now known as fbo_away | 18:18 | |
| fungi | and rackspace | 18:18 |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/gearman-plugin: Rework starting/stopping executors https://review.openstack.org/41488 | 18:21 |
| jeblair | zaro: ^ | 18:21 |
| clarkb | fungi: jeblair https://review.openstack.org/#/c/41486/ lgtm and mordred. Did you want to review sdague's robustification before I approve it? | 18:21 |
| *** sarob has quit IRC | 18:21 | |
| * fungi looks | 18:21 | |
| jeblair | clarkb: yeah, i'd like to take a look | 18:22 |
| sdague | clarkb: mordred already +2ed it actually | 18:22 |
| sdague | jeblair's eyes would be good | 18:22 |
| fungi | lgtm. mostly just wanted to confirm we weren't doing unsafe things with the query string | 18:25 |
| * fungi is probably unnecessarily paranoid about web-facing code | 18:26 | |
| * anteaya is grateful for fungi's paranoia | 18:26 | |
| clarkb | me too | 18:26 |
| anteaya | stay paranoid, fungi | 18:27 |
| zaro | jeblair: looking | 18:27 |
| *** dina_belova has joined #openstack-infra | 18:27 | |
| *** pentameter has joined #openstack-infra | 18:28 | |
| clarkb | jeblair: does that change enforce a 1:1 slave to executor mapping? | 18:28 |
| clarkb | or is a Computer roughly equivalent to an executor? | 18:29 |
| jeblair | clarkb: no, a Computer contains Executors | 18:30 |
| jeblair | clarkb: so it may well have some problems with multiple executors | 18:30 |
| jeblair | sdague: why not open() the file, then pass the open filehandle to the generator (instead of using fileinput)? | 18:33 |
| jeblair | sdague: (and actually, you could probably open the file with fileinput.hook_compressed(fname) instead of open()) | 18:34 |
| openstackgerrit | A change was merged to openstack-infra/config: Remove oslo gerritbot posts from openstack-nova https://review.openstack.org/40686 | 18:34 |
| jeblair | s/filehandle/file object/ | 18:34 |
| sdague | jeblair: so you actually can't | 18:35 |
| sdague | I tried | 18:35 |
| fungi | if you could, that would presumably solve the race | 18:35 |
| sdague | the way it does closes is weird, and fails in the plain text case | 18:35 |
| jeblair | lying doc writers | 18:35 |
| jeblair | sdague: the way open() closes? | 18:35 |
| sdague | no, the way the implicit fileinput.hook_compressed(fname) works | 18:36 |
| clarkb | jeblair: I really like that gearman plugin change | 18:36 |
| clarkb | simplifies the race condition surface area | 18:36 |
| jeblair | clarkb: do you think it has multi-executor problems? i haven't gone through and done a pass thinking about that. | 18:36 |
| clarkb | jeblair: I think it does | 18:37 |
| clarkb | jeblair: because each executor gets a thread but we are comparing against node stuff | 18:37 |
| clarkb | jeblair: which means a node iwth more than one executor might go sideways, but I believe this has been a restriction on the gearman plugin use | 18:38 |
| clarkb | zaro: is that correct? | 18:38 |
| jeblair | sdague: in the plain text case, hook_compressed just calls "open(filename, mode)" | 18:38 |
| sdague | well, when I tried to just open the stream before the first yield, it wouldn't trigger the IOError early enough | 18:40 |
| *** jjmb has joined #openstack-infra | 18:40 | |
| jeblair | sdague: oh, ok, i meant something different | 18:40 |
| sdague | one of the other nice things about fileinput, is it means you can test on the command line really easily | 18:41 |
| sdague | as it auto handles stdin | 18:41 |
| *** pabelanger has quit IRC | 18:41 | |
| annegentle_ | basic, basic question: how do I set the fqdn for a server I've already spun up on Rackspace Cloud? IP address? (Trying to spin up a jenkins slave to test translation processing) | 18:42 |
| clarkb | annegentle_: edit /etc/hostname and /etc/hosts | 18:43 |
| *** Ryan_Lane has quit IRC | 18:44 | |
| clarkb | oh hostname should not contain the full fqdn | 18:44 |
| clarkb | so /etc/hostname should just have the hostname then you can set the fqdn in /etc/hosts | 18:44 |
| *** sarob has joined #openstack-infra | 18:45 | |
| annegentle_ | clarkb: okay, does it matter what I name a throwaway slave server? | 18:47 |
| clarkb | annegentle_: it shouldn't | 18:47 |
| annegentle_ | clarkb: you just name them slave1, slave2, etc. | 18:47 |
| clarkb | precise1 centos6-1 and so on | 18:48 |
| annegentle_ | clarkb: ok makes sense now thanks | 18:50 |
| *** openstackgerrit has quit IRC | 18:50 | |
| *** openstackgerrit has joined #openstack-infra | 18:50 | |
| *** sarob has quit IRC | 18:51 | |
| jeblair | sdague: the stdin case is handled separately anyway, so you wouldn't lose that. i left more detailed inline comments. | 18:51 |
| *** sarob has joined #openstack-infra | 18:52 | |
| jeblair | clarkb, zaro: istr that gearman-plugin should handle multiple executors, except that changing the number of executors on on online node isn't well supported | 18:55 |
| jeblair | clarkb, zaro: so we want to make sure we don't break that (we do have nodes with multiple executors) | 18:55 |
| *** annegentle_ has quit IRC | 18:58 | |
| jeblair | clarkb: heh, i actually think that patch doesn't break multi-exec support any more than currently, but i think i have found another case where the current state is broken. | 18:58 |
| clarkb | jeblair: ya, I think it may be the same degree of broken | 18:59 |
| clarkb | which is why I gave it a +2 | 18:59 |
| *** annegentle has joined #openstack-infra | 18:59 | |
| jeblair | all the exec workers get added, but only one is removed | 18:59 |
| jeblair | new patchset after lunch | 19:02 |
| sdague | jeblair: ok, I can't quite get that other approach of yours to work without generating other fileinput exceptions | 19:05 |
| sdague | so unless you really object to the current approach, I'd suggest we land it. | 19:05 |
| *** nati_ueno has quit IRC | 19:06 | |
| sdague | also, in the office for 2 - 3 more hours before vacation :) so something new and big won't be coming from me until we get to sept. | 19:06 |
| openstackgerrit | Anne Gentle proposed a change to openstack-infra/config: Point to README.rst instead of .md for Devstack Gate. https://review.openstack.org/41495 | 19:07 |
| *** nati_ueno has joined #openstack-infra | 19:07 | |
| *** ruhe has joined #openstack-infra | 19:08 | |
| zaro | jeblair, clarkb : yes, you remember correctly. handles multiple executors but does not update state when executors are added or removed. | 19:09 |
| *** fbo_away is now known as fbo | 19:14 | |
| *** jpeeler1 is now known as jpeeler | 19:16 | |
| mordred | off to the airport... | 19:17 |
| mordred | I may not be online again for several hours - I do not think that domestic flights have wifi down here | 19:17 |
| sdague | I feel like you should be wearing a cape when you say that | 19:17 |
| mordred | sdague: :) | 19:17 |
| pleia2 | mordred: enjoy! | 19:18 |
| sdague | safe travels | 19:18 |
| mrmartin | re | 19:18 |
| dtroyer | sdague: if you have time can you take a look at Sam Alba's responses in https://review.openstack.org/#/c/40759/ (Docker). I added a bit too, I'd like to not wait until you get back to finish that. | 19:18 |
| clarkb | sdague: is the problem that we need to s/IOError/FileInputError/ in the try except block because fileinput throws things other than IOError? | 19:18 |
| sdague | clarkb: no, we get an IOError | 19:19 |
| sdague | but we get it later | 19:19 |
| sdague | because that is all written as a generator | 19:19 |
| clarkb | oh I bet fobj doesn't actually open it because it is a generator itself | 19:19 |
| sdague | so we're completely outside of our code | 19:19 |
| clarkb | ya | 19:19 |
| *** mrodden has quit IRC | 19:19 | |
| clarkb | ok I am still happy with my +2 | 19:20 |
| sdague | so the real answer, is a more complicated framework like flash | 19:20 |
| sdague | flask | 19:20 |
| sdague | but... that would mean you'd need to wait for all this | 19:20 |
| sdague | dtroyer: can do | 19:20 |
| mrmartin | guys, the http://ci.openstack.org/puppet.html page mentions a common.yaml file. Where could I find an example of this common.yaml? | 19:20 |
| clarkb | sdague: ya, no rush the slight race condition in what you have doesn't bother me too much as we still get a 500 | 19:20 |
| clarkb | mrmartin: we don't currently have an example hiera data yaml file | 19:21 |
| mrmartin | ok, so grepping out hiera( from config files could provide a list of possible keys. | 19:21 |
| clarkb | mrmartin: we didn't think we needed one because we only ever refer to hiera data in site.pp. But enough people have asked for it that I may put one together | 19:21 |
| clarkb | mrmartin: yup and you only need to grep site.pp | 19:21 |
| clarkb | mrmartin: you can also forego hiera completely and hard code your own data in site.pp or an ENC | 19:22 |
| mrmartin | clarkb, thanks when I want to add a brand new server I guess I just need to add sysadmins entry | 19:22 |
| sdague | dtroyer: ok, I'm ok being 0 on that | 19:24 |
| sdague | as I get we're sort of defining new policy as we go | 19:25 |
| *** dkliban has quit IRC | 19:25 | |
| mrmartin | and we have some hard-coded value in modules/openstack_project/templates/puppet.conf.erb, like server name and reporturl parameters | 19:25 |
| clarkb | mrmartin: sysadmins is a list of email addresses for people that should receive root email | 19:25 |
| annegentle | clarkb: hm so if I ran a puppet apply pointing to my local.pp manifest, what would I see if it were successful? Just returned to the prompt for me. | 19:25 |
| mrmartin | clarkb: thanks | 19:25 |
| clarkb | mrmartin: we pulled it out of the repo so that we would stop getting email for other people's servers | 19:25 |
| sdague | I also threw russellb on the review because I think it opens up what should be a summit session about coming up with the agreed path by all the various programs and projects on how something like a new nova driver needs to land | 19:25 |
| dtroyer | sdague: thanks. it's the PPA I'm not sure what to do with for now until they get it into at least one distro | 19:26 |
| annegentle | clarkb: should I now have a docbook-properties.sh script in usr/local/jenkins/slave_scripts or would that only be if I made a jenkins server | 19:26 |
| sdague | but I don't want to overly penalize the team that trips over the fact that we haven't thought it all through first :) | 19:26 |
| openstackgerrit | Mathieu Gagné proposed a change to openstack-infra/config: Mark puppet-lint jobs as voting https://review.openstack.org/41502 | 19:26 |
| annegentle | clarkb: oh sorry have to go for a bit but I want to keep trying this | 19:26 |
| sdague | clarkb: well any chance you want to land things through? would love to see the last 2 patches I have in review hit before I leave today, which is that one, plus actually turning it on | 19:27 |
| sdague | also the ?content-type=text/plain is probably going to be an important thing for people to know, as the nova-api log at least cratered dhellmann's laptop | 19:28 |
| clarkb | annegentle: you should get that script in your slave | 19:29 |
| clarkb | sdague: I can do it, but would like jeblair to update his vote | 19:29 |
| sdague | yep, that's cool | 19:30 |
| clarkb | eg I don't want to override his -1 | 19:30 |
| sdague | he actually 0ed it | 19:30 |
| clarkb | oh | 19:30 |
| * clarkb looks again | 19:30 | |
| clarkb | ok, jeblair should be back from lunch in a bit I can approve if he doesn't feel strongly about it | 19:31 |
| *** ruhe has quit IRC | 19:32 | |
| *** mrodden has joined #openstack-infra | 19:34 | |
| *** mrodden1 has joined #openstack-infra | 19:36 | |
| *** mrodden has quit IRC | 19:39 | |
| openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: ignore instead of fail when reading empty config files https://review.openstack.org/38520 | 19:41 |
| *** jjmb has quit IRC | 19:44 | |
| *** UtahDave has quit IRC | 19:45 | |
| *** UtahDave has joined #openstack-infra | 19:45 | |
| *** adalbas has quit IRC | 19:48 | |
| *** jpmelos has joined #openstack-infra | 19:50 | |
| jpmelos | Hello. | 19:50 |
| jpmelos | I'm using the -t to test my layout file, I get the error voluptuous.MultipleInvalid @ data['pipelines'][0]['trigger']. | 19:51 |
| jpmelos | I'm following the syntax correctly, so I don't know what might be causing this. | 19:51 |
| jpmelos | This even happened with the sample provided in the repository. | 19:51 |
| jpmelos | Could someone give me some help here? | 19:52 |
| clarkb | jpmelos: can you paste the example zuul layout file? | 19:52 |
| jpmelos | clarkb: Sure, just a minute. | 19:52 |
| clarkb | jpmelos: as used by you | 19:52 |
| clarkb | I guess I want the file that -t is failing on | 19:53 |
| jpmelos | http://hastebin.com/detuminaje.sm <-- This is the one. | 19:54 |
| clarkb | jpmelos: if you remove line 5 'gerrit:' does it work? | 19:54 |
| clarkb | jpmelos: that was a recently addition and I wonder if your zuul version is old enough that they isn't valid | 19:55 |
| clarkb | s/recently/recent/ | 19:55 |
| jpmelos | clarkb: Wow, actually worked! | 19:56 |
| clarkb | jpmelos: cool, now we know what the problem is. If you use a newer version of zuul (like tip of master) you can avoid that | 19:56 |
| clarkb | jpmelos: zuul has recently grown the idea of different types of triggers. triggers may come from gerrit events or time based events and so on | 19:56 |
| jpmelos | clarkb: Yea, I saw. I installed it from PIP so I thought it was a recent version. My mistake. :) | 19:57 |
| jpmelos | I think I'll stick with the PIP version, assuming it's a tested version and the tip of the branch might be somewhat unstable and untested? | 19:58 |
| mgagne | jpmelos: I think master is used by openstack infra in prod | 19:59 |
| clarkb | jpmelos: we deploy directly from master | 19:59 |
| mgagne | jpmelos: what clarkb said | 19:59 |
| clarkb | so it is fairly well tested. The last release was the last time we support the jenkins http call backs though | 19:59 |
| clarkb | if you are using that method you will want to stick with it until you are ready to switch to gearman | 20:00 |
| clarkb | at this point I think we are recommending the gearman stuff beacuse it is awesome | 20:00 |
| jpmelos | Oh, since it's a fresh install I guess it's wiser to go straight to the tip of the branch then. | 20:00 |
| *** adalbas has joined #openstack-infra | 20:00 | |
| jpmelos | clarkb: I've looked into Gearman, thanks to Zuul, and it is indeed awesome, haha. | 20:01 |
| mgagne | clarkb: could it be possible to support both? | 20:01 |
| clarkb | mgagne: jeblair was pretty firm in only supporting the one we use | 20:01 |
| clarkb | mgagne: because we definitely won't test the old method like we do the gearman stuff | 20:02 |
| mgagne | clarkb: less code to maintain, I understand that =) | 20:02 |
| clarkb | and he feels strongly that gearman is the correct way to do this stuff | 20:02 |
| clarkb | the http call backs were a giant hack that required weird firewally rules and so on | 20:02 |
| clarkb | with gearman you can do client ssl auth and you only need to allow connections into the master | 20:03 |
| mgagne | clarkb: remind me of some dark stories... | 20:03 |
| *** woodspa has quit IRC | 20:03 | |
| *** jjmb has joined #openstack-infra | 20:05 | |
| jeblair | clarkb, sdague: lgtm | 20:06 |
| jeblair | aprvd | 20:06 |
| clarkb | jeblair: ty | 20:06 |
| clarkb | my turn to go quickly grab lunch. back soo | 20:06 |
| clarkb | *soon | 20:06 |
| jpmelos | clarkb: Installed Zuul from master, works beautifully. Thank you! | 20:07 |
| clarkb | jpmelos: no problem. Good luck! | 20:07 |
| clarkb | jeblair: maybe we should consider a new zuul release? | 20:07 |
| jpmelos | :q | 20:07 |
| jpmelos | ops... -.- | 20:07 |
| openstackgerrit | A change was merged to openstack-infra/config: robustify htmlifier for logs https://review.openstack.org/41486 | 20:08 |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/gearman-plugin: Rework starting/stopping executors https://review.openstack.org/41488 | 20:08 |
| jeblair | clarkb, zaro: ^ updated to make multi-exec work better | 20:08 |
| *** sandywalsh has quit IRC | 20:09 | |
| sdague | clarkb: once you get back from lunch, you need to +A this then - https://review.openstack.org/#/c/38785/ :) | 20:11 |
| openstackgerrit | A change was merged to openstack-infra/config: A few updates for RHEL based systems https://review.openstack.org/37789 | 20:13 |
| Shrews | jeblair: so, the ssl changes i made to gear a while back won't actually work correctly until we can do some exception handling around the send() and recv() calls. There were lots of send()'s when I looked, so I just haven't had time to make that change. | 20:16 |
| *** dina_belova has quit IRC | 20:16 | |
| jeblair | Shrews: good to know; i don't have a lot of practical experience in that area | 20:17 |
| Shrews | If it were redesigned to limit those to a single call, that would be super mcawesomesauce | 20:17 |
| Shrews | jeblair: i didn't either until i had to make python-gearman actually work :) | 20:18 |
| jog0 | looking for one last review on https://review.openstack.org/#/c/41218/ | 20:19 |
| jog0 | if anyone has a moment, its a reqs change | 20:20 |
| openstackgerrit | Doug Hellmann proposed a change to openstack-infra/config: Add logging to the openstack-metering IRC channel https://review.openstack.org/41512 | 20:20 |
| *** sandywalsh has joined #openstack-infra | 20:20 | |
| *** melwitt has joined #openstack-infra | 20:23 | |
| *** sandywalsh has quit IRC | 20:23 | |
| *** jjmb has quit IRC | 20:29 | |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Remove unneeded node label parameters https://review.openstack.org/41516 | 20:29 |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Make all devstack-vm jobs single-use https://review.openstack.org/41517 | 20:29 |
| *** dkliban has joined #openstack-infra | 20:30 | |
| fungi | mordred: when you're near an internet again, is gate-requirements-install assumed to be working yet? if so, have a look at the end of http://logs.openstack.org/18/41218/1/check/gate-requirements-install/f8cdc4d/console.html.gz from saturday | 20:31 |
| *** pentameter has quit IRC | 20:37 | |
| *** pabelanger_ has quit IRC | 20:37 | |
| *** pabelanger has joined #openstack-infra | 20:38 | |
| *** pentameter has joined #openstack-infra | 20:38 | |
| *** sandywalsh has joined #openstack-infra | 20:38 | |
| fungi | jog0: lgtm. it will still take me some time to get used to the requirements project having requirements of its own ;) | 20:38 |
| *** vipul is now known as vipul-away | 20:38 | |
| *** vipul-away is now known as vipul | 20:38 | |
| jog0 | fungi: wait, I was trying to change the cap on global requirments | 20:39 |
| fungi | jog0: oh! in that case i was going to ask whether you had tested that all current integrated projects were clean against the new hacking release :/ | 20:40 |
| fungi | but since you were just changing the version of hacking used by the pep8 job on the requirements project itself, and it succeeded, i was fine with it as is | 20:41 |
| jog0 | fungi: so my understanding was: changing in reqs would change for integration tests in devstack | 20:41 |
| *** andreaf has quit IRC | 20:41 | |
| jog0 | but not the unit tests where pep8 is done | 20:41 |
| fungi | jog0: oh, i thought you had to change global-requirements.txt to get that behavior | 20:41 |
| jog0 | because I know for a fact that not all projects pass hacking 0.7 | 20:41 |
| jog0 | fungi: gah | 20:41 |
| jog0 | fungi: but hacking is only a testing requirment | 20:43 |
| jog0 | which is why I put it in testing | 20:43 |
| * jog0 is confused | 20:43 | |
| fungi | jog0: but i could be mistaken. i think this may be a corner case since we can only really exercise it by running the pep8 jobs for all requirements-enforced projects with their requirements edited inline | 20:43 |
| fungi | jog0: a while back requirements.txt and test-requirements.txt in openstack/requirements got merged into one common list, and then renamed to global-requirements.txt | 20:44 |
| fungi | jog0: because the requirements project itself was growing supporting tools which would in turn have their own requirements and test requirements | 20:44 |
| jog0 | then what test-requirments? | 20:44 |
| *** dprince has quit IRC | 20:44 | |
| fungi | test-requirements.txt is the requirements needed for testing the requirements project itself | 20:45 |
| fungi | at this point | 20:45 |
| * fungi looks back at the commit history to confirm when that changed | 20:45 | |
| jog0 | git show f7fbf480daf8dbb09650a6de4f21c1a487bfdedb | 20:45 |
| fungi | jog0: https://review.openstack.org/39358 | 20:46 |
| jog0 | yeah your right, now for the follow up question if I change global-reqs does that mean unit tests use those? | 20:46 |
| fungi | ahh, yeah you found it | 20:46 |
| jog0 | or just devstack | 20:46 |
| fungi | jog0: if you change global-requirements.txt as of right now it will propose patches against all enforced projects to match the change | 20:47 |
| fungi | jog0: soon it will also keep projects from being able to merge any new changes until they correct their requirements lists to match what global-requirements.txt dictates | 20:47 |
| jog0 | fungi: propose patches is fine some will fail | 20:48 |
| jog0 | fungi: whoa intense | 20:48 |
| jog0 | although for hacking that is doable, just update the ignore list for a bit | 20:48 |
| fungi | jog0: i think there was a future to-do list item to perhaps run the unit and static analysis tests for all projects when a change is proposed to global-requirements.txt to confirm it won't break anyone | 20:48 |
| jog0 | fungi: we have a very basic test for that in hacking today | 20:49 |
| jog0 | it does nova only | 20:49 |
| jog0 | and always is green | 20:49 |
| jog0 | but the logs are useful | 20:49 |
| fungi | so as of right now, upping it won't break anyone, but i believe the goal is to have everyone on a common set of requirements by havana release time (mordred/sdague can confirm), so if this is something you expect to be working everywhere by then it's probably okay | 20:50 |
| openstackgerrit | Joe Gordon proposed a change to openstack/requirements: Bump hacking to 0.7 globally https://review.openstack.org/41523 | 20:50 |
| *** SergeyLukjanov has quit IRC | 20:51 | |
| jog0 | fungi: thanks for the clarification, perhaps test-reqs should have a big warning for dumb people like me | 20:51 |
| fungi | jog0: that's a great idea. and i think the pbr bug regarding comment lines in the requirements lists is fixed now, so i doubt there would be any resistance to some good boiler plate in the requirements-project-specfic requirements lists saying "these are not the global requirements you're looking for" or similar | 20:52 |
| fungi | i think the assumption when 39358 merged was that people would see the insanely short lists of requirements and test requirements and realize it was far from complete | 20:53 |
| *** vipul is now known as vipul-away | 20:53 | |
| fungi | but that's probably not a great assumption in reality | 20:53 |
| openstackgerrit | Joe Gordon proposed a change to openstack/requirements: Add note to test-requirements saying its local only https://review.openstack.org/41524 | 20:54 |
| fungi | and anyway, your previous patch won't go to waste. we'd have wanted to update the requirements requirements to match what went in global requirements | 20:54 |
| jog0 | fungi: right | 20:54 |
| *** ArxCruz has quit IRC | 20:56 | |
| clarkb | sdague: https://review.openstack.org/#/c/38785/ has been approved | 20:57 |
| clarkb | fungi: jeblair ^ I will keep an eye on that | 20:57 |
| fungi | clarkb: k | 20:57 |
| jog0 | $ source devstack/openrc | 20:58 |
| jog0 | No packages found matching xenserver-core. | 20:58 |
| jog0 | huh? | 20:58 |
| jog0 | from devstack | 20:58 |
| *** emagana has joined #openstack-infra | 20:58 | |
| fungi | run locally or in a ci job? | 20:58 |
| jog0 | fungi: locally | 20:59 |
| openstackgerrit | A change was merged to openstack-infra/config: enable htmlify on main log vhost https://review.openstack.org/38785 | 20:59 |
| clarkb | jog0: openrc sources stackrc which runs is_package_installed xenserver-core | 21:02 |
| clarkb | jog0: is that an error or just a helpful message? | 21:02 |
| jog0 | clarkb: its an anyoing message | 21:02 |
| jog0 | not useful | 21:02 |
| jog0 | I am using libvirt and qemu why do I have to ever see a xenserver msg | 21:02 |
| fungi | it's advertising xenserver support in devstack, clearly ;) | 21:02 |
| jog0 | fungi: thats what I am thinking too :) | 21:03 |
| clarkb | the /dev/null redirect for dpkg does not include stderr | 21:04 |
| jpmelos | Guys, I created a new flag 'Built' in Gerrit. I added a line "built: 1" to the layout.yaml but apparently Zuul didn't respond to that... Can I use new labels with Zuul, somehow? | 21:07 |
| openstackgerrit | A change was merged to openstack-infra/config: Mark puppet-lint jobs as voting https://review.openstack.org/41502 | 21:07 |
| *** ^d has quit IRC | 21:10 | |
| clarkb | jpmelos: https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/zuul/layout.yaml#L27-L29 is how we do it | 21:10 |
| clarkb | so you should be able to do something similar | 21:10 |
| *** ^d has joined #openstack-infra | 21:12 | |
| *** ^d has joined #openstack-infra | 21:12 | |
| *** mrmartin has quit IRC | 21:13 | |
| anteaya | is this the reference for the wheels that were onboarded over the weekend? https://pypi.python.org/pypi/wheel | 21:13 |
| jpmelos | clarkb: I didn't mean to use the new flag to trigger, but instead to be set by Zuul. | 21:13 |
| *** vipul-away is now known as vipul | 21:14 | |
| jpmelos | clarkb: And it actually worked, I had forgotten to add the permissions in Gerrit for Zuul user. I'm sorry! | 21:14 |
| clarkb | jpmelos: https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/zuul/layout.yaml#L32-L38 sets that | 21:14 |
| clarkb | cool | 21:14 |
| clarkb | anteaya: that is a tool for dealing with wheels. The actuall format is documented in PEP 427 which is linked off of that page | 21:15 |
| anteaya | ah okay thanks, will dig | 21:15 |
| anteaya | I do believe this is it: http://www.python.org/dev/peps/pep-0427/ | 21:16 |
| clarkb | yup | 21:16 |
| anteaya | ta | 21:17 |
| *** dina_belova has joined #openstack-infra | 21:17 | |
| *** sarob has quit IRC | 21:19 | |
| *** sarob has joined #openstack-infra | 21:20 | |
| *** dina_belova has quit IRC | 21:22 | |
| *** sarob has quit IRC | 21:24 | |
| openstackgerrit | Jeremy Stanley proposed a change to openstack-infra/config: Don't compress very, very recent log files https://review.openstack.org/41544 | 21:27 |
| *** dina_belova has joined #openstack-infra | 21:27 | |
| fungi | once we eventually merge that, i'll do one final cleanup of the files the compression cron job keeps spamming us about | 21:28 |
| *** jjmb has joined #openstack-infra | 21:28 | |
| *** sarob has joined #openstack-infra | 21:28 | |
| *** thomasm has quit IRC | 21:29 | |
| *** blamar has quit IRC | 21:29 | |
| *** sarob has quit IRC | 21:30 | |
| *** sarob has joined #openstack-infra | 21:30 | |
| *** sarob has quit IRC | 21:31 | |
| *** sarob has joined #openstack-infra | 21:31 | |
| openstackgerrit | A change was merged to openstack/requirements: Bump hacking to 0.7 https://review.openstack.org/41218 | 21:31 |
| *** dina_belova has quit IRC | 21:31 | |
| *** emagana has quit IRC | 21:33 | |
| *** emagana has joined #openstack-infra | 21:35 | |
| *** svarnau has joined #openstack-infra | 21:38 | |
| *** jjmb has quit IRC | 21:39 | |
| *** mriedem has quit IRC | 21:39 | |
| zaro | jeblair, clarkb: i get this when deleting slaves. http://paste.openstack.org/show/43975/ | 21:42 |
| zaro | jeblair, clarkb : problem was probably around before your patch. | 21:43 |
| *** emagana has quit IRC | 21:48 | |
| *** emagana has joined #openstack-infra | 21:49 | |
| *** sandywalsh has quit IRC | 21:50 | |
| *** jergerber has quit IRC | 21:54 | |
| *** jjmb has joined #openstack-infra | 21:54 | |
| *** jpeeler has quit IRC | 21:56 | |
| *** rfolco has quit IRC | 21:57 | |
| *** emagana has quit IRC | 21:59 | |
| *** HenryG has quit IRC | 22:01 | |
| *** HenryG has joined #openstack-infra | 22:02 | |
| openstackgerrit | A change was merged to openstack-infra/config: Create new repo to host legacy heat-cfn client. https://review.openstack.org/38226 | 22:03 |
| clarkb | stevebaker: ^ | 22:03 |
| *** burt has quit IRC | 22:03 | |
| *** lcestari has quit IRC | 22:04 | |
| * stevebaker does the happy dance | 22:04 | |
| stevebaker | thanks | 22:05 |
| openstackgerrit | Ryan Petrello proposed a change to openstack-infra/config: Add pecan project. https://review.openstack.org/41552 | 22:06 |
| *** sandywalsh has joined #openstack-infra | 22:06 | |
| clarkb | jeblair: any concern approving https://review.openstack.org/#/c/39329/5 it updates most (maybe all) of our jobs | 22:07 |
| clarkb | jeblair: but your comments indicated you were confident the changes wouldn't matter | 22:07 |
| *** koobs has quit IRC | 22:08 | |
| clarkb | fungi: where are we on releasing a new git-review so that we can push the pbr migration and unittesting through? | 22:08 |
| *** koobs has joined #openstack-infra | 22:08 | |
| jeblair | clarkb: i think it's good to go | 22:09 |
| clarkb | jeblair: ok I will approve | 22:09 |
| clarkb | fungi: I am reviewing that pair of git-review changes now | 22:10 |
| fungi | clarkb: yes, want to get some eyes on https://review.openstack.org/40799 and https://review.openstack.org/40800 as last-minute tweaks before we release | 22:10 |
| fungi | i assume those are the two you mean | 22:10 |
| fungi | basically the first gets pep8 (hacking) passing again, and the second is a cosmetic fix for the expanded multi-commit output | 22:11 |
| *** changbl has quit IRC | 22:11 | |
| clarkb | the pbr change needs updating, but I will leave those comments on the review | 22:11 |
| *** datsun180b has quit IRC | 22:12 | |
| clarkb | fungi: isn't hacking <0.6 really old? | 22:12 |
| fungi | ahh, yeah i figured the pbr change was somewhat out of date with pbr state of the art since that evolved after the patch | 22:12 |
| fungi | clarkb: alternatively we get git-review working with hacking 0.7 and then adjust the pbr addition accordingly | 22:12 |
| clarkb | fungi: put another way should we update git-review to deal with new hacking? | 22:12 |
| clarkb | fungi: what about hacking 0.6? | 22:13 |
| fungi | clarkb: either. i think 0.5 was simply the latest when that patch was originally drafted | 22:13 |
| fungi | so at the time <0.6 was the recommendation | 22:13 |
| clarkb | gotcha | 22:13 |
| fungi | but we didn't previously pin hacking at all, so we drifted out of compliance with the job | 22:14 |
| clarkb | ya | 22:14 |
| fungi | i was trying not to cause too much churn underneath the pbr and integration test patches figuring we could address latest hacking and whatever else after merging those | 22:14 |
| fungi | but i'm fine changig direction there if you think that's bad | 22:15 |
| clarkb | fungi: I am running the pep8 job locally to see how bad it is | 22:15 |
| clarkb | looks like it is just the shebang? is that something that will need to be fixed in hacking? if so I am fine with the current pin change | 22:16 |
| fungi | i think 0.7 was merely having trouble finding the apache license in the script | 22:16 |
| fungi | or might have been 0.6 actually | 22:16 |
| fungi | part of why i wasn't keen on bringing current git-review up to date with those checks is that there's some refactoring/moving of files going on in the pbrizaton change, so what is and isn't compliant will change there | 22:17 |
| fungi | thus it seemed like a lot of wasted energy to change it between them | 22:18 |
| fungi | figured i'd just stack another change on top of the integration test one to raise the hacking pin and make whatever adjustments were needed to bring that into compliance with the newer checks | 22:18 |
| clarkb | fungi: I am happy with that change now that I have thought about it | 22:20 |
| clarkb | fungi: in particular because I think hacking will need to learn how to treat shebangs | 22:20 |
| cody-somerville | Does anybody here know anything about this problem and what might the solution be?: https://answers.launchpad.net/devstack/+question/233793 | 22:20 |
| fungi | yeah, and then we'll be turning right around and getting rid of the shebang anyway (from its perspective) | 22:20 |
| *** jjmb has quit IRC | 22:20 | |
| fungi | cody-somerville: aww, you got my hopes up that someone had asked a ci/infrastructure services question on ask.o.o :( | 22:21 |
| clarkb | cody-somerville: I think it is related to the setuptools distribute merge and fedora being braindead about pip | 22:22 |
| *** HenryG has quit IRC | 22:22 | |
| clarkb | cody-somerville: apparently if you ensure pip 1.4 is installed everything just works | 22:22 |
| cody-somerville | I'm getting it on Ubuntu when I try to do python setup.py develop on heat trunk | 22:22 |
| fungi | right, lemme dig up a link to the relevant ml thread | 22:22 |
| *** pentameter has quit IRC | 22:22 | |
| *** HenryG has joined #openstack-infra | 22:22 | |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Add nodepool project https://review.openstack.org/41565 | 22:23 |
| fungi | cody-somerville: i think it was this monster thread where most of the discussion on it took place... http://lists.openstack.org/pipermail/openstack-dev/2013-August/013039.html | 22:24 |
| clarkb | fungi: https://review.openstack.org/#/c/40800/ has a comment but I +2'd it | 22:24 |
| clarkb | cody-somerville: long story short is setuptools and distribute did not merge in a very nice way | 22:24 |
| *** openstackgerrit has quit IRC | 22:24 | |
| clarkb | cody-somerville: pip is smart enough to work around the brokenness so upgrading pip is the path of least resistance | 22:25 |
| *** openstackgerrit has joined #openstack-infra | 22:25 | |
| *** HenryG_ has joined #openstack-infra | 22:25 | |
| clarkb | jeblair: does nodepool get its own project because it is doing the general purpose pool management? | 22:26 |
| *** jerryz has joined #openstack-infra | 22:26 | |
| *** jjmb has joined #openstack-infra | 22:26 | |
| clarkb | jeblair: and does that change invalidate the WIP change to add it to d-g? | 22:26 |
| *** dina_belova has joined #openstack-infra | 22:28 | |
| jeblair | clarkb: that's my current line of thinking; feel free to consider the review a referendum on that. it's not devstack-gate specific, so it doesn't make sense to clutter up that repo with it (and have it share a change queue). being in its own repo also means pbr, entrypoints, and other boilerplate makes a bit more sense. | 22:28 |
| fungi | clarkb: i'd love to override print with a call to textwrap.wrap() but that's a little harder to do in python 2.x :( | 22:29 |
| fungi | ooh, unless i pull in print_function | 22:29 |
| *** HenryG has quit IRC | 22:29 | |
| jeblair | clarkb: i have responded to your review of the wip change. if/when nodepool repo exists, i'll repropose there, but will update the existing change until then. | 22:29 |
| jeblair | though, heh, i think i did miss something. :) | 22:30 |
| fungi | clarkb: yeah, with print_function we can do print(*textwrap.wrap(stuff)) | 22:30 |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Add nodepool project https://review.openstack.org/41565 | 22:31 |
| *** jjmb has quit IRC | 22:32 | |
| openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Added more options to the Build Timeout plugin https://review.openstack.org/39329 | 22:32 |
| *** rfolco has joined #openstack-infra | 22:33 | |
| *** dina_belova has quit IRC | 22:34 | |
| *** emagana has joined #openstack-infra | 22:34 | |
| clarkb | jeblair: https://review.openstack.org/#/c/39773/ has my +2, but I figure you can approve when you are comfortable | 22:35 |
| clarkb | though it seems like a low risk change. The user input is only interpreted within the browser so worst case they pwn themselves | 22:35 |
| clarkb | and it is just doing string comparisons which should be safe anyways | 22:35 |
| jeblair | clarkb: the best kind of pwning | 22:35 |
| pleia2 | fungi: re: git replication - is it not creating the required git directories+files with the ssh sync at all, or just not `git init`-ing them? | 22:36 |
| fungi | pleia2: i do not believe it will do either | 22:36 |
| fungi | pleia2: though if you have test results to the contrary, i am likely to believe them | 22:36 |
| pleia2 | fungi: no results yet, just trying to figure out where we are | 22:37 |
| pleia2 | fungi: I am concerned that if we create and init the directories, then rsync them over it'll be weird for git | 22:37 |
| fungi | pleia2: i don't really think we need rsync | 22:37 |
| pleia2 | sorry, s/rsync/ssh | 22:37 |
| fungi | pleia2: git should push commits into them, starting from sha 0x0 | 22:38 |
| pleia2 | fungi: ok, so git init in an empty directory shouldn't cause the incoming sync via ssh to get confused? | 22:38 |
| fungi | pleia2: i don't think so. it's worth looking at manage_projects.py to see what it does in new project directories on the gerrit server, particularly the local mirror in /var/lib/git | 22:39 |
| * fungi looks too, can't remember exactly | 22:39 | |
| pleia2 | fungi: well, new project directories should be ok because it's all new, my concern is sshing data from an existing repo to a newly inited directory | 22:40 |
| clarkb | iirc it is just mkdir then chown | 22:40 |
| clarkb | or rather git init then chown | 22:40 |
| *** pcm_ has quit IRC | 22:41 | |
| *** pcm_ has joined #openstack-infra | 22:41 | |
| fungi | well, to pleia2's point, that's what it does for creating a new empty project, so the ~gerrit2 copy and the mirror start in the same state in that case | 22:41 |
| * pleia2 doing some tests | 22:42 | |
| fungi | but i believe gerrit will simply mirror any missing commits into the remote copy, though i strongly suspect it needs it to be a git repo first (thus the init is lkely required) | 22:42 |
| clarkb | fungi: it does the same thing if it is importing an existing project too | 22:42 |
| clarkb | where existing project is a project in gerrit with content, possible created in the same manage_projects run | 22:43 |
| pleia2 | fungi: ok, I'll test to confirm the strong suspicion | 22:43 |
| pleia2 | thanks | 22:43 |
| fungi | oh, good point. i was looking at the wrog directory i think | 22:43 |
| pleia2 | clarkb: ah, good | 22:43 |
| fungi | yup, git --bare init %s | 22:43 |
| *** prad_ has quit IRC | 22:44 | |
| fungi | unconditionally for the copy in /var | 22:44 |
| fungi | pleia2: https://github.com/openstack-infra/jeepyb/blob/master/jeepyb/cmd/manage_projects.py#L360 | 22:45 |
| clarkb | jeblair: https://review.openstack.org/#/c/40651/ looks like the sort of thing you may be interested in | 22:45 |
| jeblair | clarkb: indeed, thanks. | 22:49 |
| clarkb | fungi: https://review.openstack.org/#/c/40973/ is that something that should be abandoned or are we keeping it open to track the potential fix? | 22:49 |
| *** HenryG__ has joined #openstack-infra | 22:50 | |
| fungi | clarkb: i haven't had a chance yet to test whether the {{ }} escaping workaround will work | 22:50 |
| fungi | if it does, i think the fix is to just document it | 22:51 |
| fungi | but was waiting to see if anyone else had any serious judo they could suggest to make deep_format smarter there | 22:51 |
| clarkb | fungi: I don't. Weird issues like this are what make me think we should use a proper template format like jinja2 | 22:52 |
| clarkb | so that the library can worry about it | 22:52 |
| clarkb | but that is a very heavy dependency compared to what we currently depend on | 22:52 |
| openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Added ignore-cache and flush-cache options https://review.openstack.org/40651 | 22:52 |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Add nodepool project https://review.openstack.org/41565 | 22:53 |
| clarkb | google search pyghmi and google searches for pygmy instead | 22:53 |
| clarkb | thank you google | 22:53 |
| *** HenryG_ has quit IRC | 22:53 | |
| fungi | clarkb: pyghmi seems to be the python-ipmi project renamed | 22:54 |
| clarkb | ya, I wanted the pypi page and google normally gives that to me as thefirst hit | 22:54 |
| fungi | ahh, hah | 22:54 |
| clarkb | devananda: you about? | 22:54 |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/devstack-gate: WIP: new nodepool https://review.openstack.org/40952 | 22:54 |
| pleia2 | fungi: does the sshed content from gerrit include a .git directory? | 22:55 |
| *** pcm_ has quit IRC | 22:55 | |
| pleia2 | (it should, but dot files can be funny with ssh) | 22:55 |
| fungi | pleia2: no, it mirrors into a bare git repo, so your directory is roughly equivalent to what the .git directory would have been in a non-bare repo | 22:55 |
| pleia2 | ah, hm | 22:55 |
| clarkb | fungi: the first thing that strikes me as odd is the version at https://pypi.python.org/pypi/pyghmi/ | 22:55 |
| jeblair | pleia2: it's a 'git push' over ssh; so it's not a shell-level operation, it's running git receive-pack | 22:56 |
| clarkb | it definitely won't make pip happy | 22:56 |
| *** pcm_ has joined #openstack-infra | 22:56 | |
| fungi | which is why those directories typically end in .git | 22:56 |
| pleia2 | jeblair: ah, git push | 22:56 |
| fungi | clarkb: oh, ew. that is not sortable either | 22:56 |
| pleia2 | fungi: got it, thanks | 22:56 |
| clarkb | fungi: and we need to rename the project in stackforge and stuff too otherwise a lot of the publishing type tools will not work | 22:57 |
| mordred | scrollback. eww | 22:57 |
| fungi | mordred: i had something for you, way, way back up there | 22:57 |
| clarkb | mordred: don't read all of the scrollback just look at python-ipmi/pyghmi since that makes sense to you I think | 22:57 |
| mordred | fungi: awesome | 22:57 |
| mordred | ok | 22:57 |
| mordred | looking | 22:57 |
| mordred | so - the version is what you get from pbr before you tag anything | 22:58 |
| fungi | mordred: something about a requirements job which is succeeding while still chucking exceptions/tracebacks which look to me like the job should probably fail | 22:58 |
| fungi | but wanted a second opinion | 22:58 |
| *** svarnau has quit IRC | 22:58 | |
| *** HenryG_ has joined #openstack-infra | 22:58 | |
| *** pcm__ has joined #openstack-infra | 22:59 | |
| clarkb | https://review.openstack.org/#/c/41086/ reviewed with the info about pbr and what I think needs to happen to make the switch cleanly | 23:00 |
| mordred | clarkb: the first thing is not an issue with pip >= 1.4 | 23:02 |
| mordred | as the git sha has a letter, which marks it as pre-release | 23:02 |
| mordred | clarkb: what is it about the stackforge things that you think won't work? | 23:02 |
| *** pcm_ has quit IRC | 23:02 | |
| *** HenryG__ has quit IRC | 23:03 | |
| jeblair | mordred: python-ipmi half-renamed itself to pyghmi | 23:03 |
| clarkb | mordred: our publishing jobs depend on the name of the repo to do the right stuff | 23:03 |
| clarkb | so python-ipmi claiming to be pyghmi is not sufficient | 23:03 |
| jeblair | mordred: the contents of the repo have changed, but the repo itself has not; further, someone has registered pyghmi and uploaded a (poorly constructed) version to pypi, and has not made openstackci the owner. | 23:04 |
| mordred | great. thank you jeblair. that is clear | 23:04 |
| mordred | devananda: ^^ | 23:04 |
| *** rnirmal has quit IRC | 23:06 | |
| mordred | devananda: you need to be one or the other. also, which ever one you are, you need to get openstackci made owner of the pypi entry | 23:06 |
| mordred | devananda: if you decide to stick with pyghmi, you'll need us to lump in a repo rename with the next time we do a batch of renames | 23:07 |
| jeblair | we should make that an agenda topic for tomorrow | 23:07 |
| clarkb | ++ | 23:07 |
| devananda | mordred: yep. jbjohnso_ set up the pypi stuff. I'll nag him again to xfer it to me (or you guys) | 23:07 |
| mordred | jeblair: ++ | 23:08 |
| mordred | I've been not doing anything about the ones on my plate due to travel - I should be able to be involved/do it this weekend/friday/whenever | 23:09 |
| *** rcleere has quit IRC | 23:09 | |
| ttx | fungi: I do like it | 23:11 |
| clarkb | jog0: still around? | 23:11 |
| ttx | (the zuul status aligned progress bar thing) | 23:11 |
| fungi | ttx: ahh, yes | 23:11 |
| *** ^d has quit IRC | 23:11 | |
| clarkb | jog0: for your hacking bump you should actually bump it in both files as requirements should use a subset of the global requirements for its own testing | 23:11 |
| fungi | ttx: also did you see my code snippet for consuming the zuul status.json to find the jenkins job url on the right server? | 23:12 |
| fungi | ttx: was a few days ago | 23:12 |
| ttx | yes, just copypasted the url, will check later | 23:13 |
| ttx | fungi: thx | 23:13 |
| fungi | okay, great. hope that helps with the transition | 23:13 |
| *** fifieldt has joined #openstack-infra | 23:15 | |
| jog0 | clarkb: that was a seperate patch, because I didn't know better | 23:16 |
| jog0 | that was merged | 23:16 |
| jog0 | by both I mean global and test | 23:17 |
| *** weshay has quit IRC | 23:17 | |
| clarkb | jog0: I see, your second change confused me because it still shows <0.7 | 23:18 |
| clarkb | I will lift my -1 | 23:18 |
| clarkb | fungi: https://review.openstack.org/#/c/41544/1/modules/openstack_project/files/log_archive_maintenance.sh do we need to worry about order there to make sure we don't do a bunch of extra work. would it be easier on find to check the mod time after matching all of the other criteria? | 23:18 |
| jog0 | clarkb: it needs a rebase I think | 23:19 |
| openstackgerrit | Joe Gordon proposed a change to openstack/requirements: Bump hacking to 0.7 globally https://review.openstack.org/41523 | 23:20 |
| jog0 | there | 23:20 |
| fungi | clarkb: maybe... i'd have to play around with ordering and see if it works. some of find's options change their meaning a bit when reordered | 23:20 |
| clarkb | jog0: https://review.openstack.org/#/c/41524/ is the one that confused me | 23:21 |
| clarkb | we'll see what happens if I approve that one | 23:21 |
| clarkb | that one needs a rebase | 23:22 |
| jog0 | ahh, will rebase in a few | 23:23 |
| *** rfolco has quit IRC | 23:25 | |
| *** dina_belova has joined #openstack-infra | 23:28 | |
| *** pcrews has quit IRC | 23:31 | |
| *** lifeless has quit IRC | 23:32 | |
| *** dina_belova has quit IRC | 23:33 | |
| openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/zuul: Allow url to be updated https://review.openstack.org/41020 | 23:33 |
| clarkb | mordred: jog0 https://review.openstack.org/#/c/41355/1 isn't the example in the commit message something that is not py3k compatible and should result in an error? | 23:37 |
| *** ^d has joined #openstack-infra | 23:38 | |
| *** ^d has joined #openstack-infra | 23:38 | |
| *** rcleere has joined #openstack-infra | 23:39 | |
| jog0 | clarkb: I think that is py33 compat the 'print ()' is py33 | 23:44 |
| jeblair | zaro: i think that exception was the NPE we were getting earlier that you worked around; i suspect that it may have been a race condition, and we've eliminated or changed the race so that now when a node is deleted, the node does not have a computer by the time that is called. | 23:45 |
| jeblair | zaro: i'll update the change to make that better | 23:45 |
| openstackgerrit | Joe Gordon proposed a change to openstack/requirements: Add note to test-requirements saying its local only https://review.openstack.org/41524 | 23:46 |
| *** moted has quit IRC | 23:47 | |
| *** nati_ueno has quit IRC | 23:47 | |
| *** moted has joined #openstack-infra | 23:47 | |
| jog0 | clarkb: ^ | 23:50 |
| *** sdake__ has quit IRC | 23:51 | |
| *** sdake has joined #openstack-infra | 23:52 | |
| *** mberwanger has joined #openstack-infra | 23:52 | |
| *** dkliban has quit IRC | 23:53 | |
| *** dkliban has joined #openstack-infra | 23:54 | |
| ttx | jeblair: looks like we have a lot of bugs not being marked FixCommitted in LP lately | 23:55 |
| *** nati_ueno has joined #openstack-infra | 23:55 | |
| ttx | jeblair: there might be some issue in the recent patch. I thought it was just the ugs with lots of tasks doing timeouts, but last week there were several mono-tasks that failed too | 23:56 |
| *** gyee has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!