| *** rama_y has quit IRC | 00:02 | |
| *** amotoki has joined #openstack-ironic | 00:04 | |
| *** rajinir has quit IRC | 00:05 | |
| *** baoli has quit IRC | 00:06 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Expose node's network_interface field in API https://review.openstack.org/317392 | 00:09 |
|---|---|---|
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Follow-up to 317392 https://review.openstack.org/342477 | 00:09 |
| *** ijw has joined #openstack-ironic | 00:11 | |
| *** Sukhdev has quit IRC | 00:12 | |
| *** jrist has joined #openstack-ironic | 00:22 | |
| *** adu has quit IRC | 00:24 | |
| *** jerrygb has joined #openstack-ironic | 00:25 | |
| *** garthb has quit IRC | 00:26 | |
| *** piet has joined #openstack-ironic | 00:27 | |
| *** jerrygb has quit IRC | 00:30 | |
| openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 00:31 |
| *** yibo has joined #openstack-ironic | 00:31 | |
| *** clenimar_ has quit IRC | 00:38 | |
| *** Sukhdev has joined #openstack-ironic | 00:43 | |
| *** amotoki has quit IRC | 00:46 | |
| *** ijw has quit IRC | 00:47 | |
| *** ijw has joined #openstack-ironic | 00:48 | |
| *** amotoki has joined #openstack-ironic | 00:48 | |
| *** _milan_ has joined #openstack-ironic | 00:48 | |
| *** milan has quit IRC | 00:49 | |
| *** ijw has quit IRC | 00:49 | |
| *** clenimar_ has joined #openstack-ironic | 00:56 | |
| *** phuongnh has joined #openstack-ironic | 00:57 | |
| *** jrist has quit IRC | 01:02 | |
| *** clenimar_ has quit IRC | 01:07 | |
| *** tiendc has joined #openstack-ironic | 01:07 | |
| *** vmud213 has joined #openstack-ironic | 01:15 | |
| *** joprovost has quit IRC | 01:17 | |
| *** ijw has joined #openstack-ironic | 01:17 | |
| *** Sukhdev has quit IRC | 01:18 | |
| *** clenimar_ has joined #openstack-ironic | 01:19 | |
| *** amotoki has quit IRC | 01:26 | |
| *** hoangcx has joined #openstack-ironic | 01:28 | |
| *** ijw has quit IRC | 01:29 | |
| *** amotoki has joined #openstack-ironic | 01:31 | |
| *** amotoki has quit IRC | 01:32 | |
| *** rajinir has joined #openstack-ironic | 01:37 | |
| *** jcoufal has joined #openstack-ironic | 01:40 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/341698 | 01:42 |
| *** rama_y has joined #openstack-ironic | 01:52 | |
| *** yuanying has joined #openstack-ironic | 02:05 | |
| *** tiendc has quit IRC | 02:08 | |
| *** bvandewa_ has joined #openstack-ironic | 02:16 | |
| *** links has joined #openstack-ironic | 02:16 | |
| *** sdake_ has quit IRC | 02:16 | |
| *** bvandewa has quit IRC | 02:19 | |
| *** bvandewa_ has quit IRC | 02:20 | |
| *** rama_y has quit IRC | 02:27 | |
| *** piet has quit IRC | 02:28 | |
| *** ijw has joined #openstack-ironic | 02:30 | |
| *** sdake has joined #openstack-ironic | 02:31 | |
| *** ijw has quit IRC | 02:35 | |
| *** ijw has joined #openstack-ironic | 02:35 | |
| *** amotoki has joined #openstack-ironic | 02:44 | |
| *** amotoki has quit IRC | 02:44 | |
| *** vmud213 has quit IRC | 02:48 | |
| *** caoshufeng_ has joined #openstack-ironic | 02:50 | |
| *** amotoki has joined #openstack-ironic | 02:50 | |
| *** Fdaisuke has joined #openstack-ironic | 02:54 | |
| *** chlong has joined #openstack-ironic | 02:56 | |
| *** sdake_ has joined #openstack-ironic | 02:58 | |
| *** yuanying has quit IRC | 03:01 | |
| *** sdake has quit IRC | 03:02 | |
| *** sdake_ has quit IRC | 03:05 | |
| *** sdake has joined #openstack-ironic | 03:08 | |
| *** clenimar_ has quit IRC | 03:13 | |
| *** piet has joined #openstack-ironic | 03:14 | |
| *** rloo has quit IRC | 03:19 | |
| *** jaybeale has joined #openstack-ironic | 03:20 | |
| *** amotoki_ has joined #openstack-ironic | 03:25 | |
| *** jaybeale has quit IRC | 03:26 | |
| *** amotoki has quit IRC | 03:29 | |
| *** jaybeale has joined #openstack-ironic | 03:30 | |
| *** ijw has quit IRC | 03:31 | |
| *** jaybeale has quit IRC | 03:31 | |
| *** amotoki_ has quit IRC | 03:31 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Follow-up patch of 0fcf2e8b51e7dbbcde6d4480b8a7b9c807651546 https://review.openstack.org/342545 | 03:32 |
| *** jerrygb has joined #openstack-ironic | 03:32 | |
| *** ijw has joined #openstack-ironic | 03:33 | |
| *** links has quit IRC | 03:33 | |
| *** bvandewa has joined #openstack-ironic | 03:37 | |
| *** jerrygb has quit IRC | 03:39 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: [devstack]Only cleanup tftp directory if ipxe disabled https://review.openstack.org/342549 | 03:40 |
| *** jerrygb has joined #openstack-ironic | 03:40 | |
| *** jerrygb has quit IRC | 03:41 | |
| *** jerrygb has joined #openstack-ironic | 03:41 | |
| *** ijw has quit IRC | 03:41 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/342551 | 03:41 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/342552 | 03:41 |
| *** piet has quit IRC | 03:45 | |
| *** hoangcx has quit IRC | 03:50 | |
| *** yuanying has joined #openstack-ironic | 03:53 | |
| *** vmud213 has joined #openstack-ironic | 03:54 | |
| *** PollyZ has joined #openstack-ironic | 03:55 | |
| *** clenimar_ has joined #openstack-ironic | 03:55 | |
| *** bvandewa has quit IRC | 03:58 | |
| *** Nisha_away has joined #openstack-ironic | 04:01 | |
| *** sdake has quit IRC | 04:05 | |
| *** amotoki has joined #openstack-ironic | 04:05 | |
| *** jcoufal has quit IRC | 04:06 | |
| *** rbudden has joined #openstack-ironic | 04:12 | |
| *** amotoki has quit IRC | 04:12 | |
| *** PollyZ has quit IRC | 04:13 | |
| *** PollyZ has joined #openstack-ironic | 04:16 | |
| *** rbudden has quit IRC | 04:18 | |
| *** amotoki has joined #openstack-ironic | 04:22 | |
| *** amotoki has quit IRC | 04:22 | |
| *** amotoki has joined #openstack-ironic | 04:22 | |
| *** jerrygb has quit IRC | 04:34 | |
| *** PollyZ has quit IRC | 04:34 | |
| *** links has joined #openstack-ironic | 04:39 | |
| *** bvandewa has joined #openstack-ironic | 04:59 | |
| *** bvandewa has quit IRC | 05:04 | |
| *** rajinir has quit IRC | 05:05 | |
| *** sdake has joined #openstack-ironic | 05:17 | |
| *** fragatina has joined #openstack-ironic | 05:22 | |
| *** fragatin_ has joined #openstack-ironic | 05:23 | |
| *** _vsaienko has joined #openstack-ironic | 05:24 | |
| *** hoangcx has joined #openstack-ironic | 05:25 | |
| *** fragatina has quit IRC | 05:26 | |
| *** Sukhdev has joined #openstack-ironic | 05:33 | |
| *** _vsaienko has quit IRC | 05:33 | |
| *** jerrygb has joined #openstack-ironic | 05:34 | |
| *** bharath has joined #openstack-ironic | 05:35 | |
| *** _vsaienko has joined #openstack-ironic | 05:35 | |
| *** ijw has joined #openstack-ironic | 05:35 | |
| *** _vs_ has joined #openstack-ironic | 05:35 | |
| *** piet has joined #openstack-ironic | 05:37 | |
| *** jerrygb has quit IRC | 05:40 | |
| *** _vsaienko has quit IRC | 05:41 | |
| openstackgerrit | Cao ShuFeng proposed openstack/ironic: Set xtrace in devstack/plugin.sh https://review.openstack.org/342609 | 05:43 |
| *** _vs_ has quit IRC | 05:44 | |
| *** _vsaienko has joined #openstack-ironic | 05:44 | |
| *** amotoki has quit IRC | 05:44 | |
| *** _vsaienko has quit IRC | 05:44 | |
| *** ChubYann has quit IRC | 05:46 | |
| *** skramaja has joined #openstack-ironic | 05:48 | |
| *** amotoki has joined #openstack-ironic | 05:51 | |
| *** sdake has quit IRC | 05:51 | |
| *** Fdaisuke has quit IRC | 05:53 | |
| *** piet has quit IRC | 05:55 | |
| *** mjura has joined #openstack-ironic | 05:57 | |
| *** loki_ has joined #openstack-ironic | 05:58 | |
| *** rama_y has joined #openstack-ironic | 05:59 | |
| *** vmud213 has quit IRC | 06:02 | |
| *** _milan_ has quit IRC | 06:03 | |
| *** Fdaisuke has joined #openstack-ironic | 06:05 | |
| *** mjura has quit IRC | 06:05 | |
| *** mjura has joined #openstack-ironic | 06:06 | |
| *** hoangcx has quit IRC | 06:07 | |
| *** milan has joined #openstack-ironic | 06:07 | |
| *** vmud213 has joined #openstack-ironic | 06:07 | |
| *** hoangcx has joined #openstack-ironic | 06:09 | |
| *** vmud213 has quit IRC | 06:09 | |
| *** vmud213 has joined #openstack-ironic | 06:10 | |
| *** _milan_ has joined #openstack-ironic | 06:10 | |
| *** milan has quit IRC | 06:11 | |
| *** sdake has joined #openstack-ironic | 06:13 | |
| *** rcernin has joined #openstack-ironic | 06:17 | |
| *** _milan_ has quit IRC | 06:22 | |
| *** phuongnh has quit IRC | 06:22 | |
| *** milan has joined #openstack-ironic | 06:23 | |
| *** phuongnh has joined #openstack-ironic | 06:23 | |
| *** amotoki has quit IRC | 06:23 | |
| *** amotoki has joined #openstack-ironic | 06:24 | |
| *** milan has quit IRC | 06:28 | |
| *** milan has joined #openstack-ironic | 06:29 | |
| *** openstackgerrit has quit IRC | 06:33 | |
| *** openstackgerrit has joined #openstack-ironic | 06:33 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Fix doc toctree warnings https://review.openstack.org/342630 | 06:34 |
| *** chlong has quit IRC | 06:38 | |
| *** Sukhdev has quit IRC | 06:40 | |
| *** rama_y has quit IRC | 06:43 | |
| *** amotoki has quit IRC | 06:44 | |
| *** ccamacho|out has quit IRC | 06:47 | |
| *** sdake_ has joined #openstack-ironic | 06:49 | |
| *** ijw has quit IRC | 06:51 | |
| *** sdake has quit IRC | 06:52 | |
| openstackgerrit | Aparna proposed openstack/ironic: Documentation change for in-band cleaning https://review.openstack.org/338744 | 06:56 |
| *** phuongnh has quit IRC | 06:56 | |
| *** Fdaisuke_ has joined #openstack-ironic | 07:00 | |
| *** bvandewa has joined #openstack-ironic | 07:01 | |
| *** Fdaisuke has quit IRC | 07:01 | |
| *** amotoki has joined #openstack-ironic | 07:03 | |
| *** bvandewa has quit IRC | 07:06 | |
| *** Nisha_away has quit IRC | 07:13 | |
| *** tesseract- has joined #openstack-ironic | 07:15 | |
| *** sdake_ has quit IRC | 07:16 | |
| *** athomas has joined #openstack-ironic | 07:17 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: [devstack]Switch to pip_install_gr for inspector client https://review.openstack.org/342642 | 07:23 |
| *** pcaruana has joined #openstack-ironic | 07:27 | |
| milan | #pixiesay Morning, Ironic! -m flexing | 07:28 |
| PixieBoots | ᕙʕ⇀ᴥ⇀ʔᕗ: Morning, Ironic! | 07:28 |
| *** d0ugal has joined #openstack-ironic | 07:31 | |
| *** ijw has joined #openstack-ironic | 07:31 | |
| *** d0ugal has quit IRC | 07:31 | |
| *** d0ugal has joined #openstack-ironic | 07:31 | |
| *** d0ugal has quit IRC | 07:31 | |
| *** d0ugal has joined #openstack-ironic | 07:31 | |
| *** ijw has quit IRC | 07:36 | |
| *** Romanenko_K has joined #openstack-ironic | 07:36 | |
| *** jerrygb has joined #openstack-ironic | 07:37 | |
| *** hoangcx has quit IRC | 07:39 | |
| *** hoangcx has joined #openstack-ironic | 07:40 | |
| *** jerrygb has quit IRC | 07:41 | |
| *** ifarkas has joined #openstack-ironic | 07:42 | |
| *** ohamada has joined #openstack-ironic | 07:42 | |
| *** daemontool has joined #openstack-ironic | 07:43 | |
| *** amotoki has quit IRC | 07:48 | |
| *** linuxaddicts has quit IRC | 07:54 | |
| *** chenke has joined #openstack-ironic | 07:58 | |
| openstackgerrit | Yuiko Takada Mori proposed openstack/ironic: IPMITool: add IPMISocatConsole and IPMIConsole class https://review.openstack.org/293873 | 07:59 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** derekh has joined #openstack-ironic | 08:04 | |
| *** hoangcx has quit IRC | 08:05 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/341698 | 08:15 |
| aarefiev | morning! | 08:18 |
| *** MattMan has quit IRC | 08:20 | |
| *** MattMan has joined #openstack-ironic | 08:20 | |
| *** e0ne has joined #openstack-ironic | 08:21 | |
| milan | morning aarefiev ! :) | 08:21 |
| aarefiev | morning milan | 08:22 |
| *** lucas|afk is now known as lucasagomes | 08:23 | |
| milan | #pixiesay Morning, lucasagomes! ;) -m happy | 08:24 |
| PixieBoots | ʕ^ᴥ^ʔ: Morning, lucasagomes! ;) | 08:24 |
| openstackgerrit | Merged openstack/ironic-inspector: [devstack]Only cleanup tftp directory if ipxe disabled https://review.openstack.org/342549 | 08:24 |
| *** caoshufeng_ has quit IRC | 08:25 | |
| openstackgerrit | Merged openstack/ironic: Expose node's network_interface field in API https://review.openstack.org/317392 | 08:27 |
| ifarkas | morning aarefiev, milan, lucasagomes and all! | 08:29 |
| openstackgerrit | Merged openstack/ironic: Follow-up patch of 0fcf2e8b51e7dbbcde6d4480b8a7b9c807651546 https://review.openstack.org/342545 | 08:29 |
| aarefiev | morning ifarkas | 08:30 |
| *** ijw has joined #openstack-ironic | 08:32 | |
| milan | morning ifarkas! :) | 08:32 |
| lucasagomes | morning milan ifarkas aarefiev all :-) | 08:32 |
| aarefiev | morning lucasagomes> | 08:33 |
| *** ijw has quit IRC | 08:38 | |
| *** caoshufeng has joined #openstack-ironic | 08:38 | |
| caoshufeng | Hi, can someone help me with this: https://review.openstack.org/#/c/342609/ | 08:41 |
| *** dtantsur|afk is now known as dtantsur | 08:44 | |
| dtantsur | Morning Ironic, aarefiev, lucasagomes, milan, ifarkas | 08:44 |
| milan | morning dtantsur! ;) | 08:45 |
| aarefiev | morning dtantsur | 08:46 |
| openstackgerrit | Merged openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/342552 | 08:56 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Support to validate iLO SSL certificate in iLO drivers https://review.openstack.org/338791 | 08:59 |
| *** bvandewa has joined #openstack-ironic | 09:03 | |
| *** hoangcx has joined #openstack-ironic | 09:04 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: [devstack]Remove unneeded enable_service in example.local.conf https://review.openstack.org/342687 | 09:06 |
| *** bvandewa has quit IRC | 09:09 | |
| *** jrist has joined #openstack-ironic | 09:09 | |
| *** zhenguo_ is now known as zhenguo | 09:14 | |
| *** mgould|afk is now known as mgould | 09:27 | |
| mgould | morning team! | 09:27 |
| dtantsur | hey mgould | 09:28 |
| mgould | caoshufeng: I'll take a look | 09:28 |
| milan | morning mgould! :) | 09:28 |
| mgould | hi dtantsur milan aarefiev lucasagomes | 09:28 |
| caoshufeng | mgould, thanks. | 09:28 |
| dtantsur | sambetts, hi! I think the spec is approved for https://review.openstack.org/279940 | 09:29 |
| aarefiev | morning mgould | 09:29 |
| *** mbound has joined #openstack-ironic | 09:29 | |
| *** bvandewa has joined #openstack-ironic | 09:30 | |
| *** jpich has joined #openstack-ironic | 09:30 | |
| mgould | random question: does everyone know the literal meaning of "nit"? | 09:32 |
| *** ijw has joined #openstack-ironic | 09:33 | |
| milan | mgould, you mean the annoying hair-dwelling blood-sucking bug egg? | 09:34 |
| openstackgerrit | Merged openstack/ironic-specs: Collect system logs specs: Change the file name template https://review.openstack.org/336108 | 09:34 |
| loki_ | mgould review please https://review.openstack.org/#/c/272658/ | 09:35 |
| openstackgerrit | Merged openstack/ironic-inspector: [devstack]Switch to pip_install_gr for inspector client https://review.openstack.org/342642 | 09:35 |
| lucasagomes | mgould, idk, I think it's an abbreviation for "nit picking" | 09:37 |
| lucasagomes | nitpicking* | 09:37 |
| openstackgerrit | Merged openstack/ironic-specs: Update bare metal trust spec to reflect the current solution. https://review.openstack.org/338118 | 09:37 |
| openstackgerrit | Cao ShuFeng proposed openstack/ironic: Set xtrace in devstack/plugin.sh https://review.openstack.org/342609 | 09:37 |
| *** ijw has quit IRC | 09:40 | |
| *** athomas has quit IRC | 09:42 | |
| openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 09:42 |
| *** yuanying is now known as yuanying-afk | 09:43 | |
| *** piet has joined #openstack-ironic | 09:46 | |
| *** athomas has joined #openstack-ironic | 09:48 | |
| mgould | milan: yes, exactly :-) | 09:55 |
| mgould | lucasagomes: "nitpicking" literally means "picking insect eggs out of someone's hair" | 09:55 |
| mgould | loki_: looking now | 09:55 |
| milan | dtantsur, according to https://github.com/openstack-dev/grenade/blob/master/grenade.sh#L268 inspector's grenade verify will be called twice (before and after the upgrade) so we could possibly even disable the BASE_RUN_SMOKE, wdyt? | 10:00 |
| dtantsur | milan, I'm in favor of it, but dunno what infra says... we can try | 10:01 |
| milan | dtantsur, OK, maybe a separate patch | 10:02 |
| milan | dtantsur, most likely something like: "no any other project does this; -1" ;) | 10:02 |
| dtantsur | yeah | 10:04 |
| dtantsur | openstack is very nova-oriented :( | 10:04 |
| milan | or: "ask some QE first! -1" | 10:04 |
| milan | #pixiesay nova -m meh | 10:05 |
| PixieBoots | ¯\_(ツ)_/¯: nova | 10:05 |
| mgould | what do Nova hackers call themselves? | 10:08 |
| milan | dtantsur, I've no clue how to figure out whether the verify is pre- or post- upgrade; I'd say we need to do proper install-tempest-plugins call (proper path in /opt/stack/[old|new]) if to hack it | 10:08 |
| mgould | Neutron hackers call themselves "neutrinos", which I rather like | 10:08 |
| dtantsur | milan, dunno, needs experimenting :( | 10:09 |
| milan | mgould, ask on #nova? :P | 10:09 |
| milan | dtantsur, yeah, in that case I'd unabandon my infra patch to see some input while experimenting in parallel? wdyt? | 10:10 |
| dtantsur | milan, yeah, lets do it | 10:10 |
| milan | dtantsur, cool | 10:11 |
| mgould | milan: good idea | 10:13 |
| milan | dtantsur, what about un-1-ing it? | 10:13 |
| milan | mgould, yeah I'm afraid other ways are a bit insane | 10:13 |
| *** piet has quit IRC | 10:13 | |
| dtantsur | milan, you still want to run smoke tests? I think my concern is valid: it will take too much time | 10:14 |
| * milan 's been looking into the grenade script now for 2 hours | 10:14 | |
| mgould | milan: I actually meant about "ask on nova" :-) | 10:14 |
| milan | dtantsur, but the point is we'd better check the upgrade didn't break us | 10:14 |
| dtantsur | milan, lets first investigate if we can replace smoke tests with a simpler check | 10:14 |
| milan | and there's no simple&straightforward way to check it in verify | 10:14 |
| dtantsur | meh... | 10:14 |
| milan | as it's run twice | 10:15 |
| dtantsur | I worry that running smoke tests twice will lead to timeouts | 10:15 |
| *** yibo has quit IRC | 10:15 | |
| dtantsur | maybe we should run a simple check both pre- and post- upgrade? | 10:15 |
| *** linuxgeek has joined #openstack-ironic | 10:15 | |
| dtantsur | milan, ^^^ | 10:15 |
| milan | dtantsur, resource verify is run twice, if I want to exec tests I need to go to proper dir opt/stack/old/tempest first then opt/stack/new/tempest to exec our tempest smoke test | 10:16 |
| milan | but I didn't find sane way *yet* how to distinguish when the phase is being run | 10:16 |
| mgould | milan: can you touch a file the first time you run the tests, and check for its presence? | 10:17 |
| milan | grenade.sh doesn't export anything useful for that purpose or I missed it :) | 10:17 |
| milan | mgould, heh :D | 10:17 |
| dtantsur | mgould's idea is not so bad | 10:17 |
| milan | yeah, good idea, thx | 10:17 |
| dtantsur | also, I'm sad to say it, but we can run a bash-based exercise if we can't make tempest work for us... | 10:17 |
| dtantsur | (not the exercise we still have now, something much simpler) | 10:18 |
| *** piet has joined #openstack-ironic | 10:18 | |
| mgould | #pixiesay bash -m sad | 10:18 |
| * dtantsur -> place with food | 10:18 | |
| PixieBoots | ʕ•͡ᴥ•ʔ: bash | 10:18 |
| *** baoli has joined #openstack-ironic | 10:19 | |
| *** baoli_ has joined #openstack-ironic | 10:20 | |
| openstackgerrit | lokesh s proposed openstack/python-oneviewclient: log a warning and still continue if the version is greater than 2.0 https://review.openstack.org/342720 | 10:20 |
| lucasagomes | mgould, :D didn't know that | 10:21 |
| * lucasagomes is sorry for the delay, he was in his 1:1 | 10:21 | |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/342551 | 10:21 |
| milan | #pixiesay bending grenade cause... no saner way to smoke-test project's stuff -m dead | 10:21 |
| PixieBoots | ʕxᴥxʔ: bending grenade cause... no saner way to smoke-test project's stuff | 10:21 |
| mgould | yep, sounds about right | 10:23 |
| *** baoli has quit IRC | 10:24 | |
| *** baoli_ has quit IRC | 10:24 | |
| mgould | lucasagomes: when I was at primary school we were periodically checked for nits, because they spread very easily in schools | 10:25 |
| mgould | dunno if they still do that, but probably | 10:25 |
| aarefiev | milan: what is the diff between pre and post verify? | 10:25 |
| *** rcernin has quit IRC | 10:26 | |
| milan | aarefiev, one should verify that the resource created is still "alive" such as network exists | 10:26 |
| milan | aarefiev, at least that's how I understand it | 10:26 |
| milan | so those should be the same | 10:26 |
| lucasagomes | mgould, heh yeah, we had some of that too... not sure if it's still a thing | 10:26 |
| lucasagomes | maybe | 10:26 |
| * milan about to bend that call to do two different things | 10:27 | |
| mgould | I expect so, I doubt either small children or parasites have changed much :-) | 10:27 |
| milan | aarefiev, dtantsur which feels a bit abusive tbh | 10:27 |
| mgould | milan: :-( | 10:27 |
| *** chenke has quit IRC | 10:28 | |
| milan | aarefiev, best is to check the intended behaviour here: https://github.com/openstack-dev/grenade/blob/master/README.rst#basic-flow | 10:28 |
| milan | so actually me'd -1 his own patch based on that | 10:29 |
| aarefiev | milan, thanks, dunno much about grenade, I'm trying to figure out how it will be | 10:29 |
| milan | #pixiesay I wish there was a face palm mood -m dead | 10:29 |
| PixieBoots | ʕxᴥxʔ: I wish there was a face palm mood | 10:29 |
| milan | aarefiev, np, I'm learning it on the fly so I may be wrong | 10:29 |
| *** ccamacho has joined #openstack-ironic | 10:30 | |
| *** ccamacho1 has joined #openstack-ironic | 10:30 | |
| mgould | milan: so the problem is that "tempest smoke" times out, so you want to replace it with something simpler/quicker? | 10:31 |
| mgould | and the simpler/quicker thing needs to know if we're pre- or post-upgrade? | 10:31 |
| dtantsur | mmm, I don't think it needs | 10:31 |
| milan | mgould, it doesn't time out yet as the smoke tests aren't executed at all after the upgrade | 10:31 |
| dtantsur | lets just run introspection on one node, why do we need to know pre/post? | 10:32 |
| mgould | yeah, ISTM the test shouldn't care | 10:32 |
| aarefiev | dtantsur: it's what I'm trying to understand | 10:32 |
| mgould | and the ideal fix would be "make tempest smoke fast enough that it doesn't time out" | 10:32 |
| mgould | though I guess we're running into the "nested VMs are slow" problem? | 10:33 |
| dtantsur | mgould, we do; smoke tests run about 7 ironic deployments already, without even our tests | 10:33 |
| dtantsur | they were designed for nova, where launching a vm is not so slow | 10:33 |
| * milan will upgrade the pull request for the discussion so we talk about the bash "code" involved | 10:33 | |
| mgould | ah | 10:33 |
| aarefiev | btw, when we are talking about 'smoke tests', what exactly test we mean | 10:34 |
| aarefiev | for inspector case | 10:34 |
| mgould | and there's no way to override the tests run by "tempest smoke"? | 10:34 |
| dtantsur | aarefiev, I have a patch up adding a simole test | 10:34 |
| milan | aarefiev, tox -e smoke in tempest | 10:34 |
| mgould | that seems like an oversight | 10:34 |
| dtantsur | mgould, the suggestion to allow it was -2'ed | 10:34 |
| aarefiev | only inspector one, right ? | 10:34 |
| milan | aarefiev, so all that lives in that venv | 10:34 |
| milan | atm 64 tests | 10:34 |
| dtantsur | aarefiev, it will run all smoke tests | 10:34 |
| aarefiev | ahh | 10:34 |
| dtantsur | most of them are nova's and other core services' | 10:34 |
| *** piet has quit IRC | 10:35 | |
| milan | dtantsur, aarefiev volumes and neutron and nova are the majority | 10:35 |
| milan | aarefiev, the problem is grenade is designed to verify a resource is sane and lives through an upgrade without any harm | 10:36 |
| milan | but some services don't create resources | 10:36 |
| milan | so those services should do something else to "verify" the upgrade didn't break their functionality | 10:37 |
| mgould | ...like ironic-inspector? | 10:37 |
| mgould | aaaah | 10:37 |
| milan | mgould, ++ | 10:37 |
| mgould | like, say, inspecting a node, upgrading, and checking the output we get is a superset of what we got before | 10:37 |
| *** ijw has joined #openstack-ironic | 10:37 | |
| milan | mgould, or run smoke tests before and after upgrade | 10:37 |
| milan | and leave that logic to those tests | 10:37 |
| milan | * to figure out the sanity of the upgrade | 10:38 |
| mgould | can we run all the irrelevant nova tests on a faster backend? | 10:38 |
| mgould | it's wasted computation, but it's *less* wasted computation | 10:38 |
| milan | aarefiev, mgould but there's just a limited amount of particular upgrade phase "call-backs" one can hook anything into | 10:39 |
| milan | and it seems grenade was designed with nova in mind | 10:39 |
| dtantsur | mgould, we share one devstack configuration. I'm not sure how to enable ironic without enabling its nova backend.. | 10:39 |
| milan | ironic doesn't mind, cause it is the driver for the resources to be created, so it makes sense there | 10:39 |
| aarefiev | ok, if 'verify' checks resources, could we do nothing for it now, and just run tests second time for check everything is good after upgrade | 10:42 |
| *** ijw has quit IRC | 10:42 | |
| milan | aarefiev, that's my suggested patch but it got -1ed | 10:43 |
| milan | aarefiev, mgould https://review.openstack.org/#/c/342275/ | 10:43 |
| mgould | dtantsur: and nova gets confused if we have Ironic + $other_backend enabled? | 10:45 |
| *** rcernin has joined #openstack-ironic | 10:45 | |
| dtantsur | mgould, it's not so possible :) you need several nova computes for that | 10:46 |
| aarefiev | so, I see this in workflow, after upgrade "perform some sanity checking (currently tempest smoke) to ensure everything seems good" | 10:46 |
| mgould | aaah, OK | 10:46 |
| mgould | dammit | 10:46 |
| dtantsur | and then you need something to distinguish between virt and "bm", e.g. host aggregates | 10:46 |
| dtantsur | folks, aarefiev, milan. currently grenade runs around 1h 36m 50s (taken from one of my patches) | 10:46 |
| *** rcernin is now known as rcernin|lunch | 10:46 | |
| * mgould ponders a stub driver which just says "yep, done!" and then doesn't do any provisioning | 10:47 | |
| dtantsur | if we enable one more set of smoke tests, we'll probably go up to 2 hours | 10:47 |
| dtantsur | mgould, it's called "fake" in ironic :) aaannd I need to think if we can do that actually | 10:47 |
| milan | aarefiev, mgould the other thing would be to hack ironic-inspector/devstack/upgrade/resources.sh (in a separate patch | 10:47 |
| milan | dtantsur, how about we limit the amount of tests executed instead? | 10:47 |
| dtantsur | milan, how do you do that? | 10:47 |
| milan | tox -esmoke -- ironic_inspector_tests.InspectorSmokeTests | 10:48 |
| dtantsur | you mean, fix grenade to do that or hack our code? | 10:48 |
| milan | hack-in some variable into grenade to limit it | 10:48 |
| milan | yeah | 10:48 |
| milan | dtantsur, grenade I suppose | 10:48 |
| dtantsur | milan, this was -2ed already by #openstack-qa for ironic :( | 10:48 |
| dtantsur | but you can try convincing them </irony> | 10:48 |
| * milan facepalm | 10:49 | |
| mgould | I guess the grenade people want to know that upgrading inspector won't break nova or neutron | 10:49 |
| aarefiev | O_o looks like we need grenade guys | 10:49 |
| mgould | which it shouldn't ever, but we'd have to convince them of that | 10:49 |
| aarefiev | how they see this | 10:49 |
| milan | mgould, grenade guys don't care about inspector IMHO as it's not yet being executed in the tests at all | 10:49 |
| milan | aarefiev, possibly | 10:50 |
| mgould | OK | 10:50 |
| dtantsur | qa folks are so heavily overloaded, we won't grab them easily | 10:50 |
| milan | aarefiev, but mind dtantsur 's irony :D | 10:50 |
| aarefiev | :) | 10:50 |
| mgould | milan: so we're trying to test "after applying the patch under test, upgrading your cloud will not break it", yes? | 10:50 |
| * milan doesn't get the lack of autonomy in project's own gating system | 10:50 | |
| mgould | me neither | 10:51 |
| milan | mgould, exactly | 10:51 |
| milan | I mean I totally get why they want to exec all smoke test cases | 10:51 |
| mgould | and our argument is "inspector will never ever break nova or neutron upgrades, we can skip those tests in inspector gate"? | 10:52 |
| milan | but then there'd better be an alternative for projects that don't create resources :-/ | 10:52 |
| aarefiev | what if runs tests only after upgrade | 10:52 |
| aarefiev | assume before all work :) just say | 10:52 |
| milan | my patch would have to be "enhanced" with BASE_TEMPEST_RUN=false and that would mean -2ing it as "no other project does that! (tm)" :D | 10:53 |
| aarefiev | :) | 10:53 |
| dtantsur | I'm afraid the same :( | 10:53 |
| * milan face-palms his face-palm | 10:53 | |
| dtantsur | what if we try mgould's idea about using a marker file? so before we run smoke, after we run only our test? | 10:54 |
| dtantsur | this sounds like an acceptable compromise to me | 10:54 |
| milan | dtantsur, I don't mind for pragmatics sake, but it's IMHO against the design goals of grenade :-/ | 10:54 |
| milan | dtantsur, I'm hacking on it atm | 10:55 |
| milan | though | 10:55 |
| dtantsur | oh, someone here knows design goals of grenade :D | 10:55 |
| * dtantsur is kidding ofc | 10:55 | |
| aarefiev | but "no other project does that" too :) | 10:55 |
| milan | dtantsur, so I feel like -1 ing my own patch | 10:55 |
| milan | :D | 10:55 |
| milan | aarefiev, exactly lol | 10:55 |
| dtantsur | well, at least everything in our tree can be landed by us... | 10:59 |
| dtantsur | I hate that we spend to much time working around grenade limitations ofc | 10:59 |
| aarefiev | I like idea running second time only inspector test | 10:59 |
| dtantsur | yes. I think we should also check that 'introspection status' command returns something meaningful after the upgrade | 11:00 |
| aarefiev | +1 | 11:00 |
| dtantsur | that's the only resource we have after all :) | 11:00 |
| jroll | dtantsur: check with vsaienk0, I think there is a way to limit the tests before/after upgrade | 11:00 |
| * milan thinks about aliasing tox for smoke test command to f*ck-hack grenade even more | 11:00 | |
| jroll | morning y'all | 11:00 |
| milan | :D | 11:00 |
| milan | jroll, morning | 11:00 |
| aarefiev | so we don't need changes on grenade side for this right ? | 11:00 |
| aarefiev | morning jroll | 11:00 |
| dtantsur | morning jroll | 11:01 |
| dtantsur | aarefiev, I hope we don't.. | 11:02 |
| *** caoshufeng has quit IRC | 11:02 | |
| mgould | morning jroll | 11:02 |
| milan | dtantsur, aarefiev, mgould, jroll, we could spoof $TEMPEST_CONCURRENCY ;) TEMPEST_CONCURRENCY=$TEMPEST_CONCURRENCY -- ironic_inspector_tests.InspectorSmokeTest" ; see https://github.com/openstack-dev/grenade/blob/master/grenade.sh#L322 | 11:04 |
| milan | but for that to work, https://review.openstack.org/#/c/342275/ has to be merged | 11:05 |
| dtantsur | lol | 11:05 |
| jroll | milan: there's no sense in just hacking grenade, we'll end up broken, other people will be surprised when they try to debug it, etc | 11:05 |
| *** Goneri has joined #openstack-ironic | 11:05 | |
| dtantsur | lets start with something simple first, then we can infinitely improve it | 11:05 |
| dtantsur | I only care about us running at least something inspector-related after upgrade | 11:05 |
| dtantsur | milan, ^^^ | 11:06 |
| milan | jroll, I agree, but there's lot of friction to even have our own gate job do what we want it to do :-/ | 11:06 |
| *** vmud213 has quit IRC | 11:06 | |
| milan | dtantsur, my suggested hack is the easiest, in our control way how we can achieve it right now | 11:06 |
| milan | i mean in 2 minutes | 11:07 |
| jroll | milan: I'd rather run something that isn't grenade, rather than using shell injection hacks to make it do something else | 11:07 |
| dtantsur | milan, what's the problem with running what we want in "verify" phase? | 11:07 |
| dtantsur | then at least we're using public interfaces... | 11:07 |
| milan | dtantsur, we'd be bending grenade even more | 11:07 |
| dtantsur | definitely not more | 11:08 |
| mgould | jroll: that sounds like a better idea, but will that allow us to get an official "we support upgrades" stamp? | 11:08 |
| milan | dtantsur, ok, then more-or-less :D | 11:08 |
| dtantsur | we run verification in verify, that's not such a big stretch | 11:08 |
| dtantsur | mgould, jroll, we can't say "oh we won't run grenade", because the whole upgrade testing setup in devstack-gate relies on it | 11:08 |
| * milan running low on blood sugar level and caffeine | 11:08 | |
| mgould | dtantsur: drat it | 11:09 |
| dtantsur | we'll have to reproduce an insane amount of code both in our tree and in devstack-gate | 11:09 |
| mgould | it seems like a really bad fit for our needs | 11:09 |
| mgould | as well as being yet another godawful Bash hydra | 11:09 |
| dtantsur | well, if someone finally explains me why we can't use "verify" phase.... | 11:09 |
| milan | dtantsur, I'll be posting that verify phase hack in a bit, but I need to refill my coffee mug first ;) | 11:10 |
| dtantsur | okie-okie, take your time :) | 11:10 |
| mgould | dtantsur, when is "verify" run? | 11:10 |
| milan | dtantsur, yes, we can, no explanation, it's a hack-for-hack change :D | 11:10 |
| dtantsur | mgould, not sure exactly, definitely after upgrade :D | 11:10 |
| *** lucasagomes is now known as lucas-hungry | 11:10 | |
| jroll | dtantsur: ah, I see | 11:10 |
| jroll | dtantsur: maybe I can bug sean next week about making this work for folks that don't need nova/etc in place | 11:11 |
| dtantsur | would be awesome | 11:11 |
| dtantsur | milan, I'm not sure why you call it "hack" at all. we should not use "verify" for verification or what? | 11:11 |
| mgould | dtantsur: looks like "verify" is run (a) during base install, after tempest smoke, and (b) after upgrade, before tempest smoke | 11:13 |
| dtantsur | soooo? | 11:13 |
| dtantsur | sounds like a fit for us, no? | 11:13 |
| milan | dtantsur, matter of one's hack-taste :D | 11:13 |
| dtantsur | (unless there is another crazyness in the whole thing) | 11:13 |
| mgould | dtantsur: sure, seems like exactly the right place | 11:14 |
| dtantsur | ok, lets just try it :) maybe we're arguing about something that is not possible... | 11:14 |
| mgould | but we have to run tempest smoke as well, and that can time out? | 11:14 |
| * milan 's coffee dripper makes the gurgling sounds :) | 11:14 | |
| dtantsur | mgould, well, it can. however, it does not timeout now, so there are some hopes that it won't timeout if our test is quick enough | 11:15 |
| dtantsur | introspection itself takes a couple of minutes | 11:15 |
| jroll | dtantsur: does inspector CI boot instances? or just do inspection and make sure things are happy? | 11:15 |
| mgould | on second thoughts, I think "resources verify" is meant to test that the resources are there, and actual "does it work?" tests should go in tempest smoke | 11:15 |
| dtantsur | jroll, our main CI does. we don't plan that for grenade | 11:15 |
| mgould | dtantsur: OK | 11:16 |
| jroll | dtantsur: also, thinking about it more... if nova isn't present in the grenade jobs I believe most of the smoke tests would/could be skipped | 11:16 |
| dtantsur | jroll, well, if infra folks allow us to disable nova... | 11:16 |
| jroll | dtantsur: (except the smoke tests in inspector's plugin) | 11:16 |
| jroll | why not? | 11:16 |
| dtantsur | jroll, well, they had problems with us setting one boolean value allowing running smoke tests after upgrade... | 11:17 |
| dtantsur | jroll, milan has more context | 11:17 |
| * dtantsur -> afk for ~ an hour | 11:17 | |
| mgould | "disable nova because there's no way we could break it" might be an easier sell | 11:18 |
| jroll | dtantsur: I feel like there's probably something we're missing there that they know and we don't | 11:18 |
| *** adu has joined #openstack-ironic | 11:18 | |
| dtantsur | mgould, I'm not sure we can't break it at all, but it's unlikely enough | 11:18 |
| dtantsur | jroll, probably... | 11:19 |
| *** dtantsur is now known as dtantsur|brb | 11:20 | |
| *** hoangcx has quit IRC | 11:21 | |
| milan | jroll, please, see this patch: https://review.openstack.org/#/c/342275/ and my comment to Ian | 11:25 |
| milan | jroll, infra -1 me for wanting to run smoke tests (no matter which) after the upgrade | 11:26 |
| mgould | milan: wat | 11:26 |
| * mgould thought that was THE WHOLE POINT of grenade | 11:27 | |
| milan | jroll, dtantsur|brb doesn't like the second run of the smoke tests after the upgrade being afraid of making it take too long | 11:27 |
| aarefiev | milan, dtantsur: looking at greande logs in inspector, I see two tests run I'm confused | 11:27 |
| milan | aarefiev, paste please? | 11:27 |
| jroll | milan: all other projects run some tests after the upgrade, but don't set that variable, so there's something we're missing here | 11:27 |
| milan | jroll, maybe it's me | 11:28 |
| * jroll looking around grenade | 11:28 | |
| aarefiev | milan: http://logs.openstack.org/01/316801/5/check/gate-grenade-dsvm-ironic-inspector-nv/5b7a9fe/logs/grenade.sh.txt.gz | 11:28 |
| aarefiev | run base smote tests in 1360 s | 11:28 |
| aarefiev | console logs http://logs.openstack.org/01/316801/5/check/gate-grenade-dsvm-ironic-inspector-nv/5b7a9fe/console.html.gz | 11:28 |
| aarefiev | same tests in 1420s | 11:29 |
| aarefiev | Running tempest smoke tests after upgrade | 11:29 |
| milan | aarefiev, /me looks | 11:30 |
| jroll | I agree with aarefiev, it is running smoke after upgrade | 11:31 |
| mgould | aarefiev: looks like that's happening *after* grenade | 11:31 |
| mgould | see http://logs.openstack.org/01/316801/5/check/gate-grenade-dsvm-ironic-inspector-nv/5b7a9fe/console.html.gz#_2016-07-08_11_07_33_606374 | 11:31 |
| aarefiev | mgould: yeah in console logs results after upgrade | 11:31 |
| aarefiev | in grenade.sh before upgrade | 11:32 |
| mgould | OK, looks like the second tempest smoke run is being called from devstack-vm-gate.sh | 11:32 |
| openstackgerrit | lokesh s proposed openstack/python-oneviewclient: Log a warning and still continue if the version is greater than 2.0 https://review.openstack.org/342720 | 11:33 |
| milan | mgould, that would explain why smoke tests are executed even without setting the var https://github.com/openstack-dev/grenade/blob/master/grenade.sh#L319 | 11:33 |
| milan | aarefiev, thx for pointing it out, I've missed it | 11:34 |
| mgould | OK, what the hell | 11:34 |
| milan | so now what | 11:34 |
| jroll | milan: right, DEVSTACK_GATE_TEMPEST=1 will make it run tempest after grenade is done upgrading | 11:35 |
| jroll | (why it's running smoke and not something else, dunno) | 11:35 |
| milan | :-/ | 11:35 |
| jroll | oh, that's the fallthrough | 11:35 |
| jroll | https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate.sh#L788 | 11:35 |
| milan | :-/ | 11:36 |
| jroll | so you can change DEVSTACK_GATE_TEMPEST_REGEX, etc, if there's something else you want to run post-upgrade | 11:36 |
| milan | jroll, maybe it's sane | 11:36 |
| jroll | milan: it seems fine to me, though I think we should delete https://github.com/openstack-dev/grenade/blob/master/grenade.sh#L319 | 11:37 |
| jroll | if nobody is using it :) | 11:37 |
| milan | it's a bit twisted but if all the gate does by default is to run smoke tests, then grenade being the job shouldn't do it on it's own | 11:37 |
| jroll | well, it's only if you enable the job to run tempest | 11:37 |
| jroll | which is DEVSTACK_GATE_TEMPEST=1 | 11:38 |
| * milan sighs | 11:38 | |
| *** jerrygb has joined #openstack-ironic | 11:38 | |
| *** ijw has joined #openstack-ironic | 11:38 | |
| aarefiev | I guess one thing is missing inspector test | 11:39 |
| jroll | I actually like it this way, because we can run whichever tempest tests we choose after grenade runs | 11:39 |
| mgould | jroll: there are a bunch of DEVSTACK_GATE_* variables which control what tests get run | 11:39 |
| jroll | mgould: I know | 11:39 |
| mgould | ah, sorry | 11:39 |
| milan | jroll, I had the false impression grenade should be in control of what test cases are run before/after upgrade, my fault | 11:39 |
| jroll | milan: before yes, after apparently not | 11:39 |
| milan | yeah | 11:39 |
| jroll | I guess think of the upgrade as the test setup | 11:40 |
| milan | jroll, exactly, it makes sense to me now | 11:40 |
| jroll | awesome :) | 11:40 |
| milan | ;) | 11:40 |
| * milan re-abandons his pull request now with proper explanation :D | 11:41 | |
| milan | jroll, aarefiev, mgould, dtantsur|brb thanks for help, guys! | 11:41 |
| aarefiev | milan: np | 11:41 |
| jroll | welcome :D | 11:42 |
| *** jerrygb has quit IRC | 11:43 | |
| *** ijw has quit IRC | 11:43 | |
| * mgould yolo's a patch to grenade: https://review.openstack.org/342756 | 11:48 | |
| TheJulia | good morning everyone | 11:48 |
| mgould | morning TheJulia | 11:48 |
| milan | mgould, maybe someone uses it for "debugging" purposes ;) | 11:49 |
| milan | morning TheJulia | 11:49 |
| mgould | you just missed an hour of despair and confusion about grenade | 11:49 |
| mgould | milan: yeah, maybe | 11:49 |
| *** derekh has quit IRC | 11:49 | |
| mgould | if so, that should be documented | 11:49 |
| mgould | and hopefully my patch will enrage infra cores into telling me so | 11:49 |
| mgould | but it'll probably just get Warnocked | 11:49 |
| mgould | rage-driven development? | 11:50 |
| milan | lol | 11:50 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Follow-up to 317392 https://review.openstack.org/342477 | 11:50 |
| *** dtantsur|brb is now known as dtantsur | 11:55 | |
| *** piet has joined #openstack-ironic | 11:57 | |
| *** _vs_ has joined #openstack-ironic | 11:57 | |
| dtantsur | jroll, is it possible you bribe someone from qa to review https://review.openstack.org/337372 ? | 11:57 |
| dtantsur | it seems like it's the only thing we're missing | 11:57 |
| jroll | dtantsur: really? O_o | 12:00 |
| jroll | dtantsur: the plugins should already be installed in the tempest venv no? | 12:00 |
| dtantsur | jroll, it gets installed in the old version tempest, then it does not get installted in the new one | 12:00 |
| *** ccamacho1 is now known as ccamacho_ | 12:01 | |
| dtantsur | but dunno, maybe I misunderstand something | 12:01 |
| milan | dtantsur, +1 | 12:01 |
| milan | jroll, dtantsur https://github.com/openstack-dev/grenade/blob/master/grenade.sh#L250 | 12:01 |
| milan | it seems so indeed | 12:01 |
| TheJulia | vsaienk0: vdrok is back next week right? | 12:02 |
| jroll | it copies .tox over, though, which should have the plugins installed in it | 12:02 |
| jroll | milan: dtantsur: I think if you run something like tox -e all with a regex after upgrade, you will see your plugin is installed. you don't see any inspector tests now because they aren't tagged with 'smoke' | 12:03 |
| *** lucas-hungry is now known as lucasagomes | 12:03 | |
| dtantsur | jroll, they are | 12:03 |
| dtantsur | jroll, could you show me a place where .tox is copied? | 12:04 |
| *** dprince has joined #openstack-ironic | 12:04 | |
| jroll | dtantsur: https://github.com/openstack-dev/grenade/blob/master/grenade.sh#L253 | 12:04 |
| mgould | dtantsur: https://review.openstack.org/#/admin/groups/306,members are the people we need to bribe, right? | 12:04 |
| milan | jroll, to me it sounds it transplants old tempest plugins into new tempest dir to be run after in the gate | 12:04 |
| dtantsur | strange, I clearly see it not running it any more | 12:04 |
| dtantsur | patch related: https://review.openstack.org/#/c/336532/ | 12:04 |
| jroll | dtantsur: right, where do you have plugin tests tagged as smoke? | 12:05 |
| dtantsur | jroll, see patch | 12:05 |
| jroll | ah | 12:05 |
| dtantsur | and I see these tests running before upgrade, but not after | 12:05 |
| *** bvandewa has quit IRC | 12:06 | |
| milan | dtantsur, jroll anyway it's odd as old test code is being run against new service | 12:06 |
| milan | or I'm missing something? | 12:06 |
| dtantsur | here is the test being run before upgrade: http://logs.openstack.org/32/336532/11/check/gate-grenade-dsvm-ironic-inspector-nv/1bf6cdc/logs/grenade.sh.txt.gz#_2016-07-04_19_57_43_370 | 12:06 |
| milan | * w/r the rsync | 12:06 |
| jroll | hrm, and smoke runs against the normal venv | 12:07 |
| jroll | milan: the master plugin is meant to be used against all stable branches | 12:07 |
| dtantsur | and here it does not get run: http://logs.openstack.org/32/336532/11/check/gate-grenade-dsvm-ironic-inspector-nv/1bf6cdc/console.html.gz#_2016-07-04_20_35_00_972485 | 12:07 |
| milan | jroll, so old smoke tests should work with new project, that's the intent? | 12:08 |
| jroll | interesting | 12:08 |
| jroll | milan: there's no "old" smoke tests, tempest is branchless and the tempest plugins are also meant to be used without branches | 12:09 |
| milan | jroll, ah, OK :-/ that's a bit unintuitive | 12:09 |
| jroll | dtantsur: oh interesting, I thought we ran baremetal basic ops after upgrade, apparently not | 12:09 |
| jroll | (in ironic) | 12:10 |
| aarefiev | jroll: looks like there is similar nova smoke test | 12:11 |
| aarefiev | should boot instance with ironic | 12:11 |
| jroll | dtantsur: aha, this breaks it http://logs.openstack.org/32/336532/11/check/gate-grenade-dsvm-ironic-inspector-nv/1bf6cdc/logs/grenade.sh.txt.gz#_2016-07-04_20_06_47_981 | 12:11 |
| jroll | aarefiev: yeah | 12:11 |
| jroll | I just thought we also ran baremetal tests | 12:11 |
| dtantsur | oh right | 12:13 |
| jroll | I' | 12:14 |
| jroll | oops | 12:14 |
| jroll | I've gotta bounce to an appointment and a couple quick errands, be back in a couple hours | 12:15 |
| jroll | posted that grenade patch in -qa fwiw | 12:15 |
| *** clenimar_ has quit IRC | 12:17 | |
| *** _vs_ has quit IRC | 12:18 | |
| *** _vsaienko has joined #openstack-ironic | 12:20 | |
| milan | dtantsur, https://review.openstack.org/#/c/336532/11/devstack/upgrade/settings: TEMPEST_PLUGINS= '.../new/...' is OK? i.e. master plugins against stable service inside grenade (all gets confusing with the rsync) | 12:20 |
| *** _vs_ has joined #openstack-ironic | 12:20 | |
| dtantsur | I think I've copy-pasted it from somewhere | 12:20 |
| milan | dtantsur, OK :) | 12:20 |
| * milan confused^2 | 12:21 | |
| milan | :D | 12:21 |
| openstackgerrit | Merged openstack/bifrost: Updated from global requirements https://review.openstack.org/339878 | 12:24 |
| *** trown|outtypewww is now known as trown | 12:29 | |
| lucasagomes | dtantsur, btw: https://bugs.launchpad.net/ironic/+bug/1603411 | 12:30 |
| openstack | Launchpad bug 1603411 in Ironic "[RFE] Wipe disks metadata cleaning step" [Wishlist,New] - Assigned to Lucas Alvares Gomes (lucasagomes) | 12:30 |
| jroll | dtantsur: relevant to our conversation earlier, "make all services except keystone optional" https://review.openstack.org/#/c/335363/1 | 12:33 |
| jroll | :P | 12:33 |
| * jroll actually runs off now | 12:33 | |
| *** jjohnson2 has joined #openstack-ironic | 12:34 | |
| lucasagomes | someone wants to take a quick look at the soft power / nmi spec: https://review.openstack.org/#/c/186700/ ? It looks good to me | 12:37 |
| lucasagomes | it has another +2 as well, I'll approve it at the end of the day if nobody else has any other concerns about it (tl;dr: nmi is now part of the mgmt interface as we agreed before) | 12:38 |
| *** _vs_ has quit IRC | 12:38 | |
| *** _vsaienko has quit IRC | 12:38 | |
| * mgould thinks it's madness not to version the tests along with the code under test, personally | 12:40 | |
| *** ijw has joined #openstack-ironic | 12:40 | |
| TheJulia | mgould: +1 | 12:40 |
| TheJulia | but... rolling compatability check in a sense :\ | 12:40 |
| milan | mgould, at least is shouldn't be rsync mangled back and forth within grenade | 12:41 |
| mgould | milan: yep | 12:41 |
| *** _vsaienko has joined #openstack-ironic | 12:42 | |
| *** _vs_ has joined #openstack-ironic | 12:42 | |
| openstackgerrit | Merged openstack/pyghmi: Disable futile attempts to correct uncorrectable time https://review.openstack.org/316218 | 12:42 |
| milan | plus I think this should read if [[ "$UPGRADE_TEMPEST" == "True" ]] ; then ... and maybe we wouldn't need this patch: https://review.openstack.org/#/c/337372/ | 12:44 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Add "Ramdisk impact" section to spec template https://review.openstack.org/320125 | 12:44 |
| milan | dtantsur, mgould ^ | 12:44 |
| lucasagomes | devananda, ^ this was failing because new specs were merged, so I deliberated updated that | 12:44 |
| milan | * https://github.com/openstack-dev/grenade/blob/master/grenade.sh#L306 | 12:45 |
| dtantsur | milan, I think we still need it, the env is wiped after the rsync | 12:45 |
| *** ijw has quit IRC | 12:45 | |
| milan | dtantsur, yeah but I guess it's because of that line, isn't it? | 12:45 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add sanity tests for baremetal power state commands https://review.openstack.org/333361 | 12:46 |
| milan | dtantsur, funny thing: the var is checked just once :D | 12:46 |
| milan | just before upgrading tempest | 12:47 |
| dtantsur | dunno really.. I didn't dig into the code | 12:47 |
| *** jerrygb has joined #openstack-ironic | 12:48 | |
| *** jerrygb has quit IRC | 12:48 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Add a simple smoke test to be run in the grenade gate https://review.openstack.org/336532 | 12:49 |
| milan | dtantsur, just for the kick of it ^ | 12:49 |
| * milan check experimental | 12:49 | |
| milan | let's see | 12:49 |
| dtantsur | we don't need check experimental any more :) | 12:49 |
| milan | so check what then? :) | 12:50 |
| dtantsur | just wait. the job is in check queue already | 12:50 |
| milan | awesome :D | 12:50 |
| openstackgerrit | lokesh s proposed openstack/python-oneviewclient: Log a warning and still continue if the version is greater than 2.0 https://review.openstack.org/342720 | 12:51 |
| *** ccamacho has quit IRC | 12:52 | |
| *** rbudden has joined #openstack-ironic | 12:53 | |
| *** ccamacho_ has quit IRC | 12:53 | |
| openstackgerrit | Merged openstack/ironic-specs: Add "Ramdisk impact" section to spec template https://review.openstack.org/320125 | 12:57 |
| milan | dtantsur, http://logs.openstack.org/32/336532/11/check/gate-grenade-dsvm-ironic-inspector-nv/1bf6cdc/logs/grenade.sh.txt.gz#_2016-07-04_20_06_45_066 it seems indeed the case https://github.com/openstack-dev/grenade/blob/master/grenade.sh#L306; let's see how the job ends up | 13:05 |
| * milan fingers-crossed | 13:05 | |
| dtantsur | yeah, let's see | 13:06 |
| *** jpich has quit IRC | 13:06 | |
| *** mjura has quit IRC | 13:08 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add ddt package to the test-requirements.txt https://review.openstack.org/342838 | 13:09 |
| *** thiagop has joined #openstack-ironic | 13:13 | |
| thiagop | Good morning, Ironic | 13:14 |
| *** loki_ has quit IRC | 13:15 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with Node https://review.openstack.org/342060 | 13:16 |
| lucasagomes | thiagop, morning | 13:23 |
| openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration https://review.openstack.org/206144 | 13:23 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 13:24 |
| vsaienk0 | Morning lucasagomes, dtantsur, thiagop, milan, sambetts, jroll, TheJulia and all Ironic'ers! | 13:26 |
| lucasagomes | vsaienk0, hi there | 13:26 |
| TheJulia | good morning vsaienk0 | 13:26 |
| milan | morning vsaienk0 ! :) | 13:26 |
| *** ametts has joined #openstack-ironic | 13:27 | |
| mgould | morning vsaienk0 | 13:27 |
| thiagop | vsaienk0: morning! Your review on 286192 was hopely fully addressed. Please, take a look again if you have a slot. | 13:28 |
| TheJulia | speaking of 286192, it is what is on my screen right now :) | 13:28 |
| thiagop | TheJulia: hope it's way better now :) | 13:29 |
| TheJulia | :) | 13:30 |
| *** _vsaienko has quit IRC | 13:30 | |
| *** _vs_ has quit IRC | 13:30 | |
| xavierr | good morning Ironic | 13:30 |
| TheJulia | good morning xavierr | 13:30 |
| mgould | morning xavierr | 13:30 |
| xavierr | morning dtantsur lucasagomes TheJulia milan mgould :) | 13:31 |
| *** ccamacho has joined #openstack-ironic | 13:31 | |
| *** ccamacho1 has joined #openstack-ironic | 13:31 | |
| dtantsur | morning TheJulia, vsaienk0, thiagop, xavierr | 13:31 |
| TheJulia | good morning dtantsur | 13:31 |
| milan | morning xavierr thiagop ! :) | 13:31 |
| vsaienk0 | thiagop: sure I will look | 13:32 |
| lucasagomes | xavierr, TheJulia morning | 13:32 |
| * TheJulia feels like a for loop to say good morning to everyone is necessary :) | 13:32 | |
| TheJulia | good morning lucasagomes | 13:32 |
| lucasagomes | :D yeah | 13:32 |
| *** jrist has quit IRC | 13:32 | |
| *** joprovost has joined #openstack-ironic | 13:32 | |
| TheJulia | just imagine if there was a reciprical good morning script in all of our IRC clients... we could flood ourselfs out of the channel with just good morning messages bouncing around | 13:33 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 13:33 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 13:33 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 13:33 |
| TheJulia | ohhhh ahhhhh documentation! | 13:34 |
| openstackgerrit | Igor Kalnitsky proposed openstack/ironic: Fix fake.FakeBoot.prepare_ramdisk() signature https://review.openstack.org/341056 | 13:35 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add -d and -n arguments for OSC baremetal node create command https://review.openstack.org/342215 | 13:35 |
| vsaienk0 | please review: https://review.openstack.org/#/c/206144/ - ironicclient patch for Ironic/Nuetron integration | 13:36 |
| *** ccamacho2 has joined #openstack-ironic | 13:37 | |
| TheJulia | vsaienk0: will do after I'm done with the oneview driver | 13:39 |
| *** ChrisAusten has joined #openstack-ironic | 13:40 | |
| *** ccamacho2 has quit IRC | 13:41 | |
| *** ccamacho has quit IRC | 13:41 | |
| *** ccamacho1 has quit IRC | 13:41 | |
| *** ijw has joined #openstack-ironic | 13:42 | |
| TheJulia | thiagop, xavierr oneview/deploy_utils.py line 41, you may want to change the text | 13:42 |
| *** ccamacho has joined #openstack-ironic | 13:42 | |
| *** cdearborn has joined #openstack-ironic | 13:44 | |
| xavierr | TheJulia: do you mean change in favor of a more specific docstring, right? | 13:45 |
| TheJulia | yes please | 13:46 |
| TheJulia | since it doesn't really match what the method is doing :) | 13:46 |
| *** ijw has quit IRC | 13:47 | |
| xavierr | haha sure, thanks :) | 13:47 |
| TheJulia | no problem | 13:47 |
| *** sdake has joined #openstack-ironic | 13:47 | |
| thiagop | vsaienk0: you may want to hold this^ | 13:48 |
| *** adu has quit IRC | 13:49 | |
| thiagop | hold untils this is done* | 13:49 |
| TheJulia | thiagop: gerrit lets us version to version diff fairly easily :) | 13:49 |
| *** sdake_ has joined #openstack-ironic | 13:50 | |
| thiagop | or just review ignoring the docstrings :) | 13:50 |
| TheJulia | lol | 13:50 |
| *** sdake has quit IRC | 13:52 | |
| *** sdake_ has quit IRC | 13:52 | |
| openstackgerrit | Yuiko Takada Mori proposed openstack/ironic: IPMITool: add IPMISocatConsole and IPMIConsole class https://review.openstack.org/293873 | 13:54 |
| *** PollyZ has joined #openstack-ironic | 13:56 | |
| *** vishwanathj has joined #openstack-ironic | 13:56 | |
| *** rama_y has joined #openstack-ironic | 13:57 | |
| * TheJulia senses gerrit knows it is Friday | 14:01 | |
| thiagop | friday? FRIDAY! TGIF! | 14:04 |
| thiagop | sambetts: around? | 14:05 |
| mgould | TheJulia: gerrit's version-to-version diff performs horribly on rebases, IME | 14:06 |
| mgould | though that's probably git's fault | 14:06 |
| TheJulia | mgould: indeed | 14:07 |
| TheJulia | mgould: but, one can still conceptually figure out what has changed :) | 14:07 |
| * jroll returns | 14:07 | |
| *** jpich has joined #openstack-ironic | 14:09 | |
| mgould | DAE ever use darcs? It went to great lengths to hide rebases from the user, so you could apply patches individually | 14:09 |
| mgould | downside was the the underlying model was really hard to think about | 14:10 |
| *** joprovost has quit IRC | 14:11 | |
| TheJulia | I think we're going to get a gerrit restart shortly | 14:11 |
| thiagop | milan: a quick look on 342865 makes me wonder if you shouldn't be setting a default value for UPGRADE_TEMPEST | 14:12 |
| milan | mgould, w/r hate driven development ;) https://review.openstack.org/#/c/342865/ | 14:12 |
| milan | thiagop, tbh I've put it up just to see about how it goes cause imho if one does some .tox rsync magic, they maybe don't want to upgrade-destroy what they've just fabricated | 14:13 |
| milan | thiagop, maybe, yeah, default would make sense, -1 please if you like | 14:14 |
| thiagop | milan: not very fluent on grenade to review, that's why I'm asking here :) | 14:15 |
| milan | ah | 14:15 |
| thiagop | milan: just saw it on #ironic-qa (because I'm there waiting for something) | 14:15 |
| milan | thiagop, the original var didn't have a default either; actually, this was the only spot were it was used | 14:16 |
| milan | ah, OK | 14:16 |
| thiagop | curious... | 14:17 |
| TheJulia | And gerrit is getting restarted | 14:17 |
| -openstackstatus- NOTICE: Gerrit is restarting to correct memory/performance issues. | 14:18 | |
| milan | lol | 14:18 |
| *** ccamacho has quit IRC | 14:18 | |
| lucasagomes | :-/ | 14:19 |
| TheJulia | http://imgur.com/cqZ3eId <-- it's GC'ing time :( | 14:19 |
| *** caiobo has joined #openstack-ironic | 14:21 | |
| dtantsur | jroll, when you're here, do you think we can approve RFE https://bugs.launchpad.net/ironic/+bug/1603454 ? | 14:21 |
| openstack | Launchpad bug 1603454 in Ironic "[RFE] DRAC OOB inspection" [Wishlist,Confirmed] - Assigned to Imre Farkas (ifarkas) | 14:21 |
| jroll | dtantsur: go for it | 14:21 |
| dtantsur | done. ifarkas ^^^ | 14:22 |
| *** joprovost has joined #openstack-ironic | 14:23 | |
| *** links has quit IRC | 14:27 | |
| mgould | TheJulia: zomg | 14:28 |
| TheJulia | mgould: pretty much | 14:31 |
| *** ccamacho has joined #openstack-ironic | 14:31 | |
| openstackgerrit | Merged openstack/ironic: Follow-up to 317392 https://review.openstack.org/342477 | 14:33 |
| jroll | random thought: I wish follow up commit messages said what they do in the title, instead of saying it's a followup | 14:34 |
| *** daemontool has quit IRC | 14:35 | |
| thiagop | jroll: +1 | 14:35 |
| TheJulia | jroll: I could go along with that | 14:35 |
| openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration https://review.openstack.org/206144 | 14:36 |
| TheJulia | thiagop: xavierr: feedback posted, looks good, just some nits, four typos, wouldn't mind a doc string on two tests kind of stuff. Nothing major | 14:41 |
| *** rama_y has quit IRC | 14:42 | |
| *** ijw has joined #openstack-ironic | 14:43 | |
| thiagop | TheJulia: thank you! We are already looking into that. | 14:46 |
| xavierr | TheJulia: understood, thank you Julia!! :) | 14:47 |
| *** ijw has quit IRC | 14:48 | |
| *** jerrygb has joined #openstack-ironic | 14:52 | |
| TheJulia | vsaienk0: https://review.openstack.org/#/c/206144/32 reviewed, one nit and a concern with the release note. I realize that the OSC users aren't impacted by this since their client code attempts the last known version as incremented, but I feel like it might be useful for the release note to indicate the API version required. | 14:53 |
| ifarkas | dtantsur, jroll, awesome, thanks! that was super quick :-) | 14:54 |
| milan | dtantsur, so my odd workaround did have some effect: http://logs.openstack.org/32/336532/12/check/gate-grenade-dsvm-ironic-inspector-nv/7992ce5/logs/grenade.sh.txt.gz#_2016-07-15_14_08_36_910 test ran within grenade; but devstack-gate isn't happy http://logs.openstack.org/32/336532/12/check/gate-grenade-dsvm-ironic-inspector-nv/7992ce5/console.html#_2016-07-15_14_21_10_685546 | 14:55 |
| * dtantsur is subscribed to new bugs | 14:55 | |
| *** jaybeale has joined #openstack-ironic | 14:55 | |
| *** rama_y has joined #openstack-ironic | 14:56 | |
| dtantsur | milan, not sure what effect you're referring to, I still don't see inspector in http://logs.openstack.org/32/336532/12/check/gate-grenade-dsvm-ironic-inspector-nv/7992ce5/console.html#_2016-07-15_14_21_06_676667 | 14:56 |
| *** mtanino has joined #openstack-ironic | 14:56 | |
| *** amotoki has joined #openstack-ironic | 14:56 | |
| milan | dtantsur, right, so it's being reinstalled somewhere else as well :-/ but now it's failing | 14:57 |
| *** vishwanathj has quit IRC | 14:59 | |
| *** vishwanathj has joined #openstack-ironic | 15:00 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Rename classes in osc plugin unit tests https://review.openstack.org/342913 | 15:01 |
| *** rama_y has quit IRC | 15:02 | |
| openstackgerrit | Imre Farkas proposed openstack/ironic: [WIP] DRAC OOB inspection https://review.openstack.org/342915 | 15:02 |
| *** ijw has joined #openstack-ironic | 15:03 | |
| TheJulia | vsaienk0: I can't help but wonder if some of the content in https://review.openstack.org/#/c/228496/ should ultimately be split apart and merged in with https://review.openstack.org/#/c/258596/ | 15:05 |
| jroll | someone mind landing this doc warnings fixup so the next pbr release doesn't break IPA gates? :) https://review.openstack.org/#/c/339197/ | 15:05 |
| TheJulia | sure | 15:06 |
| *** jerrygb_ has joined #openstack-ironic | 15:06 | |
| TheJulia | done | 15:07 |
| jroll | thanks! | 15:07 |
| *** caiobo is now known as caiobo-afk | 15:08 | |
| *** jaybeale has quit IRC | 15:09 | |
| *** jerrygb has quit IRC | 15:09 | |
| *** jrist has joined #openstack-ironic | 15:10 | |
| *** rajinir has joined #openstack-ironic | 15:13 | |
| *** jrist has quit IRC | 15:16 | |
| *** jrist has joined #openstack-ironic | 15:17 | |
| *** ijw has quit IRC | 15:18 | |
| openstackgerrit | Bharath kumar proposed openstack/ironic: Following the hacking rule for string interpolation at logging https://review.openstack.org/342924 | 15:19 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Make OSC nmi command 'inject nmi' https://review.openstack.org/342927 | 15:26 |
| jroll | dtantsur: lucasagomes: I approved soft power / nmi, I had one nit here, wdyt ^ | 15:26 |
| lucasagomes | jroll, w00t, looking | 15:26 |
| *** rama_y has joined #openstack-ironic | 15:26 | |
| *** ametts has quit IRC | 15:27 | |
| dtantsur | jroll, I wonder if simply "nmi" would work.. otherwise "nmi inject" would be a bit more OSC-ish, but dunno. I like your proposal just as well | 15:28 |
| lucasagomes | jroll, I need to update myself about osc, lemme read a bit about it... but it sounds correct to me | 15:28 |
| *** Goneri has quit IRC | 15:28 | |
| openstackgerrit | Rajini Ram proposed openstack/ironic: Thirdparty CI link https://review.openstack.org/342336 | 15:28 |
| jroll | dtantsur: yeah, maybe, that's why I didn't just edit and approve | 15:29 |
| *** vishwanathj has quit IRC | 15:30 | |
| dtantsur | jroll, lol, if failed due to missing ramdisk impact | 15:30 |
| jroll | gdi | 15:30 |
| dtantsur | poor naohirot, it was so close :) | 15:30 |
| jroll | I'll fix it | 15:30 |
| lucasagomes | jroll, yeah it looks alright: http://docs.openstack.org/developer/python-openstackclient/commands.html#objects | 15:30 |
| *** rama_y has quit IRC | 15:30 | |
| dtantsur | jroll, then feel free to fix the command inline | 15:30 |
| *** vishwanathj has joined #openstack-ironic | 15:30 | |
| jroll | dtantsur: I'll leave it separate for the record | 15:31 |
| dtantsur | okie | 15:31 |
| lucasagomes | oh yeah the ramdisk impact :-/ we merged it today | 15:31 |
| *** bharath has quit IRC | 15:31 | |
| lucasagomes | will affect all specs in the queue | 15:31 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Enhance Driver Interface for Soft Power Off and Inject NMI https://review.openstack.org/186700 | 15:32 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Make OSC nmi command 'inject nmi' https://review.openstack.org/342927 | 15:32 |
| jroll | there we are | 15:32 |
| *** Romanenko_K has quit IRC | 15:32 | |
| dtantsur | +2 to both | 15:33 |
| lucasagomes | +2+a both too | 15:33 |
| lucasagomes | no reason to wait for another batch of reviews for adding that section | 15:33 |
| *** rama_y has joined #openstack-ironic | 15:33 | |
| jroll | thanks | 15:34 |
| *** ametts has joined #openstack-ironic | 15:35 | |
| dtantsur | soft power off is cool thing to have, I wonder if they can make it in newton | 15:35 |
| *** vishwanathj has quit IRC | 15:36 | |
| *** PollyZ has quit IRC | 15:36 | |
| *** vishwanathj has joined #openstack-ironic | 15:37 | |
| lucasagomes | dtantsur, problem will be the nova bits | 15:38 |
| *** vishwanathj has quit IRC | 15:38 | |
| lucasagomes | but if we get the ironic part merged this cycle that would be cool yeah | 15:38 |
| *** rama_y has quit IRC | 15:38 | |
| *** vishwanathj has joined #openstack-ironic | 15:38 | |
| openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration https://review.openstack.org/206144 | 15:40 |
| *** PollyZ_ has joined #openstack-ironic | 15:42 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Add a simple smoke test to be run in the grenade gate https://review.openstack.org/336532 | 15:44 |
| *** jjohnson2 has quit IRC | 15:44 | |
| *** jjohnson2 has joined #openstack-ironic | 15:44 | |
| * milan tries the other way | 15:44 | |
| *** links has joined #openstack-ironic | 15:45 | |
| *** Nisha_away has joined #openstack-ironic | 15:47 | |
| *** liliars has quit IRC | 15:50 | |
| milan | mgould, https://review.openstack.org/#/c/342865/1 seems I've broke it completely :D | 15:50 |
| dtantsur | +2, approved | 15:53 |
| dtantsur | :D | 15:53 |
| *** PollyZ__ has joined #openstack-ironic | 15:54 | |
| mgould | milan: not quite, you didn't manage to break -publish-docs | 15:55 |
| mgould | try harder next time :-) | 15:55 |
| *** vishwana_ has joined #openstack-ironic | 15:55 | |
| milan | lol | 15:55 |
| openstackgerrit | Merged openstack/ironic-specs: Enhance Driver Interface for Soft Power Off and Inject NMI https://review.openstack.org/186700 | 15:56 |
| *** PollyZ_ has quit IRC | 15:57 | |
| *** vishwanathj has quit IRC | 15:57 | |
| * milan will post some "sudo halt" as a next pull request just you know, to check the gate robustness ;) | 15:57 | |
| openstackgerrit | Merged openstack/ironic-specs: Make OSC nmi command 'inject nmi' https://review.openstack.org/342927 | 15:58 |
| * milan want's to have a tamed gate as a pet one day | 15:59 | |
| *** ijw has joined #openstack-ironic | 16:00 | |
| milan | as you know, they eat grenades and fard logs | 16:00 |
| * dtantsur lacks simple vim search in gertty | 16:00 | |
| milan | *fart | 16:00 |
| dtantsur | jroll, could you please remove your -2 from https://review.openstack.org/216730 ? | 16:01 |
| jroll | dtantsur: done | 16:01 |
| dtantsur | thnx. I've cleaned up the RFE itself | 16:03 |
| *** ifarkas has quit IRC | 16:03 | |
| *** amotoki has quit IRC | 16:04 | |
| dtantsur | oh, one more patch https://review.openstack.org/207897 I'm a bit lost which one is correct | 16:04 |
| dtantsur | jroll, ^^^ | 16:05 |
| dtantsur | jroll, and this one for ironicclient: https://review.openstack.org/247904 | 16:05 |
| *** PollyZ__ has quit IRC | 16:06 | |
| *** sdake has joined #openstack-ironic | 16:07 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 16:08 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 16:08 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add multitenancy devstack configuration examples. https://review.openstack.org/258596 | 16:08 |
| *** e0ne has quit IRC | 16:10 | |
| NobodyCam | Good Morning Ironicers | 16:10 |
| *** PollyZ_ has joined #openstack-ironic | 16:11 | |
| xavierr | hi NobodyCam, good morning | 16:11 |
| *** PollyZ_ has quit IRC | 16:11 | |
| NobodyCam | Morning xavierr | 16:11 |
| NobodyCam | :) | 16:11 |
| xavierr | reading you patch 'Update devstack section of quickstart to use agent_ipmitool' | 16:12 |
| NobodyCam | :) its just a doc update :) | 16:12 |
| dtantsur | morning NobodyCam, seems like TGIF, doesn't it? ;) | 16:12 |
| NobodyCam | suggesting folks use the ipmitool driver | 16:13 |
| NobodyCam | morning dtantsur... Yes does ... :) | 16:13 |
| xavierr | NobodyCam: yes, problems with agent\_\* or pxe_*? | 16:14 |
| *** tesseract- has quit IRC | 16:16 | |
| NobodyCam | Yea rloo pointed that out. with out the escape it thinks agent_ is a link and the the asterisk is a highlight char. thou when I escape the first ones (in agent) it works I am happy to test escaping both the agent and pxe ones | 16:17 |
| xavierr | nice, that's my comment in the patch! ;) | 16:18 |
| *** rcernin|lunch has quit IRC | 16:18 | |
| *** pcaruana has quit IRC | 16:20 | |
| *** ohamada has quit IRC | 16:22 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Update devstack section of quickstart to use agent_ipmitool https://review.openstack.org/341801 | 16:23 |
| NobodyCam | xavierr: ^^ | 16:23 |
| NobodyCam | passed flake8/pep8 locally | 16:24 |
| *** sdake has quit IRC | 16:25 | |
| *** PollyZ has joined #openstack-ironic | 16:26 | |
| xavierr | o/ | 16:28 |
| *** ametts has quit IRC | 16:29 | |
| *** cdearborn has quit IRC | 16:34 | |
| *** athomas has quit IRC | 16:35 | |
| lucasagomes | NobodyCam, morning | 16:37 |
| lucasagomes | xavierr, morning too :-) | 16:38 |
| NobodyCam | hey hey Good morning lucasagomes | 16:38 |
| xavierr | hi lucasagomes, how are you man? :) | 16:38 |
| lucasagomes | xavierr, all is well, you? | 16:39 |
| lucasagomes | almost calling a day and going for the weekend o/ | 16:39 |
| xavierr | lucasagomes: I'm good | 16:40 |
| TheJulia | vsaienk0: http://logs.openstack.org/44/206144/33/check/gate-python-ironicclient-releasenotes/3563139/console.html#_2016-07-15_15_53_52_231725 | 16:42 |
| NobodyCam | lucasagomes: any plans for the weekend? | 16:42 |
| lucasagomes | NobodyCam, friend playing at a bar today, so I'm going to check that out | 16:42 |
| *** rbudden has quit IRC | 16:43 | |
| NobodyCam | awesome have a pint for me while your there :) | 16:43 |
| lucasagomes | NobodyCam, sure thing, will have a few! | 16:43 |
| NobodyCam | hehehehe :) | 16:43 |
| *** PollyZ has quit IRC | 16:44 | |
| lucasagomes | and, I'm calling it a day | 16:44 |
| lucasagomes | talk to you all soon, enjoy the weekend! | 16:44 |
| NobodyCam | have a great weekend | 16:44 |
| *** lucasagomes is now known as lucas-afk | 16:45 | |
| *** ametts has joined #openstack-ironic | 16:45 | |
| dtantsur | oh, it's really time to go, see you all :) | 16:45 |
| rajinir | sambetts: Have a question on your thirdparty CI. Are you running the builds on devstack slave vm ? | 16:45 |
| xavierr | have a nice weekend lucas-afk and dtantsur | 16:45 |
| xavierr | NobodyCam: Verified+1 | 16:46 |
| xavierr | :) | 16:46 |
| NobodyCam | xavierr: :) w00t | 16:47 |
| NobodyCam | night dtantsur have a good weekend | 16:47 |
| devananda | morning, all | 16:48 |
| NobodyCam | good morning devananda :) | 16:48 |
| NobodyCam | happy friday | 16:48 |
| xavierr | morning devananda | 16:48 |
| *** dtantsur is now known as dtantsur|afk | 16:49 | |
| vsaienk0 | TheJulia: fixing... | 16:49 |
| TheJulia | curious.... | 16:53 |
| *** jpich has quit IRC | 16:56 | |
| * NobodyCam recalls something about curiosity and a cat :p | 16:58 | |
| * xavierr is thinking the Q release name will be Queens | 16:58 | |
| TheJulia | heh | 16:59 |
| *** trown is now known as trown|lunch | 16:59 | |
| openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration https://review.openstack.org/206144 | 17:03 |
| *** PollyZ has joined #openstack-ironic | 17:08 | |
| *** thiagop is now known as thiagop-afk | 17:10 | |
| rajinir | thiagop, sambetts, watanabe_isao: Have a question on your thirdparty CI. Are you running the builds on devstack slave vm ? | 17:11 |
| *** rama_y has joined #openstack-ironic | 17:25 | |
| JayF | Hmm. | 17:26 |
| JayF | Does anyone know what I need to do to get options to register from ironic-lib into ironic? | 17:27 |
| *** bharath has joined #openstack-ironic | 17:29 | |
| *** cdearborn has joined #openstack-ironic | 17:31 | |
| *** jaybeale has joined #openstack-ironic | 17:31 | |
| *** jjohnson2_ has joined #openstack-ironic | 17:33 | |
| JayF | aha, found it. gotta edit tools/config/ironic-config-generator.conf | 17:35 |
| *** jjohnson2 has quit IRC | 17:36 | |
| *** _vs_ has joined #openstack-ironic | 17:36 | |
| *** _vsaienko has joined #openstack-ironic | 17:36 | |
| mgould | xavierr: Q is a non-US summit, though | 17:38 |
| *** jaybeale has quit IRC | 17:38 | |
| *** harlowja has quit IRC | 17:39 | |
| NobodyCam | morning mogul :) | 17:39 |
| xavierr | mgould: hi there | 17:40 |
| xavierr | mgould: Queens - Queens Pound river in the Souther Coast region of NSW -- https://en.wikipedia.org/wiki/Queens_Pound_River | 17:40 |
| *** harlowja has joined #openstack-ironic | 17:41 | |
| xavierr | mgould: is an option for voting :) | 17:41 |
| *** Goneri has joined #openstack-ironic | 17:41 | |
| mgould | ah, thanks, I hadn't seen that | 17:41 |
| xavierr | np | 17:41 |
| *** piet has quit IRC | 17:41 | |
| * mgould got a voting email, but the link didn't work | 17:41 | |
| *** ijw has quit IRC | 17:43 | |
| openstackgerrit | Bharath kumar proposed openstack/ironic: Following the hacking rule for string interpolation at logging https://review.openstack.org/342924 | 17:48 |
| *** catintheroof has joined #openstack-ironic | 17:48 | |
| *** _vsaienko has quit IRC | 17:49 | |
| *** _vs_ has quit IRC | 17:49 | |
| *** _vsaienko has joined #openstack-ironic | 17:52 | |
| *** links has quit IRC | 17:52 | |
| *** _vs_ has joined #openstack-ironic | 17:52 | |
| *** garthb has joined #openstack-ironic | 17:53 | |
| *** ijw has joined #openstack-ironic | 17:53 | |
| * mgould looks out at the heavy rain glumly | 17:55 | |
| *** Goneri has quit IRC | 17:56 | |
| milan | mgould, http://logs.openstack.org/32/336532/13/check/gate-grenade-dsvm-ironic-inspector-nv/01e8313/console.html#_2016-07-15_17_22_07_299245 \o/ :) | 17:56 |
| *** catintheroof has quit IRC | 17:56 | |
| *** _vsaienko has quit IRC | 17:57 | |
| mgould | yay! | 17:57 |
| milan | * Depends-On: https://review.openstack.org/#/c/337372/ in the smoke test patch proves it's useful indeed | 17:57 |
| *** _vsaienko has joined #openstack-ironic | 17:57 | |
| milan | #pixiesay dtantsur|afk, https://review.openstack.org/#/c/337372/ -m rnr | 17:59 |
| PixieBoots | \m/ʕ>ᴥ<ʔ\m/: dtantsur|afk, https://review.openstack.org/#/c/337372/ | 17:59 |
| *** rcernin has joined #openstack-ironic | 17:59 | |
| mgould | and on that cheerful note, I'm going home. Have a good weekend, everyone! | 18:00 |
| *** mgould is now known as mgould|afk | 18:00 | |
| milan | :) good night mgould! | 18:00 |
| mgould|afk | good night milan! | 18:00 |
| NobodyCam | night mgould|afk | 18:00 |
| xavierr | night mgould|afk | 18:01 |
| *** garthb_ has joined #openstack-ironic | 18:03 | |
| *** garthb has quit IRC | 18:03 | |
| *** joprovost has quit IRC | 18:06 | |
| *** rbudden has joined #openstack-ironic | 18:08 | |
| *** jmccrory_away is now known as jmccrory | 18:08 | |
| *** joprovost has joined #openstack-ironic | 18:09 | |
| *** _vsaienko has quit IRC | 18:10 | |
| *** _vs_ has quit IRC | 18:11 | |
| *** _vs_ has joined #openstack-ironic | 18:11 | |
| *** _vsaienko has joined #openstack-ironic | 18:12 | |
| JayF | jroll: for something like adding the metrics, should I be putting the new configs to populate in upgrade:, or just note the new feature and that it can be enabled by config under features: | 18:13 |
| jroll | JayF: features, upgrade: is for steps to take or things to look out for when upgrading | 18:13 |
| JayF | k, so like | 18:13 |
| JayF | - With this change, ironic now emits timing metrics | 18:14 |
| JayF | for all API methods to statsd, if enabled by config. | 18:14 |
| JayF | is sufficient for a change like that? | 18:14 |
| JayF | I know you edit them at the end so trying to make sure it's easy for you ;) | 18:14 |
| jroll | JayF: maybe mention the config section(s) | 18:14 |
| JayF | added | 18:14 |
| JayF | in the [metrics] and [metrics_statsd] sections.. | 18:14 |
| jroll | cool | 18:16 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Metric chassis, driver, node, and port API calls https://review.openstack.org/301923 | 18:16 |
| *** bvandewa has joined #openstack-ironic | 18:17 | |
| *** Sukhdev has joined #openstack-ironic | 18:18 | |
| *** trown|lunch is now known as trown | 18:20 | |
| *** _vsaienko has quit IRC | 18:20 | |
| *** _vs_ has quit IRC | 18:22 | |
| *** davidlenwell has quit IRC | 18:23 | |
| *** _vsaienko has joined #openstack-ironic | 18:23 | |
| *** _vs_ has joined #openstack-ironic | 18:23 | |
| *** bharath has quit IRC | 18:28 | |
| *** bharath has joined #openstack-ironic | 18:28 | |
| *** ijw has quit IRC | 18:33 | |
| *** rbudden has quit IRC | 18:40 | |
| openstackgerrit | Bharath kumar proposed openstack/ironic: Indenting the lines https://review.openstack.org/343030 | 18:43 |
| openstackgerrit | Clif Houck proposed openstack/ironic-specs: Add spec for image caching to nodes via agent driver https://review.openstack.org/310594 | 18:43 |
| *** _vsaienko has quit IRC | 18:45 | |
| *** _vsaienko has joined #openstack-ironic | 18:47 | |
| TheJulia | looks like the namedextensionmanager changes in stevedore have broken IPA's unit tests | 18:47 |
| jroll | whaaaa | 18:47 |
| TheJulia | http://logs.openstack.org/97/339197/3/gate/gate-ironic-python-agent-python27/fdd5222/testr_results.html.gz | 18:47 |
| *** sdake has joined #openstack-ironic | 18:48 | |
| jroll | whomp | 18:48 |
| TheJulia | http://git.openstack.org/cgit/openstack/ironic-python-agent/tree/ironic_python_agent/inspector.py#n45 | 18:48 |
| jroll | yep | 18:49 |
| jroll | I've got it locally, just a moment | 18:49 |
| jroll | hrm, why did this just hit? | 18:52 |
| TheJulia | new release on the 13th | 18:53 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Update to work with latest stevedore https://review.openstack.org/343033 | 18:53 |
| jroll | yeah, today seems late | 18:53 |
| jroll | anyway, that passes tests for me | 18:53 |
| openstackgerrit | lokesh s proposed openstack/python-oneviewclient: Log a warning and still continue if the version is greater than 2.0 https://review.openstack.org/342720 | 18:54 |
| *** joprovost1 has joined #openstack-ironic | 18:57 | |
| *** alaski is now known as lascii | 18:57 | |
| TheJulia | jroll: except... that is not exactly backwards compatible so the minimum version would need to be changed :\ | 18:57 |
| jroll | TheJulia: oooo good point | 18:57 |
| * jroll wonders when this arg was added | 18:58 | |
| TheJulia | like... recently | 18:58 |
| TheJulia | I had the history up a little while ago and closed it :( | 18:58 |
| TheJulia | between 1.15 and 1.16 from the looks of it | 18:58 |
| jroll | so we need to bump the minimum then? | 18:58 |
| jroll | otherwise invalid arg | 18:58 |
| jroll | unless ** | 18:58 |
| * jroll looks | 18:59 | |
| *** joprovost has quit IRC | 18:59 | |
| *** joprovost1 is now known as joprovost | 18:59 | |
| jroll | hrm https://github.com/openstack/stevedore/commit/01b446c1fa317fd75db950bb7bcc840e6b0a2225 | 19:00 |
| TheJulia | well, minimum comes from global-reqs i think... | 19:00 |
| jroll | no that's old | 19:00 |
| jroll | TheJulia: right | 19:00 |
| TheJulia | ohh... | 19:00 |
| jroll | yeah so g-r has 1.10, which doesn't have on_missing_entrypoints_callback | 19:01 |
| * jroll bugs oslo | 19:03 | |
| TheJulia | and the exception is looking for keyerror :\ | 19:03 |
| openstackgerrit | Bharath kumar proposed openstack/ironic: Following the hacking rule for string interpolation at logging https://review.openstack.org/342924 | 19:03 |
| *** rama_y has quit IRC | 19:03 | |
| jroll | TheJulia: we could make the else: there throw a keyerror | 19:04 |
| TheJulia | it would maintain consistency, although it is also not documented on the docstring.... :\ | 19:05 |
| jroll | right, was an implicit contract :| | 19:06 |
| openstackgerrit | Clif Houck proposed openstack/ironic-specs: Add spec for image caching to nodes via agent driver https://review.openstack.org/310594 | 19:06 |
| bharath | https://review.openstack.org/#/c/343030/ , https://review.openstack.org/#/c/342924/ can someone please review these patches ? | 19:06 |
| *** ccamacho is now known as ccamacho|out | 19:08 | |
| *** d0ugal has quit IRC | 19:09 | |
| *** jerrygb_ has quit IRC | 19:10 | |
| *** piet has joined #openstack-ironic | 19:10 | |
| *** _vs_ has quit IRC | 19:13 | |
| *** _vsaienko has quit IRC | 19:13 | |
| *** _vsaienko has joined #openstack-ironic | 19:16 | |
| openstackgerrit | Bharath kumar proposed openstack/ironic: Indenting the lines https://review.openstack.org/343030 | 19:16 |
| *** _vs_ has joined #openstack-ironic | 19:17 | |
| milan | good night Ironic! | 19:17 |
| *** milan has quit IRC | 19:18 | |
| *** bharath has quit IRC | 19:19 | |
| *** e0ne has joined #openstack-ironic | 19:20 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Update to work with latest stevedore https://review.openstack.org/343033 | 19:20 |
| jroll | TheJulia: I filed a bug report with the options and such https://bugs.launchpad.net/stevedore/+bug/1603542 | 19:21 |
| openstack | Launchpad bug 1603542 in ironic-python-agent "stevedore 1.16 breaks missing inspector extension handling" [Critical,In progress] - Assigned to Jim Rollenhagen (jim-rollenhagen) | 19:21 |
| TheJulia | \o/ | 19:21 |
| jroll | I'm gonna go away for a bit, bbl | 19:22 |
| *** ijw has joined #openstack-ironic | 19:26 | |
| *** ijw has quit IRC | 19:32 | |
| *** rama_y has joined #openstack-ironic | 19:34 | |
| JayF | jroll: so, funny story. we're going to be voting on py35 without having it in the env list in our tox.ini | 19:35 |
| * JayF will push up a patch to add that to the env list | 19:35 | |
| JayF | the questino is, should it be py34 + py35 or s/py34/py35/ | 19:35 |
| JayF | ^ anyone, really | 19:35 |
| *** e0ne has quit IRC | 19:35 | |
| *** rbudden has joined #openstack-ironic | 19:38 | |
| *** e0ne has joined #openstack-ironic | 19:38 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 19:39 |
| TheJulia | i think py34+py35... i think... | 19:42 |
| JayF | yeah; that's my inclinatino as well | 19:42 |
| JayF | as soon as I finish the patch I'm working on, I'll push one up for taht | 19:42 |
| *** ijw has joined #openstack-ironic | 19:48 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Pass agent metrics config via conductor https://review.openstack.org/301919 | 19:49 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Pass agent metrics config via conductor https://review.openstack.org/301919 | 19:50 |
| *** ijw has quit IRC | 19:55 | |
| *** ametts has quit IRC | 19:56 | |
| *** _vsaienko has quit IRC | 19:58 | |
| *** _vs_ has quit IRC | 19:58 | |
| *** piet has quit IRC | 19:59 | |
| *** _vsaienko has joined #openstack-ironic | 20:01 | |
| *** _vs_ has joined #openstack-ironic | 20:02 | |
| *** piet has joined #openstack-ironic | 20:02 | |
| *** fragatin_ has quit IRC | 20:03 | |
| *** sdake has quit IRC | 20:08 | |
| *** ijw has joined #openstack-ironic | 20:10 | |
| *** _vsaienko has quit IRC | 20:10 | |
| *** ijw has quit IRC | 20:11 | |
| *** ijw has joined #openstack-ironic | 20:12 | |
| *** _vs_ has quit IRC | 20:12 | |
| *** _vsaienko has joined #openstack-ironic | 20:12 | |
| *** _vs_ has joined #openstack-ironic | 20:12 | |
| *** mbound has quit IRC | 20:18 | |
| *** joprovost has quit IRC | 20:23 | |
| thiagop-afk | vsaienk0: around? | 20:24 |
| *** joprovost has joined #openstack-ironic | 20:24 | |
| *** PollyZ has quit IRC | 20:34 | |
| *** _vsaienko has quit IRC | 20:34 | |
| *** _vs_ has quit IRC | 20:34 | |
| *** davidlenwell has joined #openstack-ironic | 20:34 | |
| *** rajinir has quit IRC | 20:35 | |
| *** _vsaienko has joined #openstack-ironic | 20:35 | |
| *** _vs_ has joined #openstack-ironic | 20:36 | |
| *** joprovost has quit IRC | 20:37 | |
| *** jjohnson2_ has quit IRC | 20:38 | |
| *** jjohnson2 has joined #openstack-ironic | 20:38 | |
| *** piet has quit IRC | 20:39 | |
| *** e0ne has quit IRC | 20:41 | |
| *** Nisha_away has quit IRC | 20:45 | |
| *** _vsaienko has quit IRC | 20:46 | |
| *** _vs_ has quit IRC | 20:49 | |
| *** PollyZ has joined #openstack-ironic | 20:49 | |
| *** _vsaienko has joined #openstack-ironic | 20:49 | |
| *** _vs_ has joined #openstack-ironic | 20:50 | |
| *** dprince has quit IRC | 20:50 | |
| *** Goneri has joined #openstack-ironic | 20:56 | |
| *** xavierr is now known as xavier-afk | 20:57 | |
| *** _vs_ has quit IRC | 20:58 | |
| *** jjohnson2 has quit IRC | 20:58 | |
| *** _vsaienko has quit IRC | 21:00 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [DEFAULT] https://review.openstack.org/309070 | 21:02 |
| *** ijw has quit IRC | 21:02 | |
| *** ijw has joined #openstack-ironic | 21:02 | |
| *** _vsaienko has joined #openstack-ironic | 21:03 | |
| *** _vs_ has joined #openstack-ironic | 21:03 | |
| *** sdake has joined #openstack-ironic | 21:03 | |
| *** Goneri has quit IRC | 21:04 | |
| *** cdearborn has quit IRC | 21:06 | |
| *** cdearborn has joined #openstack-ironic | 21:07 | |
| *** Goneri has joined #openstack-ironic | 21:08 | |
| *** rajinir has joined #openstack-ironic | 21:10 | |
| *** liliars has joined #openstack-ironic | 21:11 | |
| *** fragatina has joined #openstack-ironic | 21:13 | |
| *** trown is now known as trown|outtypewww | 21:21 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Add py35 to tox environments https://review.openstack.org/343084 | 21:23 |
| *** clenimar_ has joined #openstack-ironic | 21:29 | |
| *** piet has joined #openstack-ironic | 21:30 | |
| *** jaybeale has joined #openstack-ironic | 21:33 | |
| *** PollyZ has quit IRC | 21:37 | |
| *** ChubYann has joined #openstack-ironic | 21:41 | |
| *** PollyZ has joined #openstack-ironic | 21:42 | |
| *** _vsaienko has quit IRC | 21:42 | |
| *** _vs_ has quit IRC | 21:43 | |
| *** _vsaienko has joined #openstack-ironic | 21:48 | |
| *** _vs_ has joined #openstack-ironic | 21:48 | |
| *** Goneri has quit IRC | 21:52 | |
| *** mbound has joined #openstack-ironic | 21:57 | |
| *** Goneri has joined #openstack-ironic | 22:00 | |
| *** mbound has quit IRC | 22:02 | |
| Sukhdev | devananda : ping | 22:03 |
| Sukhdev | jroll : are you around? | 22:04 |
| Sukhdev | jroll : this is the last os the patches - see if you can approve it - https://review.openstack.org/#/c/206144/ | 22:04 |
| *** ijw has quit IRC | 22:07 | |
| *** _vs_ has quit IRC | 22:07 | |
| *** _vsaienko has quit IRC | 22:07 | |
| *** Goneri has quit IRC | 22:08 | |
| Sukhdev | Ironic cores - can any of you review and see if you can approve this one - https://review.openstack.org/#/c/206144/ - this is the last one in the list of networking patches | 22:08 |
| *** _vsaienko has joined #openstack-ironic | 22:08 | |
| *** _vs_ has joined #openstack-ironic | 22:09 | |
| jroll | Sukhdev: holy cow, yes | 22:13 |
| *** joprovost has joined #openstack-ironic | 22:13 | |
| Sukhdev | jroll : down to last patch, partner - | 22:13 |
| Sukhdev | jroll: it needs your love :-) | 22:14 |
| *** clenimar_ has quit IRC | 22:14 | |
| jroll | :) | 22:14 |
| jroll | there's still some portgroups work to do but I'm looking at this one | 22:15 |
| *** clenimar_ has joined #openstack-ironic | 22:16 | |
| *** ChrisAusten has quit IRC | 22:18 | |
| *** _vs_ has quit IRC | 22:18 | |
| *** ChrisAusten has joined #openstack-ironic | 22:19 | |
| jroll | whomp, it's missing a thing | 22:19 |
| *** sdake has quit IRC | 22:19 | |
| * jroll updates | 22:19 | |
| *** _vsaienko has quit IRC | 22:20 | |
| *** ijw has joined #openstack-ironic | 22:20 | |
| *** rbudden has quit IRC | 22:22 | |
| *** _vsaienko has joined #openstack-ironic | 22:23 | |
| *** _vs_ has joined #openstack-ironic | 22:23 | |
| *** ChrisAusten has quit IRC | 22:23 | |
| openstackgerrit | Chris Krelle proposed openstack/python-ironicclient: Add release note link for client release notes https://review.openstack.org/343103 | 22:25 |
| openstackgerrit | Jim Rollenhagen proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration https://review.openstack.org/206144 | 22:25 |
| jroll | NobodyCam: TheJulia: mind a re-review there? :) | 22:25 |
| NobodyCam | looking | 22:26 |
| jroll | also a sanity check that we do indeed allow updating network_interface >.> | 22:26 |
| TheJulia | I think we do, if I'm upgrading and want to change the network type without monkeying in the database... | 22:27 |
| TheJulia | or deleting and recreating | 22:27 |
| jroll | yeah | 22:27 |
| jroll | just seemed like an obvious thing so I'm like "am I wrong here" | 22:27 |
| *** cdearborn has quit IRC | 22:27 | |
| TheJulia | :) | 22:28 |
| *** ppiela has quit IRC | 22:29 | |
| *** ijw has quit IRC | 22:29 | |
| *** vishwana_ has quit IRC | 22:29 | |
| NobodyCam | jroll: got a quick minute to take another look at: https://review.openstack.org/#/c/341801/ | 22:30 |
| jroll | NobodyCam: +2, thanks for pushing on that :) | 22:31 |
| *** _vsaienko has quit IRC | 22:31 | |
| *** _vsaienko has joined #openstack-ironic | 22:32 | |
| *** rbudden has joined #openstack-ironic | 22:32 | |
| *** _vs_ has quit IRC | 22:32 | |
| NobodyCam | oh even a new test :) | 22:32 |
| NobodyCam | +2 before test results | 22:33 |
| *** _vs_ has joined #openstack-ironic | 22:33 | |
| *** ijw has joined #openstack-ironic | 22:35 | |
| TheJulia | Same, I can check back in on it later and give it the ye olde +a after the tests complete | 22:35 |
| jroll | no way, I've got dibs on +A here :D | 22:35 |
| TheJulia | so your just going to stare at the zuul status page? ;) | 22:36 |
| JayF | the race is won by the swift | 22:36 |
| JayF | not by "dibs" | 22:36 |
| JayF | meritocracy and all that, amirite? | 22:36 |
| jroll | hehe | 22:36 |
| TheJulia | heh | 22:36 |
| NobodyCam | lol | 22:37 |
| jroll | should I kick that multitenant patch CI on this? | 22:37 |
| jroll | I guess it didn't change in a way that would affect it | 22:37 |
| *** UForgotten has quit IRC | 22:38 | |
| TheJulia | likely a good idea regardless just to be on the safe side for visibility if there is an issue | 22:38 |
| *** UForgotten has joined #openstack-ironic | 22:39 | |
| *** sdake has joined #openstack-ironic | 22:40 | |
| jroll | oh vasyl already did :D | 22:43 |
| *** _vs_ has quit IRC | 22:43 | |
| jroll | thanks vsaienk0 | 22:43 |
| *** _vsaienko has quit IRC | 22:43 | |
| *** ijw has quit IRC | 22:43 | |
| *** rbudden has quit IRC | 22:43 | |
| *** ijw has joined #openstack-ironic | 22:44 | |
| *** sdake has quit IRC | 22:46 | |
| *** _vsaienko has joined #openstack-ironic | 22:47 | |
| *** _vs_ has joined #openstack-ironic | 22:47 | |
| *** jaybeale has quit IRC | 22:48 | |
| *** cryptarium has joined #openstack-ironic | 22:51 | |
| jroll | I'm just going to +A this now | 22:51 |
| jroll | jenkins is there for a reason :) | 22:51 |
| jroll | night y'all, thanks for making my friday :D | 22:51 |
| TheJulia | There is no jenkins, only zuul :) | 22:52 |
| jroll | er. yeah. | 22:52 |
| jroll | I like to think jenkins is the shadow CI | 22:52 |
| jroll | Sukhdev: that last one is merging, cheers :) | 22:52 |
| Sukhdev | jroll : cheers to you partner - that was a good going | 22:53 |
| Sukhdev | jroll : Monday morning bring some beer to celebrate in the meeting :-):-) | 22:53 |
| *** sdake has joined #openstack-ironic | 23:02 | |
| *** _vsaienko has quit IRC | 23:02 | |
| *** _vs_ has quit IRC | 23:02 | |
| *** _vsaienko has joined #openstack-ironic | 23:03 | |
| *** _vs_ has joined #openstack-ironic | 23:03 | |
| *** garthb_ has quit IRC | 23:10 | |
| *** clenimar_ has quit IRC | 23:11 | |
| *** UForgotten has quit IRC | 23:16 | |
| *** UForgotten has joined #openstack-ironic | 23:18 | |
| *** _vsaienko has quit IRC | 23:19 | |
| *** ijw has quit IRC | 23:19 | |
| *** _vs_ has quit IRC | 23:20 | |
| *** _vsaienko has joined #openstack-ironic | 23:22 | |
| *** _vs_ has joined #openstack-ironic | 23:22 | |
| *** sdake has quit IRC | 23:22 | |
| *** clenimar_ has joined #openstack-ironic | 23:23 | |
| *** mtanino has quit IRC | 23:25 | |
| *** fragatina has quit IRC | 23:25 | |
| *** joprovost has quit IRC | 23:26 | |
| *** rcernin has quit IRC | 23:30 | |
| *** _vsaienko has quit IRC | 23:31 | |
| *** _vs_ has quit IRC | 23:32 | |
| *** _vsaienko has joined #openstack-ironic | 23:33 | |
| *** _vs_ has joined #openstack-ironic | 23:33 | |
| *** ijw has joined #openstack-ironic | 23:35 | |
| *** PollyZ has quit IRC | 23:41 | |
| *** joprovost has joined #openstack-ironic | 23:41 | |
| *** _vsaienko has quit IRC | 23:43 | |
| *** _vs_ has quit IRC | 23:43 | |
| *** _vsaienko has joined #openstack-ironic | 23:47 | |
| *** _vs_ has joined #openstack-ironic | 23:47 | |
| *** piet has quit IRC | 23:52 | |
| *** _vsaienko has quit IRC | 23:55 | |
| *** _vs_ has quit IRC | 23:56 | |
| *** _vsaienko has joined #openstack-ironic | 23:58 | |
| *** _vs_ has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!