| *** mrodden has quit IRC | 00:01 | |
| *** mrodden has joined #openstack-infra | 00:03 | |
| *** oubiwann has joined #openstack-infra | 00:04 | |
| *** dizquierdo has left #openstack-infra | 00:07 | |
| *** alexpilotti has quit IRC | 00:10 | |
| *** datsun180b has quit IRC | 00:16 | |
| *** alexpilotti has joined #openstack-infra | 00:17 | |
| *** changbl has joined #openstack-infra | 00:20 | |
| *** matsuhashi has joined #openstack-infra | 00:21 | |
| *** prad has quit IRC | 00:22 | |
| *** anteaya has quit IRC | 00:23 | |
| *** wenlock has quit IRC | 00:31 | |
| *** gyee has quit IRC | 00:33 | |
| mordred | clarkb: about the touching of site.pp | 00:44 |
|---|---|---|
| clarkb | I had no idea either. I think we didn't notice because each of the apache passenger workers dies after 5 minutes iirc | 00:44 |
| clarkb | so the delay was ~5 minutes at most | 00:44 |
| *** Loquacities has quit IRC | 00:49 | |
| *** fifieldt_ has joined #openstack-infra | 00:52 | |
| *** markmc has quit IRC | 00:55 | |
| *** fifieldt_ has quit IRC | 00:57 | |
| *** jerryz has joined #openstack-infra | 01:03 | |
| *** prad has joined #openstack-infra | 01:06 | |
| *** katyafervent has quit IRC | 01:06 | |
| *** alexpilotti has quit IRC | 01:07 | |
| *** Loquacities has joined #openstack-infra | 01:07 | |
| *** nosnos has joined #openstack-infra | 01:08 | |
| *** nati_uen_ has joined #openstack-infra | 01:13 | |
| *** reed has quit IRC | 01:14 | |
| *** nati_ueno has quit IRC | 01:16 | |
| *** fifieldt_ has joined #openstack-infra | 01:16 | |
| *** jerryz has quit IRC | 01:17 | |
| *** nati_uen_ has quit IRC | 01:18 | |
| *** SergeyLukjanov has quit IRC | 01:20 | |
| *** prad has quit IRC | 01:21 | |
| *** katyafervent has joined #openstack-infra | 01:22 | |
| *** sarob_ has joined #openstack-infra | 01:24 | |
| zul | mordred: yaaaaay | 01:25 |
| *** ericw has quit IRC | 01:26 | |
| *** sarob has quit IRC | 01:28 | |
| *** D30 has joined #openstack-infra | 01:28 | |
| *** sarob_ has quit IRC | 01:29 | |
| *** HenryG has joined #openstack-infra | 01:32 | |
| HenryG | Is it bad for unit tests to be running "/sbin/udevadm settle --timeout=10" ? | 01:33 |
| *** UtahDave has joined #openstack-infra | 01:33 | |
| *** Loquacities has quit IRC | 01:35 | |
| *** sdague has quit IRC | 01:36 | |
| clarkb | in general I wouldn't expect unittests to mess with system level stuff | 01:37 |
| *** SergeyLukjanov has joined #openstack-infra | 01:37 | |
| *** senk has quit IRC | 01:46 | |
| HenryG | Looks like it is intended to be mocked, but with ps I see it being executed | 01:47 |
| lifeless | thats bad :) | 01:48 |
| *** sarob has joined #openstack-infra | 01:48 | |
| HenryG | I am surprised that gate-neutron-python27 passes most of the time | 01:49 |
| *** sarob has quit IRC | 01:50 | |
| HenryG | If I clone neutron and do "tox -e py27" chaos ensues. | 01:50 |
| *** sarob has joined #openstack-infra | 01:51 | |
| *** sarob has quit IRC | 01:51 | |
| *** sarob_ has joined #openstack-infra | 01:51 | |
| *** yaguang has joined #openstack-infra | 01:56 | |
| *** mriedem has quit IRC | 01:56 | |
| *** Loquacities has joined #openstack-infra | 01:56 | |
| *** SergeyLukjanov has quit IRC | 01:59 | |
| *** ericw has joined #openstack-infra | 02:08 | |
| *** senk has joined #openstack-infra | 02:13 | |
| *** ericw has quit IRC | 02:13 | |
| *** michchap has quit IRC | 02:16 | |
| *** xchu has joined #openstack-infra | 02:17 | |
| *** pabelanger has quit IRC | 02:20 | |
| *** xchu has quit IRC | 02:22 | |
| *** pabelanger has joined #openstack-infra | 02:23 | |
| *** xchu has joined #openstack-infra | 02:25 | |
| *** ericw has joined #openstack-infra | 02:28 | |
| *** fifieldt_ has quit IRC | 02:35 | |
| *** guohliu has joined #openstack-infra | 02:37 | |
| *** UtahDave has quit IRC | 02:39 | |
| *** mriedem has joined #openstack-infra | 02:41 | |
| *** mriedem has quit IRC | 02:42 | |
| *** jergerber has quit IRC | 02:43 | |
| *** senk has quit IRC | 02:45 | |
| *** senk has joined #openstack-infra | 02:46 | |
| *** nati_ueno has joined #openstack-infra | 02:50 | |
| *** julim has quit IRC | 02:59 | |
| *** guohliu has quit IRC | 03:01 | |
| *** jecarey has joined #openstack-infra | 03:03 | |
| *** matsuhashi has quit IRC | 03:04 | |
| *** matsuhashi has joined #openstack-infra | 03:05 | |
| *** matsuhashi has quit IRC | 03:09 | |
| *** guohliu has joined #openstack-infra | 03:14 | |
| *** xchu has quit IRC | 03:28 | |
| *** xchu has joined #openstack-infra | 03:29 | |
| *** matsuhashi has joined #openstack-infra | 03:39 | |
| *** pmoosh has joined #openstack-infra | 03:40 | |
| *** ericw has quit IRC | 03:41 | |
| *** senk has quit IRC | 03:42 | |
| *** guohliu has quit IRC | 03:44 | |
| *** matsuhashi has quit IRC | 03:49 | |
| *** matsuhashi has joined #openstack-infra | 03:49 | |
| *** ericw has joined #openstack-infra | 03:51 | |
| *** fifieldt has joined #openstack-infra | 03:51 | |
| *** matsuhas_ has joined #openstack-infra | 03:52 | |
| *** matsuhashi has quit IRC | 03:53 | |
| *** pmoosh has quit IRC | 03:56 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/config: Switch neutron full job to run in parallel https://review.openstack.org/51916 | 03:57 |
| *** jecarey has quit IRC | 03:57 | |
| *** oubiwann has quit IRC | 04:03 | |
| *** xchu has quit IRC | 04:12 | |
| *** nati_uen_ has joined #openstack-infra | 04:14 | |
| *** ericw has quit IRC | 04:15 | |
| *** nati_ueno has quit IRC | 04:17 | |
| *** dcramer_ has joined #openstack-infra | 04:22 | |
| *** sarob_ has quit IRC | 04:27 | |
| *** sarob has joined #openstack-infra | 04:27 | |
| *** basha has joined #openstack-infra | 04:32 | |
| *** sarob has quit IRC | 04:32 | |
| *** boris-42 has joined #openstack-infra | 04:36 | |
| *** vipul has quit IRC | 04:43 | |
| *** vipul has joined #openstack-infra | 04:43 | |
| *** yaguang has quit IRC | 04:53 | |
| *** SergeyLukjanov has joined #openstack-infra | 05:10 | |
| *** afazekas has joined #openstack-infra | 05:24 | |
| *** katyafervent has quit IRC | 05:44 | |
| *** matty_dubs|gone has quit IRC | 05:44 | |
| *** portante has quit IRC | 05:45 | |
| *** nicedice has quit IRC | 05:47 | |
| *** portante has joined #openstack-infra | 05:47 | |
| *** matty_dubs|gone has joined #openstack-infra | 05:51 | |
| *** dims has joined #openstack-infra | 05:51 | |
| *** Loquacities has quit IRC | 06:12 | |
| *** yolanda has joined #openstack-infra | 06:14 | |
| *** pblaho has joined #openstack-infra | 06:15 | |
| *** dims has quit IRC | 06:21 | |
| *** amotoki has joined #openstack-infra | 06:26 | |
| *** UtahDave has joined #openstack-infra | 06:30 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 06:34 | |
| *** _SergeyLukjanov has quit IRC | 06:35 | |
| *** SergeyLukjanov has joined #openstack-infra | 06:38 | |
| *** nati_uen_ has quit IRC | 06:52 | |
| *** basha has quit IRC | 06:52 | |
| *** nati_ueno has joined #openstack-infra | 06:53 | |
| *** yassine has joined #openstack-infra | 06:56 | |
| *** nati_uen_ has joined #openstack-infra | 06:56 | |
| *** nati_ueno has quit IRC | 06:56 | |
| *** jcoufal has joined #openstack-infra | 07:01 | |
| *** dkliban has joined #openstack-infra | 07:05 | |
| *** Loquacities has joined #openstack-infra | 07:07 | |
| *** dkliban has quit IRC | 07:12 | |
| *** odyssey4me has joined #openstack-infra | 07:17 | |
| *** SergeyLukjanov_ has joined #openstack-infra | 07:26 | |
| *** SergeyLukjanov has quit IRC | 07:28 | |
| *** slong has quit IRC | 07:30 | |
| openstackgerrit | Zhongyue Luo proposed a change to openstack-dev/hacking: Add check for comparsion operator/methods to None https://review.openstack.org/51361 | 07:32 |
| *** xchu has joined #openstack-infra | 07:32 | |
| *** SergeyLukjanov has joined #openstack-infra | 07:32 | |
| *** hashar_ has joined #openstack-infra | 07:32 | |
| *** hashar_ is now known as hashar | 07:34 | |
| *** SergeyLukjanov_ has quit IRC | 07:36 | |
| *** jcoufal has quit IRC | 07:37 | |
| *** yaguang has joined #openstack-infra | 07:41 | |
| *** yaguang has quit IRC | 07:41 | |
| *** yaguang has joined #openstack-infra | 07:42 | |
| *** nati_uen_ has quit IRC | 07:43 | |
| *** nati_ueno has joined #openstack-infra | 07:43 | |
| openstackgerrit | Zhongyue Luo proposed a change to openstack-dev/hacking: Add check for comparsion operator/methods to None https://review.openstack.org/51361 | 07:45 |
| *** SergeyLukjanov has quit IRC | 07:47 | |
| *** yaguang has quit IRC | 07:48 | |
| *** yaguang has joined #openstack-infra | 07:48 | |
| *** nati_ueno has quit IRC | 07:48 | |
| *** crank has quit IRC | 07:51 | |
| *** jcoufal has joined #openstack-infra | 07:55 | |
| *** thomasbiege1 has joined #openstack-infra | 07:55 | |
| *** rakhmerov has joined #openstack-infra | 07:56 | |
| *** Bada has joined #openstack-infra | 07:59 | |
| *** basha has joined #openstack-infra | 07:59 | |
| *** osanchez has joined #openstack-infra | 08:06 | |
| *** basha has quit IRC | 08:12 | |
| *** basha has joined #openstack-infra | 08:13 | |
| *** derekh has joined #openstack-infra | 08:16 | |
| *** jpich has joined #openstack-infra | 08:22 | |
| *** yaguang has quit IRC | 08:29 | |
| *** russellb has quit IRC | 08:35 | |
| *** russellb has joined #openstack-infra | 08:36 | |
| *** johnthetubaguy has joined #openstack-infra | 08:37 | |
| *** crank has joined #openstack-infra | 08:45 | |
| *** yaguang has joined #openstack-infra | 08:52 | |
| *** UtahDave has quit IRC | 09:20 | |
| *** matsuhas_ has quit IRC | 09:27 | |
| *** matsuhashi has joined #openstack-infra | 09:27 | |
| *** matsuhashi has quit IRC | 09:28 | |
| *** matsuhashi has joined #openstack-infra | 09:28 | |
| *** ljjjustin has joined #openstack-infra | 09:29 | |
| *** thomasbiege2 has joined #openstack-infra | 09:30 | |
| *** flaper87|afk is now known as flaper87 | 09:32 | |
| *** thomasbiege1 has quit IRC | 09:34 | |
| *** thomasbiege2 has quit IRC | 09:35 | |
| *** ljjjustin has quit IRC | 09:38 | |
| *** rpodolyaka1 has joined #openstack-infra | 09:43 | |
| rpodolyaka1 | morning all | 09:45 |
| rpodolyaka1 | it looks like iso8601 upgrade breaks oslo tests | 09:46 |
| rpodolyaka1 | https://bugs.launchpad.net/oslo/+bug/1241460 | 09:46 |
| uvirtbot | Launchpad bug 1241460 in oslo "Upgrade to iso8601 0.1.5 version breaks time_utils tests" [Undecided,New] | 09:46 |
| rpodolyaka1 | http://logs.openstack.org/20/47820/16/check/gate-oslo-incubator-python26/263c907/console.html | 09:46 |
| *** Loquacities has quit IRC | 09:53 | |
| *** yaguang has quit IRC | 09:56 | |
| *** markmc has joined #openstack-infra | 09:56 | |
| *** yaguang has joined #openstack-infra | 09:57 | |
| *** Bada has quit IRC | 09:57 | |
| *** alexpilotti has joined #openstack-infra | 10:02 | |
| *** DinaBelova has joined #openstack-infra | 10:02 | |
| *** alexpilotti has quit IRC | 10:02 | |
| *** yaguang has quit IRC | 10:04 | |
| *** ruhe has joined #openstack-infra | 10:05 | |
| *** odyssey4me2 has joined #openstack-infra | 10:06 | |
| *** odyssey4me has quit IRC | 10:06 | |
| *** xchu has quit IRC | 10:06 | |
| *** Loquacities has joined #openstack-infra | 10:06 | |
| *** ruhe has quit IRC | 10:07 | |
| *** ruhe has joined #openstack-infra | 10:11 | |
| *** Loquacities has quit IRC | 10:13 | |
| *** flaper87 is now known as flaper87|afk | 10:26 | |
| *** D30 has quit IRC | 10:29 | |
| rakhmerov | could you please take a look at https://review.openstack.org/#/c/51205 today-tomorrow | 10:33 |
| *** amotoki_ has joined #openstack-infra | 10:33 | |
| *** akscram has quit IRC | 10:39 | |
| *** akscram has joined #openstack-infra | 10:46 | |
| *** dizquierdo has joined #openstack-infra | 10:47 | |
| openstackgerrit | Ekaterina Fedorova proposed a change to openstack-infra/config: Add murano-metadataclient to stackforge https://review.openstack.org/52092 | 10:57 |
| *** ruhe has quit IRC | 10:58 | |
| *** CaptTofu has joined #openstack-infra | 11:07 | |
| *** basha has quit IRC | 11:16 | |
| *** matsuhashi has quit IRC | 11:17 | |
| *** matsuhashi has joined #openstack-infra | 11:17 | |
| *** ruhe has joined #openstack-infra | 11:18 | |
| *** sdague has joined #openstack-infra | 11:22 | |
| *** matsuhashi has quit IRC | 11:22 | |
| *** DinaBelova has quit IRC | 11:24 | |
| *** flaper87|afk is now known as flaper87 | 11:26 | |
| *** nosnos has quit IRC | 11:34 | |
| *** nosnos has joined #openstack-infra | 11:35 | |
| *** jcoufal has quit IRC | 11:35 | |
| *** nosnos has quit IRC | 11:40 | |
| *** Bada has joined #openstack-infra | 11:46 | |
| *** fbo_away is now known as fbo | 11:47 | |
| *** katyafervent has joined #openstack-infra | 11:50 | |
| *** prad has joined #openstack-infra | 11:52 | |
| *** sergmelikyan has joined #openstack-infra | 11:57 | |
| *** tsufiev has joined #openstack-infra | 11:58 | |
| *** weshay has joined #openstack-infra | 12:02 | |
| *** Bada has quit IRC | 12:03 | |
| *** alcabrera has joined #openstack-infra | 12:04 | |
| tsufiev | hi there! something strange is happening to our new murano-repository repo at stackforge, see https://review.openstack.org/#/c/52436/ Could anyone give a clue? | 12:04 |
| tsufiev | it is the first commit commit sent to gerrit | 12:05 |
| *** jcoufal has joined #openstack-infra | 12:08 | |
| rpodolyaka1 | tsufiev: try to rebase your change manually and upload a new patch set on review | 12:08 |
| tsufiev | Jenkins says that automatic merge fails, but this commit is base on the HEAD of https://github.com/stackforge/murano-repository | 12:09 |
| tsufiev | rpodolyaka, just git clone-d the repo, fetched ref from review page, submitted it - same result | 12:10 |
| sdague | yeh, looks odd | 12:12 |
| sdague | tsufiev: I just uploaded this change - https://review.openstack.org/#/c/52617/ and same results | 12:12 |
| sdague | so I suspect something is needed to be kicked on the gerrit side, will need fungi or mordred to wake up to take a look I think | 12:13 |
| tsufiev | sdague: yes, seems like some issue with Jenkins | 12:13 |
| sdague | well, I suspect the issue is the repository did not import correctly | 12:13 |
| rpodolyaka1 | sdague: hey, have you seen this https://bugs.launchpad.net/oslo/+bug/1241460 ? it looks like upgrade of iso8601 breaks tests in oslo-incubator | 12:15 |
| tsufiev | sdague: when are fungi or mordred are usually available? | 12:15 |
| uvirtbot | Launchpad bug 1241460 in oslo "Upgrade to iso8601 0.1.5 version breaks time_utils tests" [Undecided,New] | 12:15 |
| *** ruhe has quit IRC | 12:15 | |
| sdague | usually soon | 12:15 |
| *** vishy has quit IRC | 12:17 | |
| *** vishy has joined #openstack-infra | 12:18 | |
| *** mriedem has joined #openstack-infra | 12:20 | |
| *** johnthetubaguy has quit IRC | 12:23 | |
| *** dcramer_ has quit IRC | 12:25 | |
| *** prad has quit IRC | 12:25 | |
| *** ruhe has joined #openstack-infra | 12:29 | |
| katyafervent | I think jenkins jobs are not running at all. Also I did make this review, but it was cancelled (https://review.openstack.org/#/c/52018/). Could it be a reason? | 12:30 |
| *** zul has quit IRC | 12:31 | |
| *** adalbas has joined #openstack-infra | 12:32 | |
| *** dkranz has quit IRC | 12:32 | |
| *** zul has joined #openstack-infra | 12:35 | |
| *** oubiwann has joined #openstack-infra | 12:43 | |
| *** fifieldt has quit IRC | 12:44 | |
| *** steven-weston has joined #openstack-infra | 12:57 | |
| *** woodspa has quit IRC | 12:59 | |
| *** dprince has joined #openstack-infra | 13:00 | |
| *** sandywalsh has joined #openstack-infra | 13:00 | |
| *** dizquierdo has left #openstack-infra | 13:00 | |
| *** odyssey4me2 is now known as odyssey4me | 13:00 | |
| *** julim has joined #openstack-infra | 13:12 | |
| *** ruhe has quit IRC | 13:15 | |
| *** jcoufal has quit IRC | 13:23 | |
| *** jaypipes has joined #openstack-infra | 13:24 | |
| *** johnthetubaguy has joined #openstack-infra | 13:27 | |
| *** ericw has joined #openstack-infra | 13:27 | |
| *** matty_dubs|gone is now known as matty_dubs | 13:33 | |
| *** thomasbiege has joined #openstack-infra | 13:34 | |
| *** jcoufal has joined #openstack-infra | 13:35 | |
| *** wenlock has joined #openstack-infra | 13:35 | |
| mordred | hey all | 13:35 |
| mordred | tsufiev: I have seen this happen a couple of times with first commits, and I don't really understand | 13:36 |
| mordred | lemme poke | 13:36 |
| mordred | wow. same thing | 13:40 |
| mordred | fungi: ^^ I'm stumped - but I'll also still caffinating | 13:41 |
| *** rakhmerov has quit IRC | 13:42 | |
| *** rakhmerov has joined #openstack-infra | 13:43 | |
| *** rakhmerov has joined #openstack-infra | 13:44 | |
| *** rakhmerov has quit IRC | 13:44 | |
| rpodolyaka1 | mordred: hey, looks like we have a problem with the latest update of iso8601 https://bugs.launchpad.net/oslo/+bug/1241460 | 13:47 |
| uvirtbot | Launchpad bug 1241460 in oslo "Upgrade to iso8601 0.1.5 version breaks time_utils tests" [Undecided,New] | 13:47 |
| *** dhellmann-afk is now known as dhellmann | 13:47 | |
| *** ryanpetrello has joined #openstack-infra | 13:48 | |
| *** julim has quit IRC | 13:48 | |
| *** dims has joined #openstack-infra | 13:48 | |
| *** dcramer_ has joined #openstack-infra | 13:50 | |
| *** julim has joined #openstack-infra | 13:51 | |
| *** johnthetubaguy1 has joined #openstack-infra | 13:51 | |
| *** jcoufal has quit IRC | 13:52 | |
| *** jcoufal has joined #openstack-infra | 13:52 | |
| *** johnthetubaguy has quit IRC | 13:53 | |
| *** prad has joined #openstack-infra | 13:54 | |
| *** dkliban has joined #openstack-infra | 13:55 | |
| *** jcoufal-mobile has joined #openstack-infra | 13:57 | |
| *** johnthetubaguy1 is now known as johnthetubaugy | 13:57 | |
| *** salv-orlando has joined #openstack-infra | 14:00 | |
| *** jcoufal-mobile has quit IRC | 14:01 | |
| *** jcoufal has quit IRC | 14:02 | |
| *** dkliban has quit IRC | 14:05 | |
| *** odyssey4me2 has joined #openstack-infra | 14:06 | |
| *** wenlock has quit IRC | 14:07 | |
| *** odyssey4me has quit IRC | 14:08 | |
| *** eharney has joined #openstack-infra | 14:08 | |
| *** jergerber has joined #openstack-infra | 14:09 | |
| *** yolanda has quit IRC | 14:11 | |
| *** xeyed4good has joined #openstack-infra | 14:12 | |
| *** xeyed4good has left #openstack-infra | 14:12 | |
| openstackgerrit | Eric Harney proposed a change to openstack/requirements: Require python-cinderclient>=1.0.6 https://review.openstack.org/50372 | 14:13 |
| *** thomasbiege has quit IRC | 14:14 | |
| *** jergerber has quit IRC | 14:14 | |
| *** bnemec is now known as beekneemech | 14:17 | |
| annegentle | is the approval group limited on stable/relname branches for all openstack projects? (Trying to match docs up with everyone else's best practices). | 14:19 |
| *** cody-somerville has joined #openstack-infra | 14:20 | |
| *** cody-somerville has quit IRC | 14:20 | |
| *** cody-somerville has joined #openstack-infra | 14:20 | |
| openstackgerrit | David Caro proposed a change to openstack-infra/reviewstats: Added some options to openapproved https://review.openstack.org/49522 | 14:21 |
| *** steven-weston has quit IRC | 14:23 | |
| *** SergeyLukjanov has joined #openstack-infra | 14:23 | |
| *** steven-weston has joined #openstack-infra | 14:24 | |
| *** wenlock has joined #openstack-infra | 14:26 | |
| *** dizquierdo has joined #openstack-infra | 14:31 | |
| *** david-lyle_ is now known as david-lyle | 14:32 | |
| *** alexpilotti has joined #openstack-infra | 14:33 | |
| *** rnirmal has joined #openstack-infra | 14:35 | |
| mordred | annegentle: ping | 14:35 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 14:35 | |
| annegentle | mordred: pong | 14:35 |
| *** rcleere has joined #openstack-infra | 14:35 | |
| *** _SergeyLukjanov has quit IRC | 14:35 | |
| mordred | annegentle: yes, there is a general stable/* approval group - but it really only makes sense for the server projects | 14:37 |
| mordred | annegentle: other projects that have stable/* branches tend to override that setting in their ACLs | 14:38 |
| mordred | just so everyone knows - I just got this in my email: | 14:38 |
| mordred | " | 14:38 |
| mordred | For those interested in effective solutions to manage your Outlook 2007 inbox | 14:39 |
| mordred | while saving time, there will be an hands-on, 90-minute webinar: | 14:39 |
| mordred | " | 14:39 |
| mordred | So, that's pretty mind-numbingly thrilling, and I don't believe I'll be able to do any real work | 14:39 |
| mordred | today | 14:39 |
| dhellmann | there are so many things wrong with that | 14:39 |
| *** SergeyLukjanov has joined #openstack-infra | 14:40 | |
| *** dkliban has joined #openstack-infra | 14:42 | |
| *** xingchao has joined #openstack-infra | 14:42 | |
| *** salv-orlando has quit IRC | 14:46 | |
| openstackgerrit | Dan Prince proposed a change to openstack-infra/reviewday: Sort report by project name. https://review.openstack.org/52644 | 14:47 |
| *** steven-weston has quit IRC | 14:48 | |
| *** pentameter has joined #openstack-infra | 14:51 | |
| *** yaguang has joined #openstack-infra | 14:53 | |
| mordred | dprince: I just -1'd that - mainly because somewhere in my brain I need someone to explain one of those functions so I retain my sense of worth | 14:55 |
| *** datsun180b has joined #openstack-infra | 14:57 | |
| dprince | mordred: keeping me honest. I gave an explanation... | 14:57 |
| *** dmsimard has joined #openstack-infra | 14:58 | |
| mordred | dprince: k. thanks. also - weird | 14:58 |
| dmsimard | Hi guys, please excuse me - I'm not entirely familiar with the process yet but what is still necessary for https://review.openstack.org/#/c/52215/ to go through ? A core reviewer ? | 14:58 |
| mordred | dmsimard: yeah - and we've been a little swamped this week | 14:59 |
| dmsimard | mordred: You're forgiven :D | 14:59 |
| tsufiev | mordred: is there something we can do to move past first commit? | 14:59 |
| dmsimard | mordred: understandable with havana release and all | 14:59 |
| sdague | ttx: so what exactly do I need to do to set a stable/havana branch on things like grenade and devstack? | 15:01 |
| openstackgerrit | David Caro proposed a change to openstack-infra/jenkins-job-builder: Added globbed parameters to the job specification https://review.openstack.org/48688 | 15:08 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 15:09 | |
| *** steven-weston has joined #openstack-infra | 15:09 | |
| *** _SergeyLukjanov has quit IRC | 15:10 | |
| *** alexpilotti has quit IRC | 15:10 | |
| *** mrodden has quit IRC | 15:10 | |
| katyafervent | Guys some time a go it was bug with jeepyb so adding new the new repository didn't work. After it was fixed the repository was initialized but now we have jenkins-related problems (with stackforge/murano-repository). Did somebody else try to commit to just new repositories? | 15:12 |
| *** dizquierdo has left #openstack-infra | 15:13 | |
| *** SergeyLukjanov has joined #openstack-infra | 15:13 | |
| *** alexpilotti has joined #openstack-infra | 15:13 | |
| sdague | or mordred, any help one what's needed to set branches in devstack / grenade as I don't seem to have the juju to do that | 15:16 |
| *** pblaho has quit IRC | 15:21 | |
| *** mrodden has joined #openstack-infra | 15:23 | |
| *** odyssey4me has joined #openstack-infra | 15:25 | |
| *** odyssey4me2 has quit IRC | 15:29 | |
| *** ruhe has joined #openstack-infra | 15:31 | |
| *** hashar has quit IRC | 15:40 | |
| *** senk has joined #openstack-infra | 15:40 | |
| *** ruhe has quit IRC | 15:41 | |
| *** xingchao has quit IRC | 15:42 | |
| *** oubiwann has quit IRC | 15:42 | |
| *** nati_ueno has joined #openstack-infra | 15:43 | |
| *** dkliban has quit IRC | 15:46 | |
| *** UtahDave has joined #openstack-infra | 15:47 | |
| BobBall | fixup_stuff.sh doesn't take any arguments, but doesn't source stackrc either; so VIRT_DRIVER is not set, meaning the bit which skips python-lxml isn't skipped. What's the preferred fix? sourcing stackrc, or somehow making it inherit the environment from stack.sh? | 15:47 |
| *** odyssey4me has quit IRC | 15:48 | |
| *** thomasbiege has joined #openstack-infra | 15:50 | |
| *** reed has joined #openstack-infra | 15:50 | |
| *** aspiers has quit IRC | 15:51 | |
| ttx | sdague: I can create one if you give me the SHA to start it from | 15:52 |
| *** nati_ueno has quit IRC | 15:52 | |
| sdague | ttx: master on both would be fine | 15:53 |
| *** nati_ueno has joined #openstack-infra | 15:53 | |
| ttx | sdague: ok, on it | 15:54 |
| *** dkranz has joined #openstack-infra | 15:55 | |
| sdague | thanks | 15:55 |
| ttx | sdague: you have it for Grenade. Might want to push a commit like https://review.openstack.org/#/c/52598/ to set default branch on gitreview | 15:56 |
| ttx | sdague: for devstack, do we have dtroyer's signoff ? | 15:57 |
| ttx | sdague: or are you emulating it ? | 15:57 |
| sdague | I was trying to find him this morning. We talked early in the week and the last 3 patches he said he wanted in were landed | 15:57 |
| *** nati_ueno has quit IRC | 15:57 | |
| sdague | so I'm emulating it, and I think we're fine | 15:57 |
| ttx | sdague: ok, done too | 15:58 |
| sdague | thanks much | 15:58 |
| *** thomasm has joined #openstack-infra | 15:58 | |
| ttx | sdague: and congrats on your landslide victory election btw :) | 15:58 |
| sdague | thanks :) | 15:58 |
| *** thomasm is now known as Guest65238 | 15:59 | |
| *** SergeyLukjanov has quit IRC | 16:01 | |
| dtroyer | sdague, ttx: I'm ok with cutting stable/havana now, the pending stuff I'd like to add are docs-related and can easily be backported | 16:01 |
| sdague | dtroyer: cool, because we just did it :) | 16:02 |
| dtroyer | and I spent all that time reading scrollback ;) | 16:02 |
| sdague | https://review.openstack.org/#/c/52658/ | 16:02 |
| sdague | https://review.openstack.org/#/c/52659/ | 16:02 |
| sdague | those add the .gitreview changes as the first commits on the branch | 16:02 |
| sdague | dtroyer: I was also trying to think through the list of everything to do post release for grenade - https://etherpad.openstack.org/p/grenade-todos-for-havana-release | 16:03 |
| sdague | maurosr is going to start running at those | 16:03 |
| sdague | ok, lunch time here | 16:04 |
| dtroyer | I'll check it out | 16:04 |
| *** johnthetubaugy has quit IRC | 16:04 | |
| *** johnthetubaguy has joined #openstack-infra | 16:04 | |
| *** krtaylor has quit IRC | 16:08 | |
| *** sarob has joined #openstack-infra | 16:09 | |
| *** yaguang has quit IRC | 16:10 | |
| *** markmc has quit IRC | 16:10 | |
| *** jpich has quit IRC | 16:10 | |
| *** rpodolyaka1 has left #openstack-infra | 16:12 | |
| *** ruhe has joined #openstack-infra | 16:13 | |
| *** jpich has joined #openstack-infra | 16:13 | |
| *** jpich has quit IRC | 16:15 | |
| *** jasond has joined #openstack-infra | 16:16 | |
| jasond | does anybody know why https://review.openstack.org/#/c/51671/ hasn't merged yet? it was approved 2 days ago | 16:16 |
| *** SergeyLukjanov has joined #openstack-infra | 16:16 | |
| clarkb | morning. on a phone but can we make sure the oslo iso8601 thing is fixed in oslo? | 16:17 |
| clarkb | it looks like an assert needs tweaking | 16:17 |
| clarkb | jasond: recheck runs check tests not gate tests. you want reverify | 16:18 |
| jasond | clarkb: i'll try that, thanks | 16:18 |
| clarkb | jog0 any luck testing that e-r chamge? I am half tempted to put it in place and see how it does >_> | 16:20 |
| *** steven-weston has quit IRC | 16:20 | |
| *** dansmith is now known as Steely_Dan | 16:24 | |
| *** fbo is now known as fbo_away | 16:24 | |
| *** matty_dubs is now known as matty_dubs|lunch | 16:24 | |
| jog0 | clarkb: sorry haven't had a chance yet, looking now | 16:24 |
| jog0 | review 52521 ? | 16:24 |
| *** Bada has joined #openstack-infra | 16:24 | |
| jog0 | jeblair sdague: congrats | 16:24 |
| beekneemech | clarkb: I'm looking at the iso8601 thing. Do you know if it's a problem with iso8601 or Oslo? | 16:24 |
| jog0 | clarkb: running that review locally now | 16:27 |
| *** rnirmal has quit IRC | 16:28 | |
| katyafervent | mordred, Hi! It's me again with murano-repository creation :) As you now we are not able to commit anything. May be the reason the bug that was with jeepyb before? May be some reconfiguration is needed now? | 16:31 |
| *** mriedem has quit IRC | 16:31 | |
| clarkb | beekneemec: look at the failing tests. the things being compared are slightly different | 16:31 |
| clarkb | I think that oslo needs to handle either set of data | 16:32 |
| *** hogepodge has joined #openstack-infra | 16:32 | |
| *** dizquierdo has joined #openstack-infra | 16:33 | |
| *** alexpilotti has quit IRC | 16:34 | |
| mordred | clarkb: I'm stuck on the phone - if you get a chance, the murano-repository sitch is WEIRD | 16:35 |
| *** yassine has quit IRC | 16:37 | |
| *** gyee has joined #openstack-infra | 16:38 | |
| clarkb | ok I will be at a desk in ~15 minutes | 16:38 |
| *** jasond has left #openstack-infra | 16:39 | |
| beekneemech | clarkb: It was actually a bug in iso8601: https://bitbucket.org/micktwomey/pyiso8601/issue/8/015-parses-negative-timezones-incorrectly | 16:40 |
| *** Bada has quit IRC | 16:42 | |
| *** alexpilotti has joined #openstack-infra | 16:44 | |
| *** alcabrera has quit IRC | 16:46 | |
| *** hogepodge has quit IRC | 16:48 | |
| *** xeyed4good has joined #openstack-infra | 16:48 | |
| *** xeyed4good has left #openstack-infra | 16:48 | |
| jog0 | clarkb: it works! | 16:49 |
| *** derekh has quit IRC | 16:49 | |
| *** amotoki__ has joined #openstack-infra | 16:50 | |
| *** amotoki_ has quit IRC | 16:50 | |
| jog0 | mtreinish: for https://review.openstack.org/#/c/52521 the functional tests should cover the change | 16:51 |
| *** alexpilotti has quit IRC | 16:52 | |
| *** thomasbiege has quit IRC | 16:52 | |
| *** thomasbiege has joined #openstack-infra | 16:52 | |
| openstackgerrit | Ben Nemec proposed a change to openstack/requirements: Do not use iso8601-0.1.5 https://review.openstack.org/52671 | 16:53 |
| beekneemech | ^Should unblock Oslo | 16:53 |
| *** alexpilotti has joined #openstack-infra | 16:54 | |
| *** krtaylor has joined #openstack-infra | 16:54 | |
| *** Bada has joined #openstack-infra | 16:56 | |
| *** Ryan_Lane has quit IRC | 16:56 | |
| tsufiev | clarkb, katyafervent went home, if something reveals about murano-repository isssue, please address me | 16:58 |
| clarkb | tsufiev: ok, I just got to me desk will be looking at it shortly | 16:59 |
| tsufiev | clarkb, thanks! | 16:59 |
| *** hogepodge has joined #openstack-infra | 16:59 | |
| clarkb | mtreinish: re https://review.openstack.org/#/c/52521/1 what sort of unittesting did you have in mind? is it the change for getting log_url that you want a test for? | 17:00 |
| openstackgerrit | David Caro proposed a change to openstack-infra/jenkins-job-builder: Added config options to not overwrite jobs desc https://review.openstack.org/52080 | 17:01 |
| beekneemech | Crud, Oslo is still broken with iso8601-0.1.6. | 17:02 |
| clarkb | mtreinish: jog0 thinks that the functional tests cover this for us. I am still learning my way around that code base. Happy to write tests, just need a little more info | 17:02 |
| clarkb | beekneemech: same test failure? I think the data structure for the times changed slightly | 17:02 |
| clarkb | tsufiev: is the problem that all patchsets cannot be merged? | 17:03 |
| beekneemech | clarkb: It got rid of the offset errors, but the test_parse_isotime_micro_second_precision failure is still there. | 17:03 |
| tsufiev | clarkb, yes, we cannot merge the first and only patchset | 17:03 |
| beekneemech | I'm looking into what exactly is wrong. | 17:03 |
| *** nati_ueno has joined #openstack-infra | 17:05 | |
| clarkb | mordred: in zuul.openstack.org:/var/lib/zuul/git/stackforge/murano-repository git status says everything has been deleted. I believe this is the problem. I am going to move that repo aside (stash it in /root for future debugging if necessary) then zuul should reclone the repo | 17:05 |
| clarkb | this *shouldn't* require a zuul restart. I guess I am about to find out :) | 17:06 |
| *** steven-weston has joined #openstack-infra | 17:06 | |
| mordred | clarkb: wow. that project creation just messed up everything :) | 17:07 |
| *** matty_dubs|lunch is now known as matty_dubs | 17:07 | |
| BobBall | grrrr... Anyone got an idea why installing python-libguestfs in nova_plugins/hypervisor-libvirt rather than files/rpms/n-cpu might break things?... The logs at http://logs.openstack.org/42/52642/1/check/check-tempest-devstack-vm-full/b1eb92c/ show that python-libguestfs is installed, yet the log files show it's not working | 17:07 |
| mordred | BobBall: yeek. no. that makes no sense | 17:08 |
| clarkb | mordred: well that didn't work either | 17:08 |
| clarkb | mordred: but it changed the error message. I am about to look in zuul logs | 17:08 |
| BobBall | https://review.openstack.org/#/c/52642/ (first hit a different bug (too?) so I rechecked) | 17:09 |
| *** ruhe has quit IRC | 17:09 | |
| BobBall | I just have no idea on this one... not being a libvirt bloke nor having a libvirt environment is quite restricting when you're making changes to libvirt stuff :P (needed to make xenserver-core work btw - that's why I'm doing it) | 17:10 |
| openstackgerrit | Elizabeth Krumbach Joseph proposed a change to openstack-infra/config: Restore tripleo nodepool config https://review.openstack.org/52674 | 17:10 |
| pleia2 | ^^ if we could give this a shot again today I'd appreciate it | 17:10 |
| mordred | BobBall: I find just tping random commands over and over again gets the job done :) | 17:11 |
| BobBall | haha | 17:11 |
| clarkb | mordred: https://jenkins01.openstack.org/job/gate-murano-repository-python26/1/console zuul didn't reclone the repo... | 17:11 |
| *** mrmartin has joined #openstack-infra | 17:12 | |
| tsufiev | mordred, sorry, guys, we didn't intent to be so desctructive :) | 17:12 |
| mordred | tsufiev: it wasn't you | 17:13 |
| mordred | tsufiev: you just tickled a bug :) | 17:13 |
| *** osanchez has quit IRC | 17:13 | |
| clarkb | NoSuchPathError. I think there is a bug in zuul's repo recreation code. I am looking now | 17:13 |
| tsufiev | :) | 17:13 |
| clarkb | we may need to restart zuul properly | 17:14 |
| *** hogepodge has quit IRC | 17:14 | |
| *** hogepodge has joined #openstack-infra | 17:15 | |
| clarkb | or I can ninja clone the repo under zuul | 17:15 |
| *** Bada has quit IRC | 17:17 | |
| *** ruhe has joined #openstack-infra | 17:17 | |
| clarkb | I see the bug in zuul, I will write a fix for that, but first I think I can ninja clone under zuul so let me give that a shot | 17:17 |
| *** oubiwann has joined #openstack-infra | 17:19 | |
| *** oubiwann_ has joined #openstack-infra | 17:20 | |
| *** alexpilotti has quit IRC | 17:21 | |
| *** dizquierdo has quit IRC | 17:22 | |
| *** oubiwann has quit IRC | 17:24 | |
| clarkb | that didn't quite work either | 17:26 |
| *** sarob has quit IRC | 17:26 | |
| *** sarob has joined #openstack-infra | 17:27 | |
| *** steven-weston has quit IRC | 17:27 | |
| *** alexpilotti has joined #openstack-infra | 17:28 | |
| BobBall | is it possible to download from logs.openstack.org in such a way that doesn't display the txt.gz files as HTML and I can actually get the files? | 17:29 |
| *** dprince has quit IRC | 17:29 | |
| *** flaper87 is now known as flaper87|afk | 17:30 | |
| openstackgerrit | Adrian Otto proposed a change to openstack-infra/config: Adding project Solum to StackForge https://review.openstack.org/52683 | 17:30 |
| clarkb | BobBall: yes, download them with something that doesn't accept html files | 17:31 |
| clarkb | eg wget | 17:31 |
| *** sarob has quit IRC | 17:31 | |
| BobBall | ah... | 17:31 |
| BobBall | I tried to use downloadthemall from within firefox - just assumed it'd be the same1 | 17:31 |
| BobBall | thanks | 17:32 |
| clarkb | mordred: IndexError: No item found with id 'origin/HEAD' that is happening because my manual git clone didn't create a refs/remotes/origin/HEAD. Any idea of how to get a refs/remotes/origin/HEAD without writing one directly? | 17:32 |
| mordred | clarkb: gross. no! | 17:32 |
| clarkb | is that as simple as a git fetch origin/HEAD ? | 17:32 |
| * clarkb experiments | 17:33 | |
| clarkb | I think we are relly close to having this work though | 17:33 |
| *** sarob has joined #openstack-infra | 17:35 | |
| openstackgerrit | A change was merged to openstack-infra/reviewday: Sort report by project name. https://review.openstack.org/52644 | 17:35 |
| sdague | BobBall: or specify ?content-type=text/plain as a param | 17:35 |
| pleia2 | my first +A :) | 17:35 |
| pleia2 | (it's about time) | 17:35 |
| sdague | pleia2: nice | 17:35 |
| mtreinish | clarkb: I just was thinking a fake response for both formats in test_classifier to make sure a future change doesn't break anything | 17:37 |
| mtreinish | but if jog0 says it's ok we can add more tests later I guess | 17:37 |
| clarkb | mordred: I see what is happening. I was using an https remote then switching back to ssh:// but deleting removes the refs... argh | 17:37 |
| clarkb | mordred: so now I need to figure out the ssh magic zuul uses then it *should* work I think | 17:38 |
| *** alcabrera has joined #openstack-infra | 17:39 | |
| *** CaptTofu has quit IRC | 17:40 | |
| *** CaptTofu has joined #openstack-infra | 17:41 | |
| beekneemech | Hmm, I'm pretty sure the remaining Oslo issue is also a bug in iso8601. Looks like it's dropping leading 0's from the ms part of the time. | 17:41 |
| beekneemech | Hah, dims beat me to it: https://bitbucket.org/micktwomey/pyiso8601/issue/9/regression-parsing-microseconds | 17:43 |
| *** Ryan_Lane has joined #openstack-infra | 17:43 | |
| *** CaptTofu has quit IRC | 17:45 | |
| clarkb | mordred: tsufiev: success! https://jenkins01.openstack.org/job/gate-murano-repository-pep8/2/console | 17:46 |
| clarkb | mtreinish: ok, once I fix this bug in zuul (so I don't forget) I will look at writing the test, if it isn't horrible I think I should just do it now | 17:46 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 17:47 | |
| *** _SergeyLukjanov is now known as SergeyLukjanov | 17:47 | |
| clarkb | mordred: argh, but I forgot to set the user name and email | 17:48 |
| clarkb | mordred: tsufiev: I am going to reclone it again and set those properly so that we don't deal with potential problems down the road, please don't approve anything | 17:49 |
| clarkb | actually nevermind those items are in if blocks | 17:53 |
| dims | beekneemech, "0.%06d" does not do the trick :( | 17:54 |
| beekneemech | dims: I just discovered that. :-( | 17:54 |
| clarkb | mordred: tsufiev: https://jenkins01.openstack.org/job/gate-murano-repository-python27/3/console I think it is good to go now | 17:56 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 17:57 | |
| *** _SergeyLukjanov is now known as SergeyLukjanov | 17:57 | |
| * koolhead17 wonders if he can finally get some time from mordred our release is out finally | 17:58 | |
| openstackgerrit | Clark Boylan proposed a change to openstack-infra/zuul: Clone repo if it is not on disk https://review.openstack.org/52689 | 17:59 |
| *** nicedice has joined #openstack-infra | 17:59 | |
| clarkb | mordred: tsufiev ^ is the fix for zuul | 17:59 |
| clarkb | koolhead17: hahahahahaha | 17:59 |
| koolhead17 | clarkb: hi man. | 17:59 |
| clarkb | koolhead17: hi there | 17:59 |
| koolhead17 | seriously where is mordred :D | 17:59 |
| koolhead17 | BTW i have registered a talk about our infra and devops use for event in india | 18:00 |
| koolhead17 | needed some slide/pointer if someone already covered it | 18:00 |
| clarkb | koolhead17: http://docs.openstack.org/infra/publications | 18:01 |
| clarkb | there is a general overview talk there | 18:01 |
| clarkb | mtreinish: does TestClassifier query the running elasticsearch? | 18:01 |
| koolhead17 | clarkb: ya my plan was how we are having such big infra well in place with devops | 18:02 |
| mtreinish | clarkb: no it has stubs out all the elasticsearch calls | 18:02 |
| koolhead17 | and what all are our tools | 18:02 |
| openstackgerrit | Ben Nemec proposed a change to openstack/requirements: Do not use iso8601-0.1.5 or 0.1.6 https://review.openstack.org/52671 | 18:02 |
| *** ruhe has quit IRC | 18:02 | |
| mtreinish | clarkb: I have a fake method that returns what I interpreted as the dict response from pyelasticsearch (or at least enough to get through the classify method) | 18:04 |
| *** sarob has quit IRC | 18:04 | |
| clarkb | mtreinish: ya looking for that now. I see the fixture but not the method | 18:05 |
| *** sarob has joined #openstack-infra | 18:05 | |
| *** wenlock has quit IRC | 18:05 | |
| *** wenlock has joined #openstack-infra | 18:06 | |
| mtreinish | clarkb: oh it hasn't merged yet | 18:07 |
| mtreinish | oops | 18:07 |
| mtreinish | clarkb: https://review.openstack.org/#/c/50733/ | 18:07 |
| mtreinish | I saw jeblair +2 and assumed it was approved | 18:07 |
| clarkb | ah | 18:07 |
| clarkb | mtreinish: are you ok with getting my change in now, then your change, then a third change to test the new stuff? | 18:07 |
| mtreinish | clarkb: yeah that's fine | 18:08 |
| clarkb | (I am trying to make sure that all of the pieces are in place before monday so that I can do that overhaul early next week | 18:08 |
| mtreinish | clarkb: yeah that's a priority, it's not like the e-r code really has good unit tests yet | 18:08 |
| *** sarob has quit IRC | 18:10 | |
| *** alexpilotti has quit IRC | 18:11 | |
| *** ruhe has joined #openstack-infra | 18:13 | |
| sdague | mtreinish: just +Aed that | 18:13 |
| *** johnthetubaguy has quit IRC | 18:14 | |
| *** alexpilotti has joined #openstack-infra | 18:14 | |
| clarkb | sdague: gah am I going to have to rebase and write those tests now? :) | 18:16 |
| sdague | oh, ooops :) | 18:16 |
| clarkb | sdague: its fine, I will look at it now | 18:17 |
| mtreinish | clarkb: the only changes are on the test side, so as long as you didn't break the old way you shouldn't have any issues | 18:18 |
| clarkb | mtreinish: awesome, I shouldn't have broken the old way. It should be forward and backward compatible | 18:20 |
| sdague | jog0 / mtreinish: https://review.openstack.org/#/c/52509/ would be nice as well | 18:22 |
| mtreinish | clarkb: well then rebasing will be good so we can confirm with a gating check | 18:22 |
| clarkb | mtreinish: zuul will rebase for us, so as long as it rebases cleanly we shouldn't need to change anything | 18:22 |
| *** thedodd has joined #openstack-infra | 18:23 | |
| mtreinish | clarkb: true, +A'd | 18:24 |
| *** thedodd has quit IRC | 18:24 | |
| *** thedodd has joined #openstack-infra | 18:24 | |
| clarkb | woot thanks | 18:25 |
| *** Guest65238 is now known as thomasm | 18:25 | |
| *** thomasm is now known as thomasm_ | 18:25 | |
| dims | beekneemech, woot :) | 18:25 |
| *** thomasm_ is now known as thomasem | 18:25 | |
| mtreinish | clarkb: heh, it actually looks like some of the classify class changes broke my unit tests | 18:26 |
| mtreinish | it's been running for 12min | 18:26 |
| *** senk has quit IRC | 18:26 | |
| mtreinish | it'll eventually fail when is_ready gives up | 18:26 |
| beekneemech | dims: Yeah, now we just need another new release of iso8601. :-) | 18:26 |
| *** milki has joined #openstack-infra | 18:28 | |
| clarkb | mtreinish: the changes I made or other changes? | 18:28 |
| mtreinish | I'm thinking it's the required files additions | 18:28 |
| clarkb | ah | 18:28 |
| *** tsufiev_ has joined #openstack-infra | 18:28 | |
| clarkb | tsufiev_: I think you should be good to go now | 18:29 |
| clarkb | milki: hi, did you have questions about JJB? | 18:29 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Make e-r compatible with old/new logstash schemas. https://review.openstack.org/52521 | 18:30 |
| milki | clarkb: just exploring how contrib works. probably gonna fork it so i can submit changes back | 18:30 |
| *** rnirmal has joined #openstack-infra | 18:30 | |
| milki | looks like its launchpad and gerrit | 18:30 |
| clarkb | milki: yup | 18:30 |
| milki | now i wont accidentally submit a pull request on githbu | 18:31 |
| clarkb | The workflow is documented at https://wiki.openstack.org/wiki/How_To_Contribute#If_you.27re_a_developer | 18:31 |
| *** hogepodge has quit IRC | 18:32 | |
| milki | thanks, ill read up | 18:33 |
| tsufiev_ | clarkb, thank you, it seems that merging issue is solved now. there are some errors still, should investigate them on monday | 18:33 |
| jog0 | sdague: hehe | 18:35 |
| mordred | clarkb: wow. are we missing fungi and jeblair today both? and also I've been useless so far? | 18:36 |
| pleia2 | mordred: we could totally not break nodepool by adding tripleo back (I tested it!) | 18:37 |
| clarkb | mordred: yup, please don't break anyting :) | 18:37 |
| *** w_ has joined #openstack-infra | 18:37 | |
| pleia2 | or should we wait for jeblair? :) | 18:37 |
| clarkb | pleia2: mordred: I am game for trying it after lunch | 18:37 |
| mordred | clarkb: well, pleia2 says we cna add tripleo back to nodepool | 18:37 |
| mordred | neat | 18:37 |
| pleia2 | cool | 18:37 |
| *** senk has joined #openstack-infra | 18:38 | |
| *** olaph has quit IRC | 18:38 | |
| sdague | wow, I wish I had any basic understanding about why pyelasticsearch results drift back and forth from single element arrays to strings | 18:39 |
| mordred | sdague: it's because beards are awesome | 18:40 |
| clarkb | sdague: oh, check if multilien is in the tags. I think on multiline events they become lists for some reason | 18:40 |
| sdague | mordred: I'd almost understand it if it was per search result set | 18:40 |
| sdague | but it's in the middle of a result set | 18:40 |
| mordred | wow. that's impressive | 18:40 |
| clarkb | sdague: but per event right? I think multiline may be to blame. check if multiline is in the tags list | 18:40 |
| *** rwsu has quit IRC | 18:41 | |
| *** echohead has joined #openstack-infra | 18:42 | |
| sdague | clarkb: yep, I think you are right | 18:42 |
| sdague | http://paste.openstack.org/show/48728/ | 18:42 |
| clarkb | sdague: I think that is a side effect of multiline collapsing events | 18:43 |
| *** rwsu has joined #openstack-infra | 18:43 | |
| clarkb | sdague: the way we use mutliline none of the fields should change so you get [ "foo" ] but you could have ["foo", "bar"] | 18:43 |
| sdague | ok, that at least is an explanation, and one I can work with :) | 18:44 |
| sdague | so different question, anyone got a good pointer to basic meta programming attributes in python? as these result sets are going to be much nicer if we do it that way | 18:45 |
| clarkb | sdague: look at sqlalchemy? (I don't actually) | 18:45 |
| clarkb | there is a dunder method that you can do dynamic lookups with right? that might be simplest | 18:45 |
| clarkb | but I think you can set the meta class to something other than type to do the work up front | 18:46 |
| * jesusaurus tends to poke at __dict__ in ipython to find the right dunders | 18:47 | |
| *** senk has quit IRC | 19:00 | |
| *** yolanda has joined #openstack-infra | 19:04 | |
| *** senk has joined #openstack-infra | 19:06 | |
| *** alcabrera has quit IRC | 19:09 | |
| *** sarob has joined #openstack-infra | 19:15 | |
| *** sarob has quit IRC | 19:22 | |
| *** senk has quit IRC | 19:23 | |
| *** reed has quit IRC | 19:30 | |
| *** wenlock is now known as cylon | 19:32 | |
| *** nati_uen_ has joined #openstack-infra | 19:32 | |
| *** cylon has quit IRC | 19:34 | |
| *** nati_ueno has quit IRC | 19:34 | |
| bodepd_ | could I get a review here: https://review.openstack.org/#/c/52215/ | 19:38 |
| *** tsufiev_ has quit IRC | 19:41 | |
| *** thomasem has quit IRC | 19:43 | |
| *** sarob has joined #openstack-infra | 19:47 | |
| *** alexpilotti has quit IRC | 19:49 | |
| *** hashar has joined #openstack-infra | 19:51 | |
| BobBall | Some advice pls... I made a change to devstack at https://review.openstack.org/#/c/52642/ which moved python-libguestfs and enabled it - causing tests to fail in the gate. It seems that the current way it is added in the gate is just broken. I need to stop it installing automatically for xenserver-core, so I noprime'd it... This made the tests continue to pass. Should I noprime it, delete it completely, or something else? | 19:51 |
| BobBall | (the enabling it was clearly a mistake, but since it was in lib/nova it was _meant_ to be used somewhere) | 19:52 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: first iteration on ResultSet and Hit objects https://review.openstack.org/52698 | 19:53 |
| clarkb | BobBall: I think dtroyer may know best | 19:54 |
| *** senk has joined #openstack-infra | 19:54 | |
| clarkb | BobBall: that seems like a devstack issue not a gate issue | 19:54 |
| clarkb | bodepd_: looking | 19:54 |
| clarkb | pleia2: mordred: ready to give nodepool a shot? | 19:54 |
| pleia2 | yep | 19:55 |
| BobBall | sure, it's a devstack change, but by enabling it I broke the gate tests - clearly libvirt does not work with python-libguestfs as it's installed in the gate... The logs of the tests that pass that I've seen have skipped python-libguestfs | 19:55 |
| clarkb | BobBall: right but that has nothing to do with the gate | 19:55 |
| clarkb | BobBall: it sounds like it breaks devstack which causes the gate to fail | 19:56 |
| *** rcleere has quit IRC | 19:56 | |
| *** rnirmal has quit IRC | 19:56 | |
| clarkb | BobBall: I am not in a position to talk to devstack and nova's use of python-libguestfs, but dtroyer should be | 19:56 |
| BobBall | well devstack is happy in both cases - it's the nova tests that fail | 19:56 |
| BobBall | okies | 19:56 |
| clarkb | bodepd_: mgagne: re https://review.openstack.org/#/c/52215/1 I thought you wanted no CLA? should the CLA requirement be removed? | 19:56 |
| mgagne | clarkb: what's your take on it? what's the legal implications? | 19:57 |
| *** sarob has quit IRC | 19:57 | |
| *** sarob has joined #openstack-infra | 19:57 | |
| clarkb | mgagne: I think many of us in general are anti CLA around here ... For stackforge projects I think they only make sense if the intention is to become incubated | 19:58 |
| clarkb | mordred: ^ thoughts? | 19:58 |
| clarkb | mgagne: I ask, because CLA stuff was specifically asked about in regards to puppet-ceph and jeblair pointed out it is not required which made stackforge more appealing to folks | 19:59 |
| clarkb | pleia2: I am reviewing the restored change now | 19:59 |
| bodepd_ | mgagne: let's remove it | 19:59 |
| mgagne | BobBall: ok | 19:59 |
| bodepd_ | clarkb: thanks for looking into it | 19:59 |
| bodepd_ | clarkb: I always hate to be a pain, but there are tons of folks waiting for that | 20:00 |
| clarkb | bodepd_: you should see my pending review list | 20:00 |
| pleia2 | clarkb: thanks, it's prety much a manual revert of https://review.openstack.org/#/c/51469/ - but that's already in a revert-revert-revert chain so I didn't want to mess with it directly | 20:00 |
| *** oubiwann_ has quit IRC | 20:01 | |
| *** krtaylor has quit IRC | 20:01 | |
| *** oubiwann has joined #openstack-infra | 20:01 | |
| *** senk has quit IRC | 20:01 | |
| openstackgerrit | Mathieu Gagné proposed a change to openstack-infra/config: Add puppet-ceph project to StackForge https://review.openstack.org/52215 | 20:01 |
| bodepd_ | mgagne: you are the man!!! thanks!!! | 20:02 |
| mgagne | np | 20:02 |
| clarkb | pleia2: my one concern is I thought there was a config format change, double checking that now | 20:02 |
| pleia2 | clarkb: I had already rebased against that | 20:03 |
| dtroyer | BobBall: so, trying to catch up... | 20:03 |
| clarkb | pleia2: cool, lgtm. Let's see if we can get mordred to chime in before you dive into the deep end | 20:05 |
| clarkb | mordred: please review https://review.openstack.org/#/c/52674/ if possible | 20:05 |
| pleia2 | thanks | 20:05 |
| dtroyer | BobBall: you really just want python-libguestfs not installed on XenServer? can you just NOPRIME that one (leave the others alone) and do a test in lib/nova to skip the install if VIRT_DRIVER=xen? | 20:05 |
| clarkb | mordred: also https://review.openstack.org/#/c/52215/. Have we fixed manage-projects yet? or will it blow up if we merge that? | 20:06 |
| BobBall | that's what I tried dtroyer - but it wasn't installed in lib/nova before - so I added it to lib/nova_plugins/hypervisor-libvirt to install it - but it seems it has never been used in the gate (previous tests that pass fall back to using the localfs driver rather than the guestfs driver) | 20:07 |
| clarkb | looks like we haven't fixed jeepyb and manage_projects yet | 20:07 |
| *** mrmartin has quit IRC | 20:08 | |
| clarkb | bodepd_: ^ note that the automagic project creation stuff isn't currently working | 20:08 |
| BobBall | dtroyer: check out patch set 1 for https://review.openstack.org/#/c/52642/ - that was where I optionally enabled python-libguestfs for libvirt only and it caused failures | 20:08 |
| dtroyer | BobBall: in patchset 1, you added for ubuntu, rhel and suse. onle rhel has it normally…so could you either skip the install or uninstall it under xenserver. LEave the others alone as they don't use it now | 20:08 |
| BobBall | what do you use in the gate? I assumed suse... | 20:09 |
| BobBall | I meant rhel* | 20:09 |
| dtroyer | BobBall: it looks like Ian added it to rhel due to a lack of nbd support... | 20:09 |
| sdague | BobBall: the gate is ubuntu 12.04 | 20:09 |
| BobBall | oh | 20:09 |
| BobBall | that was where I went wrong! | 20:09 |
| sdague | yeh, I would love to see other distros in the gate as well, just no one else has stepped up there | 20:10 |
| * BobBall was convinced it was actually CentOS, hence the change I made to deb/suse was irrelevent... | 20:10 | |
| sdague | heh | 20:10 |
| sdague | yeh, the other way around | 20:10 |
| BobBall | we'll be adding centos with xenserver-core soon I hope :) | 20:10 |
| dtroyer | centos is used for py26 somewhere... | 20:10 |
| sdague | break suse and rhel all you want :) | 20:10 |
| BobBall | w00t. I'll quote you on that sdague. | 20:10 |
| sdague | dtroyer: centos is only used for unit tests | 20:10 |
| dtroyer | ah, ok... | 20:10 |
| *** alexpilotti has joined #openstack-infra | 20:13 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 20:14 | |
| *** _SergeyLukjanov has quit IRC | 20:15 | |
| dmsimard | clarkb: How do we get the repo created then ? (puppet-ceph) | 20:15 |
| clarkb | dmsimard: we fix jeepyb, then merge mgagne's change | 20:15 |
| mordred | clarkb: there's a fix proposed | 20:16 |
| clarkb | https://review.openstack.org/#/c/52195/ is the proposed fix. If mordred shows up | 20:16 |
| clarkb | there he is | 20:16 |
| dmsimard | clarkb, mordred: Okay ! | 20:16 |
| clarkb | mordred: should I update 52195, to address your comments then we will go with that? | 20:16 |
| mordred | clarkb: sure. sorry - I've been in meeting hell all day | 20:17 |
| mordred | clarkb: on the tripleo config - lgtm - want me to aprvf it? | 20:17 |
| clarkb | mordred: go for it | 20:18 |
| *** wenlock has joined #openstack-infra | 20:18 | |
| mordred | pleia2: patch approved. fingers crossed | 20:18 |
| pleia2 | :) | 20:19 |
| *** beagles has joined #openstack-infra | 20:19 | |
| openstackgerrit | A change was merged to openstack-infra/config: Restore tripleo nodepool config https://review.openstack.org/52674 | 20:19 |
| clarkb | mordred: I didn't get a chance to review your jeepyb changes ... I should swing around and look at them at some point. Are we no longer using temporary git repos? | 20:20 |
| clarkb | mordred: and are tracking state in external repos? | 20:20 |
| BobBall | btw - is there a way to look back at the whole of my history in gerrit? There was a patch a few months ago that expired and I want to revive :) | 20:21 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: first iteration on ResultSet and Hit objects https://review.openstack.org/52698 | 20:21 |
| clarkb | BobBall: you can do a query like status:abandoned owner:bobball project:openstack/foo then dig through the list | 20:21 |
| openstackgerrit | Clark Boylan proposed a change to openstack-infra/jeepyb: Create local git mirror _after_ creating project https://review.openstack.org/52195 | 20:23 |
| clarkb | mordred: ^ | 20:23 |
| BobBall | oh brill - it was there! thanks blarkb. | 20:23 |
| BobBall | clarkb* | 20:23 |
| clarkb | np | 20:23 |
| mordred | apprv | 20:23 |
| clarkb | mordred: will you be around to babysit the puppet-ceph thing? if so I think you should review that and approve it after 52195 is applied to review.o.o | 20:24 |
| * mordred thinks clarkb should start going by blarb on casual nick fridays | 20:24 | |
| mordred | blarkb | 20:24 |
| openstackgerrit | A change was merged to openstack-infra/jeepyb: Create local git mirror _after_ creating project https://review.openstack.org/52195 | 20:24 |
| mordred | clarkb: I'm running to grocery story - will baby sit puppet-ceph when I get back | 20:24 |
| clarkb | (I haven't kept up on all of the manage-project changes well enough to feel comfortable if it goes sideways especially with the tripleo nodepool change happening too) | 20:24 |
| clarkb | mordred: perfect | 20:25 |
| bodepd_ | mordred: thanks! | 20:25 |
| clarkb | pleia2: do you know if nodepool rereads the yaml file on its own? | 20:26 |
| clarkb | I think it does, but am double checking | 20:27 |
| pleia2 | oh, I'm not sure | 20:27 |
| clarkb | pleia2: looks like it does. while not self._stopped: config = self.loadConfig | 20:27 |
| *** ruhe has quit IRC | 20:32 | |
| *** fallenpegasus has joined #openstack-infra | 20:35 | |
| *** thomasm has joined #openstack-infra | 20:35 | |
| *** thomasm has quit IRC | 20:36 | |
| *** thomasm has joined #openstack-infra | 20:37 | |
| milki | clarkb: for jjb, is there a proper way to not overwrite existing job values (set manually in jenkins) or query for the current job values? this 52080 change suggests a None value can do that for desc, but I don't believe that since the xml is uploaded as is. | 20:38 |
| *** SergeyLukjanov has joined #openstack-infra | 20:38 | |
| clarkb | milki: there isn't really. I think the description thing is a hack | 20:40 |
| clarkb | in the general case there isn't a way for doing tha | 20:40 |
| pleia2 | clarkb: ooh, I see a template pop up on the tripleo cloud! (so it's building) | 20:40 |
| clarkb | woot | 20:40 |
| *** sandywalsh has quit IRC | 20:41 | |
| milki | clarkb: what is the common method to get passwords in since the xml uses a hashed value for that? for now, we are jsut manually setting that and grabbing the hash | 20:41 |
| clarkb | milki: I believe that is what people are doing. JJB has no way of compiling XML to yaml or an internal representation. JJB is all one way | 20:42 |
| clarkb | yaml -> XML | 20:42 |
| milki | ok | 20:43 |
| milki | i need to test if the hashes change then per instance | 20:43 |
| clarkb | pleia2: how is it going? | 20:46 |
| *** nati_uen_ has quit IRC | 20:46 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: first iteration on ResultSet and Hit objects https://review.openstack.org/52698 | 20:47 |
| *** nati_ueno has joined #openstack-infra | 20:47 | |
| clarkb | I don't see the nodes on jenkins0X | 20:47 |
| clarkb | oh because we need to build images and take snapshots first | 20:47 |
| *** rnirmal has joined #openstack-infra | 20:48 | |
| pleia2 | clarkb: yeah, it takes about 20-25 minutes to build the snapshot, I just have the view on the nova side | 20:48 |
| pleia2 | still chugging away | 20:48 |
| clarkb | pleia2: I found I can tail the image log to see it building so I am following along too | 20:48 |
| pleia2 | cool | 20:48 |
| pleia2 | you'll see apt and puppet commands trickle in here and there :) | 20:48 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 20:51 | |
| clarkb | mtreinish: if you haven't taken off for the weekend I recall seeing a parallel neutron testing change. Is that ready to go? | 20:51 |
| *** _SergeyLukjanov has quit IRC | 20:52 | |
| mtreinish | clarkb: ehh sort of, it just changes the experimental full neutron job from serial to parallel so it really doesn't effect anything | 20:52 |
| mtreinish | actually you know what it won't work | 20:52 |
| mtreinish | because tenant isolation isn't enabled for that run let me respin that | 20:53 |
| clarkb | mtreinish: ok | 20:53 |
| clarkb | pleia2: I see them! it is refreshing service[ntp] | 20:53 |
| *** thedodd has quit IRC | 20:54 | |
| clarkb | pleia2: and we have failure :( couldn't fetch setuptools-git from pypi.python.org/simple | 20:54 |
| *** sandywalsh has joined #openstack-infra | 20:54 | |
| clarkb | pleia2: I wonder, do the d-g image builds use our mirror? should we maybe use the mirror from tripleo land too? | 20:54 |
| *** thedodd has joined #openstack-infra | 20:55 | |
| lifeless | clarkb: https://review.openstack.org/#/c/46996/ | 20:55 |
| *** miqui has quit IRC | 20:55 | |
| pleia2 | clarkb: it survives | 20:55 |
| clarkb | lifeless: yes, I didn't approve because of release and feature freeze | 20:56 |
| clarkb | lifeless: mordred claims it is safe, but I defer to him because I don't want ttx hunting me down :) | 20:56 |
| *** cody-somerville has quit IRC | 20:56 | |
| clarkb | pleia2: yeah, puppet will continue but I think nodepool will discard the image and try again | 20:57 |
| pleia2 | clarkb: and re: mirror, yeah, we probably should | 20:57 |
| pleia2 | clarkb: I think I saw that error too and it finished ok | 20:57 |
| clarkb | ah | 20:57 |
| *** jaypipes has quit IRC | 20:57 | |
| clarkb | I guess we will find out shortly :) | 20:57 |
| pleia2 | indeed! | 20:57 |
| lifeless | clarkb: we're past that now though right? | 20:58 |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/config: Switch neutron full job to run in parallel https://review.openstack.org/51916 | 20:58 |
| *** mtreinish has quit IRC | 20:58 | |
| clarkb | lifeless: we are, but I am still deferring | 20:59 |
| lifeless | clarkb: I am confused :). Do you mean you want mordred to click the button ? | 20:59 |
| *** senk has joined #openstack-infra | 20:59 | |
| clarkb | lifeless: yes | 20:59 |
| lifeless | clarkb: ok | 20:59 |
| lifeless | mordred: ^ | 20:59 |
| clarkb | lifeless: ttx and mordred are the only people that seem to understand that stuff, so I defer a change that changes how we version things to them | 20:59 |
| mordred | what? | 21:00 |
| lifeless | clarkb: thats fine, it wasn't clear to me - and I suspect to mordred - that noone was going to click 'approved' :) | 21:00 |
| lifeless | mordred: you need to click on approved on https://review.openstack.org/#/c/46996/ | 21:00 |
| lifeless | mordred: it has 2x+2 | 21:00 |
| mordred | yes. I was just waiting until the relesae just to be sure | 21:00 |
| lifeless | mordred: which we've done... | 21:00 |
| lifeless | mordred: :) | 21:00 |
| mordred | and I've just hit approve | 21:01 |
| clarkb | pleia2: we should figure out a sanish way to fix the concat problems | 21:01 |
| *** thedodd has quit IRC | 21:01 | |
| clarkb | pleia2: but I think it may involve goat sacrifices | 21:01 |
| pleia2 | clarkb: BUILD! | 21:01 |
| lifeless | clarkb: thinly sliced I hope | 21:01 |
| mordred | lifeless: ignoring the python3.3 error that I will sort out | 21:01 |
| pleia2 | | 8542df8b-e2e7-401f-94a1-6cfc6e06a9aa | tripleo-precise-tripleo-test-cloud-592051.slave.openstack.org | BUILD | | | 21:01 |
| mordred | lifeless: would you look at: https://review.openstack.org/#/c/52367/ | 21:01 |
| mordred | lifeless: and more importantly tell me what you think of the output it produces | 21:01 |
| clarkb | pleia2: nice | 21:01 |
| mordred | pleia2: woot! | 21:02 |
| lifeless | https://review.openstack.org/#/c/48046/ seems to be ready to land too ? | 21:02 |
| mordred | also, clarkb https://review.openstack.org/#/c/48860/ | 21:02 |
| *** SergeyLukjanov has joined #openstack-infra | 21:02 | |
| pleia2 | now to see if it keeps 592051 & 592054 like it should and doesn't get all loopy | 21:02 |
| lifeless | mordred: where can I see the output from 52367 ? | 21:03 |
| pleia2 | lifeless: have a suggestion for a tripleo review I can run experimental against? | 21:03 |
| *** thedodd has joined #openstack-infra | 21:03 | |
| pleia2 | tripleo-precise-tripleo-test-cloud-592051 | 21:04 |
| pleia2 | 1Idle | 21:04 |
| pleia2 | \o/ | 21:04 |
| *** senk has quit IRC | 21:04 | |
| *** mtreinish has joined #openstack-infra | 21:04 | |
| pleia2 | although I thought I put one on jenkins01 and one on 02 | 21:05 |
| openstackgerrit | Monty Taylor proposed a change to openstack-dev/pbr: Make ChangeLog more condensed https://review.openstack.org/52367 | 21:05 |
| mordred | lifeless: I added link to sample output ^^ | 21:05 |
| pleia2 | hm, I did, not sure why they both live on jenkins01 | 21:06 |
| clarkb | pleia2: could be a bug in the pie slicing algorithm | 21:06 |
| clarkb | pleia2: mordred: upgrading haproxy and postgresql puppet modules will fix the silly concat during install errors | 21:07 |
| lifeless | pleia2: \o/ \o/ | 21:07 |
| clarkb | pleia2: mordred: I will propose that change, the haproxy upgrade looks safe. The postgres one is less safe, but we only use that on slaves so not too insane to deal with if something changes | 21:07 |
| hashar | hey infra! | 21:07 |
| pleia2 | clarkb: sounds good | 21:08 |
| hashar | I got a django app for you, it is a dashboard for puppet facter. It is django based https://github.com/akosiaris/servermon | 21:08 |
| openstackgerrit | Monty Taylor proposed a change to openstack-dev/pbr: Comply with pep440 https://review.openstack.org/46996 | 21:08 |
| *** ruhe has joined #openstack-infra | 21:08 | |
| hashar | that somehow list servers missing packages and report the node facters :-] | 21:08 |
| mordred | lifeless: ^^ so - pep440 actually doesn't have ANY provisions for including the git sha | 21:08 |
| *** ruhe has quit IRC | 21:09 | |
| dstufft | git shas are not orderable | 21:09 |
| dstufft | is probably why | 21:10 |
| mordred | dstufft: ah - the intent is to use source labels in metadata 2.0 isn't it? | 21:10 |
| dstufft | yea | 21:10 |
| mordred | dstufft: sure - I would not expect them to be used in release releases | 21:10 |
| dstufft | mordred: FWIW PEP440 isn't finalized so if you want changes raise an issue in the metadata | 21:10 |
| dstufft | it's on bitbucket.org/pypa/something or other | 21:10 |
| mordred | dstufft: so - pep440 mentions source labels but does not describe where to put them at all | 21:11 |
| mordred | dstufft: am I missing something? | 21:12 |
| mordred | " | 21:12 |
| mordred | "Distributions may also define a source label, which is not used by automated tools. Source labels are useful when a project internal versioning scheme requires translation to create a compliant public version identifier." | 21:12 |
| mordred | where shall I put such a thing? | 21:12 |
| mordred | oh. ok. it's just defined in 426 | 21:12 |
| dstufft | yea PEP426 defines metadata 20 | 21:16 |
| dstufft | 2.0 | 21:16 |
| dstufft | PEP440 just defines version stuff | 21:16 |
| *** matty_dubs is now known as matty_dubs|gone | 21:17 | |
| mordred | nod | 21:18 |
| *** ericw has quit IRC | 21:18 | |
| mordred | hashar: wow. that seems interesting | 21:19 |
| hashar | mordred: I can't give a demo unfortunately :/ | 21:20 |
| hashar | mordred: I send my coworker a pull request to make it pbr/tox based :-] | 21:20 |
| mordred | hashar: nice | 21:21 |
| hashar | so hopefully in a few days you guys would feel at home | 21:21 |
| hashar | kudos on pbr btw | 21:21 |
| mordred | thanks! glad if it's helpful! | 21:21 |
| hashar | it lacks some documentation, but I found it WAYYY easier to use than setuptools/distribute whatever | 21:21 |
| mordred | it needs better docs for sure! | 21:21 |
| *** _TheDodd_ has joined #openstack-infra | 21:21 | |
| *** thedodd has quit IRC | 21:21 | |
| pleia2 | clarkb: https://review.openstack.org/#/c/52706/ (ok, so it failed, but I still call this progress) | 21:21 |
| * mordred wishes annegentle would write docs for all of his stuff | 21:21 | |
| *** thomasm has quit IRC | 21:22 | |
| hashar | mordred: I might write some end user doc eventually. | 21:22 |
| *** dstufft has quit IRC | 21:22 | |
| mordred | hashar: I will give you a bunny if you do | 21:22 |
| mordred | unless you don't want a bunny | 21:22 |
| hashar | my wife is a good cooker | 21:22 |
| pleia2 | clarkb: it threw away the one it failed on and is building a new one, so yay nodepool | 21:22 |
| clarkb | pleia2: awesome | 21:23 |
| *** cody-somerville has joined #openstack-infra | 21:23 | |
| *** cody-somerville has quit IRC | 21:23 | |
| *** cody-somerville has joined #openstack-infra | 21:23 | |
| mordred | pleia2: so, it did the thing we want, which is run test code | 21:25 |
| mordred | pleia2: and then the system responded in the appropriate manner | 21:25 |
| mordred | pleia2: I think that's gREAT | 21:25 |
| *** _TheDodd_ has quit IRC | 21:25 | |
| *** dstufft has joined #openstack-infra | 21:25 | |
| pleia2 | mordred: yes :) | 21:25 |
| lifeless | what was the {{ failure about? | 21:27 |
| mordred | lifeless: you | 21:28 |
| mordred | lifeless: it was angry at you | 21:28 |
| pleia2 | lifeless: heh, looking into it | 21:30 |
| pleia2 | ok, I don't actually know what /tmp/hudson5835396063838205409.sh is supposed to be | 21:33 |
| annegentle | mordred: but my whole goal would be to coach you to write better docs :) | 21:34 |
| mordred | annegentle: oh. then our relationship would get off to a rocky start | 21:34 |
| annegentle | mordred: heh | 21:34 |
| * mordred goes off to hire more minions | 21:34 | |
| openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Upgrade puppetlabs-haproxy to fix concat module. https://review.openstack.org/52708 | 21:35 |
| openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Upgrade puppetlabs-postgresql to fix concat module https://review.openstack.org/52709 | 21:35 |
| annegentle | mordred: or bunny cookers, whatevs | 21:35 |
| clarkb | pleia2: mordred ^ none of that is tested, but reading the changelogs for the two modules I think that *should* work :) | 21:35 |
| hashar | off, have a nice weekend | 21:35 |
| clarkb | pleia2: mordred: I figure I can work on getting that stuff in place after logstash upgrading happens | 21:35 |
| clarkb | hashar: you too | 21:35 |
| clarkb | pleia2: I think that tmp file is your script that is executing in the shell builder | 21:36 |
| hashar | yup it is | 21:36 |
| hashar | crafted by Jenkins | 21:36 |
| clarkb | pleia2: the {{ stuff is probably related to how jjb escapes variables | 21:36 |
| hashar | the shell is in Jenkins Job Builder I guess | 21:36 |
| clarkb | hashar: yup | 21:37 |
| *** CaptTofu has joined #openstack-infra | 21:37 | |
| hashar | bah I should be sleeping really :] Have fun! | 21:37 |
| pleia2 | clarkb: so my script is: https://git.openstack.org/cgit/openstack-infra/tripleo-ci/tree/toci_gate_test.sh | 21:38 |
| pleia2 | not much to get confused about | 21:38 |
| clarkb | pleia2: I don't think that is the script it is confused about, it is complaining about something in the script in JJB | 21:39 |
| pleia2 | clarkb: oh, hm | 21:39 |
| clarkb | https://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/jenkins_job_builder/config/tripleo.yaml#n13 that script I think | 21:40 |
| *** dcramer_ has quit IRC | 21:40 | |
| clarkb | yup, that job is a normal job, not a job template so the double {{ }}s are not needed | 21:40 |
| pleia2 | ah! | 21:40 |
| clarkb | pleia2: the function should use single {}s there | 21:40 |
| pleia2 | it was a template, but then I converted it based on feedback | 21:41 |
| pleia2 | so didn't convert all the way I guess :) | 21:41 |
| *** hashar has quit IRC | 21:41 | |
| clarkb | pleia2: I can push a fix to that through | 21:41 |
| clarkb | pleia2: if you want to propose it | 21:41 |
| pleia2 | yep, on it | 21:41 |
| openstackgerrit | Monty Taylor proposed a change to openstack-dev/pbr: Make ChangeLog more condensed https://review.openstack.org/52367 | 21:42 |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Enable Ceilometer alarm services https://review.openstack.org/48046 | 21:43 |
| openstackgerrit | Elizabeth Krumbach Joseph proposed a change to openstack-infra/config: Fix bracketing error in tripleo JJB job https://review.openstack.org/52710 | 21:43 |
| clarkb | mordred: pip freeze says jeepyb on review.o.o is jeepyb==766973d the git sha in /opt/jeepyb is fungi's change with sha d211fa570da0761346292d7a93a81b25cba8d3f4 how do I check if they match up? | 21:43 |
| *** ryanpetrello has quit IRC | 21:44 | |
| openstackgerrit | A change was merged to openstack-infra/config: Fix bracketing error in tripleo JJB job https://review.openstack.org/52710 | 21:45 |
| *** senk has joined #openstack-infra | 21:45 | |
| clarkb | darn just missed the puppet git update on that | 21:45 |
| clarkb | Ryan_Lane: hey, any update on the wiki stuff? | 21:46 |
| openstackgerrit | Monty Taylor proposed a change to openstack-dev/pbr: Use wheels for installation https://review.openstack.org/48803 | 21:46 |
| Ryan_Lane | clarkb: nope, but we can fix that now :) | 21:46 |
| Ryan_Lane | were we going to do the sledgehammer approach? | 21:46 |
| clarkb | Ryan_Lane: I think so, it avoids weird namespace issues in the future | 21:46 |
| clarkb | openstack_project::elasticsearch should be openstack_project::elasticsearch_node or similar | 21:47 |
| Ryan_Lane | ok | 21:47 |
| clarkb | then replace as necessary | 21:47 |
| *** sarob has quit IRC | 21:47 | |
| *** sarob has joined #openstack-infra | 21:48 | |
| openstackgerrit | Ryan Lane proposed a change to openstack-infra/config: Rename openstack_project::elasticsearch https://review.openstack.org/52711 | 21:49 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Use pip install -U instead of setup.py install https://review.openstack.org/51004 | 21:51 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Remove explicit depends on transitive pip deps https://review.openstack.org/51402 | 21:51 |
| *** sarob has quit IRC | 21:52 | |
| mordred | clarkb: I don't understand the question? | 21:52 |
| clarkb | Ryan_Lane: -1'd to fix the lint problem | 21:53 |
| Ryan_Lane | heh | 21:53 |
| clarkb | mordred: how do I tell if the sha that is checked out in the git repo is what was installed | 21:53 |
| *** ryanpetrello has joined #openstack-infra | 21:53 | |
| mordred | clarkb: also, did you and I ever come to terms on https://review.openstack.org/#/c/51403/ ? | 21:53 |
| mordred | clarkb: pip freeze | 21:53 |
| mordred | clarkb: I'm pretty sure that it's not | 21:53 |
| Ryan_Lane | ah. right | 21:53 |
| mordred | you could do a diff on files though | 21:53 |
| clarkb | mordred: pip freeze doesn't show a sha though it shows some other value I thought | 21:54 |
| clarkb | mordred: or is it the sha and that means we didn't update properly? | 21:54 |
| mordred | clarkb: short sha if the thing doesn't have any tags | 21:54 |
| clarkb | mordred: ok then it must not have upgraded yet | 21:55 |
| clarkb | for whatever reason | 21:55 |
| *** ryanpetrello has quit IRC | 21:55 | |
| mordred | clarkb: mordred@review:/opt/jeepyb$ diff -Nru jeepyb/cmd/manage_projects.py /usr/local/lib/python2.7/dist-packages/jeepyb/cmd/manage_projects.py | 21:55 |
| mordred | match | 21:55 |
| clarkb | mordred: so the version is wrong? | 21:55 |
| mordred | clarkb: yes. version is wrong. Oh - I think that's potentially expected | 21:55 |
| mordred | I need to add something for forced cache invalidatio | 21:56 |
| clarkb | mordred: I think it isn't a sha that we get, instead we get a sha of something else | 21:56 |
| mordred | commit 766973dc8ff5886a432dd181c3530cd2fb2b3501 | 21:56 |
| mordred | Date: Mon Sep 9 22:59:53 2013 +0000 | 21:56 |
| mordred | it's a sha | 21:56 |
| mordred | of a jeepyb commit | 21:56 |
| mordred | it's just stale | 21:56 |
| pleia2 | clarkb: notated your rocket ship on 52709 | 21:56 |
| clarkb | mordred: does that mean the entry point for manage projects is wrong too? | 21:57 |
| clarkb | pleia2: thanks | 21:57 |
| mordred | hrm | 21:57 |
| mordred | mordred@review:/opt/jeepyb$ grep Version jeepyb.egg-info/PKG-INFO | 21:57 |
| mordred | Version: d211fa5 | 21:57 |
| mordred | which is what we expect | 21:57 |
| clarkb | but will we run with a stale entrypoint? | 21:57 |
| clarkb | there are a ton of egg infos in dist-packages for jeepyb any one of them could be used right? | 21:58 |
| mordred | it shouldn't work that way | 21:58 |
| clarkb | (I am being extra paranoid because we have seen all kinds of package weirdness) | 21:58 |
| mordred | one sec | 21:58 |
| mordred | yeah - this is one of the reasons I want to land https://review.openstack.org/51004 | 21:58 |
| mordred | btw | 21:58 |
| dstufft | oh yea | 21:59 |
| dstufft | entry points | 21:59 |
| mordred | ./jeepyb-0.0.122.f98a3a4-py2.7.egg | 21:59 |
| *** w_ is now known as olaph | 21:59 | |
| *** senk has quit IRC | 21:59 | |
| dstufft | mordred: clarkb btw pip 1.5 will probably start generating it's own entry point wrappers | 21:59 |
| mordred | that's in easy-install.pth | 21:59 |
| dstufft | without pkg_resources | 21:59 |
| mordred | dstufft: we already do that :) | 21:59 |
| dstufft | yea | 21:59 |
| mordred | dstufft: so I support your endeavor | 21:59 |
| dstufft | but just saying, pip will start doing it so you don't have to :D | 21:59 |
| clarkb | mordred: re https://review.openstack.org/#/c/51403/1 I am conflicted. on one hand we can end up with different slaves, on the other hand maybe that isn't such a bad thing | 21:59 |
| mordred | clarkb: I vote we let tox run it for now, and we keep an eye on it | 22:00 |
| mordred | clarkb: virtualenv versions really shouldn't be screwing us anymore | 22:00 |
| mordred | clarkb: >>> import jeepyb.cmd.manage_projects | 22:00 |
| mordred | >>> jeepyb.cmd.manage_projects.__file__ | 22:00 |
| mordred | '/usr/local/lib/python2.7/dist-packages/jeepyb/cmd/manage_projects.pyc' | 22:01 |
| mordred | clarkb: we are appropriately skipping all of those crappy eggs | 22:01 |
| clarkb | mordred: mordred ok +2'd | 22:01 |
| mordred | clarkb: in fact, we could delete them | 22:01 |
| clarkb | mordred: ++ to deleting crappy eggs | 22:01 |
| mordred | and probably should | 22:01 |
| mordred | those are from before we had pbr stop making eggs | 22:01 |
| mordred | oh - btw - dstufft | 22:01 |
| mordred | I believe I _AM_ responding to --single-version-externally-managed mostly appropriately | 22:01 |
| mordred | as in, I am responding to it | 22:02 |
| dstufft | mordred: awesome | 22:02 |
| mordred | but I really want to go through and do a full audit thing at some point | 22:02 |
| *** cody-somerville has quit IRC | 22:02 | |
| mordred | but as best I can tell, if you pas --single-version-externally-managed, it prevents attempts at installing stuff from pip | 22:02 |
| mordred | so I may have had a day of sanity at some point | 22:03 |
| dstufft | I had a few days of sanity | 22:03 |
| dstufft | then I started learning debian packaging | 22:03 |
| mordred | yup | 22:04 |
| mordred | clarkb: DAMMIT | 22:04 |
| clarkb | mordred: what did I do? | 22:04 |
| mordred | clarkb: nope. not you. it's python agin | 22:05 |
| clarkb | mordred: did you delete the wrong egg? | 22:05 |
| clarkb | oh | 22:05 |
| mordred | clarkb: no. that's fine. they'reall reinstallable - but I didn't do that | 22:05 |
| pleia2 | dstufft: haha | 22:05 |
| *** julim has quit IRC | 22:05 | |
| mordred | I _did_ take a stab at pip install -U . in the jeepyb dir - to test the pip install -U replacement for python setup.py install we'd been doing | 22:05 |
| mordred | guess what breaks? | 22:05 |
| mordred | python-mysqldb - we currently use it from pacakges, not from pip | 22:05 |
| mordred | but pip install -U . wants to upgrade it :) | 22:06 |
| clarkb | hahahahaha | 22:06 |
| dstufft | mixing packages a sure fire way for pain | 22:06 |
| mordred | yup | 22:06 |
| mordred | I was just about to say "maybe one day we'll learn to listen to dstufft" | 22:06 |
| clarkb | dstufft: mordred: you guys keep saying that and I really feel like there is no reason for that to be the case | 22:06 |
| BobBall | mordred: what happened to the always-run-in-venv approach? | 22:06 |
| * clarkb looks at maven | 22:06 | |
| mordred | BobBall: we haven't gotten there yet | 22:06 |
| mordred | clarkb: lemme rephrase | 22:06 |
| clarkb | dstufft: mordred: I agree that it is the case, but it shouldn't be | 22:06 |
| BobBall | the patch was very close... just the upgrade didn't work IIRC | 22:06 |
| mordred | clarkb: "as currently implemented by python packaging, mixing distro packages and python pacakging is a bad idea" | 22:07 |
| mordred | BobBall: oh - this isn't even devstack | 22:07 |
| BobBall | oh, sorry | 22:07 |
| dstufft | clarkb: I want them to get along better, but it's not a high priority item for me | 22:07 |
| mordred | BobBall: this is just a single static install on the gerrit server | 22:07 |
| * BobBall was jumping in the middle :) | 22:07 | |
| mordred | BobBall: no, you're right- I should go back to that patch | 22:07 |
| BobBall | I'm interested in the devstack patch re-emerging :) | 22:07 |
| BobBall | well I can help I hope | 22:07 |
| BobBall | just trying to get my other devstack commits to a point where I can show that patch is needed so I can work on it | 22:07 |
| openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Upgrade puppetlabs-postgresql to fix concat module https://review.openstack.org/52709 | 22:08 |
| clarkb | pleia2: ^ rocket ship aligned :) | 22:08 |
| mordred | clarkb: biggest problem right now is that there is no real good way for us to install something CD from our source, and have the source depends in that cause upgrades to happen in dependencies | 22:08 |
| mordred | clarkb: we can install once, and that's fine | 22:08 |
| clarkb | pleia2: you should be able to give the tripleo deploy job another shot, here in a minute (JJB should have applied the fix already) | 22:08 |
| mordred | the semantics of pip install . will get us what we expect on that day | 22:08 |
| dstufft | -U being recursive by default doesn't help | 22:09 |
| mordred | but, 2 months later, if we do another pip install . - it will not attempt to update any dependent libraries for hwich there are a new upstream but which still match our min-bounds | 22:09 |
| mordred | but -U tries to do too much | 22:09 |
| mordred | and will try toupdate things that were installed by distro packages | 22:09 |
| pleia2 | clarkb: started one 2 minutes ago, it's chugging along | 22:09 |
| clarkb | pleia2: woot | 22:09 |
| mordred | thus- there is actually no cross-understanding of metainfo or where something same from | 22:09 |
| mordred | so pip can't say "please upgrade everything _you_ installed, but please dont' upgrade anything you didn't install" | 22:09 |
| mordred | (we can't say to pip) | 22:10 |
| mordred | which means that the ways in which we can get systems that stay up to date | 22:10 |
| * dstufft wanders off to get some food | 22:10 | |
| clarkb | pip should by default not touch anything in the distro part of python path and vice versa | 22:10 |
| mordred | right. but that isn't the case | 22:10 |
| clarkb | if it did that none of this would be a problem. but apparently python people hate that idea | 22:10 |
| mordred | and the two paths are not even separate on redhat | 22:10 |
| dstufft | I don't hate that idea | 22:11 |
| dstufft | but "distro part of python path" isn't an actual construct | 22:11 |
| dstufft | it's just another path | 22:11 |
| clarkb | dstufft: but it should be | 22:11 |
| dstufft | sure it sohuld be | 22:11 |
| dstufft | maybe someday i'll get time to write that PEP :) | 22:11 |
| dstufft | there's a lot of should be's in packaging | 22:11 |
| dstufft | if you care a lot about this particular issue i'm happy to review a PEP ;) | 22:11 |
| clarkb | this seems like such a simple one though :) | 22:11 |
| mordred | - are to duplicate the requirements lists in puppet and to manage the upgrading of each pip installed package that way (disadvantage, we have to duplicate work) | 22:12 |
| mordred | - are to distro package everything | 22:12 |
| pleia2 | clarkb: what's nodepool's ~ ? | 22:12 |
| mordred | - are to do virtualenv-per-pip-installed-thing-we-care-about, and thus install all pip things from source | 22:12 |
| clarkb | pleia2: on the test slave? | 22:12 |
| clarkb | pleia2: or on nodepool.o.o? | 22:12 |
| mordred | - are to do a combo of 3 and 4 and use dh-virtualenv | 22:12 |
| clarkb | mordred: dstufft: is it possible to change the python path to remove the distro parts and have pip install globally but in a virtual virtualenv? | 22:13 |
| dstufft | you'd need to use uh | 22:13 |
| dstufft | --root or something | 22:13 |
| clarkb | (because there are clear advantages to not needing a venv per thing) | 22:13 |
| dstufft | or --target | 22:13 |
| *** krtaylor has joined #openstack-infra | 22:13 | |
| *** amotoki__ has quit IRC | 22:14 | |
| mordred | clarkb: honestly, I think we've got to stop fighting the venv thing - I think it exists in most python-based deployments for a reason | 22:14 |
| dstufft | pip doesn't consult the python path when asking *where* to install things too, it asks distutils for that | 22:14 |
| mordred | and that reason I think are the above things | 22:14 |
| *** amotoki_ has joined #openstack-infra | 22:14 | |
| dstufft | but it does consult it for looking for already installed distrubitons | 22:14 |
| dstufft | distributions | 22:14 |
| clarkb | mordred: right because things are broken :) | 22:14 |
| mordred | clarkb: yes. but they are broken enough that we're not going to fix them this year | 22:14 |
| clarkb | mordred: I am not a fan of fixing broken with more broken. virtualenvs are valuable but I shouldn't need 20 different versions of pyzmq compiled in a bunch of venvs | 22:15 |
| mordred | also, what you described above is the implementation of virtualenv | 22:15 |
| clarkb | I should be able to build things like that once and fix them for security once | 22:15 |
| clarkb | mordred: right, but it is global, things will just use it | 22:15 |
| clarkb | because /usr/local/bin sits in your path | 22:15 |
| dstufft | just use a global virtualenv | 22:15 |
| mordred | clarkb: "build things like that once and fix them for security once" == distro packages | 22:15 |
| dstufft | a single virtualenv | 22:15 |
| pleia2 | clarkb: test slave | 22:15 |
| dstufft | and then you don't have that problem :) | 22:15 |
| clarkb | pleia2: I think it is /home/jenkins | 22:16 |
| mordred | dstufft: I believe it's the mechanics of any virtualenvs that clarkb is pushing back against | 22:16 |
| clarkb | then the job runs in /home/jenkins/workspace/$JOB_NAME | 22:16 |
| clarkb | mordred: exactly | 22:16 |
| clarkb | virtualenvs are a hack. They work great, but they skirt around broken tools | 22:16 |
| mordred | clarkb: they are a hack - but we have so far not shown any actual interest in 'fixing' this the right way | 22:16 |
| pleia2 | clarkb: ok, it's running it from /opt which failed, so I need to do another JJB fixup | 22:16 |
| mordred | which involves apt repos | 22:16 |
| mordred | and yum repos | 22:17 |
| clarkb | pleia2: oh actually | 22:17 |
| clarkb | pleia2: /opt is wheer d-g puts things | 22:17 |
| mordred | and a micro-distro worth of backported python packages | 22:17 |
| clarkb | pleia2: but the homdir is different :) | 22:17 |
| clarkb | pleia2: the /opt stuff should be fine, and we have disk space there on the hpcloud instances so we probably don't want to move it | 22:17 |
| pleia2 | clarkb: yeah, I unpacked git stuff to homedir, that's where the test script is | 22:17 |
| clarkb | OH | 22:17 |
| *** jerryz has joined #openstack-infra | 22:17 | |
| pleia2 | test script lives in tripleo-ci repo | 22:17 |
| clarkb | mordred: I don't think you have to resort to distro packages to make this work in not insane ways | 22:18 |
| clarkb | mordred: the whole reason for /usr/local's existence for example | 22:18 |
| *** vipul is now known as vipul-away | 22:18 | |
| *** vipul-away is now known as vipul | 22:18 | |
| *** vipul is now known as vipul-away | 22:18 | |
| *** vipul-away is now known as vipul | 22:18 | |
| mordred | clarkb: it doesn't work with python packaging though | 22:18 |
| *** vipul is now known as vipul-away | 22:18 | |
| dstufft | virtualenv isn't just isolation from the system, but also from each other | 22:18 |
| mordred | clarkb: what I'm saying is that python packaging is broken enough that you DO have to use either a venv or distro packages to get what you're talking about | 22:18 |
| mordred | it might should not be - but it is | 22:19 |
| clarkb | mordred: I agree | 22:19 |
| dstufft | Also any fix that requires changes in Python itself has like a 99% chance of only being 3.5+ | 22:19 |
| clarkb | dstufft: virtualenvs are like he Go statically link everything problem | 22:19 |
| clarkb | why manage in 20 buckets what you can in one | 22:19 |
| dstufft | because my 20 buckets aren't compatible with each other :) | 22:20 |
| openstackgerrit | Ben Nemec proposed a change to openstack-dev/hacking: Enforce import grouping https://review.openstack.org/52221 | 22:20 |
| *** dmsimard has quit IRC | 22:20 | |
| openstackgerrit | Elizabeth Krumbach Joseph proposed a change to openstack-infra/config: Fix location of toci script in JJB https://review.openstack.org/52714 | 22:21 |
| clarkb | dstufft: mordred: I agree that today virtualenvs or distro packages solve the problem. But I also think that when people discuss these problems we overlook the obvious that we could just fix the problem | 22:21 |
| pleia2 | clarkb: ^^ that still might not be correct | 22:21 |
| mordred | clarkb: I agree that we shoudl fix the problem | 22:21 |
| mordred | I just think that it will take probably 6 years to fix the problem | 22:22 |
| pleia2 | but at least I got the right repo this time ;) | 22:22 |
| dstufft | clarkb: I don't think anyone thinks we shouldn't fix it | 22:22 |
| mordred | because we need to fix it in upstream python, and then get the fix into the distros | 22:22 |
| pleia2 | hmm, actually | 22:22 |
| mordred | and have the fix in all of the versions we want to use | 22:22 |
| clarkb | why does python itself needfixing? | 22:22 |
| clarkb | this seems to be just PATH related | 22:22 |
| mordred | because distutils in the stdlib is what controls it | 22:22 |
| dstufft | if you want a seperate concept of distro PYTHONPATH and not distro PYTHONPATH then distutils is where it needs fixed | 22:23 |
| dstufft | it needs a dist-purelib and a dist-platlib | 22:23 |
| dstufft | in the install schemes | 22:23 |
| clarkb | so distutils doesn't honor PYTHONPATH? | 22:23 |
| dstufft | and probably a dist-bin as well | 22:23 |
| dstufft | well | 22:23 |
| dstufft | you're talking about two different things | 22:24 |
| dstufft | PYTHONPATH is just where things get imported from | 22:24 |
| clarkb | pleia2: tripleo-incubator is the name of the project running the test no tripleo-ci | 22:24 |
| dstufft | it's not where things get installed too | 22:24 |
| dstufft | to | 22:24 |
| clarkb | pleia2: do we need to move the job in zuul/layout.yaml as well? | 22:24 |
| dstufft | it so happens that unless those two concepts over lap the installed package isn't very useful | 22:24 |
| dstufft | but they are seperate concepts in Python | 22:24 |
| clarkb | dstufft: right, I would assume that pip/distutils/whatever would not install to a place it can't import from | 22:24 |
| pleia2 | clarkb: sorry, I meant tripleo-ci is where the script we're running lives, so my path to it was wrong | 22:24 |
| dstufft | it's basically a compile time option where python packages should install too | 22:25 |
| dstufft | of python | 22:25 |
| clarkb | pleia2: right, but to get tripleo-ci cloned we need to run the job under tripleo-ci | 22:25 |
| dstufft | there's no introspection of PYTHONPATH | 22:25 |
| dstufft | I'm not sure how that'd work, there's multiple entries, how do you pick which one? | 22:25 |
| clarkb | dstufft: I think you still need separate sets that you take an intersection of | 22:26 |
| clarkb | because they are different concepts, but I wouldn't expect a tool for language lib packaging to install them someplace that language can't import from (at least not by default) | 22:26 |
| clarkb | pleia2: or does the image snapshot put the script in place for you? | 22:26 |
| *** thomasm has joined #openstack-infra | 22:26 | |
| dstufft | it doesn't by default, but it doesn't inspect the current sys.path to decide where to install them | 22:27 |
| dstufft | that's essentially a compile time option | 22:27 |
| *** thomasm has quit IRC | 22:27 | |
| *** mgagne has quit IRC | 22:27 | |
| dstufft | so if you modify sys.path, that has no bearing on where things get installed too | 22:27 |
| *** thomasm has joined #openstack-infra | 22:27 | |
| dstufft | even if you move site-packages out of sys.path | 22:27 |
| openstackgerrit | Ben Nemec proposed a change to openstack/requirements: Pin iso8601 to 0.1.4 https://review.openstack.org/52671 | 22:28 |
| *** mrodden has quit IRC | 22:28 | |
| clarkb | pleia2: looking at the prepare script I think you want ~jenkins/tripleo/tripleo-ci | 22:28 |
| openstackgerrit | Elizabeth Krumbach Joseph proposed a change to openstack-infra/config: Fix location of toci script in JJB and clone toci https://review.openstack.org/52714 | 22:28 |
| dstufft | clarkb: I'm going to be trying to get distros to drop an extra piece of metadata in their egg-info directories | 22:28 |
| dstufft | an INSTALLER file that says "os" or something | 22:29 |
| dstufft | so pip knows who installed a package | 22:29 |
| pleia2 | clarkb: tripleo-ci is what tests tripleo-incubator, so I think what I need to do is clone it during image prep | 22:29 |
| dstufft | I just haven't had the time | 22:29 |
| clarkb | pleia2: right, I think that happens in prepare_tripleo.sh | 22:29 |
| clarkb | pleia2: the pull-tools run | 22:29 |
| clarkb | dstufft: interesting | 22:29 |
| dstufft | that won't help entirely on systems like fedora | 22:30 |
| dstufft | because a system pacakge will completely block out a pip package then | 22:30 |
| dstufft | since we won't have a place to install it too | 22:30 |
| dstufft | but that may be desireable anyways | 22:30 |
| clarkb | because there is no difference between the two path prefixes | 22:30 |
| dstufft | I haven't fully sorted it out | 22:30 |
| mordred | dstufft: wow. it would be stellar if dh-virtualenv was even partially usable from its git repo | 22:30 |
| pleia2 | clarkb: it does now, pull-tools doesn't include toci | 22:30 |
| dstufft | mordred: I had to patch it | 22:30 |
| dstufft | to make it work | 22:31 |
| mordred | dstufft: I'm still working through "install it" | 22:31 |
| mordred | since that does not work | 22:31 |
| pleia2 | today, the role of jeblair, fungi and clarkb will be played by clarkb (sorry clarkb :)) | 22:31 |
| * mordred will make it work by the end of the night | 22:31 | |
| clarkb | pleia2: so you need to clone it in prepare_tripleo.sh? I would put it in ~jenkins/tripleo/tripleo-ci then | 22:31 |
| mordred | clarkb: BE MORE PEOPLE | 22:31 |
| clarkb | pleia2: since ~/tripleo seems to already be established by the existing prepare script | 22:32 |
| pleia2 | clarkb: wfm | 22:32 |
| * dstufft really goes this time | 22:32 | |
| *** pentameter has quit IRC | 22:32 | |
| mordred | sigh. they're expecting me to use the github tarballs | 22:32 |
| clarkb | mordred: "make it work" is jeepyb for manage-projects right? | 22:32 |
| mordred | clarkb: what? | 22:32 |
| clarkb | mordred: or is that on a back burner? | 22:32 |
| mordred | no. it's fine | 22:32 |
| clarkb | mordred: the whole thing that started this was fixing jeepyb | 22:32 |
| clarkb | oh | 22:32 |
| mordred | jeepyb is fine | 22:32 |
| openstackgerrit | Elizabeth Krumbach Joseph proposed a change to openstack-infra/config: Fix location of toci script in JJB and clone toci https://review.openstack.org/52714 | 22:33 |
| mordred | sorry, I should have reported that | 22:33 |
| clarkb | mordred: you should review https://review.openstack.org/#/c/52215/ then | 22:33 |
| mordred | clarkb: there does not seem to be an upstream location | 22:34 |
| clarkb | mordred: right, I think they are starting from scratch | 22:34 |
| mordred | clarkb: who in channel wanted that change? | 22:34 |
| mordred | oh. neat | 22:34 |
| clarkb | mordred: mgagne and bodepd_ | 22:34 |
| clarkb | bodepd_: can you confirm the puppet-ceph is starting from a blank slate? | 22:34 |
| mordred | I'll just plop it in and watch the carnage | 22:34 |
| lifeless | oh ceph ? | 22:35 |
| clarkb | I have no idea why https://review.openstack.org/#/c/52689/ is so unhappy in its unittests | 22:35 |
| clarkb | lifeless: puppet for ceph | 22:35 |
| openstackgerrit | A change was merged to openstack-infra/config: Add puppet-ceph project to StackForge https://review.openstack.org/52215 | 22:36 |
| *** julim has joined #openstack-infra | 22:37 | |
| *** prad has quit IRC | 22:38 | |
| *** eharney has quit IRC | 22:39 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 22:41 | |
| *** _SergeyLukjanov is now known as SergeyLukjanov | 22:41 | |
| *** mrodden has joined #openstack-infra | 22:42 | |
| *** thomasm has quit IRC | 22:43 | |
| *** oubiwann has quit IRC | 22:45 | |
| *** openstackgerrit has quit IRC | 22:46 | |
| *** openstackgerrit has joined #openstack-infra | 22:46 | |
| *** dcramer_ has joined #openstack-infra | 22:46 | |
| *** adalbas has quit IRC | 22:50 | |
| *** vipul-away is now known as vipul | 22:52 | |
| bodepd_ | clarkb: confirmed | 22:54 |
| bodepd_ | thanks mordred clarkb , there are tons of people that are ready to get this thing designed and coded ;) | 22:55 |
| bodepd_ | there's too many forks out there to find a good place to start. trying to start over on a clean blueprint, and see if we can get community consensus from the ground up | 22:56 |
| clarkb | bodepd_: cool, the project should be up now I htink | 22:56 |
| bodepd_ | great! thanks for taking time out of the middle of what I can only guess is havana madness | 22:57 |
| *** sarob has joined #openstack-infra | 22:58 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 22:59 | |
| *** _SergeyLukjanov has quit IRC | 22:59 | |
| *** rnirmal has quit IRC | 23:00 | |
| *** SergeyLukjanov has joined #openstack-infra | 23:01 | |
| *** sarob has quit IRC | 23:01 | |
| *** datsun180b has quit IRC | 23:01 | |
| *** sarob has joined #openstack-infra | 23:02 | |
| *** fifieldt has joined #openstack-infra | 23:02 | |
| *** boris-42 has quit IRC | 23:02 | |
| clarkb | bodepd_: post havana madness! | 23:02 |
| *** boris-42 has joined #openstack-infra | 23:02 | |
| sdague | clarkb: can I bug you for an easy review from earlier in the week? :) - https://review.openstack.org/#/c/51754/ | 23:03 |
| clarkb | sdague: how does that work with not setting <pre> for console.html? | 23:05 |
| sdague | console.html already has <pre> in it | 23:06 |
| clarkb | sdague: right, but aren't we setting <pre> again in console.html on line 82? | 23:07 |
| clarkb | or is the logic to not set the html stuff elsewhere? | 23:07 |
| clarkb | (this is late friday brain talking) | 23:07 |
| sdague | sorry, I wrote this on monday, so getting context again | 23:08 |
| *** davidhadas has joined #openstack-infra | 23:08 | |
| sdague | actually we just put in <pre> twice | 23:08 |
| sdague | which we are doing now | 23:08 |
| sdague | this doesn't fix this | 23:08 |
| clarkb | wasn't that causing problems? | 23:08 |
| sdague | it just gets rid of the level header | 23:08 |
| clarkb | we fixed someething related to <pre? | 23:08 |
| sdague | no, the problem was that we were escaping the html | 23:08 |
| clarkb | ah | 23:09 |
| sdague | https://github.com/openstack-infra/os-loganalyze/commit/aa76b7c7f22888f50fe159f320ea6d45827b1b48 | 23:09 |
| sdague | that was merged earlier in the week | 23:09 |
| clarkb | sdague: approved, thanks for helping me through that. I am definitely feeling like the weekend is almost here | 23:11 |
| sdague | yeh, I'm already drinking | 23:11 |
| sdague | though, I'm still coding | 23:11 |
| openstackgerrit | A change was merged to openstack-infra/os-loganalyze: only insert display level header if the file supports it https://review.openstack.org/51754 | 23:11 |
| sdague | so we'll see how that goes | 23:11 |
| sdague | ok, dinner sorting time, have a good one | 23:12 |
| openstackgerrit | Clark Boylan proposed a change to openstack-infra/zuul: Clone repo if it is not on disk https://review.openstack.org/52689 | 23:13 |
| clarkb | sdague: you too | 23:13 |
| bodepd_ | stupid question guys, but do I need to do anything stupid to push up to the blank repo? | 23:16 |
| clarkb | bodepd_: you shouldn't, just clone it, git review -s, make changes, git review, done | 23:16 |
| clarkb | bodepd_: we inject a .gitreview file into blank projects so that should work | 23:17 |
| mordred | clarkb: I really need to get the cookiecutter integration done | 23:18 |
| bodepd_ | should it be mirrored to github? Or only after I push something to it? | 23:18 |
| clarkb | bodepd_: it should be mirrored to github assuming manage-projects isn't still broken | 23:18 |
| mordred | clarkb: no commits in repo | 23:18 |
| clarkb | mordred: the .gitreview commit | 23:18 |
| clarkb | mordred: https://review.openstack.org/#/admin/projects/stackforge/puppet-ceph,access something didn't work right | 23:19 |
| clarkb | mordred: is it wrong to want to revert your manage-project changes? >_> | 23:19 |
| clarkb | mordred: it could be that the group creation stuff is still derpy | 23:20 |
| clarkb | but I don't see the group at all in gerrit so I don't think that is the problem | 23:21 |
| openstackgerrit | Ryan Lane proposed a change to openstack-infra/config: Rename openstack_project::elasticsearch https://review.openstack.org/52711 | 23:22 |
| Ryan_Lane | clarkb: sorry, got caught up on other things | 23:22 |
| clarkb | mordred: I am really wondering if trying to keep all that state is worth the trouble | 23:23 |
| clarkb | Ryan_Lane: looking | 23:23 |
| bodepd_ | clarkb: it gets cloned as a repo with no commits | 23:23 |
| bodepd_ | mordred: clarkb oh, you already figured this out :) | 23:24 |
| bodepd_ | mordred: clarkb I had been figuring it out for the last couple of minutes | 23:24 |
| clarkb | bodepd_: so mordred updated the script to make this all work recently and it all broke :) | 23:24 |
| clarkb | bodepd_: we thought we had a fix, but clearly something else isn't working | 23:24 |
| bodepd_ | I hate when that happens | 23:24 |
| bodepd_ | mordred: I'm glad I could help test your new changes :) | 23:25 |
| *** weshay has quit IRC | 23:26 | |
| *** gyee has quit IRC | 23:27 | |
| clarkb | mordred: the repo was created in the cache dir, but the remote and .gitreview were not created | 23:27 |
| mordred | clarkb: hrm | 23:27 |
| clarkb | mordred: any idea why that might happen? | 23:27 |
| mordred | clarkb: no - lemme look | 23:28 |
| mordred | bodepd_: it may take a couple of minutes | 23:28 |
| clarkb | mordred: %(remote_url) does that need to be %(remote_url)s? | 23:28 |
| mordred | clarkb: to be fair, actually, I believe there was a bug in raw repo creation before - we just used it so infrequently that we've normally fixed it by hand | 23:28 |
| mordred | clarkb: yes | 23:28 |
| clarkb | mordred: do you want to fix it and I can +2 +A? | 23:29 |
| clarkb | that will through a ValueError exception short circuiting the other processing | 23:30 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/jeepyb: Add s to the end of format string https://review.openstack.org/52721 | 23:32 |
| mordred | clarkb: ^^ thanks | 23:32 |
| clarkb | mordred: I am not sure how we want ot clean this up though. Do we delete the cached version and let it recache what is in gerrit, then propose a change for .gitreview? | 23:32 |
| clarkb | I don't think we can delete the project then recreate it sanely | 23:32 |
| mordred | clarkb: yes. delete /var/lib/jeepyb/stackforge/puppet-ceph | 23:33 |
| mordred | and then we can push in the inital change by hand (I can do that) | 23:33 |
| clarkb | mordred: do you want to clean it up and rerun manage-projects? | 23:33 |
| bodepd_ | as long as I seem to have everyone's attention, I do recall one other patch... | 23:33 |
| * bodepd_ whistles | 23:33 | |
| openstackgerrit | A change was merged to openstack-infra/jeepyb: Add s to the end of format string https://review.openstack.org/52721 | 23:33 |
| clarkb | (rerun to push the ACLs and create github org) | 23:33 |
| *** vipul is now known as vipul-away | 23:33 | |
| mordred | clarkb: yup | 23:35 |
| clarkb | mordred: ok, I will let you have at it then | 23:35 |
| clarkb | I fixed my zuul patch, I wonder if I can bother jeblair for reviews while you guys are in europeland | 23:35 |
| mordred | clarkb: he just got on a plane :) | 23:37 |
| Ryan_Lane | clarkb: https://review.openstack.org/#/c/52711/2 good to go? :) | 23:38 |
| clarkb | Ryan_Lane: yup approved | 23:38 |
| clarkb | should get in by the :45 minute git pull | 23:39 |
| openstackgerrit | A change was merged to openstack-infra/config: Rename openstack_project::elasticsearch https://review.openstack.org/52711 | 23:41 |
| clarkb | I need to rebase on top of that now | 23:41 |
| clarkb | actually rebase not necessary. So I won't bother | 23:42 |
| clarkb | Ryan_Lane: any luck? let me know how it goes | 23:45 |
| openstackgerrit | Joshua Harlow proposed a change to openstack/requirements: Add taskflow (used by cinder + others) to requirements https://review.openstack.org/52725 | 23:51 |
| *** UtahDave has quit IRC | 23:52 | |
| *** reed has joined #openstack-infra | 23:52 | |
| clarkb | Ryan_Lane: I just realized something that may be problematic. By default our elasticsearch's will try and grab 16GB of memory | 23:53 |
| Ryan_Lane | hahaha | 23:53 |
| Ryan_Lane | yeah, that could be an issue | 23:54 |
| clarkb | Ryan_Lane: modules/elasticsearch/files/elasticsearch.default should become a template where we can set the memory use. I will whip that up real quick | 23:54 |
| Ryan_Lane | ok. cool | 23:54 |
| Ryan_Lane | sorry, had run off for a phone call. it's way too busy for a friday | 23:55 |
| *** luisg has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!