| *** zackf has quit IRC | 00:08 | |
| *** bfournie has quit IRC | 00:19 | |
| rloo | jlvillal: I see you found the spec | 00:22 |
|---|---|---|
| jlvillal | rloo: :) | 00:23 |
| rloo | tonyb: wrt https://review.openstack.org/#/c/395517/. jroll and i were discussing today; we decided to decide tomorrow about whether to cancel that meeting or not | 00:23 |
| patchbot | patch 395517 - openstack-infra/irc-meetings - Remove unused Ironic v2 API meeting | 00:23 |
| rloo | tonyb: cuz yeah, we haven't had that meeting for awhile | 00:24 |
| tonyb | rloo: okay. Can you say that on the review so we know it's being looked at. | 00:24 |
| rloo | tonyb: sure, good idea | 00:24 |
| tonyb | rloo: thanks. Very kind | 00:25 |
| *** rbudden has quit IRC | 00:25 | |
| *** zackf has joined #openstack-ironic | 00:25 | |
| rloo | tonyb: thank you for pinging us :) | 00:27 |
| *** ElCoyote_ has joined #openstack-ironic | 00:29 | |
| *** zackf has quit IRC | 00:29 | |
| *** aNuposic has quit IRC | 00:29 | |
| *** rajinir has quit IRC | 00:36 | |
| *** rloo has quit IRC | 00:36 | |
| *** priteau has joined #openstack-ironic | 00:40 | |
| *** hoangcx has joined #openstack-ironic | 00:41 | |
| *** causten has joined #openstack-ironic | 00:43 | |
| *** priteau has quit IRC | 00:45 | |
| *** causten has quit IRC | 00:45 | |
| *** causten has joined #openstack-ironic | 00:46 | |
| *** causten has quit IRC | 00:46 | |
| *** causten has joined #openstack-ironic | 00:47 | |
| *** bfournie has joined #openstack-ironic | 00:50 | |
| *** causten has quit IRC | 00:51 | |
| *** causten has joined #openstack-ironic | 00:54 | |
| *** awiddersheim has quit IRC | 00:54 | |
| *** phuongnh has joined #openstack-ironic | 01:02 | |
| *** rama_y has quit IRC | 01:05 | |
| *** godp1301 has joined #openstack-ironic | 01:06 | |
| *** tuanluong has joined #openstack-ironic | 01:11 | |
| *** godp1301 has quit IRC | 01:18 | |
| *** g3ek has quit IRC | 01:20 | |
| *** Sukhdev has quit IRC | 01:23 | |
| *** g3ek has joined #openstack-ironic | 01:26 | |
| *** awiddersheim has joined #openstack-ironic | 01:30 | |
| jroll | tonyb: will vote tomorrow, thanks | 01:33 |
| *** penick has joined #openstack-ironic | 01:40 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Use __slots__ for save memory of drivers https://review.openstack.org/397103 | 01:41 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Use __slots__ for save memory of drivers https://review.openstack.org/397103 | 01:44 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Use __slots__ for save memory of drivers https://review.openstack.org/397103 | 01:44 |
| *** penick has quit IRC | 01:45 | |
| *** penick_ has joined #openstack-ironic | 01:45 | |
| *** fragatin_ has joined #openstack-ironic | 01:46 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Use __slots__ for save memory of drivers https://review.openstack.org/397103 | 01:47 |
| *** fragatin_ has quit IRC | 01:47 | |
| *** fragatin_ has joined #openstack-ironic | 01:48 | |
| *** zhangjl has joined #openstack-ironic | 01:48 | |
| *** fragatina has quit IRC | 01:49 | |
| *** fragatin_ has quit IRC | 01:50 | |
| *** causten has quit IRC | 01:56 | |
| *** causten has joined #openstack-ironic | 01:56 | |
| *** gcb has joined #openstack-ironic | 02:00 | |
| *** hw_wutianwei has joined #openstack-ironic | 02:16 | |
| *** causten has quit IRC | 02:19 | |
| *** wajdi has quit IRC | 02:22 | |
| *** mnaser has quit IRC | 02:24 | |
| *** syed__ has quit IRC | 02:25 | |
| *** shuquan has joined #openstack-ironic | 02:27 | |
| *** causten_ has joined #openstack-ironic | 02:30 | |
| *** mnaser has joined #openstack-ironic | 02:30 | |
| *** causten__ has joined #openstack-ironic | 02:33 | |
| *** causten_ has quit IRC | 02:36 | |
| *** shuquan_ has joined #openstack-ironic | 02:38 | |
| *** shuquan has quit IRC | 02:38 | |
| *** ijw_ has quit IRC | 02:40 | |
| *** priteau has joined #openstack-ironic | 02:41 | |
| *** godp1301 has joined #openstack-ironic | 02:44 | |
| *** priteau has quit IRC | 02:45 | |
| *** links has joined #openstack-ironic | 02:49 | |
| *** yuanying has quit IRC | 02:51 | |
| *** shuquan_ has quit IRC | 02:52 | |
| *** shuquan has joined #openstack-ironic | 02:54 | |
| *** hoangcx has quit IRC | 03:00 | |
| *** hoangcx has joined #openstack-ironic | 03:02 | |
| *** aaa801 has joined #openstack-ironic | 03:04 | |
| *** godp1301 has joined #openstack-ironic | 03:10 | |
| *** ijw has joined #openstack-ironic | 03:12 | |
| *** ijw has quit IRC | 03:16 | |
| *** godp1301 has quit IRC | 03:23 | |
| *** fragatina has joined #openstack-ironic | 03:29 | |
| *** yuanying has joined #openstack-ironic | 03:39 | |
| *** bdeschenes has quit IRC | 03:43 | |
| *** yuanying has quit IRC | 03:45 | |
| *** yuanying has joined #openstack-ironic | 03:49 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Use __slots__ for save memory of drivers https://review.openstack.org/397103 | 03:56 |
| *** sabeen has quit IRC | 04:01 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Use __slots__ for save memory of drivers https://review.openstack.org/397103 | 04:02 |
| *** sabeen has joined #openstack-ironic | 04:03 | |
| *** aparnav has joined #openstack-ironic | 04:04 | |
| *** godp1301 has joined #openstack-ironic | 04:06 | |
| openstackgerrit | octopuszhang proposed openstack/ironic: Update multitenancy docs https://review.openstack.org/397517 | 04:07 |
| *** penick_ has quit IRC | 04:12 | |
| *** chas_ has joined #openstack-ironic | 04:13 | |
| *** godp1301 has quit IRC | 04:15 | |
| *** chas_ has quit IRC | 04:17 | |
| *** godp1301 has joined #openstack-ironic | 04:18 | |
| *** causten__ has quit IRC | 04:26 | |
| *** causten has joined #openstack-ironic | 04:26 | |
| *** causten has quit IRC | 04:28 | |
| *** zackf has joined #openstack-ironic | 04:33 | |
| *** zackf has quit IRC | 04:37 | |
| *** shuquan has quit IRC | 04:39 | |
| *** diablo_rojo has joined #openstack-ironic | 05:01 | |
| *** Sukhdev has joined #openstack-ironic | 05:02 | |
| *** godp1301 has quit IRC | 05:02 | |
| *** khamtamtun has joined #openstack-ironic | 05:04 | |
| *** godp1301 has joined #openstack-ironic | 05:13 | |
| openstackgerrit | Merged openstack/proliantutils: Add py35 to tox environments https://review.openstack.org/396491 | 05:20 |
| *** vmud213 has joined #openstack-ironic | 05:20 | |
| *** khamtamtun has quit IRC | 05:21 | |
| *** fragatina has quit IRC | 05:30 | |
| *** fragatina has joined #openstack-ironic | 05:31 | |
| *** fragatina has quit IRC | 05:35 | |
| *** chas_ has joined #openstack-ironic | 05:36 | |
| *** shuquan has joined #openstack-ironic | 05:36 | |
| *** chas_ has quit IRC | 05:40 | |
| *** godp1301 has quit IRC | 05:41 | |
| *** syed__ has joined #openstack-ironic | 05:43 | |
| *** shuquan_ has joined #openstack-ironic | 05:47 | |
| *** shuquan has quit IRC | 05:48 | |
| *** mxburns has quit IRC | 05:57 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Use __slots__ for save memory of drivers https://review.openstack.org/397103 | 06:04 |
| tuanluong | Hello ironic | 06:04 |
| tuanluong | Hello jroll | 06:04 |
| *** jaosorior has joined #openstack-ironic | 06:07 | |
| *** sabeen has quit IRC | 06:14 | |
| *** khamtamtun has joined #openstack-ironic | 06:17 | |
| *** bks has joined #openstack-ironic | 06:21 | |
| *** khamtamtun has quit IRC | 06:35 | |
| *** jaosorior has quit IRC | 06:40 | |
| *** rbudden has joined #openstack-ironic | 06:40 | |
| *** jaosorior has joined #openstack-ironic | 06:40 | |
| *** zackf has joined #openstack-ironic | 06:41 | |
| *** priteau has joined #openstack-ironic | 06:42 | |
| *** mjura has joined #openstack-ironic | 06:43 | |
| *** moshele has joined #openstack-ironic | 06:46 | |
| *** zackf has quit IRC | 06:46 | |
| *** priteau has quit IRC | 06:47 | |
| *** khamtamtun has joined #openstack-ironic | 06:48 | |
| openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Add a new Hardware Manager for Mellanox NICs https://review.openstack.org/334816 | 06:53 |
| *** zackf has joined #openstack-ironic | 06:58 | |
| *** zackf has quit IRC | 07:02 | |
| *** wajdi has joined #openstack-ironic | 07:03 | |
| *** hshiina has joined #openstack-ironic | 07:03 | |
| *** wajdi has quit IRC | 07:05 | |
| *** chas_ has joined #openstack-ironic | 07:07 | |
| *** zackf has joined #openstack-ironic | 07:09 | |
| *** amotoki has joined #openstack-ironic | 07:09 | |
| *** ChubYann has quit IRC | 07:12 | |
| *** zackf has quit IRC | 07:13 | |
| *** rbudden has quit IRC | 07:20 | |
| *** rcernin has quit IRC | 07:23 | |
| *** _vsaienko has joined #openstack-ironic | 07:29 | |
| *** shuquan_ has quit IRC | 07:32 | |
| *** shuquan has joined #openstack-ironic | 07:32 | |
| *** rama_y has joined #openstack-ironic | 07:34 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-python-agent: Replace json.dumps(obj) with oslo_serialization.jsonutils.dumps(obj) https://review.openstack.org/363605 | 07:39 |
| *** makowals_ has joined #openstack-ironic | 07:50 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-python-agent: Python 3 Compatible JSON https://review.openstack.org/363605 | 07:51 |
| *** makowals has quit IRC | 07:53 | |
| *** syed__ has quit IRC | 07:55 | |
| *** rbartal has joined #openstack-ironic | 07:57 | |
| *** makowals_ has quit IRC | 08:00 | |
| *** makowals has joined #openstack-ironic | 08:01 | |
| *** rama_y has quit IRC | 08:03 | |
| *** shuquan_ has joined #openstack-ironic | 08:04 | |
| *** shuquan has quit IRC | 08:05 | |
| *** diablo_rojo has quit IRC | 08:10 | |
| *** ralonsoh has joined #openstack-ironic | 08:13 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic-specs: Support rolling upgrades https://review.openstack.org/299245 | 08:16 |
| *** shuquan_ has quit IRC | 08:17 | |
| *** yonglihe has joined #openstack-ironic | 08:18 | |
| *** shuquan has joined #openstack-ironic | 08:19 | |
| *** pcaruana has joined #openstack-ironic | 08:22 | |
| *** e0ne has joined #openstack-ironic | 08:24 | |
| *** e0ne has quit IRC | 08:25 | |
| vsaienk0 | morning lucasagomes, dtantsur, sambetts, vdrok, aarefie, pas-ha, milan, mgould and all Ironic'ers! | 08:32 |
| *** makowals_ has joined #openstack-ironic | 08:34 | |
| *** makowals has quit IRC | 08:37 | |
| *** Sukhdev has quit IRC | 08:39 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic-specs: Support rolling upgrades https://review.openstack.org/299245 | 08:39 |
| *** ccamacho has joined #openstack-ironic | 08:39 | |
| sergek_ | morning Ironic! | 08:40 |
| sergek_ | morning vsaienk0 :) | 08:40 |
| *** jpich has joined #openstack-ironic | 08:40 | |
| tuanluong | morning ironic | 08:45 |
| tuanluong | mornig vsaienk0, sambetts, vdrok pas-ha | 08:46 |
| *** jaosorior is now known as jaosorior_lunch | 08:52 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui: Imported Translations from Zanata https://review.openstack.org/397588 | 08:57 |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-ironic | 09:00 | |
| *** athomas has joined #openstack-ironic | 09:03 | |
| *** betherly has joined #openstack-ironic | 09:03 | |
| *** dtantsur|afk is now known as dtantsur | 09:04 | |
| dtantsur | morning Ironic | 09:04 |
| *** _vsaienko has quit IRC | 09:12 | |
| *** openstackgerrit has quit IRC | 09:18 | |
| *** openstackgerrit has joined #openstack-ironic | 09:18 | |
| *** clsacramento has joined #openstack-ironic | 09:21 | |
| pas-ha | morning dtantsur vsaienk0 and others :) | 09:21 |
| dtantsur | o/ | 09:21 |
| *** ohamada has joined #openstack-ironic | 09:24 | |
| *** priteau has joined #openstack-ironic | 09:25 | |
| *** lucas-afk is now known as lucasagomes | 09:28 | |
| lucasagomes | morning all | 09:29 |
| dtantsur | morning lucasagomes | 09:38 |
| aarefiev | morning! | 09:41 |
| *** jaosorior_lunch is now known as jaosorior | 09:42 | |
| *** roeyc has joined #openstack-ironic | 09:42 | |
| *** _vsaienko has joined #openstack-ironic | 09:43 | |
| xek | morning :) | 09:45 |
| *** derekh has joined #openstack-ironic | 09:49 | |
| *** tuanluong has quit IRC | 09:57 | |
| *** snozhka has quit IRC | 10:00 | |
| *** hoangcx has quit IRC | 10:06 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic-specs: API Evolution - ETAG identifiers https://review.openstack.org/381991 | 10:06 |
| vdrok | morning ironic, vsaienk0 sergek_ dtantsur lucasagomes aarefiev and xek ! | 10:15 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Create new fact for dib_env_vars https://review.openstack.org/397649 | 10:15 |
| lucasagomes | o/ morning | 10:15 |
| *** e0ne has joined #openstack-ironic | 10:19 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 10:23 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 10:25 |
| *** mgould|afk is now known as mgould | 10:27 | |
| mgould | morning Ironic! | 10:27 |
| dtantsur | morning mgould | 10:28 |
| sambetts|afk | Morning all | 10:30 |
| *** sambetts|afk is now known as sambetts] | 10:30 | |
| *** shuquan has quit IRC | 10:31 | |
| *** fragatina has joined #openstack-ironic | 10:31 | |
| *** shuquan has joined #openstack-ironic | 10:33 | |
| dtantsur | morning sambetts] | 10:33 |
| patchbot | Error: Spurious "]". You may want to quote your arguments with double quotes in order to prevent extra brackets from being evaluated as nested commands. | 10:33 |
| dtantsur | patchbot, nope! | 10:33 |
| *** dtantsur is now known as dtantsur|bbl | 10:34 | |
| *** watanabe_isao has joined #openstack-ironic | 10:35 | |
| *** fragatina has quit IRC | 10:36 | |
| *** milan has joined #openstack-ironic | 10:38 | |
| milan | morning Ironic! :) | 10:38 |
| *** khamtamtun has quit IRC | 10:39 | |
| *** shuquan has quit IRC | 10:45 | |
| *** phuongnh has quit IRC | 10:50 | |
| mgould | morning dtantsur|bbl sambetts milan vdrok lucasagomes xek pas-ha vsaienk0 sergek_ | 10:50 |
| milan | morning mgould! :) | 10:50 |
| lucasagomes | mgould, hi there! | 10:52 |
| sambetts] | is py34 going away soon ?? | 10:53 |
| *** sambetts] is now known as sambetts | 10:53 | |
| mgould | sambetts: in general, or from OpenStack? | 10:57 |
| *** _vsaienko has quit IRC | 10:59 | |
| *** khamtamtun has joined #openstack-ironic | 11:01 | |
| *** mgoddard_ has joined #openstack-ironic | 11:01 | |
| sambetts | mgould: I'm on latest debian and I discovered a couple of days ago I can only install py35 now not py34 | 11:03 |
| sambetts | mgould: so I wondered if py34 was being replaced by py35 completely | 11:04 |
| *** mgoddard has quit IRC | 11:04 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface APIs https://review.openstack.org/327046 | 11:05 |
| mgould | python 3.4.5 was released on 2016-06-27 | 11:06 |
| mgould | python 3.5.1 was released on 2015-12-07 | 11:07 |
| mgould | oh wait, there's also 3.5.2, which was presumably released later | 11:07 |
| mgould | no, sorry, also released on 2016-06-27 | 11:08 |
| sambetts | well thats confusing... | 11:08 |
| mgould | so I guess the PSF is still maintaining the 3.4.x branch | 11:08 |
| sambetts | seems like it then, I wonder why 3.4 has gone away in debian then, it definatly used to be avaivle | 11:10 |
| vdrok | morning mgould and sambetts | 11:13 |
| sambetts | o/ vdrok | 11:13 |
| vdrok | and good morning milan | 11:13 |
| milan | morning sambetts, vdrok! :) | 11:14 |
| *** zhangjl has left #openstack-ironic | 11:14 | |
| *** mgoddard_ has quit IRC | 11:17 | |
| *** mgoddard has joined #openstack-ironic | 11:18 | |
| *** zackf has joined #openstack-ironic | 11:22 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-specs: Update interface-attach-detach-api https://review.openstack.org/397690 | 11:23 |
| vsaienk0 | morning sambetts | 11:24 |
| sambetts | hey vsaienk0 | 11:24 |
| milan | morning vsaienk0! :) | 11:24 |
| vsaienk0 | sambetts: could you please check ^ I think we need to update return value for vif_list() | 11:24 |
| sambetts | vsaienk0: the example I gave was the plan, e.g. just a list of IDs, do we need the more complicated structure? | 11:25 |
| sambetts | vsaienk0: when its only even going to have one key | 11:26 |
| *** zackf has quit IRC | 11:26 | |
| vsaienk0 | sambetts: it will allow to do not implement custom print logic on ironicclient side, and will allow to add more data in future to return value, for example ironic port/portgroup uuid with which VIF was associated | 11:28 |
| vsaienk0 | sambetts: user may want to know if VIF was associated to port or portgroup? printing just VIF uuids is not super informational, user will required to walk over all ports/portgroups to find object with which VIF was associated | 11:31 |
| *** jkilpatr has quit IRC | 11:31 | |
| sambetts | vsaienk0: IMO users shouldn't know which VIF was associtated to a port, thats the point of the absraction | 11:32 |
| sambetts | vsaienk0: but I understand the point about custom print logic | 11:32 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic-specs: API Evolution - ETAG identifiers https://review.openstack.org/381991 | 11:34 |
| vsaienk0 | sambetts: but it is an admin API /node/{node_ident}/vifs ? It may be useful for administrator to get that information, not only list of VIFS | 11:37 |
| *** zackf has joined #openstack-ironic | 11:38 | |
| sambetts | vsaienk0: there is no guarentee how a network interface implementation maps the vifs to the pifs, an attached vif might not even be mapped to a pif until later on in the deploy process | 11:39 |
| vsaienk0 | sambetts: attaching vif to a node is a synchronous operation right? | 11:41 |
| sambetts | vsaienk0: yes | 11:41 |
| sambetts | vsaienk0: but attaching it to the node, doesn't mean mapping it to a port nessesarly | 11:41 |
| vsaienk0 | sambetts: so if request with attach vif was completed it means that network driver.attach_vif() set vif_port_id to any node pif right? | 11:42 |
| *** zackf has quit IRC | 11:43 | |
| *** links has quit IRC | 11:43 | |
| *** _vsaienko has joined #openstack-ironic | 11:43 | |
| sambetts | vsaienk0: a request to attach vif, just means "attach this vif to this node" in the neutron network interface implementation that means mapping it to a pif, but in other cases the network interface implementation might store the vifs somewhere else to do something special during configure tenant networks | 11:44 |
| vsaienk0 | sambetts: vif should be anyway associated with ironic pif somehow, because it should contain mac address of some pif | 11:47 |
| vsaienk0 | or you mean that mac address of VIF may be updated later during tenant netwroks configuration? | 11:48 |
| sambetts | vsaienk0: ironic will upate the mac address if it needs too, but in some network interface implementations the hardware has programmable mac addresses so uses the neutron generated one | 11:48 |
| vsaienk0 | sambetts: I see, return additional information may conflict with custom network drivers, but we can change return value to [{'id': 'vif-id'}] to avoid ironicclient customization | 11:53 |
| sambetts | vsaienk0: exactly :) | 11:53 |
| *** dtantsur|bbl is now known as dtantsur | 11:53 | |
| sambetts | vsaienk0: only guarenteed piece of information is the ID | 11:54 |
| vsaienk0 | sambetts: nice, can you please review the patch than https://review.openstack.org/#/c/397690/ | 11:54 |
| patchbot | patch 397690 - ironic-specs - Update interface-attach-detach-api | 11:54 |
| sambetts | yup looking at it now | 11:54 |
| *** bks has quit IRC | 11:57 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface APIs https://review.openstack.org/327046 | 11:58 |
| *** snozhka has joined #openstack-ironic | 11:58 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Retry connecting to vm after power-on https://review.openstack.org/394901 | 12:01 |
| *** priteau has quit IRC | 12:03 | |
| *** jcoufal has joined #openstack-ironic | 12:12 | |
| *** watanabe_isao has quit IRC | 12:13 | |
| *** jkilpatr has joined #openstack-ironic | 12:14 | |
| *** priteau has joined #openstack-ironic | 12:17 | |
| *** gcb has quit IRC | 12:19 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification https://review.openstack.org/348437 | 12:23 |
| vsaienk0 | sambetts: do we need to support pagination for vif_list() ? | 12:27 |
| vsaienk0 | sambetts: theoretically vif_list may be too long, the same as number of node ports | 12:27 |
| sambetts | vsaienk0: I guess we probably should | 12:28 |
| *** dprince has joined #openstack-ironic | 12:33 | |
| *** ccamacho is now known as ccamacho|lunch | 12:33 | |
| vsaienk0 | sambetts: it will require a lot of changes to API implementation | 12:33 |
| sambetts | will it? | 12:34 |
| * sambetts hasn't dug into pagination much | 12:35 | |
| *** priteau has quit IRC | 12:37 | |
| vsaienk0 | sambetts: it is implemented at db api layer for ports and other objects | 12:37 |
| sambetts | vsaienk0: how is it exposed through the API though? | 12:37 |
| sambetts | vsaienk0: its just a URL query isn't it? | 12:38 |
| vsaienk0 | sambetts: yes | 12:38 |
| sambetts | then we only need to parse that query in the controller and limit the returned data right? | 12:38 |
| vsaienk0 | sambetts: something like that | 12:39 |
| *** bdeschenes has joined #openstack-ironic | 12:42 | |
| *** vmud213 has quit IRC | 12:43 | |
| *** bdeschenes has quit IRC | 12:46 | |
| sambetts | vsaienk0: how do I run a multinode job? recheck experimental? | 12:47 |
| vsaienk0 | sambetts: recheck doesn't work for experimental pipeline, you have to do 'check experimental' | 12:47 |
| *** hw_wutianwei has quit IRC | 12:47 | |
| sambetts | ah, thats what I wanted thanks :D | 12:47 |
| *** priteau has joined #openstack-ironic | 12:48 | |
| vsaienk0 | sambetts: we have only 3 patches to ironic left and 1 to devstack-gate which already has +2 related to multinode | 12:49 |
| sambetts | vsaienk0: having to deal with pretty nasty merge conflicts from those patches on my https://review.openstack.org/392959 patch | 12:49 |
| patchbot | patch 392959 - ironic - Rework Ironic devstack baremetal network simulation | 12:49 |
| vsaienk0 | sambetts: yeah, they touch networking part but nothing serious, mostly just skip some steps based on node role | 12:51 |
| sambetts | yeah thats what I'm adjust my patch to support now, it should actually simplify things quite a bit | 12:52 |
| *** ipukha has joined #openstack-ironic | 12:52 | |
| vsaienk0 | sambetts: what do you think should go first multinode or your network refactoring? | 12:53 |
| *** lucasagomes is now known as lucas-hungry | 12:54 | |
| sambetts | vsaienk0: trying to workout how much work it'll be to make multinode work in my refactor | 12:54 |
| milan | aarefiev, dtantsur, sambetts morning guys! :) if you wish to vote for our new operator name: not_in/nin/out respond please to the mail thread thx! ;) | 12:56 |
| vsaienk0 | sambetts: multinode is low hanging fruit now, so I think it is better to refactor networking with multinode support | 12:56 |
| milan | bike shedding \o/ :D | 12:56 |
| *** e0ne has quit IRC | 12:56 | |
| * dtantsur hates naming things | 12:56 | |
| *** e0ne has joined #openstack-ironic | 12:56 | |
| milan | dtantsur better vote now otherwise you may have to review an ugly name :P | 12:57 |
| milan | btw so far not_in winning | 12:58 |
| milan | over nin | 12:58 |
| dtantsur | I prefer not_in too, but I see that they already have neq, so nin may win in the end. I don't care too much. | 12:58 |
| *** bfournie has quit IRC | 12:59 | |
| sambetts | nin definatly is more consistant with the existing forms | 12:59 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Add database fields for all interfaces https://review.openstack.org/396681 | 12:59 |
| *** ralonsoh_ has joined #openstack-ironic | 13:04 | |
| *** ralonsoh has quit IRC | 13:05 | |
| aarefiev | yeah, I thought about nin too, but we can live with not_in too :) | 13:05 |
| milan | sure :) | 13:08 |
| *** trown|outtypewww is now known as trown | 13:11 | |
| *** priteau has quit IRC | 13:12 | |
| jroll | morning y'all | 13:13 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add ironic resources CRUD notifications https://review.openstack.org/356541 | 13:14 |
| sambetts | o/ jroll | 13:14 |
| sambetts | jroll: can you remember what the name of that project that was mentioned at the summit for proiding a standalone scheduler for standalone ironic? | 13:15 |
| jroll | sambetts: nimble? | 13:15 |
| jroll | it isn't just a scheduler, it's nova with all the vm stuff ripped out :| | 13:16 |
| *** khamtamtun has quit IRC | 13:16 | |
| dtantsur | morning jroll | 13:16 |
| dtantsur | jroll, could you please check the remaining stable patches (http://bit.ly/2fE0QTj)? I'd like to make stable/newton releases soon. | 13:17 |
| jroll | dtantsur: totes | 13:17 |
| *** hamzy has quit IRC | 13:18 | |
| sambetts | jroll: ah :/ | 13:22 |
| *** priteau has joined #openstack-ironic | 13:23 | |
| milan | morning jroll! :) | 13:24 |
| *** priteau has quit IRC | 13:25 | |
| *** causten has joined #openstack-ironic | 13:25 | |
| * milan relocates | 13:27 | |
| sambetts | jroll: well thats disappointing, I thought it was going to be a thing for standalone but it all hooked into OpenStack still | 13:27 |
| jroll | sambetts: indeed | 13:27 |
| *** milan has quit IRC | 13:28 | |
| *** causten has quit IRC | 13:29 | |
| *** bfournie has joined #openstack-ironic | 13:29 | |
| aparnav | Hey jroll, Could you please take a look at this RFE https://bugs.launchpad.net/ironic/+bug/1641857 ? | 13:33 |
| openstack | Launchpad bug 1641857 in Ironic "[RFE] Allow IPA to perform disk erase using thirdparty storage controller" [Undecided,New] | 13:33 |
| jroll | aparnav: that works today, with erase_block_device, afaik | 13:34 |
| jroll | aparnav: a hardware manager can implement erase_block_device, so it's handled per-disk | 13:35 |
| *** shuquan has joined #openstack-ironic | 13:35 | |
| *** sborkows has joined #openstack-ironic | 13:35 | |
| aparnav | jroll, I am trying to erase the disks which are not visible to the OS and are in turn attached to the node using a storage controller | 13:36 |
| jroll | hrm | 13:36 |
| aparnav | jroll, so those disks won't be listed as the block device | 13:36 |
| jroll | right right | 13:37 |
| jroll | aparnav: so I'd honestly say that your hardware manager should include an extra clean step for this, but the docstring for erase_devices also implies you can just override this | 13:38 |
| jroll | aparnav: I'm going to leave a comment and ask JayF to look at this and see what he thinks | 13:39 |
| *** yolanda has quit IRC | 13:39 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Make all IronicExceptions RPC-serializable https://review.openstack.org/396355 | 13:39 |
| vdrok | good morning jroll | 13:40 |
| *** shuquan has quit IRC | 13:40 | |
| jroll | JayF: when you're around, I'd love your eyes on https://bugs.launchpad.net/ironic/+bug/1641857 | 13:41 |
| openstack | Launchpad bug 1641857 in Ironic "[RFE] Allow IPA to perform disk erase using thirdparty storage controller" [Undecided,New] | 13:41 |
| jroll | morning vdrok | 13:41 |
| *** jheroux has joined #openstack-ironic | 13:41 | |
| aparnav | jroll, it can be added as a extra clean step. As we have to fall back to 'erase_device', I thought it will be better to add it in erase_devices instead of adding a new clean step | 13:41 |
| jroll | aparnav: right, I guess I'm not sure yet :) | 13:42 |
| *** jcoufal_ has joined #openstack-ironic | 13:43 | |
| aparnav | jroll, thanks, I will wait to see what others think about this. | 13:44 |
| jroll | thanks :) | 13:44 |
| sambetts | vsaienk0: do you simulate multinode downstream? I'm just wondering what the best way to test the change is | 13:45 |
| *** jcoufal has quit IRC | 13:46 | |
| vsaienk0 | sambetts: I setup multinode manually (require to setup vxlan tunnel between nodes manually), but jlvillal worked on automation I don't know if he succeed. | 13:47 |
| jroll | dtantsur: backports queue is clear | 13:47 |
| dtantsur | cool! | 13:47 |
| *** crinkle_ has joined #openstack-ironic | 13:48 | |
| *** aparnav has quit IRC | 13:48 | |
| *** priteau has joined #openstack-ironic | 13:48 | |
| *** crinkle has quit IRC | 13:49 | |
| sambetts | vsaienk0: ah ok, do you use something like virtual box with 2 VMs? | 13:50 |
| *** ccamacho|lunch is now known as ccamacho | 13:50 | |
| *** chas_ has quit IRC | 13:52 | |
| *** chas_ has joined #openstack-ironic | 13:52 | |
| vsaienk0 | sambetts: yes, 2 KVM vms setup vxlan tunnel between them, than run stack.sh on primary node, update IPs in local.conf on subnode and run stach.sh on subnode. | 13:53 |
| *** causten has joined #openstack-ironic | 13:53 | |
| *** chas__ has joined #openstack-ironic | 13:54 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node maintenance notifications https://review.openstack.org/396239 | 13:54 |
| sambetts | vsaienk0: ah ok, I use virtual box on windows, so I'll have to experiment with the best way to simulate upstream | 13:54 |
| *** yolanda has joined #openstack-ironic | 13:55 | |
| vsaienk0 | sambetts: all that is required - L3 connection between VMs, even no need to plug them to the same network | 13:56 |
| *** chas_ has quit IRC | 13:56 | |
| *** _vsaienko has quit IRC | 13:57 | |
| sambetts | vsaienk0: oh ok, makes sense | 13:57 |
| *** crinkle_ is now known as crinkle | 13:58 | |
| *** khamtamtun has joined #openstack-ironic | 13:58 | |
| *** Goneri has joined #openstack-ironic | 13:58 | |
| *** vsaienko has joined #openstack-ironic | 14:00 | |
| *** rloo has joined #openstack-ironic | 14:01 | |
| *** causten has quit IRC | 14:01 | |
| *** causten has joined #openstack-ironic | 14:02 | |
| *** lucas-hungry is now known as lucasagomes | 14:02 | |
| *** Goneri has quit IRC | 14:03 | |
| *** causten has quit IRC | 14:06 | |
| *** yolanda has quit IRC | 14:07 | |
| *** aparnav has joined #openstack-ironic | 14:12 | |
| *** pcaruana has quit IRC | 14:13 | |
| *** awiddersheim has quit IRC | 14:15 | |
| *** Goneri has joined #openstack-ironic | 14:15 | |
| *** rbudden has joined #openstack-ironic | 14:15 | |
| *** khamtamtun has quit IRC | 14:16 | |
| *** lindycoder has joined #openstack-ironic | 14:18 | |
| *** awiddersheim has joined #openstack-ironic | 14:19 | |
| *** ElCoyote_ has quit IRC | 14:22 | |
| *** yolanda has joined #openstack-ironic | 14:23 | |
| *** chas__ has quit IRC | 14:24 | |
| *** chas_ has joined #openstack-ironic | 14:25 | |
| *** vinsh has quit IRC | 14:25 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/396134 | 14:28 |
| *** baoli has joined #openstack-ironic | 14:29 | |
| *** baoli has quit IRC | 14:29 | |
| openstackgerrit | Stephen Finucane proposed openstack/ironic: Remove ClusteredComputeManager https://review.openstack.org/397762 | 14:29 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Add node (database and objects) fields for all interfaces https://review.openstack.org/396681 | 14:29 |
| *** chas_ has quit IRC | 14:30 | |
| openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Add interface attach/detach support https://review.openstack.org/364420 | 14:31 |
| jroll | vsaienk0: sambetts: for multitenant network testing, do we use any of tempests network_basic_ops tests or is it all in the ironic tempest plugin? | 14:35 |
| vsaienk0 | jroll: all is in ironic tempest plugin | 14:35 |
| *** baoli_ has joined #openstack-ironic | 14:35 | |
| jroll | cool, thanks | 14:35 |
| *** zackf has joined #openstack-ironic | 14:36 | |
| *** jaosorior has quit IRC | 14:36 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Turn off tempest's multitenant network tests https://review.openstack.org/397767 | 14:37 |
| jroll | ignore that for a moment :) | 14:38 |
| * jroll needs to make sure tempest is okay with that new flag's name | 14:38 | |
| sambetts | jroll: do we need that? don't we do that by just passing in the regex? or is this for running full tempest with no regex? | 14:40 |
| jroll | sambetts: that's a good point, but I'd say yes for the latter | 14:41 |
| jroll | sambetts: just replacing existing 'is baremetal enabled' checks with something more sane | 14:41 |
| *** vsaienko has quit IRC | 14:41 | |
| jroll | e.g. https://review.openstack.org/#/c/355586/16/tempest/scenario/manager.py | 14:41 |
| patchbot | patch 355586 - tempest - Removing baremetal tests from tempest tree | 14:41 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Add node (database and objects) fields for all interfaces https://review.openstack.org/396681 | 14:41 |
| sambetts | jroll: if we're running full tempest shouldn't we test all the multitenant stuff? | 14:41 |
| jroll | sambetts: we aren't yet... but yes | 14:42 |
| jroll | sambetts: first goal is get stuff out of the tempest tree, then we'll move on to the plugin repo thing, and then make our tempest testing not terrible | 14:42 |
| jroll | :) | 14:42 |
| *** tuan_luong has joined #openstack-ironic | 14:42 | |
| jroll | hope that makes sense | 14:43 |
| sambetts | jroll: totally :) | 14:44 |
| *** godp1301 has joined #openstack-ironic | 14:44 | |
| *** godp1301_ has joined #openstack-ironic | 14:45 | |
| *** khamtamtun has joined #openstack-ironic | 14:45 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: DNM: Testing 355586 https://review.openstack.org/397775 | 14:47 |
| jroll | aaaand making sure everything works | 14:47 |
| *** godp1301 has quit IRC | 14:49 | |
| *** bdeschenes has joined #openstack-ironic | 14:49 | |
| *** khamtamtun has quit IRC | 14:50 | |
| *** makowals_ has quit IRC | 14:52 | |
| *** makowals has joined #openstack-ironic | 14:52 | |
| *** causten has joined #openstack-ironic | 14:59 | |
| *** rbudden has quit IRC | 14:59 | |
| *** hshiina has quit IRC | 15:00 | |
| jroll | dtantsur: sanity check me on tempest plugin repos? https://review.openstack.org/#/c/397781/ | 15:02 |
| patchbot | patch 397781 - openstack-infra/project-config - Create repo for ironic tempest plugins | 15:02 |
| *** Goneri has quit IRC | 15:02 | |
| dtantsur | jroll, I dunno if they'll expect it to be tempest-plugin-ironic or ironic-tempest-plugin. have you had a chance to talk to qa folks about their expectation? | 15:03 |
| sambetts | or tempest-ironic-plugin ;) | 15:04 |
| jroll | dtantsur: the projects with separate plugins seem to be *tempest-plugin http://docs.openstack.org/developer/tempest/plugin-registry.html | 15:04 |
| jroll | though, there aren't many, wow | 15:04 |
| dtantsur | openstack/tempest-horizon | 15:04 |
| jroll | I thought most were separate plugins | 15:04 |
| jroll | ugh | 15:04 |
| *** vinsh has joined #openstack-ironic | 15:04 | |
| dtantsur | openstack/designate-tempest-plugin | 15:04 |
| jroll | I'll ask | 15:05 |
| dtantsur | yes please | 15:05 |
| jroll | yeah, and trove-tempest-plugin | 15:05 |
| jroll | I hate naming things | 15:05 |
| dtantsur | naming \o/ | 15:05 |
| jroll | (also, why should qa care) | 15:05 |
| dtantsur | well, they're an obvious point of contact for such questions.. and they might issue a policy on that | 15:06 |
| jroll | right, I meant more of ... why does it matter :) | 15:06 |
| *** chas_ has joined #openstack-ironic | 15:08 | |
| *** chas__ has joined #openstack-ironic | 15:10 | |
| *** syed__ has joined #openstack-ironic | 15:10 | |
| *** chas_ has quit IRC | 15:13 | |
| *** vinsh has quit IRC | 15:14 | |
| dtantsur | for me - it doesn't :) I just don't want end up having to renaming the repo | 15:15 |
| *** vinsh has joined #openstack-ironic | 15:15 | |
| *** chas__ has quit IRC | 15:15 | |
| jroll | right | 15:15 |
| *** Goneri has joined #openstack-ironic | 15:15 | |
| *** vinsh has quit IRC | 15:16 | |
| *** vinsh has joined #openstack-ironic | 15:17 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Update CRUD and node states notification spec https://review.openstack.org/397792 | 15:17 |
| *** vinsh has quit IRC | 15:18 | |
| *** igordcar1 has quit IRC | 15:19 | |
| *** igordcard has joined #openstack-ironic | 15:19 | |
| dtantsur | meh, I tried to make the next driver comp patch small, but it pulls more and more things in >_< | 15:20 |
| *** vinsh has joined #openstack-ironic | 15:21 | |
| yuriyz | hi ironic mariojv looks like we need new notification action for console https://review.openstack.org/#/c/397792/ | 15:26 |
| patchbot | patch 397792 - ironic-specs - Update CRUD and node states notification spec | 15:26 |
| mariojv | thanks yuriyz, already opened it and taking a look | 15:26 |
| mariojv | yuriyz: +1 | 15:29 |
| *** wajdi has joined #openstack-ironic | 15:29 | |
| *** rbrud has joined #openstack-ironic | 15:30 | |
| NobodyCam | Good Morning Ironic'ers :) | 15:33 |
| *** nicodemos has joined #openstack-ironic | 15:34 | |
| dtantsur | morning NobodyCam | 15:34 |
| NobodyCam | :) | 15:34 |
| *** mjturek has joined #openstack-ironic | 15:35 | |
| openstackgerrit | Karthik S proposed openstack/ironic-inspector-specs: Spec for retrieving NUMA node information https://review.openstack.org/396147 | 15:35 |
| jkilpatr | dtantsur, do you know why ironic node-show has fields for introspection details that are never populated | 15:36 |
| jkilpatr | ? | 15:36 |
| dtantsur | jkilpatr, which fields are you referring to? | 15:36 |
| jkilpatr | inspection_start_at and inspection_finished_at | 15:36 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Support defining and loading hardware types https://review.openstack.org/336626 | 15:37 |
| mjturek | rloo lucasagomes vsaienk0: was wondering if you could re-review https://review.openstack.org/#/c/185987 now that the RFE has been accepted | 15:37 |
| patchbot | patch 185987 - ironic - Enable PXE for systems using petitboot | 15:37 |
| dtantsur | jkilpatr, they do get populated when you use "inspect" provision state (but not when you use ironic-inspector API/CLI directly) | 15:37 |
| lucasagomes | mjturek, hi there, will do | 15:37 |
| mjturek | thanks :) | 15:37 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add user and project domains to ironic context https://review.openstack.org/295371 | 15:37 |
| lucasagomes | just give me few minutes | 15:37 |
| mjturek | no rush | 15:38 |
| jkilpatr | dtantsur, so if I set nodes to the inspect provision state introspection is started exactly as if I had run the direct command? | 15:38 |
| dtantsur | jkilpatr, not exactly, it depends on a driver, for exampled | 15:38 |
| dtantsur | ref: http://docs.openstack.org/developer/ironic/deploy/inspection.html | 15:39 |
| jkilpatr | dtantsur, thanks I'll go off an investigate, because watching for the node to turn itself off is a horrible way to gather metrics on introspection | 15:40 |
| lucasagomes | mjturek, +2, the only update since my last review was the method's name | 15:41 |
| sambetts | jkilpatr: we're are adding a new command for ironic-inspector for listing the state of inspection too, https://review.openstack.org/#/c/375045/ | 15:42 |
| patchbot | patch 375045 - ironic-inspector-specs - Endpoint to list introspection statuses | 15:42 |
| openstackgerrit | Stephen Finucane proposed openstack/ironic: Remove ClusteredComputeManager https://review.openstack.org/397762 | 15:42 |
| *** rama_y has joined #openstack-ironic | 15:43 | |
| vdrok | good morning NobodyCam | 15:43 |
| *** aparnav has quit IRC | 15:43 | |
| NobodyCam | Good Morning Vdok :) | 15:44 |
| *** jvgrant__ has quit IRC | 15:45 | |
| *** jvgrant has joined #openstack-ironic | 15:45 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Update CRUD and node states notification spec https://review.openstack.org/397792 | 15:46 |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Consolidate IMM date string parsing https://review.openstack.org/397801 | 15:46 |
| jkilpatr | dtantsur, pxe_ipmitool does not support inspect | 15:46 |
| *** milan has joined #openstack-ironic | 15:47 | |
| *** openstackgerrit has quit IRC | 15:48 | |
| *** chas_ has joined #openstack-ironic | 15:48 | |
| *** openstackgerrit has joined #openstack-ironic | 15:48 | |
| dtantsur | jkilpatr, it has to be enabled in configuration | 15:49 |
| dtantsur | [inspector]enabled=True | 15:49 |
| jlvillal | sambetts: I do have a way to simulate a multi-node setup. | 15:49 |
| *** aparnav has joined #openstack-ironic | 15:49 | |
| jlvillal | sambetts: If you look in the devstack-gate-test repository there is a multi.*.sh script and an ansible playbook. | 15:49 |
| *** absubram has joined #openstack-ironic | 15:50 | |
| jlvillal | sambetts: Kind of OSIC cloud dependent, but I think it can give you an idea. Basically it is setting up the /etc/nodepool/ directory on each node | 15:50 |
| sambetts | jlvillal: I don't think it'll work for my situation becuase I want to use 2 virtualbox VMs, but I think I'm working it out now :) | 15:51 |
| nicodemos | lucasagomes and/or vdrok. I have a question about: https://review.openstack.org/#/c/396698/. I get this bug from another patch (https://review.openstack.org/#/c/358041), and forgot to move the unittests too... so I copy the tests as they were and then resolve the conflicts later? | 15:51 |
| patchbot | patch 396698 - ironic - Missing or not needed return values in OneView dep... | 15:51 |
| patchbot | patch 358041 - ironic - Reusing oneview_client when possible | 15:51 |
| openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent: Trying to create a package for biosdevname https://review.openstack.org/396406 | 15:51 |
| jlvillal | sambetts: You can look at it and see what is done. The basic thing is making the /etc/nodepool/directory. And that the master can SSH into the subnode | 15:51 |
| *** chas_ has quit IRC | 15:53 | |
| jkilpatr | dtantsur, so wait if I had to enable inspector what was I using before? | 15:53 |
| jkilpatr | there is a difference between the pxe image and some other method? | 15:53 |
| * jkilpatr is confused | 15:54 | |
| dtantsur | jkilpatr, no idea, sorry. I don't have context and what you were using, probably ironic-inspector CLI directly | 15:54 |
| vdrok | nicodemos: not sure I get it, https://review.openstack.org/#/c/358041 was a part of https://review.openstack.org/#/c/396698/, then you decided to split those and forgot to move the unittests? | 15:54 |
| patchbot | patch 358041 - ironic - Reusing oneview_client when possible | 15:54 |
| patchbot | patch 396698 - ironic - Missing or not needed return values in OneView dep... | 15:54 |
| lucasagomes | nicodemos, lemme check | 15:54 |
| lucasagomes | nicodemos, so, if patch https://review.openstack.org/#/c/396698/ conflicts with https://review.openstack.org/#/c/358041 (or vice-versa) you can always create a chain of changes | 15:56 |
| patchbot | patch 396698 - ironic - Missing or not needed return values in OneView dep... | 15:56 |
| patchbot | patch 358041 - ironic - Reusing oneview_client when possible | 15:56 |
| lucasagomes | you can rebase a patch on top of the other and resolve the conflicts on that chain | 15:56 |
| nicodemos | vdrok, kind of. Dmitry asked to seprate the bug in a new patch, but the test was created in the previous patch. | 15:57 |
| vdrok | nicodemos: I think the best way is to add unittests in https://review.openstack.org/#/c/358041 and rebase https://review.openstack.org/#/c/396698/ on top. what lucasagomes already said :) | 15:57 |
| patchbot | patch 358041 - ironic - Reusing oneview_client when possible | 15:57 |
| patchbot | patch 396698 - ironic - Missing or not needed return values in OneView dep... | 15:57 |
| lucasagomes | nicodemos, right, so just put the new patch on top of the one dmitry asked to separate | 15:57 |
| nicodemos | ok. Thanks. vdrok lucasagomes | 15:57 |
| lucasagomes | nicodemos, as two commits, and then "git review" them | 15:58 |
| lucasagomes | gerrit will take care of keeping the chain in place | 15:58 |
| *** rajinir has joined #openstack-ironic | 16:01 | |
| *** rpioso has joined #openstack-ironic | 16:02 | |
| openstackgerrit | Merged openstack/python-ironicclient: Adds --chassis-uuid to osc 'baremetal node set' https://review.openstack.org/363999 | 16:02 |
| *** pcaruana has joined #openstack-ironic | 16:02 | |
| JayF | jroll: aparnav: I'm going to suggest we reject that RFE, and instead implement the feature in a third party hardware manager. Basically it looks like even detecting these disks require a third-party tool, that I don't think it's wise to ship with our ramdisk | 16:04 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Update CRUD and node states notification spec https://review.openstack.org/397792 | 16:04 |
| *** sborkows has quit IRC | 16:04 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Test stable/newton to see if working gate https://review.openstack.org/397803 | 16:05 |
| jroll | JayF: well, the RFE is about using hw manager things to dispatch erase_devices, rather than dispatching erase_block_device for each disk | 16:07 |
| jroll | JayF: it isn't about writing the actual codd | 16:08 |
| jroll | code | 16:08 |
| JayF | jroll: someone could simply subclass GenericHardwareManager and override list_block_devices to also detect the "undetectable" devices | 16:09 |
| JayF | jroll: I don't know what needs to change here | 16:09 |
| jroll | ah, that's true | 16:09 |
| jroll | JayF: the misinformation needs to change, I guess? :) | 16:09 |
| jroll | can you leave a comment on the rfe for aparnav ? | 16:10 |
| JayF | I already left one comment, I'll also paste this short irc chat | 16:10 |
| *** Goneri has quit IRC | 16:11 | |
| JayF | jroll: although to be clear; for this use case I do *not* reccomend that course of action | 16:12 |
| *** roeyc has quit IRC | 16:12 | |
| jroll | JayF: well, I guess I'm looking for your expertise on what the best action for this person to take is | 16:12 |
| JayF | jroll: if I were trying to solve this problem, I'd write a hardware specific hardware manager, and add a clean step specifically to wipe these "invisible" disks | 16:12 |
| JayF | which is what you originally suggeted | 16:12 |
| jroll | JayF: yeah, that's one option I mentioned | 16:12 |
| JayF | *suggested | 16:12 |
| jroll | right | 16:12 |
| *** chas_ has joined #openstack-ironic | 16:19 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node console notifications https://review.openstack.org/397812 | 16:22 |
| *** Goneri has joined #openstack-ironic | 16:23 | |
| *** chas_ has quit IRC | 16:23 | |
| aparnav | JayF, jroll, The idea is to call a method similar to 'erase_block_device' say 'erase_controller_disks' from the erase_devices method. The actual implementation of erasing the disks using the thirdparty hardware manager utility doesn't comes under the GenericHardwareManager. | 16:28 |
| aparnav | If the thirdparty hardware manager doesn't have the method then fall back to current erase_device steps | 16:29 |
| JayF | aparnav: IMO that makes the existing method much more complicated when there's no reason; we already have a plugin system for adding extra cleaning steps, what's the downside in using that? | 16:30 |
| *** Nisha has joined #openstack-ironic | 16:30 | |
| *** mgoddard_ has joined #openstack-ironic | 16:30 | |
| *** Nisha is now known as Guest39619 | 16:30 | |
| openstackgerrit | Stenio Araujo proposed openstack/ironic: Adds another validation step when using dynamic allocation https://review.openstack.org/377073 | 16:31 |
| *** baoli_ has quit IRC | 16:31 | |
| openstackgerrit | Félix Bouliane proposed openstack/ironic: Add virtualpdu to ironic devstack plugin https://review.openstack.org/388154 | 16:31 |
| openstackgerrit | Félix Bouliane proposed openstack/ironic: DONOTMERGE-change IRONIC_DEPLOY_DRIVER to pxe_snmp https://review.openstack.org/388168 | 16:31 |
| dtantsur | hmm, wut? http://logs.openstack.org/41/396741/1/check/gate-tempest-dsvm-ironic-multitenant-network-ubuntu-xenial/c771f22/console.html#_2016-11-15_15_17_09_261371 | 16:32 |
| dtantsur | https://review.openstack.org/396741 fails with this | 16:32 |
| patchbot | patch 396741 - ironic (stable/newton) - Add PS4 for better logfile information of devstack... | 16:32 |
| dtantsur | and ditto for inspector job Oo http://logs.openstack.org/41/396741/1/check/gate-tempest-dsvm-ironic-inspector-nv/bf64714/console.html#_2016-11-15_15_22_07_591304 | 16:32 |
| dtantsur | have we broken stable branches? | 16:33 |
| *** e0ne has quit IRC | 16:33 | |
| dtantsur | SKIPPED: Port state change feature isn't supported by Ironic | 16:33 |
| dtantsur | sambetts, vsaienk0 maybe you have ideas ^^^? | 16:33 |
| *** mgoddard has quit IRC | 16:33 | |
| *** e0ne has joined #openstack-ironic | 16:34 | |
| dtantsur | ugh, our tests are skipped completely on stable/newton, only API tests are run :( | 16:34 |
| *** rama_y has quit IRC | 16:35 | |
| sambetts | dtantsur: just git grepped tempest and ironic and have no idea where that message is coming from... | 16:36 |
| *** godp1301_ has quit IRC | 16:36 | |
| aparnav | JayF, It can be added as extra clean step. Then it will be calling two clean steps instead of one. Also we have to fall back to 'erase_device', I thought it will be better to add it in erase_devices instead of adding a new clean step. | 16:36 |
| sambetts | dtantsur: literally can't find the words "Port state" anywhere... | 16:37 |
| *** godp1301 has joined #openstack-ironic | 16:38 | |
| dtantsur | Oo | 16:38 |
| jroll | dtantsur: that sounds like my tempest patch | 16:38 |
| dtantsur | right, it may be it | 16:38 |
| jroll | wtf | 16:38 |
| sambetts | wtf has the gate smooshed those patches together somehow?! | 16:39 |
| JayF | aparnav: No, IMO this is exactly why we made the clean steps pluggable, for a case like that. | 16:39 |
| JayF | aparnav: and I have no problem with having it be an openstack project for a 3rd party hw manager, my only objection to 100% upstreaming this is that we generally reject things that require thirdparty/proprietary utilities in IPA | 16:40 |
| *** tuan_luong has quit IRC | 16:40 | |
| jroll | sambetts: it's in there | 16:40 |
| jroll | OH | 16:40 |
| jroll | this is dumb, lol | 16:41 |
| jroll | so | 16:41 |
| jlvillal | I'll chime in with yeah the stable/newton branch appears broken: http://logs.openstack.org/03/397803/1/check/gate-tempest-dsvm-ironic-multitenant-network-ubuntu-xenial/89b22e4/ | 16:41 |
| jlvillal | That is from a patch with no changes. | 16:41 |
| jroll | the tempest config thing we brought over | 16:41 |
| jroll | checks port_admin_state_change | 16:41 |
| jroll | skips if set | 16:41 |
| * jlvillal goes to update whiteboard, unless already done | 16:41 | |
| jroll | which may be silly, dunno | 16:41 |
| jroll | but | 16:41 |
| jroll | we use master tempest plugin | 16:41 |
| jroll | and stable devstack | 16:41 |
| jroll | this is fine on master because devstack plugin sets it accordingly. | 16:41 |
| dtantsur | aha, wonderful world of branchless things >_< | 16:41 |
| jroll | indeed | 16:42 |
| jroll | so uh, backport-wise | 16:42 |
| jroll | I can backport the whole patch, even though tempest plugin changes won't be used | 16:42 |
| dtantsur | ++ | 16:42 |
| jroll | or I can only backport the devstack changes | 16:42 |
| jroll | dtantsur: so, whole patch you think? | 16:43 |
| dtantsur | whole patch. imagine some people will try using stable tempest plugin | 16:44 |
| jroll | heh | 16:45 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Bring configurations from tempest to ironic_tempest_plugin https://review.openstack.org/397824 | 16:45 |
| jroll | bah, merge conflict for mitaka | 16:46 |
| dtantsur | woohoo | 16:46 |
| dtantsur | and should we bother with liberty? | 16:46 |
| jlvillal | jroll: dtantsur: Whiteboard updated | 16:46 |
| aparnav | JayF, I understand. The thirdparty utility related changes goes under Thirdparty hardware manager and not under IPA. But 'erase_devices' in IPA needs changes for the call 'erase_controller_disks' to the hardware managers. | 16:46 |
| jroll | dtantsur: probably not, dunno | 16:48 |
| jlvillal | dtantsur: What do you think of this __slots__ patch. I'm not understanding the value of it yet: https://review.openstack.org/#/c/397103/ | 16:48 |
| patchbot | patch 397103 - ironic - Use __slots__ for save memory of drivers | 16:48 |
| JayF | aparnav: I disagree with us overloading that one step with also erasing disk controller disks. There's no need for it, you literally have the exact use case for a custom additional clean step. Unless there's some requirement you haven't laid out, I don't think I'm going to be onboard for approving that as an RFE. | 16:48 |
| jlvillal | dtantsur: I think the inspiration was a comment you made in a spec. | 16:49 |
| dtantsur | jlvillal, just like you, I'd prefer to see at least simple memory comparison | 16:49 |
| jlvillal | dtantsur: Good :) Thanks. | 16:49 |
| *** chas_ has joined #openstack-ironic | 16:49 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Bring configurations from tempest to ironic_tempest_plugin https://review.openstack.org/397828 | 16:49 |
| dtantsur | should not be hard to do, like enroll 10000 fake nodes and check :) | 16:49 |
| jroll | dtantsur: ok, there's mitaka | 16:49 |
| dtantsur | cool, will wait for CI | 16:50 |
| jroll | dtantsur: let's be sure not to merge anything in stable for now I guess :D | 16:50 |
| aparnav | JayF, okay I understand. Thank you | 16:50 |
| JayF | aparnav: I'll be glad to review whatever code you get for a third party hardware manager, and help you make sure it works :) | 16:50 |
| dtantsur | jroll, you're missing cherry-pick lines in both cases | 16:50 |
| dtantsur | * only mitaka | 16:50 |
| aparnav | JayF, thank you :) | 16:51 |
| jroll | dtantsur: oops, thanks | 16:51 |
| JayF | aparnav: are you OK with me marking that RFE as invalid/rejected? I'm not a dictator so you can talk to other folks and try to get consensus around your idea instead, but if you're going off to try what I suggested I wanna cleanup the bug :D | 16:52 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Bring configurations from tempest to ironic_tempest_plugin https://review.openstack.org/397828 | 16:53 |
| *** ipukha has left #openstack-ironic | 16:53 | |
| jroll | dtantsur: fixed | 16:53 |
| dtantsur | k | 16:53 |
| *** chas_ has quit IRC | 16:54 | |
| *** fragatina has joined #openstack-ironic | 16:54 | |
| *** fragatina has quit IRC | 16:54 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 16:57 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic: Missing or not needed return values in OneView deploy interface https://review.openstack.org/396698 | 16:58 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic: Reusing oneview_client when possible https://review.openstack.org/358041 | 16:58 |
| nicodemos | vdrok, lucasagomes. Done ^. Thanks o/ | 17:00 |
| lucasagomes | nicodemos, will take a look later, thanks! | 17:00 |
| *** g3ek has quit IRC | 17:02 | |
| *** chuckC_ has joined #openstack-ironic | 17:02 | |
| *** mjura has quit IRC | 17:03 | |
| *** chuckC_ has quit IRC | 17:04 | |
| *** causten has quit IRC | 17:04 | |
| *** moshele has quit IRC | 17:06 | |
| *** causten has joined #openstack-ironic | 17:08 | |
| *** jcoufal_ has quit IRC | 17:10 | |
| *** ccamacho is now known as ccamacho|afk | 17:10 | |
| *** Guest39619 is now known as Nisha_ | 17:14 | |
| *** jcoufal has joined #openstack-ironic | 17:14 | |
| Nisha_ | JayF, hi | 17:15 |
| Nisha_ | JayF, aparnav discussed with me on the RFE just now. | 17:16 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 17:16 |
| jroll | rloo: no word from devananda on api meeting, I guess we turn it off | 17:16 |
| Nisha_ | JayF, am sorry but i am still missing why it needs a new clean step and why it cant be in current erase_devices step | 17:17 |
| JayF | Nisha_: there are two separate questions here, and I'll answer them separately: | 17:17 |
| Nisha_ | JayF, sure | 17:18 |
| JayF | Nisha_: 1) Should we have upstream support for detecting and erasing these disks? My answer is "no", simply because as an implementation detail, we'd have to include a nonfree HP utility, and that's something we generally have rejected in upstream IPA. So then the question becomes... | 17:18 |
| Nisha_ | aparnav, ^^^ | 17:19 |
| *** chas_ has joined #openstack-ironic | 17:19 | |
| Nisha_ | aparnav, are u calling hpssacli in IPA? | 17:19 |
| JayF | Nisha_: 2) What's the best way to do this out of tree? aparnav suggested adding a hook to erase_devices, I rejected that because this is the exact use case we created custom cleaning steps for. A small hardware manager can add a single clean step to erase the disks in a controller, and deploy it as a separate hardware manger. There is no value whatsoever in modifying the existing erase_devices step | 17:19 |
| JayF | workflow. | 17:19 |
| rloo | jroll: yeah, let's turn it off. We can always add it back in when/if we want. | 17:19 |
| Nisha_ | aparnav, as i understand it must be in proliant hardware manager | 17:20 |
| rloo | jroll: you going to send out email about it? and I'll let you do the honor: https://review.openstack.org/#/c/395517/ | 17:20 |
| patchbot | patch 395517 - openstack-infra/irc-meetings - Remove unused Ironic v2 API meeting | 17:20 |
| aparnav | Nisha_, hpssacli is not getting called from IPA | 17:20 |
| jroll | rloo: just +1'd the thing, and bah was hoping you'd send mail :) | 17:20 |
| jroll | rloo: I can send email and update wiki after lunch though | 17:21 |
| Nisha_ | JayF, we already have hardware manager and IPA just invokes calling that | 17:21 |
| rloo | jroll: ha ha. i can send it | 17:21 |
| aparnav | Nisha_, The changes related to hpssacli goes to proliant hardware manager | 17:21 |
| *** e0ne has quit IRC | 17:21 | |
| Nisha_ | JayF, ^^^ | 17:21 |
| jroll | rloo: you rule, but I really don't mind :) | 17:21 |
| * jroll lunches now | 17:21 | |
| Nisha_ | JayF, does this helps to answer that the third party tool is not getting called in IPA upstream | 17:21 |
| JayF | Nisha_: the third party tool is needed /to see the disks at all/ | 17:22 |
| JayF | Nisha_: which means upstream IPA can't even add detection support for it | 17:22 |
| Nisha_ | JayF, yes but i didnt get ur second part | 17:22 |
| JayF | Nisha_: and I am not OK with a blind hook into IPA hardware manager to call a method to erase controller-attached disks that we don't even implement upstream. There's no point in it. | 17:22 |
| Nisha_ | JayF, i didnt get this | 17:23 |
| Nisha_ | "which means upstream IPA can't even add detection support for it" | 17:23 |
| JayF | Nisha_: tl;dr: Just add a clean step in proliant manager, and implement it there | 17:23 |
| JayF | Nisha_: that's ^ the right thing to do in this case | 17:23 |
| JayF | Nisha_: what I'm saying is the flow today is list_block_devices -> erase devices (for device in devices -> erase_block_device) | 17:23 |
| Nisha_ | JayF, there is a reason why we say it shud be done in current erase_devices step | 17:23 |
| JayF | Nisha_: why? | 17:23 |
| JayF | Nisha_: so without IPA being able to detect the blcok device to dispatch the erasure to a third party manager, it doens't fit into our current design | 17:24 |
| Nisha_ | JayF, 1. currently IPA does the hardware disk erase as its first choice and then fall back to software disk erase | 17:24 |
| Nisha_ | 2. the above step is insufficient for HP hardwares completely | 17:24 |
| *** chas_ has quit IRC | 17:25 | |
| JayF | that seems completely irrelevent to the conversation at hand? | 17:25 |
| *** ChubYann has joined #openstack-ironic | 17:25 | |
| Nisha_ | 3. we also want to do the hardware disk erase as the first step and if it doesnt work then fall back to software disk erase | 17:25 |
| Nisha_ | aparnav, correct me if i go wrong anywhere | 17:25 |
| JayF | Nisha_: so let me tell you the problem statement as it was given to me | 17:25 |
| JayF | Nisha_: there are disks attached to a storage controller. Today IPA doesn't even detect those disks exist, much less try to erase them. You would like to add support for erasing these disks. Correct? | 17:26 |
| Nisha_ | JayF, IPA hardware disk erase doesnt support those disks | 17:26 |
| JayF | Nisha_: that's not the questino: does IPA even *Attempt* to erase these disks? i.e. do we detect them at all? | 17:27 |
| Nisha_ | aparnav, could u answer above | 17:27 |
| aparnav | JayF, Nisha_, Yes, IPA doesnt detect those disk | 17:28 |
| Nisha_ | JayF, but IPA software disk erase workson those disks | 17:28 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Move `deploy_forces_oob_reboot` to deploy drivers https://review.openstack.org/397843 | 17:28 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from most drivers https://review.openstack.org/397844 | 17:28 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from iLO drvers https://review.openstack.org/397845 | 17:28 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from OneView drivers https://review.openstack.org/397846 | 17:28 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove iBoot, WoL and AMT drivers https://review.openstack.org/397847 | 17:28 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru completely https://review.openstack.org/397848 | 17:28 |
| JayF | Nisha_: aparnav: Those are conflicting statements. | 17:28 |
| JayF | Nisha_: aparnav: one of them is wrong, which one? lol | 17:28 |
| Nisha_ | JayF, aparnav is saying even software disk erase doesnt work | 17:29 |
| Nisha_ | JayF, ok one ques | 17:30 |
| JayF | Nisha_: so that means IPA doesn't even detect those disks, and AIUI, can't without the third party tool | 17:30 |
| Nisha_ | JayF, if IPA is able to erase the partitions of the OS | 17:30 |
| aparnav | JayF, Nisha_, software disk erase erases only the disks visible to the OS. When we say storage controller, we cannot directly trigger erase commands to those disks | 17:30 |
| JayF | Nisha_: that means it's completely out of scope of IPA to be able to erase those disks today. It can't erase what it can't detect. | 17:30 |
| Nisha_ | doesnt it mean that it detects the disk | 17:30 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru completely https://review.openstack.org/397848 | 17:30 |
| JayF | Nisha_: what you're saying and what aparnav is saying can't be correct simultaneously :) | 17:31 |
| Nisha_ | JayF, see last statement which she said just now | 17:31 |
| JayF | Nisha_: but my point is; if IPA can't see those block devices at all, having a separate step+detection mechanism is the sanest way to get those erased -- and that shoudl be done entirely in the proliant manager. | 17:32 |
| dtantsur | jroll, +A on both patches | 17:32 |
| JayF | Nisha_: if IPA can detect the disks, then simply implementing erase_block_device, and detecting when a disk is attached to the storage controller, then acting to erase it, is the right path | 17:32 |
| Nisha_ | JayF, my ques here is can IPA see all disks whether RAID is configured or not on other hardwares | 17:32 |
| JayF | Nisha_: neither path requires IPA upstream to have any changes. | 17:32 |
| jroll | dtantsur: awesome, I assume they ran tests? :) | 17:32 |
| * dtantsur double-checks again | 17:33 | |
| JayF | Nisha_: IPA calls `lsblk` and filters out for physical disks. This means that disks that are members of software raid appear separately, and hardware raids appear to IPA as a single disk | 17:33 |
| Nisha_ | JayF, and if they dont have raid configured | 17:33 |
| Nisha_ | are those disks visible to IPA | 17:33 |
| Nisha_ | ? | 17:33 |
| dtantsur | test_baremetal_server_ops [307.338578s] ... ok | 17:34 |
| pas-ha | Nisha_: are you a maintainer for iLO drivers? | 17:34 |
| dtantsur | test_baremetal_multitenancy [737.217236s] ... ok | 17:34 |
| Nisha_ | pas-ha, :D lol | 17:34 |
| Nisha_ | pas-ha, no i am just the contributor | 17:34 |
| JayF | Nisha_: It would depend on the raid controller + the hardware. You can determine this yourself, IPA generates the list via: https://gist.githubusercontent.com/jayofdoom/e4144b1bac119a56c97847110b3934ba/raw/ff48f13b5c38ed86342a1e6cee47cfd9664a4e36/gistfile1.txt -- just run that on your hardware and see what shows up | 17:34 |
| JayF | oh bad lnike | 17:35 |
| jroll | dtantsur: <3 | 17:35 |
| JayF | Nisha_: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L115 was what I was trying to link | 17:35 |
| * jroll actually lunches now | 17:35 | |
| pas-ha | I am just looking for folks "responsible"/knowledgeable in iLO and OneView drivers | 17:36 |
| JayF | pas-ha: of the cores, I've probably reviews as much oneview code as anyone, but I haven't ever used teh driver | 17:36 |
| Nisha_ | JayF, i understand what you are trying to say but still am not convinced why it should be a seperate clean step. iLO drivers already call IPA erase_devices for software disk erase | 17:36 |
| JayF | pas-ha: maybe just ask your question generally? | 17:36 |
| sambetts | vsaienk0, jlvillal: have you got an example of a working subnode local.conf, the example of how to do it on the devstack docs is terrible, and trying to pick apart devstack-gate isn't working for me | 17:36 |
| pas-ha | there's someone called "nisha" pointed on https://wiki.openstack.org/wiki/Ironic/Drivers, Nisha_ would that be you? | 17:37 |
| jlvillal | sambetts: I told you about my script :) | 17:37 |
| dtantsur | see you tomorrow! | 17:37 |
| jlvillal | sambetts: https://github.com/JohnVillalovos/devstack-gate-test | 17:37 |
| *** dtantsur is now known as dtantsur|afk | 17:37 | |
| Nisha_ | so we shud be able to call hardware disk erase also same way | 17:37 |
| Nisha_ | pas-ha, yes | 17:37 |
| JayF | Nisha_: if you can implement it in your hardware manager using erase_block_device, sure, do that. I am *not* on board with aparnav's request that we add a method to erase_devices specifically to call fro erasing devices on a disk controller | 17:37 |
| jlvillal | sambetts: https://github.com/JohnVillalovos/devstack-gate-test/blob/master/multi-node-setup.sh | 17:37 |
| jlvillal | sambetts: https://github.com/JohnVillalovos/devstack-gate-test/blob/master/playbook-multi-node.yml | 17:38 |
| pas-ha | JayF: I've just posted patches removing old agent passthru from all drivers. OneView and iLO were non-trivial, so I'd like to get responsible people to the reviewers | 17:38 |
| nicodemos | pas-ha: I and xavierr can answer questions about the oneview | 17:38 |
| Nisha_ | JayF, ok let me discuss on implementation with her | 17:38 |
| jlvillal | sambetts: It won't work out of the box for you, but should point you in the correct direction | 17:38 |
| Nisha_ | But please dont mark the RFE as invalid | 17:38 |
| *** fragatina has joined #openstack-ironic | 17:38 | |
| *** rama_y has joined #openstack-ironic | 17:38 | |
| JayF | Nisha_: Well, right now I'm very explicitly not approving of it, and even with our chat here I'm fairly convinced that all the good solutions do not involve changes for IPA upstream | 17:38 |
| *** fragatina has quit IRC | 17:38 | |
| pas-ha | Nisha_: then sorru for stupid question - what is your Gerrit email/full name? :) so I can add you | 17:39 |
| * dtantsur|afk records his overall agreement with JayF as this point | 17:39 | |
| JayF | Nisha_: so I won't mark the RFE as invalid, but I'll link this chat in there and log my disagreement which will likely prevent it from being approved unless you all are better able to articulate what you're trying to do | 17:39 |
| *** fragatina has joined #openstack-ironic | 17:39 | |
| Nisha_ | JayF, ok. but still wait for a day | 17:39 |
| Nisha_ | its night here | 17:39 |
| Nisha_ | i will discuss with her tomorrow and update on the RFE | 17:39 |
| pas-ha | nicodemos: then please review and share with your team/add them to reviewers https://review.openstack.org/#/c/397846 | 17:39 |
| patchbot | patch 397846 - ironic - Remove agent vendor passthru from OneView drivers | 17:39 |
| Nisha_ | you can mark it as invalid tomorrow if really needs to be done | 17:40 |
| nicodemos | pas-ha: ack | 17:40 |
| sambetts | jlvillal: thanks, yeah I was hoping for a non-script based full local.conf that I can see, there are a bunch of things to enable and disable I think, but I'm not doing a very good job of identifying which from all the scripts that are around | 17:40 |
| *** fragatin_ has joined #openstack-ironic | 17:40 | |
| Nisha_ | pas-ha, my Gerrit ID is Nisha | 17:40 |
| Nisha_ | pas-ha, full name is Nisha Agarwal | 17:41 |
| Nisha_ | is that sufficient | 17:41 |
| pas-ha | Nisha_: done, added you (and Shivanand too). please take a look https://review.openstack.org/#/c/397845 | 17:41 |
| patchbot | patch 397845 - ironic - Remove agent vendor passthru from iLO drvers | 17:41 |
| *** fragatin_ has quit IRC | 17:42 | |
| *** g3ek has joined #openstack-ironic | 17:44 | |
| *** fragatina has quit IRC | 17:44 | |
| jlvillal | sambetts: Ah. Well make a patch that prints out the contents of /etc/nodepool/ :) | 17:44 |
| *** fragatina has joined #openstack-ironic | 17:44 | |
| *** amotoki has quit IRC | 17:44 | |
| jlvillal | sambetts: And whatever other information you want. | 17:44 |
| sambetts | jlvillal: I've dug up http://logs.openstack.org/73/368173/41/experimental/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-multinode-nv/3594741/logs/subnode-2/localrc.txt.gz which should help | 17:44 |
| jlvillal | sambetts: Also have you looked at our multi-node patch set? | 17:45 |
| jlvillal | sambetts: Right. Also look in the _zuul_ansible/ directory | 17:45 |
| sambetts | jlvillal: basically my big network sim refactor *should* simplify the multinode network setup a little, just trying to build that into my patch to see | 17:46 |
| jlvillal | sambetts: Cool. Of course the multi-node work is depending on a few more patches to get merged. One in devstack-gate | 17:46 |
| jlvillal | Maybe one in devstack, but I forget if it got merged or not. | 17:47 |
| sambetts | jlvillal: yeah, I've just been looking at that too | 17:47 |
| *** derekh has quit IRC | 17:48 | |
| lucasagomes | folks, I'm calling it a day | 17:48 |
| lucasagomes | have a great evening all! | 17:48 |
| jlvillal | lucasagomes: Good night | 17:48 |
| sambetts | o/ lucasagomes | 17:48 |
| lucasagomes | o/ | 17:48 |
| *** lucasagomes is now known as lucas-afk | 17:48 | |
| *** mgoddard_ has quit IRC | 17:48 | |
| *** mgoddard has joined #openstack-ironic | 17:49 | |
| *** godp1301 has quit IRC | 17:49 | |
| mgould | lucas-afk: o/ | 17:49 |
| *** baoli has joined #openstack-ironic | 17:49 | |
| *** jlvillal is now known as jlvill-travel | 17:50 | |
| *** jpich has quit IRC | 17:51 | |
| *** aparnav has quit IRC | 17:51 | |
| *** charlledaniel has joined #openstack-ironic | 17:52 | |
| *** baoli_ has joined #openstack-ironic | 17:52 | |
| pas-ha | lucas-afk: o/ | 17:52 |
| *** vsaienko has joined #openstack-ironic | 17:52 | |
| kvcobb | Hello Ironicals, could someone point me to some good docs for image creation for my bare metal server? I'm using this guide: http://docs.openstack.org/project-install-guide/baremetal/draft/configure-integration.html#configure-the-image-service | 17:53 |
| Nisha_ | jroll, JayF , dtantsur|afk rloo could you look at https://bugs.launchpad.net/ironic/+bug/1526382 and let me knnow if spec is really required for this RFE | 17:54 |
| openstack | Launchpad bug 1526382 in Ironic "[RFE] Enhance ilo drivers to do inband inspection along with OOB inspection" [Wishlist,In progress] - Assigned to Aparna (aparnavtce) | 17:54 |
| kvcobb | Unfortunately after adding images as both partition and whole disk, I'm still unable to boot my server. | 17:54 |
| *** baoli has quit IRC | 17:55 | |
| sambetts | kvcobb: what failure are you seeing? | 17:55 |
| kvcobb | with whole disk I don't see any errors in the conductor logs, but when it comes back up after disk clean the machine tries to PXE boot again and fails with: | 17:56 |
| kvcobb | Boot from SAN device 0x80 failed: | 17:56 |
| kvcobb | Exec format error | 17:56 |
| kvcobb | PXE boot failed! No configuration for any of the present NICs. | 17:57 |
| sambetts | kvcobb: with whole disk image they should boot from the local disk after a sucessful deployment, it seems like your node is failing to do that for some reason | 17:58 |
| rloo | Nisha_: jroll had already tagged that as needing a spec. why would you remove that tag and ask us if it needs a spec? | 17:58 |
| JayF | Nisha_: if jroll added needs-spec, he's saying it needs a spec | 17:58 |
| kvcobb | hmmm... let me see whether it boots after forcing disk boot through BIOS | 17:58 |
| kvcobb | that sambetts | 17:58 |
| Nisha_ | rloo, JayF jroll did it because it was added as it already has a spec | 17:58 |
| JayF | I mean, let me put it this way: | 17:59 |
| JayF | spec vs rfe is really a question of where the design is going | 17:59 |
| JayF | since you already have a spec, you should just finish it and get it approved | 17:59 |
| JayF | not having a spec doesn't lower the bar you need for design work, it just means we think a consensus is easy to reach in the bug | 17:59 |
| Nisha_ | but when i discussed this with jroll last week he told me to update the RFE and it will be renalalzed for it | 17:59 |
| JayF | wiping out the gerrit review with the spec is useless, and actually sets you back a bit imo | 17:59 |
| openstackgerrit | Merged openstack/ironic: Timing metrics: OneView drivers https://review.openstack.org/379619 | 18:00 |
| Nisha_ | JayF, the spec is there from a year or more | 18:00 |
| JayF | Then clearly it's not one that's easy to get consensus on :) | 18:00 |
| Nisha_ | and secondly and main point is | 18:00 |
| rloo | Nisha_: we aren't aware of the history/discussion you had with jroll. if he suggested something, then you should have mentioned it in the rfe/bug. | 18:00 |
| Nisha_ | that it implements the "Boot management for ironic/inspector" spec from Dmitry | 18:00 |
| Nisha_ | rloo, i would do that | 18:01 |
| JayF | Nisha_: linking IRC chat logs, like I did in the RFE we were talking about earlier, is the best way to reference an older irc conversation | 18:01 |
| Nisha_ | JayF, rloo ^^^ as it implements the spec which is in review and on which it depends on also | 18:01 |
| *** athomas has quit IRC | 18:01 | |
| openstackgerrit | Merged openstack/ironic: Bring configurations from tempest to ironic_tempest_plugin https://review.openstack.org/397828 | 18:01 |
| Nisha_ | i think it doesnt require the spec | 18:01 |
| Nisha_ | JayF, i will just find out and paste in bug | 18:01 |
| rloo | Nisha_: i don't want to get into a discussion about your spec right now i'm in the middle of something. but please help us in the future by providing more information. there is ALOT of work to be done in ironic | 18:02 |
| JayF | ++ | 18:02 |
| JayF | and if the conversation isn't persisted in the bug | 18:02 |
| JayF | it might as well have not happened to anyone who comes up afterwards who wasn't in that chat | 18:02 |
| *** ralonsoh_ has quit IRC | 18:02 | |
| *** mgould is now known as mgould|afk | 18:03 | |
| *** ijw has joined #openstack-ironic | 18:04 | |
| *** ohamada has quit IRC | 18:07 | |
| *** ijw has quit IRC | 18:08 | |
| *** ijw has joined #openstack-ironic | 18:09 | |
| *** ijw has quit IRC | 18:09 | |
| *** ijw has joined #openstack-ironic | 18:09 | |
| *** e0ne has joined #openstack-ironic | 18:11 | |
| kvcobb | sambetts: question, how does the conductor no which drive to target and configure the node for boot of that drive? My server has 2 2TB SSDs in it and 2 mechanical 2TB HDs. | 18:11 |
| kvcobb | know* | 18:11 |
| *** ijw_ has joined #openstack-ironic | 18:12 | |
| *** ijw has quit IRC | 18:12 | |
| *** vsaienko has quit IRC | 18:12 | |
| JayF | kvcobb: search docs for "root device hints" | 18:12 |
| JayF | kvcobb: :) | 18:12 |
| sambetts | kvcobb: http://docs.openstack.org/project-install-guide/baremetal/draft/advanced.html#specifying-the-disk-for-deployment-root-device-hints | 18:12 |
| mariojv | JayF: jcook: why is there no UNRESCUEWAIT state here: https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/implement-rescue-mode.html#state-machine-impact ? | 18:12 |
| JayF | mariojv: because you never have to boot a ramdisk to unrescue. | 18:13 |
| JayF | mariojv: so there's never waiting | 18:13 |
| mariojv | ah, right | 18:13 |
| mariojv | thanks | 18:13 |
| JayF | sambetts: pfft, showing up my customer service by providing actual links :P | 18:13 |
| Nisha_ | JayF, rloo please see the bug when u find time | 18:13 |
| kvcobb | sambetts: Will do. Figured I had to be missing something import | 18:13 |
| sambetts | JayF: hahaha ;) | 18:14 |
| Nisha_ | rloo, i know ironic core is pretty busy but if RFEs are there isnt it approved/rejected by ironic core? | 18:14 |
| Nisha_ | rloo, means whom shud i ask to review the RFE if i am not getting eyes on the RFE | 18:14 |
| JayF | Nisha_: yes, ironic cores approve/reject rfes, but we also prioritize our work based on upstream priorities; that feature isn't a priority so it falls into "if we have time" :) | 18:14 |
| sambetts | kvcobb: np | 18:14 |
| rloo | Nisha_: it can be, but it is the community's responsibility to help out and comment on things. | 18:14 |
| Nisha_ | JayF, vendor things are never and will never be priority for ironic | 18:15 |
| rloo | Nisha_: you are more than welcome to help reduce my workload by reviewing, reviewing, reviewing, so i have more time to do things like look at proposed rfe's. which are low on my list of things to do. | 18:16 |
| *** rbrud has quit IRC | 18:17 | |
| kvcobb | JayF: lol :) just noticed root hints advice came from you also - so thanks to you both. sambetts did sorta one up you there :) | 18:18 |
| JayF | kvcobb: I tend to give links to docs via "here's a list of google keywords to use" lol | 18:19 |
| sambetts | JayF: the funny thing is that searching for ironic root device, results in tripleo docs first, which link to the old ironic docs, which then link to the new ironic docs | 18:20 |
| kvcobb | JayF: I'll take free help any way I can get it | 18:20 |
| JayF | sambetts: at least we kept the links alive, go us | 18:20 |
| JayF | sambetts: someone should go fix ooo docs though | 18:20 |
| sambetts | yeah | 18:20 |
| *** dprince has quit IRC | 18:21 | |
| *** dprince has joined #openstack-ironic | 18:21 | |
| *** e0ne has quit IRC | 18:23 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Move `deploy_forces_oob_reboot` to deploy drivers https://review.openstack.org/397843 | 18:25 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from OneView drivers https://review.openstack.org/397846 | 18:25 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove iBoot, WoL and AMT drivers https://review.openstack.org/397847 | 18:25 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from most drivers https://review.openstack.org/397844 | 18:25 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from iLO drvers https://review.openstack.org/397845 | 18:25 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru completely https://review.openstack.org/397848 | 18:25 |
| *** fragatina has quit IRC | 18:28 | |
| *** trown is now known as trown|lunch | 18:28 | |
| *** fragatina has joined #openstack-ironic | 18:29 | |
| *** flaviodsr has quit IRC | 18:34 | |
| mariojv | jcook: added some comments on https://review.openstack.org/#/c/350831 | 18:34 |
| patchbot | patch 350831 - ironic - Add API and conductor methods for [un]rescue | 18:34 |
| *** godp1301 has joined #openstack-ironic | 18:35 | |
| *** e0ne has joined #openstack-ironic | 18:37 | |
| *** vsaienko has joined #openstack-ironic | 18:38 | |
| *** harlowja has quit IRC | 18:38 | |
| *** wajdi has quit IRC | 18:44 | |
| *** fragatina has quit IRC | 18:44 | |
| *** fragatina has joined #openstack-ironic | 18:45 | |
| *** vsaienko has quit IRC | 18:45 | |
| *** vsaienko has joined #openstack-ironic | 18:45 | |
| *** jheroux has quit IRC | 18:49 | |
| *** Sukhdev has joined #openstack-ironic | 18:51 | |
| *** wajdi has joined #openstack-ironic | 18:52 | |
| *** milan has quit IRC | 18:52 | |
| *** vsaienko has quit IRC | 18:58 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs: Add bonding configuration fields to portgroups https://review.openstack.org/396610 | 18:59 |
| *** rcernin has joined #openstack-ironic | 19:02 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs: Add bonding configuration fields to portgroups https://review.openstack.org/396610 | 19:04 |
| *** harlowja has joined #openstack-ironic | 19:06 | |
| *** priteau has quit IRC | 19:07 | |
| openstackgerrit | Merged openstack/ironic: Bring configurations from tempest to ironic_tempest_plugin https://review.openstack.org/397824 | 19:08 |
| *** jheroux has joined #openstack-ironic | 19:14 | |
| jcook | mariojv righteous, I'll take a look later. I haven't touched the other two patches yet | 19:17 |
| *** dprince has quit IRC | 19:23 | |
| JayF | anyone wanna take a look at https://review.openstack.org/#/c/397792/3 before I land it? | 19:24 |
| patchbot | patch 397792 - ironic-specs - Update CRUD and node states notification spec | 19:24 |
| JayF | mariojv: ^ your review is probably a good thing to have there | 19:24 |
| mariojv | JayF: i already reviewed PS1, will re review | 19:25 |
| mariojv | JayF: +1 feel free to land | 19:26 |
| *** krtaylor has quit IRC | 19:27 | |
| JayF | So, on our weekly priorities is reviews for notifications -- https://review.openstack.org/#/c/348437/23 is the first patch in the series, has my +2 and mariojv's +1, so if someone wants to land something impactful today that could be a good choice | 19:28 |
| patchbot | patch 348437 - ironic - Add node provision state change notification | 19:28 |
| JayF | the rest of them obviously could use reviews too, but have comments as well that are actionable | 19:28 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic: Onetime boot when set_boot_device isn't persistent https://review.openstack.org/340596 | 19:28 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/ironic-lib: Add support for BIOS local boot for GPT label https://review.openstack.org/397897 | 19:28 |
| JayF | cloudnull: ^ that, in mitaka, is only going to affect iscsi deployments (pxe_* drivers) | 19:29 |
| JayF | cloudnull: just making sure you're aware :) | 19:29 |
| mariojv | that provision state notification patch also had a +1 from vdrok with clarification questionson patch set 22, fwiw | 19:29 |
| cloudnull | from the bug it looks like it effects iscsi and agent | 19:30 |
| openstackgerrit | Merged openstack/ironic-specs: Update CRUD and node states notification spec https://review.openstack.org/397792 | 19:30 |
| JayF | cloudnull: I'm saying, in stable/mitaka, I don't think IPA uses ironic-lib to do that yet? | 19:30 |
| JayF | cloudnull: Hmm. Maybe for partition images, I may need to refresh context | 19:30 |
| cloudnull | oh. maybe not. | 19:30 |
| cloudnull | that was one of the issues I was bumping up against. | 19:31 |
| cloudnull | just circling back on things now. | 19:31 |
| *** e0ne has quit IRC | 19:31 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic-specs: Support rolling upgrades https://review.openstack.org/299245 | 19:32 |
| cloudnull | I can pull in a later version of IPA os long as i'm not ahead of 1.5.0 | 19:32 |
| cloudnull | if that'd be a better overall solution . | 19:32 |
| JayF | cloudnull: yeah, it looks like that code isn't used in IPA in mitaka. Also I'm curious how we handle stable for ironic-lib | 19:32 |
| JayF | cloudnull: https://github.com/openstack/ironic-python-agent/blob/stable/mitaka/ironic_python_agent/utils.py I think is the relevant code in mitaka, and it doesn't use ironic-lib :( | 19:33 |
| cloudnull | right now my larger deployment is running mitaka, is there a version of IPA that is recommened under mitaka ? | 19:33 |
| cloudnull | :'( | 19:34 |
| JayF | let m,e look at the release notes | 19:34 |
| JayF | see if there's an intersection of vendor_passthru lookup + using ironic_lib for partial disk images | 19:34 |
| cloudnull | 1.5.0 seems to work, I was using the head of master until just a few days ago | 19:35 |
| JayF | cloudnull: you should be able to use a stable/newton IPA, we didn't merge the code removing teh fallback for old vendor-passthru api until ocata series, unless someone missed their release note :() | 19:35 |
| JayF | cloudnull: and it appears the code to flip over to ironic-lib was in 1.2.0, so using stable/newton might enable you to pull in your ironic-lib fix, lemme look at the requirements.txt there | 19:36 |
| cloudnull | kk thanks JayF | 19:36 |
| JayF | cloudnull: so, if the fix you need was in ironic-lib 2.0.0, that's what stable/newton ipa uses | 19:37 |
| *** trown|lunch is now known as trown | 19:37 | |
| JayF | cloudnull: and AFAIK (I'm hedging just because I haven't tested it), there's nothing in newton ipa that would break against mitaka ironic | 19:37 |
| cloudnull | ok, building that now. | 19:38 |
| JayF | jroll: dtantsur|afk: Can we get a release of IPA stable/newton? It looks like it has unreleased backported fixes. | 19:38 |
| *** ijw_ has quit IRC | 19:38 | |
| *** ijw has joined #openstack-ironic | 19:39 | |
| JayF | jroll: dtantsur|afk: I can put up the patch for it, unless there's some reason we shouldn't release it yet | 19:39 |
| *** ijw has quit IRC | 19:44 | |
| *** amotoki has joined #openstack-ironic | 19:44 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add notification documentation to administrator's guide https://review.openstack.org/389344 | 19:45 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add notification documentation to install guide https://review.openstack.org/395787 | 19:45 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add notification documentation to install guide https://review.openstack.org/395787 | 19:46 |
| cloudnull | testing stable/newton IPA now, | 19:47 |
| *** ijw has joined #openstack-ironic | 19:47 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/bifrost: Updated from global requirements https://review.openstack.org/397903 | 19:48 |
| *** Nisha_ has quit IRC | 19:50 | |
| *** amotoki has quit IRC | 19:50 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/397908 | 19:50 |
| *** causten has quit IRC | 19:55 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru completely https://review.openstack.org/397848 | 19:56 |
| *** ijw has quit IRC | 19:57 | |
| jroll | JayF: I think dmitry was going to release all the things this week, idk if there's a reason not to do it now, so I'll leave it up to him | 19:58 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic: Reusing oneview_client when possible https://review.openstack.org/358041 | 19:58 |
| JayF | jroll: ah, cool, I just noticed | 19:58 |
| jroll | JayF: we cleared the queue this morning in anticipation of doing so | 19:58 |
| JayF | jroll: another question: we haven't done stable things with ironic-lib yet, have we? | 19:58 |
| jroll | JayF: also ironic-lib... yeah that question | 19:58 |
| *** causten has joined #openstack-ironic | 19:58 | |
| jroll | it's the same process | 19:58 |
| JayF | jroll: I'd presume we'd release a point-release of the lib released with that version, and then bump the requirements in the stable branch? | 19:58 |
| jroll | JayF: I'm not sure when reqs are usually bumped (sometimes they aren't?) but yeah | 19:59 |
| jroll | JayF: I think the general assumption is a released thing will get used anyway, only bump if huge major bug or something, would need to ask stable folks | 20:00 |
| *** wajdi_ has joined #openstack-ironic | 20:00 | |
| jroll | I think that's english, let me know if it doesn't make sense | 20:00 |
| JayF | Yeah, I get it | 20:01 |
| jroll | cpp; | 20:01 |
| jroll | cool. | 20:01 |
| *** wajdi has quit IRC | 20:02 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic: Missing or not needed return values in OneView deploy interface https://review.openstack.org/396698 | 20:05 |
| *** krtaylor has joined #openstack-ironic | 20:05 | |
| rloo | jroll: weren't you going to make JayF part of stable or something? (I forgot what) | 20:06 |
| *** moshele has joined #openstack-ironic | 20:06 | |
| jroll | rloo: maybe, there is a process to do that sort of thing | 20:07 |
| *** ijw has joined #openstack-ironic | 20:07 | |
| jroll | I would like to, yes | 20:07 |
| rloo | jroll: ok, i remember you mentioning something about that at the summit. about assigning JayF to everything :) | 20:08 |
| jroll | rloo: oh, that was about making JayF do all the work, not stable specifically :) | 20:08 |
| JayF | I do everything really well | 20:08 |
| JayF | except for something | 20:09 |
| rloo | jroll: yes, that was it! :D | 20:09 |
| JayF | something I don't do well at all | 20:09 |
| *** krtaylor has quit IRC | 20:10 | |
| *** Goneri has quit IRC | 20:11 | |
| JayF | If anyone is interested in helping with design fro specific fault support, mariojv and I are working on it in a video session right now. Happy to invite anyone who wants to join. | 20:11 |
| *** ijw has quit IRC | 20:12 | |
| JayF | mariojv: https://etherpad.openstack.org/p/ironic-specific-fault-support-spec | 20:12 |
| mariojv | thanks | 20:12 |
| *** e0ne has joined #openstack-ironic | 20:13 | |
| *** krtaylor has joined #openstack-ironic | 20:14 | |
| *** krtaylor has quit IRC | 20:14 | |
| *** krtaylor has joined #openstack-ironic | 20:15 | |
| *** krtaylor has quit IRC | 20:17 | |
| *** baoli_ has quit IRC | 20:19 | |
| *** ijw has joined #openstack-ironic | 20:20 | |
| *** baoli has joined #openstack-ironic | 20:23 | |
| *** krtaylor has joined #openstack-ironic | 20:24 | |
| *** causten has quit IRC | 20:24 | |
| *** baoli_ has joined #openstack-ironic | 20:24 | |
| *** causten has joined #openstack-ironic | 20:24 | |
| *** ijw has quit IRC | 20:25 | |
| *** krtaylor has quit IRC | 20:26 | |
| mrda | Morning Ironic | 20:27 |
| *** baoli has quit IRC | 20:28 | |
| *** causten has quit IRC | 20:29 | |
| *** jkilpatr_ has joined #openstack-ironic | 20:29 | |
| jroll | mjturek: around? | 20:30 |
| mjturek | hey jroll | 20:30 |
| jroll | mjturek: curious what you wish the release model to be around molteniron | 20:30 |
| * jroll finds docs | 20:30 | |
| mjturek | hmmm that's a good question | 20:30 |
| jroll | mjturek: see the release:x tags here http://governance.openstack.org/reference/tags/index.html#release-management-tags | 20:30 |
| mjturek | looking | 20:30 |
| jroll | I suspect you don't need stable branches so release:independent is probably correct | 20:31 |
| *** chas_ has joined #openstack-ironic | 20:31 | |
| *** jkilpatr has quit IRC | 20:31 | |
| mjturek | jroll: yeah just read it, seems right | 20:32 |
| jroll | cool, thanks | 20:32 |
| mjturek | thanks! | 20:32 |
| *** rbudden has joined #openstack-ironic | 20:33 | |
| *** ijw has joined #openstack-ironic | 20:33 | |
| *** causten has joined #openstack-ironic | 20:35 | |
| *** krtaylor has joined #openstack-ironic | 20:35 | |
| openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers https://review.openstack.org/361451 | 20:35 |
| Sukhdev | jroll : I added three tests based upon one of your comments - have a look https://review.openstack.org/361451 | 20:35 |
| patchbot | patch 361451 - ironic - Add support for Security Groups for baremetal servers | 20:35 |
| Sukhdev | jroll : will ping you after lunch regarding your second comment on this patch | 20:36 |
| jroll | Sukhdev: added to my queue | 20:37 |
| *** causten has quit IRC | 20:39 | |
| openstackgerrit | Charlle Daniel proposed openstack/ironic: Ensures that OneView nodes are free for use by Ironic https://review.openstack.org/372762 | 20:40 |
| *** vsaienko has joined #openstack-ironic | 20:47 | |
| jroll | mjturek: btw, https://review.openstack.org/#/c/397937/ | 20:48 |
| patchbot | patch 397937 - governance - Add openstack/molteniron to ironic governance | 20:48 |
| openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers https://review.openstack.org/361451 | 20:49 |
| *** amotoki has joined #openstack-ironic | 20:50 | |
| *** amotoki has quit IRC | 20:55 | |
| mjturek | jroll: awesome! will review | 20:57 |
| *** ijw has quit IRC | 20:58 | |
| *** moshele has quit IRC | 20:59 | |
| *** ijw has joined #openstack-ironic | 21:00 | |
| *** ijw has quit IRC | 21:02 | |
| *** krtaylor has quit IRC | 21:02 | |
| *** ccamacho|afk has quit IRC | 21:02 | |
| *** ijw has joined #openstack-ironic | 21:02 | |
| *** MattMan has quit IRC | 21:02 | |
| *** MattMan has joined #openstack-ironic | 21:03 | |
| *** e0ne has quit IRC | 21:04 | |
| *** priteau has joined #openstack-ironic | 21:08 | |
| *** roeyc has joined #openstack-ironic | 21:08 | |
| *** causten has joined #openstack-ironic | 21:12 | |
| *** priteau has quit IRC | 21:12 | |
| *** jkilpatr_ has quit IRC | 21:13 | |
| *** krtaylor has joined #openstack-ironic | 21:14 | |
| *** sambetts is now known as sambetts|afk | 21:18 | |
| *** ccamacho has joined #openstack-ironic | 21:20 | |
| *** bdeschenes has quit IRC | 21:20 | |
| *** jkilpatr_ has joined #openstack-ironic | 21:26 | |
| *** lindycoder has quit IRC | 21:28 | |
| *** krtaylor has quit IRC | 21:29 | |
| *** bdeschenes has joined #openstack-ironic | 21:32 | |
| *** pcaruana has quit IRC | 21:32 | |
| openstackgerrit | Merged openstack/ironic: Add PS4 for better logfile information of devstack runs https://review.openstack.org/396741 | 21:37 |
| *** jkilpatr_ has quit IRC | 21:43 | |
| *** wajdi_ has quit IRC | 21:51 | |
| *** harlowja has quit IRC | 21:55 | |
| *** roeyc has quit IRC | 21:55 | |
| *** vsaienko has quit IRC | 21:55 | |
| *** roeyc has joined #openstack-ironic | 21:56 | |
| *** xavierr is now known as xavierr|afk | 21:58 | |
| *** wajdi has joined #openstack-ironic | 21:59 | |
| *** rbudden has quit IRC | 22:00 | |
| *** trown is now known as trown|outtypewww | 22:02 | |
| *** jkilpatr has joined #openstack-ironic | 22:10 | |
| *** harlowja has joined #openstack-ironic | 22:13 | |
| *** khamtamtun has joined #openstack-ironic | 22:29 | |
| *** jheroux has quit IRC | 22:32 | |
| *** ijw has quit IRC | 22:34 | |
| *** godp1301 has quit IRC | 22:39 | |
| *** rcernin has quit IRC | 22:42 | |
| *** ijw has joined #openstack-ironic | 22:43 | |
| kvcobb | If anyone is still around, thoughts on this error in my conductor log: http://paste.openstack.org/show/589335/ | 22:57 |
| kvcobb | ? | 22:57 |
| kvcobb | So I've been playing with root device hints all afternoon and when ever I specify them, ironic isn't able to find my devices | 22:58 |
| *** khamtamtun has quit IRC | 22:59 | |
| *** vinsh has quit IRC | 23:00 | |
| *** roeyc has quit IRC | 23:07 | |
| *** ccamacho has quit IRC | 23:11 | |
| *** wajdi has quit IRC | 23:11 | |
| *** alexpilotti has quit IRC | 23:14 | |
| jcook | mrda o/ | 23:26 |
| *** bfournie has quit IRC | 23:27 | |
| jcook | mrda I've been driving OKRs for OSIC if you want to take a look and participate: https://etherpad.openstack.org/p/osic-okrs-ocata (Ironic is around line 106). Individual OKRs are at the bottom. | 23:27 |
| jcook | mrda when people say Jira in OSIC, they'll mostly be referring to the stuff under: Objective: Improve enterprise adoption of OpenStack | 23:29 |
| *** vinsh has joined #openstack-ironic | 23:30 | |
| *** vinsh has quit IRC | 23:30 | |
| *** vinsh has joined #openstack-ironic | 23:31 | |
| *** chas_ has quit IRC | 23:33 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Add spec for specific faults support https://review.openstack.org/334113 | 23:34 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Add spec for specific faults support https://review.openstack.org/334113 | 23:36 |
| mrda | thanks jcook | 23:43 |
| jcook | mrda sure | 23:44 |
| Sukhdev | jroll : got a moment now? | 23:48 |
| jroll | Sukhdev: it's almost 7pm so, no, sorry | 23:49 |
| Sukhdev | jroll : are you not in CA? | 23:49 |
| jroll | Sukhdev: feel free to drop questions here or in PM, though, and I'll reply in the morning | 23:49 |
| jroll | or email is fine | 23:49 |
| jroll | I am not | 23:49 |
| *** jvgrant_ has joined #openstack-ironic | 23:49 | |
| Sukhdev | jroll : my bad - for some reason I thought you were somewhere in sacramento - | 23:50 |
| jroll | Sukhdev: I have since moved | 23:50 |
| jroll | it's fine, don't worry about it, I sat down here for other reasons | 23:50 |
| jroll | just not interested in staying here for more than 2 more :)minutes | 23:50 |
| jroll | 2 more minutes :) | 23:50 |
| Sukhdev | jroll : understood - will shoot an email | 23:51 |
| jroll | sounds good, thanks | 23:51 |
| *** jvgrant has quit IRC | 23:52 | |
| *** chas_ has joined #openstack-ironic | 23:58 | |
| *** baoli_ has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!