| *** baoli has joined #openstack-ironic | 00:22 | |
| *** baoli has quit IRC | 00:23 | |
| *** baoli has joined #openstack-ironic | 00:23 | |
| *** fragatina has joined #openstack-ironic | 00:35 | |
| *** baoli has quit IRC | 00:38 | |
| *** hoangcx has joined #openstack-ironic | 00:42 | |
| *** hw_wutianwei has joined #openstack-ironic | 00:43 | |
| *** jkilpatr has quit IRC | 00:45 | |
| *** SerenaFeng has joined #openstack-ironic | 00:55 | |
| *** phuongnh has joined #openstack-ironic | 01:04 | |
| *** amoralej|off has quit IRC | 01:19 | |
| *** amoralej has joined #openstack-ironic | 01:22 | |
| *** zhangjl has joined #openstack-ironic | 01:29 | |
| *** crushil has joined #openstack-ironic | 01:30 | |
| *** tuanluong has joined #openstack-ironic | 01:41 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic master: Add supported_power_states field to node APIs https://review.openstack.org/382194 | 01:49 |
|---|---|---|
| openstackgerrit | Naohiro Tamura proposed openstack/ironic master: Add supported_power_states field to api-ref https://review.openstack.org/427000 | 01:50 |
| *** pmannidi has quit IRC | 01:50 | |
| *** pmannidi has joined #openstack-ironic | 01:59 | |
| *** baoli has joined #openstack-ironic | 02:04 | |
| *** SerenaFeng has quit IRC | 02:05 | |
| *** SerenaFeng has joined #openstack-ironic | 02:24 | |
| *** baoli has quit IRC | 02:33 | |
| *** sdake has joined #openstack-ironic | 02:36 | |
| *** sdake_ has joined #openstack-ironic | 02:40 | |
| *** sdake has quit IRC | 02:43 | |
| *** SerenaFeng has quit IRC | 02:49 | |
| *** SerenaFeng has joined #openstack-ironic | 02:51 | |
| *** SerenaFeng has quit IRC | 02:54 | |
| *** SerenaFe_ has joined #openstack-ironic | 02:54 | |
| *** crushil has quit IRC | 02:55 | |
| openstackgerrit | liusheng proposed openstack/ironic master: Don't need to define versionedobjects classes repeatedly https://review.openstack.org/432819 | 02:56 |
| *** tovin07_ has joined #openstack-ironic | 02:57 | |
| *** stendulker has joined #openstack-ironic | 02:58 | |
| openstackgerrit | liusheng proposed openstack/ironic master: Don't need to define versionedobjects classes repeatedly https://review.openstack.org/432819 | 03:12 |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic-specs master: Support additional capabilities discovery for iRMC driver https://review.openstack.org/409044 | 03:14 |
| *** sdake has joined #openstack-ironic | 03:17 | |
| *** sdake_ has quit IRC | 03:18 | |
| *** crushil has joined #openstack-ironic | 03:31 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 03:39 | |
| *** crushil has quit IRC | 03:43 | |
| *** sdake has quit IRC | 03:49 | |
| *** SerenaFe_ has quit IRC | 03:51 | |
| *** MattMan_1 has quit IRC | 03:53 | |
| *** MattMan_1 has joined #openstack-ironic | 03:53 | |
| *** aparnav has joined #openstack-ironic | 04:05 | |
| *** ppiela has quit IRC | 04:26 | |
| *** openstackgerrit has quit IRC | 04:32 | |
| *** hoonetorg has quit IRC | 04:34 | |
| *** SerenaFeng has joined #openstack-ironic | 04:38 | |
| *** SerenaFeng has quit IRC | 04:46 | |
| *** faizy has joined #openstack-ironic | 04:46 | |
| *** hoonetorg has joined #openstack-ironic | 04:47 | |
| *** openstackgerrit has joined #openstack-ironic | 04:52 | |
| openstackgerrit | liusheng proposed openstack/ironic master: Don't need to define versionedobjects classes repeatedly https://review.openstack.org/432819 | 04:52 |
| *** udesale has joined #openstack-ironic | 05:01 | |
| *** crushil has joined #openstack-ironic | 05:12 | |
| *** mtreinish has quit IRC | 05:20 | |
| *** jaosorior has joined #openstack-ironic | 05:28 | |
| *** mtreinish has joined #openstack-ironic | 05:31 | |
| *** amotoki has joined #openstack-ironic | 05:33 | |
| *** Nisha_ has joined #openstack-ironic | 05:34 | |
| *** amotoki has quit IRC | 05:36 | |
| *** Nisha_Agarwal has quit IRC | 05:37 | |
| *** zhangjl1 has joined #openstack-ironic | 05:45 | |
| *** SerenaFeng has joined #openstack-ironic | 05:46 | |
| *** amotoki has joined #openstack-ironic | 05:48 | |
| *** zhangjl has quit IRC | 05:48 | |
| *** moshele has joined #openstack-ironic | 05:49 | |
| *** crushil has quit IRC | 05:55 | |
| *** zhangjl1 has quit IRC | 05:59 | |
| *** zhangjl has joined #openstack-ironic | 06:00 | |
| *** gcb has joined #openstack-ironic | 06:00 | |
| *** jrist has joined #openstack-ironic | 06:04 | |
| *** Nisha_ has quit IRC | 06:08 | |
| *** Nisha_ has joined #openstack-ironic | 06:09 | |
| *** amotoki has quit IRC | 06:10 | |
| *** Nisha_ has quit IRC | 06:17 | |
| *** Nisha_ has joined #openstack-ironic | 06:17 | |
| *** amotoki has joined #openstack-ironic | 06:23 | |
| *** jaganathan has joined #openstack-ironic | 06:25 | |
| *** faizy has quit IRC | 06:31 | |
| *** faizy has joined #openstack-ironic | 06:31 | |
| *** faizy has quit IRC | 06:35 | |
| *** faizy has joined #openstack-ironic | 06:36 | |
| *** faizy has quit IRC | 06:37 | |
| *** faizy has joined #openstack-ironic | 06:38 | |
| *** moshele has quit IRC | 06:39 | |
| *** faizy has quit IRC | 06:41 | |
| *** faizy has joined #openstack-ironic | 06:41 | |
| *** fragatina has quit IRC | 06:58 | |
| *** nmathew has joined #openstack-ironic | 07:02 | |
| *** Nisha_ has quit IRC | 07:13 | |
| *** rcernin has joined #openstack-ironic | 07:15 | |
| *** tesseract has joined #openstack-ironic | 07:28 | |
| *** tesseract is now known as tesseract-RH | 07:29 | |
| *** ijw has joined #openstack-ironic | 07:38 | |
| *** fragatina has joined #openstack-ironic | 07:40 | |
| *** ralonsoh has joined #openstack-ironic | 07:46 | |
| *** pmannidi has quit IRC | 07:47 | |
| *** fragatina has quit IRC | 07:48 | |
| *** pcaruana has joined #openstack-ironic | 07:49 | |
| *** gcb has quit IRC | 07:57 | |
| *** zhangjl has left #openstack-ironic | 08:00 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests https://review.openstack.org/423556 | 08:04 |
| *** hwoarang has quit IRC | 08:16 | |
| *** faizy has quit IRC | 08:19 | |
| *** faizy has joined #openstack-ironic | 08:20 | |
| *** mgoddard has joined #openstack-ironic | 08:35 | |
| *** amarao has joined #openstack-ironic | 08:35 | |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic master: Add additional capabilities discovery for iRMC driver https://review.openstack.org/417347 | 08:36 |
| *** ijw has quit IRC | 08:48 | |
| *** kromanenko has joined #openstack-ironic | 08:59 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-ironic | 09:00 | |
| *** faizy has quit IRC | 09:01 | |
| *** faizy has joined #openstack-ironic | 09:02 | |
| *** faizy_ has joined #openstack-ironic | 09:11 | |
| *** dtantsur|afk is now known as dtantsur | 09:15 | |
| *** faizy has quit IRC | 09:15 | |
| dtantsur | morning ironic! | 09:15 |
| *** racedo has joined #openstack-ironic | 09:18 | |
| *** faizy_ has quit IRC | 09:24 | |
| *** udesale__ has joined #openstack-ironic | 09:26 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 09:27 | |
| *** udesale has quit IRC | 09:29 | |
| *** derekh has joined #openstack-ironic | 09:32 | |
| dtantsur | jroll, I don't remember if I pinged you already about it, but could you check https://review.openstack.org/#/c/430366/ please? | 09:32 |
| patchbot | patch 430366 - python-ironicclient (stable/ocata) - Bump the LAST_KNOWN_API_VERSION to 29 in OSC | 09:32 |
| *** lucas-afk is now known as lucasagomes | 09:36 | |
| lucasagomes | morning all | 09:36 |
| dtantsur | hey lucasagomes | 09:39 |
| lucasagomes | o/ | 09:39 |
| openstackgerrit | Béla Vancsics proposed openstack/ironic master: Use more specific asserts in tests https://review.openstack.org/432941 | 09:40 |
| *** athomas has joined #openstack-ironic | 09:40 | |
| Nisha_Agarwal | hey dtantsur, lucasagomes morning | 09:40 |
| Nisha_Agarwal | :) | 09:41 |
| *** faizy has joined #openstack-ironic | 09:42 | |
| dtantsur | morning/afternoon Nisha_Agarwal | 09:43 |
| *** faizy_ has joined #openstack-ironic | 09:43 | |
| Nisha_Agarwal | here afternoon :) | 09:43 |
| Nisha_Agarwal | dtantsur, ironic released on Friday? | 09:44 |
| dtantsur | Nisha_Agarwal, nope, still pending | 09:44 |
| Nisha_Agarwal | oh | 09:44 |
| Nisha_Agarwal | dtantsur, i thought it would have been done | 09:45 |
| Nisha_Agarwal | :) | 09:45 |
| dtantsur | we don't tend to release on Fridays | 09:45 |
| *** mgoddard has quit IRC | 09:46 | |
| Nisha_Agarwal | dtantsur, so it would be released by today? or by tomorrow? | 09:47 |
| *** faizy has quit IRC | 09:47 | |
| dtantsur | Nisha_Agarwal, I need to sync with jroll, but I think we have to release by tomorrow | 09:47 |
| *** hwoarang has joined #openstack-ironic | 09:47 | |
| Nisha_Agarwal | :) | 09:47 |
| Nisha_Agarwal | dtantsur, great | 09:47 |
| *** e0ne has joined #openstack-ironic | 09:52 | |
| *** e0ne has quit IRC | 09:54 | |
| *** e0ne has joined #openstack-ironic | 09:58 | |
| *** amotoki has quit IRC | 09:59 | |
| *** amotoki has joined #openstack-ironic | 10:01 | |
| *** Nisha_Agarwal has quit IRC | 10:02 | |
| *** e0ne has quit IRC | 10:02 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 10:03 | |
| *** faizy_ has quit IRC | 10:06 | |
| *** e0ne has joined #openstack-ironic | 10:07 | |
| openstackgerrit | Merged openstack/ironic master: Minor updates to multi-tenancy documentation https://review.openstack.org/426022 | 10:08 |
| *** e0ne has quit IRC | 10:09 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic master: Add ironic portgroup CRUD notifications https://review.openstack.org/427281 | 10:12 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: iLO drivers fails to do basic power operations https://review.openstack.org/432958 | 10:13 |
| *** e0ne has joined #openstack-ironic | 10:13 | |
| *** e0ne_ has joined #openstack-ironic | 10:15 | |
| *** e0ne has quit IRC | 10:15 | |
| *** dims_ has quit IRC | 10:16 | |
| vdrok | morning ironic, dtantsur lucasagomes and Nisha_Agarwal | 10:17 |
| lucasagomes | o/ | 10:17 |
| Nisha_Agarwal | vdrok, :) morning | 10:18 |
| *** sdake has joined #openstack-ironic | 10:18 | |
| *** e0ne_ has quit IRC | 10:21 | |
| pas-ha | morning all | 10:21 |
| *** udesale has joined #openstack-ironic | 10:22 | |
| *** mgoddard has joined #openstack-ironic | 10:23 | |
| *** dims has joined #openstack-ironic | 10:23 | |
| *** udesale__ has quit IRC | 10:23 | |
| dtantsur | morning vdrok, pas-ha | 10:25 |
| *** e0ne has joined #openstack-ironic | 10:25 | |
| pas-ha | morning dtantsur :) | 10:25 |
| openstackgerrit | Sofiia Andriichenko proposed openstack/ironic master: Add tempest plugin API tests for driver: https://review.openstack.org/408531 | 10:26 |
| *** owalsh-afk has joined #openstack-ironic | 10:26 | |
| *** owalsh-afk is now known as owalsh | 10:26 | |
| *** karthiks has joined #openstack-ironic | 10:27 | |
| *** e0ne has quit IRC | 10:27 | |
| *** owalsh_ has joined #openstack-ironic | 10:28 | |
| *** ralonsoh_ has joined #openstack-ironic | 10:30 | |
| *** owalsh__ has joined #openstack-ironic | 10:31 | |
| *** owalsh has quit IRC | 10:32 | |
| *** ralonsoh_ has quit IRC | 10:32 | |
| *** e0ne has joined #openstack-ironic | 10:32 | |
| *** ralonsoh_ has joined #openstack-ironic | 10:32 | |
| *** ralonsoh has quit IRC | 10:33 | |
| *** owalsh_ has quit IRC | 10:34 | |
| *** owalsh_ has joined #openstack-ironic | 10:35 | |
| *** owalsh__ has quit IRC | 10:36 | |
| tuanluong | morning ironic | 10:37 |
| *** owalsh__ has joined #openstack-ironic | 10:37 | |
| tuanluong | morning vdrok lucasagomes Nisha_Agarwal pas-ha dtantsur | 10:37 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Fix iLO drivers for SNMPv3 https://review.openstack.org/432958 | 10:38 |
| Nisha_Agarwal | tuanluong, o/ | 10:38 |
| Nisha_Agarwal | dtantsur, ^^^ above is a very small patch just to fix the latest ironic to work with old proliantutils which doesnt has SNMP enhancement | 10:39 |
| *** udesale has quit IRC | 10:39 | |
| *** owalsh__ is now known as owalsh | 10:39 | |
| *** owalsh_ has quit IRC | 10:40 | |
| *** e0ne has quit IRC | 10:41 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Adds wsgi to ironic-api in devstack https://review.openstack.org/430851 | 10:42 |
| *** e0ne has joined #openstack-ironic | 10:45 | |
| dtantsur | Nisha_Agarwal, none of my review comments were fixed apparently | 10:47 |
| Nisha_Agarwal | on which patch? | 10:47 |
| Nisha_Agarwal | the follow up patch? | 10:48 |
| dtantsur | Nisha_Agarwal, the patch you've just asked me to review | 10:48 |
| *** tuanluong has quit IRC | 10:49 | |
| dtantsur | I reviewed patchset 1, but patchset 2 was posted without fixing my comments, hence my surprise :) | 10:49 |
| Nisha_Agarwal | oh i didnt see that u reviewed already | 10:49 |
| Nisha_Agarwal | i am sorry | 10:49 |
| dtantsur | no worries, I just got confused at first :) | 10:49 |
| Nisha_Agarwal | i will just see and address them | 10:49 |
| dtantsur | Nisha_Agarwal, I'm actually not sure if it need a release note, as the original patch was just merged | 10:50 |
| dtantsur | but please fix the comments in the code | 10:50 |
| Nisha_Agarwal | dtantsur, sure | 10:51 |
| Nisha_Agarwal | it is just to make sure it works with old proliantutils | 10:51 |
| *** e0ne has quit IRC | 10:51 | |
| dtantsur | yeah | 10:52 |
| *** owalsh_ has joined #openstack-ironic | 10:55 | |
| *** phuongnh has quit IRC | 10:57 | |
| *** sambetts|afk is now known as sambetts | 10:58 | |
| *** owalsh has quit IRC | 10:58 | |
| *** ccamacho has quit IRC | 10:59 | |
| *** hoangcx has quit IRC | 11:00 | |
| *** ccamacho has joined #openstack-ironic | 11:02 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: [install-guide] add section on Glance+Swift config https://review.openstack.org/379422 | 11:02 |
| openstackgerrit | Merged openstack/ironic-python-agent master: imagebuild: tinyipa: Make the install-deps.sh SUSE aware https://review.openstack.org/428704 | 11:06 |
| openstackgerrit | Debayan Ray proposed openstack/proliantutils master: [WIP] Introducing Redfish API support https://review.openstack.org/406431 | 11:10 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Always install Ansible with pip https://review.openstack.org/431614 | 11:11 |
| *** owalsh has joined #openstack-ironic | 11:12 | |
| sambetts | Morning all | 11:14 |
| *** owalsh_ has quit IRC | 11:16 | |
| *** owalsh_ has joined #openstack-ironic | 11:16 | |
| *** stendulker has quit IRC | 11:17 | |
| *** owalsh__ has joined #openstack-ironic | 11:17 | |
| lucasagomes | sambetts, o/ | 11:17 |
| sambetts | hey lucasagomes u | 11:17 |
| *** openstackgerrit has quit IRC | 11:18 | |
| dtantsur | morning sambetts | 11:18 |
| *** openstackgerrit has joined #openstack-ironic | 11:18 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Walk over all objects when doing VIF detach https://review.openstack.org/432995 | 11:19 |
| dtantsur | sambetts, mind checking https://review.openstack.org/#/c/421285/ please? this is something we might want to get into the release.. | 11:19 |
| patchbot | patch 421285 - ironic-inspector - Find a node by multiple attributes | 11:19 |
| *** owalsh has quit IRC | 11:19 | |
| sambetts | dtantsur: I'll try to get to in a just a minute :) | 11:21 |
| *** owalsh_ has quit IRC | 11:21 | |
| dtantsur | thnx | 11:21 |
| * dtantsur tries to wrap up things before the release | 11:21 | |
| *** owalsh has joined #openstack-ironic | 11:21 | |
| *** owalsh__ has quit IRC | 11:22 | |
| *** mgould has joined #openstack-ironic | 11:23 | |
| vdrok | morning sambetts | 11:24 |
| *** Nisha_Agarwal has quit IRC | 11:25 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 11:27 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Walk over all objects when doing VIF detach https://review.openstack.org/432995 | 11:30 |
| *** owalsh_ has joined #openstack-ironic | 11:31 | |
| vsaienk0 | good morning dtantsur, lucasagomes, vdrok, sambetts, milan, vdrok and all Ironic'ers! | 11:31 |
| vdrok | morning vsaienk0 | 11:31 |
| *** owalsh__ has joined #openstack-ironic | 11:33 | |
| *** owalsh has quit IRC | 11:34 | |
| dtantsur | morning vsaienk0 | 11:34 |
| dtantsur | and morning jroll, I see you ;) | 11:34 |
| jroll | >.> | 11:34 |
| jroll | morning :) | 11:34 |
| *** e0ne has joined #openstack-ironic | 11:34 | |
| * jroll wonders how dtantsur doesn't get buried if he sees gerrit emails | 11:34 | |
| vdrok | morning jroll | 11:35 |
| jroll | dtantsur: +1 for release today | 11:35 |
| jroll | going to look at reno cleanup comments shortly | 11:35 |
| *** owalsh_ has quit IRC | 11:36 | |
| * jroll busy day today again | 11:36 | |
| sambetts | having my email client smoosh all review emails for a specifc patch into one conversation tree makes it much easier to parse, | 11:36 |
| dtantsur | jroll, I don't necessary read all them, just pick ones I'm waiting for feedback for | 11:37 |
| jroll | dtantsur: fair :P | 11:38 |
| dtantsur | folks, ironic-staging-drivers is not under our governance, right? | 11:38 |
| dtantsur | so it's up to lucasagomes to release them? | 11:38 |
| lucasagomes | dtantsur, yeah it is | 11:39 |
| dtantsur | ok, please do :) | 11:39 |
| dtantsur | lucasagomes, do we have stable branches for it? | 11:39 |
| lucasagomes | we probably should talk whether have it under our gov or not, but for now I'm doing the releases (I think you can do too and so can vdrok) | 11:39 |
| lucasagomes | dtantsur, not atm :-/ | 11:39 |
| des_consolado | hey I'm just looking at the python-ironicclient code. There's a lot of this self.app.client_manager stuff, what is this? | 11:39 |
| dtantsur | lucasagomes, I think we need to start having stable branches, because ironic internal API is not stable | 11:40 |
| vdrok | dtantsur: lucasagomes we now sync the requiremetns, so yeah, I'm all for stable branches ;) | 11:40 |
| dtantsur | des_consolado, this is provided by OSC (python-openstackclient) to fetch various clients (ironic, nova, ...) | 11:40 |
| lucasagomes | des_consolado, I think this is related to the OSC arch | 11:40 |
| lucasagomes | yeah like dtantsur said | 11:40 |
| des_consolado | oh right, thanks | 11:40 |
| *** nmathew has quit IRC | 11:40 | |
| lucasagomes | dtantsur, vdrok agreed | 11:41 |
| lucasagomes | I'll release it today and then we can go from there | 11:41 |
| dtantsur | ++ thanks | 11:41 |
| *** owalsh__ is now known as owalsh | 11:41 | |
| dtantsur | now, is someone speaking native English (quick glance at sambetts) going to go over ironic-inspector release notes? | 11:41 |
| dtantsur | ditto for IPA actually? | 11:41 |
| sambetts | dtantsur: I can take a read through them today | 11:43 |
| dtantsur | sambetts, thanks! | 11:43 |
| *** e0ne has quit IRC | 11:45 | |
| *** e0ne has joined #openstack-ironic | 11:47 | |
| *** e0ne has quit IRC | 11:48 | |
| *** e0ne has joined #openstack-ironic | 11:48 | |
| *** e0ne has quit IRC | 11:48 | |
| *** e0ne has joined #openstack-ironic | 11:49 | |
| *** e0ne has quit IRC | 11:49 | |
| *** e0ne has joined #openstack-ironic | 11:50 | |
| *** e0ne has quit IRC | 11:50 | |
| *** e0ne has joined #openstack-ironic | 11:50 | |
| *** e0ne has quit IRC | 11:51 | |
| *** e0ne has joined #openstack-ironic | 11:51 | |
| *** jkilpatr has joined #openstack-ironic | 11:51 | |
| *** e0ne has quit IRC | 11:52 | |
| *** ralonsoh__ has joined #openstack-ironic | 11:52 | |
| des_consolado | what's the difference betwen | 11:54 |
| des_consolado | oops... what's the difference between a chassis and a node | 11:55 |
| *** ralonsoh_ has quit IRC | 11:55 | |
| des_consolado | a chassis is a collection of nodes? | 11:57 |
| mgould | des_consolado: yes. https://www.techopedia.com/definition/2167/server-chassis | 12:00 |
| mgould | a physical collection in the same metal box rather than a logical connection, though :-) | 12:01 |
| mgould | *collection | 12:02 |
| *** ccamacho has quit IRC | 12:02 | |
| *** rbudden has joined #openstack-ironic | 12:02 | |
| mgould | dtantsur: sure. Is there a review open, or should I just look at the reno directory? | 12:03 |
| *** ccamacho has joined #openstack-ironic | 12:03 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Clean up release notes for 7.0.0 https://review.openstack.org/431188 | 12:06 |
| jroll | TheJulia: soliosg ^ updated | 12:06 |
| jroll | mgould: you'd want to read through the first (unreleased) section in http://docs.openstack.org/releasenotes/ironic-python-agent/current-series.html | 12:07 |
| mgould | jroll: will do, thanks | 12:11 |
| *** milan has joined #openstack-ironic | 12:13 | |
| sambetts | dtantsur: is this: http://docs.openstack.org/releasenotes/ironic-inspector/unreleased.html the best way to read through the release notes | 12:14 |
| sambetts | ? | 12:14 |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: Now it also displays information like name, uuid , maintenance and maintenance_reason of the node when we executes "ironic node-show-states <node-name/node-uuid>". When we are provisioning a baremetal we are checking the state of the node with the comman https://review.openstack.org/433029 | 12:14 |
| dtantsur | sambetts, yep | 12:14 |
| sambetts | dtantsur: cool :D | 12:15 |
| *** dims has quit IRC | 12:23 | |
| *** vsaienko has joined #openstack-ironic | 12:26 | |
| *** owalsh has quit IRC | 12:27 | |
| *** SerenaFeng has quit IRC | 12:27 | |
| des_consolado | https://github.com/openstack/python-ironicclient/blob/master/ironicclient/osc/v1/baremetal_node.py#L422 - so how do I know where this function is defined? | 12:28 |
| des_consolado | so I'm looking for self.app.client_manager.baremetal.node.create | 12:29 |
| mgould | des_consolado: yep | 12:31 |
| * mgould dimly remembers having a lot of trouble following the logic when he last looked at that stuff | 12:31 | |
| des_consolado | like... it's saying self.app... but self.app isn't defined anywhere in the damn file here... so where the hell are we... lol | 12:32 |
| mgould | the "osc" code is for the "openstack" CLI tool, which is meant to load plugins at runtime | 12:32 |
| des_consolado | so I have to go check out the python-openstackclient code and check for the baremetal stuff in there I guess? Will have a look | 12:32 |
| dtantsur | des_consolado, self.app.client_manager.baremetal is the very ironic client object we define in ironicclient | 12:33 |
| dtantsur | which you get by calling get_client | 12:33 |
| dtantsur | just like e.g. self.app.client_manager.orchestration will be heat client defined by heatclient (I guess) | 12:33 |
| openstackgerrit | Merged openstack/ironic master: Follow-up iRMC power driver for soft reboot/poff https://review.openstack.org/425593 | 12:33 |
| des_consolado | but what is just self.app and where is that defined | 12:33 |
| dtantsur | des_consolado, I think it's part of either cliff or OSC itself and is provided by OSC | 12:34 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Use SwiftAPI for temporary url generation in glance https://review.openstack.org/410764 | 12:34 |
| mgould | des_consolado: yeah, python-openstackclient basically just defines a plugin loader, IIRC | 12:34 |
| mgould | ironicclient/osc defines a plugin for openstackclient that loads the rest of ironicclient as a plugin | 12:35 |
| mgould | so you can type `openstack baremetal` instead of `ironic` | 12:35 |
| milan | morning Ironic! :) | 12:35 |
| mgould | morning milan | 12:35 |
| milan | morning mgould! :) | 12:36 |
| *** g3ek has quit IRC | 12:36 | |
| *** sdake_ has joined #openstack-ironic | 12:36 | |
| openstackgerrit | Béla Vancsics proposed openstack/ironic master: Use more specific asserts in tests https://review.openstack.org/432941 | 12:36 |
| dtantsur | des_consolado, I think this is self.app: https://github.com/openstack/python-openstackclient/blob/master/openstackclient/shell.py#L38 | 12:36 |
| milan | and morning des_consolado dtantsur sambetts and everyone! :) | 12:36 |
| vdrok | morning milan and mgould | 12:36 |
| dtantsur | morning milan :) | 12:36 |
| des_consolado | morning lol | 12:36 |
| milan | morning vdrok! :) | 12:36 |
| des_consolado | actually it's afternoon here now | 12:36 |
| des_consolado | ty dtantsur | 12:36 |
| milan | des_consolado, same here ;) | 12:37 |
| dtantsur | des_consolado, we use http://www.total-knowledge.com/~ilya/mips/ugt.html ;) | 12:37 |
| mgould | des_consolado: I think the method you're looking for is https://github.com/openstack/python-ironicclient/blob/a07cbc907481a735f4a7cc29440562c33db6c82e/ironicclient/common/base.py#L214 | 12:37 |
| * dtantsur also finds openstack clients architecture quite confusing | 12:38 | |
| * mgould too | 12:39 | |
| *** sdake has quit IRC | 12:39 | |
| *** owalsh has joined #openstack-ironic | 12:39 | |
| mgould | des_consolado: it's apparently possible to hook a DevStack VM up to PyCharm so you can follow the control-flow in a debugger: http://aakash01.github.io/openstack/2014/08/31/part-iii-debugging-devstack-pycharm | 12:42 |
| mgould | I think I just ended up adding lots of log statements by hand while I was trying to work out what was going on | 12:43 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient master: ETAGs for node in ironicclient https://review.openstack.org/400335 | 12:43 |
| dtantsur | mgould, O__o | 12:43 |
| dtantsur | jroll, I'm not sure if I asked or not. Do we need some x-project sessions with nova? If so, we may need to get a slot planned (either one of ours, or one of theirs, or one of x-project slots) | 12:44 |
| *** g3ek has joined #openstack-ironic | 12:45 | |
| openstackgerrit | Merged openstack/ironic master: Change misc to test_utils for tempest test https://review.openstack.org/431809 | 12:45 |
| jroll | dtantsur: yeah, I talked to matt a bit last week... they were going to schedule a certain time for placement things. other than that I think there's a couple topics that could maybe use x-project sessions but I'm not sure if it's needed | 12:45 |
| jroll | I wanted to talk about single region stuff but idk if I have enough info on that yet | 12:46 |
| dtantsur | jroll, well, we may recap the RAID topic, but as we haven't had much progress with it, it may be more like "hmm, let's finally do that" kind of thing | 12:46 |
| dtantsur | jroll, single region? | 12:46 |
| jroll | dtantsur: see wednesday afternoon at the bottom https://etherpad.openstack.org/p/nova-ptg-pike | 12:46 |
| jroll | dtantsur: yeah, ironic+kvm in one region without cells, it's kind of awkward :) | 12:46 |
| dtantsur | jroll, well, we did that with host aggregates for tripleo, so it's not impossible.. | 12:47 |
| dtantsur | but primarily because I don't know how to use cells :) | 12:47 |
| jroll | no, it isn't impossible, it just isn't a great user experience. think about quotas, knowing what actions one can take on some instance, etc | 12:47 |
| dtantsur | yeah, yeah | 12:48 |
| *** sdake_ is now known as sdake | 12:48 | |
| dtantsur | jroll, vdrok suggested "Splitting nodes into separate pools", this may fall under the same discussion | 12:48 |
| jroll | dtantsur: well, I think it's the opposite, really :) | 12:49 |
| jroll | whoa, our ptg etherpad lost all of its colors :( | 12:49 |
| dtantsur | dunno, I haven't put any thoughts in it, just saw it this morning | 12:49 |
| dtantsur | jroll, yep, a wrong button pressed by one of contributors (/me is not pointing fingers) | 12:50 |
| *** gcb has joined #openstack-ironic | 12:50 | |
| mgould | hey everyone, I'm confused by https://review.openstack.org/#/c/334816/23/releasenotes/notes/add_mellanox_hardware_manager-edfae87964737df1.yaml | 12:50 |
| patchbot | patch 334816 - ironic-python-agent - Add a new Hardware Manager for Mellanox NICs (MERGED) | 12:50 |
| vdrok | yup, that splitting of nodes does not fit that good in nova's model atm | 12:50 |
| mgould | it looks to me like the ReNo means "we calculate a MAC address and a client-ID according to RFC 4390" | 12:51 |
| *** Madasi has quit IRC | 12:51 | |
| mgould | but RFC 4390 actually says "the chaddr MUST be zeroed" | 12:51 |
| mgould | which is not what we're doing | 12:51 |
| mgould | have I misunderstood the reno? | 12:51 |
| *** lucasagomes is now known as lucas-hungry | 12:53 | |
| dtantsur | mgould, I haven't checked the RFE back then.. | 12:53 |
| mgould | dtantsur: https://tools.ietf.org/html/rfc4390#page-2 | 12:54 |
| dtantsur | huh | 12:55 |
| mgould | but that's kinda out of scope for what I'm doing now, which is "fix spelling and grammar in release notes" | 12:56 |
| dtantsur | mgould, let's describe what the code actually does then.. maybe we should drop a email to the author (I don't see him here) | 12:56 |
| mgould | cool, makes sense | 12:56 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: Fix some grammer and spelling issues in release notes https://review.openstack.org/433043 | 12:57 |
| sambetts | mgould: ^^ my WIP patch for it | 12:57 |
| mgould | sambetts: *grammar :-) | 12:58 |
| sambetts | mgould: ;) | 12:58 |
| mgould | sambetts: looks good | 12:59 |
| dtantsur | :D | 12:59 |
| * mgould is looking at the IPA renos | 12:59 | |
| sambetts | mgould: ok awesome :D | 12:59 |
| *** bfournie has joined #openstack-ironic | 13:00 | |
| * jroll bbiab | 13:01 | |
| *** rpioso has joined #openstack-ironic | 13:01 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient master: Add command RAID logical disk properties https://review.openstack.org/362047 | 13:02 |
| *** vsaienko has quit IRC | 13:03 | |
| *** aparnav has quit IRC | 13:04 | |
| TheJulia | Good morning | 13:06 |
| dtantsur | morning TheJulia | 13:06 |
| *** athomas has quit IRC | 13:07 | |
| *** athomas has joined #openstack-ironic | 13:09 | |
| mat128 | morning group | 13:10 |
| mat128 | congrats dtantsur | 13:10 |
| mgould | morning TheJulia | 13:11 |
| mgould | morning mat128 | 13:11 |
| *** vsaienko has joined #openstack-ironic | 13:11 | |
| dtantsur | morning mat128 :) | 13:12 |
| *** nmathew has joined #openstack-ironic | 13:12 | |
| *** nmathew- has joined #openstack-ironic | 13:14 | |
| *** dprince has joined #openstack-ironic | 13:15 | |
| *** trown|outtypewww is now known as trown | 13:16 | |
| *** nmathew has quit IRC | 13:18 | |
| openstackgerrit | Miles Gould proposed openstack/ironic-python-agent master: Correct error in Infiniband release note https://review.openstack.org/433050 | 13:20 |
| openstackgerrit | Miles Gould proposed openstack/ironic-python-agent master: Improve grammar/readability in Ocata release notes https://review.openstack.org/433051 | 13:20 |
| mgould | ^^^ IPA reno cleanups | 13:20 |
| *** dims has joined #openstack-ironic | 13:20 | |
| dtantsur | thanks! | 13:21 |
| dtantsur | huh, I see many folks are planning formal slots for their PTG work | 13:21 |
| mat128 | mgould: thanks for doing this | 13:23 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Updated from global requirements https://review.openstack.org/432673 | 13:24 |
| *** vsaienko has quit IRC | 13:27 | |
| *** owalsh is now known as owalsh-afk | 13:29 | |
| mgould | mat128: np, I'll take a look at the Ironic ones after lunch | 13:29 |
| *** mgould is now known as mgould|brb | 13:30 | |
| *** vinsh has joined #openstack-ironic | 13:30 | |
| *** mikal_ has joined #openstack-ironic | 13:30 | |
| *** owalsh has joined #openstack-ironic | 13:31 | |
| *** mikal has quit IRC | 13:32 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: Fix some grammar and spelling issues in release notes https://review.openstack.org/433043 | 13:32 |
| TheJulia | dtantsur: I was just reading some of those emails | 13:32 |
| openstackgerrit | Merged openstack/ironic-inspector master: Find a node by multiple attributes https://review.openstack.org/421285 | 13:32 |
| *** owalsh_ has joined #openstack-ironic | 13:32 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Add functional tests for VIFs with ironicclient commands https://review.openstack.org/430881 | 13:32 |
| *** owalsh-afk has quit IRC | 13:34 | |
| *** jheroux has joined #openstack-ironic | 13:34 | |
| *** vsaienko has joined #openstack-ironic | 13:35 | |
| *** Nisha_Agarwal has quit IRC | 13:35 | |
| *** owalsh has quit IRC | 13:36 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 13:36 | |
| sambetts | dtantsur, mgould|brb: I've updated my patch for inspector, I hope I've covered all the release notes for this release | 13:36 |
| *** owalsh has joined #openstack-ironic | 13:37 | |
| *** owalsh_ has quit IRC | 13:37 | |
| *** owalsh_ has joined #openstack-ironic | 13:38 | |
| *** owalsh__ has joined #openstack-ironic | 13:39 | |
| openstackgerrit | Merged openstack/ironic-ui master: standardize release notes filenames and fix contents https://review.openstack.org/429896 | 13:40 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Use SwiftAPI for temporary url generation in glance https://review.openstack.org/410764 | 13:41 |
| *** owalsh has quit IRC | 13:42 | |
| *** ccamacho is now known as ccamacho|lunch | 13:42 | |
| *** dprince has quit IRC | 13:42 | |
| *** owalsh_ has quit IRC | 13:43 | |
| *** dims_ has joined #openstack-ironic | 13:43 | |
| *** owalsh__ has quit IRC | 13:44 | |
| *** dims has quit IRC | 13:45 | |
| openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/431073 | 13:45 |
| rpioso | I've added a few items to the end of the Pike PTG proposed topics etherpad, https://etherpad.openstack.org/p/ironic-pike-ptg. Apologies for any inconvenience. I had some technical difficulties on Friday. | 13:45 |
| rpioso | Please take a look and let me know what you think. | 13:45 |
| openstackgerrit | Merged openstack/ironic-ui master: Remove py34 env from tox https://review.openstack.org/431284 | 13:46 |
| openstackgerrit | Merged openstack/ironic-ui master: Correct horizon dir in installation doc https://review.openstack.org/431851 | 13:46 |
| rpioso | They're about 1) enhancing RAID config expressiveness, 2) configuring BIOS settings during cleaning/deployment, and 3) setting the boot device via the management interface. | 13:48 |
| *** mtanino has joined #openstack-ironic | 13:49 | |
| dtantsur | rpioso, please add your name to all items, and some details about "setting the boot device" (we already do that, but on different level of granularity) | 13:49 |
| *** ralonsoh__ is now known as ralonsoh | 13:57 | |
| *** sasha2 has joined #openstack-ironic | 13:58 | |
| rpioso | dtantsur, ty for reminding me to add my name :) I've clarified what I'm interested in adding to "setting the boot device". | 13:59 |
| *** amotoki has quit IRC | 13:59 | |
| *** Goneri has joined #openstack-ironic | 14:01 | |
| dtantsur | rpioso, ok, copies your topics to the schedule, let's see if we get to them (please come to the meeting today in case people have questions) | 14:01 |
| rpioso | dtantsur, ty. I'll be there. | 14:04 |
| *** crushil has joined #openstack-ironic | 14:05 | |
| TheJulia | If any bifrost can take few minutes and look at https://review.openstack.org/#/c/431614/. It be awesome. | 14:05 |
| patchbot | patch 431614 - bifrost - Always install Ansible with pip | 14:05 |
| *** amotoki has joined #openstack-ironic | 14:08 | |
| *** lucas-hungry is now known as lucasagomes | 14:10 | |
| openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client master: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 14:10 |
| *** dprince has joined #openstack-ironic | 14:10 | |
| *** amoralej is now known as amoralej|lunch | 14:11 | |
| vdrok | morning TheJulia mat128 and rpioso | 14:15 |
| TheJulia | Good morning vdrok | 14:15 |
| *** rloo has joined #openstack-ironic | 14:16 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient master: Add command RAID logical disk properties https://review.openstack.org/362047 | 14:16 |
| *** cdearborn has joined #openstack-ironic | 14:16 | |
| *** moshele has joined #openstack-ironic | 14:17 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Correct error in Infiniband release note https://review.openstack.org/433050 | 14:21 |
| *** ccamacho|lunch is now known as ccamacho | 14:23 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Pass session directly to swiftclient https://review.openstack.org/432353 | 14:25 |
| dtantsur | TheJulia, vdrok, why do we hold https://review.openstack.org/#/c/431899/ ? should be fine to merge.. | 14:28 |
| patchbot | patch 431899 - python-ironicclient - Updated from global requirements | 14:28 |
| dtantsur | ditto for https://review.openstack.org/#/c/431954/ | 14:28 |
| patchbot | patch 431954 - ironic-lib - Updated from global requirements | 14:28 |
| vdrok | dtantsur: I'm not sure, we have the stable versions in upper constraints already? | 14:29 |
| dtantsur | vdrok, these projects already got stable/ocata anyway, unlike ironic, inspector and IPA | 14:29 |
| vdrok | hm, ok, I'll merge then | 14:29 |
| TheJulia | Yeah, I was just about to say we've already cut that release | 14:29 |
| TheJulia | tl;dr fine to merge | 14:30 |
| dtantsur | thnx | 14:31 |
| *** Nisha_Agarwal has quit IRC | 14:31 | |
| *** nmathew- has quit IRC | 14:31 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Add a summary release note for ocata https://review.openstack.org/431187 | 14:33 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Clean up release notes for 7.0.0 https://review.openstack.org/431188 | 14:33 |
| jroll | TheJulia: vdrok: rloo: dtantsur: ^ I hope we're finally done here :) | 14:34 |
| vdrok | :) | 14:34 |
| dtantsur | heh | 14:34 |
| rloo | hi jroll. I was hoping that would be done before I got to work today :-( | 14:34 |
| jroll | rloo: well, we got the rest of the notes done, and a cycle or three of reviews | 14:35 |
| dtantsur | morning rloo | 14:36 |
| rloo | jroll: am going through my 100+ emails and see that :) will get to them this AM after I get a bunch of stuff out of the way. Unless of course, I'm too late which wouldn't be too bad ;) | 14:36 |
| vdrok | good morning rloo | 14:36 |
| jroll | rloo: cool, thank you | 14:36 |
| *** amoralej|lunch is now known as amoralej | 14:37 | |
| rloo | hi and morning week before PTG all, dtantsur, jroll, vdrok, TheJulia | 14:37 |
| TheJulia | good morning rloo | 14:37 |
| jroll | morning :) | 14:37 |
| *** amotoki has quit IRC | 14:37 | |
| rloo | whoever mvp nitesh is, i hope someone has time to help them with their commit titles... | 14:38 |
| *** bfournie1 has joined #openstack-ironic | 14:45 | |
| *** vinsh_ has joined #openstack-ironic | 14:45 | |
| *** bfournie has quit IRC | 14:45 | |
| *** vinsh has quit IRC | 14:45 | |
| rloo | oh man, fedex truck is stuck in front of my house. i wonder if they know how to drive or if it is really slippery outside. i gave him my shovel and a bag of salt and left him :) | 14:47 |
| dtantsur | :D | 14:48 |
| jroll | hah | 14:48 |
| *** pcaruana has quit IRC | 14:50 | |
| lucasagomes | rloo, hah | 14:52 |
| lucasagomes | morning | 14:52 |
| *** baoli has joined #openstack-ironic | 14:53 | |
| rloo | morning lucasagomes! I'm not sure the guy knows what he's doing so maybe I should have stayed but I was worried he'd run me over or something | 14:53 |
| lucasagomes | rloo, he will figure out eventually, he's learning the hard way | 14:54 |
| lucasagomes | dtantsur, vdrok anything we urgently need to merge in the staging drivers before I release it ? | 14:54 |
| mgould|brb | morning rloo | 14:54 |
| rloo | lucasagomes: one can only hope :) | 14:55 |
| rloo | hi mgould|brb! | 14:55 |
| * jroll disappears for meetings for a couple of hours | 14:55 | |
| *** mgould|brb is now known as mgould | 14:55 | |
| jroll | hopefully we can release when I return | 14:55 |
| jroll | :) | 14:55 |
| lucasagomes | ++ | 14:55 |
| *** pcaruana has joined #openstack-ironic | 14:55 | |
| *** jcoufal has joined #openstack-ironic | 14:56 | |
| vdrok | lucasagomes: I think pas-ha wanted this one https://review.openstack.org/414644 for testing... will look at it now | 14:57 |
| patchbot | patch 414644 - ironic-staging-drivers - Enable testing of particular staging driver | 14:57 |
| * lucasagomes looks too | 14:57 | |
| rpioso | vdrok: Yes, good morning! | 14:59 |
| TheJulia | rloo: I would have loved to see the look on the fedex driver's face | 14:59 |
| *** moshele has quit IRC | 14:59 | |
| openstackgerrit | Xavier proposed openstack/ironic master: Validation before perform node deallocation https://review.openstack.org/386014 | 15:00 |
| rloo | TheJulia: he's been out there for awhile now. i think he ran out of the salt. my neighbour just gave him something else. i'm debating whether to talk to my neighbour about it (they are the ones that called me to ask me to help him out). | 15:00 |
| *** udesale has joined #openstack-ironic | 15:03 | |
| vdrok | lucasagomes: hah, it also depends on a re-use tinyipa patch, sorry :) | 15:03 |
| openstackgerrit | Merged openstack/python-ironicclient master: Handle log message interpolation by the logger https://review.openstack.org/430273 | 15:03 |
| openstackgerrit | Merged openstack/python-ironicclient master: Updated from global requirements https://review.openstack.org/431899 | 15:03 |
| lucasagomes | vdrok, it's ok, will take a look in a sec | 15:03 |
| dtantsur | any objections to approving this RFE: https://bugs.launchpad.net/ironic/+bug/1660292 ? it essentially continues notifications, now for port groups | 15:07 |
| openstack | Launchpad bug 1660292 in Ironic "[RFE] Portgroup notifications" [Wishlist,In progress] - Assigned to Yuriy Zveryanskyy (yzveryanskyy) | 15:07 |
| *** beekneemech is now known as bnemec | 15:08 | |
| vdrok | not from me :) | 15:09 |
| TheJulia | dtantsur: seems more like a bug when considering consistency :) | 15:09 |
| dtantsur | yep, I think I'll just rfe-approve it | 15:09 |
| rloo | dtantsur: i didn't look but i think it should be a simple approve | 15:10 |
| openstackgerrit | Merged openstack/ironic-lib master: Updated from global requirements https://review.openstack.org/431954 | 15:12 |
| *** jcoufal_ has joined #openstack-ironic | 15:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/431952 | 15:12 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/431953 | 15:12 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements https://review.openstack.org/431956 | 15:12 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Updated from global requirements https://review.openstack.org/431957 | 15:12 |
| *** owalsh_ has joined #openstack-ironic | 15:12 | |
| dtantsur | mgould, re https://bugs.launchpad.net/ironic/+bug/1656858 : did you enable boot_mode discovery in inspector.conf? | 15:12 |
| openstack | Launchpad bug 1656858 in Ironic "Inspector does not set boot_mode property when inspecting UEFI DRAC nodes" [Undecided,New] | 15:12 |
| *** owalsh_ is now known as owalsh | 15:13 | |
| mgould | dtantsur: not sure, I'll check | 15:13 |
| mgould | if it's not the default, then no | 15:15 |
| *** jcoufal has quit IRC | 15:15 | |
| dtantsur | dsneddon_afk, FYI https://bugs.launchpad.net/ironic/+bug/1658964 | 15:16 |
| openstack | Launchpad bug 1658964 in Ironic "[RFE] Implement neutron routed networks support in Ironic" [Wishlist,Confirmed] | 15:16 |
| dtantsur | mgould, it's not | 15:16 |
| *** Nisha_Agarwal has joined #openstack-ironic | 15:16 | |
| *** causten has joined #openstack-ironic | 15:17 | |
| *** causten has quit IRC | 15:17 | |
| *** athomas has quit IRC | 15:18 | |
| *** athomas has joined #openstack-ironic | 15:18 | |
| *** causten_ has joined #openstack-ironic | 15:19 | |
| *** Syed__ has joined #openstack-ironic | 15:19 | |
| mgould | dtantsur: OK, it's probably an "mgould missed the relevant config option" problem, then | 15:23 |
| dtantsur | ok :) | 15:26 |
| mgould | let me check, though | 15:26 |
| mgould | are we ever planning to change the default value? I'm surprised that's configurable | 15:27 |
| rloo | dtantsur, jroll: dumb question. how experimental is dynamic drivers? Wondering if we should put 'experimental' in the affected config descriptions. | 15:30 |
| dtantsur | rloo, not to an extend of being unusable or breaking user data | 15:32 |
| rloo | dtantsur: might be good to know what 'experimental' means then. | 15:32 |
| rloo | dtantsur: does it just mean 'not fully supported'? | 15:32 |
| rloo | dtantsur: wrt equivs for existing classic drivers? | 15:33 |
| dtantsur | "finished, but may have rough edges we're not aware of" | 15:33 |
| JayF | I think the bigger thing to flag is the significantly fewer drivers supported by hardware types | 15:33 |
| rloo | dtantsur: but 'rough edges' does not include unusable or breaking user's data/nodes? | 15:33 |
| dtantsur | dunno, I'm not strong with English wording, and I'm fine with people inventing their own definition | 15:33 |
| JayF | I'd be super confused as an operator if I tried to migrate from a legacy driver when the dynamic driver for my hardware didn't exist yet. | 15:34 |
| dtantsur | rloo, not necessary. rather what JayF mentions or maybe potential weirdnesses in API | 15:34 |
| *** rajinir_ has joined #openstack-ironic | 15:34 | |
| rloo | dtantsur: what dynamic driver do we have that is equiv to classic? ipmi is the only hw type, right? | 15:34 |
| rloo | dtantsur: maybe as JayF sez, we could mention the classic drivers and their equiv dynamic driver? | 15:35 |
| rloo | dtantsur: for the ones that have equivalents | 15:35 |
| dtantsur | rloo, ipmi and irmc now | 15:35 |
| rloo | dtantsur: and we don't want to rush through the other hw types I gues | 15:36 |
| dtantsur | I'm open to reviewing a patch, if somebody provides it, but it's too hard for me to invent a wording satisfying everyone | 15:36 |
| *** dyasny has quit IRC | 15:38 | |
| TheJulia | I kind of thought the prelude in the summary kind of put it well, of course, in configuration files, someone who didn't read that might not grok that it is a fairly new thing | 15:38 |
| *** dyasny has joined #openstack-ironic | 15:38 | |
| *** spartacloud has joined #openstack-ironic | 15:39 | |
| *** jaganathan has quit IRC | 15:39 | |
| rloo | dtantsur: i'm not sure that 'experimental' is the right word vs not-all-classic-drivers-have-equiv-dynamic-drivers. | 15:39 |
| rloo | dtantsur: but will leave for others to comment if they want it changed. | 15:39 |
| JayF | Maybe even something like | 15:40 |
| jroll | words are hard | 15:40 |
| dtantsur | 'experimental' is erring at a safe side | 15:40 |
| JayF | "Dyanmic drivers for Ironic is implemented in Ocata, only for the following drivers: \n[table of old->new driver mapping]\n. Use of these in Ocata is completely optional." | 15:40 |
| JayF | except spell dynamic correctly /o\ | 15:41 |
| dtantsur | also the fact that we don't yet have voting CI coverage for these drivers justifies 'experimental' IMO | 15:41 |
| NobodyCam | Good Morning Ironic'ers | 15:41 |
| *** owalsh_ has joined #openstack-ironic | 15:41 | |
| JayF | oh that's a good point re: ci | 15:41 |
| dtantsur | morning NobodyCam | 15:41 |
| NobodyCam | morning dtantsur :) | 15:41 |
| jroll | when I use the word experimental here, I meant to imply 1) the lack of drivers, 2) it isn't battle tested (see also CI) | 15:42 |
| jroll | rloo: dtantsur JayF ^ | 15:42 |
| dtantsur | jroll++ | 15:42 |
| mgould | beta? | 15:43 |
| JayF | beta implies a structure that doesn't exist here, lol | 15:43 |
| JayF | I like experimental w/an explanation | 15:43 |
| rloo | jroll: I think we should mention those two then. (whether you want to call it experimental or alpha, beta, gamma...) | 15:43 |
| JayF | I'd ask it this way: do we want any deployers who are not in this channel running dynamic drivers (yet)? | 15:43 |
| JayF | It sounds like the general consensus is "probably not" | 15:44 |
| *** owalsh has quit IRC | 15:44 | |
| rloo | jroll: you already mention in the release notes that not all classic have corresp. dynamic, maybe clarify that. and mention no CI coverage yet. | 15:44 |
| dtantsur | JayF, not in production | 15:44 |
| dtantsur | well, maybe even in production, if they feel brave | 15:44 |
| jroll | I'm fine with people running it in prod | 15:44 |
| JayF | dtantsur: Then I think experimental is the perfect word | 15:44 |
| * JayF whips up a patch | 15:44 | |
| jroll | see also onmetal using agent_ipmitool before CI existed | 15:44 |
| dtantsur | but the reason I was rushing the driver comp into the Ocata release was *specifically* for people to run it in their dev/experiment/QA environments | 15:44 |
| jroll | rloo: please leave comments in the patch, I'm still stuck in meetings for another hour | 15:45 |
| jroll | JayF: a patch for... | 15:45 |
| jroll | ? | 15:45 |
| rloo | JayF: i haven't yet look at release notes, just the summary. i think we'd want a patch for both places. | 15:45 |
| JayF | jroll: do you already have a config desscription change patch up? | 15:45 |
| * JayF wonders if he missed something | 15:45 | |
| rloo | jroll: will do, i need to get something else done first before i forget. | 15:45 |
| jroll | JayF: not for config descriptions, no | 15:45 |
| jroll | I'm only half following this conversation | 15:45 |
| JayF | That's what I'm writing a patch for. | 15:45 |
| jroll | so I wasn't sure | 15:45 |
| jroll | ok | 15:45 |
| JayF | If for no other reason than to give us a gerrit place to have this arument | 15:45 |
| rloo | JayF: oh. i was thinking about our release notes. i suppose adding something to configs might help too. | 15:46 |
| jroll | that seems... overkill, if we're going to have it in docs + release notes, but ¯\_(ツ)_/¯ | 15:46 |
| JayF | I mean, if you both think it's overkill I can skip it | 15:46 |
| jroll | idk | 15:46 |
| * dtantsur does think it's an overkill | 15:46 | |
| JayF | I tend to like updating the config descriptions, just because I know that's one of the first things I do when deploying an unfamiliar piece of software: read the sample config | 15:46 |
| dtantsur | well, not unless we put our whole docs to the config descriptions | 15:47 |
| * JayF :q! and stops working on it | 15:47 | |
| rloo | JayF: it might be overkill. i was going to wait to see what you proposed :) | 15:47 |
| JayF | don't let me waste time on something three other cores are meh about :P | 15:47 |
| *** vinsh_ has quit IRC | 15:47 | |
| *** vinsh has joined #openstack-ironic | 15:48 | |
| rloo | JayF: you might have a great idea? | 15:48 |
| -openstackstatus- NOTICE: We are currently investigating an issue with our AFS mirrors which is causing some projects jobs to fail. We are working to correct the issue. | 15:49 | |
| dtantsur | release day \o/ | 15:49 |
| TheJulia | woot! | 15:49 |
| *** amarao has quit IRC | 15:50 | |
| *** rloo has quit IRC | 15:51 | |
| *** rloo has joined #openstack-ironic | 15:51 | |
| *** owalsh_ is now known as owalsh | 15:51 | |
| *** vinsh has quit IRC | 15:52 | |
| *** milan has quit IRC | 15:56 | |
| *** vinsh has joined #openstack-ironic | 15:56 | |
| *** milan has joined #openstack-ironic | 15:56 | |
| *** dsneddon_afk is now known as dsneddon | 16:02 | |
| *** jcoufal_ has quit IRC | 16:05 | |
| *** jcoufal has joined #openstack-ironic | 16:06 | |
| *** dtantsur is now known as dtantsur|mtg | 16:08 | |
| openstackgerrit | Merged openstack/ironic master: Walk over all objects when doing VIF detach https://review.openstack.org/432995 | 16:09 |
| openstackgerrit | Merged openstack/ironic master: Add a summary release note for ocata https://review.openstack.org/431187 | 16:09 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Add functional tests for VIFs in OSC plugin https://review.openstack.org/430904 | 16:11 |
| *** [1]rpioso has joined #openstack-ironic | 16:12 | |
| openstackgerrit | Merged openstack/bifrost master: Always install Ansible with pip https://review.openstack.org/431614 | 16:17 |
| *** dyasny has quit IRC | 16:18 | |
| *** rcernin has quit IRC | 16:19 | |
| *** dims_ has quit IRC | 16:19 | |
| *** pcaruana has quit IRC | 16:20 | |
| TheJulia | \o/ | 16:21 |
| *** dims has joined #openstack-ironic | 16:24 | |
| *** jcoufal_ has joined #openstack-ironic | 16:28 | |
| *** rpioso has quit IRC | 16:29 | |
| *** jcoufal has quit IRC | 16:30 | |
| *** causten_ has quit IRC | 16:31 | |
| *** causten has joined #openstack-ironic | 16:32 | |
| openstackgerrit | Merged openstack/bifrost master: Cleanup python requirements https://review.openstack.org/431496 | 16:33 |
| *** pcaruana has joined #openstack-ironic | 16:34 | |
| *** jcoufal has joined #openstack-ironic | 16:34 | |
| *** e0ne has joined #openstack-ironic | 16:35 | |
| *** [1]rpioso is now known as rpioso | 16:35 | |
| *** jcoufal_ has quit IRC | 16:37 | |
| *** hshiina has joined #openstack-ironic | 16:40 | |
| pas-ha | lucasagomes: if you have time, all comments to https://review.openstack.org/#/c/339742/ are resloved ;) | 16:45 |
| patchbot | patch 339742 - ironic-staging-drivers - Re-use tinyipa image for ansbile-deploy | 16:45 |
| lucasagomes | pas-ha, hi there, cool! It's on my list to look but I'm currently in a meeting :-/ | 16:45 |
| pas-ha | np, just a friendly ping/heads-up :) | 16:46 |
| *** udesale has quit IRC | 16:47 | |
| *** dims has quit IRC | 16:51 | |
| dtantsur|mtg | jroll, is it me he runs today's meeting or still you? :) | 16:51 |
| jroll | dtantsur|mtg: I always can when you don't want to, but I think it should be you :) | 16:52 |
| dtantsur|mtg | ok, just asking :) | 16:52 |
| *** jcoufal_ has joined #openstack-ironic | 16:53 | |
| *** e0ne has quit IRC | 16:54 | |
| *** dims has joined #openstack-ironic | 16:55 | |
| *** sdake_ has joined #openstack-ironic | 16:55 | |
| *** tesseract-RH has quit IRC | 16:55 | |
| *** jcoufal has quit IRC | 16:56 | |
| dtantsur|mtg | aarefiev_afk, milan, either of you want to check https://etherpad.openstack.org/p/ptg-pike-wsgi from inspector side? | 16:57 |
| *** sdake__ has joined #openstack-ironic | 16:57 | |
| *** dtantsur|mtg is now known as dtantsur | 16:58 | |
| * dtantsur is ready to jump from one meeting to another :) | 16:58 | |
| *** pcaruana has quit IRC | 16:58 | |
| *** pcaruana has joined #openstack-ironic | 16:58 | |
| *** sdake has quit IRC | 16:59 | |
| *** sdake_ has quit IRC | 17:00 | |
| *** mgoddard has quit IRC | 17:01 | |
| *** mgoddard has joined #openstack-ironic | 17:02 | |
| milan | dtantsur, yeah, I'll look into that | 17:04 |
| *** dims_ has joined #openstack-ironic | 17:06 | |
| *** athomas has quit IRC | 17:06 | |
| *** dims has quit IRC | 17:07 | |
| pas-ha | lucasagomes: thanks! no I'll go and nag the infra team about that experimental job I've requested :) | 17:08 |
| lucasagomes | pas-ha, awesome, and I will put a new release out for staging as soon as it merges | 17:08 |
| jroll | rloo: eh, idk how to feel about public transit in ATL | 17:09 |
| *** jcoufal__ has joined #openstack-ironic | 17:09 | |
| JayF | What's wrong with MARTA? I took it to/from the airport when the summit was there without any issue. | 17:10 |
| rloo | jroll: oh, ok. i enjoy checking out public transit when i go somewhere. | 17:10 |
| rloo | i agree, walking is the best solution. easier to stagger back to hotel. | 17:10 |
| jroll | idk, I haven't heard great things about it | 17:10 |
| TheJulia | MARTA light rail is fairly limited, and there have been some problems on it in the past | 17:10 |
| lucasagomes | JayF, ++ I used it too, no problem | 17:10 |
| jroll | let me put it this way, my friends that live in ATL area wouldn't recommend it :P | 17:11 |
| *** jcoufal_ has quit IRC | 17:11 | |
| TheJulia | Also, after a particular hour, the service essentially shuts down except one run | 17:11 |
| jroll | (afaik) | 17:11 |
| TheJulia | I also would recommend avoiding marta | 17:11 |
| TheJulia | I guess uber, and taxis are the best bet if your not renting a car. | 17:13 |
| lucasagomes | interesting, I was counting on it... I will have a plan B then | 17:13 |
| jroll | surely it depends on the route and time of day, too | 17:13 |
| *** aNuposic has joined #openstack-ironic | 17:14 | |
| * JayF has to brb from the meeting, contractors arrived | 17:14 | |
| -openstackstatus- NOTICE: AFS replication issue has been addressed. Mirrors are currently re-syncing and coming back online. | 17:19 | |
| *** bnemec has quit IRC | 17:20 | |
| *** trown is now known as trown|lunch | 17:21 | |
| *** bnemec has joined #openstack-ironic | 17:24 | |
| openstackgerrit | Merged openstack/bifrost master: roles: bifrost-ironic-install: Always download the latest iPXE files https://review.openstack.org/427682 | 17:35 |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic master: WIP/DNM: Baremetal Devstack Network Simulation Developer Doc https://review.openstack.org/422582 | 17:36 |
| *** rama_y has joined #openstack-ironic | 17:38 | |
| *** causten has quit IRC | 17:39 | |
| *** vsaienko has quit IRC | 17:40 | |
| *** awiddersheim has quit IRC | 17:42 | |
| NobodyCam | :) | 17:43 |
| *** owalsh has quit IRC | 17:44 | |
| *** baoli has quit IRC | 17:44 | |
| *** pcaruana has quit IRC | 17:44 | |
| *** baoli has joined #openstack-ironic | 17:45 | |
| *** baoli has quit IRC | 17:45 | |
| dtantsur | JayF, do you have awesome On Metal ones left? | 17:47 |
| JayF | No. Those were very limited edition and are gone, and will never be printed again | 17:47 |
| dtantsur | I see | 17:49 |
| *** dprince has quit IRC | 17:49 | |
| *** derekh has quit IRC | 17:52 | |
| *** strigazi is now known as strigazi_AFK | 17:54 | |
| *** ralonsoh has quit IRC | 17:56 | |
| *** mgoddard has quit IRC | 17:59 | |
| NobodyCam | Good Meeting | 18:00 |
| * jroll needs lunch and errand running | 18:00 | |
| * dtantsur needs some rest :) | 18:00 | |
| lucasagomes | NobodyCam, morning | 18:00 |
| lucasagomes | folks I'm calling it a day, have a great evening all! | 18:00 |
| vdrok | morning NobodyCam and jlvillal and good nigth everyone! | 18:01 |
| *** fragatina has joined #openstack-ironic | 18:01 | |
| mgould | lucasagomes: o/ | 18:01 |
| *** fragatina has quit IRC | 18:01 | |
| lucasagomes | mgould, vdrok g'night! | 18:01 |
| *** lucasagomes is now known as lucas-afk | 18:01 | |
| jlvillal | vdrok: Hi! And good night. | 18:01 |
| dtantsur | calling it a day as well, unless anybody has any questions/requests to me | 18:01 |
| NobodyCam | night dtantsur | 18:01 |
| *** fragatina has joined #openstack-ironic | 18:01 | |
| *** dtantsur is now known as dtantsur|afk | 18:02 | |
| NobodyCam | night vdrok :) | 18:02 |
| NobodyCam | morning / good night lucas-afk | 18:03 |
| pas-ha | g'night all | 18:06 |
| *** david-lyle_ has joined #openstack-ironic | 18:08 | |
| mgould | pas-ha: good night | 18:08 |
| *** jaosorior has quit IRC | 18:08 | |
| *** mgould is now known as mgould|afk | 18:08 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Do not pass SNMPv3 credentials always https://review.openstack.org/432958 | 18:09 |
| openstackgerrit | Miles Gould proposed openstack/ironic-python-agent master: Improve grammar/readability in Ocata release notes https://review.openstack.org/433051 | 18:12 |
| *** david-lyle_ is now known as david-lyle | 18:12 | |
| aNuposic | mariojv rechecked https://review.openstack.org/#/c/407639/, can you please re-review | 18:16 |
| patchbot | patch 407639 - ironic - Config drive support for Ceph | 18:16 |
| aNuposic | jlvillal vdrok vsaienk0 TheJulia ^ | 18:16 |
| *** milan has quit IRC | 18:17 | |
| *** hshiina has quit IRC | 18:21 | |
| *** baoli has joined #openstack-ironic | 18:24 | |
| *** baoli has quit IRC | 18:25 | |
| *** baoli has joined #openstack-ironic | 18:25 | |
| mariojv | i'll look after the cycle's more underway aNuposic | 18:28 |
| *** e0ne has joined #openstack-ironic | 18:30 | |
| aNuposic | mariojv: ok sure | 18:32 |
| *** mgoddard has joined #openstack-ironic | 18:32 | |
| TheJulia | jroll: When the time comes, 6c7a80e2307d4cad3120716bdd5132e868907b36 for bifrost | 18:34 |
| TheJulia | Also, any cores, a review of https://review.openstack.org/#/c/431188 will make jroll happy | 18:35 |
| patchbot | patch 431188 - ironic - Clean up release notes for 7.0.0 | 18:35 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing multi-tenant multi-node grenade job https://review.openstack.org/422149 | 18:41 |
| *** sdake has joined #openstack-ironic | 18:42 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Do not pass SNMPv3 credentials always https://review.openstack.org/432958 | 18:42 |
| *** sdake__ has quit IRC | 18:43 | |
| *** Goneri has quit IRC | 18:43 | |
| *** e0ne has quit IRC | 18:44 | |
| *** sdake has quit IRC | 18:45 | |
| JayF | TheJulia: jroll I'm about 1/2way thru that so far | 18:46 |
| *** trown|lunch is now known as trown | 18:47 | |
| openstackgerrit | Merged openstack/ironic-ui master: Updated from global requirements https://review.openstack.org/431957 | 18:47 |
| *** sdake has joined #openstack-ironic | 18:47 | |
| *** sdake has quit IRC | 18:47 | |
| *** baoli has quit IRC | 18:48 | |
| *** baoli has joined #openstack-ironic | 18:48 | |
| *** vsaienko has joined #openstack-ironic | 18:49 | |
| *** patriciadomin has quit IRC | 18:50 | |
| *** Sukhdev has joined #openstack-ironic | 18:51 | |
| *** jcoufal has joined #openstack-ironic | 18:53 | |
| *** Nisha_Agarwal has quit IRC | 18:55 | |
| JayF | TheJulia: jroll: dtantsur|afk: should I land https://review.openstack.org/#/c/431188/ ? -- I just put the 2nd +2 on it | 18:56 |
| *** jcoufal__ has quit IRC | 18:56 | |
| patchbot | patch 431188 - ironic - Clean up release notes for 7.0.0 | 18:56 |
| JayF | I don't wanna step on anyone else who might be reviewing | 18:56 |
| *** fragatina has quit IRC | 18:59 | |
| *** awiddersheim has joined #openstack-ironic | 18:59 | |
| rloo | JayF: i am reviewing found 2 nits (not important). one question. also, we wanted to add more info about dynamic drivers, but i haven't gettn to that reno yet. | 19:01 |
| JayF | The dynamic drivers reno was A++ IMO | 19:01 |
| JayF | but I'll let you finish reviewing and vote, nbd | 19:01 |
| *** crushil has quit IRC | 19:05 | |
| *** dyasny has joined #openstack-ironic | 19:08 | |
| *** vsaienko has quit IRC | 19:17 | |
| *** dprince has joined #openstack-ironic | 19:26 | |
| *** crushil has joined #openstack-ironic | 19:28 | |
| rloo | JayF: ok, reviewed. I'm fine if it gets +A and a followup or not. I've already forgotten what I commented on. | 19:29 |
| *** amoralej is now known as amoralej|off | 19:37 | |
| *** jcoufal_ has joined #openstack-ironic | 19:38 | |
| *** jcoufal has quit IRC | 19:39 | |
| *** mgoddard has quit IRC | 19:43 | |
| *** dsneddon is now known as dsneddon_afk | 19:45 | |
| *** owalsh has joined #openstack-ironic | 19:50 | |
| *** owalsh is now known as owalsh-afk | 19:50 | |
| *** baoli has quit IRC | 19:50 | |
| *** baoli has joined #openstack-ironic | 19:51 | |
| *** gk-1wm-su has joined #openstack-ironic | 19:51 | |
| *** gk-1wm-su has quit IRC | 19:54 | |
| *** jcoufal_ has quit IRC | 19:54 | |
| *** dsneddon has joined #openstack-ironic | 19:58 | |
| *** crushil has quit IRC | 20:02 | |
| *** crushil has joined #openstack-ironic | 20:05 | |
| *** jcoufal has joined #openstack-ironic | 20:07 | |
| *** zer0c00l has joined #openstack-ironic | 20:08 | |
| *** amoralej|off is now known as amoralej | 20:11 | |
| *** owalsh has joined #openstack-ironic | 20:13 | |
| *** owalsh-afk has quit IRC | 20:16 | |
| *** bfournie1 has quit IRC | 20:17 | |
| *** fragatina has joined #openstack-ironic | 20:17 | |
| *** vsaienko has joined #openstack-ironic | 20:20 | |
| *** ijw has joined #openstack-ironic | 20:20 | |
| rloo | dtantsur|afk: we didn't mention at today's meeting but i'm assuming we are skipping next Monday's meeting? We ok with the following Monday Feb 27? | 20:21 |
| * jroll sighs | 20:22 | |
| jroll | new reno patch shortly | 20:22 |
| *** jcoufal has quit IRC | 20:24 | |
| *** ijw has quit IRC | 20:25 | |
| *** ijw has joined #openstack-ironic | 20:25 | |
| *** bfournie has joined #openstack-ironic | 20:26 | |
| *** Goneri has joined #openstack-ironic | 20:26 | |
| *** jcoufal has joined #openstack-ironic | 20:29 | |
| mrda | Morning Ironic | 20:29 |
| *** causten has joined #openstack-ironic | 20:34 | |
| TheJulia | Good morning mrda | 20:39 |
| *** sdake has joined #openstack-ironic | 20:41 | |
| mrda | o/ | 20:41 |
| *** ianychoi has quit IRC | 20:43 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Clean up release notes for 7.0.0 https://review.openstack.org/431188 | 20:44 |
| jroll | rloo: done ^ | 20:44 |
| rloo | jroll: that was quick :) | 20:44 |
| *** crushil has quit IRC | 20:44 | |
| jroll | rloo: not that quick :/ | 20:44 |
| jroll | see some of my responses, there's a couple I didn't change | 20:44 |
| *** crushil has joined #openstack-ironic | 20:45 | |
| *** linggao has joined #openstack-ironic | 20:45 | |
| jroll | rloo: btw, we're late enough on the release that I'm personally looking for "good enough" on the IPA and inspector cleanups | 20:45 |
| jroll | jfyi if you review those | 20:45 |
| *** linggao_ has joined #openstack-ironic | 20:45 | |
| jroll | we need to get the release out asap :( | 20:45 |
| *** causten has quit IRC | 20:45 | |
| *** linggao_ has quit IRC | 20:45 | |
| *** causten has joined #openstack-ironic | 20:46 | |
| *** causten has quit IRC | 20:46 | |
| *** causten has joined #openstack-ironic | 20:47 | |
| *** linggao has quit IRC | 20:47 | |
| rloo | jroll: you can't rush... something... now what was it... | 20:48 |
| JayF | jroll: I'll seek and review the ipa ones | 20:49 |
| JayF | if someone else wants to look at inspector, I know less about that so someone else may be a better reviewer | 20:49 |
| jroll | JayF: https://review.openstack.org/#/c/433051/ | 20:49 |
| patchbot | patch 433051 - ironic-python-agent - Improve grammar/readability in Ocata release notes | 20:49 |
| jroll | rloo and dtantsur|afk have reviews on inspector, I'll update them | 20:49 |
| JayF | jroll: +2 on ironic | 20:50 |
| JayF | looking at ipa | 20:50 |
| *** ianychoi has joined #openstack-ironic | 20:51 | |
| JayF | jroll: actually there's one person who does use a config file for ipa | 20:51 |
| jroll | JayF: neat :) | 20:52 |
| JayF | that's why we have so much fixing for it in | 20:52 |
| JayF | basically they wrote a feature for cert authentication in IPA, and needed config file support/documentation to do it sanely in their ramdisk | 20:53 |
| *** jcoufal has quit IRC | 20:54 | |
| JayF | jroll: I had a nit on that ipa change, nothign major. Going to go ahead and land it if nobody else is reviewing it? | 20:54 |
| *** mikal_ is now known as mikal | 20:54 | |
| *** Syed__ has quit IRC | 20:55 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-inspector master: Fix some grammar and spelling issues in release notes https://review.openstack.org/433043 | 20:55 |
| * jroll ain't stopping you | 20:55 | |
| *** Sukhdev has quit IRC | 20:56 | |
| jroll | rloo: ^ that should fix your/dmitry's comments except the last one (because idk) | 20:56 |
| *** jcoufal has joined #openstack-ironic | 20:57 | |
| rloo | jroll: EXCEPT inspector seems to use 'Add', not 'Adds' (whatever tense that is called) | 20:57 |
| JayF | TheJulia: I didn't know you practiced ninjitzu :P | 20:58 |
| jroll | rloo: ............................... | 20:58 |
| * jroll fixes | 20:58 | |
| TheJulia | JayF: only when cats don't cause trouble | 20:58 |
| * TheJulia goes and cleans up a disaster a cat has created | 20:59 | |
| jroll | TheJulia: no, it's okay, I've got the release notes handled just fine | 20:59 |
| * jroll giggles | 20:59 | |
| rloo | jroll: fwiw, i prefer your way. but ... | 20:59 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Improve grammar/readability in Ocata release notes https://review.openstack.org/433051 | 21:00 |
| TheJulia | jroll: right after JayF mentioned me, my cat pushed my coffee from earlier off the dash of the RV on to the floor :( | 21:00 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-inspector master: Fix some grammar and spelling issues in release notes https://review.openstack.org/433043 | 21:00 |
| jroll | rloo: I think that fixes the ones I've broken | 21:01 |
| rloo | jroll: ok, /me looks again | 21:01 |
| jroll | rloo: it's just incredibly frustrating watching how much we all argue about/for consistency, yet nothing is ever consistent | 21:01 |
| jroll | TheJulia: :( | 21:01 |
| * TheJulia is all for "does this make it better" today | 21:01 | |
| rloo | jroll: i agree. it is consistent in ironic. if dmitry was here, i'd ask him about inspector but he isn't. maybe we should just make the changes while he isn't here. | 21:01 |
| jroll | rloo: no, it's consistent in ironic every 6 months when I spend an entire week on it | 21:02 |
| jroll | s/when/after | 21:02 |
| jroll | you're welcome to change inspector's release notes, I'm not, I'm never touching release notes again, this is silly | 21:02 |
| rloo | jroll: well, i make it consistent when i review. but there's only so much i will review and nits don't always get addressed in reno. | 21:02 |
| rloo | jroll: ha ha. | 21:02 |
| openstackgerrit | Merged openstack/ironic master: Clean up release notes for 7.0.0 https://review.openstack.org/431188 | 21:03 |
| JayF | I gotta get tougher on enforcing better release notes in the original patch, too. It feels like a shitty thing to block a patch on, but someone having to spend a week mopping them up is even worse :( | 21:03 |
| jroll | rloo: not even joking | 21:03 |
| jroll | idk who will do it, but it won't be me | 21:03 |
| rloo | jroll: the problem is that we developers are expected to be good at everything, documentation, testing, releasing, let alone development! | 21:03 |
| rloo | jroll: unless you're in a start up, i seriously doubt that companies expect that from each developer. | 21:04 |
| * rloo will remember to buy a beer for jroll next week | 21:04 | |
| jroll | rloo: sounds like something to tell our various corporate sponsors which apparently don't contribute enough people that are good at these things | 21:04 |
| TheJulia | +2'ed 433043 | 21:04 |
| rloo | jroll: that is something that the openstack something should be saying, no? where something != me | 21:06 |
| jroll | rloo: maybe, though the foundation can't possibly see everything, they need people on the front lines to give them feedback | 21:07 |
| rloo | jroll: that would be the PTLs. I'm towards the back, can you see me waving? no. QED. | 21:07 |
| rloo | I'm going to +A release notes for inspector unless anyone wants me to hold off? I'll wait one minute. maybe. | 21:08 |
| mariojv | it'd be cool if we had asked something about documentation readability during the user survey | 21:08 |
| mariojv | (maybe we did? i never read the responses, if they're out yet) | 21:09 |
| jroll | rloo: more feedback never hurts | 21:09 |
| *** amarao has joined #openstack-ironic | 21:09 | |
| * TheJulia chews on words for a moment regarding the current topic | 21:09 | |
| * rloo wonders if TheJulia will bite and say something ... :) | 21:10 | |
| TheJulia | rloo: with tact... perhaps! | 21:10 |
| rloo | Unlike me, TheJulia is the Queen of Tact! | 21:11 |
| JayF | to be blunt; all forms of contribution to openstack seem to be on a downward trend | 21:11 |
| TheJulia | ROFL | 21:11 |
| JayF | it makes me really suspicious that we'd be able to get more contribution, from anyone, in an area typically neglected by sponsoring companies | 21:11 |
| rloo | JayF: i'd trade a few developers for one great doc person | 21:11 |
| rloo | oh, what did i just say. did i mean it? not sure. | 21:12 |
| TheJulia | The problem is that there are broken feedback loops. It is the responsibility every contributor, at least every contributor who contributes on behalf of their organization, to convey the project wants, needs, and direction back to their org, and then help facilitate negotiation. That is if they care. In some cases, the contribution is more so for internal | 21:12 |
| TheJulia | consumption/making our peers or our own lives easier. | 21:12 |
| * JayF just wondering if rloo has him on the trade block | 21:13 | |
| JayF | Please don't trade me to Apache! /s | 21:13 |
| rloo | JayF: oh, if I had the power, the power! :D | 21:13 |
| TheJulia | But a company wanting to make a product, doesn't really care as much about a project's documentation or contributing to improve it because of NIH syndrome, and confusion actually becomes a profit center in the terms of support | 21:13 |
| * TheJulia gets off the soapbox | 21:13 | |
| *** sambetts is now known as sambetts|afk | 21:14 | |
| sambetts|afk | night all | 21:14 |
| rloo | night sambetts|afk | 21:15 |
| TheJulia | goodnight sambetts|afk | 21:15 |
| JayF | o/ | 21:16 |
| TheJulia | For what its worth.... I have a friend who is a technical writer who has had to reference ironic's documentation, and they expressed that we do put out some exceptionally verbose and clear documentation. I've spoken with some other technical writers, and really the perception I've gotten is that we just need to learn to chunk information into more | 21:19 |
| TheJulia | digestible bits, but that is a continuous improvement thing. | 21:19 |
| *** causten has quit IRC | 21:19 | |
| *** crushil has quit IRC | 21:19 | |
| *** aNuposic has quit IRC | 21:19 | |
| *** causten has joined #openstack-ironic | 21:20 | |
| rloo | TheJulia: that's great feedback. and yeah to NIH :-( | 21:20 |
| *** amarao has quit IRC | 21:20 | |
| JayF | TheJulia: fwiw, I've taken a couple of stabs at reorganizing, and just hit a wall. | 21:20 |
| JayF | TheJulia: maybe with such a direct ... problem statement, we could get some help from docs team? | 21:20 |
| JayF | TheJulia: if they would help us figure out an outline, I'd be happy to do the work to sort things into place | 21:21 |
| TheJulia | JayF: That is actually a really good idea! I was trying to lure a technical writer mid last year to help contribute to going through our docs, but sadly that didn't go anywhere. | 21:23 |
| *** amoralej is now known as amoralej|off | 21:23 | |
| *** causten has quit IRC | 21:24 | |
| *** jcoufal has quit IRC | 21:24 | |
| TheJulia | JayF: Out of curiosity, what was the wall that you felt you hit? | 21:25 |
| JayF | TheJulia: more or less, not knowing where to start. | 21:26 |
| JayF | TheJulia: the task I looked at was splitting our developer docs (for developers) away from admin guide (for deployers) | 21:27 |
| JayF | TheJulia: and it's all so mixed together it's hard to find a cutline. | 21:27 |
| TheJulia | yeah | 21:27 |
| *** jcoufal has joined #openstack-ironic | 21:27 | |
| JayF | TheJulia: I feel like if we had an idea of what should go where, I could have more success at moving it around to be better | 21:27 |
| TheJulia | That is a hard one to delineate given our blend in ironic. :( | 21:27 |
| JayF | Yes, exactly | 21:29 |
| JayF | Which is why I wonder if actual technical documentation expertise would be helpful | 21:29 |
| JayF | rather than just one of us taking a stab at it, which is what we've always done in the past | 21:29 |
| TheJulia | Well, the first step is to admit we have a problem... likely the third step is codifying that in git somehow. :) Second would be, at least I think it would be coming to terms that we can't just do it as a oneshot revision, that we just have to iterate. | 21:30 |
| *** aNuposic has joined #openstack-ironic | 21:30 | |
| JayF | Are you going to be at the full week of ptg? | 21:30 |
| TheJulia | I am | 21:31 |
| JayF | We should corner asettle early in the week and talk to her about this, see if she has any specific suggestions (or maybe if she volunteers some help formt he docs team) | 21:31 |
| TheJulia | Summer and I are driving down Sunday | 21:31 |
| JayF | awesome | 21:32 |
| *** jkilpatr has quit IRC | 21:32 | |
| *** dprince has quit IRC | 21:34 | |
| jroll | rloo: (or anyone else) any reason not to land the inspector reno cleanup? https://review.openstack.org/#/c/433043/ | 21:39 |
| patchbot | patch 433043 - ironic-inspector - Fix some grammar and spelling issues in release notes | 21:39 |
| rloo | jroll: no reason. i was waiting for 1 min to see if anyone wanted to look at it, and then forgot. thought i had landed it. | 21:40 |
| jroll | ah, thanks! | 21:40 |
| * jroll does all the releases | 21:40 | |
| JayF | \o/ | 21:41 |
| TheJulia | woot! Thank you jroll | 21:44 |
| *** causten has joined #openstack-ironic | 21:45 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Fix some grammar and spelling issues in release notes https://review.openstack.org/433043 | 21:46 |
| jroll | TheJulia: major version bump for bifrost? | 21:46 |
| jroll | I see some stuff in upgrade notes that imply yes, just checking | 21:47 |
| TheJulia | jroll: yes | 21:50 |
| jroll | TheJulia: cool, submitted as such | 21:50 |
| TheJulia | Thanks! | 21:50 |
| *** jcoufal_ has joined #openstack-ironic | 21:54 | |
| *** jcoufal has quit IRC | 21:55 | |
| *** jcoufal_ has quit IRC | 21:56 | |
| *** jheroux has quit IRC | 21:56 | |
| *** vsaienko has quit IRC | 21:57 | |
| *** vsaienko has joined #openstack-ironic | 22:11 | |
| aNuposic | JayF: Is this still a bug? https://bugs.launchpad.net/ironic/+bug/1611135 | 22:12 |
| openstack | Launchpad bug 1611135 in Ironic "Nodes should enter 'clean wait' state during a cleaning reboot" [Medium,Triaged] | 22:12 |
| *** jkilpatr has joined #openstack-ironic | 22:13 | |
| JayF | aNuposic: afaik, yes | 22:15 |
| JayF | aNuposic: nothing in upstream that can trigger it though (there are no IPA hardware managers that require a cleaning reboot) | 22:15 |
| *** trown is now known as trown|outtypewww | 22:15 | |
| aNuposic | JayF: I see. I tried to put the node in clean wait state and forcefully sent requested_reboot to true | 22:18 |
| JayF | yeah, I don't think that's going to work | 22:18 |
| aNuposic | JayF: but then got to know _cleaning_reboot https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L122 is called on out of band reboot | 22:18 |
| JayF | ooooh. interesting. | 22:19 |
| JayF | I know it doens't work for /in band reboots/ | 22:19 |
| JayF | at least as of when I filed that bug | 22:19 |
| JayF | and have not seen any code come thru that seems to fix it | 22:19 |
| JayF | I could make a DNM commit to IPA which would add a noop step that wants a cleaning reboot, if it'd help? | 22:19 |
| *** rajinir_ has quit IRC | 22:20 | |
| aNuposic | Yeah that might help, or if you can let me know how to reproduce this problem? I mean i believe interface should be different and not deploy for out band | 22:21 |
| aNuposic | *out of band | 22:21 |
| JayF | yeah gimme a sec, I'll push one up | 22:21 |
| aNuposic | sure | 22:21 |
| JayF | oh sweet, we already have two stpes | 22:24 |
| JayF | aNuposic: instead of pushing it up, I'm going to tell you what to do? | 22:24 |
| JayF | aNuposic: just change reboot_requested: True for erase_devices in hardware.py | 22:24 |
| aNuposic | JayF: yeah works. Okay | 22:25 |
| JayF | aNuposic: then build a new agent, upload it to glance, and assign it as the deploy ramdisk for the node | 22:25 |
| *** Goneri has quit IRC | 22:25 | |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic-python-agent master: Update reno for stable/ocata https://review.openstack.org/433307 | 22:25 |
| openstackgerrit | OpenStack Release Bot proposed openstack/bifrost master: Update reno for stable/ocata https://review.openstack.org/433312 | 22:26 |
| aNuposic | oh, i see. but what should be the interface in --clean steps when putting the node in clean provision state? | 22:26 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add Cinder storage driver https://review.openstack.org/366197 | 22:29 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add base cinder common interface https://review.openstack.org/355625 | 22:29 |
| aNuposic | i mean when running ironic-node-set-provision-state clean --clean steps {interface: ?, step: erase_devices}, should it be deploy? | 22:29 |
| aNuposic | JayF | 22:30 |
| JayF | aNuposic: instead of using manual cleaning to test, I'd suggest automated cleaning | 22:30 |
| JayF | aNuposic: if you do the steps I said above, and change the erase_devices step to requrie a reboot | 22:31 |
| JayF | just node-set-provision-state manage; node-set-provision-state provide will send the node through automated cleaning | 22:31 |
| *** causten has quit IRC | 22:31 | |
| *** causten has joined #openstack-ironic | 22:32 | |
| *** causten has quit IRC | 22:32 | |
| *** causten has joined #openstack-ironic | 22:32 | |
| * TheJulia raises an eyebrow at the release bot's proposal | 22:33 | |
| aNuposic | JayF: sweet, will follow that. | 22:34 |
| jroll | TheJulia: our pages are a bit non-standard, I actually meant to fix that before ocata :/ | 22:34 |
| jroll | which leads to that sort of thing | 22:34 |
| TheJulia | Ahh, non-standard as we've been putting the range on them? | 22:35 |
| jroll | TheJulia: and the current-series thing | 22:37 |
| jroll | everyone else just has unreleased | 22:37 |
| TheJulia | ahh | 22:37 |
| jroll | I updated ironic-ui after that release to what it should be, if you want to see it | 22:37 |
| *** pmannidi has joined #openstack-ironic | 22:38 | |
| TheJulia | Okay, I'll fix bifrost tomorrow morning. Been derailed too many times today | 22:38 |
| *** patriciadomin has joined #openstack-ironic | 22:38 | |
| jroll | I don't mind doing it, I can just roll through them all | 22:38 |
| TheJulia | jroll: up to you dude, I can happily just deal with some of it in the morning for bifrost :) | 22:39 |
| jroll | TheJulia: yeah, all good, I'll do it :) | 22:41 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add base cinder common interface https://review.openstack.org/355625 | 22:42 |
| JayF | jroll: link 'em up to me and I can be your review machine | 22:43 |
| * JayF gets his +2 stamp inked up | 22:43 | |
| jroll | JayF: patience | 22:43 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add Cinder storage driver https://review.openstack.org/366197 | 22:43 |
| * jroll not doing that now, it's far too late for work | 22:43 | |
| *** dhellmann has left #openstack-ironic | 22:43 | |
| JayF | jroll: wanna give me the tl;dr and I can post them? | 22:43 |
| * JayF looks for the ironic-ui commit | 22:44 | |
| jroll | JayF: https://github.com/openstack/ironic-ui/commit/5e28a1dc7378b3d8afa57e01d7f80f6eb1bdc892 | 22:44 |
| jroll | something to that effect | 22:44 |
| JayF | I'll take a stab at it, seems fairly simple | 22:44 |
| jroll | ok fiiine | 22:45 |
| jroll | all you people trying to find work :P | 22:46 |
| JayF | I just need stuff that's simple and doesn't require fast internet | 22:46 |
| jroll | heh | 22:46 |
| JayF | as I'm tethered from my new place, waiting for the new HVAC install to be done | 22:46 |
| jroll | have a good night | 22:49 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic master: Update release nodes for Ocata https://review.openstack.org/433320 | 22:53 |
| *** cdearborn has quit IRC | 22:54 | |
| *** vsaienko has quit IRC | 22:55 | |
| JayF | ^ hopefully that's what jroll wanted | 22:56 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Wire in storage interface attach/detach operations https://review.openstack.org/406290 | 23:02 |
| *** dhellmann has joined #openstack-ironic | 23:04 | |
| *** dhellmann has quit IRC | 23:05 | |
| *** srart has quit IRC | 23:05 | |
| *** gk--1wm- has joined #openstack-ironic | 23:05 | |
| *** gk--1wm- has left #openstack-ironic | 23:05 | |
| *** srart has joined #openstack-ironic | 23:09 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic master: Update release nodes for Ocata https://review.openstack.org/433320 | 23:18 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent master: Clean up release notes post-ocata release https://review.openstack.org/433329 | 23:18 |
| *** jtomasek has quit IRC | 23:31 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-inspector master: Update release notes for Ocata release https://review.openstack.org/433331 | 23:34 |
| *** jtomasek has joined #openstack-ironic | 23:35 | |
| *** g3ek has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!