| *** absubram has quit IRC | 00:02 | |
| *** krtaylor has joined #openstack-ironic | 00:04 | |
| *** godp1301_ has joined #openstack-ironic | 00:05 | |
| *** godp1301 has quit IRC | 00:07 | |
| *** godp1301 has joined #openstack-ironic | 00:07 | |
| *** chas has quit IRC | 00:09 | |
| *** chas has joined #openstack-ironic | 00:09 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add storage_interface to base driver class https://review.openstack.org/348006 | 00:10 |
|---|---|---|
| *** godp1301_ has quit IRC | 00:10 | |
| openstackgerrit | Merged openstack/python-ironicclient: Add basic tests for OSC plugin baremetal node commands https://review.openstack.org/301687 | 00:11 |
| *** godp1301 has quit IRC | 00:12 | |
| *** chas has quit IRC | 00:14 | |
| *** causten has quit IRC | 00:17 | |
| *** moshele has quit IRC | 00:22 | |
| *** rbrud has joined #openstack-ironic | 00:24 | |
| *** ElCoyote_ has joined #openstack-ironic | 00:26 | |
| *** david-lyle_ is now known as david-lyle | 00:35 | |
| *** rajinir has quit IRC | 00:36 | |
| *** hoangcx has joined #openstack-ironic | 00:37 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Trying to test grenade multi-tenant experimental job https://review.openstack.org/389268 | 00:39 |
| *** bfournie has quit IRC | 00:50 | |
| *** phuongnh has joined #openstack-ironic | 00:58 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: [WIP] Add base cinder common interface https://review.openstack.org/355625 | 01:10 |
| *** godp1301 has joined #openstack-ironic | 01:13 | |
| *** godp1301 has quit IRC | 01:17 | |
| *** bfournie has joined #openstack-ironic | 01:21 | |
| *** Sukhdev has quit IRC | 01:24 | |
| *** shobhar_ has joined #openstack-ironic | 01:25 | |
| *** shobhar_ has quit IRC | 01:29 | |
| *** aNuposic has joined #openstack-ironic | 01:32 | |
| *** zhangjl has joined #openstack-ironic | 01:35 | |
| *** rama_y has joined #openstack-ironic | 01:44 | |
| *** zhangyufei has joined #openstack-ironic | 01:45 | |
| *** tuanluong has joined #openstack-ironic | 02:06 | |
| *** hoangcx has quit IRC | 02:23 | |
| *** rloo has quit IRC | 02:26 | |
| *** aNuposic has quit IRC | 02:28 | |
| *** shuquan has joined #openstack-ironic | 02:30 | |
| *** nicodemos has quit IRC | 02:38 | |
| *** nicodemos has joined #openstack-ironic | 02:38 | |
| *** g3ek has quit IRC | 02:38 | |
| *** shuquan_ has joined #openstack-ironic | 02:40 | |
| *** shuquan has quit IRC | 02:41 | |
| *** fragatin_ has joined #openstack-ironic | 02:47 | |
| *** g3ek has joined #openstack-ironic | 02:48 | |
| *** fragatina has quit IRC | 02:50 | |
| *** fragatin_ has quit IRC | 02:51 | |
| *** fragatina has joined #openstack-ironic | 02:57 | |
| *** fragatina has quit IRC | 02:58 | |
| *** shuquan_ has quit IRC | 03:01 | |
| *** shuquan has joined #openstack-ironic | 03:02 | |
| *** fragatina has joined #openstack-ironic | 03:24 | |
| *** ijw has quit IRC | 03:27 | |
| *** ijw has joined #openstack-ironic | 03:28 | |
| *** shuquan has quit IRC | 03:29 | |
| *** fragatina has quit IRC | 03:29 | |
| *** vikrant has joined #openstack-ironic | 03:31 | |
| *** ijw has quit IRC | 03:33 | |
| *** Nisha has joined #openstack-ironic | 03:34 | |
| *** Nisha is now known as Guest57346 | 03:35 | |
| *** shuquan has joined #openstack-ironic | 03:35 | |
| *** Guest57346 has quit IRC | 03:39 | |
| *** shuquan has quit IRC | 03:47 | |
| *** shuquan_ has joined #openstack-ironic | 03:47 | |
| *** aNuposic has joined #openstack-ironic | 03:56 | |
| *** yonglihe has quit IRC | 03:56 | |
| *** aNuposic has quit IRC | 04:19 | |
| *** hoangcx has joined #openstack-ironic | 04:24 | |
| *** shuquan_ has quit IRC | 04:40 | |
| *** shuquan has joined #openstack-ironic | 05:06 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Use function is_valid_mac from oslo.utils https://review.openstack.org/389513 | 05:06 |
| *** vmud213 has joined #openstack-ironic | 05:10 | |
| *** shuquan has quit IRC | 05:14 | |
| *** vmud213 has quit IRC | 05:14 | |
| *** vmud213 has joined #openstack-ironic | 05:15 | |
| *** shuquan has joined #openstack-ironic | 05:17 | |
| *** vmud213 has quit IRC | 05:17 | |
| *** vmud213 has joined #openstack-ironic | 05:18 | |
| *** sabeen has quit IRC | 05:27 | |
| *** shuquan_ has joined #openstack-ironic | 05:34 | |
| *** shuquan has quit IRC | 05:37 | |
| *** fragatina has joined #openstack-ironic | 05:47 | |
| *** amotoki has joined #openstack-ironic | 05:51 | |
| *** fragatina has quit IRC | 05:52 | |
| *** moshele has joined #openstack-ironic | 05:53 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Timing metrics: iRMC drivers https://review.openstack.org/371284 | 05:53 |
| *** chas has joined #openstack-ironic | 05:54 | |
| *** chas has quit IRC | 06:01 | |
| *** shuquan has joined #openstack-ironic | 06:05 | |
| *** shuquan_ has quit IRC | 06:05 | |
| *** chas has joined #openstack-ironic | 06:06 | |
| *** chas has quit IRC | 06:11 | |
| *** jaosorior has joined #openstack-ironic | 06:14 | |
| *** vmud213 has quit IRC | 06:16 | |
| *** Sukhdev has joined #openstack-ironic | 06:27 | |
| *** hw_wutianwei has joined #openstack-ironic | 06:28 | |
| *** moshele has quit IRC | 06:30 | |
| *** rbrud has quit IRC | 06:37 | |
| *** ChubYann has quit IRC | 06:38 | |
| *** chas has joined #openstack-ironic | 06:38 | |
| *** jaosorior has quit IRC | 06:41 | |
| *** jaosorior has joined #openstack-ironic | 06:42 | |
| *** chas has quit IRC | 06:43 | |
| *** hshiina has joined #openstack-ironic | 06:48 | |
| *** fragatina has joined #openstack-ironic | 06:49 | |
| *** moshele has joined #openstack-ironic | 06:53 | |
| *** fragatina has quit IRC | 06:54 | |
| openstackgerrit | octopuszhang proposed openstack/ironic-inspector: add ipa-api-url pxe default config https://review.openstack.org/398775 | 06:59 |
| *** vmud213 has joined #openstack-ironic | 07:00 | |
| openstackgerrit | octopuszhang proposed openstack/ironic-inspector: add ipa-api-url in pxe default config https://review.openstack.org/398775 | 07:01 |
| *** fxpester has joined #openstack-ironic | 07:04 | |
| *** aNuposic has joined #openstack-ironic | 07:07 | |
| *** aNuposic has quit IRC | 07:09 | |
| *** shuquan has quit IRC | 07:09 | |
| *** mjura has joined #openstack-ironic | 07:13 | |
| *** mjura has quit IRC | 07:17 | |
| *** rbartal has joined #openstack-ironic | 07:18 | |
| *** pcaruana has joined #openstack-ironic | 07:18 | |
| *** vsaienko has joined #openstack-ironic | 07:19 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update iptables rules and services IPs for multinode https://review.openstack.org/368648 | 07:19 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add devstack setup_vxlan_network() https://review.openstack.org/368647 | 07:19 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Skip some steps for multinode case https://review.openstack.org/368646 | 07:19 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 07:19 |
| *** shuquan has joined #openstack-ironic | 07:25 | |
| *** mjura has joined #openstack-ironic | 07:29 | |
| vsaienk0 | morning dtantsur, I think they are skipped because of https://review.openstack.org/#/c/355591/13/ironic_tempest_plugin/tests/scenario/baremetal_manager.py | 07:29 |
| patchbot | patch 355591 - ironic - Bring configurations from tempest to ironic_tempes... (MERGED) | 07:29 |
| vsaienk0 | dtantsur: InspectorScenarioTest doesn't redefine skip_ckeck() method so tempest uses this check https://review.openstack.org/#/c/355591/13/ironic_tempest_plugin/tests/scenario/baremetal_manager.py@61 | 07:31 |
| patchbot | patch 355591 - ironic - Bring configurations from tempest to ironic_tempes... (MERGED) | 07:31 |
| *** Sukhdev has quit IRC | 07:38 | |
| *** SerenaFeng has joined #openstack-ironic | 07:45 | |
| *** athomas has joined #openstack-ironic | 07:57 | |
| *** vsaienko has quit IRC | 08:11 | |
| *** jroll has quit IRC | 08:27 | |
| *** moshele has quit IRC | 08:34 | |
| *** moshele has joined #openstack-ironic | 08:34 | |
| *** moshele has quit IRC | 08:39 | |
| *** jroll has joined #openstack-ironic | 08:40 | |
| *** moshele has joined #openstack-ironic | 08:41 | |
| *** moshele has quit IRC | 08:42 | |
| *** SerenaFeng has quit IRC | 08:43 | |
| *** rama_y has quit IRC | 08:43 | |
| *** rama_y has joined #openstack-ironic | 08:44 | |
| tuanluong | morning ironic members | 08:45 |
| tuanluong | mornig vsaienk0, dtantsur | 08:45 |
| *** lennyb has quit IRC | 08:45 | |
| vsaienk0 | morning dtantsur, sambetts, lucasagomes, mgould, tuanluong, pas-ha, vdrok, aarefiev and all Ironic'ers! | 08:46 |
| galyna | morning, all! :) | 08:48 |
| *** jpich has joined #openstack-ironic | 08:49 | |
| *** shuquan has quit IRC | 08:51 | |
| vsaienk0 | morning galyna! | 08:51 |
| *** ralonsoh has joined #openstack-ironic | 08:51 | |
| *** lennyb has joined #openstack-ironic | 08:53 | |
| *** moshele has joined #openstack-ironic | 08:53 | |
| *** shuquan has joined #openstack-ironic | 08:57 | |
| *** SerenaFeng has joined #openstack-ironic | 08:59 | |
| *** lucas-afk is now known as lucasagomes | 08:59 | |
| lucasagomes | morning all | 08:59 |
| *** zzzeek has quit IRC | 09:00 | |
| *** Nisha has joined #openstack-ironic | 09:00 | |
| *** Nisha is now known as Guest30737 | 09:00 | |
| *** Guest30737 is now known as Nisha_ | 09:01 | |
| *** zzzeek has joined #openstack-ironic | 09:01 | |
| hshiina | morning ironic | 09:05 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Remove deprecated DHCP provider methods https://review.openstack.org/398861 | 09:05 |
| *** betherly_ has joined #openstack-ironic | 09:07 | |
| *** jaosorior is now known as jaosorior_lunch | 09:08 | |
| *** fragatina has joined #openstack-ironic | 09:12 | |
| *** fragatina has quit IRC | 09:18 | |
| tuanluong | hello vsaienk0, | 09:18 |
| tuanluong | I am trying portgroup-create in new api 2.14 | 09:19 |
| tuanluong | ironic portgroup-create: error: argument -a/--address is required | 09:19 |
| tuanluong | "ironic --ironic-api-version 1.24 portgroup-create" | 09:20 |
| vsaienk0 | tuanluong --address and --node are mandatory fields | 09:24 |
| tuanluong | Is that correct when we have to give address for create portgroup? Because I saw in spec don't have | 09:24 |
| tuanluong | should we should update specs? https://review.openstack.org/#/c/396610/5/specs/approved/ironic-ml2-integration.rst,edit Line 1098 | 09:27 |
| patchbot | patch 396610 - ironic-specs - Add bonding configuration fields to portgroups | 09:27 |
| *** roeyc has joined #openstack-ironic | 09:31 | |
| vdrok | good morning ironic, vsaienk0 galyna lucasagomes and hshiina | 09:32 |
| openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Dispatched out network interface info to all hardware managers https://review.openstack.org/398885 | 09:33 |
| openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Add a new Hardware Manager for Mellanox NICs https://review.openstack.org/398886 | 09:33 |
| openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Add a new Hardware Manager for Mellanox NICs https://review.openstack.org/334816 | 09:35 |
| vsaienk0 | tuanluong: we are going to update the spec https://review.openstack.org/#/c/396610/ I've left comment there | 09:35 |
| patchbot | patch 396610 - ironic-specs - Add bonding configuration fields to portgroups | 09:35 |
| vdrok | lucasagomes: a request for you, could you approve https://bugs.launchpad.net/ironic/+bug/1641972 ? pretty straight-forward | 09:35 |
| openstack | Launchpad bug 1641972 in Ironic "[RFE] Add domains to ironic context" [Undecided,In progress] - Assigned to Vladyslav Drok (vdrok) | 09:35 |
| tuanluong | vsaienk0, Thanks, Could you explaint to me why we need to --address and if needed how can we have -address? | 09:37 |
| tuanluong | morning vdrok | 09:38 |
| vdrok | good morning tuanluong | 09:38 |
| vsaienk0 | tuanluong: bond interface as regular network interface in system should have an address, portgroup in Ironic is an representation of bond interface in system, for example ironic port is a representation of regular (Ethernet) network interface. | 09:41 |
| *** Nisha_ has quit IRC | 09:41 | |
| *** SerenaFeng has quit IRC | 09:42 | |
| *** jaosorior_lunch is now known as jaosorior | 09:43 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Followup to 0335e81a8787 https://review.openstack.org/398515 | 09:47 |
| *** openstackgerrit has quit IRC | 09:48 | |
| *** openstackgerrit has joined #openstack-ironic | 09:49 | |
| lucasagomes | vdrok, hi there, yes will take a look | 09:54 |
| lucasagomes | vdrok, I was coming to the office, sorry for the delay | 09:55 |
| *** betherly has quit IRC | 09:55 | |
| lucasagomes | vdrok, done :-) yeah it's pretty straight forward | 09:55 |
| *** betherly_ is now known as betherly | 09:56 | |
| vdrok | lucasagomes: no problem at all :) thanks! | 09:56 |
| *** betherly is now known as betherly_ | 09:56 | |
| *** betherly_ is now known as betherly | 09:56 | |
| *** derekh has joined #openstack-ironic | 09:59 | |
| *** moshele has left #openstack-ironic | 10:13 | |
| *** moshele has joined #openstack-ironic | 10:13 | |
| *** e0ne has joined #openstack-ironic | 10:15 | |
| *** hoangcx has quit IRC | 10:17 | |
| *** hshiina has quit IRC | 10:19 | |
| *** sambetts|afk is now known as sambetts | 10:23 | |
| *** zhangyufei has quit IRC | 10:23 | |
| *** zhangyufei has joined #openstack-ironic | 10:25 | |
| *** zhangjl has quit IRC | 10:31 | |
| *** nicodemos has quit IRC | 10:34 | |
| *** nicodemos has joined #openstack-ironic | 10:34 | |
| *** fragatina has joined #openstack-ironic | 10:42 | |
| *** strigazi_AFK is now known as strigazi | 10:42 | |
| openstackgerrit | Merged openstack/ironic-ui: Imported Translations from Zanata https://review.openstack.org/397588 | 10:44 |
| *** fragatina has quit IRC | 10:47 | |
| *** jtomasek has quit IRC | 10:47 | |
| *** shuquan has quit IRC | 10:49 | |
| *** vsaienko has joined #openstack-ironic | 10:50 | |
| *** e0ne has quit IRC | 10:52 | |
| *** e0ne has joined #openstack-ironic | 10:55 | |
| *** vsaienko has quit IRC | 10:56 | |
| *** phuongnh has quit IRC | 10:58 | |
| *** khamtamtun has joined #openstack-ironic | 11:06 | |
| *** fragatina has joined #openstack-ironic | 11:07 | |
| openstackgerrit | Stephen Finucane proposed openstack/ironic: Remove ClusteredComputeManager https://review.openstack.org/397762 | 11:08 |
| *** vsaienko has joined #openstack-ironic | 11:11 | |
| *** fragatina has quit IRC | 11:12 | |
| openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Dispatched out network interface info to all hardware managers https://review.openstack.org/398885 | 11:12 |
| openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Add a new Hardware Manager for Mellanox NICs https://review.openstack.org/334816 | 11:12 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: Support ETAGs in Ironic CLI https://review.openstack.org/398933 | 11:12 |
| *** khamtamtun has quit IRC | 11:12 | |
| *** kamtamtun has joined #openstack-ironic | 11:14 | |
| mrtenio | Good Morning Ironic. | 11:21 |
| mrtenio | lucasagomes, vdrok, could you rereview this patch? Thank you https://review.openstack.org/#/c/377073/ | 11:21 |
| patchbot | patch 377073 - ironic - Adds another validation step when using dynamic al... | 11:21 |
| vdrok | morning mrtenio, will look today | 11:22 |
| lucasagomes | mrtenio, added to the list, I will take a look later on | 11:22 |
| mrtenio | vdrok, lucasagome, thank you very much. | 11:24 |
| mrtenio | lucasagomes* :P | 11:25 |
| sambetts | lucasagomes: do you know what our stance on doing bulk actions in the python client is? | 11:27 |
| sambetts | lucasagomes: e.g node-delete <uuid <uuid> <uuid> | 11:27 |
| *** roeyc has quit IRC | 11:29 | |
| lucasagomes | sambetts, not sure... I think delete already worked right ? | 11:29 |
| sambetts | lucasagomes: I don't know I've never tried, I'm just reviewing a spec which is adding a command to bulk delete portgroups and wasn't sure if we allowed it for anything else | 11:30 |
| lucasagomes | sambetts, I see... so AFAIR, node-delete might be the only one that we support for multiple nodes | 11:32 |
| *** athomas has quit IRC | 11:32 | |
| sambetts | lucasagomes: ah ok /me wonders if we should support it or not then | 11:33 |
| lucasagomes | sambetts, I haven't read the spec, but, it makes sense to allow it | 11:34 |
| sambetts | cool :) | 11:37 |
| *** jkilpatr has quit IRC | 11:41 | |
| *** Nisha_ has joined #openstack-ironic | 11:41 | |
| *** kamtamtun has quit IRC | 11:45 | |
| *** kamtamtun has joined #openstack-ironic | 11:52 | |
| *** kamtamtun has quit IRC | 11:53 | |
| *** roeyc has joined #openstack-ironic | 11:57 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs: Add bonding configuration fields to portgroups https://review.openstack.org/396610 | 11:57 |
| *** jkilpatr has joined #openstack-ironic | 12:00 | |
| *** vsaienko has quit IRC | 12:00 | |
| *** vsaienko has joined #openstack-ironic | 12:01 | |
| *** zhugaoxiao has quit IRC | 12:01 | |
| *** zhugaoxiao has joined #openstack-ironic | 12:01 | |
| *** betherly has quit IRC | 12:03 | |
| *** tuanluong has quit IRC | 12:04 | |
| *** kamtamtun has joined #openstack-ironic | 12:04 | |
| *** kamtamtun has quit IRC | 12:05 | |
| *** betherly has joined #openstack-ironic | 12:05 | |
| *** jkilpatr has quit IRC | 12:06 | |
| *** fragatina has joined #openstack-ironic | 12:08 | |
| *** zhangyufei has quit IRC | 12:09 | |
| *** jkilpatr has joined #openstack-ironic | 12:11 | |
| *** leduardo has joined #openstack-ironic | 12:11 | |
| vdrok | morning sambetts | 12:12 |
| *** fragatina has quit IRC | 12:13 | |
| vdrok | sambetts: looking at the client code, we have nargs='+' for delete commands for all resources | 12:15 |
| vdrok | so makes sense to add this to portgroups too | 12:15 |
| sambetts | vdrok: yup :) I'm happy with it, I just wanted to check, I think we don't allow bulk operations in the API which is why I was confused | 12:16 |
| *** karthiks has quit IRC | 12:18 | |
| *** jkilpatr has quit IRC | 12:19 | |
| vdrok | yeah, we do not in the api | 12:19 |
| *** awiddersheim has quit IRC | 12:19 | |
| *** mgoddard_ has joined #openstack-ironic | 12:20 | |
| *** mgoddard has quit IRC | 12:23 | |
| vdrok | sambetts: sorry, missed your comments, I replied inline | 12:32 |
| vdrok | on the portgroups spec update | 12:32 |
| *** jkilpatr has joined #openstack-ironic | 12:32 | |
| *** karthiks has joined #openstack-ironic | 12:33 | |
| vsaienk0 | sambetts: I have a question regarding default for portgroup bonding mode | 12:34 |
| vsaienk0 | sambetts: I think it should be lacp, as we already have standalone_ports_supported=True, and default mode should allow fallback | 12:34 |
| vdrok | my opinion on this - let's not have the default that may not work, just ignoring such portgroups should be ok I think | 12:35 |
| openstackgerrit | Aparna proposed openstack/proliantutils: Add Sanitize disk erase support using SSA https://review.openstack.org/398202 | 12:36 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Use oslo-config-generator for sample config https://review.openstack.org/398989 | 12:38 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Configure and use SSL-related requests options https://review.openstack.org/398992 | 12:39 |
| *** lucasagomes is now known as lucas-hungry | 12:42 | |
| ricardoas | morning! | 12:44 |
| *** Nisha_ has quit IRC | 12:44 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update the alembic migration section in the developer FAQ https://review.openstack.org/398486 | 12:44 |
| sambetts | vsaienk0: having a default bond mode that requires the switch to support something doen't make any sense to me, mode 1 works entirly operating system side, standalone_ports_supported is a flag for cases when its *required* to only have a single port, not for cases when the portgroup is invalid | 12:44 |
| *** Nisha_ has joined #openstack-ironic | 12:44 | |
| sambetts | IMO it shouldn't be possible to define an invalid portgroup | 12:44 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update the alembic migration section in the developer FAQ https://review.openstack.org/398486 | 12:45 |
| leduardo | Good morning Ironic. I am trying to execute an inspection in devstack but I only was able to do it by removing the interface (ens3) from the deploy bridge (br-deploy) and attaching it to the inspector bridge (br-inspector). My question is, Is it possible to run inspector and deploy in the same interface? If yes, how can I do this? | 12:46 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Configure and use SSL-related requests options https://review.openstack.org/398992 | 12:46 |
| sambetts | leduardo: it is possible to do yes, inspector manages its own dhcp/firefall, you will need to configure the /etc/ironic-inspector/dnsmasq.conf and /etc/ironic-inspector/inspector.conf to use the same network interface as ironic is using to talk to the BM nodes | 12:49 |
| vsaienk0 | sambetts: I see your point, it definitely makes sense | 12:49 |
| vdrok | vsaienk0: sambetts: what if we do the mode required for new api versions, and still do None for older api versions? | 12:49 |
| *** karthiks has quit IRC | 12:49 | |
| sambetts | vdrok: thats why I suggest a configurable default for cases when it is == None | 12:50 |
| *** hw_wutianwei has quit IRC | 12:50 | |
| vdrok | sambetts: ah, configurable. that makes sense then | 12:50 |
| nicodemos | good morning, ironci | 12:54 |
| nicodemos | *ironic | 12:54 |
| vdrok | morning leduardo and nicodemos | 12:54 |
| nicodemos | hey, vdrok o/ | 12:56 |
| *** baoli has joined #openstack-ironic | 12:57 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update the alembic migration section in the developer FAQ https://review.openstack.org/398486 | 12:58 |
| *** Nisha_ has quit IRC | 12:58 | |
| *** chlong has joined #openstack-ironic | 13:00 | |
| *** baoli has quit IRC | 13:01 | |
| leduardo | Hmm, I see, but if I have 2 BM nodes and I wish to run at the same time deploy one BM and inspector to another BM, using the same interface. The dnsmasq.conf allows me select the inspector bridge but OVS does not allow to put the same interface in both bridges (br-deploy and br-inspector). | 13:01 |
| TheJulia | Good morning everyone | 13:02 |
| sambetts | leduardo: its not that you want to put the interface in both bridges, you want to configure inspector to use the deploy bridge | 13:02 |
| vdrok | morning TheJulia | 13:03 |
| nicodemos | morning, TheJulia | 13:03 |
| *** ccamacho is now known as ccamacho|lunch | 13:03 | |
| *** baoli has joined #openstack-ironic | 13:04 | |
| *** baoli_ has joined #openstack-ironic | 13:05 | |
| *** karthiks has joined #openstack-ironic | 13:06 | |
| leduardo | Oh, I see now. The bridge (br-inspector) that is created by devstack is used both deploy and inspector. I had configurate another bridge (br-deploy) only to deploy. | 13:08 |
| *** baoli has quit IRC | 13:09 | |
| leduardo | thanks | 13:09 |
| leduardo | morning vdrok | 13:09 |
| *** vikrant has quit IRC | 13:16 | |
| *** vmud213 has quit IRC | 13:17 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Configure and use SSL-related requests options https://review.openstack.org/398992 | 13:19 |
| jroll | morning folks | 13:21 |
| TheJulia | good morning jroll | 13:22 |
| jroll | anyone here remember offhand the best way to pull in a patch for a library? specifically, tooz | 13:22 |
| jroll | iirc it's LIBS_FROM_GIT+=',tooz' and TOOZ_GIT_BRANCH='the review branch', right? | 13:22 |
| *** athomas has joined #openstack-ironic | 13:23 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Serve iPXE configs from Ironic API https://review.openstack.org/369438 | 13:24 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get the iPXE boot config https://review.openstack.org/394399 | 13:24 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: DNM test enabled dynamic iPXE https://review.openstack.org/391943 | 13:24 |
| sambetts | jroll: that sounds right to me | 13:24 |
| jroll | ok sweet, thanks sambetts | 13:24 |
| sambetts | jroll: actually taking a quick look I think its GITBRANCH['tooz'] = '...' | 13:25 |
| jroll | sambetts: GITBRANCH["tooz"]=${TOOZ_BRANCH:-master} | 13:26 |
| jroll | so I think you can use TOOZ_BRANCH | 13:26 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add storage_interface DB field and object https://review.openstack.org/348005 | 13:26 |
| sambetts | ah :) | 13:26 |
| jroll | but yeah, I can just change this in d-s-g I think? | 13:27 |
| *** Nisha_ has joined #openstack-ironic | 13:27 | |
| *** lucas-hungry is now known as lucasagomes | 13:28 | |
| lucasagomes | morning jroll TheJulia | 13:28 |
| *** athomas has quit IRC | 13:28 | |
| TheJulia | Good morning lucasagomes | 13:29 |
| *** jheroux has joined #openstack-ironic | 13:46 | |
| *** trown|outtypewww is now known as trown | 13:48 | |
| vsaienk0 | morning TheJulia | 13:48 |
| *** rloo has joined #openstack-ironic | 13:48 | |
| *** shuquan has joined #openstack-ironic | 13:49 | |
| *** godp1301 has joined #openstack-ironic | 13:49 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Rely on portgroup standalone_ports_supported https://review.openstack.org/360647 | 13:51 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for provision state commands https://review.openstack.org/337296 | 13:53 |
| sambetts | jroll: I've run into another issue with our API, and I'm not sure what the right approch for solving it is, basically the unit tests are testing /nodes/validate?node= but tempest is testing /nodes/<ident>/validate, the way the controller is coded means that /nodes/<ident>/validate is actually a bug, but its way thats documented on the API docs | 13:55 |
| openstackgerrit | Merged openstack/ironic: Followup to 0335e81a8787 https://review.openstack.org/398515 | 13:57 |
| *** e0ne has quit IRC | 13:58 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification https://review.openstack.org/348437 | 13:59 |
| jroll | sambetts: uhhhh, I feel like /nodes/ident/validate seems intentional then :) | 14:00 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add ironic resources CRUD notifications https://review.openstack.org/356541 | 14:00 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Remove deprecated DHCP provider methods https://review.openstack.org/398861 | 14:01 |
| lucasagomes | sambetts, jroll yeah I think it was intentional as well, same for /detail | 14:02 |
| lucasagomes | but wsme/pecan does some magic that allows you to use ?<param name>=<value> as well | 14:02 |
| lucasagomes | gotta love it :-) | 14:02 |
| *** ccamacho|lunch is now known as ccamacho | 14:03 | |
| *** Goneri has joined #openstack-ironic | 14:04 | |
| *** ralonsoh has quit IRC | 14:06 | |
| *** mgoddard_ has quit IRC | 14:06 | |
| *** mgoddard has joined #openstack-ironic | 14:06 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-specs: Amend dynamic iPXE configuration spec https://review.openstack.org/392290 | 14:06 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node maintenance notifications https://review.openstack.org/396239 | 14:08 |
| *** fragatina has joined #openstack-ironic | 14:10 | |
| *** lindycoder has joined #openstack-ironic | 14:13 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Move to tooz hash ring implementation https://review.openstack.org/399028 | 14:14 |
| *** fragatina has quit IRC | 14:15 | |
| *** ElCoyote_ has quit IRC | 14:17 | |
| vdrok | morning jroll | 14:19 |
| *** causten has joined #openstack-ironic | 14:19 | |
| cloudnull | mornings all. | 14:20 |
| cloudnull | qq -- is there any way to inject arbitrary commands into IPA when provisioning a host? Basically I want to be able to run a series of hpssacli commands before the image is written? | 14:22 |
| TheJulia | Good morning | 14:22 |
| TheJulia | cloudnull: Via a hardware manager it could be done, in a sense | 14:23 |
| TheJulia | cloudnull: proliant-utils has a ipa hardware manager included in their source tree :) | 14:23 |
| lucasagomes | cloudnull, ^ yeah you will need to create a custom hardware manager (see https://review.openstack.org/#/c/334816/ for example) | 14:23 |
| patchbot | patch 334816 - ironic-python-agent - Add a new Hardware Manager for Mellanox NICs | 14:23 |
| *** Nisha_ has quit IRC | 14:24 | |
| TheJulia | yolanda, NobodyCam: You both now have magical powers on the bifrost repo. Enjoy! :) | 14:24 |
| jroll | cloudnull: good example: https://github.com/openstack/proliantutils/blob/master/proliantutils/ipa_hw_manager/hardware_manager.py | 14:24 |
| *** shuquan has quit IRC | 14:25 | |
| jroll | cloudnull: you'd want to override get_os_install_device, have it run the things, and then return the install device you want | 14:25 |
| jroll | cloudnull: this "do some extra stuff before/after deployment" is the plumbing I mentioned that we have planned to do raid/partitioning/etc, we call it 'deploy steps' | 14:25 |
| yolanda | TheJulia, great! thanks | 14:25 |
| cloudnull | makes sense. | 14:26 |
| * cloudnull looking at the hw manager now. | 14:26 | |
| cloudnull | thanks | 14:26 |
| jroll | np | 14:26 |
| *** jcoufal has joined #openstack-ironic | 14:26 | |
| TheJulia | yolanda: fwiw, I think cinerama is out until next week. I haven't had a chance to talk to her since last week. | 14:26 |
| jroll | cloudnull: if the goal is to build a raid, we have some things to do that, but it's a bit manual (you'd set it up when you first online a thing) | 14:27 |
| cloudnull | essentially I'm thinking (hoping) i can build 2 or more raids. | 14:27 |
| cloudnull | which will limit the storage capacity on the OS device | 14:28 |
| cloudnull | and provide unpartitioned space for the user. | 14:28 |
| jroll | ah right | 14:28 |
| cloudnull | I can do that manually like so http://cdn.pasteraw.com/ea360antm5lg5zwbkeuj4fp9twj3cy1 | 14:28 |
| jroll | cloudnull: this is the 'official' way to do that, as a one-time thing http://docs.openstack.org/developer/ironic/deploy/raid.html | 14:29 |
| jroll | and should just work if the hardware manager I linked is installed in the IPA image | 14:29 |
| cloudnull | I have that hw manager in the deploy image which was installed w/ dib | 14:30 |
| *** vsaienko has quit IRC | 14:30 | |
| yolanda | hi TheJulia , so i will update my +1 with +2 now | 14:31 |
| jroll | cool, there's some examples in that doc and the process in the bottom in 'workflow' | 14:31 |
| jroll | I'd make it part of the node enrollment script, assuming you always want it to be the same | 14:32 |
| *** dstepanenko has quit IRC | 14:32 | |
| *** vsaienko has joined #openstack-ironic | 14:32 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Update playbooks to support os_client_config https://review.openstack.org/380224 | 14:32 |
| TheJulia | yolanda: \o/, don't be afraid to +w for stuff that has been sitting around for a while with plenty of positive review feedback. :) | 14:34 |
| sambetts | jroll, lucasagomes: the way the controller is coded is to use the query strings only, the nodes/detail endpoint is deisgned to work witout an ID because it lists nodes with more detail, /nodes/validate without a query string including an ID doesn't work | 14:38 |
| sambetts | lucasagomes: if /nodes/validate requires an ID always then I say that /nodes/<ident>/validate is the right way to implment it, but that means fixing the unit tests... | 14:39 |
| jroll | sambetts: well, if we document and test /nodes/ident/validate, it seems intentional, we need to keep it | 14:39 |
| sambetts | jroll: can I blow up /nodes/validate? then? | 14:40 |
| sambetts | jroll: we don't test that outside of UTs and don't document it | 14:40 |
| *** jaosorior has quit IRC | 14:41 | |
| lucasagomes | sambetts, I think so... what nodes/validate without an id would do ? Validate all nodes ? | 14:41 |
| sambetts | lucasagomes: currently it does nothing but blow up because it doesn't have an ID | 14:42 |
| jroll | sambetts: what's the benefit to blowing up /nodes/validate?node_ident=foo | 14:45 |
| *** vsaienko has quit IRC | 14:45 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add sanity tests for baremetal power state commands https://review.openstack.org/333361 | 14:49 |
| sambetts | jroll: basically making the validate function properly require an ID should mean that the pecan routing works correctly for /nodes/<ident>/validate, and _lookup is used as a last resort like it suppose to be instead of calling _lookup because it thinks validate is a subcontroller | 14:49 |
| jroll | sambetts: confused though, what's the benefit of making this change? | 14:50 |
| jroll | other than code cleanliness maybe? | 14:50 |
| *** syed__ has joined #openstack-ironic | 14:50 | |
| sambetts | it means that we can handle if a subcontroller doesn't exist or isn't allowed in this api version better | 14:51 |
| TheJulia | Additionally, what would the impact of having to revert the act of blowing it up when we find out someone was using it and it was not deprecated? | 14:51 |
| sambetts | thats a good point I wonder what version of the API the client is using | 14:53 |
| sambetts | client is using nodes/<node ident>/validate | 14:54 |
| jroll | TheJulia: ++ | 14:54 |
| TheJulia | Not everyone uses the client library... | 14:54 |
| sambetts | yeah :/ | 14:54 |
| TheJulia | I'm just playing devils advocate here, but I think to remove it essentially forces us to cut /v2 | 14:54 |
| sambetts | If we only document the other way, then I mean isn't it a case of this | 14:55 |
| sambetts | http://xkcd.com/1172/ | 14:55 |
| TheJulia | pretty much :) | 14:56 |
| jroll | sambetts: I think what we're saying is creating an API break has to bring a large amount of value | 14:56 |
| jroll | to be worth it | 14:56 |
| lucasagomes | sambetts, 1.9 by default | 14:57 |
| lucasagomes | sambetts, when you say blow up, what http error does it return ? | 14:57 |
| TheJulia | jroll: ++ | 14:57 |
| sambetts | lucasagomes: 400 Expected a logical name or UUID but received None. | 14:59 |
| *** mgoddard_ has joined #openstack-ironic | 15:01 | |
| *** mgoddard has quit IRC | 15:04 | |
| sambetts | the other issue the fix for this would solve so I also discovered along side this issue if you do /nodes/<ident>/<anything> it does the equivolent of GET /nodes/<ident>?fields=<anything> instead of returning a 404 | 15:06 |
| sambetts | even though /<anything> isn't a valid url | 15:06 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for maintenance mode commands https://review.openstack.org/337259 | 15:09 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal port commands https://review.openstack.org/365692 | 15:12 |
| *** e0ne has joined #openstack-ironic | 15:12 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Remove deprecated Neutron DHCP provider methods https://review.openstack.org/398861 | 15:13 |
| *** Nisha_ has joined #openstack-ironic | 15:16 | |
| *** yufei has joined #openstack-ironic | 15:20 | |
| *** rbrud has joined #openstack-ironic | 15:23 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add ironic resources CRUD notifications https://review.openstack.org/356541 | 15:23 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node maintenance notifications https://review.openstack.org/396239 | 15:25 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Move to tooz hash ring implementation https://review.openstack.org/399028 | 15:26 |
| JayF | jroll: the shared hash ring code went in tooz instead of some oslo project? | 15:27 |
| JayF | til | 15:27 |
| jroll | JayF: indeed | 15:27 |
| jroll | well, is going | 15:27 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for 'baremetal port create' command https://review.openstack.org/332880 | 15:28 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal chassis commands https://review.openstack.org/366158 | 15:30 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal driver commands https://review.openstack.org/367359 | 15:30 |
| JayF | jroll: if the point is to test ci, you wanna look at the comment on my -1 | 15:30 |
| JayF | jroll: and if it doesn't fail ci with that error, it's scary :) | 15:30 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Use oslo-config-generator for sample config https://review.openstack.org/398989 | 15:30 |
| jroll | JayF: idk why you're reviewing that but will look in a moment | 15:31 |
| JayF | jroll: I was curious, and found that you're calling the tooz lib with incorrect capitalization, so the whole thing should break. I hope :) | 15:31 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Move to tooz hash ring implementation https://review.openstack.org/399028 | 15:32 |
| jroll | JayF: well, it was a different incorrect, but you were right | 15:32 |
| JayF | yeah I figured it'd be either bad caps or needing to be called outta the module | 15:33 |
| jroll | thanks for that | 15:33 |
| JayF | np | 15:33 |
| JayF | I love reviewing net-minus-hundredplus-lines patches | 15:33 |
| JayF | lol | 15:33 |
| jroll | I love proposing them :D | 15:33 |
| jroll | JayF: the nova patch is +11, -284 | 15:33 |
| *** absubram has joined #openstack-ironic | 15:34 | |
| *** absubram_ has joined #openstack-ironic | 15:35 | |
| *** rbartal has quit IRC | 15:35 | |
| *** fragatina has joined #openstack-ironic | 15:37 | |
| *** absubram has quit IRC | 15:38 | |
| *** absubram_ is now known as absubram | 15:38 | |
| openstackgerrit | Merged openstack/ironic: Remove ClusteredComputeManager https://review.openstack.org/397762 | 15:39 |
| *** fragatina has quit IRC | 15:41 | |
| *** fragatina has joined #openstack-ironic | 15:42 | |
| jroll | ummmm. #awkward. | 15:42 |
| jroll | vdrok: yuriyz: lucasagomes: ^ that wasn't past the 3 month deprecation period | 15:42 |
| jroll | the deprecation merged september 20 https://review.openstack.org/#/c/352991/ | 15:43 |
| patchbot | patch 352991 - ironic - Deprecate ClusteredComputeManager (MERGED) | 15:43 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add negative tests for baremetal node commands https://review.openstack.org/337719 | 15:43 |
| vdrok | ouch | 15:43 |
| vdrok | jroll: revert? | 15:44 |
| lucasagomes | jroll, urgh... I got confused because it was saying ocata I thought we had already deprecated it for longer | 15:44 |
| jroll | vdrok: dunno, lemme figure out what to do about that | 15:44 |
| vdrok | yep, seems like i confused that with removal from docs, which was long time ago i think :( | 15:45 |
| jroll | it's okay, it happens | 15:46 |
| mariojv | does it still meet deprecation criteria if there's no release before the 3 month deadline? | 15:46 |
| * JayF hands jroll an sh for his it | 15:46 | |
| jroll | mariojv: yes | 15:46 |
| jroll | the 3 months is about people that deploy from master | 15:46 |
| jroll | er | 15:46 |
| jroll | it does not meet criteria, sorry | 15:47 |
| mariojv | ok | 15:47 |
| lucasagomes | jroll, let's revert it then | 15:47 |
| lucasagomes | and remove it after december 20 | 15:47 |
| jroll | lucasagomes: I think I agree, but poked a couple TC folks to sanity check | 15:47 |
| lucasagomes | ack | 15:47 |
| * jroll puts it up to get CI going | 15:47 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Revert "Remove ClusteredComputeManager" https://review.openstack.org/399100 | 15:48 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node console notifications https://review.openstack.org/397812 | 15:50 |
| *** mgoddard has joined #openstack-ironic | 15:50 | |
| *** mgoddard_ has quit IRC | 15:50 | |
| *** wajdi has joined #openstack-ironic | 15:50 | |
| *** fxpester has quit IRC | 15:52 | |
| jroll | ok, ttx confirmed that's the right thing to do | 15:52 |
| lucasagomes | +a | 15:54 |
| lucasagomes | jroll, sorry for approving the other one in the first place | 15:55 |
| jroll | lucasagomes: things happen | 15:55 |
| lucasagomes | yeah | 15:55 |
| jroll | all good man :) | 15:55 |
| JayF | lucasagomes: it's nice to see the long-time cores screw up sometimes, makes me feel less like a noob when I do it ;) | 15:55 |
| lucasagomes | JayF, hah yeah nobody picks everything in a review I guess :-( | 15:56 |
| lucasagomes | glad we use git at least, git revert makes it easier to fix | 15:56 |
| jroll | the hardest part about these | 15:56 |
| jroll | is making sure you look at the merge date, not the commit date | 15:57 |
| yufei | hello, ironicers, I’m reading ironic-python-agent, and found a sentence in readme: | 15:57 |
| yufei | The ironic-python-agent may also be used with the original Ironic pxe drivers as of the kilo release. | 15:57 |
| yufei | Does this mean that Ironic-python-agent doesn’t support to provision with pxe drivers after kilo? But I still can provision with pxe_ipmi tool driver and coreos image in newton. Or is there something I understand wrong? | 15:57 |
| jroll | yufei: it means ironic-python-agent works with pxe drivers for kilo and later | 15:58 |
| jroll | before kilo, it did not | 15:58 |
| yufei | ok…. thank you, sorry for my english. | 15:58 |
| jroll | no problem, you're welcome :) | 15:58 |
| yufei | :D | 15:59 |
| lucasagomes | yufei, ^ that... it's a historical thing. In the past the pxe_ drivers used the original bash ramdisk and the agent_ the IPA. But we decided to consolidade it and have only 1 supported ramdisk in Ironic (the IPA one) | 15:59 |
| lucasagomes | and since kilo both drivers supports it | 15:59 |
| *** rama_y has quit IRC | 16:01 | |
| yufei | thanks, the first version of ironic I use is Liberty, so i don’t find that before | 16:01 |
| JayF | yufei: also, thanks for reading the docs \o/ | 16:02 |
| yufei | :D | 16:02 |
| *** penick has joined #openstack-ironic | 16:02 | |
| *** absubram has quit IRC | 16:03 | |
| *** moshele has quit IRC | 16:05 | |
| *** dtrainor has joined #openstack-ironic | 16:05 | |
| dtrainor | Howdy. I'm running in to errors like this that are pretty persistent when trying to do a deployment: Error in deploy of node 5e8b0d2b-1914-4cbf-b1c2-25a35111be96: DRAC operation failed. Reason: Unfinished config jobs found ... ... | 16:06 |
| dtrainor | This results in Nova not being able to find a valid host, with the classic "There are not enough hosts available., Code: 500" error. | 16:07 |
| *** amotoki has quit IRC | 16:07 | |
| dtrainor | that being said, i'm having difficulty understanding if this is an issue with ironic not being able ot handle exceptions properly based on unexpected results from the drac or.... what. i'm pretty out of my element right now. | 16:09 |
| *** pcaruana has quit IRC | 16:10 | |
| *** causten has quit IRC | 16:11 | |
| *** causten has joined #openstack-ironic | 16:12 | |
| pas-ha | dtrainor: try to check manually with dracclient what is the list / status of unfinished jobs on those nodes | 16:13 |
| sambetts | dtrainor: That error is coming from the drac driver when it trys to validate if a config job is finished or not, before it does certain actions like setting the boot device, it seems like there is something running on your server that is preventing the drac driver from proceeding | 16:14 |
| pas-ha | +1 | 16:14 |
| pas-ha | probably there is leftover unfinished job which e.g. requires reboot until any other changes can be applied | 16:15 |
| dtrainor | that makes sense. i have suspicions that this error is caused in large part to ironic itself, since i had been using these systems just fine the other day and nobody has used them since | 16:15 |
| *** fragatina has quit IRC | 16:15 | |
| *** causten has quit IRC | 16:16 | |
| *** mjura has quit IRC | 16:16 | |
| *** causten has joined #openstack-ironic | 16:17 | |
| *** vsaienko has joined #openstack-ironic | 16:18 | |
| *** yufei has quit IRC | 16:19 | |
| *** e0ne has quit IRC | 16:19 | |
| *** causten has quit IRC | 16:21 | |
| openstackgerrit | William Stevenson proposed openstack/ironic-inspector: Fixed incorrect string type checking https://review.openstack.org/399123 | 16:22 |
| *** baoli_ has quit IRC | 16:28 | |
| dtrainor | thanks pas-ha, sambetts | 16:28 |
| *** fragatina has joined #openstack-ironic | 16:30 | |
| *** penick has quit IRC | 16:30 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Configure and use SSL-related requests options https://review.openstack.org/398992 | 16:31 |
| *** penick has joined #openstack-ironic | 16:32 | |
| lucasagomes | dtrainor, apparently there's also a "list_unfinished_jobs" and "{abandon, commit, set}_bios_config" vendor passthru for DRAC | 16:34 |
| *** rpioso has joined #openstack-ironic | 16:34 | |
| lucasagomes | where you could check those pending jobs | 16:34 |
| dtrainor | i'll do just that, at least for a learning exercise | 16:34 |
| *** causten has joined #openstack-ironic | 16:36 | |
| *** gcb has quit IRC | 16:40 | |
| *** fragatina has quit IRC | 16:41 | |
| *** gcb has joined #openstack-ironic | 16:41 | |
| openstackgerrit | Merged openstack/ironic: Enable PXE for systems using petitboot https://review.openstack.org/185987 | 16:41 |
| JayF | vgadiraj: aslezil: I want to attend a cultural event :P | 16:43 |
| jroll | lunchtime! \o/ \o/ \o/ | 16:43 |
| aslezil | JayF you should have been in castle :) | 16:44 |
| dtrainor | lucasagomes, the field guys are convinced that pxe_drac can be unpredictable and even unreliable, and suggest using pxe_ipmitool instead, even when communicating with a DRAC. do you have any thoughts or opinions on this? | 16:45 |
| TheJulia | jroll: you speak of lunch, when I've not even had breakfast :) | 16:45 |
| jroll | TheJulia: nor have I but I has spaghetti :D | 16:46 |
| lucasagomes | dtrainor, as for now, I also think pxe_ipmitool is better | 16:46 |
| dtrainor | good to know. thanks. just wanted to hear it from another trusted source. | 16:47 |
| lucasagomes | dtrainor, as long as you don't need any vendor specific stuff from DRAC | 16:47 |
| dtrainor | understood | 16:50 |
| *** rama_y has joined #openstack-ironic | 16:51 | |
| *** penick has quit IRC | 16:52 | |
| *** derekh has quit IRC | 16:55 | |
| *** penick has joined #openstack-ironic | 17:00 | |
| *** aNuposic has joined #openstack-ironic | 17:03 | |
| dtrainor | lucasagomes, pas-ha, sambetts, pxe_ipmitool seems to be much more tolerant of pending jobs so far | 17:06 |
| lucasagomes | dtrainor, yeah, IPMI doesn't really have any concept of "jobs list" | 17:06 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add ability to time out on individual clean steps https://review.openstack.org/391554 | 17:06 |
| mariojv | vsaienk0: i left a comment there for you ^ tests should pass now | 17:07 |
| dtrainor | right | 17:07 |
| mariojv | aNuposic: hey, inspector unit tests are passing with the ENROLL check added: https://review.openstack.org/#/c/398546/ | 17:07 |
| patchbot | patch 398546 - ironic-inspector - [WIP/DNM] Test discovered nodes are in ENROLL state | 17:07 |
| mariojv | i'll update the commit message and mark that as not WIP | 17:08 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic-inspector: Test discovered nodes are in ENROLL state and fix typo https://review.openstack.org/398546 | 17:09 |
| aNuposic | mariojv: Ah okay i was just going to try that. Let me check the tests | 17:09 |
| mariojv | aNuposic: they passed. i just updated commit message, but you can see the old tests passing if you click "show CI" at the bottom left | 17:09 |
| aNuposic | mariojv: Okay | 17:10 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add storage_interface DB field and object https://review.openstack.org/348005 | 17:10 |
| aNuposic | mariojv: then i need to see why in my environment nodes were not getting enrolled | 17:10 |
| aNuposic | mariojv: i guess the way i was doing or creating the nodes might be wrong | 17:11 |
| mariojv | aNuposic: up to you, i just wanted to make sure you didn't waste time on something being tested in the gate | 17:11 |
| aNuposic | mariojv: hmm. okay | 17:11 |
| aNuposic | mariojv: thanks | 17:12 |
| mariojv | np | 17:12 |
| *** rbrud has quit IRC | 17:14 | |
| *** strigazi is now known as strigazi_AFK | 17:16 | |
| *** fragatina has joined #openstack-ironic | 17:19 | |
| *** rpioso has quit IRC | 17:19 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add storage_interface to base driver class https://review.openstack.org/348006 | 17:23 |
| vdrok | good night everyone! | 17:24 |
| nicodemos | night, vdrok | 17:25 |
| rloo | good night vdrok | 17:26 |
| *** fragatina has quit IRC | 17:29 | |
| *** fragatina has joined #openstack-ironic | 17:30 | |
| lucasagomes | vdrok, night | 17:30 |
| *** causten has quit IRC | 17:32 | |
| *** causten has joined #openstack-ironic | 17:33 | |
| lucasagomes | calling it a day too | 17:33 |
| lucasagomes | have a great evening all! | 17:33 |
| *** causten_ has joined #openstack-ironic | 17:33 | |
| sambetts | cya lucasagomes | 17:33 |
| nicodemos | good night, lucasagomes | 17:35 |
| lucasagomes | o/ | 17:35 |
| *** lucasagomes is now known as lucas-afk | 17:35 | |
| *** causten has quit IRC | 17:37 | |
| *** e0ne has joined #openstack-ironic | 17:39 | |
| *** causten_ has quit IRC | 17:43 | |
| *** causten has joined #openstack-ironic | 17:44 | |
| *** baoli has joined #openstack-ironic | 17:46 | |
| *** rpioso has joined #openstack-ironic | 17:47 | |
| *** causten has quit IRC | 17:49 | |
| * sambetts -> home | 17:50 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal port commands https://review.openstack.org/365692 | 17:51 |
| sambetts | jroll, rloo: I'm on PTO tomorrow, if I'm needed for anything I'll be avaible via email :) | 17:51 |
| *** sambetts is now known as sambetts|afk | 17:51 | |
| rloo | sambetts: ok, Enjoy! (I'm off tomorrow too, jroll can hold up the fort) | 17:51 |
| sambetts|afk | thanks you too :) | 17:52 |
| rloo | sambetts|afk: :D | 17:52 |
| *** ijw has joined #openstack-ironic | 18:02 | |
| *** fragatin_ has joined #openstack-ironic | 18:04 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal driver commands https://review.openstack.org/367359 | 18:05 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for OSC baremetal chassis create command https://review.openstack.org/382496 | 18:06 |
| *** fragatina has quit IRC | 18:07 | |
| *** wajdi has quit IRC | 18:07 | |
| *** roeyc has quit IRC | 18:08 | |
| NobodyCam | gah connection issues this morning: so belated Good Morning Ironic'ers :) | 18:15 |
| JayF | o/ | 18:15 |
| *** causten has joined #openstack-ironic | 18:17 | |
| rloo | morning NobodyCam! | 18:18 |
| NobodyCam | morning JayF, rloo :) | 18:18 |
| *** causten has quit IRC | 18:21 | |
| openstackgerrit | Merged openstack/ironic: Revert "Remove ClusteredComputeManager" https://review.openstack.org/399100 | 18:23 |
| *** trown is now known as trown|lunch | 18:25 | |
| *** vsaienko has quit IRC | 18:25 | |
| *** vsaienko has joined #openstack-ironic | 18:25 | |
| *** strigazi_AFK is now known as strigazi | 18:33 | |
| nicodemos | morning, NobodyCam | 18:33 |
| NobodyCam | morning nicodemus :) | 18:35 |
| *** krtaylor has quit IRC | 18:38 | |
| *** betherly has quit IRC | 18:39 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Move to tooz hash ring implementation https://review.openstack.org/399028 | 18:40 |
| *** baoli_ has joined #openstack-ironic | 18:41 | |
| *** baoli has quit IRC | 18:43 | |
| TheJulia | NobodyCam: good morning, again :) | 18:45 |
| NobodyCam | :) | 18:46 |
| *** jpich has quit IRC | 18:47 | |
| *** dhellmann has joined #openstack-ironic | 18:52 | |
| *** moshele has joined #openstack-ironic | 18:53 | |
| *** vinsh has quit IRC | 18:57 | |
| *** rbrud has joined #openstack-ironic | 19:01 | |
| *** fragatin_ has quit IRC | 19:01 | |
| *** fragatina has joined #openstack-ironic | 19:02 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add storage_interface to base driver class https://review.openstack.org/348006 | 19:08 |
| *** Nisha_ has quit IRC | 19:11 | |
| *** e0ne has quit IRC | 19:17 | |
| *** penick has quit IRC | 19:17 | |
| *** galyna has quit IRC | 19:22 | |
| *** strigazi is now known as strigazi_AFK | 19:25 | |
| *** penick has joined #openstack-ironic | 19:28 | |
| *** causten has joined #openstack-ironic | 19:29 | |
| *** iurygregory has quit IRC | 19:29 | |
| *** penick has quit IRC | 19:31 | |
| *** ijw has quit IRC | 19:32 | |
| *** ijw has joined #openstack-ironic | 19:35 | |
| *** ijw has quit IRC | 19:35 | |
| *** ijw has joined #openstack-ironic | 19:35 | |
| *** ChubYann has joined #openstack-ironic | 19:37 | |
| *** ijw has quit IRC | 19:38 | |
| *** krtaylor has joined #openstack-ironic | 19:42 | |
| *** Sukhdev has joined #openstack-ironic | 19:44 | |
| *** trown|lunch is now known as trown | 19:46 | |
| *** openstackgerrit has quit IRC | 19:48 | |
| *** openstackgerrit has joined #openstack-ironic | 19:49 | |
| mariojv | dtantsur|afk: lol, i totally didn't realize that i put "bearmetal" instead of "baremetal" in https://review.openstack.org/#/c/398546 | 19:50 |
| patchbot | patch 398546 - ironic-inspector - Test discovered nodes are in ENROLL state and fix ... | 19:50 |
| mariojv | dtantsur|afk: you want me to change it? | 19:50 |
| xavierr | mariojv: +1 for bearmetal | 19:50 |
| mrtenio | bearmetal sounds like a nice slogan | 19:51 |
| *** sambetts|afk has quit IRC | 19:53 | |
| *** sambetts_ has joined #openstack-ironic | 19:55 | |
| *** e0ne has joined #openstack-ironic | 19:56 | |
| JayF | mariojv: you gonna fix that or should I land it/ lol | 19:57 |
| mariojv | JayF: land away | 19:57 |
| JayF | mariojv: I think I'll just land it :) easter egg for someone else to find later | 19:57 |
| JayF | lol | 19:57 |
| *** ijw has joined #openstack-ironic | 19:58 | |
| *** vsaienko has quit IRC | 19:59 | |
| jroll | mrtenio: https://wiki.openstack.org/w/images/d/df/Ironic_mascot_color.png :) | 19:59 |
| jroll | metal bear | 19:59 |
| xavierr | JayF: o/ | 20:01 |
| JayF | o/ | 20:01 |
| rloo | mariojv: i looked at your notification doc changes. it seems to me that we should land them either before or after yuriyz's notification changes, since his patches adds new notifications. | 20:02 |
| *** ijw has quit IRC | 20:03 | |
| mariojv | rloo: right - i'm find with either order | 20:03 |
| mariojv | the changes to the state machine diagram should land after his FSM changes i believe | 20:04 |
| mariojv | i haven't looked at that in a while though | 20:04 |
| rloo | i haven't looked into those at all | 20:04 |
| mariojv | i think the install guide bit can land independent of yuriy's changes but imbw | 20:05 |
| rloo | mariojv: any idea how many notification patches there are, and what state they are in? | 20:05 |
| mariojv | yup, give me a minute | 20:05 |
| rloo | mariojv: the install guide has a pointer to the deploy guide | 20:05 |
| rloo | and your patch creates the page in the deploy guide! | 20:06 |
| mariojv | rloo: 4 notification changes ready for review: https://review.openstack.org/#/q/topic:bug/1606520+status:open | 20:06 |
| mariojv | ah, you're totally right | 20:06 |
| rloo | mariojv: maybe we should just try to land your notification patch and have the notification patches be rebased. | 20:06 |
| mariojv | fine by me | 20:06 |
| JayF | rloo: mariojv: +1 that's what I'd suggest. | 20:06 |
| JayF | mariojv: wanna respond to comments on your patch and then I can have a look? We should be able to land tha ttoday | 20:07 |
| rloo | mariojv: do you have time to do another revision today. i'm here for another 2-3 hours. I'm sure JayF will help :) | 20:07 |
| mariojv | i have 20 mins before lolmeetings | 20:07 |
| mariojv | rloo: this one right? https://review.openstack.org/#/c/389344/ | 20:07 |
| patchbot | patch 389344 - ironic - Add notification documentation to administrator's ... | 20:07 |
| JayF | 20m is enough to revise it mariojv :) | 20:07 |
| JayF | mariojv: I will boycott review party if your patch isn't updated yet :P | 20:07 |
| *** jkilpatr has quit IRC | 20:12 | |
| *** jkilpatr has joined #openstack-ironic | 20:13 | |
| mrda | Morning Ironic | 20:15 |
| nicodemos | morning, mrda | 20:15 |
| *** ijw has joined #openstack-ironic | 20:15 | |
| mrtenio | jroll, how could I not know the name of the mascot? :P thanks. | 20:16 |
| jroll | :D | 20:16 |
| mrtenio | morning mrda, nicodemos | 20:16 |
| mariojv | rloo: can you clarify your L79 comment on https://review.openstack.org/#/c/389344/4/doc/source/index.rst a bit? | 20:18 |
| patchbot | patch 389344 - ironic - Add notification documentation to administrator's ... | 20:18 |
| rloo | mariojv: yeah, it doesn't belong in the 'Getting Started' section. | 20:18 |
| mariojv | do you want a separate section from getting started, called "Advanced development", on the same level as "getting started"? | 20:18 |
| rloo | mariojv: just leave it there for now, we can change it later. | 20:18 |
| mariojv | alright | 20:19 |
| rloo | mariojv: yeah, that's what i meant. | 20:19 |
| mariojv | i'll do the auditing move though, that's simple | 20:19 |
| rloo | mariojv: just that i don't know what the name of the section should be and i dont' want to bikeshed on that | 20:19 |
| mariojv | k | 20:19 |
| mariojv | regenerating locally then should have another patch set up | 20:20 |
| rloo | thx mariojv | 20:20 |
| mariojv | np, thanks for reviewing | 20:20 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add notification documentation to administrator's guide https://review.openstack.org/389344 | 20:24 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add notification documentation to install guide https://review.openstack.org/395787 | 20:24 |
| mariojv | rloo: JayF ^ | 20:24 |
| *** ijw has quit IRC | 20:30 | |
| JayF | mariojv: lgtm, looks like you address all of Ruby's issues. | 20:30 |
| mariojv | \o/ | 20:30 |
| JayF | mariojv: a couple of things in the install guide one, can you address them quickly/ | 20:32 |
| *** ijw has joined #openstack-ironic | 20:34 | |
| rloo | mariojv: +A'd | 20:38 |
| *** ijw has quit IRC | 20:39 | |
| JayF | rloo: so something I'm going to work on, and maybe I should send something to the list about this, is looking at all the doc content we have, and thinking about how to better organize it. Sorta trying to get a true dev/admin/install guide split so that we can forklift hte admin guide when needed | 20:41 |
| JayF | rloo: I think that's relevant to the comment in the patch you just landed of mario's, about what goes where in index | 20:41 |
| *** vinsh has joined #openstack-ironic | 20:41 | |
| rloo | JayF: my suggestion is to propose something so we can bikeshed on that. but good idea. | 20:41 |
| rloo | JayF: i was going to ask you. did you delete the ironic stuff that is in the official admin guide? | 20:42 |
| anish | anyone feel like reviewing a one line change ? https://review.openstack.org/#/c/398837/ | 20:42 |
| patchbot | patch 398837 - python-dracclient - Add missing RAID Status | 20:42 |
| JayF | rloo: not yet, I was going to wait on that until I knew exactly where new admin guide would live. If you think that should be done sooner I can jfdi right now and just patch it later if we have a bette rlanding page to link it to | 20:42 |
| JayF | anish: I don't work in python-dracclient, and aiui it's about to be dropped from ironic governance, IDK who would be the best people to review that, but I don't think it's anyone active in channel at this time of day | 20:43 |
| rloo | JayF: we don't know when they're going to have an admin plugin. i think until that happens, we should just keep the admin stuff in our tree. | 20:43 |
| anish | JayF: gotcha, thanks ! | 20:43 |
| JayF | rloo: I mean more, if we do a split in-tree, like with a legit admin guide hosted in our dev docs location | 20:43 |
| JayF | rloo: that might have a "Admin guide" landing page better than what we have now | 20:44 |
| anish | JayF: I read about the governance part, what does that imply exactly ? | 20:44 |
| rloo | JayF: OH. i probably wouldn't do that if it means moving admin pages to diff urls. cuz we would have to maintain the old links. | 20:44 |
| *** ijw has joined #openstack-ironic | 20:44 | |
| JayF | rloo: yeah... I guess we'd have to maintain those spots anyway, so sounds like I should jfdi and remove the bad content from the central one | 20:45 |
| rloo | JayF: right, i don't care about the central ones -- see if anyone there cares about those pages/links disappearing. Oh. or i suppose you could just update those pages with links to our dev pages. dunno. | 20:45 |
| jroll | anish: it just means the ironic team isn't responsible for it | 20:49 |
| *** ijw has quit IRC | 20:49 | |
| jroll | anish: the folks that work on drac stuff will continue to maintain and release it | 20:49 |
| anish | thanks | 20:50 |
| JayF | and more directly relevant to your ask: folks with core on ironic won't automatically get it on dracclient | 20:50 |
| JayF | which is part of why I don't wanna review that even if it's dirt simple | 20:50 |
| jroll | you don't have core anyway, JayF :P | 20:50 |
| anish | heh | 20:50 |
| JayF | jroll: I thought ironic-core had core on all ironic projects? | 20:50 |
| jroll | JayF: no, that's on my list to fix | 20:50 |
| JayF | oh, TIL | 20:50 |
| JayF | ah, but supposed to | 20:51 |
| jroll | once the governance changes go through | 20:51 |
| JayF | that makes more sense | 20:51 |
| *** vsaienko has joined #openstack-ironic | 20:51 | |
| openstackgerrit | Stephane Robert proposed openstack/ironic: Caching of PDU autodiscovery https://review.openstack.org/399234 | 20:55 |
| openstackgerrit | Merged openstack/ironic-inspector: Test discovered nodes are in ENROLL state and fix typo https://review.openstack.org/398546 | 20:56 |
| *** srobert has joined #openstack-ironic | 20:57 | |
| mariojv | thanks for the +A | 20:58 |
| rloo | belated morning mrda | 20:58 |
| mrda | hi rloo | 20:58 |
| rloo | mariojv: yw, thx for the revision! | 20:58 |
| openstackgerrit | Merged openstack/ironic: Timing metrics: iRMC drivers https://review.openstack.org/371284 | 21:02 |
| *** MattMan has quit IRC | 21:03 | |
| *** e0ne has quit IRC | 21:03 | |
| *** MattMan has joined #openstack-ironic | 21:03 | |
| *** penick has joined #openstack-ironic | 21:03 | |
| openstackgerrit | Merged openstack/ironic: Add notification documentation to administrator's guide https://review.openstack.org/389344 | 21:04 |
| *** ijw has joined #openstack-ironic | 21:05 | |
| *** ijw has quit IRC | 21:10 | |
| *** ijw has joined #openstack-ironic | 21:10 | |
| *** penick has quit IRC | 21:12 | |
| *** Goneri has quit IRC | 21:18 | |
| *** penick has joined #openstack-ironic | 21:18 | |
| *** ijw has quit IRC | 21:20 | |
| *** jkilpatr has quit IRC | 21:29 | |
| *** godp1301 has quit IRC | 21:30 | |
| *** penick has quit IRC | 21:40 | |
| *** ijw has joined #openstack-ironic | 21:42 | |
| *** ijw has quit IRC | 21:46 | |
| *** ijw has joined #openstack-ironic | 21:47 | |
| *** ijw has quit IRC | 21:49 | |
| *** ijw has joined #openstack-ironic | 21:49 | |
| *** jkilpatr has joined #openstack-ironic | 21:53 | |
| *** penick has joined #openstack-ironic | 21:56 | |
| *** srobert has quit IRC | 21:59 | |
| *** trown is now known as trown|outtypewww | 22:00 | |
| openstackgerrit | Xavier proposed openstack/ironic: Move `deploy_forces_oob_reboot` to deploy drivers https://review.openstack.org/397843 | 22:02 |
| openstackgerrit | Xavier proposed openstack/ironic: Remove agent vendor passthru from OneView drivers https://review.openstack.org/397846 | 22:02 |
| openstackgerrit | Xavier proposed openstack/ironic: Remove agent vendor passthru from most drivers https://review.openstack.org/397844 | 22:02 |
| openstackgerrit | Xavier proposed openstack/ironic: Remove agent vendor passthru from iLO drvers https://review.openstack.org/397845 | 22:02 |
| *** jheroux has quit IRC | 22:08 | |
| *** moshele has quit IRC | 22:09 | |
| *** jcoufal has quit IRC | 22:22 | |
| *** penick has quit IRC | 22:23 | |
| rloo | mariojv: so that works for you? https://review.openstack.org/398486 | 22:24 |
| patchbot | patch 398486 - ironic - Update the alembic migration section in the develo... | 22:24 |
| *** godp1301 has joined #openstack-ironic | 22:28 | |
| *** krtaylor has quit IRC | 22:30 | |
| *** vsaienko has quit IRC | 22:30 | |
| *** ijw has quit IRC | 22:31 | |
| *** ijw has joined #openstack-ironic | 22:33 | |
| mariojv | rloo: you're right - that doesn't work. i'd tried similar commands that worked, but something about the -- breaks it | 22:34 |
| mariojv | i'll change my vote | 22:34 |
| mariojv | thanks for pointing it out | 22:35 |
| rloo | thx for checking mariojv. if it was just me, I'd OK it, but glad to know it isn't just my set up! | 22:35 |
| mariojv | yup | 22:36 |
| mariojv | using regular ol bash | 22:36 |
| rloo | I mean, if it didn't work for me only :) | 22:36 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: [WIP] Add base cinder common interface https://review.openstack.org/355625 | 22:38 |
| *** Goneri has joined #openstack-ironic | 22:38 | |
| *** lindycoder has quit IRC | 22:44 | |
| *** rama_y has quit IRC | 23:00 | |
| *** ijw has quit IRC | 23:03 | |
| *** ijw has joined #openstack-ironic | 23:09 | |
| cloudnull | is it possible to go from manageable > available without having to go through cleaning ? | 23:12 |
| JayF | cloudnull: only if you have automatic cleaning disabled. We explicitly require nodes to be cleaned when automatic cleaning is enabled, so an operator can know for certain all nodes in AVAILABLE have been cleaned. | 23:13 |
| rloo | no, but you could turn cleaning off via the configs. | 23:13 |
| cloudnull | i've got around 120 nodes that I had to re-enroll w/ different RAID configs and I'd like to simply initiallize the nodes and go w/out cleaning on this first pass. | 23:13 |
| *** ijw has quit IRC | 23:13 | |
| JayF | cloudnull: to do otherwise would be probably a security vulnerability :/ | 23:13 |
| cloudnull | fair enough | 23:13 |
| JayF | cloudnull: yeah; the only way to do that would be get them all in "available" with the config to disable cleaning on, then re-enabled cleaning ... but I don't think that's a good pattern | 23:14 |
| cloudnull | just figured I'd ask before disabling cleaning for this first pass | 23:14 |
| cloudnull | in this case all of the nodes have been cleaned already. | 23:14 |
| JayF | yeah; in OnMetal we actually would use that first pass as a form on QC | 23:14 |
| cloudnull | but I've changed from allowing 1 raid 1+0 to several raids so that my OS device is <1TB | 23:14 |
| JayF | i.e. if a node was unable to boot a ramdisk, we'd find it out at the "cleaning" step when moving them to available the first time, and not on the first time a deploy was attempted | 23:15 |
| *** godp1301 has quit IRC | 23:15 | |
| cloudnull | I am making them all go through a "cleaning" by using destory_configuration && create_configuration I just dont want them sitting and spinning for the next 3 hours doing ata secure delete. | 23:16 |
| JayF | hmm... | 23:16 |
| cloudnull | but i'll play with it | 23:16 |
| JayF | cloudnull: you could disable erase_devices explicitly, not just all of cleaning | 23:17 |
| JayF | cloudnull: https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L965 | 23:17 |
| *** pawnesh has quit IRC | 23:17 | |
| cloudnull | in the end I really want all of the actual cleaning. so that nodes are properly cared for as users consume them | 23:17 |
| cloudnull | i was just looking for a quick cheat on this first go | 23:18 |
| JayF | I'm thinking about how viable it would be to allow a node into available without cleaning, i.e. if it could be done safely | 23:18 |
| *** causten has quit IRC | 23:18 | |
| *** ijw has joined #openstack-ironic | 23:19 | |
| JayF | but skipping cleaning on first enroll of nodes is kinda an antipattern except in a limited enough set of circumstances that it probably shouldn't be allowed in the api | 23:19 |
| cloudnull | i mean I can go edit the DB :) | 23:19 |
| JayF | egad, -2 | 23:19 |
| JayF | lol | 23:19 |
| *** causten has joined #openstack-ironic | 23:19 | |
| JayF | fwiw; I'm the person who advocated for cleaning to always be mandated :) so you might find people with other opinions | 23:19 |
| *** Goneri has quit IRC | 23:20 | |
| *** causten has quit IRC | 23:24 | |
| cloudnull | thanks again JayF | 23:24 |
| * TheJulia wonders if there is measurable sarcasm :) | 23:24 | |
| TheJulia | but in a good way | 23:24 |
| cloudnull | no sarcasm, I appreciate the answers. | 23:24 |
| * TheJulia just has a weird sense of humor then | 23:25 | |
| cloudnull | i mean I turned off cleaning, and will turn it back on later. | 23:25 |
| cloudnull | but i appreciate knowing how its supposed to be done. | 23:25 |
| TheJulia | I've spoken with some that have done that, and it is a YMMV based on the situation thing | 23:25 |
| TheJulia | At the same time, it can be handy if you want to update firmware to level set things prior to the hardware being used | 23:26 |
| cloudnull | ++ in the after now I need to go figure out firmware thing-a-mes | 23:27 |
| TheJulia | Hardware manager all the things! | 23:27 |
| cloudnull | we have proliant servers so it looks like it'll just work-tm but we'll see. | 23:27 |
| *** rama_y has joined #openstack-ironic | 23:28 | |
| TheJulia | I guess that is the other way with manual cleaning on proliants. That worked in mitaka, so if your using Newton you should be golden. Feel free to ping me if you have any issues, I can point you to the people to talk to if you do. | 23:32 |
| *** krtaylor has joined #openstack-ironic | 23:32 | |
| *** vinsh has quit IRC | 23:33 | |
| *** sabeen has joined #openstack-ironic | 23:33 | |
| *** vinsh has joined #openstack-ironic | 23:34 | |
| *** rbrud has quit IRC | 23:35 | |
| cloudnull | I'm stuck in mitaka | 23:41 |
| cloudnull | sadly | 23:41 |
| cloudnull | however i'm using IPA from newton :) | 23:41 |
| *** bnemec has quit IRC | 23:42 | |
| *** baoli_ has quit IRC | 23:45 | |
| TheJulia | Well, that ilo specific driver feature is out of band and does not use IPA. I seem to remember we had to fix the documentation, but nothing really major beyond that I remember. | 23:45 |
| *** ijw has quit IRC | 23:48 | |
| cloudnull | ah. good to know. | 23:50 |
| *** vinsh_ has joined #openstack-ironic | 23:57 | |
| *** vinsh has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
