| *** hamzy has joined #openstack-ironic | 00:00 | |
| *** yolanda has joined #openstack-ironic | 00:01 | |
| *** yuanying_ has joined #openstack-ironic | 00:03 | |
| *** yolanda has quit IRC | 00:05 | |
| *** aweeks has quit IRC | 00:06 | |
| *** yuanying has quit IRC | 00:06 | |
| *** ijw has quit IRC | 00:06 | |
| *** ijw has joined #openstack-ironic | 00:07 | |
| *** yuanying_ has quit IRC | 00:07 | |
| *** baoli has quit IRC | 00:07 | |
| *** yolanda has joined #openstack-ironic | 00:19 | |
| *** yolanda has quit IRC | 00:23 | |
| *** yolanda has joined #openstack-ironic | 00:26 | |
| *** yolanda has quit IRC | 00:29 | |
| *** yolanda has joined #openstack-ironic | 00:30 | |
| openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Documentation for Security Groups for baremetal servers https://review.openstack.org/393962 | 00:34 |
|---|---|---|
| *** devlaps has quit IRC | 00:37 | |
| *** devlaps has joined #openstack-ironic | 00:38 | |
| *** yolanda has quit IRC | 00:38 | |
| *** mtanino has quit IRC | 00:39 | |
| *** soliosg has quit IRC | 00:40 | |
| *** jkilpatr has quit IRC | 00:43 | |
| *** jkilpatr has joined #openstack-ironic | 00:43 | |
| *** yolanda has joined #openstack-ironic | 00:44 | |
| *** rama_y has quit IRC | 00:47 | |
| *** yolanda has quit IRC | 00:49 | |
| *** tuanluong has joined #openstack-ironic | 00:51 | |
| tuanluong | Morning Ironic | 00:51 |
| *** SerenaFeng has joined #openstack-ironic | 00:54 | |
| NobodyCam | morning tuanluong | 00:58 |
| NobodyCam | and good night :p | 00:58 |
| tuanluong | morning NobodyCam | 00:58 |
| NobodyCam | hehehe :) | 00:58 |
| tuanluong | :) | 00:58 |
| tuanluong | I will waiting here until you getup | 00:59 |
| *** yolanda has joined #openstack-ironic | 00:59 | |
| *** ijw has quit IRC | 00:59 | |
| NobodyCam | oh my | 01:01 |
| NobodyCam | did you have a question? | 01:01 |
| *** yuanying has joined #openstack-ironic | 01:04 | |
| *** yolanda has quit IRC | 01:07 | |
| tuanluong | No, Thanks NobodyCam | 01:08 |
| NobodyCam | :) have a good night :) | 01:09 |
| tuanluong | good night:0 | 01:12 |
| *** zhangjl has joined #openstack-ironic | 01:14 | |
| *** vinsh has joined #openstack-ironic | 01:19 | |
| *** zhangjl has quit IRC | 01:24 | |
| *** Syed__ has quit IRC | 01:27 | |
| *** yolanda has joined #openstack-ironic | 01:31 | |
| *** david-lyle has joined #openstack-ironic | 01:33 | |
| *** yolanda has quit IRC | 01:33 | |
| *** yolanda has joined #openstack-ironic | 01:34 | |
| *** yolanda has quit IRC | 01:36 | |
| *** yolanda has joined #openstack-ironic | 01:37 | |
| *** yolanda has quit IRC | 01:41 | |
| *** Goneri has quit IRC | 01:47 | |
| *** ijw has joined #openstack-ironic | 01:51 | |
| *** yolanda has joined #openstack-ironic | 01:53 | |
| *** Sukhdev has quit IRC | 01:54 | |
| *** yolanda has quit IRC | 01:58 | |
| *** ijw has quit IRC | 01:58 | |
| *** aNuposic has joined #openstack-ironic | 02:09 | |
| *** yolanda has joined #openstack-ironic | 02:10 | |
| *** baoli has joined #openstack-ironic | 02:12 | |
| *** baoli has quit IRC | 02:17 | |
| *** zhangjl has joined #openstack-ironic | 02:22 | |
| *** yolanda has quit IRC | 02:33 | |
| *** yolanda has joined #openstack-ironic | 02:34 | |
| *** hogepodge has quit IRC | 02:38 | |
| *** yolanda has quit IRC | 02:42 | |
| *** vsaienko has joined #openstack-ironic | 02:47 | |
| *** yolanda has joined #openstack-ironic | 02:54 | |
| *** aNuposic has quit IRC | 02:56 | |
| *** yuanying has quit IRC | 02:58 | |
| *** vsaienko has quit IRC | 02:58 | |
| *** yolanda has quit IRC | 02:58 | |
| *** zhangjl has quit IRC | 03:02 | |
| *** rloo has quit IRC | 03:05 | |
| *** yolanda has joined #openstack-ironic | 03:05 | |
| *** fragatina has quit IRC | 03:06 | |
| *** yolanda has quit IRC | 03:11 | |
| *** yolanda has joined #openstack-ironic | 03:11 | |
| *** yolanda has quit IRC | 03:16 | |
| *** zhangjl has joined #openstack-ironic | 03:17 | |
| *** links has joined #openstack-ironic | 03:22 | |
| *** yolanda has joined #openstack-ironic | 03:28 | |
| *** yolanda has quit IRC | 03:33 | |
| *** yolanda has joined #openstack-ironic | 03:34 | |
| *** vikrant has joined #openstack-ironic | 03:37 | |
| *** galyna has quit IRC | 03:40 | |
| *** yolanda has quit IRC | 03:40 | |
| *** yuanying has joined #openstack-ironic | 03:43 | |
| *** rajinir has quit IRC | 03:46 | |
| *** yolanda has joined #openstack-ironic | 03:47 | |
| *** yuanying has quit IRC | 03:48 | |
| *** yuanying has joined #openstack-ironic | 03:49 | |
| *** yufei has joined #openstack-ironic | 03:50 | |
| *** yolanda has quit IRC | 03:51 | |
| *** yuanying has quit IRC | 03:53 | |
| *** SerenaFeng has quit IRC | 03:53 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 03:56 | |
| *** devlaps has quit IRC | 03:58 | |
| *** yolanda has joined #openstack-ironic | 04:05 | |
| *** yolanda has quit IRC | 04:10 | |
| *** baoli has joined #openstack-ironic | 04:12 | |
| *** SerenaFeng has joined #openstack-ironic | 04:13 | |
| *** SerenaFeng has quit IRC | 04:16 | |
| *** links has quit IRC | 04:20 | |
| *** yolanda has joined #openstack-ironic | 04:22 | |
| *** watanabe_isao has joined #openstack-ironic | 04:22 | |
| *** yolanda has quit IRC | 04:26 | |
| *** yolanda has joined #openstack-ironic | 04:29 | |
| *** yolanda has quit IRC | 04:35 | |
| *** yolanda has joined #openstack-ironic | 04:37 | |
| *** yuanying has joined #openstack-ironic | 04:40 | |
| *** misty-cloud has joined #openstack-ironic | 04:41 | |
| *** yolanda has quit IRC | 04:41 | |
| *** links has joined #openstack-ironic | 04:57 | |
| *** yolanda has joined #openstack-ironic | 04:57 | |
| *** yolanda has quit IRC | 05:02 | |
| *** baoli has quit IRC | 05:02 | |
| *** baoli has joined #openstack-ironic | 05:03 | |
| *** baoli has quit IRC | 05:07 | |
| *** aparnav has joined #openstack-ironic | 05:08 | |
| *** yuanying has quit IRC | 05:13 | |
| *** yuanying has joined #openstack-ironic | 05:13 | |
| *** robcresswell has quit IRC | 05:14 | |
| *** zhenguo_ has joined #openstack-ironic | 05:14 | |
| *** yolanda has joined #openstack-ironic | 05:14 | |
| *** natorious has quit IRC | 05:14 | |
| *** fyxim has quit IRC | 05:14 | |
| *** mgagne has quit IRC | 05:15 | |
| *** zhenguo has quit IRC | 05:15 | |
| *** zhenguo_ is now known as zhenguo | 05:15 | |
| *** ildikov has quit IRC | 05:15 | |
| *** ildikov has joined #openstack-ironic | 05:16 | |
| *** jkilpatr has quit IRC | 05:16 | |
| *** krtaylor has quit IRC | 05:16 | |
| *** betherly has quit IRC | 05:16 | |
| *** fragatina has joined #openstack-ironic | 05:17 | |
| *** robcresswell has joined #openstack-ironic | 05:18 | |
| *** fyxim has joined #openstack-ironic | 05:19 | |
| *** yolanda has quit IRC | 05:19 | |
| *** betherly has joined #openstack-ironic | 05:20 | |
| *** fragatina has quit IRC | 05:26 | |
| *** fragatina has joined #openstack-ironic | 05:27 | |
| *** krtaylor has joined #openstack-ironic | 05:28 | |
| *** Sukhdev has joined #openstack-ironic | 05:31 | |
| *** jkilpatr has joined #openstack-ironic | 05:31 | |
| *** SerenaFeng has joined #openstack-ironic | 05:41 | |
| *** absubram has joined #openstack-ironic | 05:49 | |
| *** absubram has quit IRC | 05:57 | |
| *** jaosorior has joined #openstack-ironic | 06:02 | |
| *** Nisha_Agarwal has quit IRC | 06:05 | |
| *** yolanda has joined #openstack-ironic | 06:12 | |
| openstackgerrit | Takanori Miyagishi proposed openstack/ironic: Fix double dots at the end of a message to single dot https://review.openstack.org/407281 | 06:23 |
| *** fragatina has quit IRC | 06:26 | |
| *** nandal has joined #openstack-ironic | 06:28 | |
| *** yolanda has quit IRC | 06:28 | |
| *** yolanda has joined #openstack-ironic | 06:30 | |
| *** yolanda has quit IRC | 06:37 | |
| *** yolanda has joined #openstack-ironic | 06:40 | |
| *** ChubYann has quit IRC | 06:42 | |
| *** yolanda has quit IRC | 06:45 | |
| *** yolanda has joined #openstack-ironic | 06:46 | |
| openstackgerrit | zhangguoqing proposed openstack/python-ironicclient: Fix 'X-Subject-Token' format which should without dashed https://review.openstack.org/407882 | 06:53 |
| *** Sukhdev has quit IRC | 07:01 | |
| *** fxpester has joined #openstack-ironic | 07:05 | |
| *** yuanying has quit IRC | 07:05 | |
| *** mjura has joined #openstack-ironic | 07:09 | |
| *** moshele has joined #openstack-ironic | 07:09 | |
| *** yolanda has quit IRC | 07:10 | |
| *** yolanda has joined #openstack-ironic | 07:14 | |
| *** mjura has quit IRC | 07:16 | |
| *** e0ne has joined #openstack-ironic | 07:16 | |
| *** yuanying has joined #openstack-ironic | 07:17 | |
| *** mjura has joined #openstack-ironic | 07:17 | |
| *** fragatina has joined #openstack-ironic | 07:18 | |
| *** jaosorior has quit IRC | 07:22 | |
| *** yolanda has quit IRC | 07:23 | |
| *** jaosorior has joined #openstack-ironic | 07:23 | |
| *** yolanda has joined #openstack-ironic | 07:24 | |
| *** aNuposic has joined #openstack-ironic | 07:25 | |
| *** zhangjl1 has joined #openstack-ironic | 07:27 | |
| *** zhangjl has quit IRC | 07:28 | |
| openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: Skip API related work if no api url configured https://review.openstack.org/399857 | 07:30 |
| *** zhangjl1 has quit IRC | 07:31 | |
| *** zhangjl has joined #openstack-ironic | 07:32 | |
| *** rcernin has joined #openstack-ironic | 07:34 | |
| *** rbartal has joined #openstack-ironic | 07:41 | |
| *** pcaruana has joined #openstack-ironic | 07:41 | |
| *** jpich has joined #openstack-ironic | 07:44 | |
| *** rbartal has quit IRC | 07:46 | |
| *** rbartal has joined #openstack-ironic | 07:48 | |
| *** rybridges2 has quit IRC | 07:54 | |
| *** fragatina has quit IRC | 07:58 | |
| openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: Skip API related work if no api url configured https://review.openstack.org/399857 | 07:59 |
| openstackgerrit | octopuszhang proposed openstack/ironic: Update multitenancy docs https://review.openstack.org/397517 | 08:00 |
| *** jrcloud has joined #openstack-ironic | 08:00 | |
| *** moshele has quit IRC | 08:01 | |
| *** zackf has quit IRC | 08:02 | |
| *** UForgotten has quit IRC | 08:04 | |
| *** UForgotten has joined #openstack-ironic | 08:05 | |
| *** vsaienko has joined #openstack-ironic | 08:06 | |
| *** rybridges2 has joined #openstack-ironic | 08:07 | |
| *** yuanying has quit IRC | 08:08 | |
| *** aNuposic has quit IRC | 08:14 | |
| *** xiaoyang has joined #openstack-ironic | 08:16 | |
| *** magic has quit IRC | 08:20 | |
| *** magic has joined #openstack-ironic | 08:23 | |
| *** xiaoyang has quit IRC | 08:26 | |
| *** amoralej|off is now known as amoralej | 08:29 | |
| *** vsaienko has quit IRC | 08:36 | |
| *** roeyc has joined #openstack-ironic | 08:42 | |
| *** roeyc has left #openstack-ironic | 08:44 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 08:47 | |
| *** vsaienko has joined #openstack-ironic | 08:48 | |
| *** e0ne has quit IRC | 08:49 | |
| *** ralonsoh has joined #openstack-ironic | 08:53 | |
| *** priteau has joined #openstack-ironic | 08:55 | |
| *** yuanying has joined #openstack-ironic | 08:59 | |
| *** daemontool_ has quit IRC | 08:59 | |
| *** athomas has joined #openstack-ironic | 08:59 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-ironic | 09:01 | |
| openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: Skip API related work if no api url configured https://review.openstack.org/399857 | 09:04 |
| sergek | Good morning Ironicers! | 09:04 |
| *** vsaienko has quit IRC | 09:09 | |
| *** vsaienko has joined #openstack-ironic | 09:11 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catch unknown exceptions in validate driver ifaces https://review.openstack.org/406157 | 09:15 |
| *** zhangjl1 has joined #openstack-ironic | 09:17 | |
| *** zhangjl has quit IRC | 09:20 | |
| openstackgerrit | paresh sao proposed openstack/ironic: Boot from iSCSI volume https://review.openstack.org/308425 | 09:21 |
| *** ashestakov has joined #openstack-ironic | 09:21 | |
| *** sborkows has joined #openstack-ironic | 09:23 | |
| *** vsaienko has quit IRC | 09:25 | |
| *** vsaienko has joined #openstack-ironic | 09:25 | |
| *** ashestakov has quit IRC | 09:26 | |
| *** moshele has joined #openstack-ironic | 09:26 | |
| *** jtomasek has joined #openstack-ironic | 09:31 | |
| *** lucas-afk is now known as lucasagomes | 09:32 | |
| *** Romanenko_K has joined #openstack-ironic | 09:36 | |
| *** vsaienko has quit IRC | 09:37 | |
| *** jpich has quit IRC | 09:37 | |
| *** ohamada has joined #openstack-ironic | 09:38 | |
| *** derekh has joined #openstack-ironic | 09:40 | |
| openstackgerrit | Markos Chandras proposed openstack/bifrost: README.rst: Add openSUSE Leap to list of supported distributions https://review.openstack.org/407970 | 09:41 |
| *** jpich has joined #openstack-ironic | 09:47 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Make "enabled_drivers" config option more resilient to failures https://review.openstack.org/407596 | 09:47 |
| *** e0ne has joined #openstack-ironic | 09:51 | |
| *** e0ne has quit IRC | 10:01 | |
| *** dtantsur|afk is now known as dtantsur | 10:03 | |
| dtantsur | morning Ironic | 10:03 |
| *** e0ne has joined #openstack-ironic | 10:03 | |
| *** pshedimb has left #openstack-ironic | 10:04 | |
| lucasagomes | dtantsur, morning | 10:06 |
| *** Nisha_Agarwal has quit IRC | 10:06 | |
| dtantsur | how is it going? | 10:06 |
| lucasagomes | dtantsur, all is well, apart from the internet that is down here so I'm tethering from my mobile heh | 10:08 |
| lucasagomes | you ? | 10:08 |
| *** abalfour has quit IRC | 10:09 | |
| *** abalfour has joined #openstack-ironic | 10:09 | |
| dtantsur | ETOOMANYTASKS :) otherwise pretty good | 10:09 |
| dtantsur | heh, we have 24-hour internet outage from Monday evening to Tuesday morning | 10:09 |
| dtantsur | quite a few things to finish in the 10 days before my PTO | 10:10 |
| lucasagomes | yeah :-/ | 10:11 |
| tuanluong | Morning Ironic | 10:11 |
| tuanluong | mornig dtantsur lucasagomes | 10:11 |
| dtantsur | hey-hey tuanluong | 10:12 |
| dtantsur | lucasagomes, for me it's driver composition, undercloud cleaning, undercloud cinder integration + reviews of everything :) | 10:12 |
| *** SerenaFeng has quit IRC | 10:14 | |
| lucasagomes | dtantsur, what's missing for cleaning ? | 10:15 |
| pas-ha | morning Ironic :) | 10:15 |
| dtantsur | lucasagomes, I think only https://review.openstack.org/406197 | 10:17 |
| patchbot | patch 406197 - instack-undercloud - Set Ironic cleaning network to ctlplane | 10:17 |
| dtantsur | morning pas-ha | 10:17 |
| dtantsur | lucasagomes, and some testing :) | 10:17 |
| lucasagomes | pas-ha, morning! | 10:17 |
| *** tuanluong has quit IRC | 10:17 | |
| pas-ha | morning dtantsur lucasagomes | 10:17 |
| lucasagomes | dtantsur, fair enuff! And apparently their multinode job is broken, seem unrelated to the patch tho http://logs.openstack.org/97/406197/1/check/gate-tripleo-ci-centos-7-nonha-multinode/a816601/console.html#_2016-12-06_16_27_56_980234 | 10:18 |
| dtantsur | lucasagomes, yep, it was broken yesterday, I've just rechecked | 10:19 |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic: Add object version pinning https://review.openstack.org/407997 | 10:23 |
| lucasagomes | dtantsur, btw, based on the planning yesterday https://bugs.launchpad.net/ironic/+bug/1648036 | 10:27 |
| openstack | Launchpad bug 1648036 in Ironic "[RFE] Add support for HCTL root device hint in Ironic" [Low,New] - Assigned to Lucas Alvares Gomes (lucasagomes) | 10:27 |
| *** ashestakov has joined #openstack-ironic | 10:28 | |
| dtantsur | lucasagomes, +1. do you have any plans on ironic-lib root device hints in inspector? | 10:29 |
| dtantsur | I'm a bit worried that they might diverge.. | 10:29 |
| ashestakov | @bcornec1 @bcornec2 around? | 10:29 |
| lucasagomes | dtantsur, yes, I can work on that too | 10:29 |
| lucasagomes | dtantsur, we def should reuse the same code | 10:29 |
| dtantsur | cool | 10:29 |
| dtantsur | also will appreciate any reviews on the next driver comp patch: https://review.openstack.org/336626 | 10:34 |
| patchbot | patch 336626 - ironic - Support defining and loading hardware types | 10:34 |
| dtantsur | (I do realize it's pretty crazily huge and involved) | 10:35 |
| *** david-lyle_ has joined #openstack-ironic | 10:35 | |
| *** milan has joined #openstack-ironic | 10:36 | |
| milan | morning Ironic! :) | 10:36 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Introduce generic hardware types https://review.openstack.org/400678 | 10:37 |
| *** david-lyle has quit IRC | 10:37 | |
| dtantsur | morning milan | 10:37 |
| milan | morning dtantsur! :) | 10:37 |
| *** yufei has quit IRC | 10:41 | |
| lucasagomes | o/ | 10:42 |
| *** jaosorior has quit IRC | 10:42 | |
| *** nandal has quit IRC | 10:46 | |
| *** mgagne has joined #openstack-ironic | 10:47 | |
| *** mgagne is now known as Guest2615 | 10:47 | |
| *** SerenaFeng has joined #openstack-ironic | 10:49 | |
| *** SerenaFeng has quit IRC | 10:49 | |
| sborkows | Hi, can someone look at https://review.openstack.org/#/c/391836 ? I need more comments on that, because I don't know which approach is more valid. | 10:50 |
| patchbot | patch 391836 - ironic-python-agent - Add a new Hardware Manager for CNA network card | 10:50 |
| *** natorious has joined #openstack-ironic | 10:50 | |
| *** dtantsur is now known as dtantsur|brb | 10:56 | |
| *** ildikov has quit IRC | 11:00 | |
| *** sambetts|afk is now known as sambetts | 11:00 | |
| *** ildikov has joined #openstack-ironic | 11:00 | |
| milan | morning lucasagomes, sborkows! :) | 11:05 |
| milan | morning sambetts! :) | 11:05 |
| sborkows | morning milan | 11:05 |
| *** sborkows has quit IRC | 11:06 | |
| *** zhangjl1 has quit IRC | 11:14 | |
| sambetts | Hey milan | 11:15 |
| *** misty-cloud has quit IRC | 11:20 | |
| *** gcb has quit IRC | 11:24 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Make _get_sensors_data concurrent https://review.openstack.org/407429 | 11:27 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Make "enabled_drivers" config option more resilient to failures https://review.openstack.org/407596 | 11:30 |
| *** openstackgerrit has quit IRC | 11:32 | |
| *** openstackgerrit has joined #openstack-ironic | 11:35 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Add HCTL to BlockDevices https://review.openstack.org/408032 | 11:35 |
| *** ptx0 has quit IRC | 11:45 | |
| *** ptx0 has joined #openstack-ironic | 11:46 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session glance and swift https://review.openstack.org/352289 | 11:47 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session glance and swift https://review.openstack.org/352289 | 11:47 |
| *** jkilpatr has quit IRC | 11:49 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Check for HCTL as part of root device hints https://review.openstack.org/408042 | 11:52 |
| *** e0ne has quit IRC | 11:57 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Remove the VALID_ROOT_DEVICE_HINTS list https://review.openstack.org/408045 | 11:58 |
| *** dtantsur|brb is now known as dtantsur | 12:00 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Document HCTL for root device hints https://review.openstack.org/408051 | 12:05 |
| *** jkilpatr has joined #openstack-ironic | 12:07 | |
| *** baoli has joined #openstack-ironic | 12:26 | |
| *** baoli has quit IRC | 12:26 | |
| *** baoli has joined #openstack-ironic | 12:26 | |
| *** misty-cloud has joined #openstack-ironic | 12:28 | |
| jroll | morning y'all | 12:31 |
| jroll | sambetts: matt is asking if we can get this updated and make some progress https://review.openstack.org/#/c/364413/ | 12:31 |
| patchbot | patch 364413 - nova - [WIP] Support Ironic interface attach/detach in no... | 12:31 |
| *** xiaoyang has joined #openstack-ironic | 12:31 | |
| *** jaosorior has joined #openstack-ironic | 12:32 | |
| lucasagomes | jroll, morning | 12:33 |
| *** magic has quit IRC | 12:33 | |
| vsaienk0 | morning lucasagomes, jroll, dtantsur, milan, sambetts and all Ironic'ers! | 12:34 |
| milan | morning jroll, vsaienk0! :) | 12:34 |
| sambetts | jroll: So I believe the reason I couldn't use a depends on with the nova patch is because you can't depends-on an ironic-client change becuase we don't install it from source, which is why I ended up creating this patch https://review.openstack.org/#/c/367948/4 | 12:34 |
| patchbot | patch 367948 - ironic - DO NOT MERGE Testing Attach/Detach changes | 12:34 |
| *** magic has joined #openstack-ironic | 12:34 | |
| jroll | sambetts: which is fine, matt is wondering if we can get the code updated so they can see progress | 12:35 |
| sambetts | jroll: yeah sure :) | 12:35 |
| *** vikrant has quit IRC | 12:36 | |
| jroll | sambetts: also I think you need the gerrit branch there :) | 12:36 |
| *** xiaoyang has quit IRC | 12:36 | |
| jroll | GITBRANCH["python-ironicclient"]=refs/changes/20/364420/4 | 12:36 |
| jroll | (from the "download" thing in the upper right corner of the ironicclient change) | 12:36 |
| sambetts | jroll: in the devstack/lib/ironic? that all gets handled by the zuul wizardry and the depends-on in the commit message :) | 12:37 |
| jroll | sambetts: neat, TIL | 12:37 |
| sambetts | jroll: when you use a depends-on "master" becomes the patch you've depended on | 12:37 |
| jroll | I assumed specifying master would grab master | 12:37 |
| jroll | ya | 12:37 |
| jroll | cool | 12:37 |
| sambetts | yeah its pretty cool, I only learned that when setting up my CI | 12:37 |
| sambetts | zuul is actually really neat | 12:38 |
| jroll | tis | 12:38 |
| sambetts | and jenkins is a ball ache | 12:38 |
| jroll | but yeah, he wants to see some progress there I guess, you can depends-on 367948 when you have time :) | 12:38 |
| *** xiaoyang has joined #openstack-ironic | 12:40 | |
| dtantsur | morning jroll, vsaienk0, sambetts | 12:42 |
| sambetts | hey dtantsur | 12:42 |
| *** magic has quit IRC | 12:42 | |
| jroll | hi dtantsur :) | 12:43 |
| *** magic has joined #openstack-ironic | 12:43 | |
| nicodemos | good morning! | 12:45 |
| *** xiaoyang has quit IRC | 12:45 | |
| *** lucasagomes is now known as lucas-hungry | 12:49 | |
| *** gcb has joined #openstack-ironic | 12:50 | |
| *** baoli has quit IRC | 12:51 | |
| *** misty-cloud has quit IRC | 12:58 | |
| *** jaosorior has quit IRC | 12:59 | |
| *** misty-cloud has joined #openstack-ironic | 13:04 | |
| milan | morning nicodemos! :) | 13:06 |
| *** amoralej is now known as amoralej|lunch | 13:08 | |
| *** bfournie has quit IRC | 13:08 | |
| *** jaosorior has joined #openstack-ironic | 13:10 | |
| *** jpich has quit IRC | 13:11 | |
| *** heroux has joined #openstack-ironic | 13:17 | |
| *** srobert has joined #openstack-ironic | 13:18 | |
| *** hw_wutianwei has quit IRC | 13:24 | |
| *** aparnav has quit IRC | 13:25 | |
| *** mgould|afk is now known as mgould | 13:26 | |
| gcb | hi, is it possible boot from volume for ironic ? | 13:27 |
| jroll | gcb: it's a thing we're working on, not yet | 13:28 |
| *** watanabe_isao has quit IRC | 13:28 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/408091 | 13:32 |
| gcb | jroll , cool , a customer want to this function :-) any link about this ? | 13:33 |
| *** trown|outtypewww is now known as trown | 13:34 | |
| jroll | gcb: here's the spec http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/boot-from-volume-reference-drivers.html | 13:35 |
| vsaienk0 | jroll, jlvillal: I'm trying to drop intree ironic multitenancy tests, and replace them by tempest one, this is first patch to tempest that stop relying on multitenancy by baremetal.driver_enabled https://review.openstack.org/#/c/368173/ Please review once have a time. Ironic CI passed https://review.openstack.org/#/c/368173/ | 13:36 |
| patchbot | patch 368173 - ironic - Testing multinode stuff | 13:36 |
| patchbot | patch 368173 - ironic - Testing multinode stuff | 13:36 |
| *** vsaienk0 has left #openstack-ironic | 13:36 | |
| *** tuan_luong has joined #openstack-ironic | 13:36 | |
| *** vsaienk0 has joined #openstack-ironic | 13:36 | |
| vsaienk0 | jroll, jlvillal related tempest patch is https://review.openstack.org/#/c/406999 | 13:37 |
| patchbot | patch 406999 - tempest - Multitenancy not restricted by Ironic anymore | 13:37 |
| openstackgerrit | Miles Gould proposed openstack/ironic: Add release names & numbers to API version history https://review.openstack.org/404211 | 13:37 |
| gcb | jroll, thanks | 13:37 |
| jroll | gcb: np | 13:37 |
| jroll | vsaienk0: please rebase that work on https://review.openstack.org/#/c/355586/ | 13:38 |
| patchbot | patch 355586 - tempest - Removing baremetal tests from tempest tree | 13:38 |
| openstackgerrit | Milan Kováčik proposed openstack/python-ironic-inspector-client: List introspection statuses support https://review.openstack.org/408116 | 13:41 |
| * jroll runs to a dentist appt, back later | 13:42 | |
| *** e0ne has joined #openstack-ironic | 13:42 | |
| *** bfournie has joined #openstack-ironic | 13:46 | |
| *** jcoufal has joined #openstack-ironic | 13:48 | |
| *** SerenaFeng has joined #openstack-ironic | 13:48 | |
| *** rcernin has quit IRC | 13:49 | |
| *** rcernin has joined #openstack-ironic | 13:51 | |
| vsaienk0 | jroll: will do, wondered why we introduced new shared_physical_network variable which has the same meaning as create_isolated_networks https://github.com/openstack/tempest/blob/32a9e308e65b99a80575354532dbca98dea5ea76/tempest/config.py#L84 | 13:53 |
| *** Guest2615 is now known as mgagne | 13:54 | |
| *** mgagne has quit IRC | 13:54 | |
| *** mgagne has joined #openstack-ironic | 13:54 | |
| *** lucas-hungry is now known as lucasagomes | 13:58 | |
| *** jpich has joined #openstack-ironic | 13:58 | |
| *** links has quit IRC | 13:59 | |
| *** glonlas has joined #openstack-ironic | 14:01 | |
| *** glonlas has quit IRC | 14:01 | |
| *** amoralej|lunch is now known as amoralej | 14:07 | |
| *** glonlas has joined #openstack-ironic | 14:11 | |
| *** dprince has joined #openstack-ironic | 14:14 | |
| *** baoli has joined #openstack-ironic | 14:15 | |
| *** lindycoder has joined #openstack-ironic | 14:18 | |
| *** mtanino has joined #openstack-ironic | 14:22 | |
| *** gcb has quit IRC | 14:24 | |
| *** krtaylor has quit IRC | 14:28 | |
| *** krtaylor has joined #openstack-ironic | 14:29 | |
| *** rbudden has joined #openstack-ironic | 14:29 | |
| *** rbartal has quit IRC | 14:33 | |
| *** vsaienko has joined #openstack-ironic | 14:38 | |
| *** tuan_luong has quit IRC | 14:38 | |
| *** dprince has quit IRC | 14:38 | |
| *** vsaienko has quit IRC | 14:40 | |
| *** cdearborn has joined #openstack-ironic | 14:42 | |
| *** jaosorior has quit IRC | 14:43 | |
| *** trown|brb has joined #openstack-ironic | 14:47 | |
| *** trown|brb is now known as trown_ | 14:47 | |
| *** trown has quit IRC | 14:48 | |
| *** ohamada has quit IRC | 14:48 | |
| *** ohamada has joined #openstack-ironic | 14:49 | |
| jroll | vsaienk0: because that's what qa team wanted ¯\_(ツ)_/¯ | 14:49 |
| *** yufei has joined #openstack-ironic | 14:49 | |
| *** trown has joined #openstack-ironic | 14:50 | |
| *** trown_ has quit IRC | 14:50 | |
| *** Goneri has joined #openstack-ironic | 14:52 | |
| *** dprince has joined #openstack-ironic | 14:52 | |
| *** rama_y has joined #openstack-ironic | 14:53 | |
| *** SerenaFeng has quit IRC | 14:53 | |
| *** yolanda has quit IRC | 14:59 | |
| *** yolanda has joined #openstack-ironic | 14:59 | |
| *** glonlas_ has joined #openstack-ironic | 15:01 | |
| *** links has joined #openstack-ironic | 15:03 | |
| *** glonlas has quit IRC | 15:04 | |
| *** Syed__ has joined #openstack-ironic | 15:07 | |
| *** rama_y has quit IRC | 15:09 | |
| vsaienk0 | dtantsur, aarefiev: when ironic is removed from tempest inspector start failing with http://logs.openstack.org/75/397775/5/check/gate-tempest-dsvm-ironic-inspector-ubuntu-xenial-nv/05b3378/console.html#_2016-12-07_14_21_10_759451 it should be fixed before https://review.openstack.org/#/c/355586/ is mergedf | 15:09 |
| patchbot | patch 355586 - tempest - Removing baremetal tests from tempest tree | 15:09 |
| vsaienk0 | not sure if you already know about it, just fyi ^ | 15:10 |
| dtantsur | vsaienk0, any ideas how to fix it? | 15:11 |
| dtantsur | also milan fyi ^^^ | 15:11 |
| *** dprince has quit IRC | 15:12 | |
| *** misty-cloud is now known as nmathew | 15:13 | |
| vsaienk0 | dtantsur: something like this https://github.com/openstack/ironic/commit/ab6f73f7a06a9c9e497762e1dc917a9626cccd26#diff-3e7370b7b0ad85701e43800afdd4c305 | 15:14 |
| dtantsur | vsaienk0, hmm, maybe we should update ironic to keep them in "self" | 15:15 |
| dtantsur | i.e. make self.default_params_with_timeout_values still work | 15:15 |
| * dtantsur writes a patch | 15:15 | |
| *** nmathew has quit IRC | 15:15 | |
| vsaienk0 | dtantsur: but ironic-inspector client manager is inherited directly from tempest manager not ironic https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/test/inspector_tempest_plugin/services/introspection_client.py#L23 | 15:16 |
| dtantsur | hmm, I wonder if we even need these parameters | 15:16 |
| *** nmathew has joined #openstack-ironic | 15:16 | |
| *** nmathew has quit IRC | 15:17 | |
| vsaienk0 | dtantsur: looks like we can not pass them https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/test/inspector_tempest_plugin/services/introspection_client.py#L34 and pick default in the base class | 15:17 |
| jroll | I thought there was a patch for this | 15:17 |
| * jroll looks | 15:17 | |
| aarefiev | dtantsur: we can just delete them | 15:18 |
| jroll | otherwise, yes, something like the commit vasyl linked | 15:18 |
| dtantsur | yeah, feels like this is not needed | 15:18 |
| dtantsur | why do we even pass some timeouts to inspector client? Oo | 15:18 |
| dtantsur | btw I think it's also wrong for ironic | 15:19 |
| jroll | ah, I thought it was inherited from ironic, that's why I didn't patch | 15:19 |
| *** fxpester has quit IRC | 15:20 | |
| jroll | dtantsur: that's not the python client, it's the client in the tempest plugin | 15:20 |
| *** glonlas_ has quit IRC | 15:20 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: DO NOT MERGE Testing Attach/Detach changes https://review.openstack.org/367948 | 15:21 |
| jroll | https://github.com/openstack/ironic/blob/master/ironic_tempest_plugin/services/baremetal/base.py#L43 | 15:21 |
| *** anush has quit IRC | 15:21 | |
| dtantsur | still, I wonder if we need these timeouts at all... (maybe we do) | 15:21 |
| jroll | although maybe you're right | 15:21 |
| jroll | https://github.com/openstack/tempest/blob/master/tempest/lib/common/rest_client.py#L90 | 15:22 |
| *** glonlas has joined #openstack-ironic | 15:22 | |
| jroll | default is 60 | 15:22 |
| patchbot | Error: 'supybot.is' is not a valid configuration variable. | 15:22 |
| jroll | so we probably need it :) | 15:22 |
| jroll | /kick patchbot | 15:22 |
| *** daemontool has joined #openstack-ironic | 15:23 | |
| * dtantsur is confused | 15:23 | |
| aarefiev | looks like they should go to tempest | 15:24 |
| jroll | what should go to tempest? | 15:25 |
| aarefiev | this configuration options | 15:25 |
| aarefiev | they are in compute group | 15:26 |
| *** rama_y has joined #openstack-ironic | 15:27 | |
| jroll | the config options *are* in tempest | 15:27 |
| jroll | but the definition of self.default_params_with_timeout_values is no longer in the base class | 15:28 |
| *** jpich has quit IRC | 15:28 | |
| jroll | (because that was something only baremetal tests used) | 15:28 |
| aarefiev | yeah, I was confused, why they are not in ironic then :) | 15:28 |
| *** jpich has joined #openstack-ironic | 15:29 | |
| jroll | they are :) | 15:29 |
| *** anush has joined #openstack-ironic | 15:29 | |
| jroll | https://github.com/openstack/ironic/blob/master/ironic_tempest_plugin/clients.py#L39 | 15:29 |
| jroll | I can fix inspector, sorry, didn't realize it needed it | 15:30 |
| aarefiev | ok, we can just remove it from inspector | 15:30 |
| jroll | yeah, I suspect I agree, we'll see what CI says :) | 15:31 |
| *** baoli has quit IRC | 15:32 | |
| *** ashestakov has quit IRC | 15:32 | |
| *** absubram has joined #openstack-ironic | 15:32 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-inspector: Remove default_params_with_timeout_values from tempest client https://review.openstack.org/408179 | 15:32 |
| jroll | ^ | 15:32 |
| dtantsur | thanks! | 15:32 |
| aarefiev | the default seems fine, lets see | 15:33 |
| *** ohamada has quit IRC | 15:35 | |
| *** ohamada has joined #openstack-ironic | 15:35 | |
| *** athomas has quit IRC | 15:35 | |
| *** moshele has quit IRC | 15:37 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Add HCTL to BlockDevices https://review.openstack.org/408032 | 15:38 |
| rama_y | Hi dtantsur, this is regarding https://review.openstack.org/#/c/406790/ | 15:39 |
| patchbot | patch 406790 - ironic-inspector-specs - Collect Bios, Memory and Processor Info from Dmide... | 15:39 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Update mock variable name in unit tests https://review.openstack.org/407729 | 15:41 |
| rama_y | dtantsur, is the recommendation not to add a new plugin, but to add to capabilities? | 15:42 |
| dtantsur | yep | 15:43 |
| TheJulia | good morning everyone | 15:43 |
| dtantsur | morning TheJulia | 15:43 |
| *** Sukhdev has joined #openstack-ironic | 15:44 | |
| rama_y | dtantsur, ok, thanks. I will update the spec accordingly. | 15:44 |
| *** athomas has joined #openstack-ironic | 15:44 | |
| *** soliosg has joined #openstack-ironic | 15:45 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [ipmitool] Do not try powering off nodes that are already off when rebooting https://review.openstack.org/402052 | 15:45 |
| dtantsur | yuriyz, attached a bug ^^^ | 15:46 |
| aslezil | Good Morning Ironic | 15:48 |
| *** heroux has quit IRC | 15:48 | |
| aslezil | dtantsur: quick question regarding the spec for collecting biosdevname https://review.openstack.org/#/c/404372/8 | 15:49 |
| patchbot | patch 404372 - ironic-inspector-specs - Collect NIC name given by BIOS | 15:49 |
| *** hogepodge has joined #openstack-ironic | 15:50 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Update mock variable name in unit tests https://review.openstack.org/407729 | 15:50 |
| aslezil | dtanstsur: I have commented on the spec, does that make sense. | 15:50 |
| *** Sukhdev_ has joined #openstack-ironic | 15:51 | |
| *** rama_y has quit IRC | 15:51 | |
| *** jheroux has joined #openstack-ironic | 15:52 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 15:53 | |
| dtantsur | aslezil, no, sorry. I don't see how your comments change the fact that we don't store invontory or any of its parts in node.extra or port.exttra | 15:56 |
| * jroll also commented, doesn't understand why we need biosdevname for bonding | 15:57 | |
| *** chlong has joined #openstack-ironic | 15:58 | |
| *** rbartal has joined #openstack-ironic | 15:58 | |
| JayF | dtantsur: jroll: Thanks for commenting, aslezil and I are working on getting a more detailed use case or the design. | 15:59 |
| jroll | also curious what "config files to script provisioning of nodes" means | 15:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DON NOT REVIEW https://review.openstack.org/408195 | 15:59 |
| dtantsur | jroll, wanna some fun long read? ;) https://review.openstack.org/336626 it is | 15:59 |
| patchbot | patch 336626 - ironic - Support defining and loading hardware types | 15:59 |
| *** snecklifter has joined #openstack-ironic | 15:59 | |
| dtantsur | JayF, aslezil, if you want some nice CLI, you can consider extending effort in http://specs.openstack.org/openstack/ironic-inspector-specs/specs/lldp-reporting.html | 16:00 |
| dtantsur | I'm not against nice reports of various inventory parts, I'm against storing it in ironic as opposed to our (already existing) inventory storage | 16:00 |
| *** jheroux has quit IRC | 16:00 | |
| JayF | dtantsur: yeah, that's clearly the missing piece. We'll get a valid use case or change the plan. | 16:01 |
| aslezil | dtantsur: thanks for reviewing I got you. Iam trying to get a good use case as JayF said | 16:01 |
| *** mgould is now known as mgould|afk | 16:01 | |
| jroll | JayF: fwiw, ironic or inspector using the extra field is a pretty hard no, that's for operators | 16:01 |
| jroll | dtantsur: wow, that's a patch | 16:02 |
| JayF | jroll: yep, I suggested to aslezil that it be updated to use node.properties if it gets stored in the node | 16:02 |
| dtantsur | jroll, yep :( | 16:02 |
| dtantsur | I tried to make it smaller, honestly | 16:02 |
| jroll | JayF: if it's being put in ironic, it should be something that ironic uses | 16:02 |
| jroll | dtantsur: yeah, I understand :) | 16:03 |
| JayF | jroll: I agree, that's why we're trying to track down a use case | 16:03 |
| yufei | hello, vsaienk0, I see your comments at https://review.openstack.org/#/c/397517/, looks good, but I have one question. Is the provision network suggested to be a non-shared network in admin or service? If so, Which tenant is better? | 16:03 |
| patchbot | patch 397517 - ironic - Update multitenancy docs | 16:03 |
| *** rloo has joined #openstack-ironic | 16:04 | |
| dtantsur | lucasagomes, created a bug for you https://bugs.launchpad.net/ironic-inspector/+bug/1648133 | 16:04 |
| openstack | Launchpad bug 1648133 in Ironic Inspector "Use root device hints from ironic-lib" [Medium,Triaged] - Assigned to Lucas Alvares Gomes (lucasagomes) | 16:04 |
| lucasagomes | dtantsur, cheers! | 16:06 |
| *** jheroux has joined #openstack-ironic | 16:06 | |
| *** rama_y has joined #openstack-ironic | 16:07 | |
| *** rpioso has joined #openstack-ironic | 16:09 | |
| *** aparnav has joined #openstack-ironic | 16:09 | |
| * jroll lunches | 16:12 | |
| JayF | jlvillal: TheJulia: https://review.openstack.org/#/c/399857/19 this appears ready now, I just readded my +2, if one of you want to land it now :D | 16:12 |
| patchbot | patch 399857 - ironic-python-agent - Skip API related work if no api url configured | 16:12 |
| * TheJulia loads | 16:13 | |
| *** links has quit IRC | 16:13 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Use identity api version 3 for OSC plugin tests https://review.openstack.org/408205 | 16:16 |
| *** jpich has quit IRC | 16:17 | |
| *** pcaruana has quit IRC | 16:18 | |
| *** rcernin has quit IRC | 16:19 | |
| NobodyCam | Good Morning Ironic'ers :) | 16:23 |
| *** mjura has quit IRC | 16:23 | |
| dtantsur | morning NobodyCam | 16:24 |
| NobodyCam | Morning dtantsur :) | 16:25 |
| rloo | morning NobodyCam, dtantsur, and everyone else! | 16:26 |
| dtantsur | morning rloo | 16:26 |
| NobodyCam | Morning rloo :) | 16:26 |
| JayF | o/ | 16:26 |
| NobodyCam | morning JayF :) | 16:26 |
| TheJulia | Good morning NobodyCam | 16:28 |
| NobodyCam | good morning TheJulia :) | 16:29 |
| NobodyCam | I'm out for a bit while I run to the doctors office :p | 16:29 |
| mariojv | lucasagomes: is it possible for hctl to be missing from a device when parsing root device hints? | 16:36 |
| mariojv | regarding https://review.openstack.org/#/c/408042 | 16:36 |
| patchbot | patch 408042 - ironic-lib - Check for HCTL as part of root device hints | 16:36 |
| lucasagomes | mariojv, yeah, if it's not an SCSI device it won't have a HCTL | 16:36 |
| lucasagomes | e.g IDE | 16:36 |
| mariojv | right | 16:36 |
| lucasagomes | or virtio if testing with libvirt | 16:37 |
| mariojv | thanks | 16:38 |
| lucasagomes | mariojv, this is the patch that gets the values from the devices itself:https://review.openstack.org/#/c/408032/2/ironic_python_agent/hardware.py | 16:38 |
| patchbot | patch 408032 - ironic-python-agent - Add HCTL to BlockDevices | 16:38 |
| lucasagomes | mariojv, thank you for reviewing it! | 16:38 |
| mariojv | ah, great | 16:38 |
| mariojv | np | 16:39 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector-specs: Endpoint to list introspection statuses https://review.openstack.org/375045 | 16:40 |
| dtantsur | rloo, I've already asked about the two Valence spec. They seem to work completely independently from each other. | 16:41 |
| mariojv | lucasagomes: yeah, i'm not sure that the IPA doc there needs to be udpated | 16:42 |
| mariojv | *updated | 16:42 |
| mariojv | seems like that's for inventory, not root device info | 16:42 |
| *** devlaps has joined #openstack-ironic | 16:42 | |
| mariojv | actually, it will update the "disks" section, i think | 16:43 |
| mariojv | lucasagomes: here: http://docs.openstack.org/developer/ironic-python-agent/#hardware-inventory | 16:43 |
| mariojv | so, perhaps both ironic (already done) and ipa docs need updates | 16:43 |
| lucasagomes | mariojv, oh good point, I didn't know we had a documentation for such fields | 16:43 |
| lucasagomes | mariojv, I will update it in the next patch-set | 16:43 |
| mariojv | thx | 16:44 |
| JayF | dtantsur: does ironic-inspector have keystone policy support yet? | 16:45 |
| rloo | dtantsur: oh, thx for asking! did you get any answer? | 16:46 |
| rloo | dtantsur: we should just get them to do one spec, like we did with the console stuff. | 16:47 |
| dtantsur | JayF, no | 16:47 |
| JayF | dtantsur: that's probably a good reason someone wouldn't want to expose inspector API to a larger group of users | 16:47 |
| dtantsur | rloo, no, I didn't. +1 on one spec | 16:47 |
| dtantsur | JayF, well, it may be a reason to add keystone policy ;) should not be hard, we have like 6-7 endpoints | 16:48 |
| yufei | hi, dtantsur, rloo, I’m the author of one of the valence spec, if you have any question about that, please @yufei_, I will response when I see it. | 16:48 |
| rloo | dtantsur: my other thought about that is that it might make more sense, at least as a first step, for the rsd intelligence (to compose nodes) to be outside ironic. make life easier. | 16:48 |
| rloo | yufei: ^^ | 16:49 |
| dtantsur | rloo, maybe? I just want to see one complete spec, not one placeholder and one incomplete | 16:49 |
| rloo | dtantsur: agreed. | 16:49 |
| rloo | dtantsur: i asked in the spec, but i'll mention that we just want one spec. | 16:50 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Add HCTL to BlockDevices https://review.openstack.org/408032 | 16:51 |
| vsaienk0 | yufei: by design openstack services uses service tenant accounts, so it doesn't matter. It is important that network should be hidden from non-admin tenants/users (non-shared networks are visible only for admins and tenant where they were created) | 16:52 |
| *** chlong has quit IRC | 16:56 | |
| yufei | get it, but I think that maybe we can suggest user to create provision network in service project, as I remember that rdo suggest to create floating network in service tenant. | 16:57 |
| *** snecklifter has quit IRC | 16:59 | |
| yufei | This will help avoid admin user create instance in provision network by mistake. | 16:59 |
| *** jrcloud is now known as zackf | 17:00 | |
| openstackgerrit | Joanna Taryma proposed openstack/ironic: Fail IPA config if no protocol prefix in ironic api address https://review.openstack.org/404975 | 17:04 |
| openstackgerrit | Merged openstack/ironic-python-agent: Skip API related work if no api url configured https://review.openstack.org/399857 | 17:05 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV https://review.openstack.org/397126 | 17:05 |
| dtantsur | JayF, do you think we should backport https://review.openstack.org/#/c/399857 ? I've seen some folks confused by ironic_api_client erroe messages after inspection | 17:07 |
| patchbot | patch 399857 - ironic-python-agent - Skip API related work if no api url configured (MERGED) | 17:07 |
| JayF | dtantsur: I have no objection to it whatsoever | 17:07 |
| JayF | dtantsur: not something I would've picked out myself for a stable patch, but I'd +2 a clean backport that passed CI | 17:08 |
| dtantsur | merge conflict, hmm | 17:08 |
| * dtantsur wishes gerrit actually showed the conflict | 17:08 | |
| *** aNuposic has joined #openstack-ironic | 17:08 | |
| jlvillal | JayF: Thanks for the ping on the patch | 17:09 |
| jlvillal | dtantsur: Yeah, I asked jeblair about adding a more informative message. He has it on his TODO list. | 17:09 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add ironic resources CRUD notifications https://review.openstack.org/356541 | 17:09 |
| *** moshele has joined #openstack-ironic | 17:09 | |
| jlvillal | vsaienk0: I'll take a look at that patch set. | 17:09 |
| dtantsur | jlvillal, cool! | 17:09 |
| jlvillal | dtantsur: No idea if it will happen. I took a peek at the Zuul code, but couldn't figure it out in 5 minutes. So gave up... | 17:10 |
| *** Sukhdev_ has quit IRC | 17:10 | |
| *** rcernin has joined #openstack-ironic | 17:11 | |
| jlvillal | vsaienk0: In your message above did you mean to mention the same patch twice? Or were there two patches I should look at? | 17:11 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: Skip API related work if no api url configured https://review.openstack.org/408222 | 17:12 |
| dtantsur | not exactly clean, but let's see ^^^ | 17:12 |
| *** milan has quit IRC | 17:12 | |
| JayF | dtantsur: that couldn't have been a bad conflict though, the code looks pretty clean? | 17:13 |
| *** aparnav has quit IRC | 17:13 | |
| *** rama_y has quit IRC | 17:13 | |
| dtantsur | JayF, yep, it's because of removing driver_name in Ocata | 17:13 |
| dtantsur | small conflict in agent.py | 17:13 |
| JayF | dtantsur: aha, that makes sense, I'll wait for CI but it looks fine to me | 17:13 |
| openstackgerrit | octopuszhang proposed openstack/ironic: Update multitenancy docs https://review.openstack.org/397517 | 17:13 |
| *** aparnav has joined #openstack-ironic | 17:14 | |
| *** jtomasek has quit IRC | 17:14 | |
| jlvillal | dtantsur: Do you like to keep the "conflicts" in the commit message. | 17:15 |
| * jlvillal usually deletes them | 17:15 | |
| JayF | jlvillal: stable policy suggests keeping them | 17:16 |
| JayF | jlvillal: so folks can see where you manually modified the patch | 17:16 |
| dtantsur | jlvillal, I used to delete them, then I read somewhere (probably the policy) that it's better to keep | 17:16 |
| dtantsur | yes, this | 17:16 |
| jlvillal | JayF: dtantsur: Ah thanks | 17:16 |
| *** moshele has quit IRC | 17:17 | |
| dtantsur | time to bail out, see you tomorrow | 17:19 |
| *** dtantsur is now known as dtantsur|afk | 17:19 | |
| JayF | o/ | 17:20 |
| *** sacharya has joined #openstack-ironic | 17:20 | |
| jlvillal | dtantsur|afk: Good night | 17:20 |
| *** aNuposic has quit IRC | 17:20 | |
| *** aNuposic has joined #openstack-ironic | 17:21 | |
| *** aparnav has quit IRC | 17:21 | |
| *** aparna has joined #openstack-ironic | 17:21 | |
| *** e0ne has quit IRC | 17:26 | |
| *** baoli has joined #openstack-ironic | 17:28 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node maintenance notifications https://review.openstack.org/396239 | 17:29 |
| *** chlong has joined #openstack-ironic | 17:30 | |
| *** priteau has quit IRC | 17:31 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node console notifications https://review.openstack.org/397812 | 17:32 |
| *** dhellmann has quit IRC | 17:33 | |
| *** rbartal has quit IRC | 17:34 | |
| lucasagomes | o/ calling it a day as well | 17:35 |
| lucasagomes | have a great evening all! | 17:35 |
| sambetts | cya lucasagomes | 17:35 |
| *** lucasagomes is now known as lucas-afk | 17:35 | |
| *** daemontool has quit IRC | 17:35 | |
| *** aparna has quit IRC | 17:38 | |
| *** rajinir has joined #openstack-ironic | 17:39 | |
| *** xiaoyang has joined #openstack-ironic | 17:40 | |
| openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: change mocked object name to follow naming conventions https://review.openstack.org/408233 | 17:40 |
| *** Nisha_Agarwal has quit IRC | 17:41 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add os_project_name to functional tests config https://review.openstack.org/408235 | 17:41 |
| *** magic has quit IRC | 17:43 | |
| openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: change mocked object name to follow naming conventions https://review.openstack.org/408233 | 17:44 |
| *** strigazi is now known as strigazi_AFK | 17:46 | |
| *** glonlas has quit IRC | 17:49 | |
| openstackgerrit | Merged openstack/ironic: Make "enabled_drivers" config option more resilient to failures https://review.openstack.org/407596 | 17:51 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 17:52 |
| JayF | does anyone know if Szymon Borkowski is on IRC? | 17:53 |
| *** jkilpatr_ has joined #openstack-ironic | 17:54 | |
| *** trown is now known as trown|lunch | 17:55 | |
| Romanenko_K | hi! | 17:55 |
| *** sacharya_ has joined #openstack-ironic | 17:56 | |
| *** jkilpatr has quit IRC | 17:56 | |
| JayF | jcook: are you going to have time today to update https://review.openstack.org/#/c/407236/ or should I take care of it? | 17:56 |
| patchbot | patch 407236 - ironic-specs - Update rescue spec | 17:56 |
| *** derekh has quit IRC | 17:56 | |
| *** rama_y has joined #openstack-ironic | 17:56 | |
| Romanenko_K | Something happened with fake nodes network interface on functional testing CI. Please look: https://bugs.launchpad.net/ironic/+bug/1648179 | 17:57 |
| openstack | Launchpad bug 1648179 in Ironic "Fake node validation error: gate-ironicclient-dsvm-functional fails on test_node_validate" [Undecided,New] | 17:57 |
| *** sacharya has quit IRC | 17:57 | |
| JayF | Romanenko_K: any idea when this failure started? | 17:59 |
| *** dhellmann has joined #openstack-ironic | 18:00 | |
| *** Sukhdev has quit IRC | 18:01 | |
| Romanenko_K | i am not sure, possibly something was merged today? I have one yesterday deployed Devstack and there fake node validation passes successfully. | 18:01 |
| *** athomas has quit IRC | 18:01 | |
| JayF | yeah, I didn't see anything in recent commits that jumped out at me | 18:02 |
| JayF | at least in ironic | 18:02 |
| *** david-lyle_ is now known as david-lyle | 18:02 | |
| *** fragatina has joined #openstack-ironic | 18:03 | |
| *** fragatin_ has joined #openstack-ironic | 18:04 | |
| *** ralonsoh has quit IRC | 18:04 | |
| jcook | mariojv JayF responded | 18:05 |
| JayF | jcook: I agree, I think you're right | 18:06 |
| *** krtaylor has quit IRC | 18:06 | |
| *** sacharya_ has quit IRC | 18:07 | |
| *** fragatina has quit IRC | 18:08 | |
| *** sacharya has joined #openstack-ironic | 18:08 | |
| *** vsaienko has joined #openstack-ironic | 18:08 | |
| *** jralph_ has quit IRC | 18:12 | |
| *** Romanenko_K has left #openstack-ironic | 18:12 | |
| *** yufei has quit IRC | 18:12 | |
| *** jralph has joined #openstack-ironic | 18:13 | |
| *** dhellmann has quit IRC | 18:15 | |
| mariojv | i'll look | 18:16 |
| *** ohamada has quit IRC | 18:17 | |
| mariojv | jcook: JayF: i thought we'd have to wait for that work to be complete | 18:17 |
| mariojv | jcook: JayF: or should we just flip nets etc immediately after finalize_rescue is called? | 18:17 |
| JayF | mariojv: there's no way for the agent to report back | 18:17 |
| mariojv | aka have it be a sync call | 18:17 |
| JayF | mariojv: if it's changing network config in the ramdisk | 18:17 |
| mariojv | i thought that was the point of having prepare_rescue and finalize_rescue, i guess | 18:17 |
| mariojv | oh | 18:18 |
| mariojv | got it | 18:18 |
| mariojv | so, it should be immediate | 18:18 |
| jcook | mariojv so with prepare and finalize it wasn't verified either | 18:18 |
| mariojv | right | 18:18 |
| mariojv | k, i'll update some code today | 18:18 |
| JayF | yeah; I'm going to work on the client code today | 18:19 |
| openstackgerrit | Merged openstack/ironic-inspector: Remove default_params_with_timeout_values from tempest client https://review.openstack.org/408179 | 18:19 |
| *** anush has quit IRC | 18:20 | |
| *** vsaienko has quit IRC | 18:23 | |
| *** amoralej is now known as amoralej|off | 18:25 | |
| *** MattMan_ has joined #openstack-ironic | 18:28 | |
| *** glonlas has joined #openstack-ironic | 18:30 | |
| *** mat128 has quit IRC | 18:31 | |
| *** MattMan has quit IRC | 18:32 | |
| *** mat128 has joined #openstack-ironic | 18:37 | |
| *** chlong has quit IRC | 18:38 | |
| * sambetts -> home | 18:40 | |
| *** mcarden has quit IRC | 18:40 | |
| sambetts | cya guys later | 18:40 |
| jlvillal | sambetts: good night | 18:40 |
| *** sambetts is now known as sambetts|afk | 18:40 | |
| *** anush has joined #openstack-ironic | 18:41 | |
| *** mcarden has joined #openstack-ironic | 18:41 | |
| jlvillal | jroll: Can I be of any assistance with the tempest stuff? I think my work is overlapping with that. | 18:43 |
| *** yufei has joined #openstack-ironic | 18:43 | |
| JayF | https://review.openstack.org/#/c/407236/ needs a review -- some updates to the rescue spec to match up with some stuff we found implementing it | 18:47 |
| patchbot | patch 407236 - ironic-specs - Update rescue spec | 18:47 |
| *** Nakato has quit IRC | 18:48 | |
| *** Nakato has joined #openstack-ironic | 18:48 | |
| jroll | jlvillal: I'm just waiting for project-config to unfreeze, land that piece, and then the rest can go | 18:51 |
| jroll | jlvillal: where are you overlapping? feel free to just rebase on top of mine | 18:52 |
| *** chlong has joined #openstack-ironic | 18:57 | |
| jlvillal | jroll: Like this patch: https://review.openstack.org/406376 | 19:04 |
| patchbot | patch 406376 - ironic - Configure tempest for multitenancy/flat network | 19:04 |
| jlvillal | jroll: Do you want me to rebase it on top of yours? | 19:05 |
| zer0c00l | Is it possible to run multiple ironic conductors on the same host? | 19:05 |
| jroll | jlvillal: yes please, as soon as the project-config change lands (after the xenial stuff is done) my things should be landing | 19:05 |
| jroll | zer0c00l: yes, you'll need to set host= differently in config | 19:06 |
| jlvillal | jroll: Okay. Will do | 19:06 |
| zer0c00l | jroll: Is there some documentation on this? | 19:07 |
| jroll | zer0c00l: I don't think so, I wouldn't recommend it | 19:07 |
| *** [1]rpioso has joined #openstack-ironic | 19:08 | |
| jroll | zer0c00l: one of the scaling limitations for conductor hosts is tftp, and that would be shared between conductors on the same host | 19:08 |
| *** milan has joined #openstack-ironic | 19:08 | |
| *** rpioso has quit IRC | 19:10 | |
| mariojv | jroll: JayF: had a question with aNuposic about the ceph configdrive RFE | 19:10 |
| zer0c00l | jroll: i see. we have multi core processors | 19:10 |
| zer0c00l | conductor always end up using just one cpu core | 19:10 |
| mariojv | should he have tempest tests? it doesn't look like swift configdrive is tested in the gate: https://github.com/openstack/ironic/search?utf8=%E2%9C%93&q=configdrive_use_swift | 19:10 |
| zer0c00l | all other cores are almost always idle :( | 19:10 |
| mariojv | defaults to false, only has unit tests | 19:10 |
| zer0c00l | Also we have 2 conductors running on 2 seperate nodes handling more than 8k machines | 19:10 |
| jroll | zer0c00l: yay python | 19:10 |
| mariojv | unless we still have some out of tree tempest tests? | 19:10 |
| zer0c00l | power_sync_state kills the whole thing | 19:11 |
| jroll | zer0c00l: I'd run conductors in VMs, to be honest | 19:11 |
| zer0c00l | during sync_power_state the cpu usage shoots up for ironic-conductor to almost 100% | 19:11 |
| zer0c00l | i see | 19:11 |
| jroll | zer0c00l: but if you want to run some on the same hosts... set host= (which is the rabbit queue and such), and maybe the tftp and image cache locations | 19:12 |
| jroll | I'm not sure if the latter is necessary | 19:12 |
| zer0c00l | i see | 19:12 |
| zer0c00l | the conductor cannot use the same tftp path? | 19:13 |
| zer0c00l | i mean multiple conductors cannot use the same tftp path | 19:13 |
| zer0c00l | (? | 19:13 |
| *** hemna_ has joined #openstack-ironic | 19:13 | |
| JayF | mariojv: touche' | 19:14 |
| JayF | zer0c00l: jroll: You can't colocate them if you are using tftp, mainly because a tftp server can only have one path, and the conductors will "fight" over what configs go there | 19:15 |
| JayF | zer0c00l: jroll: It's not like with tftp you can set a different port or something | 19:15 |
| *** hemna_ has quit IRC | 19:15 | |
| zer0c00l | So we do use tftp to bootstrap ipxe | 19:15 |
| jroll | JayF: oh, true | 19:16 |
| JayF | yeah; I'm not 100% certain but I suspect that fighting over tftp configs is where all the shenanigans would lie | 19:16 |
| jroll | I think the tftp stuff is partitioned by node though | 19:16 |
| *** fragatin_ has quit IRC | 19:16 | |
| jroll | e.g. /tftproot/node-id/blah | 19:16 |
| JayF | jroll: okay; so conductor1 fails, nodes fail over to conductor2, conductor2 takes over | 19:16 |
| JayF | jroll: when conductor1 comes back and takes over, conductor2 won't ever try to remove the "old" config? | 19:16 |
| jroll | JayF: I'm honestly not sure | 19:17 |
| JayF | jroll: my concern is more that -- cleanup actions taken by one conductor could disrupt actual configs from another one | 19:17 |
| *** krtaylor has joined #openstack-ironic | 19:17 | |
| jroll | sure | 19:17 |
| JayF | zer0c00l: if you wanna be certain it'll work; use containers + distinct IPs per conductor | 19:17 |
| jroll | that's why I recommended separate, and wasn't sure if necessary | 19:17 |
| zer0c00l | i see | 19:18 |
| zer0c00l | So VMs/containers prevents conductors from stomping over each other | 19:18 |
| zer0c00l | Also if i have n conductors when sync_power_state runs does it run on all the nodes | 19:18 |
| jroll | no, just the nodes that conductor manages | 19:19 |
| jroll | roughly 1/n | 19:19 |
| JayF | no, the conductors consistently hash nodes between them | 19:19 |
| *** ijw has joined #openstack-ironic | 19:19 | |
| zer0c00l | okay | 19:19 |
| zer0c00l | So that helps. | 19:19 |
| * NobodyCam is back :) | 19:19 | |
| zer0c00l | Scaling the conductor out helps with sync_power_state problems | 19:20 |
| jroll | yes | 19:20 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Configure tempest for multitenancy/flat network https://review.openstack.org/406376 | 19:21 |
| *** e0ne has joined #openstack-ironic | 19:22 | |
| jlvillal | jroll: ^^ rebased on yours | 19:22 |
| *** vsaienko has joined #openstack-ironic | 19:22 | |
| jroll | jlvillal: cool, thanks | 19:22 |
| jlvillal | jroll: Did you figure out the issue you were seeing with the tempest patches? Was it the project-config change that fixed it? | 19:22 |
| *** rama_y has quit IRC | 19:22 | |
| * jlvillal vaguely remembers something from earlier this week | 19:23 | |
| rloo | mgould|afk: are you still around or gone for the day? | 19:23 |
| rloo | mgould|afk: I'll comment in your patch 404211 | 19:23 |
| patchbot | https://review.openstack.org/#/c/404211/ - ironic - Add release names & numbers to API version history | 19:23 |
| jroll | jlvillal: the project-config patch should fix it, yes. grenade wasn't loading the tempest plugin, so CONF.service_available.ironic wasn't being registered, therefore this hack wasn't working (which we intentionally left in tempest, line 813): https://review.openstack.org/#/c/355586/22/tempest/scenario/manager.py | 19:24 |
| patchbot | patch 355586 - tempest - Removing baremetal tests from tempest tree | 19:24 |
| jroll | I can't test the project-config change easily, but that was the cause and the patch should fix it | 19:24 |
| jlvillal | jroll: Thanks. Yeah it is hard to test those changes. | 19:25 |
| *** Sukhdev has joined #openstack-ironic | 19:25 | |
| jlvillal | jroll: I can only figure out how to generate the XML files from project-config. | 19:25 |
| jroll | jlvillal: zuul v3 will make us able to 1) put that in our tree, and 2) use depends-on to test it \o/ | 19:26 |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Correct offset of a capability bit https://review.openstack.org/408267 | 19:26 |
| *** e0ne has quit IRC | 19:26 | |
| jlvillal | jroll: That will be great :) | 19:27 |
| jlvillal | jroll: Any word on how much longer on the Xenial work? | 19:27 |
| jroll | jlvillal: they're chugging along in alphabetical order for `ls jenkins/jobs/`, I see they are in the p's now (started yesterday) | 19:28 |
| jroll | so likely friday or monday I hope | 19:28 |
| jlvillal | Sweet :) | 19:28 |
| *** fragatina has joined #openstack-ironic | 19:29 | |
| *** chlong has quit IRC | 19:36 | |
| *** hw_wutianwei has joined #openstack-ironic | 19:39 | |
| *** fragatina has quit IRC | 19:40 | |
| *** ChubYann has joined #openstack-ironic | 19:41 | |
| *** trown|lunch is now known as trown | 19:42 | |
| *** glonlas has quit IRC | 19:43 | |
| *** vsaienko has quit IRC | 19:51 | |
| *** jkilpatr_ has quit IRC | 19:51 | |
| *** jkilpatr has joined #openstack-ironic | 19:52 | |
| *** dhellmann_ has joined #openstack-ironic | 19:52 | |
| *** lindycoder has quit IRC | 19:54 | |
| *** fragatina has joined #openstack-ironic | 19:58 | |
| *** dhellmann_ is now known as dhellmann | 19:59 | |
| *** chlong has joined #openstack-ironic | 20:00 | |
| *** clenimar has quit IRC | 20:01 | |
| openstackgerrit | Stenio Araujo proposed openstack/ironic: Adds another validation step when using dynamic allocation https://review.openstack.org/377073 | 20:02 |
| *** glonlas has joined #openstack-ironic | 20:07 | |
| *** anush has quit IRC | 20:10 | |
| *** Nakato has quit IRC | 20:11 | |
| *** Nakato has joined #openstack-ironic | 20:12 | |
| openstackgerrit | Merged openstack/pyghmi: Correct offset of a capability bit https://review.openstack.org/408267 | 20:13 |
| *** glonlas_ has joined #openstack-ironic | 20:20 | |
| *** [1]rpioso has quit IRC | 20:21 | |
| *** glonlas has quit IRC | 20:23 | |
| *** anush has joined #openstack-ironic | 20:24 | |
| mariojv | JayF: question about rescue | 20:27 |
| *** Nakato has quit IRC | 20:28 | |
| mariojv | JayF: are we going to have to have separate tftp_* config options for rescue, if using a separate ramdisk? | 20:28 |
| mariojv | [pxe]/tftp_* to be slightly more specific | 20:28 |
| *** Nakato has joined #openstack-ironic | 20:29 | |
| mariojv | i suppose you'd really just need to change the pxe config, not all of that | 20:31 |
| mariojv | and have a separate labeled section here: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe_config.template | 20:31 |
| JayF | mariojv: you can't run multiple tftp servers on a given server | 20:31 |
| mariojv | yeah | 20:31 |
| *** [1]rpioso has joined #openstack-ironic | 20:32 | |
| *** lindycoder has joined #openstack-ironic | 20:33 | |
| JayF | mariojv: so you got it now? I think that's probalby right | 20:33 |
| *** hw_wutianwei has quit IRC | 20:33 | |
| mariojv | yeah | 20:33 |
| JayF | mariojv: I'm just not certain how ironic triggers what section of that config to boot | 20:33 |
| mariojv | mostly | 20:33 |
| mariojv | that's the bit i don't know but i'll find out | 20:33 |
| JayF | mariojv: if you wanna pair up on any of that just ask; I am about to jump into the client work and I don't expect it to take too long (well, I'll get the shell done, almost impossible to actually test until more rescue code is ready in ironic-proper) | 20:34 |
| mariojv | thanks, i think i'll try to get a little further | 20:34 |
| *** hw_wutianwei has joined #openstack-ironic | 20:35 | |
| mariojv | maybe tomorrow if i'm stuck, if you'll have some time before team sync | 20:35 |
| JayF | yeah; all I have tomorrow is standup and sync | 20:35 |
| *** aNuposic has quit IRC | 20:38 | |
| *** harlowja has quit IRC | 20:41 | |
| JayF | How are we supposed to handle nodes with states that don't exist in older microversions? | 20:50 |
| JayF | For instance, if a node is in RESCUE, which was added in the latest microversion, what state would a client running (latest-1) see? | 20:51 |
| *** glonlas_ has quit IRC | 20:52 | |
| openstackgerrit | Xavier proposed openstack/ironic: Shutdown server before change boot order for agent drivers https://review.openstack.org/408298 | 20:53 |
| *** glonlas has joined #openstack-ironic | 20:53 | |
| *** Nakato has quit IRC | 20:55 | |
| *** Nakato has joined #openstack-ironic | 20:56 | |
| *** Sukhdev has quit IRC | 20:57 | |
| rloo | jlvillal: did you see the comment in https://review.openstack.org/#/c/407729? do you want to change the name of the mock in your patch or in the other one? | 20:59 |
| patchbot | patch 407729 - ironic-python-agent - Update mock variable name in unit tests | 20:59 |
| mrda | Morning Ironic | 21:00 |
| JayF | o/ | 21:00 |
| jroll | JayF: we've always just exposed those states to all microversions | 21:00 |
| *** anush has quit IRC | 21:00 | |
| jroll | the client gets it as a string and just displays it | 21:00 |
| JayF | That's what I thought | 21:00 |
| jroll | NONE -> AVAILABLE was the exception | 21:01 |
| jroll | but e.g. adopted will show as adopted in version 1.1 | 21:01 |
| jroll | or 1.2 or whatever the first was | 21:01 |
| *** Goneri has quit IRC | 21:01 | |
| *** Sukhdev has joined #openstack-ironic | 21:07 | |
| *** jcoufal has quit IRC | 21:08 | |
| jlvillal | rloo: I did. I'll rebase my patch. | 21:08 |
| jlvillal | rloo: Not sure which is a better name... | 21:09 |
| rloo | jlvillal: ok. | 21:09 |
| rloo | jlvillal: mock_smaker :) | 21:09 |
| jlvillal | heh | 21:09 |
| rloo | jlvillal: mock_make_server makes more sense than server_maker, since the method is make_server. | 21:10 |
| jlvillal | rloo: Sounds good. I'll go with that third option then. | 21:11 |
| rloo | jlvillal: and bonus points if you change all mocked to mock | 21:11 |
| jlvillal | rloo: Doing that now | 21:11 |
| rloo | thx! | 21:11 |
| jlvillal | rloo: So I'm pondering if I want to do it in all unit test files. s/mocked_/mock_/g | 21:14 |
| *** Goneri has joined #openstack-ironic | 21:14 | |
| rloo | jlvillal: please. just this one file for this patch. | 21:14 |
| jlvillal | Whew :) | 21:15 |
| rloo | jlvillal: only cuz i don't know that i want to review so many files. although i guess it is easy. go for it if you want! | 21:15 |
| JayF | jcook: mariojv: got a sec to talk rescue api? I think I found an issue in the patch that's up for API but not certain | 21:15 |
| rloo | if the changes don't conflict with too many other patches... | 21:15 |
| jcook | JayF k | 21:16 |
| JayF | jcook: I jumped in my vidyo room real quick | 21:17 |
| mariojv | trying to finish something | 21:17 |
| mariojv | i'll be around tomorrow morning if you still need help then | 21:17 |
| JayF | it's fine, I was fishing for both of you but one is fine :P | 21:18 |
| *** Nakato has quit IRC | 21:20 | |
| *** Nakato has joined #openstack-ironic | 21:20 | |
| *** e0ne has joined #openstack-ironic | 21:22 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/408091 | 21:22 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Update mock variable name in unit tests https://review.openstack.org/407729 | 21:24 |
| jlvillal | rloo: ^^^ | 21:24 |
| rloo | jlvillal: thx, will wait for jenkins to smile first | 21:25 |
| * jlvillal feels full of confidence since he did 'tox -vvv' with success before pushing. | 21:26 | |
| *** e0ne has quit IRC | 21:29 | |
| *** srobert has quit IRC | 21:33 | |
| rloo | jlvillal: you want to update the commit msg before I +A https://review.openstack.org/#/c/407729 | 21:35 |
| patchbot | patch 407729 - ironic-python-agent - Update mock variable name in unit tests | 21:35 |
| jlvillal | rloo: Looking | 21:37 |
| *** aNuposic has joined #openstack-ironic | 21:37 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Update mock variable name in unit tests https://review.openstack.org/407729 | 21:38 |
| jlvillal | rloo: Commit message updated. Thanks. | 21:38 |
| rloo | thx jlvillal. TheJulia, if you have a minute to put your +2 again on https://review.openstack.org/407729 | 21:39 |
| patchbot | patch 407729 - ironic-python-agent - Update mock variable name in unit tests | 21:39 |
| TheJulia | Done | 21:39 |
| * TheJulia wonders if she should be tested for some form of dyslexia | 21:40 | |
| rloo | You're OK TheJulia :) | 21:40 |
| TheJulia | Wait, I'm not crazy? *shows signs of hope!* | 21:41 |
| *** harlowja has joined #openstack-ironic | 21:41 | |
| *** jkilpatr has quit IRC | 21:41 | |
| rloo | TheJulia: I don't think you are. Does it take one to recognize another one? :) | 21:42 |
| TheJulia | I don't know *looks around shifty eyed* | 21:43 |
| *** milan has quit IRC | 21:43 | |
| rloo | TheJulia: heh | 21:44 |
| openstackgerrit | Merged openstack/ironic-python-agent: Update mock variable name in unit tests https://review.openstack.org/407729 | 21:45 |
| *** RomanenkoK has joined #openstack-ironic | 21:46 | |
| *** glonlas_ has joined #openstack-ironic | 21:46 | |
| *** glonlas_ has quit IRC | 21:47 | |
| *** glonlas_ has joined #openstack-ironic | 21:47 | |
| *** abalfour has left #openstack-ironic | 21:48 | |
| *** glonlas has quit IRC | 21:50 | |
| *** aNuposic has quit IRC | 21:52 | |
| *** aNuposic has joined #openstack-ironic | 21:52 | |
| *** aNuposic has quit IRC | 21:55 | |
| *** trown is now known as trown|outtypewww | 21:58 | |
| *** krtaylor has quit IRC | 21:58 | |
| *** aNuposic has joined #openstack-ironic | 22:00 | |
| *** bfournie has quit IRC | 22:01 | |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic: WIP/DNM: Trying to test grenade multi-tenant experimental job https://review.openstack.org/403895 | 22:04 |
| openstackgerrit | Merged openstack/ironic: Remove the VALID_ROOT_DEVICE_HINTS list https://review.openstack.org/408045 | 22:06 |
| JayF | sambetts|afk: jfyi, in the future, if you set importance on a bug, set the status to Triaged (not Confirmed) to keep it from showing up in the queue for bug triage :) | 22:11 |
| *** lindycoder has quit IRC | 22:11 | |
| jroll | confirmed shows up in triage? | 22:11 |
| jroll | that seems... wrong | 22:11 |
| jroll | new->triaged->confirmed is always the workflow as I've thought about it | 22:12 |
| JayF | no, the workflow as LP determines it is more | 22:12 |
| JayF | new (1 person) -> confirmed (addl person) -> triaged (bug supervisor) | 22:12 |
| jroll | ah, I misunderstood | 22:12 |
| JayF | since we do bug supervisor by committee, the best case is for cores to file bugs w/importance set and status:triaged | 22:12 |
| *** baoli has quit IRC | 22:12 | |
| JayF | folks outside hte project can do new and confirm new bugs | 22:12 |
| JayF | but all that is imaginary since there's no acls | 22:13 |
| JayF | and anyone can join the team and wreck havoc on our bug settings :P | 22:13 |
| *** baoli has joined #openstack-ironic | 22:13 | |
| *** RomanenkoK has quit IRC | 22:16 | |
| *** baoli has quit IRC | 22:18 | |
| openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Documentation for Security Groups for baremetal servers https://review.openstack.org/393962 | 22:19 |
| *** jheroux has quit IRC | 22:23 | |
| *** krtaylor has joined #openstack-ironic | 22:42 | |
| *** absubram has quit IRC | 22:43 | |
| *** bfournie has joined #openstack-ironic | 22:44 | |
| openstackgerrit | Merged openstack/ironic: Fix double dots at the end of a message to single dot https://review.openstack.org/407281 | 22:45 |
| *** rcernin has quit IRC | 22:48 | |
| *** Goneri has quit IRC | 22:53 | |
| openstackgerrit | Merged openstack/ironic: Use IronicObject._from_db_object_list method https://review.openstack.org/407616 | 22:54 |
| openstackgerrit | Merged openstack/ironic: Add release names & numbers to API version history https://review.openstack.org/404211 | 22:54 |
| openstackgerrit | Jay Faulkner proposed openstack/python-ironicclient: Add support for RESCUE and UNRESCUE provision states https://review.openstack.org/408341 | 22:54 |
| *** hw_wutianwei has quit IRC | 22:55 | |
| *** chlong has quit IRC | 23:06 | |
| *** glonlas_ has quit IRC | 23:09 | |
| *** cdearborn has quit IRC | 23:15 | |
| *** harlowja has quit IRC | 23:16 | |
| *** sacharya has quit IRC | 23:23 | |
| *** sacharya has joined #openstack-ironic | 23:24 | |
| *** rama_y has joined #openstack-ironic | 23:26 | |
| *** rbudden has quit IRC | 23:35 | |
| * mrda waits for jlvillal to ask his question :) | 23:36 | |
| jlvillal | Am I correct that with the multi-tenant support that bare-metal nodes can be attached to the same network that VMs are attached to? My feeling (98% sure) is yes on this. But I wanted to double check before I tell people that. | 23:37 |
| jlvillal | mrda: :P | 23:37 |
| *** trown|outtypewww has quit IRC | 23:40 | |
| JayF | jlvillal: absolutely! | 23:41 |
| JayF | jlvillal: otherwsie we are doin' it wrong | 23:41 |
| JayF | jlvillal: neutron networks are neutron networks, and as long as the deployer set them up to be accessible via vm or bare metal that can work on both | 23:41 |
| jlvillal | JayF: Thanks that is what I thought. But I had someone say, we want to have Ironic support that feature. And I'm thinking, it already does unless I'm confused. | 23:41 |
| jlvillal | JayF: Thanks! | 23:41 |
| JayF | jlvillal: at Rackspace, in our (admittedly very-divergent-from-upstream) network implementation, we support that already | 23:41 |
| JayF | jlvillal: I find that happening a lot; people assume we don't do things that we do ;) | 23:42 |
| jlvillal | heh | 23:42 |
| *** trown has joined #openstack-ironic | 23:42 | |
| NobodyCam | anyone ever have to mount a iso manually via Dirac v8 (ssh) | 23:42 |
| NobodyCam | the command I am using is: racism remoteimage -u user@domain.local -p demo -c -l //ip.ip.ip.ip:8080/CentOS-7-x86_64-Minimal-1511.iso | 23:43 |
| mrda | yay, I was right \o/ | 23:44 |
| jlvillal | racism? That's an odd name for a command. | 23:44 |
| NobodyCam | hahha | 23:44 |
| * jlvillal thinks those dell people should change their command names. | 23:44 | |
| mrda | lol | 23:44 |
| NobodyCam | its the auto correct on my mac | 23:44 |
| jlvillal | Ah | 23:44 |
| mrda | NobodyCam: your mac is racist? | 23:44 |
| NobodyCam | racadm | 23:45 |
| NobodyCam | lol | 23:45 |
| jlvillal | NobodyCam: No idea. I don't have any Dell hardware. Sorry :( | 23:45 |
| NobodyCam | just bought my mac a new word | 23:45 |
| NobodyCam | lol | 23:45 |
| mrda | NobodyCam: don't you normally buy a vowel? | 23:45 |
| NobodyCam | jlvillal: the hardware is in your lab | 23:45 |
| mrda | I'm here until Thursday, try the veal! | 23:46 |
| *** ijw has quit IRC | 23:46 | |
| NobodyCam | hehehe | 23:46 |
| NobodyCam | it's been one of those days | 23:46 |
| jlvillal | NobodyCam: Well there is some here. But not any that I control... | 23:46 |
| *** rbudden has joined #openstack-ironic | 23:46 | |
| *** ijw has joined #openstack-ironic | 23:46 | |
| NobodyCam | :) | 23:46 |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic: WIP/DNM: Trying to test grenade multi-tenant experimental job https://review.openstack.org/403895 | 23:48 |
| JayF | aNuposic: you might wanna read scrollback from me+mario this morning, he pointed out we don't test swift+configdrive in the gate so you're probably fine w/o it for ceph+configdrive | 23:51 |
| JayF | aNuposic: It's not ideal, but it is what it is. And saves you a lot of pain tracking down howtf to gate against ceph :) | 23:51 |
| aNuposic | JayF: Yes I was discussing that time with mariojv but continued to investigate more on how to run ceph in gates | 23:52 |
| JayF | aNuposic: yeah, it's your choice if you wanna continue pointing your brain in that direction... but I don't want you to think I'm -1 if you don't have that working :) | 23:53 |
| jlvillal | aNuposic: I think you are on trail to pain and heartache in trying to get ceph running in the gate ;) | 23:53 |
| aNuposic | JayF: good to know that no need | 23:53 |
| JayF | jlvillal: on the flipside, if aNuposic figured it out and documented it, it's like pain relief for the next person :P | 23:54 |
| jlvillal | True! :) | 23:54 |
| aNuposic | JayF: I had not got confirmed "Don't need to do" from you or others so continued to dig | 23:54 |
| JayF | aNuposic: there is rarely ever a confirmed anything with upstream :) | 23:54 |
| aNuposic | JayF: ha ha okay. Good to know. Yeah otherwise it looked like a pain | 23:54 |
| *** rbudden has quit IRC | 23:54 | |
| jlvillal | aNuposic: JayF: But the thing is we are trying to shrink our job count. As there is some unspoken limit on gate jobs. So adding a whole new gate job might also be a problem. | 23:55 |
| JayF | aNuposic: I'd still suggest putting a comment in the RFE summarizing your research into it, with a focus on us not testing it for swift :) | 23:55 |
| jlvillal | Unspoken as in it has never been spoken to me :) | 23:55 |
| JayF | jlvillal: Yeah, I suspect for pike we'll have to make improved testing matrixes a priority | 23:55 |
| JayF | where we can test >1 thing in each job | 23:55 |
| JayF | like it seems a little nuts to me we don't have a single job testing configdrive in swift | 23:55 |
| jlvillal | JayF: I don't think we have a single drive that has Swift at all. | 23:56 |
| jlvillal | aNuposic: JayF: Though you do realize that the goal is to eliminate the configdrive. Not add it to being stored in Swift. | 23:56 |
| jlvillal | At least my RFE got rejected because the goal was to remove the storing of configdrives. | 23:57 |
| jlvillal | Storing in Ironic that is. | 23:57 |
| jlvillal | And not add it to Ceph either. | 23:57 |
| * jlvillal is not sure what is needed to accomplish that. | 23:57 | |
| aNuposic | JayF: Yes of course. I will add my findings. :) | 23:58 |
| *** ElCoyote_ has joined #openstack-ironic | 23:58 | |
| aNuposic | jlvillal: is it! i was not knowing that | 23:58 |
| jlvillal | aNuposic: It is. Let me find my RFE | 23:58 |
| aNuposic | jlvillal: why to remove configdrive support at all? | 23:58 |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic: WIP/DNM: Trying to test grenade multi-tenant experimental job https://review.openstack.org/403895 | 23:58 |
| JayF | aNuposic: not to remove configdrive support; to stop storing it in ironic | 23:59 |
| jlvillal | aNuposic: Because of painful issues like trying to store it | 23:59 |
| JayF | aNuposic: basically we store it in ironic primarily b/c we thought it was needed for rebuild, but in nova, it rebuilds a new configdrive if you do a rebuild | 23:59 |
| JayF | aNuposic: so by preserving+reusing the configdrive we're actually breaking the api | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!