| *** causten has quit IRC | 00:00 | |
| *** aignatov has joined #openstack-ironic | 00:02 | |
| *** david-lyle has quit IRC | 00:05 | |
| *** jkilpatr has joined #openstack-ironic | 00:11 | |
| *** krtaylor has joined #openstack-ironic | 00:12 | |
| *** rpioso has quit IRC | 00:23 | |
| *** awiddersheim has joined #openstack-ironic | 00:23 | |
| TheJulia | rloo: posting fix in a moment | 00:26 |
|---|---|---|
| *** dsneddon_afk has quit IRC | 00:30 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Move to tooz hash ring implementation https://review.openstack.org/399028 | 00:31 |
| *** sacharya has quit IRC | 00:35 | |
| *** nicodemos has quit IRC | 00:38 | |
| *** nicodemos has joined #openstack-ironic | 00:38 | |
| *** rama_y has quit IRC | 00:41 | |
| *** rama_y_ has quit IRC | 00:41 | |
| *** hoangcx has joined #openstack-ironic | 00:50 | |
| *** hw_wutianwei has joined #openstack-ironic | 00:52 | |
| *** harlowja has joined #openstack-ironic | 00:52 | |
| jroll | TheJulia: rloo: nice, thanks for that | 00:53 |
| *** phuongnh has joined #openstack-ironic | 00:57 | |
| rloo | thx TheJulia! | 00:57 |
| *** SerenaFeng has joined #openstack-ironic | 00:58 | |
| phuongnh | morning ironic! | 00:59 |
| TheJulia | Good morning! | 00:59 |
| *** rloo has quit IRC | 00:59 | |
| phuongnh | thank TheJulia :) | 01:00 |
| *** causten has joined #openstack-ironic | 01:07 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Testing multinode job https://review.openstack.org/422149 | 01:18 |
| *** tuanluong has joined #openstack-ironic | 01:23 | |
| *** mtanino has quit IRC | 01:24 | |
| *** Syed__ has quit IRC | 01:35 | |
| *** ppiela has quit IRC | 01:37 | |
| openstackgerrit | jiang wei proposed openstack/python-ironicclient: Support i18n for baremetal node cmds https://review.openstack.org/357639 | 01:45 |
| openstackgerrit | Tovin Seven proposed openstack/python-ironicclient: Add OSProfiler support to Ironicclient https://review.openstack.org/362331 | 01:46 |
| *** sacharya has joined #openstack-ironic | 01:46 | |
| *** yonglihe has joined #openstack-ironic | 01:52 | |
| *** baoli has quit IRC | 01:55 | |
| *** rama_y has joined #openstack-ironic | 02:03 | |
| *** rama_y_ has joined #openstack-ironic | 02:03 | |
| *** gcb has joined #openstack-ironic | 02:14 | |
| *** wrouesnel has quit IRC | 02:28 | |
| *** wrouesnel has joined #openstack-ironic | 02:32 | |
| *** awidders_ has joined #openstack-ironic | 02:32 | |
| *** sacharya_ has joined #openstack-ironic | 02:33 | |
| *** links has joined #openstack-ironic | 02:33 | |
| *** izaakk_ has joined #openstack-ironic | 02:34 | |
| *** betherly_ has joined #openstack-ironic | 02:34 | |
| *** bcornec2 has joined #openstack-ironic | 02:35 | |
| *** hwoarang_ has joined #openstack-ironic | 02:36 | |
| *** mcarden_ has joined #openstack-ironic | 02:37 | |
| *** hoangcx_ has joined #openstack-ironic | 02:38 | |
| *** wrouesnel has quit IRC | 02:38 | |
| *** wrouesnel has joined #openstack-ironic | 02:39 | |
| *** mtreinish_ has joined #openstack-ironic | 02:41 | |
| *** sacharya has quit IRC | 02:41 | |
| *** hoangcx has quit IRC | 02:41 | |
| *** awiddersheim has quit IRC | 02:41 | |
| *** krtaylor has quit IRC | 02:41 | |
| *** jkilpatr has quit IRC | 02:41 | |
| *** aNuposic has quit IRC | 02:41 | |
| *** hwoarang has quit IRC | 02:41 | |
| *** bcornec1 has quit IRC | 02:41 | |
| *** rm_work has quit IRC | 02:41 | |
| *** izaakk has quit IRC | 02:41 | |
| *** mcarden has quit IRC | 02:41 | |
| *** betherly has quit IRC | 02:41 | |
| *** raghusab- has quit IRC | 02:41 | |
| *** bcornec has quit IRC | 02:41 | |
| *** mtreinish has quit IRC | 02:41 | |
| *** mtreinish_ is now known as mtreinish | 02:41 | |
| *** izaakk_ is now known as izaakk | 02:42 | |
| *** wrouesnel has quit IRC | 02:42 | |
| *** betherly_ is now known as betherly | 02:43 | |
| *** wrouesnel has joined #openstack-ironic | 02:44 | |
| *** yarkot has quit IRC | 02:44 | |
| *** wrouesnel has quit IRC | 02:45 | |
| *** wrouesnel has joined #openstack-ironic | 02:45 | |
| *** wrouesnel has quit IRC | 02:47 | |
| *** wrouesnel has joined #openstack-ironic | 02:47 | |
| *** wrouesnel has quit IRC | 02:49 | |
| *** wrouesnel has joined #openstack-ironic | 02:49 | |
| *** wrouesnel has quit IRC | 02:49 | |
| *** wrouesnel has joined #openstack-ironic | 02:49 | |
| *** wrouesnel has quit IRC | 02:50 | |
| *** yarkot has joined #openstack-ironic | 02:50 | |
| *** wrouesnel has joined #openstack-ironic | 02:50 | |
| *** jkilpatr has joined #openstack-ironic | 02:50 | |
| *** krtaylor has joined #openstack-ironic | 02:50 | |
| *** rm_work has joined #openstack-ironic | 03:00 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers https://review.openstack.org/372428 | 03:06 |
| *** wrouesnel has quit IRC | 03:07 | |
| *** wrouesnel has joined #openstack-ironic | 03:08 | |
| *** bcornec has joined #openstack-ironic | 03:12 | |
| *** wrouesnel has quit IRC | 03:15 | |
| *** wrouesnel has joined #openstack-ironic | 03:16 | |
| *** wrouesnel has quit IRC | 03:16 | |
| *** wrouesnel has joined #openstack-ironic | 03:17 | |
| *** wrouesnel has quit IRC | 03:18 | |
| *** wrouesnel has joined #openstack-ironic | 03:19 | |
| *** wrouesnel has quit IRC | 03:20 | |
| *** wrouesnel has joined #openstack-ironic | 03:21 | |
| *** wrouesnel has quit IRC | 03:22 | |
| *** ijw has quit IRC | 03:23 | |
| *** wrouesnel has joined #openstack-ironic | 03:23 | |
| *** wrouesnel has quit IRC | 03:23 | |
| *** raghusab has joined #openstack-ironic | 03:24 | |
| *** wrouesnel has joined #openstack-ironic | 03:24 | |
| *** wrouesnel has joined #openstack-ironic | 03:24 | |
| *** wrouesnel has quit IRC | 03:25 | |
| *** wrouesnel has joined #openstack-ironic | 03:26 | |
| *** wrouesnel has joined #openstack-ironic | 03:26 | |
| *** wrouesnel has quit IRC | 03:27 | |
| *** wrouesnel has joined #openstack-ironic | 03:27 | |
| *** wrouesnel has quit IRC | 03:27 | |
| *** wrouesnel has joined #openstack-ironic | 03:28 | |
| *** wrouesnel has quit IRC | 03:28 | |
| *** wrouesnel has joined #openstack-ironic | 03:28 | |
| *** vsaienko has joined #openstack-ironic | 03:32 | |
| *** vsaienko has quit IRC | 03:34 | |
| *** vikrant has joined #openstack-ironic | 03:45 | |
| *** sacharya_ has quit IRC | 03:57 | |
| *** tiendc has joined #openstack-ironic | 04:02 | |
| *** jerrygb has quit IRC | 04:10 | |
| openstackgerrit | Merged openstack/bifrost: Need install ansible in source case https://review.openstack.org/420737 | 04:10 |
| *** ijw has joined #openstack-ironic | 04:11 | |
| *** baoli has joined #openstack-ironic | 04:15 | |
| *** SerenaFeng has quit IRC | 04:18 | |
| *** absubram has joined #openstack-ironic | 04:33 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:39 | |
| *** Sukhdev has joined #openstack-ironic | 04:41 | |
| *** krtaylor has quit IRC | 04:52 | |
| *** sacharya has joined #openstack-ironic | 04:57 | |
| *** causten has quit IRC | 05:02 | |
| *** causten has joined #openstack-ironic | 05:02 | |
| *** wrouesnel has quit IRC | 05:03 | |
| *** absubram has quit IRC | 05:03 | |
| *** sacharya has quit IRC | 05:03 | |
| *** wrouesnel has joined #openstack-ironic | 05:03 | |
| *** wrouesnel has quit IRC | 05:03 | |
| *** wrouesnel has joined #openstack-ironic | 05:03 | |
| *** wrouesnel has quit IRC | 05:05 | |
| *** wrouesnel has joined #openstack-ironic | 05:05 | |
| *** wrouesnel has quit IRC | 05:06 | |
| *** wrouesnel has joined #openstack-ironic | 05:07 | |
| *** causten has quit IRC | 05:07 | |
| *** wrouesnel has quit IRC | 05:07 | |
| *** wrouesnel has joined #openstack-ironic | 05:08 | |
| *** jerrygb has joined #openstack-ironic | 05:10 | |
| *** jerrygb has quit IRC | 05:15 | |
| *** wrouesnel has quit IRC | 05:20 | |
| *** wrouesnel has joined #openstack-ironic | 05:21 | |
| *** wrouesnel has quit IRC | 05:21 | |
| *** wrouesnel has joined #openstack-ironic | 05:23 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI https://review.openstack.org/348191 | 05:24 |
| *** udesale has joined #openstack-ironic | 05:31 | |
| *** tiendc has quit IRC | 05:33 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool management driver for Inject NMI https://review.openstack.org/348721 | 05:33 |
| openstackgerrit | rajiv proposed openstack/ironic-python-agent: Enable release notes translation https://review.openstack.org/422396 | 05:34 |
| *** SerenaFeng has joined #openstack-ironic | 05:40 | |
| *** ijw has quit IRC | 05:43 | |
| *** aparnav has joined #openstack-ironic | 05:44 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: iRMC management driver for Inject NMI https://review.openstack.org/348732 | 05:48 |
| *** baoli has quit IRC | 05:54 | |
| *** wrouesnel has quit IRC | 05:58 | |
| *** sacharya has joined #openstack-ironic | 05:58 | |
| *** wrouesnel has joined #openstack-ironic | 05:59 | |
| *** wrouesnel has quit IRC | 06:00 | |
| *** wrouesnel has joined #openstack-ironic | 06:01 | |
| *** wrouesnel has quit IRC | 06:02 | |
| *** rbudden has quit IRC | 06:02 | |
| *** rama_y_ has quit IRC | 06:03 | |
| *** rama_y has quit IRC | 06:03 | |
| *** sacharya has quit IRC | 06:03 | |
| *** wrouesnel has joined #openstack-ironic | 06:04 | |
| *** wrouesnel has quit IRC | 06:18 | |
| *** wrouesnel has joined #openstack-ironic | 06:20 | |
| *** wrouesnel has quit IRC | 06:24 | |
| *** tiendc has joined #openstack-ironic | 06:26 | |
| *** wrouesnel has joined #openstack-ironic | 06:26 | |
| *** rama_y_ has joined #openstack-ironic | 06:28 | |
| *** rama_y has joined #openstack-ironic | 06:28 | |
| *** wrouesnel has quit IRC | 06:28 | |
| *** amotoki has quit IRC | 06:28 | |
| *** amotoki has joined #openstack-ironic | 06:28 | |
| *** wrouesnel has joined #openstack-ironic | 06:29 | |
| *** rama_y_ has quit IRC | 06:32 | |
| *** rama_y has quit IRC | 06:32 | |
| *** wrouesnel has quit IRC | 06:33 | |
| *** wrouesnel has joined #openstack-ironic | 06:34 | |
| *** wrouesnel has joined #openstack-ironic | 06:35 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: snmp inspection for disk size https://review.openstack.org/394858 | 06:46 |
| *** mjura has joined #openstack-ironic | 06:46 | |
| openstackgerrit | Naohiro Tamura proposed openstack/python-ironicclient: Add a new OSC command for Inject NMI https://review.openstack.org/357661 | 06:49 |
| *** vsaienko has joined #openstack-ironic | 06:53 | |
| *** vsaienko has quit IRC | 06:54 | |
| *** hoangcx_ has quit IRC | 06:56 | |
| *** jerrygb has joined #openstack-ironic | 07:00 | |
| *** jerrygb has quit IRC | 07:05 | |
| *** ChubYann has quit IRC | 07:07 | |
| *** rajinir has quit IRC | 07:16 | |
| *** hoangcx has joined #openstack-ironic | 07:22 | |
| pas-ha | morning ironicers :) | 07:22 |
| *** SerenaFeng has quit IRC | 07:23 | |
| *** ohamada has joined #openstack-ironic | 07:26 | |
| *** rbartal has joined #openstack-ironic | 07:48 | |
| *** e0ne has joined #openstack-ironic | 07:54 | |
| *** sacharya has joined #openstack-ironic | 08:00 | |
| *** hoangcx_ has joined #openstack-ironic | 08:01 | |
| *** hoangcx has quit IRC | 08:03 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Validate the generated swift temp url https://review.openstack.org/370875 | 08:06 |
| *** sacharya has quit IRC | 08:06 | |
| *** tesseract has joined #openstack-ironic | 08:13 | |
| *** e0ne has quit IRC | 08:16 | |
| *** amoralej|off is now known as amoralej | 08:17 | |
| *** ccamacho has joined #openstack-ironic | 08:23 | |
| *** ralonsoh has joined #openstack-ironic | 08:25 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: Add `latest` to baremetal-api-version https://review.openstack.org/420659 | 08:27 |
| *** pmannidi has quit IRC | 08:29 | |
| vdrok | morning pas-ha and ironic! | 08:29 |
| *** openstackgerrit has quit IRC | 08:33 | |
| *** openstackgerrit has joined #openstack-ironic | 08:33 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Retry to connect to RPC if failed https://review.openstack.org/376462 | 08:33 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Retry to connect to RPC if failed https://review.openstack.org/376462 | 08:40 |
| tuanluong | morning ironic. pas-ha vdrok | 08:40 |
| *** Sukhdev has quit IRC | 08:43 | |
| galyna | morning all! :) | 08:43 |
| *** hoangcx_ is now known as hoangcx | 08:45 | |
| *** amarao has joined #openstack-ironic | 08:59 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-ironic | 09:00 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: Add `latest` to baremetal-api-version https://review.openstack.org/420659 | 09:03 |
| vdrok | morning tuanluong and galyna | 09:09 |
| openstackgerrit | paresh sao proposed openstack/proliantutils: Add support to get cpu virtualization capability https://review.openstack.org/422490 | 09:12 |
| *** ccamacho has quit IRC | 09:23 | |
| *** athomas has joined #openstack-ironic | 09:25 | |
| *** yonglihe has quit IRC | 09:27 | |
| *** e0ne has joined #openstack-ironic | 09:30 | |
| *** lucas-afk is now known as lucasagomes | 09:35 | |
| lucasagomes | morning all | 09:35 |
| *** ccamacho has joined #openstack-ironic | 09:40 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: Add supported boot modes to server capabilities https://review.openstack.org/275053 | 09:45 |
| *** derekh has joined #openstack-ironic | 09:48 | |
| *** yuikotakadamori has joined #openstack-ironic | 09:50 | |
| *** MattMan has quit IRC | 09:51 | |
| *** MattMan has joined #openstack-ironic | 09:52 | |
| *** tuanluong has quit IRC | 09:56 | |
| *** nmathew has joined #openstack-ironic | 10:00 | |
| *** hoangcx has quit IRC | 10:07 | |
| *** priteau has joined #openstack-ironic | 10:08 | |
| *** phuongnh has quit IRC | 10:17 | |
| *** sambetts|afk is now known as sambetts | 10:17 | |
| openstackgerrit | Aparna proposed openstack/proliantutils: Add HPSUM firmware update support https://review.openstack.org/422004 | 10:17 |
| sambetts | Morning all | 10:18 |
| yuikotakadamori | sambetts: morning :) | 10:19 |
| sambetts | hey yuikotakadamori long time no speak! | 10:19 |
| yuikotakadamori | sambetts: yeah, actually now I have a baby in myself, so I'm trying to go home early everyday :) | 10:20 |
| sambetts | yuikotakadamori: :O congrats!!!! | 10:21 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Find a node by multiple attributes https://review.openstack.org/421285 | 10:21 |
| yuikotakadamori | sambetts: thanks :D | 10:21 |
| lucasagomes | sambetts, yuikotakadamori good morning/afternoon | 10:24 |
| yuikotakadamori | lucasagomes: good afternoon :) | 10:25 |
| lucasagomes | yuikotakadamori, oh congrats! | 10:26 |
| lucasagomes | [all] if you have some time mind taking a look at https://review.openstack.org/#/c/374988/ ? Last bit in Ironic for being able to test UEFI in the gate, cheers! :-) | 10:26 |
| patchbot | patch 374988 - ironic - DevStack: Configure nodes/environment to boot in U... | 10:26 |
| yuikotakadamori | lucasagomes: hehe, thanks! so that I cannot go to summit? PTG? for a while :p | 10:26 |
| lucasagomes | yuikotakadamori, yeah, well the PTG is not that important compared with it :-) | 10:27 |
| *** jerrygb has joined #openstack-ironic | 10:28 | |
| yuikotakadamori | lucasagomes, exactly. please enjoy! | 10:28 |
| *** tiendc has quit IRC | 10:28 | |
| *** jerrygb has quit IRC | 10:33 | |
| vdrok | morning lucasagomes sambetts and yuikotakadamori ! :) | 10:39 |
| lucasagomes | o/ morning | 10:39 |
| yuikotakadamori | vdrok: morning :) | 10:41 |
| *** nmathew has quit IRC | 10:52 | |
| vdrok | lucasagomes: did you try to set IRONIC_BOOT_MODE=uefi in gate with that patch? | 10:52 |
| lucasagomes | vdrok, I haven't, I have to put a patch for project-config to create a nv job for it | 10:53 |
| lucasagomes | vdrok, I tested it locally only | 10:54 |
| lucasagomes | lemme create a patch for it in project-config, hang on | 10:54 |
| * lucasagomes will finish something quick first | 10:54 | |
| vdrok | lucasagomes: no, I mean just do-not-merge one, that will always set IRONIC_BOOT_MODE=uefi? or is it not possible without project-config chnage? | 10:54 |
| lucasagomes | vdrok, oh, got what you mean... I haven't tried it, but sounds like a good test | 10:55 |
| lucasagomes | vdrok, lemme do it, I haven't tried | 10:55 |
| vdrok | lucasagomes: I'd feel safer approving that if I'd see a pass there :) | 10:55 |
| lucasagomes | vdrok, sure thing, will put one up now | 10:56 |
| lucasagomes | vdrok, the non-ipxe tests will fail tho, but that expected | 10:57 |
| openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent: iscsi: Listen for connections on both IPv4 and IPv6 ports https://review.openstack.org/411819 | 10:57 |
| openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent: Agent: Listen for connections on both IPv4 and IPv6 ports https://review.openstack.org/411818 | 10:57 |
| openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent: Add IPv6 unit test for _get_route_source https://review.openstack.org/419533 | 10:57 |
| openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent: Convert _get_route_source to use the socket module https://review.openstack.org/419534 | 10:57 |
| vdrok | yup, that's fine | 10:57 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DO-NOT-MERGE: Test UEFI in gate https://review.openstack.org/422549 | 10:58 |
| lucasagomes | vdrok, ^ | 10:58 |
| *** udesale has quit IRC | 10:58 | |
| vdrok | thanks :) | 10:58 |
| *** dtantsur|afk is now known as dtantsur | 11:05 | |
| dtantsur | morning Ironic | 11:05 |
| *** Nisha_Agarwal has quit IRC | 11:06 | |
| vdrok | morning dtantsur | 11:06 |
| *** Nisha_Agarwal has joined #openstack-ironic | 11:06 | |
| yuikotakadamori | dtantsur: morning :) | 11:06 |
| openstackgerrit | Karthik S proposed openstack/ironic-inspector-specs: Spec for retrieving NUMA node information https://review.openstack.org/396147 | 11:08 |
| dtantsur | hey vdrok, hey yuikotakadamori, long time no see :) | 11:11 |
| yuikotakadamori | dtantsur, hehe, yeah. | 11:12 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Use only Glance V2 by default (with a compatibility option) https://review.openstack.org/421994 | 11:12 |
| dtantsur | yuikotakadamori, how are you doing? :) | 11:13 |
| *** zhugaoxiao has joined #openstack-ironic | 11:14 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [DNM] Try disabling Glance V1 https://review.openstack.org/422557 | 11:15 |
| yuikotakadamori | dtantsur, I'm busy for downstream work, and now I have a baby in myself :) | 11:18 |
| dtantsur | oh, congratulations! (for the latter, not for the former :) | 11:19 |
| yuikotakadamori | dtantsur, hehe, I see. My baby will come on April :D | 11:19 |
| yuikotakadamori | (come? will be born??) | 11:20 |
| dtantsur | (both sound fine to me personally) cool, spring seems like a good time | 11:20 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Make _send_sensors_data concurrent https://review.openstack.org/407429 | 11:20 |
| yuikotakadamori | dtantsur, yeah, I and my family are looking forward to meet him! | 11:22 |
| *** formica has joined #openstack-ironic | 11:22 | |
| yuikotakadamori | I will go home, see you ironickers o/ | 11:25 |
| *** yuikotakadamori has quit IRC | 11:25 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Make _send_sensors_data concurrent https://review.openstack.org/407429 | 11:25 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Do not use user token in neutron client https://review.openstack.org/422561 | 11:28 |
| *** athomas has quit IRC | 11:28 | |
| *** zhugaoxiao has quit IRC | 11:29 | |
| *** zhugaoxiao has joined #openstack-ironic | 11:29 | |
| *** athomas has joined #openstack-ironic | 11:35 | |
| openstackgerrit | Aparna proposed openstack/ironic: Add `get_clean_step` to IloManagement interface https://review.openstack.org/422572 | 11:47 |
| *** jkilpatr has quit IRC | 11:47 | |
| lucasagomes | vdrok, wholediskimages in gate + UEFI seems to be failing as well because the bootloader might not be UEFI compat: http://logs.openstack.org/49/422549/1/check/gate-tempest-dsvm-ironic-ipa-wholedisk-pxe_ipmitool-tinyipa-ubuntu-xenial/6ed3130/logs/ironic-bm-logs/node-0_console_19-01-2017-11:44:39.txt.gz | 11:52 |
| lucasagomes | the chainboot fails | 11:52 |
| lucasagomes | partition images seems grand tho | 11:52 |
| lucasagomes | vdrok, great idea to have this patch up to catch these use cases | 11:52 |
| openstackgerrit | Aparna proposed openstack/proliantutils: Support to get hardware raid support https://review.openstack.org/422575 | 11:52 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Refactor active conductor check https://review.openstack.org/421465 | 11:53 |
| vdrok | lucasagomes: hmm, I thought we have the plan to ditch chainloading for whole-disk images? | 11:53 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: API changes for driver-composition https://review.openstack.org/417970 | 11:53 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Enable manual-management hardware type in devstack https://review.openstack.org/417920 | 11:53 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add hardware types to the hash ring https://review.openstack.org/419065 | 11:53 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Register/unregister hardware interfaces for conductors https://review.openstack.org/412631 | 11:53 |
| jroll | y'all are brutal on these reviews, I think this is finally done though ^ | 11:53 |
| jroll | morning. | 11:53 |
| vdrok | morning jroll :) | 11:53 |
| lucasagomes | vdrok, we had a check before that prevented it | 11:53 |
| lucasagomes | vdrok, but I removed, cause, it's not that we can't chainload to wholedisk image | 11:54 |
| lucasagomes | it just that the images should support it | 11:54 |
| lucasagomes | e.g have a efi compat bootloader installed in it | 11:54 |
| lucasagomes | but apparently cirros doesn't | 11:54 |
| lucasagomes | jroll, morning | 11:54 |
| dtantsur | morning jroll | 11:54 |
| dtantsur | heh, I remember the patch with loading hardware types did take a few cycles to merge as well :) | 11:55 |
| vdrok | lucasagomes: so you think we can take care of whole disk images later? If so, please add some comment to that devstack change. | 11:55 |
| lucasagomes | vdrok, yeah will do. I will just wait the whole jobs to run | 11:56 |
| lucasagomes | and return | 11:56 |
| vdrok | sure | 11:56 |
| lucasagomes | vdrok, but yeah for now we can enable UEFI test on partition images | 11:56 |
| jroll | dtantsur: so I just have properties/passthru to do on the API, and then a TODO to allow loading zero classic drivers and checking instead if a classic OR dynamic is loaded. then I guess we just need an ipmi hardware type and some CI? | 11:56 |
| lucasagomes | which is fine cause image generation is outside the real of ironic | 11:56 |
| lucasagomes | vdrok, we will need to change the image to test local boot anyway | 11:57 |
| lucasagomes | cirros also doesn't have grub2 installed in it so we can test in gate :-( | 11:57 |
| lucasagomes | loads of work ahead o/ | 11:57 |
| aparnav | hey jroll | 11:57 |
| dtantsur | jroll, yep. we can proceed with IPMI/CI as soon as we get the first of the patches (registration) landed, I think | 11:58 |
| jroll | dtantsur: thinking we can just do that CI on the multitenant job, that'll test hardware types, it'll test non-default network interface, and we have multiple nodes so could test mixing drivers/hardware types | 11:58 |
| openstackgerrit | Karthik S proposed openstack/ironic-inspector-specs: Spec for retrieving NUMA node information https://review.openstack.org/396147 | 11:59 |
| jroll | hi aparnav \o | 11:59 |
| dtantsur | jroll, cool idea | 11:59 |
| jroll | :) | 12:00 |
| aparnav | jroll, related to the rfe https://bugs.launchpad.net/ironic/+bug/1648448, I submitted a spec. Based on the review comments I have abondoned the spec and updated the rfe description | 12:00 |
| openstack | Launchpad bug 1648448 in Ironic "[RFE]HPSUM firmware update as cleaning step for iLO drivers" [Wishlist,In progress] - Assigned to Aparna (aparnavtce) | 12:00 |
| aparnav | jroll, Can you confirm still there a need for the spec or not? | 12:00 |
| vdrok | dtantsur: couple of comments on glance patch | 12:01 |
| dtantsur | thnx | 12:02 |
| *** tiendc has joined #openstack-ironic | 12:04 | |
| *** tiendc has quit IRC | 12:05 | |
| *** jkilpatr has joined #openstack-ironic | 12:05 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Retry to connect to RPC if failed https://review.openstack.org/376462 | 12:06 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Retry to connect to RPC if failed https://review.openstack.org/376462 | 12:07 |
| jroll | aparnav: ah, nice. approved. dtantsur if you want to sanity check this: https://bugs.launchpad.net/ironic/+bug/1648448 | 12:08 |
| openstack | Launchpad bug 1648448 in Ironic "[RFE]HPSUM firmware update as cleaning step for iLO drivers" [Wishlist,In progress] - Assigned to Aparna (aparnavtce) | 12:08 |
| *** awidders_ has quit IRC | 12:08 | |
| dtantsur | lgtm | 12:09 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: Add `latest` to baremetal-api-version https://review.openstack.org/420659 | 12:09 |
| *** krtaylor has joined #openstack-ironic | 12:10 | |
| jroll | thanks | 12:10 |
| aparnav | jroll, thank you :) | 12:10 |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic: WIP/DNM: Baremetal Devstack Network Simulation Developer Doc https://review.openstack.org/422582 | 12:10 |
| jroll | aparnav: no problem, thanks for bringing it up! | 12:10 |
| lucasagomes | folks, if you have some time take a look at https://review.openstack.org/#/c/411818/ and next patch in series, it would be cool to have IPv6 support in Ironic/IPA merged in ocata | 12:11 |
| patchbot | patch 411818 - ironic-python-agent - Agent: Listen for connections on both IPv4 and IPv... | 12:11 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Do not use user token in neutron client https://review.openstack.org/422561 | 12:22 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Use only Glance V2 by default (with a compatibility option) https://review.openstack.org/421994 | 12:23 |
| dtantsur | vdrok, ^^^ | 12:23 |
| *** Nisha_Agarwal has quit IRC | 12:25 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 12:25 | |
| *** lucasagomes is now known as lucas-hungry | 12:27 | |
| *** jerrygb has joined #openstack-ironic | 12:29 | |
| *** vikrant has quit IRC | 12:29 | |
| *** aparnav has quit IRC | 12:34 | |
| *** jerrygb has quit IRC | 12:34 | |
| vsaienk0 | sambetts: I can't get why returning None there is dangerous https://review.openstack.org/#/c/364413/24/nova/virt/ironic/driver.py@1242 | 12:36 |
| patchbot | patch 364413 - nova - Support Ironic interface attach/detach in nova virt | 12:36 |
| sambetts | vsaienk0: didn't we add the function network_binding_host_id function specifically for this purpose in Newton? | 12:39 |
| *** hw_wutianwei has quit IRC | 12:39 | |
| *** hwoarang_ is now known as hwoarang | 12:40 | |
| jroll | vsaienk0: I don't see why either, maybe just ask johnthetubaguy :) | 12:40 |
| *** baoli has joined #openstack-ironic | 12:40 | |
| jroll | rather I don't see the 'could steal a port from another user' | 12:40 |
| sambetts | jroll, vsaienk0: I'm a little lost on that part too | 12:41 |
| vsaienk0 | sambetts: yeah we added network_binding_host_id to allow override binding_host_id for ironic virt driver, but can't get why it is dangerous... | 12:42 |
| sambetts | jroll, vsaienk0: binding:host_id has nothing to do with which user owns it, only which nova compute host the port should be bound too | 12:42 |
| jroll | right | 12:42 |
| * jroll wonders if binding:host_id=None is equivalent to the lack of binding:host_id in the request to neutron | 12:43 | |
| vsaienk0 | johnthetubaguy ^^ | 12:43 |
| vsaienk0 | jroll: https://github.com/openstack/neutron/blob/970e565efc8a579eff90e093f6e5cf897c574e80/neutron/plugins/ml2/plugin.py#L389 | 12:44 |
| sambetts | jroll: but that shouldn't matter | 12:44 |
| jroll | sambetts: if that line changes to have "and bind_host_id" it will :) | 12:44 |
| sambetts | all that'll do is skip populating the binding_profile which we don't need anything from it anyway right? | 12:45 |
| vsaienk0 | jroll: it shouldn't otherwise our neutron driver shouldn't work right? | 12:46 |
| jroll | I guess what I'm wondering is, if binding:host_id is not in the request, will neutron try to bind the port immediately, or will it treat it like null and not bind it | 12:46 |
| jroll | vsaienk0: well, we send None now | 12:46 |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic: WIP/DNM: Baremetal Devstack Network Simulation Developer Doc https://review.openstack.org/422582 | 12:46 |
| *** dtantsur is now known as dtantsur|lunch | 12:47 | |
| sambetts | jroll: it just leaves the port alone | 12:47 |
| jroll | sambetts: ok, that answers my question | 12:47 |
| jroll | :) | 12:47 |
| sambetts | jroll: neutron will not bind a port without a host_id | 12:47 |
| jroll | right | 12:47 |
| jroll | I now know this | 12:47 |
| sambetts | :D | 12:48 |
| jroll | that was my question :) | 12:48 |
| jroll | sambetts: I was curious if we'd suddenly break if nova decided not to send it at all | 12:48 |
| *** baoli has quit IRC | 12:48 | |
| sambetts | jroll: I'm just very confused about the stealing ports thing | 12:49 |
| jroll | sambetts: agree | 12:49 |
| jroll | john is pinged, now we play the waiting game :P | 12:50 |
| sambetts | +1 | 12:50 |
| jroll | dtantsur|lunch: do we have code for "conductor will fail to start if there is a name clash between classic and dynamic" yet? | 12:52 |
| jroll | (when you're back from lunch ofc) | 12:52 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Driver composition: add default column to new table https://review.openstack.org/422599 | 12:58 |
| jroll | dtantsur|lunch: ^ spec update, btw | 12:58 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add RPC and object version pinning https://review.openstack.org/407491 | 12:59 |
| dtantsur|lunch | jroll, (while I haven't started eating yet): 1. no, we don't; 2. thanks!; 3. I realized we might need to expose vendor_interface still :( e.g. for IPMI hardware type we can well expect a user to switch between ipmitool vendor and ipminative vendor | 12:59 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add RPC and object version pinning https://review.openstack.org/407491 | 12:59 |
| *** gcb has quit IRC | 12:59 | |
| jroll | dtantsur|lunch: booooooo for 3 | 13:00 |
| dtantsur|lunch | yep :( | 13:00 |
| * dtantsur|lunch heads towards food for real | 13:00 | |
| jroll | ok, I'll stick with the default vendor for passthru for now | 13:00 |
| *** Nisha_Agarwal has quit IRC | 13:01 | |
| jroll | and get that validation written along with the new loading validation | 13:01 |
| * jroll finds breakfast | 13:01 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 13:01 | |
| *** links has quit IRC | 13:11 | |
| *** jerrygb has joined #openstack-ironic | 13:13 | |
| *** lucas-hungry is now known as lucasagomes | 13:17 | |
| *** dtantsur|lunch is now known as dtantsur | 13:17 | |
| *** ohamada_ has joined #openstack-ironic | 13:18 | |
| *** ohamada has quit IRC | 13:18 | |
| dtantsur | EmilienM, hey! so, what's the status of the glance issue? I tried disabling glance v1 in devstack, seems to pass with my change: https://review.openstack.org/#/c/422557/ | 13:20 |
| patchbot | patch 422557 - ironic - [DNM] Try disabling Glance V1 | 13:20 |
| lucasagomes | vdrok, added a comment summarizing the UEFI situation at https://review.openstack.org/#/c/374988/ | 13:22 |
| patchbot | patch 374988 - ironic - DevStack: Configure nodes/environment to boot in U... | 13:22 |
| lucasagomes | jroll, ^ you might be interested as wel | 13:23 |
| *** trown|outtypewww is now known as trown | 13:23 | |
| lucasagomes | dtantsur, btw, we might need changes in the inspector ipxe template to be able to boot UEFI. Seems something's missing in the kernel cmdline: logs.openstack.org/49/422549/1/check/gate-tempest-dsvm-ironic-inspector-ubuntu-xenial-nv/513627c/logs/ironic-bm-logs/node-0_console.txt.gz | 13:26 |
| lucasagomes | I don't know if there's a configuration in inspector for uefi or anything, this was just a gate test with: https://review.openstack.org/#/c/422549/ | 13:27 |
| patchbot | patch 422549 - ironic - DO-NOT-MERGE: Test UEFI in gate (ABANDONED) | 13:27 |
| lucasagomes | so maybe it's fixed already | 13:27 |
| dtantsur | lucasagomes, I'm open to suggestions :) | 13:27 |
| lucasagomes | dtantsur, sure, yeah I will try it out | 13:27 |
| lucasagomes | I just saw that, so letting ya know | 13:28 |
| dtantsur | thanks! | 13:28 |
| *** nmathew has joined #openstack-ironic | 13:29 | |
| *** Nisha_Agarwal has quit IRC | 13:30 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: test uefi boot https://review.openstack.org/422623 | 13:33 |
| *** jheroux has joined #openstack-ironic | 13:33 | |
| lucasagomes | vsaienk0, https://review.openstack.org/#/c/422549/ | 13:33 |
| patchbot | patch 422549 - ironic - DO-NOT-MERGE: Test UEFI in gate (ABANDONED) | 13:33 |
| *** ccamacho is now known as ccamacho|lunch | 13:35 | |
| vsaienk0 | lucasagomes: it seems that node capabilities for uefi enabled and oneview case will be malformed ironicclient call will look like (ironic node-create .... -p capabilities=boot_mode:uefi, p capabilities=server_hardware_type_uri:$server_hardware_type_uri,enclosure_group_uri:$enclosure_group_uri,) | 13:40 |
| lucasagomes | vsaienk0, oh, /me checks | 13:40 |
| lucasagomes | vsaienk0, L1451 from https://review.openstack.org/#/c/374988/15/devstack/lib/ironic should handle it, no ? | 13:42 |
| patchbot | patch 374988 - ironic - DevStack: Configure nodes/environment to boot in U... | 13:42 |
| lucasagomes | if node_capabilities is already set just append the oneview specific options | 13:43 |
| vsaienk0 | lucasagomes: right! | 13:44 |
| *** nmathew has quit IRC | 13:46 | |
| *** lindycoder has joined #openstack-ironic | 13:48 | |
| openstackgerrit | Merged openstack/ironic: Validate the generated swift temp url https://review.openstack.org/370875 | 13:50 |
| sambetts | vsaienk0: are you in openstack-nova? | 13:52 |
| vsaienk0 | sambetts: just joined | 13:52 |
| sambetts | vsaienk0: me and john are talking about it, basically its a weird race if you have two nova deploys which were both given the same --nic port-id=<port_id> | 13:53 |
| *** causten has joined #openstack-ironic | 13:53 | |
| vsaienk0 | sambetts: it is related only for concurrent deploys? | 13:54 |
| sambetts | vsaienk0: yeah, its a real edge case | 13:55 |
| *** [1]cdearborn has joined #openstack-ironic | 13:55 | |
| vsaienk0 | sambetts: so how Nova assumes that port is already in use? when binding host id is set? | 13:58 |
| sambetts | vsaienk0: so the race is between two deploys both reaching the update binding profile call at the same time | 14:00 |
| EmilienM | dtantsur: hey, excellent. So I tried this patch in tripleo and it didn't work because we didn't build your patch in the package I think | 14:00 |
| EmilienM | dtantsur: in other words, I haven't managed to test your work yet | 14:00 |
| EmilienM | dtantsur: it's on my list for today | 14:00 |
| sambetts | vsaienk0: the issue with ours is that if it goes second it'll nuke anything that was set by the first one instead of checking if its overriding anything | 14:01 |
| vsaienk0 | sambetts: it should be solved by implementing ETAGS on Newtron side, looks like lost update problem... | 14:01 |
| *** udesale has joined #openstack-ironic | 14:01 | |
| sambetts | vsaienk0: yeah apparently thats happening the next release | 14:02 |
| vsaienk0 | sambetts: I still don't understand how Nova can avoid that without using etags | 14:02 |
| sambetts | vsaienk0: no... its a race regardless of it being None or another binding host is | 14:03 |
| sambetts | s/is/id | 14:03 |
| vsaienk0 | sambetts: but neutron allows to override binding host id, I mean if it was set to something, to update it to another value you don't need to remove it explicitly | 14:05 |
| *** dyasny has quit IRC | 14:05 | |
| *** dprince has joined #openstack-ironic | 14:05 | |
| vsaienk0 | I don't understand why if both request contain binding host id it will not lead to stolen port | 14:05 |
| *** baoli has joined #openstack-ironic | 14:06 | |
| jroll | nova attach/detach/portgroups has +Ws \o/ | 14:09 |
| jroll | great work to everyone involved in that | 14:09 |
| jroll | it's been a riiiide | 14:09 |
| * jroll earmarks vlan aware stuff for next cycle | 14:09 | |
| vdrok | whee \o/ | 14:10 |
| vsaienk0 | sambetts: so according to johns comment they somehow check in nova that port doesn't contain binding host id already | 14:10 |
| *** rloo has joined #openstack-ironic | 14:11 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector: Add plugin to process basic management LLDP TLVs https://review.openstack.org/406496 | 14:13 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add version column https://review.openstack.org/412397 | 14:14 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add RPC and object version pinning https://review.openstack.org/407491 | 14:14 |
| lucasagomes | jroll, o/ | 14:15 |
| lucasagomes | so only api-ref needed irght ? | 14:15 |
| lucasagomes | "needed" I mean, still up for review | 14:15 |
| sambetts | jroll: did we fix the check on the nova side for waiting until the node has moved into clean wait vs cleaning to say that it is torn down? | 14:15 |
| dtantsur | EmilienM, ok, thanks for the update | 14:15 |
| jroll | lucasagomes: yeah, I think so | 14:15 |
| jroll | sambetts: not that I know of | 14:15 |
| dtantsur | oh, so it's done code-wise, awesome | 14:16 |
| *** dyasny has joined #openstack-ironic | 14:18 | |
| vsaienk0 | sambetts: so looks like we have the same problem in ironic, if we attach port to node that is already used by someone, it will be stolen... | 14:18 |
| sambetts | vsaienk0: yeah we probably need to add a device_id check into our neutron driver | 14:19 |
| vsaienk0 | sambetts: we should set device_owner to baremetal:node.uuid for the port here https://github.com/openstack/ironic/blob/master/ironic/common/neutron.py#L196 | 14:19 |
| sambetts | vsaienk0: in vif_attach we need if port.device_id && device_id != instance_id raise PortAlreadyInUse() | 14:20 |
| vsaienk0 | sambetts: what about no nova case? | 14:21 |
| vsaienk0 | wo nova, instance_uuid might not be set | 14:22 |
| *** kjw3 has joined #openstack-ironic | 14:22 | |
| *** Goneri has joined #openstack-ironic | 14:24 | |
| sambetts | nova will deal with the nova case :) | 14:25 |
| jroll | why not just check if binding:host_id is set before we send it? | 14:26 |
| *** udesale has quit IRC | 14:27 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 14:28 |
| sambetts | jroll, vsaienk0: we should probably check both, device_id should match the ironic node's instance_id if we have on, and the host_id shouldn't be set because it means the port is already bound | 14:29 |
| jroll | fair enough | 14:30 |
| vsaienk0 | jroll: all this will work only when etags or something is implemented on neutron side, so we 100% sured that state of object we know wasn't changed by someone else while we updating... | 14:30 |
| jroll | yerp | 14:30 |
| vsaienk0 | damn, nova patches cant be merged bue to py35 job failure... | 14:32 |
| jroll | yep :( | 14:33 |
| jroll | that's alright though, we know they'll get in by now | 14:33 |
| dtantsur | now, we're left with soft power pending in nova, right? | 14:37 |
| jroll | dtantsur: indeed, I submitted a client release yesterday | 14:38 |
| lucasagomes | dtantsur, yeah, and bfv stuff | 14:38 |
| * jroll checks | 14:38 | |
| jroll | lucasagomes: no chance we're getting BFV in nova this cycle :P | 14:38 |
| lucasagomes | jroll, :-( yeah | 14:38 |
| jroll | dtantsur: I guess we need this but it doesn't block CI on that patch https://review.openstack.org/#/c/422287/ | 14:39 |
| patchbot | patch 422287 - requirements - Update python-ironicclient minimum to 1.10.0 | 14:39 |
| jroll | lucasagomes: no need to be sad, we didn't expect to get that done | 14:39 |
| jroll | just the plumbing in ironic | 14:39 |
| rloo | ^ yes, that was my understanding too. | 14:39 |
| rloo | hi and good morning and great work (phew) on portgroups and attach/detach! (almost there anyway) | 14:40 |
| lucasagomes | cool, yeah we can get it in early in pike hopefully | 14:40 |
| lucasagomes | rloo, morning | 14:40 |
| * jroll points at http://specs.openstack.org/openstack/ironic-specs/priorities/ocata-priorities.html#generic-boot-from-volume | 14:40 | |
| jroll | lucasagomes: rloo: if you didn't see, I got that driver composition chain fixed up, it should be good to go now | 14:40 |
| jroll | and I think I can finish the API patch today | 14:40 |
| rloo | jroll: yup, on my list to look at today. going to look at rolling upgrades stuff first though | 14:41 |
| jroll | cool | 14:41 |
| jroll | rloo: how's that looking, it's been a couple days for me | 14:41 |
| rloo | jroll: sigh. 'better'. ask me after i look at this next round... | 14:42 |
| jroll | heh. alright | 14:42 |
| lucasagomes | jroll, awesome! Will take a look | 14:43 |
| jroll | thanks! | 14:43 |
| openstackgerrit | Michael Turek proposed openstack/python-ironicclient: Add support for RESCUE and UNRESCUE provision states https://review.openstack.org/408341 | 14:44 |
| mjturek | JayF: sorry for the delay ^ didn't look into functional tests yet but I added a couple of unit tests. Also, the rescuepassword wasn't getting sent to set_provision_state. Fixed that as well | 14:45 |
| openstackgerrit | Merged openstack/ironic: Move to tooz hash ring implementation https://review.openstack.org/399028 | 14:46 |
| jroll | wooooo | 14:47 |
| *** milan has joined #openstack-ironic | 14:49 | |
| milan | morning Ironic! :) | 14:49 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 14:50 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Validate port info before assume we may use it https://review.openstack.org/415881 | 14:51 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Keystone session for swift-temp-url generation https://review.openstack.org/410764 | 14:52 |
| vdrok | morning rloo and milan ! | 14:53 |
| rloo | hi vdrok, milan! | 14:54 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Keystone session for swift-temp-url generation https://review.openstack.org/410764 | 14:54 |
| milan | morning rloo, vdrok! :) | 14:54 |
| rloo | vdrok: hope you're ok that i updated that nova patch (so that matt could +2 it) | 14:54 |
| vdrok | rloo: sure I am, thank you by the way :) | 14:54 |
| rloo | vdrok: yw. matt would have done it himself anyway :) | 14:55 |
| openstackgerrit | Michael Turek proposed openstack/python-ironicclient: Add support for RESCUE and UNRESCUE provision states https://review.openstack.org/408341 | 14:56 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Validate port info before assume we may use it https://review.openstack.org/415881 | 14:57 |
| *** rbartal has quit IRC | 14:57 | |
| jlvillal | vdrok, vsaienk0 I have been working on multitenant with grenade. | 14:58 |
| jlvillal | vdrok, vsaienk0 And ran into this issue: http://paste.openstack.org/show/595633/ | 14:58 |
| *** mtanino has joined #openstack-ironic | 14:58 | |
| jlvillal | vdrok, vsaienk0 I'm wondering if you have ideas on how we should setup the grenade code for multitenant. I can upload the full log somewhere, if desired. | 14:59 |
| *** dprince has quit IRC | 14:59 | |
| *** srobert has joined #openstack-ironic | 14:59 | |
| vsaienk0 | jlvillal: in Ocata nova deprecate attaching floating-ip via nova CLI https://review.openstack.org/#/c/410701/16/projects/60_nova/resources.sh@110 | 14:59 |
| patchbot | patch 410701 - openstack-dev/grenade - Add floating ip with process creating instance | 14:59 |
| jlvillal | vsaienk0, Hmm. Okay. But still working for other grenade job. So a little confused. Let me look at patch. | 15:00 |
| *** baoli has quit IRC | 15:00 | |
| jlvillal | vsaienk0, I will test that patch with my code. Thanks! :) | 15:01 |
| TheJulia | Good morning everyone | 15:02 |
| vdrok | jlvillal: morning! did not know anything on your question :) | 15:02 |
| vdrok | morning TheJulia | 15:02 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Testing multinode job https://review.openstack.org/422149 | 15:02 |
| jlvillal | vdrok, Me either! :( | 15:03 |
| vsaienk0 | jlvillal for newton nova it should work | 15:03 |
| dtantsur | morning rloo, jlvillal | 15:03 |
| jlvillal | vsaienk0, Thanks. I'll test it here locally in about an hour. I appreciate the help! | 15:03 |
| jlvillal | Good morning dtantsur vsaienk0 vdrok rloo jroll milan lucasagomes TheJulia and everyone else :) | 15:04 |
| milan | morning all! :D | 15:04 |
| *** causten has quit IRC | 15:05 | |
| *** udesale has joined #openstack-ironic | 15:05 | |
| *** causten has joined #openstack-ironic | 15:05 | |
| dtantsur | good early morning milan :) | 15:05 |
| jroll | morning jlvillal | 15:06 |
| dtantsur | jroll, you haven't started a hardware type for IPMI, have you? if not, I can take a look, must be quick | 15:06 |
| jroll | dtantsur: I have not, that would be great if you did | 15:06 |
| * jlvillal wonders why it is early morning for milan. jlvillal though it would be afternoon for milan | 15:06 | |
| jlvillal | s/though/thought/ | 15:06 |
| milan | jlvillal is of course right, it's just me joining the channel late ;) | 15:07 |
| jlvillal | milan, Ah :) | 15:07 |
| dtantsur | jlvillal, our regular kidding, ignore it :) we're in the same timezone | 15:07 |
| jlvillal | Still early morning for me. I'm still in bed :) | 15:07 |
| milan | lol :D | 15:07 |
| *** baoli has joined #openstack-ironic | 15:07 | |
| sambetts | jroll: Re: the ordering thing I said earlier: https://review.openstack.org/#/c/422678/ | 15:08 |
| patchbot | patch 422678 - nova - Change order of _cleanup_deploy and _unprovision i... | 15:08 |
| jroll | sambetts: thanks, added to my list | 15:08 |
| lucasagomes | jlvillal, hi there | 15:08 |
| jroll | probably would need a bug on that | 15:08 |
| sambetts | jroll: yeah, I was hoping to see if it breaks anything first ;) | 15:09 |
| jroll | sambetts: and wait for the priority stuff to land, that conflicts with all of it :P | 15:09 |
| rloo | hi dtantsur, good early morning jlvillal, hope you feel like getting out of bed :) | 15:09 |
| *** causten has quit IRC | 15:09 | |
| sambetts | jroll: very yes :-P | 15:09 |
| sambetts | jroll: I hope we can get that issue fixed though, because calling vif_detach in _cleanup_deploy only makes the problem worse:-P | 15:10 |
| jroll | sambetts: yeah, indeed | 15:10 |
| rloo | xek: if you have a change to update this today, you may get my +2: https://review.openstack.org/#/c/407491/ | 15:12 |
| patchbot | patch 407491 - ironic - Add RPC and object version pinning | 15:12 |
| *** dyasny has quit IRC | 15:14 | |
| lucasagomes | jroll, reviewed... I'm fine also approving https://review.openstack.org/#/c/412631/ I left some comments there but nothing major | 15:16 |
| patchbot | patch 412631 - ironic - Register/unregister hardware interfaces for conduc... | 15:16 |
| *** baoli has quit IRC | 15:16 | |
| TheJulia | rloo: I guess that raises the question if we're ready to begin landing that feature (and if I should remove my -2) | 15:16 |
| * jroll looks | 15:17 | |
| rloo | TheJulia: no, I think we should keep it as -2. If we don't, we need to update the config to say it isn't working yet. | 15:17 |
| rloo | TheJulia: so I'm fine leaving it as -2 for now. | 15:17 |
| TheJulia | ok | 15:17 |
| jroll | lucasagomes: I don't quite get your comment on that intersection | 15:17 |
| jroll | either that or you found a bug | 15:18 |
| rloo | TheJulia: the only problem would be rebasing, but i'm hopeful the feature will land soooooooon | 15:18 |
| jroll | or maybe I do get it | 15:18 |
| * lucasagomes looks again | 15:18 | |
| jroll | lucasagomes: https://review.openstack.org/#/c/412631/15/ironic/common/driver_factory.py@357 | 15:18 |
| patchbot | patch 412631 - ironic - Register/unregister hardware interfaces for conduc... | 15:18 |
| jroll | I think I see what you're saying now, either you're right or I missed something | 15:18 |
| jroll | no, you're right, nice | 15:18 |
| jroll | I can clean up in a follow up too | 15:18 |
| lucasagomes | jroll, well you are always adding to enabled in the same look as you are adding "name" to supported | 15:19 |
| lucasagomes | then you just intersect both | 15:19 |
| lucasagomes | yeah... | 15:19 |
| lucasagomes | jroll, cool, no worries pushing the +a now | 15:19 |
| *** rama_y has joined #openstack-ironic | 15:19 | |
| *** rama_y_ has joined #openstack-ironic | 15:19 | |
| lucasagomes | jroll, it's not a big deal anyway | 15:19 |
| jroll | lucasagomes: thanks, I'll fix it up later today | 15:19 |
| jroll | good catch | 15:19 |
| lucasagomes | jroll, no rush, if you want I can push a patch on top fixing it too | 15:19 |
| TheJulia | rloo: to save the rebasing headache, it would be ideal if we could get it in order, and then focus on the series and try and land it in rapid succession... but that just sounds like crazy talk | 15:19 |
| lucasagomes | s/look/loop | 15:20 |
| jroll | lucasagomes: either way, I need to dive into this code later today anyway | 15:20 |
| rloo | TheJulia: yes, that makes sense. Should be doable. | 15:20 |
| lucasagomes | cool | 15:20 |
| *** dyasny has joined #openstack-ironic | 15:20 | |
| *** makowals has joined #openstack-ironic | 15:21 | |
| openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support https://review.openstack.org/264257 | 15:22 |
| lucasagomes | folks if you have time, wants to take a last look at https://review.openstack.org/#/c/374988/ ? There's 2+2 already and once that's in we can enable some UEFI test in gate (for partition images + ipxe, check the comments at the end of the page) | 15:24 |
| patchbot | patch 374988 - ironic - DevStack: Configure nodes/environment to boot in U... | 15:24 |
| openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent: iscsi: Listen for connections on both IPv4 and IPv6 ports https://review.openstack.org/411819 | 15:25 |
| openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent: Agent: Listen for connections on both IPv4 and IPv6 ports https://review.openstack.org/411818 | 15:25 |
| openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent: Add IPv6 unit test for _get_route_source https://review.openstack.org/419533 | 15:25 |
| openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent: Convert _get_route_source to use the socket module https://review.openstack.org/419534 | 15:25 |
| rloo | lucasagomes: could one of those that +2, +A? (I'll try to take a look later if needed and if I get a chance) | 15:27 |
| *** dyasny has quit IRC | 15:28 | |
| *** sacharya has joined #openstack-ironic | 15:30 | |
| lucasagomes | rloo, no rush | 15:30 |
| *** baoli has joined #openstack-ironic | 15:30 | |
| *** jaosorior has joined #openstack-ironic | 15:31 | |
| *** rama_y_ has quit IRC | 15:32 | |
| *** rama_y has quit IRC | 15:32 | |
| *** causten has joined #openstack-ironic | 15:34 | |
| *** rajinir has joined #openstack-ironic | 15:35 | |
| milan | dtantsur, sambetts https://review.openstack.org/#/c/375045/ please consider reviewing, it has been up since Dec 7 and has a+2 and couple of +1 ;) | 15:39 |
| patchbot | patch 375045 - ironic-inspector-specs - Endpoint to list introspection statuses | 15:39 |
| *** ccamacho|lunch is now known as ccamacho | 15:41 | |
| *** rbudden has joined #openstack-ironic | 15:44 | |
| *** mtanino has quit IRC | 15:46 | |
| *** dprince has joined #openstack-ironic | 15:47 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Add hardware type for IPMI using ipmitool https://review.openstack.org/422708 | 15:47 |
| dtantsur | kind of WIP still ^^^ | 15:49 |
| jroll | yeah | 15:49 |
| jroll | dtantsur: if we do want to enable that by default, we should also change devstack to load ipmi instead of manual management | 15:50 |
| jroll | dtantsur: looks straightforward, though, I guess the defaults are my only question there, do we want to do that yet | 15:51 |
| *** dyasny has joined #openstack-ironic | 15:51 | |
| * jroll leaves as a comment | 15:53 | |
| *** rpioso has joined #openstack-ironic | 15:54 | |
| *** Syed__ has joined #openstack-ironic | 15:58 | |
| dtantsur | jroll, yeah, didn't update it yet. waiting for your patches to land and make merge conflict :) | 16:00 |
| jroll | dtantsur: aha, cool | 16:00 |
| dtantsur | master does not even have the devstack option to enable hardware types | 16:00 |
| *** mjura has quit IRC | 16:04 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Detailed sequence-diagram for deploy https://review.openstack.org/414512 | 16:05 |
| *** david-lyle has joined #openstack-ironic | 16:06 | |
| *** jcoufal has joined #openstack-ironic | 16:08 | |
| * swatson is back (gone 40:49:50) | 16:08 | |
| *** udesale has quit IRC | 16:11 | |
| *** rama_y has joined #openstack-ironic | 16:12 | |
| *** absubram has joined #openstack-ironic | 16:22 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Cap memory and CPU usage when running qemu-img https://review.openstack.org/422738 | 16:27 |
| openstackgerrit | Merged openstack/ironic: Register/unregister hardware interfaces for conductors https://review.openstack.org/412631 | 16:29 |
| rloo | lucasagomes: that ironic-lib patch, is it urgent? (aren't we freezing libs this week?) | 16:30 |
| lucasagomes | rloo, not sure it's urgent, "it's a nice thing to have" I believe | 16:31 |
| lucasagomes | our QE guy hit that memory problem | 16:31 |
| rloo | lucasagomes: ok. | 16:31 |
| lucasagomes | and the machines they use are quite fat | 16:31 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Cap memory and CPU usage when running qemu-img https://review.openstack.org/422738 | 16:32 |
| * lucasagomes bumped the CPU time to match nova's | 16:32 | |
| lucasagomes | rloo, actually... thinking well I think we may want to include it, I just bumped into: https://bugs.launchpad.net/ossa/+bug/1449062 | 16:36 |
| openstack | Launchpad bug 1449062 in python-oslo.concurrency (Ubuntu Wily) "[OSSA 2016-012] qemu-img calls need to be restricted by ulimit (CVE-2015-5162)" [Medium,Fix committed] | 16:36 |
| lucasagomes | jroll, ^ | 16:36 |
| lucasagomes | ironic is not in the list of affected projects but def can be exploited | 16:36 |
| *** links has joined #openstack-ironic | 16:37 | |
| jroll | O_o | 16:37 |
| rloo | lucasagomes: ok, if jroll agrees, i'll review it | 16:37 |
| lucasagomes | ack | 16:38 |
| jroll | lucasagomes: can you add ironic-lib to affected projects? | 16:38 |
| jroll | oh that's old af | 16:38 |
| jroll | never mind | 16:38 |
| lucasagomes | jroll, yeah that's old | 16:38 |
| jroll | fun | 16:38 |
| lucasagomes | but still, we don't limit the memory when invoking qemu-img | 16:38 |
| lucasagomes | I think we should | 16:38 |
| jroll | right, I agree | 16:39 |
| JayF | lucasagomes: +2 on that ironic-lib, but with a question | 16:39 |
| jroll | is this kinda like a tarbomb? | 16:39 |
| jroll | er zipbomb* | 16:39 |
| lucasagomes | jroll, apparently, you can deploy an image which will cause qemu-img to just use a lot of memory (and potently stop the whole node) | 16:40 |
| jroll | yeah, sounds like a zipbomb, heh | 16:40 |
| jroll | lucasagomes: mind hitting this one too? https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/shell/write_image.sh#L42 | 16:40 |
| JayF | jroll: -1, actually | 16:40 |
| lucasagomes | JayF, hmm we can remove the CPU limitation yeah, I think the main concern is the memory | 16:41 |
| JayF | yeah | 16:41 |
| JayF | IPA owns all the CPU | 16:41 |
| JayF | so limiting CPU seems strange in the IPA case | 16:41 |
| jroll | we use the ironic-lib stuff in ironic, though, too | 16:41 |
| jroll | so ironic-lib should use it, It hink | 16:41 |
| jroll | I think. | 16:42 |
| lucasagomes | JayF, I bumped it to 8s, which should be grand tho... I wonder how much it would impact in performance | 16:42 |
| TheJulia | It makes sense to throttle on the conductor, but not ipa | 16:42 |
| *** ijw has joined #openstack-ironic | 16:42 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Add hardware type for IPMI using ipmitool https://review.openstack.org/422708 | 16:42 |
| dtantsur | jroll, rebased on top of your patch and added devstack change ^^^ | 16:42 |
| jroll | dtantsur: awesome | 16:43 |
| jroll | lucasagomes: one more question, this ossa bug appears to only be related to qemu-img, do we need it on the convert call? | 16:43 |
| *** ohamada_ has quit IRC | 16:43 | |
| *** aNuposic has joined #openstack-ironic | 16:44 | |
| lucasagomes | jroll, it says about qemu-img info yeah. I'm playing on the safe side here cause I don't think nova (or other projects) needs to convert the images like us | 16:44 |
| lucasagomes | so I added to both | 16:44 |
| lucasagomes | but lemme investigate more | 16:44 |
| jroll | mmm, true | 16:44 |
| lucasagomes | (this was reported to me downstream today, so I still have to dig some stuff) | 16:45 |
| jroll | lucasagomes: btw, deadline for ironic-lib releases is today, so we need to get this in asap | 16:45 |
| * jroll will ping release team | 16:45 | |
| lucasagomes | jroll, ack | 16:45 |
| *** rama_y has quit IRC | 16:46 | |
| * dtantsur reminds that bug fixes can still go in, even after we branch ironic-lib | 16:46 | |
| jroll | dtantsur: I think ocata releases will be frozen for it, though, even if we backport | 16:46 |
| *** absubram has quit IRC | 16:47 | |
| jroll | so we'd be waiting until the day after final release | 16:47 |
| rloo | jroll: if that is a security issue, shouldn't we also backport to newton and ... ? | 16:48 |
| jroll | rloo: yes, but those are not affected by deadlines | 16:48 |
| *** ccamacho has quit IRC | 16:48 | |
| jroll | we'll send it allll the way back | 16:48 |
| *** rama_y has joined #openstack-ironic | 16:48 | |
| rloo | jroll: gotcha | 16:48 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: Add supported boot modes to server capabilities https://review.openstack.org/275053 | 16:48 |
| *** absubram has joined #openstack-ironic | 16:49 | |
| jroll | heh, I googled "qemu-img convert ram usage" and got lucas' bug | 16:49 |
| lucasagomes | exactly I can't find anything heh | 16:50 |
| lucasagomes | but it's a bit scary to let it out heh | 16:50 |
| jroll | yeah, weird | 16:50 |
| jroll | maybe we cap convert, then too | 16:51 |
| jroll | just to be safe | 16:51 |
| lucasagomes | I can try to email richard w jones (he works for red hat) | 16:51 |
| jroll | so I'm +2 on this but would like a release note if possible | 16:51 |
| lucasagomes | and see if we actually need it | 16:51 |
| jroll | that would be cool but let's not block on it | 16:51 |
| lucasagomes | jroll, oh true, lemme ad one | 16:51 |
| lucasagomes | jroll, the release note goes to Ironic right ? | 16:52 |
| JayF | no | 16:52 |
| lucasagomes | ironic-lib doesn't seem to have one | 16:52 |
| JayF | no notes on ironic-lib | 16:52 |
| lucasagomes | s/one/reno | 16:52 |
| jroll | lucasagomes: aw, good point | 16:52 |
| JayF | when we bump the version in ipa/ironic, we should add teh release note | 16:52 |
| jroll | ^^ | 16:52 |
| lucasagomes | JayF, cool | 16:52 |
| *** e0ne has quit IRC | 16:52 | |
| jroll | there's also one to be fixed in ipa | 16:53 |
| jroll | in write_image.sh | 16:53 |
| jroll | (I think) | 16:53 |
| lucasagomes | yeah lemme take a look at it | 16:53 |
| jroll | cool, thanks, let me know :) | 16:54 |
| lucasagomes | rloo, you mean we should have a parameter on those methods to know whether cap CPU time or not ? | 16:54 |
| rloo | lucasagomes: yup. was also thinking a similar thing for memory. | 16:54 |
| rloo | lucasagomes: i don't like hardcoded values. | 16:54 |
| jroll | is qemu-img even multithreaded? | 16:54 |
| JayF | jroll: no | 16:54 |
| *** david-lyle has quit IRC | 16:55 | |
| lucasagomes | jroll, don't think so, tho needs to confirm | 16:55 |
| jroll | then just kill the cpu throttle | 16:55 |
| lucasagomes | rloo, hmm | 16:55 |
| JayF | In my testing, I've seen qemu-img pin a single cpu | 16:55 |
| jroll | also, if we add parameters, we change the API, and it's no longer just a bug fix | 16:55 |
| rloo | lucasagomes: and with parameters, we can fix/change easier at conductor/ipa level. | 16:55 |
| JayF | that was on an older debian ramdisk in 2014 though :) | 16:55 |
| *** absubram has quit IRC | 16:55 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Validate port info before assume we may use it https://review.openstack.org/415881 | 16:56 |
| lucasagomes | rloo, ok hang on, so let's not cap CPU time... But parametize memory usage will require more patches as jroll suggested | 16:56 |
| rloo | jroll: oh. in that case, hardcode for now. and parametize later? | 16:56 |
| jroll | can we not? | 16:56 |
| lucasagomes | rloo, ack | 16:57 |
| rloo | jroll: can we not parametize? is that what you're asking? | 16:57 |
| jroll | 1Gib seems fine to me | 16:57 |
| jroll | yes | 16:57 |
| * lucasagomes do it | 16:57 | |
| JayF | imo, in a perfect world we make that configurable, how much ram to keep qemu-img to | 16:57 |
| jroll | at least for now, if someone wants it different they can do that thing | 16:57 |
| rloo | jroll: i'm fine with that. for now anyway. we can make it configurable lata. | 16:57 |
| JayF | but I'm also OK with seeing that come later and/or if requested | 16:57 |
| *** jistr is now known as jistr|afk | 16:57 | |
| vdrok | good night everyone! | 16:57 |
| rloo | night vdrok! | 16:57 |
| *** causten has quit IRC | 16:58 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Limit memory usage when running qemu-img https://review.openstack.org/422738 | 16:58 |
| lucasagomes | JayF, rloo yeah I will add a parameter later if you guys are ok with it | 16:58 |
| lucasagomes | ^ just removed the CPU limitation | 16:58 |
| * lucasagomes fix IPA as well | 16:58 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector: Add plugin to process basic management LLDP TLVs https://review.openstack.org/406496 | 16:58 |
| * jroll +2s that | 16:58 | |
| *** causten has joined #openstack-ironic | 16:59 | |
| *** absubram has joined #openstack-ironic | 16:59 | |
| jroll | ok, I should eat unless someone urgently needs something from me | 17:00 |
| *** causten_ has joined #openstack-ironic | 17:03 | |
| *** causten has quit IRC | 17:03 | |
| *** rama_y has quit IRC | 17:03 | |
| *** Marx314 has quit IRC | 17:06 | |
| *** Marx314 has joined #openstack-ironic | 17:06 | |
| *** ccamacho has joined #openstack-ironic | 17:06 | |
| *** absubram has quit IRC | 17:08 | |
| *** links has quit IRC | 17:11 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Fix DBDeadlock during clean up https://review.openstack.org/422770 | 17:13 |
| *** aarefiev is now known as aarefiev_afk | 17:14 | |
| *** david-lyle has joined #openstack-ironic | 17:18 | |
| *** david-lyle has quit IRC | 17:18 | |
| *** causten_ has quit IRC | 17:18 | |
| *** causten has joined #openstack-ironic | 17:19 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Limit memory usage when running qemu-img convert https://review.openstack.org/422775 | 17:21 |
| *** causten has quit IRC | 17:23 | |
| *** david-lyle has joined #openstack-ironic | 17:28 | |
| openstackgerrit | Anup Navare proposed openstack/ironic: Allow logical name along with UUID in port creation https://review.openstack.org/405586 | 17:30 |
| *** causten has joined #openstack-ironic | 17:32 | |
| jlvillal | vsaienk0: The patch fixed that issue. Thanks! :) | 17:35 |
| jlvillal | Of course there is now a new issue. Failure in swift upgrade. But it is progress. | 17:35 |
| vsaienk0 | jlvillal: where did you checked it? | 17:36 |
| *** ashestakov has joined #openstack-ironic | 17:37 | |
| jlvillal | vsaienk0: Locally. I take the current multitenant job reproduce.sh and convert it into a grenade job | 17:37 |
| jlvillal | vsaienk0: http://paste.openstack.org/show/595676/ | 17:37 |
| vsaienk0 | jlvillal: I see, thanks, waiting for experimental job here https://review.openstack.org/#/c/422149/ | 17:38 |
| patchbot | patch 422149 - ironic - WIP/DNM: Testing multinode job | 17:38 |
| rloo | JayF, TheJulia: what's the status/story with the nested kvm issue? did we disable it? are we still keeping track of failures? happened again: https://review.openstack.org/#/c/417920/ | 17:38 |
| patchbot | patch 417920 - ironic - Enable manual-management hardware type in devstack | 17:38 |
| jlvillal | vsaienk0: I do wonder if the experimental job is bad. I need to compare reproduce.sh and see if I made a mistake in project-config | 17:39 |
| JayF | rloo: https://review.openstack.org/#/c/421581/ needs reviews and merging. | 17:39 |
| patchbot | patch 421581 - ironic - Default ironic to not use nested KVM | 17:39 |
| JayF | rloo: afaict, folks are onboard with changing it, there was a code issue last patchset or else it would've merged | 17:39 |
| rloo | JayF: oh. we should open a bug for this? | 17:40 |
| JayF | rloo: I mean, I don't think it's a bug per se | 17:40 |
| JayF | rloo: and it can't be caught by elastic recheck | 17:40 |
| vsaienk0 | jlvillal: looks like you running resource setup on master branch locally | 17:40 |
| JayF | rloo: sorta a grey area, that's why I hit the ML :/ | 17:40 |
| rloo | JayF: well, just wondering if we will remember in the future, the issues but i guess the info is in the ML | 17:40 |
| jlvillal | vsaienk0: really? In the gate job? Or in my local job? | 17:41 |
| vsaienk0 | jlvillal in your local job | 17:41 |
| JayF | rloo: I mean, we were also warned by infra this would happen, and we tried it anyway, so not 100% sure history will prevent future issues either, lol | 17:41 |
| jlvillal | Hmmm | 17:41 |
| rloo | JayF: it won't prevent, but hopefully help folks move forward instead of doing a similar thing again. but other than that, i don't really care. | 17:42 |
| jlvillal | vsaienk0: Why do you say that? What did I do wrong? | 17:42 |
| rloo | JayF: I commented on that. TheJulia always seems to sneak in before me. I can change my vote if you want. | 17:45 |
| JayF | rloo: I'll fix it | 17:45 |
| TheJulia | rloo: I can't help but wonder if that is my super hero power.... | 17:47 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Default ironic to not use nested KVM https://review.openstack.org/421581 | 17:47 |
| JayF | rloo: TheJulia ^^ | 17:47 |
| rloo | TheJulia: I too, was wondering :) | 17:47 |
| vsaienk0 | jlvillal: the patch you've applied fixes that issue when performing upgrade from master to master | 17:48 |
| rloo | TheJulia: yours to +A if you want :) | 17:48 |
| TheJulia | Merci | 17:48 |
| rloo | TheJulia: de rien. | 17:49 |
| vsaienk0 | jlvillal: so if it works for you, I assume your base release is master :) | 17:49 |
| jlvillal | vsaienk0: Hmmm. I'll dig in and look. But I thought the /opt/stack/old/ directory are stable/newton | 17:49 |
| * jlvillal goes and double checks | 17:50 | |
| jlvillal | vsaienk0: For sure /opt/stack/old/ironic is stable/newton | 17:50 |
| vsaienk0 | jlvillal what about nova and nova client? | 17:50 |
| *** aNuposic has quit IRC | 17:51 | |
| jlvillal | vsaienk0: Looks like it. The only commits in the log say "cherry-picked" | 17:52 |
| jlvillal | vsaienk0: For nova. I don't see a nova-client. Or python-novaclient | 17:52 |
| jlvillal | I'm guessing it is pip installed | 17:52 |
| *** absubram has joined #openstack-ironic | 17:56 | |
| *** aNuposic has joined #openstack-ironic | 17:56 | |
| vsaienk0 | sambetts: once have a time could you please review https://review.openstack.org/#/c/422561 | 17:56 |
| patchbot | patch 422561 - ironic - Do not use user token in neutron client | 17:56 |
| jlvillal | vsaienk0: I copied the logs here: http://jirantis.com/openstack-logs/multitenant-grenade/logs/ if you want to look. | 17:57 |
| aNuposic | TheJulia: Hi Julia, wrt https://review.openstack.org/#/c/407639/ radosgw use swift API and hence use same [swift]username/password | 17:58 |
| patchbot | patch 407639 - ironic - Config drive support for Ceph | 17:58 |
| *** causten has quit IRC | 18:00 | |
| *** causten has joined #openstack-ironic | 18:01 | |
| aNuposic | TheJulia: config option "configdrive_use_swift" tells that is configdrive getting stored in some object store or will be stored locally in Mysql db and "configdrive_endpoint_type" tells whether swift or radosgw | 18:01 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Limit memory usage when running qemu-img convert https://review.openstack.org/422775 | 18:01 |
| *** jistr|afk is now known as jistr | 18:01 | |
| TheJulia | aNuposic: Okay, that makes a little more sense to me, regarding swift api usage. Your commit message hints at the authentication mechanism required for radosgw not being supported though? | 18:02 |
| *** derekh has quit IRC | 18:03 | |
| TheJulia | aNuposic: I honestly think we can do better regarding the general switch dedicated towards storing the config drive in the database or in an object store. | 18:03 |
| *** causten_ has joined #openstack-ironic | 18:04 | |
| *** ralonsoh has quit IRC | 18:04 | |
| *** athomas has quit IRC | 18:04 | |
| aNuposic | TheJulia: Yeah that was because so far Ironic don't directly support communication with radosgw using username/password. It always goes through keystone auth token, that's why the commit message like that | 18:04 |
| lucasagomes | vsaienk0, JayF how do I know if a test failed due that KVM bug ? | 18:05 |
| lucasagomes | cause http://logs.openstack.org/38/422738/3/check/gate-tempest-dsvm-ironic-lib-wholedisk-pxe_ipmitool-ubuntu-xenial/78565a9/logs/ironic-bm-logs/node-0_console.txt.gz is empty | 18:05 |
| JayF | lucasagomes: logs/libvirt/qemu/node-* log | 18:05 |
| lucasagomes | so I assume the machine haven't started ? | 18:05 |
| JayF | lucasagomes: will have a KVM error in it | 18:05 |
| lucasagomes | cool elmem check | 18:05 |
| aNuposic | TheJulia: so do you think there should be some different name than 'configdrive_use_swift'? | 18:05 |
| JayF | lucasagomes: yep, that's the other symptom | 18:05 |
| TheJulia | aNuposic: So does radosgw support keystone auth tokens? | 18:05 |
| *** krtaylor has quit IRC | 18:05 | |
| vsaienk0 | lucasagomes: The key log message, "KVM: entry failed, hardware error 0x0”, shows up in logs/libvirt/qemu/node-*.txt.gz. | 18:06 |
| patchbot | Error: No closing quotation | 18:06 |
| lucasagomes | JayF, http://logs.openstack.org/38/422738/3/check/gate-tempest-dsvm-ironic-lib-wholedisk-pxe_ipmitool-ubuntu-xenial/78565a9/logs/libvirt/qemu/node-0.txt.gz | 18:06 |
| *** causten has quit IRC | 18:06 | |
| JayF | lucasagomes: bingo: KVM: entry failed, hardware error 0x0 | 18:06 |
| lucasagomes | yeah seems we hit that in one of the runs for the ironic-lib patch | 18:06 |
| lucasagomes | rehecking | 18:06 |
| JayF | yeah, it happens pretty often, that's why I sent the mail list post and such | 18:06 |
| lucasagomes | JayF, yeah, thanks for looking into it | 18:06 |
| JayF | gate unreliability peeves me pretty badly, because it's a good enough system we shouldn't have any | 18:06 |
| TheJulia | Over the holiday I don't think I saw it once, but it picked up after the new year :( | 18:07 |
| aNuposic | TheJulia: Yeah i believe it does, but my patch will be useful when operator don't want Keystone to be used and can make use of simple username/password technique | 18:08 |
| JayF | aNuposic: so for standalone ironic :) | 18:08 |
| lucasagomes | JayF, ++ it's time/resource wasted every time it fails for something like that so, yeah | 18:08 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Deprecate setting IPMI credentials https://review.openstack.org/422788 | 18:08 |
| aNuposic | JayF: TheJulia: exactly | 18:09 |
| TheJulia | aNuposic: I would prefer to see us deprecate the global parameter, who knows, maybe I'd want to add "store in the http folder" option next week, which could also work for standalone usage :) | 18:09 |
| *** ppiela has joined #openstack-ironic | 18:10 | |
| aNuposic | TheJulia: It can also be used as a whole and not stand alone Ironic but by bypassing auth token | 18:11 |
| * TheJulia pokes docs.ceph.com hoping it will load eventually | 18:12 | |
| lucasagomes | folks I'm calling it a day | 18:13 |
| lucasagomes | jroll, I had to recheck the ironic-lib memory patch due that problem in KVM so, if you guys merge it later on feel free to put up the patches backporting it (if not, I will do it tomorrow morning) | 18:14 |
| lucasagomes | have a great evening all | 18:14 |
| TheJulia | goodnight lucasagomes | 18:14 |
| lucasagomes | o/ see ya | 18:14 |
| *** lucasagomes is now known as lucas-afk | 18:15 | |
| *** trown is now known as trown|lunch | 18:16 | |
| jaosorior | h | 18:17 |
| TheJulia | aNuposic: Thinking about it more, that additional context does change my perception overall. I'm going to read the keystone/radosgw integration docs at some point in the next day, that is if the site loads so I can read them, before reviewing again. :) | 18:17 |
| TheJulia | heh, just loaded | 18:17 |
| *** david-lyle has quit IRC | 18:18 | |
| aNuposic | TheJulia: Haha, I never had site loading prob.. | 18:19 |
| aNuposic | TheJulia: Sure :) | 18:19 |
| aNuposic | TheJulia: Yeah it is taking for me as well to load the site now | 18:20 |
| *** derekh has joined #openstack-ironic | 18:20 | |
| dtantsur | see you tomorrow | 18:21 |
| *** dtantsur is now known as dtantsur|afk | 18:21 | |
| *** Sukhdev has joined #openstack-ironic | 18:21 | |
| aNuposic | TheJulia: This is one more link which might to useful to go over http://docs.openstack.org/developer/ironic/deploy/radosgw.html | 18:22 |
| rloo | bye dtantsur|afk, lucas-afk | 18:22 |
| rloo | lucas-afk, JayF, etc. wrt the kvm errors, there is info about it (how to know if due to that) in our whiteboard | 18:22 |
| TheJulia | aNuposic: So there is one concern that I would have with swift api use (which... looks like given keystone integration and swift api endpoint support, I'm wondering why we even really need to do anything in a sense..), at least on a wider scale, is that it doesn't support object expiration :\ | 18:23 |
| JayF | rloo: yes, there was | 18:23 |
| JayF | rloo: line 38-53 | 18:23 |
| rloo | JayF: which reminds me and can't remember if i asked. do we still want to note down when we see the problem? | 18:23 |
| JayF | rloo: I don't think it's useful at this point. The fix is in the gate. I found multitudes of examples when digging through it by hand before sending that email. | 18:24 |
| *** ashestakov has quit IRC | 18:24 | |
| rloo | JayF: my thinking too. ok, when that patch lands, we can move that info to the other etherpad. | 18:24 |
| JayF | +1 | 18:25 |
| aNuposic | TheJulia: A minute, will read object expiration support for swift | 18:27 |
| TheJulia | aNuposic: http://docs.ceph.com/docs/giant/radosgw/swift/ | 18:29 |
| *** vsaienko has joined #openstack-ironic | 18:32 | |
| aNuposic | TheJulia: From the above link ^ seems so | 18:32 |
| aNuposic | TheJulia: http://docs.openstack.org/developer/swift/overview_expiring_objects.html says object expiration is dependent on X-delete at and X-delete after headers in PUT request | 18:33 |
| aNuposic | TheJulia: I didn't payed attention to these headers when I made a PUT request, but do not remember to see these headers | 18:34 |
| TheJulia | aNuposic: the reason it comes to mind is expiring objects are used elsewhere in for deployments/virtual media | 18:34 |
| aNuposic | TheJulia: hmm, but as of now when configdrive is getting stored in swift are we paying attention to object expiration? | 18:35 |
| *** derekh has quit IRC | 18:35 | |
| aNuposic | TheJulia: Just wondering it, is that something taken care of now | 18:36 |
| *** absubram has quit IRC | 18:36 | |
| *** dsneddon has joined #openstack-ironic | 18:41 | |
| TheJulia | aNuposic: I don't remember off hand, I just remember some things use expiration. | 18:42 |
| *** vsaienko has quit IRC | 18:43 | |
| *** vsaienko has joined #openstack-ironic | 18:44 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Allow diskless nodes introspection https://review.openstack.org/326620 | 18:47 |
| jroll | JayF: I just approved lucas-afk's ironic-lib patch, and cherry-picked it to ocata here: https://review.openstack.org/#/c/422813/ | 18:52 |
| patchbot | patch 422813 - ironic-lib (stable/ocata) - Limit memory usage when running qemu-img | 18:52 |
| JayF | +2, but we gotta make sure to wait to approve until the master one lands | 18:53 |
| milan | guys, please consider a review here: https://review.openstack.org/#/c/396147/14 especially the NUMA data structure. Thanks! Much apprec | 18:53 |
| patchbot | patch 396147 - ironic-inspector-specs - Spec for retrieving NUMA node information | 18:53 |
| milan | * appreciated :D | 18:53 |
| jroll | JayF: yep, just letting you know | 18:53 |
| aNuposic | TheJulia: Also it seems Giant is very old release | 18:53 |
| aNuposic | TheJulia: new doc https://github.com/ceph/ceph/blob/master/doc/radosgw/swift.rst says it support | 18:54 |
| TheJulia | aNuposic: Oh cool! It was the first page that came up when I googled. | 18:54 |
| aNuposic | gmmaha: ^ | 18:54 |
| gmmaha | howdy aNuposic TheJulia | 18:55 |
| gmmaha | TheJulia: was looking at the ceph sources for aNuposic and i see that they have an object expirer to look at radosgw objects and garbage collect them. | 18:55 |
| gmmaha | Would still be good to run a test on the latest system just to prove it does purge the fie | 18:56 |
| TheJulia | gmmaha: aNuposic just pointed to the more up to date documentation :) | 18:56 |
| gmmaha | file* | 18:56 |
| *** krtaylor has joined #openstack-ironic | 18:56 | |
| gmmaha | TheJulia: yup.. the file says it is supported | 18:57 |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic: Add RPC and object version pinning https://review.openstack.org/407491 | 19:00 |
| *** derekh has joined #openstack-ironic | 19:01 | |
| *** vsaienko has quit IRC | 19:02 | |
| *** trown|lunch is now known as trown | 19:05 | |
| *** ChubYann has joined #openstack-ironic | 19:20 | |
| *** baoli has quit IRC | 19:22 | |
| *** vsaienko has joined #openstack-ironic | 19:24 | |
| *** mtanino has joined #openstack-ironic | 19:25 | |
| *** absubram has joined #openstack-ironic | 19:25 | |
| *** vsaienko has quit IRC | 19:27 | |
| *** jkilpatr has quit IRC | 19:29 | |
| *** tesseract has quit IRC | 19:29 | |
| *** aNuposic has quit IRC | 19:29 | |
| *** vsaienko has joined #openstack-ironic | 19:29 | |
| openstackgerrit | Merged openstack/ironic-lib: Limit memory usage when running qemu-img https://review.openstack.org/422738 | 19:31 |
| *** dprince has quit IRC | 19:32 | |
| *** openstackgerrit has quit IRC | 19:33 | |
| *** vsaienko has quit IRC | 19:34 | |
| *** openstackgerrit has joined #openstack-ironic | 19:35 | |
| openstackgerrit | Merged openstack/ironic-specs: Driver composition: add default column to new table https://review.openstack.org/422599 | 19:35 |
| rloo | JayF: ^^, you or someone else can approve https://review.openstack.org/#/c/422813/ | 19:37 |
| patchbot | patch 422813 - ironic-lib (stable/ocata) - Limit memory usage when running qemu-img | 19:37 |
| JayF | rloo: it needs an additional +2 from dtantsur|afk or devananda | 19:37 |
| JayF | rloo: although this might be a case where it's OK anyway *shrug* | 19:37 |
| rloo | JayF: we need it in today i think? jroll can do it, he just cherry-picked it. | 19:37 |
| *** vsaienko has joined #openstack-ironic | 19:38 | |
| rloo | JayF: i give you permission to +2/+A but I don't think it is urgent yet. When we have 1 hour left, then it will be, but I don't know when the drop dead date is :) | 19:38 |
| jroll | JayF: no, stable rules say 1x +2 if a stable core did the patch? | 19:38 |
| jroll | s/?// | 19:38 |
| JayF | jroll: oh, yeah, you're right | 19:38 |
| JayF | jroll: good point, although I just landed it anyway | 19:38 |
| jroll | I also ran that by doug because weird time state, he said that's fine | 19:38 |
| jroll | lol, thanks | 19:38 |
| rloo | thx JayF :) | 19:39 |
| *** aNuposic has joined #openstack-ironic | 19:40 | |
| *** derekh has quit IRC | 19:41 | |
| EmilienM | hey folks! this patch to default Glance API to v2 is ready for review: https://review.openstack.org/#/c/421994/ - thanks! | 19:44 |
| patchbot | patch 421994 - ironic - Use only Glance V2 by default (with a compatibilit... | 19:44 |
| *** jkilpatr has joined #openstack-ironic | 19:45 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent: Reboot and Poweroff fails with coreos IPA image https://review.openstack.org/412594 | 19:45 |
| *** david-lyle has joined #openstack-ironic | 19:48 | |
| *** vsaienko has quit IRC | 19:48 | |
| aNuposic | TheJulia: Yup.. seems radosgw handles auto object expiration as I didn't see any obj that were created 2 months back. :) | 19:50 |
| aNuposic | TheJulia: do you think 'configdrive_use_swift' should be changed to something generic? I think can be kept same as we are going to use swift api | 19:51 |
| *** formica has quit IRC | 19:53 | |
| TheJulia | aNuposic: I think so yes, I think what is really needed is additional context. I need to re-review the code though because I'm not sure we necessarily have to put the credentials in form you did though, given keystoneauth does have some various options that could be used in this case. I just need to let it spin in my brain a little bit. | 19:55 |
| aNuposic | TheJulia: Ah, sure will rework later on it after your review. Thanks :) | 19:57 |
| TheJulia | aNuposic: What may be useful in the mean time is to update the commit message and documentation to explicitly call out that it is using the swift API compatibility in radosgw. I'm just really wondering if it speaks to keystone, why is there anything that actually has to be done at this point since ceph's docs list registering radosgw as the swift endpoint. | 19:59 |
| TheJulia | :) | 19:59 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: Discover Storage details https://review.openstack.org/341370 | 19:59 |
| JayF | aNuposic: also a good idea I've found is to add comments to make things clear if someone brought up a question in review that wasn't something that needed a fix | 20:00 |
| openstackgerrit | Merged openstack/ironic: Enable manual-management hardware type in devstack https://review.openstack.org/417920 | 20:00 |
| JayF | aNuposic: i.e. assume someone reading that code later would have the same question, so add a comment so they won't, if that makes sense? | 20:00 |
| TheJulia | ++ | 20:01 |
| aNuposic | JayF: yup, that would be a good. I will add comments as needed in next patches | 20:02 |
| TheJulia | aNuposic: Thanks :) | 20:03 |
| *** sambetts is now known as sambetts|afk | 20:03 | |
| sambetts|afk | night all | 20:03 |
| jroll | \o | 20:03 |
| TheJulia | goodnight sambetts|afk | 20:03 |
| aNuposic | TheJulia: yes will go ahead and update the commit message and docs | 20:03 |
| rloo | Oh Jaaaaaayyyyyyy JayF: https://review.openstack.org/#/c/421581/ | 20:04 |
| patchbot | patch 421581 - ironic - Default ironic to not use nested KVM | 20:04 |
| aNuposic | JayF: TheJulia: i should put habit of including more comments on the code :) | 20:04 |
| TheJulia | rloo: ouch :( | 20:04 |
| openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 20:05 |
| rloo | TheJulia: yeah. Am wondering if it was cuz of the one for driver comp that was just merged | 20:05 |
| JayF | dammit | 20:05 |
| JayF | :( | 20:05 |
| JayF | aslezil: ^ literally the same thing happened to my patch we're talking about in pm, someone merge conflicted me, except mine happened in the gate :( | 20:06 |
| jroll | omg, how does a 37 line refactor end up with 8 comments after three revs | 20:06 |
| jroll | 2 revs. | 20:06 |
| rloo | jroll: cuz the reviewers are awesome? or ... ? :) | 20:06 |
| *** absubram has quit IRC | 20:06 | |
| rloo | jroll: remember, it isn't the quantity, it is the quality :) | 20:07 |
| jroll | also | 20:07 |
| jroll | what suddenly broke our unit tests | 20:07 |
| jroll | 2017-01-19 18:55:30.207535 | AttributeError: 'HashRing' object has no attribute 'hosts' | 20:08 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Default ironic to not use nested KVM https://review.openstack.org/421581 | 20:08 |
| jroll | wat | 20:08 |
| *** aNuposic has quit IRC | 20:08 | |
| JayF | rloo: TheJulia ^ rebased it | 20:09 |
| rloo | JayF: thx. | 20:09 |
| *** amoralej is now known as amoralej|off | 20:09 | |
| jroll | ah I see what happened | 20:10 |
| jroll | laaaaame | 20:10 |
| jroll | unit tests are fine, ignore me :) | 20:10 |
| * jroll fixes everything | 20:10 | |
| *** kjw3 has quit IRC | 20:14 | |
| *** kjw3 has joined #openstack-ironic | 20:14 | |
| *** dprince has joined #openstack-ironic | 20:17 | |
| *** Sukhdev has quit IRC | 20:26 | |
| *** aNuposic has joined #openstack-ironic | 20:26 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-lib: Limit memory usage when running qemu-img https://review.openstack.org/422854 | 20:30 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-lib: Limit memory usage when running qemu-img https://review.openstack.org/422855 | 20:30 |
| jroll | JayF: those are for you, for newton/mitaka | 20:31 |
| openstackgerrit | Leif Madsen proposed openstack/bifrost: Make path to ironic-rootwrap dynamic https://review.openstack.org/422856 | 20:31 |
| JayF | jroll: I don't remember, for stable policy is it master-only for ordering, or n-1 | 20:31 |
| JayF | jroll: i.e. should I not land mitaka until newton lands, etc | 20:31 |
| jroll | JayF: the latter, yeah | 20:31 |
| jroll | I'm 95% sure | 20:31 |
| *** trown is now known as trown|brb | 20:33 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent: [WIP] Trying to create a package for biosdevname https://review.openstack.org/396406 | 20:33 |
| *** e0ne has joined #openstack-ironic | 20:35 | |
| *** amarao has quit IRC | 20:37 | |
| *** e0ne has quit IRC | 20:38 | |
| *** milan has quit IRC | 20:40 | |
| *** milan has joined #openstack-ironic | 20:41 | |
| jroll | rloo: hope this clears things up https://review.openstack.org/#/c/412631/15/ironic/common/driver_factory.py@357 | 20:41 |
| patchbot | patch 412631 - ironic - Register/unregister hardware interfaces for conduc... (MERGED) | 20:41 |
| *** trown|brb is now known as trown | 20:42 | |
| rloo | jroll: oh yeah. obvious when i actually look at the code. | 20:43 |
| rloo | jroll: thx! | 20:43 |
| jroll | rloo: :) | 20:44 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: API changes for driver-composition https://review.openstack.org/417970 | 20:44 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add hardware types to the hash ring https://review.openstack.org/419065 | 20:44 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clean up driver_factory.enabled_supported_interfaces https://review.openstack.org/422862 | 20:44 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Turn NOTE into docstring https://review.openstack.org/422863 | 20:44 |
| jroll | on a related note, fixed up the hash ring patch and a couple nits | 20:45 |
| jroll | where'd my refactor go, though | 20:46 |
| *** baoli has joined #openstack-ironic | 20:46 | |
| *** baoli has quit IRC | 20:47 | |
| rloo | jroll: so the driver composition stuff looks like it is in decent shape | 20:47 |
| *** baoli has joined #openstack-ironic | 20:47 | |
| jroll | urgh, I merged it into the hash ring patch by mistake | 20:48 |
| jroll | that actually looks cleaner though imo | 20:48 |
| jroll | rloo: yeah, just gotta finish the API and do CI | 20:48 |
| rloo | jroll: right, that's all :) | 20:48 |
| jroll | rloo: it's close, promise | 20:49 |
| jroll | CI should be easy | 20:49 |
| rloo | jroll: should be done by end of next week i think | 20:49 |
| jroll | yep, that's my goal | 20:49 |
| *** wrouesnel has quit IRC | 20:49 | |
| jroll | (CI is probably harder than I think) | 20:49 |
| jroll | rloo: how's rolling upgrades look? | 20:50 |
| rloo | jroll: i'm not so sure. there isn't that much code but for this one the devil is in the details | 20:50 |
| rloo | jroll: there's one patch that I'm not convinced about yet | 20:50 |
| jroll | rloo: does the code look ready, though? | 20:50 |
| jroll | ok | 20:50 |
| rloo | jroll: and then there are other patches that are work in progress, i'm not sure what those are for | 20:50 |
| * jroll looks | 20:51 | |
| rloo | jroll: and then there's the CI stuff. i thought we could discuss next week. if we can't test in gate... ?? | 20:51 |
| jroll | right, I'm worried even if we finish it we won't get CI working in time | 20:51 |
| jroll | and CI is a hard requirement for this IMO | 20:51 |
| rloo | jroll: yeah. which is why i thought next week mon? we could discuss with jlvillal and vsaienk0 and whoever. | 20:52 |
| jroll | sure, soudns good | 20:52 |
| jroll | if those two and xek are in the meeting we can figure it out | 20:52 |
| jlvillal | I'll be there | 20:52 |
| rloo | jroll: so i'm trying to get 'everything else' out of the way. ha ha. | 20:52 |
| jroll | rloo: yep. I think BFV and rescue are the only other big ones that need oomph | 20:53 |
| jroll | oh, and we need to review the nova soft power thing | 20:53 |
| *** e0ne has joined #openstack-ironic | 20:54 | |
| rloo | jroll: am looking at the soft power thing now. there are two patches in nova. i can continue with bfv this month but i don't know if i have it in me, to get into a new feature like rescue. | 20:54 |
| jroll | two for soft power? | 20:54 |
| jroll | rloo: yeah, fair | 20:54 |
| rloo | jroll: one reboot, one power off i think. | 20:54 |
| jroll | silly | 20:54 |
| rloo | jroll: am just looking at the reboot one right now. and yeah, i wondered why two but... | 20:55 |
| rloo | jroll: i have to say, i'm quite impressed with matt's reviews (in novaland). | 20:55 |
| jroll | rloo: he didn't wind up as ptl by doing nothing :P | 20:56 |
| jroll | but yes, I'm happy matt has helped us out there | 20:56 |
| rloo | jroll: :) seems like he is doing A LOT! | 20:56 |
| *** pablo|500| has quit IRC | 20:57 | |
| openstackgerrit | anusree proposed openstack/pyghmi: Remove py26 support https://review.openstack.org/422867 | 20:58 |
| *** ccamacho has quit IRC | 20:59 | |
| *** e0ne has quit IRC | 21:00 | |
| *** vsaienko has joined #openstack-ironic | 21:02 | |
| *** e0ne has joined #openstack-ironic | 21:02 | |
| *** jjohnson2 has joined #openstack-ironic | 21:04 | |
| jjohnson2 | mat128, hah, you posted the same comment as I did | 21:04 |
| mat128 | yup :) I dont think the commiter will read the message though, it seems like a mass-edit | 21:05 |
| mat128 | s/edit/commit | 21:05 |
| jjohnson2 | RHEL6/CentOS6 longevity has had a strong impact on what I'm called to support | 21:05 |
| jjohnson2 | feels like it's holding on harder than RHEL5 did | 21:06 |
| * jroll posts the same thing | 21:06 | |
| *** ijw has quit IRC | 21:06 | |
| jroll | jjohnson2: fwiw, that looks like someone doing a grep across all of openstack's git namespace and doing drive-by patches for those | 21:06 |
| jroll | oh yeah, mat said that | 21:07 |
| *** e0ne has quit IRC | 21:07 | |
| jjohnson2 | it will became hard to support RHEL6/CentOS6 even for just pyghmi | 21:08 |
| jjohnson2 | so many nice toys in newer python | 21:08 |
| *** ccamacho has joined #openstack-ironic | 21:09 | |
| *** milan has quit IRC | 21:10 | |
| *** jkilpatr has quit IRC | 21:10 | |
| *** milan has joined #openstack-ironic | 21:11 | |
| *** jkilpatr has joined #openstack-ironic | 21:12 | |
| *** absubram has joined #openstack-ironic | 21:15 | |
| TheJulia | jroll: your +1 will be required on https://review.openstack.org/#/c/422873/ to complete the clean-up of ironic-webclient | 21:15 |
| patchbot | patch 422873 - openstack-infra/project-config - Retire ironic-webclient zuul/gerrit config | 21:15 |
| jroll | TheJulia: done, thanks | 21:16 |
| TheJulia | np, sorry for not getting that taken care of sooner, even though it is a super quick change. :\ | 21:16 |
| *** vsaienko has quit IRC | 21:18 | |
| *** vsaienko has joined #openstack-ironic | 21:18 | |
| jroll | no worries | 21:18 |
| jroll | wsmanclient is still on my todo list, I know how it is :P | 21:18 |
| *** ElCoyote_ has left #openstack-ironic | 21:19 | |
| *** jkilpatr has quit IRC | 21:20 | |
| *** krtaylor has quit IRC | 21:21 | |
| *** priteau has quit IRC | 21:21 | |
| TheJulia | :) | 21:22 |
| *** ElCoyote_ has joined #openstack-ironic | 21:22 | |
| *** vsaienko has quit IRC | 21:23 | |
| *** vsaienko has joined #openstack-ironic | 21:25 | |
| *** dprince has quit IRC | 21:28 | |
| *** dprince has joined #openstack-ironic | 21:28 | |
| *** sayantani01 has joined #openstack-ironic | 21:29 | |
| jroll | blah, broken CI in mitaka | 21:29 |
| *** jcoufal has quit IRC | 21:30 | |
| *** sayantani01 has quit IRC | 21:30 | |
| *** Sukhdev has joined #openstack-ironic | 21:30 | |
| * jroll gives up for today | 21:31 | |
| jroll | see y'all | 21:31 |
| *** sayantani01 has joined #openstack-ironic | 21:31 | |
| *** jkilpatr has joined #openstack-ironic | 21:34 | |
| *** david-lyle has quit IRC | 21:35 | |
| *** krtaylor has joined #openstack-ironic | 21:35 | |
| TheJulia | goodnight jroll | 21:36 |
| *** alexpilotti has joined #openstack-ironic | 21:38 | |
| *** alexpilo_ has quit IRC | 21:38 | |
| *** david-lyle has joined #openstack-ironic | 21:39 | |
| *** david-lyle has quit IRC | 21:39 | |
| *** david-lyle has joined #openstack-ironic | 21:39 | |
| *** milan has quit IRC | 21:40 | |
| *** milan has joined #openstack-ironic | 21:41 | |
| openstackgerrit | Merged openstack/ironic-lib: Limit memory usage when running qemu-img https://review.openstack.org/422854 | 21:41 |
| *** jkilpatr has quit IRC | 21:42 | |
| openstackgerrit | Xavier proposed openstack/ironic: Adds validation before delete profile in oneview https://review.openstack.org/386014 | 21:48 |
| JayF | any idea what's up with stable/mitaka ci for ironic-lib | 21:48 |
| JayF | it's failing more jobs than are passing :x | 21:48 |
| JayF | mainly looking at https://review.openstack.org/#/c/422855/ | 21:49 |
| patchbot | patch 422855 - ironic-lib (stable/mitaka) - Limit memory usage when running qemu-img | 21:49 |
| *** alexpilotti has quit IRC | 21:56 | |
| *** alexpilotti has joined #openstack-ironic | 21:57 | |
| TheJulia | interesting... | 22:03 |
| *** ccamacho has quit IRC | 22:05 | |
| rloo | JayF: no idea. wrt the doc failure. ironic-lib in mitaka doesn't have a doc/ subdirectory so no conf.py either. | 22:06 |
| *** milan has quit IRC | 22:10 | |
| *** milan has joined #openstack-ironic | 22:11 | |
| TheJulia | the nova compute proxy blew up in one along with virtualbmc appearing to have gone awol | 22:15 |
| *** jkilpatr has joined #openstack-ironic | 22:17 | |
| *** Goneri has quit IRC | 22:19 | |
| *** ccamacho has joined #openstack-ironic | 22:21 | |
| *** trown is now known as trown|outtypewww | 22:25 | |
| TheJulia | two of them blew up shortly after one of the periodic resource tracker tasks fired in the compute service | 22:26 |
| soliosg | How is states.svg (the finite state machine diagram) generated from states.py? | 22:27 |
| mariojv | soliosg: tox -egenstates | 22:27 |
| TheJulia | ^^^ that | 22:27 |
| mariojv | https://github.com/openstack/ironic/blob/master/tools/states_to_dot.py#L56 | 22:27 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add base cinder common interface https://review.openstack.org/355625 | 22:28 |
| * soliosg thanks faster that the speed light replies from @mariojv and TheJulia | 22:28 | |
| TheJulia | stepping away, I'll keep looking at the stable/mitaka failures, but the last change to nova stable/mitaka was three weeks ago :\ | 22:29 |
| *** baoli has quit IRC | 22:30 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent: [WIP] Trying to create a package for biosdevname https://review.openstack.org/396406 | 22:32 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Agent driver rescue implementation https://review.openstack.org/400437 | 22:33 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 22:33 |
| mariojv | haven't responded to comments on the api/conductor patch yet, just a rebase ^ | 22:33 |
| *** jheroux has quit IRC | 22:33 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Agent: Listen for connections on both IPv4 and IPv6 ports https://review.openstack.org/411818 | 22:33 |
| openstackgerrit | Merged openstack/ironic-python-agent: iscsi: Listen for connections on both IPv4 and IPv6 ports https://review.openstack.org/411819 | 22:34 |
| openstackgerrit | Merged openstack/ironic-python-agent: Add IPv6 unit test for _get_route_source https://review.openstack.org/419533 | 22:34 |
| jlvillal | vsaienk0: FYI: Changing the grenade-multitenant job reproduce.sh http://logs.openstack.org/49/422149/3/experimental/gate-grenade-dsvm-ironic-multitenant-ubuntu-xenial-nv/4f615ca/logs/reproduce.sh | 22:35 |
| jlvillal | vsaienk0: IRONIC_TEMPEST_WHOLE_DISK_IMAGE=True IRONIC_VM_EPHEMERAL_DISK=0 seems to have fixed the ssh error. | 22:36 |
| jlvillal | vsaienk0: No idea why. I'm double checking that is the reason. As my previous test also changed deploy_driver to 'agent_ssh'. But only changing 'agent_ssh' didn't change the failure. | 22:37 |
| *** pmannidi has joined #openstack-ironic | 22:37 | |
| *** milan has quit IRC | 22:40 | |
| *** wrouesnel has joined #openstack-ironic | 22:40 | |
| *** milan has joined #openstack-ironic | 22:41 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add the new capabilities to the capabilities list https://review.openstack.org/422902 | 22:41 |
| *** vsaienko has quit IRC | 22:46 | |
| * swatson is away: I'm away | 22:46 | |
| *** vsaienko has joined #openstack-ironic | 22:48 | |
| *** wrouesnel has quit IRC | 22:48 | |
| *** wrouesnel has joined #openstack-ironic | 22:51 | |
| jlvillal | vsaienk0: Okay, need to be agent_ssh also. | 22:53 |
| openstackgerrit | Merged openstack/ironic: Default ironic to not use nested KVM https://review.openstack.org/421581 | 22:55 |
| rloo | ^^ yay! | 22:55 |
| *** srobert has quit IRC | 22:57 | |
| *** ijw has joined #openstack-ironic | 22:59 | |
| *** ijw_ has joined #openstack-ironic | 23:02 | |
| *** ijw has quit IRC | 23:02 | |
| *** david-lyle has quit IRC | 23:03 | |
| *** lindycoder_ has joined #openstack-ironic | 23:03 | |
| *** lindycoder_ has quit IRC | 23:03 | |
| *** absubram has quit IRC | 23:05 | |
| *** lindycoder has quit IRC | 23:05 | |
| *** aNuposic has quit IRC | 23:07 | |
| *** absubram has joined #openstack-ironic | 23:09 | |
| *** krtaylor has quit IRC | 23:10 | |
| *** milan has quit IRC | 23:10 | |
| *** milan has joined #openstack-ironic | 23:11 | |
| *** [1]cdearborn has quit IRC | 23:12 | |
| *** makowals has quit IRC | 23:17 | |
| JayF | Do we have any concept of what % of ironic users are standalone (vs w/nova+full openstack?) | 23:17 |
| *** aNuposic has joined #openstack-ironic | 23:20 | |
| *** priteau has joined #openstack-ironic | 23:22 | |
| *** ccamacho has quit IRC | 23:22 | |
| *** amotoki has quit IRC | 23:22 | |
| *** sacharya_ has joined #openstack-ironic | 23:23 | |
| *** absubram has quit IRC | 23:23 | |
| *** sacharya has quit IRC | 23:23 | |
| rloo | JayF: not me. would the last user survey have that info? | 23:23 |
| rloo | JayF: guess that'd be a good question to put in the next survey but too late. | 23:24 |
| rloo | JayF: if you have a chance/time and feel like it, lucas-afk wanted this reviewed. i don't know enough to +A it and I have to take off... https://review.openstack.org/#/c/374988/ | 23:24 |
| patchbot | patch 374988 - ironic - DevStack: Configure nodes/environment to boot in U... | 23:24 |
| jlvillal | JayF: If you have time, can you look at this devstack-gate change. https://review.openstack.org/#/c/422911/ ? | 23:25 |
| patchbot | patch 422911 - openstack-infra/devstack-gate - Remove setting IRONIC_VM_SPECS_RAM in devstack-gate | 23:25 |
| JayF | probably no time today | 23:25 |
| rloo | JayF: good answer :) | 23:26 |
| *** priteau has quit IRC | 23:27 | |
| *** rloo has quit IRC | 23:31 | |
| *** milan has quit IRC | 23:33 | |
| *** wrouesnel has quit IRC | 23:35 | |
| *** wrouesnel has joined #openstack-ironic | 23:38 | |
| *** jaosorior has quit IRC | 23:38 | |
| *** jvgrant has quit IRC | 23:38 | |
| *** jvgrant has joined #openstack-ironic | 23:38 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent: Reboot and Poweroff fails with coreos IPA image https://review.openstack.org/412594 | 23:40 |
| openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 23:43 |
| *** hurricanerix has joined #openstack-ironic | 23:46 | |
| *** makowals has joined #openstack-ironic | 23:55 | |
| *** rpioso has quit IRC | 23:56 | |
| *** vsaienko has quit IRC | 23:56 | |
| *** rbudden has quit IRC | 23:57 | |
| *** sayantani01 has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!