| *** makowals has quit IRC | 00:01 | |
| *** rbudden has joined #openstack-ironic | 00:01 | |
| *** wrouesnel has quit IRC | 00:02 | |
| *** wrouesnel has joined #openstack-ironic | 00:06 | |
| *** sayantani01 has joined #openstack-ironic | 00:07 | |
| *** sayantani01 has quit IRC | 00:11 | |
| *** mtanino has quit IRC | 00:13 | |
| *** hw_wutianwei has joined #openstack-ironic | 00:24 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Limit memory usage when running qemu-img convert https://review.openstack.org/422775 | 00:26 |
|---|---|---|
| *** dsneddon is now known as dsneddon_afk | 00:27 | |
| *** ppiela has quit IRC | 00:29 | |
| *** dsneddon has joined #openstack-ironic | 00:30 | |
| *** awiddersheim has joined #openstack-ironic | 00:33 | |
| *** wrouesnel has quit IRC | 00:35 | |
| *** wrouesnel has joined #openstack-ironic | 00:43 | |
| *** awiddersheim has left #openstack-ironic | 00:44 | |
| *** awiddersheim has joined #openstack-ironic | 00:45 | |
| *** hoangcx has joined #openstack-ironic | 00:53 | |
| *** bradjones has quit IRC | 00:55 | |
| *** phuongnh has joined #openstack-ironic | 00:55 | |
| TheJulia | JayF: w/r/t standalone vs full stack, I don't know of a percentage, but I think I took counts of hands in the bcn baremetal session which was kind of interesting if memory serves. | 00:55 |
| *** wrouesnel has quit IRC | 01:02 | |
| jlvillal | vsaienk0: My theory is that multitenant is not compatible with using pxe_ssh deploy driver. At least with our current devstack setup. | 01:04 |
| * jlvillal feels he made some good progress today :) | 01:13 | |
| *** amotoki has joined #openstack-ironic | 01:15 | |
| *** wrouesnel has joined #openstack-ironic | 01:22 | |
| TheJulia | jroll: JayF: looks like ironic-lib's gate has been borked for a long time. https://review.openstack.org/#/c/397897/ 9 weeks ago. Same failure pattern :( | 01:23 |
| patchbot | patch 397897 - ironic-lib (stable/mitaka) - Add support for BIOS local boot for GPT label (ABANDONED) | 01:23 |
| *** Syed__ has quit IRC | 01:25 | |
| *** makowals has joined #openstack-ironic | 01:26 | |
| *** wrouesnel has quit IRC | 01:35 | |
| *** aNuposic has quit IRC | 01:36 | |
| *** sayantani01 has joined #openstack-ironic | 01:39 | |
| *** aNuposic has joined #openstack-ironic | 01:41 | |
| *** Sukhdev has quit IRC | 01:49 | |
| *** wrouesnel has joined #openstack-ironic | 01:59 | |
| *** wrouesnel has quit IRC | 01:59 | |
| *** wrouesnel has joined #openstack-ironic | 02:08 | |
| openstackgerrit | Anup Navare proposed openstack/ironic: Config drive support for Ceph https://review.openstack.org/407639 | 02:15 |
| *** causten_ has quit IRC | 02:15 | |
| *** dprince has quit IRC | 02:19 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI https://review.openstack.org/348191 | 02:20 |
| *** tiendc has joined #openstack-ironic | 02:28 | |
| *** wrouesnel has quit IRC | 02:32 | |
| *** wrouesnel has joined #openstack-ironic | 02:33 | |
| *** aNuposic has quit IRC | 02:36 | |
| *** wrouesnel has quit IRC | 02:37 | |
| *** wrouesnel has joined #openstack-ironic | 02:38 | |
| *** wrouesnel has quit IRC | 02:39 | |
| *** baoli has joined #openstack-ironic | 02:40 | |
| *** wrouesnel has joined #openstack-ironic | 02:44 | |
| *** links has joined #openstack-ironic | 02:53 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Update the existing APIs due to adding get_supported_power_states https://review.openstack.org/382194 | 02:58 |
| *** wrouesnel has quit IRC | 03:03 | |
| *** wrouesnel has joined #openstack-ironic | 03:08 | |
| *** alexpilotti has quit IRC | 03:09 | |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient: Add args to CLI 'node-create' for selecting hardware interfaces https://review.openstack.org/418359 | 03:10 |
| *** wrouesnel has quit IRC | 03:19 | |
| *** tovin07 has joined #openstack-ironic | 03:22 | |
| *** wrouesnel has joined #openstack-ironic | 03:27 | |
| *** wrouesnel has quit IRC | 03:35 | |
| *** wrouesnel has joined #openstack-ironic | 03:37 | |
| *** baoli has quit IRC | 03:42 | |
| *** baoli_ has joined #openstack-ironic | 03:47 | |
| *** amotoki has quit IRC | 03:49 | |
| *** lindycoder_ has joined #openstack-ironic | 03:51 | |
| *** wrouesnel has quit IRC | 03:54 | |
| *** wrouesnel has joined #openstack-ironic | 03:54 | |
| *** alexpilotti has joined #openstack-ironic | 03:55 | |
| *** vikrant has joined #openstack-ironic | 03:57 | |
| *** lindycoder_ has quit IRC | 03:58 | |
| *** lindycoder_ has joined #openstack-ironic | 04:11 | |
| *** wrouesnel has quit IRC | 04:12 | |
| *** wrouesnel has joined #openstack-ironic | 04:14 | |
| *** wrouesnel has quit IRC | 04:15 | |
| *** links has quit IRC | 04:16 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool management driver for Inject NMI https://review.openstack.org/348721 | 04:17 |
| *** lindycoder_ has quit IRC | 04:17 | |
| *** wrouesnel has joined #openstack-ironic | 04:18 | |
| *** links has joined #openstack-ironic | 04:18 | |
| *** amotoki has joined #openstack-ironic | 04:19 | |
| *** amotoki_ has joined #openstack-ironic | 04:25 | |
| *** udesale has joined #openstack-ironic | 04:25 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: iRMC management driver for Inject NMI https://review.openstack.org/348732 | 04:27 |
| *** amotoki has quit IRC | 04:27 | |
| *** udesale has quit IRC | 04:30 | |
| *** wrouesnel has quit IRC | 04:37 | |
| *** wrouesnel has joined #openstack-ironic | 04:39 | |
| *** aparnav has joined #openstack-ironic | 04:57 | |
| *** krtaylor has joined #openstack-ironic | 04:57 | |
| *** baoli_ has quit IRC | 05:03 | |
| *** baoli has joined #openstack-ironic | 05:04 | |
| *** baoli has quit IRC | 05:05 | |
| *** jerrygb has quit IRC | 05:05 | |
| *** udesale has joined #openstack-ironic | 05:09 | |
| *** sacharya_ has quit IRC | 05:10 | |
| *** udesale has quit IRC | 05:10 | |
| *** deray has joined #openstack-ironic | 05:10 | |
| *** udesale has joined #openstack-ironic | 05:11 | |
| *** sacharya has joined #openstack-ironic | 05:11 | |
| *** priteau has joined #openstack-ironic | 05:24 | |
| *** priteau has quit IRC | 05:29 | |
| *** aNuposic has joined #openstack-ironic | 05:32 | |
| *** aNuposic has quit IRC | 05:36 | |
| *** jerrygb has joined #openstack-ironic | 05:39 | |
| *** ijw_ has quit IRC | 05:52 | |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic: WIP/DNM: Baremetal Devstack Network Simulation Developer Doc https://review.openstack.org/422582 | 05:59 |
| *** srart has quit IRC | 06:12 | |
| *** dsneddon_ has joined #openstack-ironic | 06:14 | |
| *** rbudden has quit IRC | 06:14 | |
| *** dsneddon has quit IRC | 06:15 | |
| *** absubram has joined #openstack-ironic | 06:19 | |
| *** srart has joined #openstack-ironic | 06:22 | |
| *** phuongnh has quit IRC | 06:30 | |
| *** phuongnh has joined #openstack-ironic | 06:31 | |
| *** sayantani01 has quit IRC | 06:31 | |
| *** baoli has joined #openstack-ironic | 06:40 | |
| *** baoli has quit IRC | 06:44 | |
| *** mjura has joined #openstack-ironic | 06:46 | |
| *** tiendc has quit IRC | 06:57 | |
| *** aNuposic has joined #openstack-ironic | 06:58 | |
| *** dsneddon_ has quit IRC | 07:01 | |
| *** ijw has joined #openstack-ironic | 07:04 | |
| *** ijw_ has joined #openstack-ironic | 07:07 | |
| *** absubram has quit IRC | 07:08 | |
| *** ijw has quit IRC | 07:10 | |
| *** ijw__ has joined #openstack-ironic | 07:11 | |
| *** ijw_ has quit IRC | 07:11 | |
| *** ChubYann has quit IRC | 07:12 | |
| *** rajinir has quit IRC | 07:16 | |
| *** tiendc has joined #openstack-ironic | 07:17 | |
| *** phuongnh has quit IRC | 07:20 | |
| pas-ha | morning itonic :) | 07:20 |
| * pas-ha apparently not awake enough, --> coffee | 07:21 | |
| openstackgerrit | Nam Nguyen Hoai proposed openstack/ironic: Fix typo https://review.openstack.org/423094 | 07:24 |
| *** ohamada has joined #openstack-ironic | 07:25 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add RPC and object version pinning https://review.openstack.org/407491 | 07:29 |
| *** dsneddon has joined #openstack-ironic | 07:36 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add version column https://review.openstack.org/412397 | 07:40 |
| xek | morning pas-ha! | 07:40 |
| pas-ha | morning xek :) | 07:41 |
| *** pcaruana has joined #openstack-ironic | 07:43 | |
| *** early` has joined #openstack-ironic | 07:44 | |
| openstackgerrit | Ha Dung proposed openstack/ironic: Fixes all current typo bugs of Ironic project. https://review.openstack.org/423109 | 07:46 |
| *** melwitt_ has joined #openstack-ironic | 07:48 | |
| *** ijw__ has quit IRC | 07:50 | |
| *** joanna has quit IRC | 07:51 | |
| *** early has quit IRC | 07:51 | |
| *** ptx0 has quit IRC | 07:51 | |
| *** melwitt has quit IRC | 07:51 | |
| *** ptx0 has joined #openstack-ironic | 07:53 | |
| *** SerenaFeng has joined #openstack-ironic | 08:01 | |
| *** tesseract has joined #openstack-ironic | 08:05 | |
| *** pmannidi has quit IRC | 08:09 | |
| *** phuongnh has joined #openstack-ironic | 08:13 | |
| *** SerenaFeng has quit IRC | 08:19 | |
| *** ralonsoh has joined #openstack-ironic | 08:23 | |
| *** tovin07 has quit IRC | 08:30 | |
| *** nicodemos has quit IRC | 08:30 | |
| *** afazekas has quit IRC | 08:30 | |
| *** tovin07 has joined #openstack-ironic | 08:30 | |
| openstackgerrit | Cuong Nguyen proposed openstack/ironic: Use context manager for better file handling https://review.openstack.org/423121 | 08:31 |
| openstackgerrit | Nam Nguyen Hoai proposed openstack/ironic: Fix typo https://review.openstack.org/423094 | 08:35 |
| *** afazekas has joined #openstack-ironic | 08:35 | |
| *** SerenaFeng has joined #openstack-ironic | 08:36 | |
| openstackgerrit | Nam Nguyen Hoai proposed openstack/ironic: Fix typo in the metrics.rst file https://review.openstack.org/423094 | 08:37 |
| *** aNuposic has quit IRC | 08:39 | |
| *** amoralej|off is now known as amoralej | 08:39 | |
| *** ohamada has quit IRC | 08:45 | |
| *** ohamada has joined #openstack-ironic | 08:49 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-ironic | 09:00 | |
| *** ccamacho has joined #openstack-ironic | 09:01 | |
| *** openstackgerrit has quit IRC | 09:02 | |
| *** amarao has joined #openstack-ironic | 09:05 | |
| *** SerenaFeng has quit IRC | 09:07 | |
| *** openstackgerrit has joined #openstack-ironic | 09:08 | |
| openstackgerrit | Cuong Nguyen proposed openstack/ironic: Use context manager for better file handling https://review.openstack.org/423121 | 09:08 |
| *** SerenaFeng has joined #openstack-ironic | 09:08 | |
| *** zackf has quit IRC | 09:09 | |
| *** mkrai has quit IRC | 09:13 | |
| *** mkrai has joined #openstack-ironic | 09:13 | |
| *** derekh has joined #openstack-ironic | 09:15 | |
| *** johnthetubaguy has quit IRC | 09:16 | |
| *** johnthetubaguy has joined #openstack-ironic | 09:19 | |
| *** fxpester has joined #openstack-ironic | 09:22 | |
| *** nmathew has joined #openstack-ironic | 09:23 | |
| vdrok | good morning ironic, pas-ha and xek | 09:24 |
| *** SerenaFeng has quit IRC | 09:27 | |
| *** ohamada has quit IRC | 09:27 | |
| *** ohamada has joined #openstack-ironic | 09:27 | |
| *** SerenaFeng has joined #openstack-ironic | 09:31 | |
| *** ohamada has quit IRC | 09:32 | |
| *** ohamada has joined #openstack-ironic | 09:32 | |
| *** joanna has joined #openstack-ironic | 09:32 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add version column https://review.openstack.org/412397 | 09:34 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 09:34 |
| *** lucas-afk is now known as lucasagomes | 09:34 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add compatibility with Newton when creating a node https://review.openstack.org/420728 | 09:35 |
| lucasagomes | morning all | 09:35 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: [PoC] Add fake fields in Node object and RPC https://review.openstack.org/415462 | 09:35 |
| openstackgerrit | Merged openstack/ironic-ui: Add Python 3.5 classifier and venv https://review.openstack.org/421695 | 09:35 |
| openstackgerrit | Merged openstack/ironic-ui: Imported Translations from Zanata https://review.openstack.org/418781 | 09:39 |
| openstackgerrit | Merged openstack/ironic-ui: Updated from global requirements https://review.openstack.org/420396 | 09:39 |
| *** jerrygb_ has joined #openstack-ironic | 09:41 | |
| *** jerrygb has quit IRC | 09:43 | |
| *** jerrygb has joined #openstack-ironic | 09:46 | |
| *** jerrygb_ has quit IRC | 09:48 | |
| *** MattMan has quit IRC | 09:52 | |
| *** MattMan has joined #openstack-ironic | 09:52 | |
| *** tiendc has quit IRC | 09:53 | |
| *** zackf has joined #openstack-ironic | 09:57 | |
| *** sacharya has quit IRC | 09:57 | |
| *** sacharya has joined #openstack-ironic | 09:58 | |
| *** zackf has quit IRC | 10:01 | |
| phuongnh | morning lucasagomes | 10:02 |
| vdrok | good morning lucasagomes and phuongnh ! | 10:02 |
| *** mrtenio has quit IRC | 10:02 | |
| phuongnh | good morning vdrok | 10:02 |
| phuongnh | and maybe a nice weekend to you all :) | 10:03 |
| *** priteau has joined #openstack-ironic | 10:05 | |
| lucasagomes | o/ | 10:05 |
| lucasagomes | jroll, thanks for all the backports dude! | 10:05 |
| phuongnh | lucasagomes, sorry about my inconvenience, if you have time, please help me and Nisha to review our spec document about "Add new capabilities to ironic" : https://review.openstack.org/#/c/338138/ | 10:06 |
| patchbot | patch 338138 - ironic-specs - Add new capabilities to ironic | 10:06 |
| *** tiendc has joined #openstack-ironic | 10:06 | |
| lucasagomes | phuongnh, oh true, sure will do | 10:07 |
| phuongnh | thank you a lot | 10:07 |
| *** priteau has quit IRC | 10:07 | |
| *** athomas has joined #openstack-ironic | 10:10 | |
| *** e0ne has joined #openstack-ironic | 10:12 | |
| *** jerrygb has quit IRC | 10:13 | |
| *** deray has quit IRC | 10:13 | |
| *** sambetts|afk is now known as sambetts | 10:13 | |
| sambetts | morning all | 10:14 |
| *** jerrygb has joined #openstack-ironic | 10:14 | |
| *** gcb has joined #openstack-ironic | 10:14 | |
| *** hw_wutianwei has quit IRC | 10:18 | |
| vdrok | good morning sambetts | 10:19 |
| lucasagomes | sambetts, morning | 10:20 |
| *** sacharya has quit IRC | 10:23 | |
| *** sacharya has joined #openstack-ironic | 10:24 | |
| *** hoangcx has quit IRC | 10:25 | |
| *** aarefiev_afk is now known as aarefiev | 10:26 | |
| aarefiev | morning all! | 10:26 |
| sambetts | hey lucasagomes vdrok | 10:26 |
| sambetts | o/ aarefiev | 10:26 |
| vdrok | morning aarefiev | 10:26 |
| lucasagomes | o/ aarefiev morning | 10:26 |
| aarefiev | morning sambetts, vdrok, lucasagomes | 10:27 |
| *** amotoki_ has quit IRC | 10:28 | |
| *** priteau has joined #openstack-ironic | 10:29 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic: Use global vars for storing image deploy path's https://review.openstack.org/420724 | 10:30 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Change tenant to project in docs and unit tests https://review.openstack.org/420055 | 10:32 |
| *** gcb has quit IRC | 10:34 | |
| *** tiendc has quit IRC | 10:34 | |
| openstackgerrit | Taku Izumi proposed openstack/ironic-python-agent: Add support for finalize_rescue https://review.openstack.org/417351 | 10:35 |
| *** amotoki has joined #openstack-ironic | 10:38 | |
| *** SerenaFeng has quit IRC | 10:40 | |
| *** phuongnh has quit IRC | 10:40 | |
| *** zackf has joined #openstack-ironic | 10:41 | |
| *** dtantsur|afk is now known as dtantsur | 10:43 | |
| dtantsur | morning Ironic | 10:45 |
| *** zackf has quit IRC | 10:46 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Rolling upgrades related dev documentation https://review.openstack.org/419439 | 10:46 |
| *** dsneddon has quit IRC | 10:50 | |
| dtantsur | sambetts, aarefiev, hey! mind checking https://review.openstack.org/#/c/422788/ please? we need it landed before the FF | 10:52 |
| patchbot | patch 422788 - python-ironic-inspector-client - Deprecate setting IPMI credentials | 10:52 |
| aarefiev | dtantsur: morning, wouldn't you mind update client docs | 10:55 |
| dtantsur | oh, right, gimme a minute | 10:59 |
| *** priteau has quit IRC | 11:02 | |
| *** priteau has joined #openstack-ironic | 11:02 | |
| openstackgerrit | Merged openstack/ironic: Add hardware types to the hash ring https://review.openstack.org/419065 | 11:02 |
| openstackgerrit | Vadim Hmyrov proposed openstack/ironic: Disable guessing content type from extension in node name. https://review.openstack.org/408597 | 11:03 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Deprecate setting IPMI credentials https://review.openstack.org/422788 | 11:03 |
| dtantsur | aarefiev, sambetts ^^^ with docs | 11:03 |
| dtantsur | jroll, hey! when you're here, please check https://review.openstack.org/#/c/422018/ to unblock inspector stable | 11:04 |
| patchbot | patch 422018 - ironic (stable/newton) - Remove service argument from tempest plugin client... | 11:04 |
| dtantsur | lucasagomes, mind checking https://review.openstack.org/#/c/421994/ please? it's needed to unblock glance v1 removal in tripleo | 11:05 |
| patchbot | patch 421994 - ironic - Use only Glance V2 by default (with a compatibilit... | 11:05 |
| *** priteau has quit IRC | 11:08 | |
| lucasagomes | dtantsur, sure thing! 1 sec | 11:10 |
| dtantsur | ugh, folks, how do I abort cleaning stuck in "clean wait" except for editing database? | 11:17 |
| dtantsur | if node-set-provision-state abort is NOT the answer | 11:18 |
| sambetts | node-set-provison-state manage? | 11:18 |
| dtantsur | nope | 11:18 |
| sambetts | :( | 11:18 |
| dtantsur | I'd call it a bug that you can't abort damned shredding (which has been running for many days already) | 11:19 |
| * dtantsur reports | 11:19 | |
| sambetts | node-set-power-state hard-off | 11:19 |
| sambetts | ;) | 11:19 |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic: [PoC] Add fake fields in Node object and RPC https://review.openstack.org/415462 | 11:20 |
| mat128 | dtantsur: you can't get out of clean wait | 11:21 |
| mat128 | we've had issues with specifically this because we are getting physical appliances into Ironic | 11:21 |
| mat128 | and they need to be manually cleaned (yeah.. I know) | 11:22 |
| dtantsur | yeah, it's been there for 10 days already | 11:22 |
| mat128 | oh god | 11:22 |
| mat128 | you can always use my trick | 11:22 |
| mat128 | https://github.com/mat128/ipa_cleaning_success | 11:22 |
| vdrok | good morning dtantsur and mat128 | 11:22 |
| dtantsur | I'd better try to fix it. reported as https://bugs.launchpad.net/ironic/+bug/1658061 | 11:22 |
| openstack | Launchpad bug 1658061 in Ironic "No way to abort even abortable in-band cleaning step" [High,Triaged] - Assigned to Dmitry Tantsur (divius) | 11:22 |
| mat128 | subscribing, thanks | 11:23 |
| mat128 | dtantsur: is there any shred process running? | 11:23 |
| dtantsur | "The current clean step "erase_devices" for node 5983b961-2c47-4c9b-962b-0a9dfc194c86 is not abortable" wut? | 11:24 |
| dtantsur | no, really, wut? https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L913-L917 | 11:24 |
| mat128 | dtantsur: do a GET on /v1/commands | 11:26 |
| mat128 | you will see running commands and their details | 11:26 |
| *** nmathew has quit IRC | 11:27 | |
| dtantsur | I see abortable=True in the cached clean steps | 11:27 |
| dtantsur | aha, I seem to understand. I used *manual* cleaning, and it does not take abortable information | 11:36 |
| sambetts | lucasagomes, dtantsur: what was that leader election process we were looking at before? | 11:36 |
| sambetts | lucasagomes, dtantsur: I've lost the diagrams/paper about it | 11:36 |
| * lucasagomes reads | 11:36 | |
| lucasagomes | sambetts, you mean before what ? | 11:37 |
| sambetts | lucasagomes: before now ;) it was a few months back when we were looking at lock managers I think | 11:37 |
| lucasagomes | sambetts, we talked about using tooz (with zookeeper) right ? | 11:37 |
| lucasagomes | that's what I remember heh | 11:38 |
| sambetts | yeah, but we talked about a leader election process, it might have been for the inspector and who was running the periodic tasks | 11:38 |
| sambetts | dtantsur: ^ you mgiht rememeber | 11:38 |
| dtantsur | toox+zookeper again, I think | 11:38 |
| sambetts | wasn't it a specifc "type" of leader election, we had a white paper about it right? | 11:39 |
| dtantsur | dunno, let's wait for milan | 11:39 |
| sambetts | milan: perhaps you remember | 11:40 |
| sambetts | ah I think I found something that I remember https://raft.github.io/ | 11:42 |
| *** jkilpatr has quit IRC | 11:43 | |
| sambetts | lucasagomes, dtantsur: remember this ^ | 11:44 |
| lucasagomes | sambetts, oh yeah | 11:44 |
| lucasagomes | that's the algorithm | 11:44 |
| lucasagomes | etcd and others use it | 11:44 |
| *** amotoki has quit IRC | 11:44 | |
| lucasagomes | cool stuff | 11:44 |
| openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent: Mock out get_wildcard_address https://review.openstack.org/423227 | 11:53 |
| *** jkilpatr has joined #openstack-ironic | 11:57 | |
| *** pcaruana has quit IRC | 11:59 | |
| *** milan has joined #openstack-ironic | 12:04 | |
| *** milan has left #openstack-ironic | 12:05 | |
| *** jkilpatr has quit IRC | 12:05 | |
| *** pcaruana has joined #openstack-ironic | 12:06 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Rolling upgrade procedure documentation https://review.openstack.org/416565 | 12:07 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Fix DBDeadlock during clean up https://review.openstack.org/422770 | 12:07 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Rolling upgrade procedure documentation https://review.openstack.org/416565 | 12:07 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Fix DBDeadlock during clean up https://review.openstack.org/422770 | 12:09 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Correctly cache "abortable" flag for manual clean steps https://review.openstack.org/423235 | 12:12 |
| dtantsur | mat128, something like ^^^, will test now | 12:12 |
| mat128 | dtantsur: so it's only in the case of user-provided clean steps that it wasn't cached? | 12:14 |
| dtantsur | seems like that | 12:14 |
| mat128 | (re: the abortable flag) | 12:14 |
| mat128 | hmm | 12:14 |
| dtantsur | now that I fixed that, I can try figuring out if abort is working | 12:14 |
| openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent: Mock out get_wildcard_address https://review.openstack.org/423227 | 12:17 |
| *** vikrant has quit IRC | 12:18 | |
| *** nmathew has joined #openstack-ironic | 12:18 | |
| *** jkilpatr has joined #openstack-ironic | 12:19 | |
| openstackgerrit | Aparna proposed openstack/proliantutils: Add HPSUM firmware update support https://review.openstack.org/422004 | 12:19 |
| *** moshele has joined #openstack-ironic | 12:21 | |
| dtantsur | mat128, aborting manual cleaning works for me after this patch (though we need to set last_error to something like "aborted") | 12:21 |
| openstackgerrit | Merged openstack/ironic: Use only Glance V2 by default (with a compatibility option) https://review.openstack.org/421994 | 12:22 |
| dtantsur | EmilienM, ^^^ | 12:22 |
| mat128 | dtantsur: nice! | 12:22 |
| cdaniel | morning all | 12:22 |
| EmilienM | dtantsur: nice, thanks. I'll test it again with TripleO. My results have been bad until now. Maybe I missed something | 12:24 |
| lucasagomes | cdaniel, mat128 morning! | 12:28 |
| mat128 | good morning lucasagomes | 12:28 |
| *** ohamada has quit IRC | 12:28 | |
| *** xuao has joined #openstack-ironic | 12:29 | |
| *** ohamada has joined #openstack-ironic | 12:30 | |
| *** ohamada has quit IRC | 12:30 | |
| jroll | morning y'all | 12:31 |
| dtantsur | morning jroll | 12:31 |
| jroll | TheJulia: boooo re mitaka | 12:31 |
| jroll | lucasagomes: you're welcome | 12:31 |
| jroll | dtantsur: +A | 12:31 |
| dtantsur | thnx! | 12:31 |
| lucasagomes | jroll, morning :-) | 12:31 |
| jroll | np | 12:31 |
| * dtantsur figures out why aborting cleaning leaves empty last_error, despite us setting it | 12:32 | |
| dtantsur | because I'm looking at a wrong node, obviously! | 12:34 |
| jroll | heh | 12:35 |
| jroll | blame nova | 12:35 |
| * dtantsur always blames nova | 12:35 | |
| *** slagle_ is now known as slagle | 12:37 | |
| *** lucasagomes is now known as lucas-hungry | 12:37 | |
| dtantsur | mat128, to clarify: what did you try to abort that didn't work? | 12:38 |
| mat128 | dtantsur: my workaround was built even before the -wait states | 12:38 |
| mat128 | dtantsur: from the code you posted, abort probably works, unless you pass in manual clean steps | 12:39 |
| dtantsur | yep, seems like that | 12:39 |
| dtantsur | ok, now I need to figure out how to fix the node that is already stuck :) | 12:39 |
| vdrok | morning cdaniel and jroll | 12:40 |
| dtantsur | folks, what's our story for non-abortable steps getting stuck? | 12:41 |
| dtantsur | do we just wait for them, probably forever? | 12:41 |
| jroll | I think currently we just hack the db | 12:42 |
| jroll | it's pretty terrible | 12:42 |
| dtantsur | mmmm | 12:42 |
| *** appu103 has joined #openstack-ironic | 12:42 | |
| jroll | we talked about adding a timeout feature at some point, but is hard to guess what the timeout should be :) | 12:42 |
| dtantsur | do we need something like $ openstack baremetal node abort --yes-really-kill-it-with-fire | 12:43 |
| vdrok | dtantsur: restarting conductor moves them to clean fail iirc? | 12:43 |
| jroll | possibly, yeah | 12:43 |
| vdrok | ah, that's for cleaning state | 12:43 |
| dtantsur | jroll, timeout may be more dangerous, I'd prefer a conscious decision to brick the BMC :) | 12:43 |
| dtantsur | vdrok, yeah, not for clean wait | 12:43 |
| jroll | heh | 12:44 |
| * dtantsur adds to the PTG etherpad | 12:44 | |
| *** appu103 has quit IRC | 12:45 | |
| *** aparnav has quit IRC | 12:46 | |
| *** aparnav has joined #openstack-ironic | 12:46 | |
| *** jerrygb_ has joined #openstack-ironic | 12:46 | |
| *** jerrygb has quit IRC | 12:49 | |
| *** links has quit IRC | 12:55 | |
| *** amotoki has joined #openstack-ironic | 12:56 | |
| *** nmathew has quit IRC | 12:59 | |
| *** dprince has joined #openstack-ironic | 12:59 | |
| *** yolanda has quit IRC | 13:01 | |
| *** yolanda has joined #openstack-ironic | 13:02 | |
| *** xuao has quit IRC | 13:10 | |
| TheJulia | Good morning everyone | 13:10 |
| *** athomas has quit IRC | 13:10 | |
| TheJulia | I am onboard with the --yes-really-kill-it-with-fire option or maybe --yes-i-checked-just-do-it | 13:12 |
| mat128 | morning TheJulia | 13:14 |
| mat128 | dtantsur, TheJulia: +1 to kill-it-with-fire | 13:14 |
| *** ohamada has joined #openstack-ironic | 13:15 | |
| dtantsur | morning TheJulia | 13:15 |
| *** athomas has joined #openstack-ironic | 13:16 | |
| TheJulia | Also, giant warnings would likely be appropriate :) | 13:16 |
| *** gcb has joined #openstack-ironic | 13:19 | |
| *** trown|outtypewww is now known as trown | 13:22 | |
| *** xavierr has quit IRC | 13:22 | |
| *** clenimar has quit IRC | 13:23 | |
| *** gabriel-bezerra has quit IRC | 13:23 | |
| *** ricardoas has quit IRC | 13:23 | |
| *** cdaniel has quit IRC | 13:23 | |
| *** jheroux has joined #openstack-ironic | 13:25 | |
| *** sayantani01 has joined #openstack-ironic | 13:26 | |
| jlvillal | vsaienk0, vdrok: Not sure if you saw my message about multi-tenant not being compatible with pxe_ssh deploy driver in the gate. | 13:30 |
| jlvillal | vsaienk0, vdrok jroll: I proposed https://review.openstack.org/422917 | 13:31 |
| patchbot | patch 422917 - openstack-infra/project-config - ironic: grenade.*multitenant job use agent_ssh | 13:31 |
| jroll | can we not use _ssh please? :) | 13:31 |
| *** Goneri has joined #openstack-ironic | 13:31 | |
| jlvillal | jroll, Not sure I went with what the standard multitenant gate job is already using (agent_ssh). I will try agent_ipmitool later on. | 13:32 |
| *** aparnav has quit IRC | 13:32 | |
| jlvillal | later on this morning... | 13:33 |
| jroll | sure | 13:34 |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: [WIP] Implement os_client_config support https://review.openstack.org/423268 | 13:36 |
| dtantsur | jlvillal, is it just because of using whole disk images and hence local boot? | 13:36 |
| jlvillal | dtantsur, I'm not sure exactly on the root cause. But multitenant fails if using pxe_ssh. It could be that. | 13:37 |
| jroll | well, if the node is on the tenant network, we can't pxe boot, right | 13:38 |
| dtantsur | I see. yeah, I think it's because we default to partition images for pxe_* drivers, and to netboot for partitions images | 13:38 |
| *** bradjones has joined #openstack-ironic | 13:38 | |
| *** bradjones has quit IRC | 13:38 | |
| *** bradjones has joined #openstack-ironic | 13:38 | |
| jlvillal | jroll, ++ most likely | 13:38 |
| openstackgerrit | Merged openstack/ironic: Remove service argument from tempest plugin client manager https://review.openstack.org/422018 | 13:40 |
| *** Nisha_Agarwal has joined #openstack-ironic | 13:41 | |
| *** lindycoder_ has joined #openstack-ironic | 13:41 | |
| *** baoli has joined #openstack-ironic | 13:42 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Allow using resource classes https://review.openstack.org/407651 | 13:43 |
| *** baoli has quit IRC | 13:43 | |
| jroll | ^ we should probably get that in soonish | 13:43 |
| dtantsur | wow interesting | 13:43 |
| * jroll makes a test patch | 13:43 | |
| *** baoli has joined #openstack-ironic | 13:43 | |
| jroll | that's usage in devstack to be clear :) | 13:43 |
| dtantsur | ah | 13:43 |
| dtantsur | jroll, O | 13:43 |
| dtantsur | damn | 13:44 |
| dtantsur | jroll, I'm completely outdated on this work: should we switch tripleo to it as well? | 13:44 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: DNM: turn on resource classes https://review.openstack.org/423269 | 13:44 |
| jroll | dtantsur: hm, probably | 13:44 |
| jroll | I need to poke around a bit more but deployers should likely start setting that in ocata | 13:44 |
| jroll | and probably require it for pike upgrade | 13:45 |
| *** moshele has quit IRC | 13:45 | |
| * jroll lost track of it a bit himself | 13:45 | |
| dtantsur | hmm, tripleo is preparing for feature freeze, soooo :) | 13:47 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/423270 | 13:47 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/423271 | 13:47 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui: Updated from global requirements https://review.openstack.org/423272 | 13:47 |
| jroll | seems odd for tripleo to FF at the same time as services, no? | 13:47 |
| jroll | would tripleo backport something like this? | 13:48 |
| *** lucas-hungry is now known as lucasagomes | 13:48 | |
| dtantsur | jroll, yes. this decision was done, however, knowing the amount of bugs we face every release | 13:48 |
| dtantsur | jroll, if it's something required for clean upgrade to pike - we can definitely talk about it. but I don't understand it myself :) | 13:48 |
| jroll | hm | 13:48 |
| jroll | yeah | 13:48 |
| jroll | I need to dig more on what progress nova folks have made | 13:48 |
| dtantsur | ok, please lemme know what you figure out. or even send it out with [ironic] [tripleo] to the ML :) | 13:49 |
| jroll | yeah, I'm just going to join the scheduler team meeting on monday | 13:49 |
| dtantsur | cool | 13:49 |
| * dtantsur gets back to debugging cleaning in the overcloud, sigh.. | 13:50 | |
| TheJulia | jroll: So interesting on stable/mitaka... looks like installing the tempest plugin is causing the current ironic-lib to get pulled in overwriting the one installed from git | 13:54 |
| jroll | are folks fine with the IRONIC_USE_RESOURCE_CLASSES variable name in this patch, so I can move ahead on creating an experimental CI job? https://review.openstack.org/#/c/407651/4/devstack/lib/ironic | 13:54 |
| patchbot | patch 407651 - ironic - Allow using resource classes | 13:54 |
| jroll | TheJulia: O_o | 13:54 |
| TheJulia | http://logs.openstack.org/55/422855/1/check/gate-tempest-dsvm-ironic-lib-partition-pxe_ipmitool-ubuntu-trusty/378dcb7/logs/devstack-early.txt.gz#_2017-01-19_20_57_19_924 | 13:55 |
| jroll | I guess that makes sense, though I thought tempest was in a venv | 13:55 |
| jroll | tox -evenv-tempest -- pip install git+git://git.openstack.org/openstack/ironic | 13:55 |
| TheJulia | also wrong version of eventless is getting pulled in 0.20 where as stable/mitaka upper constraint is 0.18 | 13:56 |
| TheJulia | err, eventlet | 13:56 |
| jroll | ugh | 13:56 |
| TheJulia | that _might_ begin to explain some things if they are not constrained in a venv, but it looks like it at least finds the other ironic-lib and squashes it | 13:56 |
| TheJulia | at least, sifting through the logs | 13:57 |
| * dtantsur wonders if splitting the plugin will make it better or worse | 13:58 | |
| jroll | it should make it better, as we won't be installing all of ironic's deps | 13:58 |
| *** xuao has joined #openstack-ironic | 13:59 | |
| dtantsur | yeah | 14:00 |
| dtantsur | sambetts, mind re-reviewing https://review.openstack.org/#/c/422788/ please? we need to release ironic-inspector-client in the beginning of the next week | 14:01 |
| patchbot | patch 422788 - python-ironic-inspector-client - Deprecate setting IPMI credentials | 14:01 |
| * jroll doesn't understand why this happens when tempest is in a venv | 14:01 | |
| jroll | TheJulia: I don't think it's actually doing the wrong thing | 14:04 |
| jroll | or at least I don't think that's what is breaking CI | 14:04 |
| *** hamzy has quit IRC | 14:04 | |
| jroll | oh, lol | 14:04 |
| jroll | http://logs.openstack.org/55/422855/1/check/gate-tempest-dsvm-ironic-lib-partition-pxe_ipmitool-ubuntu-trusty/378dcb7/logs/screen-ir-cond.txt.gz?level=WARNING | 14:04 |
| jroll | that would do it, I think | 14:05 |
| *** hamzy has joined #openstack-ironic | 14:05 | |
| jroll | maybe not | 14:05 |
| jroll | it says attempt 1 of 3 but never runs again | 14:06 |
| sambetts | dtantsur: thanks :) | 14:06 |
| TheJulia | jroll: I think that is after the nova compute process goes belly up | 14:07 |
| TheJulia | http://logs.openstack.org/55/422855/1/check/gate-tempest-dsvm-ironic-lib-wholedisk-agent_ipmitool-ubuntu-trusty/ae3110d/logs/screen-n-cpu.txt.gz#_2017-01-19_20_52_04_757 | 14:07 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add RPC and object version pinning https://review.openstack.org/407491 | 14:08 |
| *** [1]cdearborn has joined #openstack-ironic | 14:08 | |
| dtantsur | sambetts, thank you :) ok, now I need to fix https://review.openstack.org/#/c/409789/, but it's pretty optional. then we can ask jroll to request a release ;) | 14:08 |
| patchbot | patch 409789 - python-ironic-inspector-client - Clarify that node names can be used in addition to... | 14:09 |
| jroll | TheJulia: oh yeah, that's after devstack cleans up | 14:09 |
| jroll | bah | 14:09 |
| jroll | so what the f | 14:09 |
| * jroll steps away for a few | 14:09 | |
| TheJulia | hmm... like 13 seconds | 14:10 |
| TheJulia | I think I found one where it was ~1 minute off before devstack actually started cleaning up. | 14:10 |
| *** rbudden has joined #openstack-ironic | 14:13 | |
| *** jcoufal has joined #openstack-ironic | 14:13 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Clarify that node names can be used in addition to UUIDs https://review.openstack.org/409789 | 14:14 |
| *** sayantani01 has quit IRC | 14:14 | |
| *** sayantani01 has joined #openstack-ironic | 14:15 | |
| *** ppiela has joined #openstack-ironic | 14:18 | |
| *** causten has joined #openstack-ironic | 14:20 | |
| *** sacharya has quit IRC | 14:23 | |
| *** sacharya_ has joined #openstack-ironic | 14:23 | |
| *** sacharya_ has quit IRC | 14:24 | |
| *** sacharya has joined #openstack-ironic | 14:25 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add version column https://review.openstack.org/412397 | 14:29 |
| *** srobert has joined #openstack-ironic | 14:29 | |
| *** sacharya has quit IRC | 14:30 | |
| TheJulia | hmm, does look like it should be in it the env only. whisky tango | 14:33 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Deprecate setting IPMI credentials https://review.openstack.org/422788 | 14:34 |
| TheJulia | and the version in pip freeze are right. And https://review.openstack.org/#/c/419516/ worked, only difference seemingly just being ironic-lib from git :\ | 14:38 |
| patchbot | patch 419516 - ironic (stable/mitaka) - Allow to set min,max API microversion in tempest (MERGED) | 14:38 |
| *** ohamada has quit IRC | 14:40 | |
| *** amoralej is now known as amoralej|lunch | 14:41 | |
| jroll | iiiinteresting. | 14:42 |
| *** absubram has joined #openstack-ironic | 14:42 | |
| vdrok | morning TheJulia and jlvillal | 14:45 |
| jlvillal | good morning vdrok | 14:45 |
| vdrok | jroll: yeah, I'm good with IRONIC_USE_RESOURCE_CLASSES | 14:45 |
| TheJulia | Good morning vdrok | 14:45 |
| *** mjura has quit IRC | 14:46 | |
| openstackgerrit | Michael Turek proposed openstack/python-ironicclient: Add support for RESCUE and UNRESCUE provision states https://review.openstack.org/408341 | 14:47 |
| *** jcoufal__ has joined #openstack-ironic | 14:48 | |
| *** jcoufal has quit IRC | 14:49 | |
| openstackgerrit | Michael Turek proposed openstack/python-ironicclient: Add support for RESCUE and UNRESCUE provision states https://review.openstack.org/408341 | 14:50 |
| vdrok | jroll: we might need to do some refactor before that devstack change | 14:51 |
| *** srobert_ has joined #openstack-ironic | 14:51 | |
| *** clenimar has joined #openstack-ironic | 14:51 | |
| *** sayantani01 has quit IRC | 14:51 | |
| *** cdaniel has joined #openstack-ironic | 14:51 | |
| *** mrtenio has joined #openstack-ironic | 14:51 | |
| jlvillal | jroll, FYI, I did a test using 'agent_ipmitool' and it failed with the grenade.*multitenant job. 'agent_ssh' is working. | 14:52 |
| lucasagomes | :-( | 14:52 |
| jlvillal | An error about securitygroup in use. I'd rather move forward with agent_ssh at this time. as it is working up into the create phase of grenade. While the other drivers are failing on the first tempest smoke test. | 14:53 |
| lucasagomes | jlvillal, what failed exactly ? Got the logs ? | 14:53 |
| jlvillal | lucasagomes, This isn't something that has ever worked before. I mean it isn't something that was working and now is broken. If that makes sense. | 14:53 |
| vdrok | jlvillal: that's a usual thing when node gets stuck in clean wait, and then tear down starts cleaning up security groups and other stuff | 14:54 |
| *** srobert has quit IRC | 14:54 | |
| jlvillal | vdrok, Okay. I can try it again... | 14:54 |
| *** melwitt_ is now known as melwitt | 14:55 | |
| vdrok | jlvillal: yup, if you have an env, please do, it might be something easy to fix. if it's not, maybe that's fine to go with _ssh :) | 14:55 |
| lucasagomes | jlvillal, oh right, yeah does makes sense | 14:55 |
| lucasagomes | jlvillal, I just wondering cause the SSH drivers will be removed soon | 14:55 |
| jlvillal | vdrok, Okay I'm spinning up a new VM right now | 14:55 |
| lucasagomes | (I think, unless the plans have changed) | 14:55 |
| jlvillal | lucasagomes, Ah, good point. | 14:55 |
| *** ricardoas has joined #openstack-ironic | 14:58 | |
| *** xuao has quit IRC | 15:01 | |
| *** xuao has joined #openstack-ironic | 15:02 | |
| *** mjturek has quit IRC | 15:04 | |
| lucasagomes | folks, if you have some time mind taking a look at https://review.openstack.org/#/c/374988/ (See my last comment for the plans of testing UEFI in gate) | 15:04 |
| patchbot | patch 374988 - ironic - DevStack: Configure nodes/environment to boot in U... | 15:04 |
| lucasagomes | cheerios :D | 15:04 |
| *** jcoufal__ has quit IRC | 15:05 | |
| *** xuao has quit IRC | 15:07 | |
| jlvillal | lucasagomes, I ate Cheerios for breakfast :) | 15:07 |
| lucasagomes | jlvillal, lol | 15:07 |
| * lucasagomes haven't tried it yet | 15:08 | |
| lucasagomes | I'm not really into the cereal stuff but I like the rice ones with chocolate | 15:08 |
| lucasagomes | forgot the name | 15:08 |
| *** xavierr has joined #openstack-ironic | 15:08 | |
| lucasagomes | choco pops I think | 15:08 |
| *** gcb has quit IRC | 15:09 | |
| dtantsur | bfournie, yeah, sorry, I forgot about that patch. well, we're still in good shape, I guess | 15:10 |
| *** mtanino has joined #openstack-ironic | 15:10 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add version column https://review.openstack.org/412397 | 15:10 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 15:10 |
| bfournie | dtantsur: thanks, yeah I think so | 15:10 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add compatibility with Newton when creating a node https://review.openstack.org/420728 | 15:11 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: [PoC] Add fake fields in Node object and RPC https://review.openstack.org/415462 | 15:11 |
| *** gabriel-bezerra has joined #openstack-ironic | 15:11 | |
| *** jcoufal has joined #openstack-ironic | 15:12 | |
| dtantsur | bfournie, +1053 loc, WOW | 15:13 |
| *** amoralej|lunch is now known as amoralej | 15:13 | |
| * dtantsur will review it after a break then :D | 15:13 | |
| *** Syed__ has joined #openstack-ironic | 15:14 | |
| TheJulia | jroll: trying to reproduce the failure in a blank vm, going to let that spin for a little while | 15:14 |
| bfournie | dtantsur: you mean the plugin. yeah, I know :-(, lots of tests though adding to loc, the python-ironic-inspector change is +484 | 15:18 |
| jroll | TheJulia: thanks | 15:18 |
| jroll | vdrok: thank you, what did you want to refactor before this? | 15:18 |
| vdrok | jroll: the network interface thing. both have the same problem | 15:19 |
| *** fxpester has quit IRC | 15:19 | |
| jroll | vdrok: oh, the enroll thing, right | 15:19 |
| jroll | I guess I could pick that up, probably | 15:20 |
| *** udesale has quit IRC | 15:21 | |
| *** udesale has joined #openstack-ironic | 15:21 | |
| *** udesale has quit IRC | 15:24 | |
| *** amotoki has quit IRC | 15:24 | |
| *** udesale has joined #openstack-ironic | 15:25 | |
| *** absubram has quit IRC | 15:25 | |
| *** jcoufal has quit IRC | 15:28 | |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic: [PoC] Add fake fields in Node object and RPC https://review.openstack.org/415462 | 15:28 |
| openstackgerrit | Merged openstack/ironic: Clean up driver_factory.enabled_supported_interfaces https://review.openstack.org/422862 | 15:29 |
| *** jcoufal has joined #openstack-ironic | 15:30 | |
| *** baoli has quit IRC | 15:30 | |
| *** baoli has joined #openstack-ironic | 15:33 | |
| *** bnemec is now known as beekneemech | 15:33 | |
| openstackgerrit | Eric Xie proposed openstack/python-ironicclient: Fix ImportError when providing a meaningless API version. https://review.openstack.org/423333 | 15:33 |
| *** sacharya has joined #openstack-ironic | 15:35 | |
| openstackgerrit | Miles Gould proposed openstack/ironic: Allow setting UEFI boot devices in DRAC driver https://review.openstack.org/420107 | 15:38 |
| *** jaosorior has joined #openstack-ironic | 15:39 | |
| *** mgoddard has joined #openstack-ironic | 15:39 | |
| jkilpatr | hey has anyone used the statsd integration in ironic? what should it look like in graphite? trying to pick it out of our existing metrics | 15:41 |
| lucasagomes | I believe jroll JayF mariojv ^ uses it | 15:42 |
| jkilpatr | lucasagomes, thanks mostly trying to figure out if I set it up right at this stage | 15:42 |
| jroll | jkilpatr: I think the default is something like com.example.ironic-conductor01.ironic.api.v1.drivers.DriverController.list or so | 15:43 |
| jroll | that is, (reversed hostname).(python method in module path style) | 15:43 |
| JayF | lucasagomes: fwiw we have not deployed the upstream metrics stuff in OnMetal, so no practical knowledge about it yet :( | 15:43 |
| jkilpatr | jroll, what log would I look in to make sure it's turned on and reporting correctly statsd is happy but I can't find where ironic logs about it | 15:44 |
| mariojv | yeah, all similar but downstream | 15:44 |
| lucasagomes | JayF, oh right, I thought you guys were mostly upstreaming what you have downstream already | 15:44 |
| jroll | jkilpatr: I don't think ironic logs about it, you should see them coming through in statsd though | 15:44 |
| jroll | if ironic logged that would be a ton of logs | 15:44 |
| mariojv | yep, that's the best verification ^ | 15:44 |
| mariojv | check the timestamps of the metrics too | 15:44 |
| jroll | oh prepend_host defaults to false | 15:45 |
| mariojv | if they're recent, it's probably working fine | 15:45 |
| jroll | so may just be ironic.foo.bar.Class.method | 15:45 |
| *** [1]cdearborn has quit IRC | 15:45 | |
| jroll | jkilpatr: I assume you've looked at the stuff here, right? https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L2120-L2143 | 15:46 |
| jkilpatr | cool, what would you guys be interested in visualizing from this? I'm going to get 50 beefy nodes for the sole purpose of deployment testing I intend to run a few thousand introspections / boots / whatever else is worth finding out about. | 15:46 |
| JayF | lucasagomes: yeah, but it gets changed enough that it doesn't always work out clearly enough | 15:46 |
| JayF | jkilpatr: I sorta like using metrics in a situation like that as a discovery method | 15:47 |
| JayF | jkilpatr: i.e. looking for things that take a long time, figure out why | 15:47 |
| *** jcoufal has quit IRC | 15:49 | |
| mariojv | after https://bugs.launchpad.net/ironic/+bug/1594242 landed, we can launch ironic instances with security groups right? i.e. openstack server create --flavor baremetal --security-group sec_group ... | 15:51 |
| openstack | Launchpad bug 1594242 in Ironic "[RFE] Security Groups support for baremetal servers" [Wishlist,Fix released] - Assigned to Sukhdev Kapur (sukhdev-8) | 15:51 |
| mariojv | (wondering about this for https://review.openstack.org/#/c/419634/) | 15:51 |
| patchbot | patch 419634 - nova - Add ironic security group support to support matri... | 15:51 |
| *** jcoufal has joined #openstack-ironic | 15:51 | |
| mariojv | wasn't sure if there's anything that needed to be done in the nova driver to make it aware of the capability | 15:52 |
| *** [1]cdearborn has joined #openstack-ironic | 15:58 | |
| openstackgerrit | Miles Gould proposed openstack/ironic: Allow setting UEFI boot devices in DRAC driver https://review.openstack.org/420107 | 15:59 |
| *** sayantani01 has joined #openstack-ironic | 16:02 | |
| *** rbudden has quit IRC | 16:02 | |
| *** pcaruana has quit IRC | 16:03 | |
| jlvillal | jroll, vdrok: FYI: I ran two more tests with 'agent_ipmitool' and they both failed for the grenade.*multitenant job. While 'agent_ssh' is working. | 16:05 |
| openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 16:05 |
| *** amotoki has joined #openstack-ironic | 16:05 | |
| jroll | so weird | 16:05 |
| jlvillal | jroll: vdrok vsaienk0: So I would like to get: https://review.openstack.org/422917 merged as is using 'agent_ssh' | 16:05 |
| patchbot | patch 422917 - openstack-infra/project-config - ironic: grenade.*multitenant job use agent_ssh | 16:05 |
| jroll | sure, we need to work hard on killing that driver in pike imo | 16:06 |
| *** amotoki has quit IRC | 16:06 | |
| vsaienk0 | jlvillal: why *_ssh? multinode multitenant uses agent_ipmitool it works fine | 16:06 |
| *** baoli_ has joined #openstack-ironic | 16:07 | |
| jlvillal | vsaienk0: No it doesn't | 16:07 |
| *** baoli has quit IRC | 16:07 | |
| jlvillal | vsaienk0: Oh multinode... | 16:07 |
| jlvillal | vsaienk0: Not sure. I have run three tests locally and they all failed... | 16:08 |
| openstackgerrit | Miles Gould proposed openstack/ironic: Allow setting UEFI boot devices in DRAC driver https://review.openstack.org/420107 | 16:08 |
| *** tovin07 has quit IRC | 16:08 | |
| *** ijw has joined #openstack-ironic | 16:08 | |
| vsaienk0 | jroll I'm thinking we can remove singlenode multitenant job, and made multinode multitenant job voting in Pike, wdt? | 16:11 |
| jroll | vsaienk0: yeah, I think that seems fine, don't see why not | 16:12 |
| jroll | I'd like to see many of the CI refactors we talked about in barcelona done in pike /cc dtantsur :P | 16:13 |
| *** nicodemos has joined #openstack-ironic | 16:13 | |
| jlvillal | vsaienk0: It seems like it has been fairly reliable | 16:13 |
| dtantsur | yeah.. | 16:13 |
| *** tovin07 has joined #openstack-ironic | 16:14 | |
| dtantsur | jroll, unrelated: wdyt about concerns voiced by vsaienk0 on https://review.openstack.org/#/c/422708/2/ironic/drivers/agent.py ? | 16:15 |
| patchbot | patch 422708 - ironic - Add hardware type for IPMI using ipmitool | 16:15 |
| *** zackf has joined #openstack-ironic | 16:15 | |
| dtantsur | I guess I can reimport this thing in the old module.. | 16:16 |
| dtantsur | dunno if we should really care or what | 16:16 |
| jroll | dtantsur: I feel like people shouldn't be inheriting those, but they could be | 16:16 |
| jroll | I'm not sure I see the benefit in moving them, though | 16:16 |
| jroll | I'd rather just delete :) | 16:16 |
| *** jcoufal_ has joined #openstack-ironic | 16:17 | |
| jroll | (yes, I know that isn't an option right now) | 16:17 |
| dtantsur | well, it was my long dream to consolidate related bits in one place. Anyway, I can revert it, if we find it concerning | 16:17 |
| *** rpioso has joined #openstack-ironic | 16:17 | |
| jroll | dtantsur: we could do that, but also "alias" it in the old file to prevent breaking | 16:17 |
| *** jcoufal has quit IRC | 16:18 | |
| dtantsur | yeah, this was my first thought | 16:18 |
| TheJulia | jroll: different error while attempting to reproduce, looks like pbr weirdness in that it is reporting that it couldn't find openstack.__version__ when OSC commands are invoked 8( | 16:19 |
| jroll | wtf | 16:19 |
| lucasagomes | jroll, jlvillal just to confirm here, *_ssh driver will stay in for ocata then ? | 16:27 |
| jlvillal | lucasagomes: According to the deprecation note agent_ssh could be removed in Ocata cycle or later | 16:31 |
| *** jcoufal_ has quit IRC | 16:31 | |
| jroll | lucasagomes: guess so, our CI depends on it still :( | 16:32 |
| lucasagomes | :-/ | 16:32 |
| openstackgerrit | Merged openstack/ironic: Generic management I/F for Inject NMI https://review.openstack.org/348191 | 16:36 |
| jroll | nice! | 16:36 |
| * jroll wonders if we'll get that thing done | 16:36 | |
| jroll | btw, I just did some periodic reporting up of the things. we did pretty well on priorities this cycle :D | 16:36 |
| dtantsur | yep :) | 16:38 |
| openstackgerrit | Merged openstack/ironic: DevStack: Configure nodes/environment to boot in UEFI mode https://review.openstack.org/374988 | 16:38 |
| *** jcoufal has joined #openstack-ironic | 16:38 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Add hardware type for IPMI using ipmitool https://review.openstack.org/422708 | 16:43 |
| dtantsur | updated ^^^ | 16:43 |
| jroll | cool, will review this afternoon | 16:44 |
| *** e0ne has quit IRC | 16:44 | |
| * jroll plans to eat lunch, finish API work, then review the things | 16:45 | |
| jroll | bbl | 16:45 |
| *** jaosorior has quit IRC | 16:48 | |
| *** jaosorior has joined #openstack-ironic | 16:49 | |
| jcook | What should the target provision state of a node that is deleting? | 16:50 |
| * dtantsur suspects "available" | 16:50 | |
| *** jaosorior has quit IRC | 16:51 | |
| jcook | I'll take it | 16:51 |
| jcook | dtantsur thanks | 16:52 |
| JayF | deleting -> cleaning -> available iirc | 16:56 |
| aslezil | dtantsur: Could you plese review once you get chance the spec for collecting bios given name of NIC https://review.openstack.org/#/c/404372/. The data collected is kept in inventory json file itself | 16:59 |
| patchbot | patch 404372 - ironic-inspector-specs - Collect NIC name given by BIOS | 16:59 |
| *** derekh has quit IRC | 17:00 | |
| dtantsur | aslezil, will put to my queue, but my work day is essentially over. will get to it on Monday, I guess | 17:00 |
| *** jcoufal_ has joined #openstack-ironic | 17:00 | |
| aslezil | Thanks dtantsur: :) | 17:00 |
| *** udesale has quit IRC | 17:02 | |
| *** jcoufal__ has joined #openstack-ironic | 17:02 | |
| *** jcoufal has quit IRC | 17:02 | |
| openstackgerrit | Sean Dague proposed openstack/ironic: DNM: testing devstack-gate local.conf https://review.openstack.org/423376 | 17:03 |
| *** aNuposic has joined #openstack-ironic | 17:04 | |
| *** jcoufal has joined #openstack-ironic | 17:04 | |
| *** rbudden has joined #openstack-ironic | 17:04 | |
| *** ralonsoh has quit IRC | 17:05 | |
| *** jcoufal_ has quit IRC | 17:05 | |
| *** jcoufal__ has quit IRC | 17:07 | |
| dtantsur | have a great weekend! | 17:08 |
| *** dtantsur is now known as dtantsur|afk | 17:08 | |
| *** amarao has quit IRC | 17:10 | |
| TheJulia | jroll: something with my python install, just starting over on a new fresh vm and trying again :( | 17:10 |
| openstackgerrit | Merged openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/423271 | 17:11 |
| jcook | jroll ImportError: No module named tooz. Was that a recent refactor / depends that wasn't added to requirements? | 17:13 |
| jcook | weird, I have it installed | 17:14 |
| lucasagomes | jcook, tooz is in the requirements.txt | 17:15 |
| lucasagomes | oh, hmm that's odd then | 17:15 |
| lucasagomes | jcook, running stuff with different python versions ? | 17:15 |
| jcook | yeah tests are failing for me | 17:15 |
| JayF | jcook: try tox -r | 17:15 |
| JayF | jcook: to reinstall your deps | 17:15 |
| lucasagomes | oh unittests, sometimes you have to rebuild the .tox/<test env> | 17:15 |
| jcook | k | 17:15 |
| lucasagomes | either -r or just delete rm .tox/py27 for example | 17:15 |
| jcook | yeah I can import manually in python interpreter | 17:16 |
| jcook | running -r now | 17:16 |
| jcook | that's probably it, thanks | 17:16 |
| lucasagomes | cool | 17:16 |
| jcook | wow screen of red on py34 | 17:16 |
| aNuposic | jcook: Yes i installed new tooz and removed .tox and then ran py27 | 17:20 |
| *** aNuposic has quit IRC | 17:21 | |
| *** aNuposic has joined #openstack-ironic | 17:22 | |
| jcook | yeah tox -r fixed it | 17:22 |
| JayF | anytime I get a strange unit test error I don't expect, I use tox with -r | 17:22 |
| JayF | and 90% of the time it fixes it | 17:22 |
| lucasagomes | :D | 17:23 |
| lucasagomes | folks I'm calling it a day | 17:23 |
| *** dprince has quit IRC | 17:23 | |
| lucasagomes | have a great evening all, enjoy the weekend | 17:23 |
| jlvillal | lucasagomes: Good night! | 17:23 |
| lucasagomes | I'm off on monday so I talk to you all on tueday! | 17:23 |
| lucasagomes | tuesday* | 17:23 |
| lucasagomes | jlvillal, night :-) | 17:23 |
| *** lucasagomes is now known as lucas-afk | 17:23 | |
| jlvillal | vsaienk0: Not sure why. I just ran two more tests. One with 'agent_ssh' (works) and one with 'agent_ipmitool' (fails). Which I agree is strange since the multi-node multi-tenant job does use 'agent_ipmitool' and works... | 17:24 |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/423270 | 17:24 |
| *** jcoufal has quit IRC | 17:30 | |
| *** jcoufal has joined #openstack-ironic | 17:31 | |
| *** jaosorior has joined #openstack-ironic | 17:32 | |
| *** aparnav has joined #openstack-ironic | 17:36 | |
| *** aparnav has quit IRC | 17:44 | |
| *** baoli_ has quit IRC | 17:45 | |
| *** athomas has quit IRC | 17:45 | |
| *** baoli has joined #openstack-ironic | 17:46 | |
| *** jcoufal has quit IRC | 17:49 | |
| *** mgoddard has quit IRC | 17:52 | |
| *** sacharya has quit IRC | 17:54 | |
| openstackgerrit | Miles Gould proposed openstack/ironic: Allow setting UEFI boot devices in DRAC driver https://review.openstack.org/420107 | 17:54 |
| *** mgould|a1k is now known as mgould|afk | 17:54 | |
| *** tovin07 has quit IRC | 17:56 | |
| *** jcoufal has joined #openstack-ironic | 17:57 | |
| *** dprince has joined #openstack-ironic | 18:01 | |
| TheJulia | jroll: http://paste.openstack.org/show/gvzwoPPNZQ1UGPtsVerQ/ I feel like I need a beer | 18:06 |
| *** e0ne has joined #openstack-ironic | 18:06 | |
| jroll | wow, that's odd | 18:07 |
| *** tovin07 has joined #openstack-ironic | 18:12 | |
| *** jralph has quit IRC | 18:14 | |
| openstackgerrit | Jesse J. Cook proposed openstack/ironic: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 18:15 |
| *** jcoufal_ has joined #openstack-ironic | 18:17 | |
| *** jcoufal has quit IRC | 18:19 | |
| *** jcoufal has joined #openstack-ironic | 18:20 | |
| *** jcoufal_ has quit IRC | 18:22 | |
| *** sayantani01 has quit IRC | 18:25 | |
| *** ccamacho has quit IRC | 18:26 | |
| *** mgoddard has joined #openstack-ironic | 18:28 | |
| *** rama_y has joined #openstack-ironic | 18:33 | |
| *** rama_y_ has joined #openstack-ironic | 18:33 | |
| *** sacharya has joined #openstack-ironic | 18:34 | |
| *** e0ne has quit IRC | 18:35 | |
| *** jaosorior has quit IRC | 18:35 | |
| *** trown is now known as trown|lunch | 18:35 | |
| *** rama_y has quit IRC | 18:37 | |
| *** jralph has joined #openstack-ironic | 18:37 | |
| *** rama_y_ has quit IRC | 18:37 | |
| *** rama_y has joined #openstack-ironic | 18:38 | |
| *** hecliunyx has quit IRC | 18:40 | |
| *** oomichi has joined #openstack-ironic | 18:42 | |
| *** jcoufal_ has joined #openstack-ironic | 18:43 | |
| *** ccamacho has joined #openstack-ironic | 18:45 | |
| *** rpioso has quit IRC | 18:45 | |
| *** jcoufal has quit IRC | 18:46 | |
| *** ppiela has quit IRC | 18:47 | |
| *** hecliunyx has joined #openstack-ironic | 18:50 | |
| openstackgerrit | Jesse J. Cook proposed openstack/ironic: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 18:54 |
| *** rpioso has joined #openstack-ironic | 18:55 | |
| *** sacharya has quit IRC | 19:02 | |
| *** jcoufal_ has quit IRC | 19:04 | |
| aslezil | "https://review.openstack.org/#/c/412594/" This is bug fix which now poweroff/reboot the coreOS IPA image in chroot. I would be appreciated if any core reviewers could review it. The patch have lot of +1 on it. | 19:05 |
| patchbot | patch 412594 - ironic-python-agent - Reboot and Poweroff fails with coreos IPA image | 19:05 |
| *** vsaienko has joined #openstack-ironic | 19:06 | |
| *** jcoufal has joined #openstack-ironic | 19:06 | |
| JayF | ^ that patch is good if someone can take a look, I withheld my +2 because I helped with it | 19:09 |
| JayF | mikal: ^ that is the patch to fix your pet bug (coreos reboot/shutdown) | 19:11 |
| *** rama_y has quit IRC | 19:14 | |
| *** sayantani01 has joined #openstack-ironic | 19:17 | |
| jroll | aslezil: a couple comments inline | 19:17 |
| JayF | jroll: for some reason, the side effect array stuff didn't work in the tests | 19:19 |
| aslezil | jroll: Thanks :) I will fix it jroll and upload a patch set now | 19:19 |
| jroll | JayF: do you know which patchset that was? | 19:19 |
| JayF | OH! I think I just figured it out | 19:19 |
| JayF | jroll: maybe local only? I was pairing w/ aslezil on it | 19:20 |
| jroll | should be side_effect = (('', ''), ('', 'ignoring ...'), ('', '')) | 19:20 |
| jroll | ah | 19:20 |
| JayF | yeah, | 19:20 |
| JayF | I think we only did 2 in the array before | 19:20 |
| jroll | ah | 19:20 |
| jroll | that'll do it | 19:20 |
| JayF | because I forgot that sync ran | 19:20 |
| JayF | and now looking at this it's more clear | 19:20 |
| jroll | :) | 19:20 |
| *** dsneddon_afk is now known as dsneddon | 19:21 | |
| *** baoli has quit IRC | 19:25 | |
| openstackgerrit | Joanna Taryma proposed openstack/ironic: Fixed allowing non-UUID value as UUID for port and portgroup patches. https://review.openstack.org/403910 | 19:28 |
| *** jcoufal_ has joined #openstack-ironic | 19:30 | |
| *** rama_y has joined #openstack-ironic | 19:31 | |
| *** tesseract has quit IRC | 19:32 | |
| *** jcoufal has quit IRC | 19:32 | |
| *** sambetts is now known as sambetts|afk | 19:36 | |
| *** mgoddard has quit IRC | 19:36 | |
| *** trown|lunch is now known as trown | 19:36 | |
| openstackgerrit | Anup Navare proposed openstack/ironic: Config drive support for Ceph https://review.openstack.org/407639 | 19:42 |
| *** vsaienko has quit IRC | 19:43 | |
| *** Sukhdev has joined #openstack-ironic | 19:47 | |
| *** jcoufal_ has quit IRC | 19:49 | |
| *** Nisha_Agarwal has quit IRC | 19:50 | |
| *** jaosorior has joined #openstack-ironic | 19:52 | |
| *** dsneddon is now known as dsneddon_afk | 19:54 | |
| *** jcoufal has joined #openstack-ironic | 19:54 | |
| JayF | soliosg: so basically if you want a mock to return a different value each time it's called, you can use mock_object.side_effect = [], where the array is taken as the return codes in roder, i.e. [0] is the return value first time it's called, [1] is the return value the second time, and so on | 20:00 |
| *** moshele has joined #openstack-ironic | 20:08 | |
| JayF | our gate is broken | 20:12 |
| JayF | the api-ref release thing mentioned in the mailing list is impacting us | 20:12 |
| *** hecliunyx has quit IRC | 20:12 | |
| JayF | because our api-ref has some warnings in it now, and we bail on warnings | 20:12 |
| jroll | wooooo | 20:12 |
| JayF | I'm going to push up a quick patch to remove the "no warnings" flag | 20:12 |
| jroll | thanks sdague | 20:12 |
| jroll | how many warnings do we have? | 20:12 |
| JayF | 2017-01-20 18:20:26.375363 | Warning, treated as error: | 20:13 |
| JayF | 2017-01-20 18:20:26.375414 | /home/jenkins/workspace/gate-ironic-api-ref/api-ref/source/api-ref/source/baremetal-api-v1-nodes-vifs.inc:24 .rst:: WARNING: No path parameter ``node_ident`` found in rest_parameter stanza. | 20:13 |
| JayF | but tbh, I have no idea whatsoever if it would print all of them | 20:13 |
| JayF | or just the first one | 20:13 |
| *** aNuposic has quit IRC | 20:13 | |
| * jroll checks | 20:13 | |
| JayF | I'm assuming your thought is, fix the warnings | 20:14 |
| jroll | yes | 20:14 |
| jroll | it might be trivial | 20:14 |
| JayF | I'm giong to push my patch up, lets get it landed, then if we can followup with re-enabling the -W and fixing the warnings, bonus | 20:14 |
| jroll | sure, go ahead | 20:14 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Turn NOTE into docstring https://review.openstack.org/422863 | 20:14 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: API changes for driver-composition https://review.openstack.org/417970 | 20:15 |
| JayF | https://bugs.launchpad.net/ironic/+bug/1658187 | 20:18 |
| openstack | Launchpad bug 1658187 in Ironic "api-ref generation now has warnings" [Undecided,New] | 20:18 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Do not treat api-ref warnings as errors https://review.openstack.org/423505 | 20:18 |
| JayF | jroll: and any other available core ^^ | 20:18 |
| JayF | I'm going to update the whiteboard | 20:18 |
| JayF | whiteboard updated | 20:19 |
| *** hecliunyx has joined #openstack-ironic | 20:20 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector: Add plugin to process basic management LLDP TLVs https://review.openstack.org/406496 | 20:20 |
| * jroll waits for CI | 20:20 | |
| JayF | btw jroll, I made that bug explicitly so it won't be resolved until we fix the warnings + re-enable -W as well | 20:22 |
| jroll | cool | 20:22 |
| *** hecliunyx has quit IRC | 20:26 | |
| *** amoralej is now known as amoralej|off | 20:27 | |
| jroll | JayF: +A | 20:28 |
| *** srobert_ has quit IRC | 20:29 | |
| jroll | there's a few warnings here, rolling through them now | 20:29 |
| *** srobert has joined #openstack-ironic | 20:30 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Fix api-ref warnings https://review.openstack.org/423509 | 20:31 |
| jroll | actually, -2'd your thing | 20:31 |
| jroll | ^ that should fix things | 20:31 |
| JayF | you already put it in the gate? | 20:31 |
| jroll | right, so I -2'd to not let it merge, since we can land the real fix without having to wait or rebase around | 20:31 |
| jroll | only costs us 5 minutes ¯\_(ツ)_/¯ | 20:32 |
| JayF | okie. I'll wait for CI and vote on yours. | 20:32 |
| jroll | or I can let it merge, I don't mind, just more work | 20:32 |
| JayF | I thought changing a vote after you W+1 | 20:32 |
| JayF | still ran all the gate jobs | 20:32 |
| jroll | it does | 20:32 |
| JayF | and just stopped it at the very last minute | 20:32 |
| jroll | but it checks again before merge | 20:32 |
| *** srobert_ has joined #openstack-ironic | 20:33 | |
| *** hecliunyx has joined #openstack-ironic | 20:34 | |
| *** srobert has quit IRC | 20:36 | |
| *** aNuposic has joined #openstack-ironic | 20:38 | |
| *** jcoufal_ has joined #openstack-ironic | 20:39 | |
| *** jcoufal has quit IRC | 20:42 | |
| JayF | anyone around to look at https://review.openstack.org/#/c/423509/ with me? | 20:43 |
| patchbot | patch 423509 - ironic - Fix api-ref warnings | 20:43 |
| JayF | otherwise I'm going to land it with only +2 since it fixes the gate + is docs only | 20:43 |
| *** jcoufal has joined #openstack-ironic | 20:43 | |
| jroll | :P | 20:44 |
| jroll | TheJulia: ^^ | 20:44 |
| * JayF jfdi | 20:44 | |
| TheJulia | I just added my +2 | 20:45 |
| jroll | sweet, thanks y'all | 20:45 |
| jroll | nice work catching that JayF | 20:45 |
| *** jcoufal__ has joined #openstack-ironic | 20:45 | |
| JayF | a patch I was reviewing failed CI, and I saw it was only the api-ref job | 20:45 |
| JayF | remembered that post and cursed under my breath | 20:46 |
| JayF | lol. | 20:46 |
| *** jcoufal_ has quit IRC | 20:46 | |
| *** vsaienko has joined #openstack-ironic | 20:46 | |
| TheJulia | heh | 20:46 |
| jroll | nice :P | 20:47 |
| *** ijw has quit IRC | 20:48 | |
| *** jcoufal has quit IRC | 20:48 | |
| openstackgerrit | Merged openstack/ironic: Fix api-ref warnings https://review.openstack.org/423509 | 20:51 |
| JayF | ^^^ gate fixed, hurrah | 20:52 |
| *** vsaienko has quit IRC | 20:52 | |
| jroll | huzzah | 20:53 |
| TheJulia | So, long story short, setuptools>=31.0.0 apparently does the wrong thing and breaks stable/mitaka installs of openstacksdk. Trying to reproduce the ironic-lib failure, newer setuptools seems to keep getting sucked in. Taking one more try at it. :\ | 20:55 |
| JayF | even if you fix that, we still have to skip the docs job on stable/mitaka in project-config, right? | 20:56 |
| * JayF starting to wonder if it's worth it to fix ironic-lib mitaka ci | 20:56 | |
| TheJulia | yup | 20:56 |
| TheJulia | JayF: I'm thinking not :\ It is in security fixes phase though | 20:56 |
| JayF | jroll: ^ wdyt | 20:56 |
| TheJulia | EOL on April ?10th? | 20:56 |
| openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent: Reboot and Poweroff fails with coreos IPA image https://review.openstack.org/412594 | 20:57 |
| *** jaosorior has quit IRC | 20:57 | |
| JayF | something like that | 20:57 |
| jroll | TheJulia: JayF: I think it's worth it, this is essentially a security fix | 20:58 |
| jroll | I'd hit the mailing list about the current problem though, that must break more than just us | 20:58 |
| TheJulia | I found an open bug filed last month against openstacksdk, not triaged :\ | 20:59 |
| jroll | ping dtroyer / stevemar maybe? | 20:59 |
| TheJulia | but it is likely totally unrelated to the failure we're seeing in CI | 20:59 |
| jroll | hrm | 20:59 |
| * TheJulia double checks | 20:59 | |
| TheJulia | heh, pip freeze list doesn't list setuptools | 21:00 |
| stevemar | jroll: o/ | 21:02 |
| jroll | ohai | 21:03 |
| stevemar | was getting food | 21:03 |
| jroll | now that I go to spell this out, I think I have it backwards, we blame setuptools, not openstacksdk | 21:03 |
| jroll | stevemar: how dare you take a whole 4 minutes to respond | 21:03 |
| *** sayantani01 has quit IRC | 21:06 | |
| stevemar | jroll: woo hoo | 21:07 |
| TheJulia | well, blame it for the loss of every ounce of my remaining sanity | 21:07 |
| TheJulia | time will tell if devstack does the right thing on my other laptop | 21:07 |
| * TheJulia wonders "Why do I not have a squid cache setup locally..." | 21:07 | |
| *** rbudden has quit IRC | 21:08 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic-python-agent: WIP: IPA rescue extension https://review.openstack.org/423521 | 21:10 |
| mariojv | JayF: ^ I just put up a new patch set, we can merge the 2 later if needed / wanted | 21:10 |
| JayF | mariojv: wfm, you might wanna comment in that other patchset about it | 21:11 |
| mariojv | done | 21:14 |
| *** sayantani01 has joined #openstack-ironic | 21:15 | |
| *** jcoufal has joined #openstack-ironic | 21:17 | |
| *** jcoufal__ has quit IRC | 21:19 | |
| *** moshele has quit IRC | 21:21 | |
| TheJulia | woot, yeah, latest setuptools got installed | 21:30 |
| * TheJulia wonders if now is a good time to call it the day | 21:30 | |
| *** jcoufal has quit IRC | 21:32 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Testing multinode job https://review.openstack.org/422149 | 21:34 |
| *** jcoufal has joined #openstack-ironic | 21:34 | |
| *** jerrygb_ has quit IRC | 21:41 | |
| *** ijw has joined #openstack-ironic | 21:42 | |
| *** rajinir has joined #openstack-ironic | 21:42 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Agent driver rescue implementation https://review.openstack.org/400437 | 21:46 |
| soliosg | JayF: thanks for explanation on mock_object.side_effect | 21:49 |
| JayF | soliosg: no problem man, anytime. Mock is really hard to grok until you see all the various things in use, even if you read t he docs. | 21:50 |
| *** jcoufal has quit IRC | 21:51 | |
| *** rama_y has quit IRC | 21:53 | |
| JayF | jroll: I think aslezil addressed your comments well here: https://review.openstack.org/#/c/412594/23 if you wanna re-review :) | 21:54 |
| patchbot | patch 412594 - ironic-python-agent - Reboot and Poweroff fails with coreos IPA image | 21:54 |
| jroll | uno momento | 21:54 |
| aslezil | :) | 21:54 |
| *** ChubYann has joined #openstack-ironic | 21:54 | |
| *** mriedem has joined #openstack-ironic | 21:58 | |
| mriedem | jroll: or someone, can you tell me which of these is the real patch? https://review.openstack.org/#/q/topic:bp/soft-reboot-poweroff | 21:58 |
| mriedem | see the 2 at the top, one is merge conflict and the other has my +2 | 21:58 |
| mriedem | oh nvm | 21:59 |
| mriedem | power off vs reboot | 21:59 |
| jroll | answered without talking, damn I'm good | 21:59 |
| jroll | aslezil: +2, thanks | 21:59 |
| jroll | mriedem: thanks for staying on top of that stuff | 22:00 |
| aslezil | thanks jroll: | 22:00 |
| aslezil | Thanks TheJulia :) | 22:03 |
| JayF | \o/ | 22:07 |
| *** ccamacho has quit IRC | 22:13 | |
| *** rbudden has joined #openstack-ironic | 22:16 | |
| *** trown is now known as trown|outtypewww | 22:17 | |
| jroll | dtantsur|afk: this vendor interface thing is terrible, thanks to mixins. we either need to interface-ify the vendor mixins or change up the vendor class. the first two api patches are ready though, I think. | 22:23 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: support dynamic drivers in driver properties API https://review.openstack.org/423536 | 22:25 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: vendor passthru for hw types https://review.openstack.org/423537 | 22:25 |
| jroll | well, they still need a decent commit message, release notes, some docs | 22:25 |
| jroll | but yeah | 22:25 |
| *** bfournie has quit IRC | 22:26 | |
| *** vsaienko has joined #openstack-ironic | 22:28 | |
| *** lindycoder_ has quit IRC | 22:32 | |
| openstackgerrit | Eric Xie proposed openstack/python-ironicclient: Fix ImportError when providing a meaningless API version. https://review.openstack.org/423333 | 22:33 |
| *** jheroux has quit IRC | 22:34 | |
| *** vsaienko has quit IRC | 22:34 | |
| *** mriedem has quit IRC | 22:39 | |
| *** jerrygb has joined #openstack-ironic | 22:40 | |
| *** vsaienko has joined #openstack-ironic | 22:44 | |
| *** srobert_ has quit IRC | 22:46 | |
| *** srobert has joined #openstack-ironic | 22:46 | |
| *** srobert has quit IRC | 22:51 | |
| *** dprince has quit IRC | 22:52 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Reboot and Poweroff fails with coreos IPA image https://review.openstack.org/412594 | 22:53 |
| *** [1]cdearborn has quit IRC | 23:25 | |
| *** sayantani01 has quit IRC | 23:26 | |
| *** aNuposic has quit IRC | 23:30 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to use no nova installation https://review.openstack.org/420688 | 23:50 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests https://review.openstack.org/423556 | 23:50 |
| *** sacharya has joined #openstack-ironic | 23:58 | |
| *** vsaienko has quit IRC | 23:59 | |
| *** aNuposic has joined #openstack-ironic | 23:59 | |
| *** rama_y has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!