| *** r-daneel has quit IRC | 00:01 | |
| *** srart has joined #openstack-ironic | 00:04 | |
| *** gcb has quit IRC | 00:05 | |
| *** robcresswell has quit IRC | 00:12 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 00:16 | |
| *** AlexeyAbashkin has quit IRC | 00:21 | |
| *** yangyapeng has quit IRC | 00:26 | |
| *** jkilpatr has quit IRC | 00:28 | |
| *** rbudden has quit IRC | 00:43 | |
| *** sambetts|afk has quit IRC | 00:54 | |
| *** sambetts_ has joined #openstack-ironic | 00:54 | |
| *** d0ugal has quit IRC | 00:55 | |
| *** lbragstad has quit IRC | 00:57 | |
| *** d0ugal has joined #openstack-ironic | 00:59 | |
| *** crushil has joined #openstack-ironic | 01:01 | |
| *** rloo has quit IRC | 01:03 | |
| *** phuongnh has joined #openstack-ironic | 01:06 | |
| *** yangyapeng has joined #openstack-ironic | 01:13 | |
| *** Goneri has quit IRC | 01:14 | |
| *** tiendc has joined #openstack-ironic | 01:17 | |
| *** cjloader has quit IRC | 01:26 | |
| *** hshiina has joined #openstack-ironic | 01:46 | |
| *** kiseok7_ has quit IRC | 01:55 | |
| *** kiseok7_ has joined #openstack-ironic | 01:56 | |
| *** kiseok7_ has quit IRC | 01:57 | |
| *** kiseok7_ has joined #openstack-ironic | 01:58 | |
| *** kiseok7_ has quit IRC | 01:59 | |
| *** kiseok7_ has joined #openstack-ironic | 02:01 | |
| *** kiseok7_ has quit IRC | 02:08 | |
| *** kiseok7_ has joined #openstack-ironic | 02:08 | |
| *** kiseok7_ has quit IRC | 02:08 | |
| *** yangyapeng has quit IRC | 02:08 | |
| *** yolanda has joined #openstack-ironic | 02:09 | |
| *** kiseok7_ has joined #openstack-ironic | 02:09 | |
| *** robcresswell has joined #openstack-ironic | 02:09 | |
| *** yangyapeng has joined #openstack-ironic | 02:13 | |
| *** lbragstad has joined #openstack-ironic | 02:14 | |
| *** harlowja has quit IRC | 02:27 | |
| *** ijw has quit IRC | 02:49 | |
| *** jaganathan has quit IRC | 02:57 | |
| *** hieulq has joined #openstack-ironic | 03:09 | |
| *** sasha2 has quit IRC | 03:13 | |
| *** jaganathan has joined #openstack-ironic | 03:14 | |
| *** ptx0 has quit IRC | 03:22 | |
| *** fragatina has quit IRC | 03:25 | |
| *** hieulq_ has joined #openstack-ironic | 03:25 | |
| *** ptx0 has joined #openstack-ironic | 03:26 | |
| *** phuongnh has quit IRC | 03:27 | |
| *** tiendc has quit IRC | 03:27 | |
| *** hieulq has quit IRC | 03:28 | |
| *** tiendc has joined #openstack-ironic | 03:32 | |
| *** phuongnh has joined #openstack-ironic | 03:32 | |
| *** wong has joined #openstack-ironic | 03:34 | |
| *** hshiina has quit IRC | 03:43 | |
| *** hshiina has joined #openstack-ironic | 03:47 | |
| *** fragatina has joined #openstack-ironic | 04:01 | |
| *** sapd_ has quit IRC | 04:02 | |
| *** sapd_ has joined #openstack-ironic | 04:02 | |
| *** links has joined #openstack-ironic | 04:02 | |
| *** fragatina has quit IRC | 04:05 | |
| *** links has quit IRC | 04:07 | |
| *** fragatina has joined #openstack-ironic | 04:10 | |
| *** fragatina has quit IRC | 04:11 | |
| *** links has joined #openstack-ironic | 04:19 | |
| *** tiendc has quit IRC | 04:28 | |
| *** tiendc has joined #openstack-ironic | 04:28 | |
| *** yolanda has quit IRC | 04:32 | |
| *** cjloader has joined #openstack-ironic | 04:34 | |
| *** cjloader has quit IRC | 04:36 | |
| *** cjloader has joined #openstack-ironic | 04:36 | |
| *** fragatina has joined #openstack-ironic | 04:45 | |
| *** jaganathan has quit IRC | 04:48 | |
| *** cjloader has quit IRC | 04:53 | |
| *** cjloader_ has joined #openstack-ironic | 04:55 | |
| *** dsneddon has quit IRC | 04:56 | |
| *** cjloader_ has quit IRC | 04:56 | |
| *** stendulker has joined #openstack-ironic | 04:59 | |
| *** dpawar has joined #openstack-ironic | 05:05 | |
| *** links has quit IRC | 05:06 | |
| *** links has joined #openstack-ironic | 05:08 | |
| *** cjloader has joined #openstack-ironic | 05:11 | |
| *** harlowja has joined #openstack-ironic | 05:12 | |
| *** harlowja has quit IRC | 05:13 | |
| *** cjloader has quit IRC | 05:20 | |
| *** cjloader has joined #openstack-ironic | 05:21 | |
| *** crushil has quit IRC | 05:21 | |
| *** pcaruana has joined #openstack-ironic | 05:24 | |
| *** yee379 has quit IRC | 05:25 | |
| *** yee379 has joined #openstack-ironic | 05:26 | |
| *** pcaruana has quit IRC | 05:36 | |
| *** amotoki has quit IRC | 06:03 | |
| *** amotoki has joined #openstack-ironic | 06:04 | |
| *** liuzz has joined #openstack-ironic | 06:08 | |
| *** rbartal has joined #openstack-ironic | 06:08 | |
| *** hshiina has quit IRC | 06:25 | |
| *** jaganathan has joined #openstack-ironic | 06:25 | |
| *** cjloader has quit IRC | 06:34 | |
| *** saop has joined #openstack-ironic | 06:39 | |
| *** jtomasek has joined #openstack-ironic | 06:40 | |
| *** makowals has joined #openstack-ironic | 06:50 | |
| *** etingof has quit IRC | 06:52 | |
| *** ionutbalutoiu has quit IRC | 06:53 | |
| *** ionutbalutoiu has joined #openstack-ironic | 06:54 | |
| *** marios has joined #openstack-ironic | 07:00 | |
| *** tuanla____ has joined #openstack-ironic | 07:01 | |
| *** etingof has joined #openstack-ironic | 07:05 | |
| *** mjura has joined #openstack-ironic | 07:07 | |
| *** rbartal has quit IRC | 07:07 | |
| *** e0ne has joined #openstack-ironic | 07:13 | |
| *** jrist has quit IRC | 07:15 | |
| *** fragatina has quit IRC | 07:18 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: Implementation for UEFI iSCSI boot for ILO https://review.openstack.org/468288 | 07:31 |
|---|---|---|
| *** pcaruana has joined #openstack-ironic | 07:33 | |
| *** e0ne has quit IRC | 07:34 | |
| *** saop_ has joined #openstack-ironic | 07:41 | |
| *** saop has quit IRC | 07:44 | |
| *** jrist has joined #openstack-ironic | 07:53 | |
| *** pcaruana has quit IRC | 07:58 | |
| *** rcernin has quit IRC | 08:00 | |
| *** natorious has quit IRC | 08:04 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 08:05 | |
| *** etingof has quit IRC | 08:07 | |
| *** tesseract has joined #openstack-ironic | 08:09 | |
| *** natorious has joined #openstack-ironic | 08:09 | |
| *** pcaruana has joined #openstack-ironic | 08:13 | |
| *** sapd_ has quit IRC | 08:16 | |
| *** sapd_ has joined #openstack-ironic | 08:16 | |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add support to bind type vlan networks https://review.openstack.org/532349 | 08:21 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add dsvm job https://review.openstack.org/530409 | 08:21 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Docs - Add multi-tenant devstack - n-g-s vlans https://review.openstack.org/534459 | 08:21 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Make the agent distributed using hashring and notifications https://review.openstack.org/524709 | 08:21 |
| *** tiendc has quit IRC | 08:38 | |
| *** tuanla____ has quit IRC | 08:38 | |
| *** phuongnh has quit IRC | 08:38 | |
| *** tiendc has joined #openstack-ironic | 08:38 | |
| *** phuongnh has joined #openstack-ironic | 08:39 | |
| *** tuanla____ has joined #openstack-ironic | 08:39 | |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Update proliantutils version for Queens release https://review.openstack.org/535895 | 08:46 |
| *** vmud213 has joined #openstack-ironic | 08:47 | |
| vmud213 | Morning Ironic! | 08:48 |
| *** dsneddon has joined #openstack-ironic | 08:53 | |
| *** priteau has joined #openstack-ironic | 09:02 | |
| *** athomas has joined #openstack-ironic | 09:06 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add rescue related methods to network interface https://review.openstack.org/509342 | 09:07 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 09:07 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Agent rescue implementation https://review.openstack.org/400437 | 09:07 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add API methods for [un]rescue https://review.openstack.org/350831 | 09:07 |
| pas-ha | morning ironic :) | 09:18 |
| *** rbartal has joined #openstack-ironic | 09:21 | |
| *** stendulker has quit IRC | 09:30 | |
| *** tesseract-RH has joined #openstack-ironic | 09:33 | |
| *** tesseract-RH has quit IRC | 09:34 | |
| *** tesseract-RH has joined #openstack-ironic | 09:35 | |
| *** tesseract has quit IRC | 09:37 | |
| *** derekh has joined #openstack-ironic | 09:39 | |
| *** etingof has joined #openstack-ironic | 09:48 | |
| *** e0ne has joined #openstack-ironic | 09:53 | |
| *** tuanla____ has quit IRC | 09:55 | |
| *** phuongnh has quit IRC | 09:55 | |
| *** phuongnh has joined #openstack-ironic | 09:56 | |
| *** tuanla____ has joined #openstack-ironic | 09:56 | |
| *** dtantsur|afk is now known as dtantsur | 09:58 | |
| *** stendulker has joined #openstack-ironic | 09:59 | |
| mgoddard | morning pas-ha & co | 10:02 |
| stendulker | morning mgoddard, pas-ha | 10:05 |
| vmud213 | dtantsur: Thanks for the review on https://review.openstack.org/#/c/535895/. After a rough test, I agree that it won't break the code. So giving workflow -1 | 10:10 |
| patchbot | patch 535895 - ironic - Update proliantutils version for Queens release | 10:10 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Testing multinode grenade fix https://review.openstack.org/536746 | 10:11 |
| *** vsaienk0 has joined #openstack-ironic | 10:12 | |
| dtantsur | vmud213: cool! thanks for investigating | 10:12 |
| dtantsur | stendulker: hey! I'm looking for a 2nd core to back my "classic drivers deprecations" FFE proposal. will you by chance have time to review it during the FF window? | 10:13 |
| stendulker | dtantsur: yes, i can do that. | 10:14 |
| dtantsur | stendulker: thanks! I'll put your name there then | 10:14 |
| stendulker | dtantsur: ok | 10:14 |
| openstackgerrit | paresh sao proposed openstack/ironic master: Implementation for UEFI iSCSI boot for ILO https://review.openstack.org/468288 | 10:18 |
| *** chas has joined #openstack-ironic | 10:18 | |
| *** marios has quit IRC | 10:30 | |
| *** sambetts_ is now known as sambetts | 10:32 | |
| sambetts | Morning all | 10:32 |
| *** marios has joined #openstack-ironic | 10:32 | |
| *** lucas-afk is now known as lucasagomes | 10:35 | |
| dtantsur | hey sambetts | 10:41 |
| *** e0ne has quit IRC | 10:51 | |
| sambetts | hey dtantsur | 10:57 |
| *** makowals has quit IRC | 10:59 | |
| *** stendulker has quit IRC | 11:00 | |
| *** AlexeyAbashkin has quit IRC | 11:06 | |
| *** e0ne has joined #openstack-ironic | 11:06 | |
| *** tuanla____ has quit IRC | 11:12 | |
| dtantsur | sambetts: mind reviewing https://review.openstack.org/#/c/460564/ please? this has been around since April, I'd like to get it in before the FF | 11:13 |
| patchbot | patch 460564 - ironic - Allow setting {provisioning,cleaning}_network in n... | 11:13 |
| sambetts | dtantsur: sure | 11:13 |
| hjensas | sambetts: Hi, if you could also look at topic:bug/1658964 that would be great. | 11:15 |
| *** saop_ has quit IRC | 11:16 | |
| *** saop_ has joined #openstack-ironic | 11:16 | |
| dtantsur | EEEHHHMMMMM.. did the multitenant job just self-heal? Oo I see it passing on https://review.openstack.org/#/c/509342/ | 11:18 |
| patchbot | patch 509342 - ironic - Add rescue related methods to network interface | 11:18 |
| *** vmud213 has quit IRC | 11:19 | |
| *** saop_ has quit IRC | 11:20 | |
| *** gmonteiro has joined #openstack-ironic | 11:20 | |
| gmonteiro | Good Morning Ironic | 11:20 |
| *** AlexeyAbashkin has joined #openstack-ironic | 11:23 | |
| *** phuongnh has quit IRC | 11:26 | |
| dtantsur | morning gmonteiro | 11:28 |
| gmonteiro | o/ dtantsur | 11:29 |
| *** tiendc has quit IRC | 11:31 | |
| gmonteiro | dtantsur: you can review this https://review.openstack.org/#/c/524310/ if you have time :) | 11:34 |
| patchbot | patch 524310 - ironic - Migrate oneview hardware type to use python-hpOneView | 11:34 |
| openstackgerrit | chen jiao proposed openstack/ironic master: remeber spelling error https://review.openstack.org/536781 | 11:35 |
| gmonteiro | I already had a review of julia and shivanand, I need it before the feature freeze on Thursday | 11:36 |
| *** e0ne has quit IRC | 11:41 | |
| *** e0ne has joined #openstack-ironic | 11:41 | |
| dtantsur | gmonteiro: will look today. please consider requesting an FFE just in case, I can support it. | 11:41 |
| *** e0ne has quit IRC | 11:42 | |
| gmonteiro | understood, thanks dtantsur | 11:42 |
| dtantsur | gmonteiro: commented, this is very close to landing | 11:49 |
| *** dtantsur is now known as dtantsur|brb | 11:50 | |
| *** tesseract-RH has quit IRC | 11:52 | |
| *** liuzz has quit IRC | 12:03 | |
| *** jkilpatr has joined #openstack-ironic | 12:13 | |
| *** vsaienk0 has left #openstack-ironic | 12:22 | |
| *** vsaienk0 has joined #openstack-ironic | 12:22 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Migrate oneview hardware type to use python-hpOneView https://review.openstack.org/524310 | 12:22 |
| vsaienk0 | dtantsur|brb: please review https://review.openstack.org/#/c/536746/ depends-on patches to devstack that will unblock multinode grenade, It failed on schedule to multiple nodes which is not related to nova-cpu stuck, we can fix it later separately | 12:22 |
| patchbot | patch 536746 - ironic - Testing multinode grenade fix | 12:22 |
| *** dtantsur|brb is now known as dtantsur | 12:23 | |
| dtantsur | vsaienk0: I think there are two parallel efforts to unblock this job, could you please sync with TheJulia? | 12:24 |
| *** jjohnson2 has joined #openstack-ironic | 12:25 | |
| vsaienk0 | dtantsur: I see she is working with jroll on microversion negotiation which is good, my patches are a bit different, they will work until negotiation is implemented | 12:28 |
| dtantsur | vsaienk0: well, the negotiation patch is already proposed, we just need nova cores on it | 12:29 |
| dtantsur | anyway, let's wait for either TheJulia or jroll for an up-to-date status | 12:29 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Devstack - Add ironic-neutron-agent https://review.openstack.org/530117 | 12:29 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add support to bind type vlan networks https://review.openstack.org/532349 | 12:29 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add dsvm job https://review.openstack.org/530409 | 12:29 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Docs - Add multi-tenant devstack - n-g-s vlans https://review.openstack.org/534459 | 12:29 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Make the agent distributed using hashring and notifications https://review.openstack.org/524709 | 12:29 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Use reporting_interval option from neutron https://review.openstack.org/536792 | 12:29 |
| vsaienk0 | dtantsur: ok, np if you think there is no need for that change in devstack I will just abandon them | 12:29 |
| dtantsur | vsaienk0: I'm not sure, that's why I suggest to wait | 12:42 |
| vsaienk0 | dtantsur: I will talk with TheJulia when she is around | 12:43 |
| *** milan_ has joined #openstack-ironic | 12:46 | |
| *** tesseract-RH has joined #openstack-ironic | 12:47 | |
| dtantsur | vsaienk0: btw, if you have a minute: https://review.openstack.org/#/c/460564/ | 12:48 |
| patchbot | patch 460564 - ironic - Allow setting {provisioning,cleaning}_network in n... | 12:48 |
| *** yangyapeng has quit IRC | 12:50 | |
| *** yangyapeng has joined #openstack-ironic | 12:51 | |
| *** jlabarre has joined #openstack-ironic | 12:54 | |
| *** tesseract-RH has quit IRC | 12:54 | |
| openstackgerrit | Michael Turek proposed openstack/ironic-python-agent master: Use lshw in place of dmidecode for the default hardware manager https://review.openstack.org/501799 | 12:55 |
| *** yangyapeng has quit IRC | 12:55 | |
| *** tesseract has joined #openstack-ironic | 12:55 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Allow data migrations to accept options https://review.openstack.org/535772 | 12:56 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Automatically migrate nodes to hardware types https://review.openstack.org/534373 | 12:56 |
| dtantsur | had to rebase ^^^ :( | 12:56 |
| *** zhurong has joined #openstack-ironic | 12:57 | |
| *** yolanda has joined #openstack-ironic | 13:05 | |
| dtantsur | also looking for a 2nd core review on https://review.openstack.org/535772 this is needed for hardware types migration | 13:06 |
| patchbot | patch 535772 - ironic - Allow data migrations to accept options | 13:06 |
| *** zhurong has quit IRC | 13:14 | |
| *** e0ne has joined #openstack-ironic | 13:14 | |
| *** vsaienk0 has quit IRC | 13:21 | |
| *** etingof has quit IRC | 13:21 | |
| *** cdearborn has joined #openstack-ironic | 13:21 | |
| *** vsaienk0 has joined #openstack-ironic | 13:22 | |
| *** dims has quit IRC | 13:22 | |
| *** alex_xu has quit IRC | 13:24 | |
| *** dims has joined #openstack-ironic | 13:25 | |
| *** MattMan has quit IRC | 13:25 | |
| *** MattMan has joined #openstack-ironic | 13:25 | |
| *** yangyapeng has joined #openstack-ironic | 13:26 | |
| *** alex_xu has joined #openstack-ironic | 13:27 | |
| *** jjohnson2 has quit IRC | 13:27 | |
| openstackgerrit | zhengyong proposed openstack/ironic-python-agent master: get IPv6 address of network interface in function get_interface_info https://review.openstack.org/536136 | 13:28 |
| *** SerenaFeng has joined #openstack-ironic | 13:30 | |
| *** rloo has joined #openstack-ironic | 13:32 | |
| openstackgerrit | zhengyong proposed openstack/ironic-python-agent master: get IPv6 address of network interface in function get_interface_info https://review.openstack.org/536136 | 13:32 |
| *** SerenaFeng has quit IRC | 13:32 | |
| *** jcoufal has joined #openstack-ironic | 13:32 | |
| *** SerenaFeng has joined #openstack-ironic | 13:34 | |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Devstack - use neutron segments (routed provider networks) https://review.openstack.org/531275 | 13:39 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Wait for ironic-neutron-agent to report state https://review.openstack.org/531637 | 13:39 |
| *** cjloader has joined #openstack-ironic | 13:43 | |
| * TheJulia rubs eyes | 13:44 | |
| *** dprince has joined #openstack-ironic | 13:46 | |
| rloo | Good morning everyone | 13:51 |
| dtantsur | morning rloo, TheJulia | 13:52 |
| * rloo passes a hot cup of coffee to TheJulia | 13:52 | |
| rloo | hi dtantsur | 13:52 |
| rloo | dtantsur: was there magic last night, to fix the grenade job? | 13:52 |
| dtantsur | I don't know, it seems to pass | 13:52 |
| rloo | dtantsur: are you ok if I +A this while we have this window of opportunity: https://review.openstack.org/#/c/353156/71 | 13:53 |
| patchbot | patch 353156 - ironic - Add rescuewait timeout periodic task | 13:53 |
| rloo | dtantsur: i'm good with a followup patch | 13:53 |
| rloo | dtantsur: normally i'd want them to fix but fixing that could take more than one revision | 13:53 |
| dtantsur | rloo: totally, let's fix it later | 13:53 |
| rloo | dtantsur: good | 13:54 |
| *** sasha2 has joined #openstack-ironic | 13:54 | |
| dtantsur | I'll have to rebase https://review.openstack.org/#/c/460564/ sigh... | 13:54 |
| patchbot | patch 460564 - ironic - Allow setting {provisioning,cleaning}_network in n... | 13:54 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: API: Node Traits API https://review.openstack.org/532269 | 13:54 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add node traits to API reference https://review.openstack.org/536384 | 13:54 |
| jroll | morning | 13:56 |
| dtantsur | rloo: if you have time after traits and rescue: https://review.openstack.org/535772 (may involve some bikeshedding) | 13:56 |
| patchbot | patch 535772 - ironic - Allow data migrations to accept options | 13:56 |
| dtantsur | morning jroll | 13:56 |
| jroll | dtantsur: I didn't think to update the whiteboard, sorry, I have a status | 13:56 |
| jroll | though now you say it's passing? | 13:56 |
| rloo | dtantsur: yup, on my list for today. along with the client patches | 13:56 |
| *** trown|outtypewww is now known as trown|ruck | 13:57 | |
| dtantsur | jroll: seems so! maybe someone else noticed the segfaults? | 13:57 |
| rloo | jroll: what is your status? | 13:57 |
| rloo | and good morning jroll :) | 13:57 |
| jroll | dtantsur: talked with dansmith yesterday, he is pretty -2 on changing upgrade order to work around this, him and matt were digging in a bit on the segv but didn't find anything last I checked | 13:57 |
| *** Goneri has joined #openstack-ironic | 13:58 | |
| rloo | jroll: so what is wrong with changing the upgrade order? wouldn't we want it to be flexible anyway, and isn't that going to happen with the version negotiation stuff anyway? | 13:58 |
| *** rbudden has joined #openstack-ironic | 13:58 | |
| *** etingof has joined #openstack-ironic | 13:59 | |
| jroll | rloo: making it flexible is fine, making it backwards for this cycle only, and only in some configurations is not. AIUI. | 13:59 |
| jroll | rloo: as this situation is a MUST change the order, not MAY | 13:59 |
| rloo | jroll: ha, so no hack/exceptions. i see. | 13:59 |
| jroll | basically, yes | 13:59 |
| rloo | jroll: i wondered about that. ok, so we need to try to get the version negotiation stuff landed, to avoid this problem in the future. | 14:00 |
| *** cjloader has quit IRC | 14:00 | |
| TheJulia | jroll: I commented on the review this morning, at least trying to provide reasoning why this is a good thing and helps us out | 14:01 |
| jroll | rloo: we do need to get that landed, but I don't think that's a good way to avoid the problem in the future (though helps work around it) | 14:01 |
| jroll | fwiw, my latest suspect was an `apt-get install` shortly before the segfaults which upgraded python-libvirt and some related things. so I don't have much of a clue on how to track down if there was an update there | 14:03 |
| jroll | I don't see anything immediately in nova (master or stable/pike) or g-r that look like a fix here | 14:03 |
| rloo | jroll: my thinking is that once some libraries are upgraded, you don't know what is going to happen. so each service should really be upgraded in a container | 14:04 |
| jroll | rloo: I 100% believe that people (and the gate) should always isolate python applications with virtualenvs, containers, VMs, etc. but good luck convincing OS-level packagers and the folks in charge of the gate :( | 14:05 |
| johnthetubaguy | Is this multi-node grenade, as I am curious if only running the services that are going to be upgraded on the second node would help? | 14:05 |
| *** SerenaFeng has quit IRC | 14:05 | |
| jroll | last time I tried to convince an OS packager, he told me to shove pip up my ass | 14:05 |
| *** links has quit IRC | 14:05 | |
| rloo | jroll: wow | 14:05 |
| johnthetubaguy | those folks really like following their rules, and I can see why I guess | 14:06 |
| jroll | johnthetubaguy: we upgrade most everything except nova atm | 14:06 |
| rloo | jroll: hey, why don't we put nova on the 2nd node -- the one that doesn't get upgraded? | 14:07 |
| johnthetubaguy | rloo: yeah, that's what I was meaning | 14:07 |
| jroll | right, I think that's more complicated than it seems on the surface, but it's certainly possible | 14:07 |
| johnthetubaguy | grenade only really works if you upgrade the whole thing | 14:08 |
| rloo | johnthetubaguy: good idea! :) | 14:08 |
| TheJulia | jroll: I feel like because it is an external influence, realistically outside of any of our ability to control, that the fix should just get landed because it provides a much better user experience, otherwise it is a hard nova-compute service break for users | 14:08 |
| johnthetubaguy | yeah, its not really modular like that in devstack, but it seems the only way, old node vs new node | 14:08 |
| rloo | yeah, so that's what i don't understand. do we really want a strict ordering of projects when upgrading? | 14:08 |
| johnthetubaguy | so jroll, I am +1 having flexibility, so I could re-apply my +2 on those groups | 14:08 |
| jroll | TheJulia: I'm not the one to convince :) | 14:08 |
| johnthetubaguy | s/groups/grounds/ | 14:09 |
| jroll | TheJulia: but, it's not about negotiation being the wrong thing to do, it's about using it as a hack and enforcing a strict upgrade order that's different than usual | 14:09 |
| TheJulia | jroll: my perception of their perception is not a great one right now... which might also lead into why openstack struggles discussions, but this is neither the place, time, nor is there beer to discuss this | 14:09 |
| *** e0ne has quit IRC | 14:10 | |
| rloo | ptg is coming up soon. question is whether we can wait til then to discuss. guess it depends on if grenade fails again due to this. | 14:10 |
| johnthetubaguy | we did agree a little while ago to fix the order services are upgrade, after some operator requests for an order, I think this is about keeping "the order" the same from release to release, unless you deprecated for a release, I guess? | 14:10 |
| jroll | TheJulia: for sure. | 14:10 |
| *** e0ne has joined #openstack-ironic | 14:10 | |
| johnthetubaguy | If my memory is right, I think the operators didn't care what the order was, as long as it worked, and stayed the same release to release | 14:11 |
| johnthetubaguy | well, I mean, consensus landed somewhere near that | 14:11 |
| johnthetubaguy | sucks though | 14:12 |
| *** melwitt has quit IRC | 14:12 | |
| jroll | so going back, can we talk about this | 14:13 |
| jroll | 13:52:44 dtantsur | I don't know, it seems to pass | 14:13 |
| jroll | nobody knows why? | 14:13 |
| jroll | is it still passing? | 14:13 |
| * TheJulia deletes what she was saying | 14:14 | |
| TheJulia | jroll: ci-watch says? | 14:14 |
| * TheJulia is still on cup #1 | 14:14 | |
| * jroll is still on scrollback | 14:14 | |
| * jroll looks | 14:14 | |
| *** melwitt has joined #openstack-ironic | 14:14 | |
| jroll | ci-watch says mostly passing, yes | 14:14 |
| jroll | wtf | 14:14 |
| *** lucasagomes is now known as lucas-hungry | 14:15 | |
| rloo | jroll: don't know if coincidence, but this looks like the first patch that it passed in: https://review.openstack.org/#/c/535928/2 | 14:16 |
| patchbot | patch 535928 - ironic - DNM testing whether lib/neutron switch breaks this... | 14:16 |
| TheJulia | johnthetubaguy: Eh, there was also negative feedback on of rocking the boat by trying to break plugins out of the built-ins... and that seemed to reach an impass. Besides, until the work is actually done, and teams are invested enough, or are forced to take it in-tree, we will just end up in the same catch-22 next cycle without the ability to be a little more friendly instead of break hard in some | 14:16 |
| TheJulia | situations. :( Really, in a sense, this all goes back to our hard python api pin | 14:16 |
| TheJulia | jroll: I will toss things at walls in support of the wtf moment, after I've had my second cup of coffee | 14:16 |
| jroll | rloo: innnnteresting. things happening during the neutron upgrade were suspect. | 14:17 |
| *** etingof has quit IRC | 14:17 | |
| jroll | alas, the dependencies did not land, so that isn't it | 14:17 |
| *** crushil has joined #openstack-ironic | 14:18 | |
| rloo | jroll: right, which is why i think it might be a coincidence. but maybe there is some other neutron-related/similar patch that fixed it. | 14:19 |
| jroll | could be, yeah | 14:19 |
| * jroll is still poking around | 14:20 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: Implementation for UEFI iSCSI boot for ILO https://review.openstack.org/468288 | 14:20 |
| jroll | timing is a bit off, but suspect none the less https://github.com/openstack-dev/devstack/commit/614cab33c40159f0bc10d92c9f8dc3f9783708d9 | 14:28 |
| rloo | jroll: you think the mirror was 'bad'? | 14:29 |
| jroll | rloo: more along the lines of some newer package in there that broke things | 14:29 |
| jroll | but just throwing that out there, more of a note while I search | 14:31 |
| rloo | jroll: that mirror has been in use for awhile, since oct or before. if that is true, that a newer package broke us, seems like it might be just a matter of time before that package appears again? | 14:33 |
| jroll | rloo: correct | 14:33 |
| jroll | like I said, just making a note of it, my confidence isn't high that it's related | 14:33 |
| rloo | jroll: thx for looking. i will focus on getting as much code landed as possible while this window of opportunity stays open | 14:34 |
| jroll | ++ | 14:35 |
| *** crushil has quit IRC | 14:38 | |
| *** crushil_ has quit IRC | 14:40 | |
| *** baha has joined #openstack-ironic | 14:44 | |
| *** fragatina has joined #openstack-ironic | 14:47 | |
| TheJulia | since it is passing as-is with some frequency, I guess we can go ahead and land rpc items with confidence now | 14:48 |
| dtantsur | aaaand https://review.openstack.org/532268 failed it :( | 14:49 |
| patchbot | patch 532268 - ironic - Add RPC object for traits | 14:49 |
| TheJulia | GAH | 14:49 |
| dtantsur | I did not check if it's the same failure though | 14:49 |
| TheJulia | I'll check in a moment | 14:49 |
| jroll | vsaienk0 mentioned a different failure case, right | 14:51 |
| openstackgerrit | Merged openstack/ironic-lib stable/pike: Use dd conv=sparse when writing images to nodes https://review.openstack.org/535307 | 14:52 |
| jroll | so I don't have a large sample size, but... | 14:52 |
| jroll | in pass AND fail cases, we start with libmysqlclient20 5.7.20-0ubuntu0.16.04.1 and libmysqlclient-dev 5.7.20-0ubuntu0.16.04.1 | 14:53 |
| jroll | in only the fail case, we upgrade to libmysqlclient20 5.7.21-0ubuntu0.16.04.1 and libmysqlclient-dev 5.7.21-0ubuntu0.16.04.1 | 14:53 |
| * jroll will check more cases | 14:53 | |
| vsaienk0 | jroll I was seen many times when default retries in nova are exhausted since we didn't finish upgrading ironic in time for example http://logs.openstack.org/42/536242/1/check/ironic-grenade-dsvm-multinode-multitenant/e6ad982/logs/screen-n-cpu.txt.gz | 14:54 |
| *** cjloader has joined #openstack-ironic | 14:54 | |
| jroll | vsaienk0: ah, got it | 14:54 |
| jroll | thanks | 14:54 |
| *** crushil_ has joined #openstack-ironic | 14:55 | |
| *** etingof has joined #openstack-ironic | 14:55 | |
| *** yolanda has quit IRC | 14:56 | |
| jroll | urgh, found a fail case where we don't upgrade those :/ | 14:56 |
| *** jjohnson2 has joined #openstack-ironic | 14:57 | |
| jroll | it could be anything, this feels like a wild goose chase | 14:59 |
| TheJulia | yup | 15:01 |
| jroll | gotta run for an hour+, bbl | 15:03 |
| *** arnewiebalck has joined #openstack-ironic | 15:03 | |
| *** dpawar has quit IRC | 15:05 | |
| TheJulia | dtantsur: completely unrelated to the other failures... 3 nova failures, looks like it goes back to the baremetal nodes being locked, so it could just be timing... It does have a lot of errors w/r/t vif detach attempts when there is an outstanding lock. :\ | 15:15 |
| *** vabada2 has joined #openstack-ironic | 15:19 | |
| *** vabada has quit IRC | 15:20 | |
| *** lucas-hungry is now known as lucasagomes | 15:20 | |
| dtantsur | moar failures! | 15:20 |
| *** links has joined #openstack-ironic | 15:21 | |
| rpioso | Good morning :) | 15:24 |
| *** vabada2 has quit IRC | 15:32 | |
| *** vabada has joined #openstack-ironic | 15:32 | |
| dtantsur | morning rpioso | 15:33 |
| * dtantsur is not entirely sure about the "good" bit | 15:33 | |
| rpioso | dtantsur: I hope your day improves. | 15:34 |
| dtantsur | thanks | 15:40 |
| TheJulia | dtantsur: fwiw, I fired off a recheck of my fix for the vif detachment | 15:43 |
| dtantsur | which one, sorry? | 15:43 |
| rpioso | dtantsur: yw | 15:44 |
| TheJulia | the one that moves the exclusive locking for detach into the actual logic of detachment dependent upon node state, since we only want to block the record being removed in an active (or now rescue) state | 15:44 |
| TheJulia | so since it is in deleting, the vif record can get ripped out | 15:45 |
| TheJulia | which will make nova much happier | 15:45 |
| TheJulia | and returns to the original pattern it experienced since .extra updates wouldn't be blocked as far as we remember/are aware | 15:46 |
| rloo | TheJulia: thx for looking into the grenade failure for 532268. I did a recheck. the inspector job timed out too :-( | 15:48 |
| rloo | good morning rpioso | 15:49 |
| dtantsur | TheJulia: good point | 15:49 |
| rpioso | rloo: Good morning | 15:50 |
| TheJulia | rloo: ugh | 15:50 |
| NobodyCam | Good Morning Ironic'ers | 15:56 |
| ricardoas | morning! | 15:57 |
| NobodyCam | Morning ricardoas :) | 15:58 |
| *** r-daneel has joined #openstack-ironic | 15:59 | |
| openstackgerrit | Merged openstack/ironic master: Add rescue related methods to network interface https://review.openstack.org/509342 | 16:00 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Switch the CI to hardware types https://review.openstack.org/536875 | 16:01 |
| dtantsur | morning NobodyCam | 16:01 |
| NobodyCam | Morning dtantsur :) | 16:02 |
| NobodyCam | happy not Monday :) | 16:02 |
| *** ni291187 has joined #openstack-ironic | 16:02 | |
| *** openstackgerrit has quit IRC | 16:03 | |
| *** etingof has quit IRC | 16:04 | |
| *** rbartal has quit IRC | 16:05 | |
| *** ni291187 has quit IRC | 16:06 | |
| ricardoas | morning NobodyCam :D | 16:09 |
| *** e0ne has quit IRC | 16:11 | |
| *** trown|ruck is now known as trown|brb | 16:11 | |
| ricardoas | hey dtantsur, I´ve just catch up with gmonteiro about opening a FFE for python lib change in oneview interfaces... thanks for the suggestion! I´m not sure I´ll be here to keep up with it the next cycle if we postpone it again... :( | 16:12 |
| gmonteiro | o/ NobodyCam | 16:12 |
| *** openstackgerrit has joined #openstack-ironic | 16:12 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Allow setting {provisioning,cleaning,rescuing}_network in driver_info https://review.openstack.org/460564 | 16:12 |
| dtantsur | vsaienk0, sambetts, had to rebase because of rescue_network ^^^ | 16:12 |
| dtantsur | ricardoas: I can imagine, it starts taking time | 16:13 |
| *** mjura has quit IRC | 16:15 | |
| *** SerenaFeng has joined #openstack-ironic | 16:16 | |
| *** trown|brb is now known as trown|ruck | 16:17 | |
| *** pc_m has quit IRC | 16:20 | |
| *** etingof has joined #openstack-ironic | 16:21 | |
| ricardoas | TheJulia: would you have any time to review the oneview stuff if we open a ffe? I ´m asking because you have already seen most part of it... | 16:21 |
| *** pc_m has joined #openstack-ironic | 16:22 | |
| *** tesseract has quit IRC | 16:22 | |
| openstackgerrit | Merged openstack/ironic master: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 16:25 |
| TheJulia | ricardoas: I kind of do | 16:29 |
| *** penick has joined #openstack-ironic | 16:29 | |
| *** jaganathan has quit IRC | 16:32 | |
| *** vsaienk0 has quit IRC | 16:34 | |
| dtantsur | pas-ha: hey, mind checking if the direction is right here? https://review.openstack.org/#/c/536875/ | 16:39 |
| patchbot | patch 536875 - ironic - [WIP] Switch the CI to hardware types | 16:39 |
| *** jaganathan has joined #openstack-ironic | 16:39 | |
| dtantsur | I think you introduced that devstack variable | 16:40 |
| ricardoas | TheJulia: thank you! | 16:41 |
| *** pcaruana has quit IRC | 16:41 | |
| pas-ha | dtantsur: looking | 16:46 |
| *** chas has quit IRC | 16:49 | |
| *** chas has joined #openstack-ironic | 16:49 | |
| pas-ha | dtantsur: +1, one nit in comment | 16:51 |
| dtantsur | yeah, good point. will update other jobs now | 16:51 |
| *** jaganathan has quit IRC | 16:52 | |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: Restores sushy session functionality. https://review.openstack.org/535910 | 16:53 |
| *** chas has quit IRC | 16:54 | |
| rpioso | etingof: Are you still working on https://review.openstack.org/#/c/526773/ ? | 16:56 |
| patchbot | patch 526773 - ironic - boot mode support at ManagementInterface | 16:56 |
| etingof | rpioso, yes, I do | 16:57 |
| NobodyCam | hey hey Morning gmonteiro :) | 16:57 |
| etingof | rpioso, the code is done, I'm fighting the mysterious CI job failure | 16:57 |
| *** jkilpatr_ has joined #openstack-ironic | 16:58 | |
| *** jkilpatr has quit IRC | 17:00 | |
| rpioso | etingof: Cool! I am reviewing it from the perspective of the drac driver. Also, I am working on adding UEFI support to that driver. Your change may align well with that. | 17:01 |
| etingof | rpioso, \o/ | 17:02 |
| etingof | rpioso, I hope to defeat the CI thing next week | 17:02 |
| etingof | rpioso, (got distracted by a local conference) | 17:02 |
| *** AlexeyAbashkin has quit IRC | 17:02 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Switch the CI to hardware types https://review.openstack.org/536875 | 17:04 |
| rpioso | etingof: My hope is to complete my review of that patch by tomorrow. No promises, though. I have been distracted by other upstream work, and there is more coming my way. | 17:04 |
| etingof | rpioso, thank you! | 17:05 |
| rpioso | etingof: Please wait until I submit my comments :) | 17:06 |
| etingof | rpioso, no worries, I can't work on that this week anyway | 17:06 |
| rpioso | etingof: Thank you for the update. Please stay tuned. | 17:07 |
| *** jaganathan has joined #openstack-ironic | 17:10 | |
| *** jkilpatr_ has quit IRC | 17:12 | |
| *** yolanda has joined #openstack-ironic | 17:17 | |
| *** gmonteiro has quit IRC | 17:20 | |
| *** cjloader has quit IRC | 17:20 | |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Devstack - use neutron segments (routed provider networks) https://review.openstack.org/531275 | 17:21 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Wait for ironic-neutron-agent to report state https://review.openstack.org/531637 | 17:21 |
| *** cjloader has joined #openstack-ironic | 17:21 | |
| *** baha has quit IRC | 17:22 | |
| * jroll returns | 17:27 | |
| *** anupn has quit IRC | 17:29 | |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Add routed networks support to flat interfaces https://review.openstack.org/536040 | 17:30 |
| *** amarao has quit IRC | 17:30 | |
| *** penick has quit IRC | 17:32 | |
| openstackgerrit | Tapio Tallgren proposed openstack/bifrost master: Rewrite iptables rules with Ansible module https://review.openstack.org/536911 | 17:34 |
| *** penick has joined #openstack-ironic | 17:36 | |
| *** akhilaki has joined #openstack-ironic | 17:39 | |
| *** akhilaki_ has joined #openstack-ironic | 17:39 | |
| *** makowals has joined #openstack-ironic | 17:39 | |
| *** yolanda has quit IRC | 17:47 | |
| *** trown|ruck is now known as trown|lunch | 17:48 | |
| *** etingof has quit IRC | 17:48 | |
| *** vsaienk0 has joined #openstack-ironic | 17:49 | |
| *** chas has joined #openstack-ironic | 17:53 | |
| *** vabada has quit IRC | 17:53 | |
| *** etingof has joined #openstack-ironic | 17:53 | |
| *** vabada has joined #openstack-ironic | 17:53 | |
| *** chas has quit IRC | 17:57 | |
| *** vsaienk0 has quit IRC | 17:58 | |
| *** derekh has quit IRC | 18:01 | |
| *** milan_ has quit IRC | 18:03 | |
| *** SerenaFeng has quit IRC | 18:04 | |
| *** ijw has joined #openstack-ironic | 18:04 | |
| *** dpawar has joined #openstack-ironic | 18:14 | |
| *** jkilpatr has joined #openstack-ironic | 18:20 | |
| *** baha has joined #openstack-ironic | 18:25 | |
| *** links has quit IRC | 18:26 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Clean up CI playbooks https://review.openstack.org/535896 | 18:29 |
| *** yolanda has joined #openstack-ironic | 18:29 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 18:31 | |
| *** vsaienk0 has joined #openstack-ironic | 18:33 | |
| *** lucasagomes is now known as lucas-afk | 18:35 | |
| *** AlexeyAbashkin has quit IRC | 18:35 | |
| *** links has joined #openstack-ironic | 18:40 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Deprecate classic drivers https://review.openstack.org/536928 | 18:42 |
| dtantsur | BOOOOOM | 18:42 |
| * dtantsur drops this mic and walks out | 18:42 | |
| *** vsaienk0 has quit IRC | 18:43 | |
| jroll | \o/ | 18:43 |
| *** vsaienk0 has joined #openstack-ironic | 18:44 | |
| rloo | encore, encore dtantsur! | 18:44 |
| dtantsur | :) | 18:45 |
| mgoddard | Classic drivers are dead. Long live classic drivers! | 18:45 |
| dtantsur | rloo: thanks for review on the options thingy. I'll give it a thought tomorrow - no energy left for today | 18:45 |
| rloo | dtantsur: no worries. there is room for enhancements, but if need be, we can push out something 'slim' now. | 18:46 |
| dtantsur | now, time to get some wine and do a premature celebration :D | 18:47 |
| rloo | dtantsur: enjoy! :) | 18:48 |
| *** fragatina has quit IRC | 18:48 | |
| dtantsur | thnx | 18:48 |
| *** makowals has quit IRC | 18:48 | |
| *** dtantsur is now known as dtantsur|afk | 18:48 | |
| *** fragatina has joined #openstack-ironic | 18:49 | |
| *** dpawar has quit IRC | 18:51 | |
| Tengu | hello there! | 18:53 |
| Tengu | anyone still up? would like to discuss ipmi in a libvirt env :). | 18:53 |
| Tengu | (and yep, it's kind of related to ironic - I'd like to manage VM for a tripleo lab and still use ironic) | 18:54 |
| Tengu | I know redhat has something on their formation laptop for that, but either I'm unable to provide the correct words to google, or it's maybe not really open | 18:55 |
| mgoddard | Tengu: tried virtualbmc? https://github.com/openstack/virtualbmc | 18:57 |
| Tengu | mgoddard: aaahhh. didn't see that. I think it will be the right one. Reading, thanks! | 18:58 |
| TheJulia | anyone seen anything like http://logs.openstack.org/68/532268/12/check/ironic-tempest-dsvm-ipa-wholedisk-agent_ipmitool-tinyipa-multinode/e026e4a/logs/screen-n-cond-cell1.txt.gz#_Jan_23_17_58_57_614336 ? | 18:59 |
| Tengu | mgoddard: is there a package? probably in delorean? | 18:59 |
| *** jkilpatr has quit IRC | 18:59 | |
| *** cjloader has quit IRC | 19:00 | |
| mgoddard | Tengu: at a guess, python-virtualbmc | 19:00 |
| *** cjloader has joined #openstack-ironic | 19:00 | |
| mgoddard | Tengu: otherwise, use pip/PyPI? | 19:01 |
| mgoddard | Tengu: http://markelov.blogspot.co.uk/2017/05/how-to-configure-openstack-tripleo.html | 19:01 |
| mgoddard | Tengu: python2-virtualbmc | 19:01 |
| Tengu | mgoddard: will check both in order to get the best of it :). | 19:02 |
| mgoddard | TheJulia: eww, that doesn't look good | 19:02 |
| TheJulia | no, it looks rather bad | 19:02 |
| Tengu | mgoddard: aahh, perfect. Will bookmark it and apply it tomorrow :). | 19:02 |
| NobodyCam | also could easily roll package your own with FPM: https://github.com/jordansissel/fpm | 19:03 |
| NobodyCam | wow that came out all wrong: | 19:04 |
| NobodyCam | you could also easily roll your own package with FPM :p | 19:04 |
| mgoddard | TheJulia: https://bugs.launchpad.net/nova/+bug/1733933 looks related | 19:05 |
| openstack | Launchpad bug 1736946 in OpenStack Compute (nova) ocata "duplicate for #1733933 Conductor: fails to clean up networking resources due to _destroy_build_request CantStartEngineError" [High,In progress] - Assigned to Matt Riedemann (mriedem) | 19:05 |
| mgoddard | TheJulia: and the bug that that is a dup of: https://bugs.launchpad.net/nova/+bug/1736946 | 19:05 |
| openstack | Launchpad bug 1736946 in OpenStack Compute (nova) ocata "Conductor: fails to clean up networking resources due to _destroy_build_request CantStartEngineError" [High,In progress] - Assigned to Matt Riedemann (mriedem) | 19:05 |
| TheJulia | fun | 19:10 |
| *** links has quit IRC | 19:19 | |
| *** vsaienk0 has quit IRC | 19:25 | |
| *** vsaienk0 has joined #openstack-ironic | 19:26 | |
| *** penick has quit IRC | 19:29 | |
| *** harlowja has joined #openstack-ironic | 19:34 | |
| TheJulia | ugh, more failed jobs | 19:35 |
| *** marios has quit IRC | 19:37 | |
| *** ricardoas is now known as ricardoas-away | 19:37 | |
| *** fragatina has quit IRC | 19:38 | |
| *** trown|lunch is now known as trown | 19:40 | |
| *** trown is now known as trown|ruck | 19:40 | |
| *** vsaienk0 has quit IRC | 19:49 | |
| *** chas has joined #openstack-ironic | 19:50 | |
| rloo | TheJulia: multi-grenade, or other jobs? | 19:50 |
| TheJulia | rloo: on one patch, two other jobs, but looks like network timeouts | 19:50 |
| openstackgerrit | Merged openstack/ironic master: remeber spelling error https://review.openstack.org/536781 | 19:51 |
| rloo | TheJulia: network timeouts are better than weird code things... sounds like the usual gate issues :-( not surprising, given the deadlines coming up | 19:51 |
| *** akhilaki_ has quit IRC | 19:52 | |
| *** akhilaki has quit IRC | 19:52 | |
| *** chas has quit IRC | 19:54 | |
| hjensas | sambetts: still around? Do you have a bug number for the one you mentioned on the flat driver patch? | 19:58 |
| *** dsneddon has quit IRC | 20:00 | |
| *** dsneddon has joined #openstack-ironic | 20:01 | |
| sambetts | hjensas: I'm not sure if there was one created or not, I can't seem to find it, however the bug reveals itself if you run openstack and don't run a neutron-OVS-agent service on the host that is running nova-compute with the ironic virt driver, when we try to bind the port it uses the nova compute host id, and neutron rejects the request with a no host matches that host id error | 20:03 |
| sambetts | hjensas: and even with things set up correctly today, if you inspect a neutron flat network ironic port it isn't ever successfuly bound and therefore in state "ACTIVE" | 20:05 |
| hjensas | sambetts: ah, yes got it. I'll put something in the fixes part of releasenotes. | 20:05 |
| sambetts | hjensas: thats the reason we created the networking-baremetal mech driver in the first place :) | 20:05 |
| sambetts | cool thanks :D | 20:06 |
| * sambetts --> away | 20:06 | |
| sambetts | o/ cya'll tomorrow | 20:06 |
| hjensas | sambetts: no, thank you! :) | 20:06 |
| *** sambetts is now known as sambetts|afk | 20:06 | |
| *** penick has joined #openstack-ironic | 20:08 | |
| *** e0ne has joined #openstack-ironic | 20:13 | |
| TheJulia | rloo: easy stable core review https://review.openstack.org/#/c/535474 | 20:13 |
| patchbot | patch 535474 - sushy (stable/queens) - Update UPPER_CONSTRAINTS_FILE for stable/queens | 20:13 |
| rloo | TheJulia: ok, easy is nice for a change :) | 20:13 |
| TheJulia | rloo: another https://review.openstack.org/#/c/535453/1 | 20:15 |
| *** e0ne_ has joined #openstack-ironic | 20:15 | |
| patchbot | patch 535453 - ironic-lib (stable/queens) - Update UPPER_CONSTRAINTS_FILE for stable/queens | 20:15 |
| *** vsaienk0 has joined #openstack-ironic | 20:16 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic-tempest-plugin master: Add node trait tests https://review.openstack.org/535908 | 20:18 |
| *** e0ne has quit IRC | 20:19 | |
| *** e0ne has joined #openstack-ironic | 20:20 | |
| *** e0ne_ has quit IRC | 20:21 | |
| *** e0ne has quit IRC | 20:21 | |
| rloo | TheJulia: how many of these patches do we *need* to get merged by Thursday, vs how many do we want? Trying to figure out priority/which to review | 20:23 |
| rloo | TheJulia: https://review.openstack.org/#/c/531029/ | 20:23 |
| patchbot | patch 531029 - python-ironicclient - Facilitate latest Rest API use | 20:23 |
| openstackgerrit | Merged openstack/sushy master: Update reno for stable/queens https://review.openstack.org/535475 | 20:25 |
| TheJulia | rloo: I'm basically ready to WF+1 the entire traits stack if I see a multinode job succeed somewhere at or past rpc changes :) | 20:26 |
| TheJulia | I think... last I looked at rescue, it was not staged such to prevent a merge conflict with traits | 20:27 |
| rloo | TheJulia: heh. that makes sense though. | 20:27 |
| rloo | TheJulia: oh, there is a conflict. i think.both have new Node versions if I recall. | 20:27 |
| TheJulia | but, I've been so heads down with 5000000 things and house purchase that.... ugh | 20:27 |
| rloo | TheJulia: i will check, can't keep track of what has landed in each of them. | 20:27 |
| TheJulia | rloo: indeed, and they were both claiming 1.37 as their microversion | 20:27 |
| TheJulia | granted, rescue can be changed fast | 20:28 |
| TheJulia | if we land traits | 20:28 |
| rloo | TheJulia: oh yeah. that too! but there will be a conflict after one of them lands. | 20:28 |
| TheJulia | hell, we should just land it and sort it later.... | 20:28 |
| TheJulia | if we're going to, we need to start letting the patches land very very very soon like yesterday | 20:28 |
| rloo | TheJulia: which patches? there are too many things to try to land this week | 20:30 |
| *** hw_wutianwei has quit IRC | 20:31 | |
| *** hw_wutianwei has joined #openstack-ironic | 20:32 | |
| TheJulia | traits rpc I guess | 20:32 |
| jroll | rloo: TheJulia: I can help with whichever doesn't land | 20:32 |
| TheJulia | You know... it is under the hood, the important thing is the API. we've got +2's all the way through the traits api at this point | 20:32 |
| TheJulia | we can fix under the hood next week | 20:32 |
| rloo | TheJulia: well, we can +A and see what happens then. | 20:33 |
| rloo | i don't know if many patches are passing CI now though. i'm guessing issues in infra-land. | 20:33 |
| TheJulia | looks like they are taking one of the clouds out now | 20:34 |
| rloo | jroll: you can help with testing the traits stuff. although i need you to update the client patch for traits. | 20:35 |
| *** penick has quit IRC | 20:36 | |
| *** vsaienk0 has quit IRC | 20:36 | |
| jroll | rloo: yep, I saw that and will be doing it. what testing does it need, just general poking the api? | 20:36 |
| *** penick has joined #openstack-ironic | 20:36 | |
| rloo | jroll: make sure rolling upgrades doesn't break (although I"m fairly sure it doesn't) and yes, API. cuz changing API behaviour after the fact is a mess wrt being backwards compatible etc. | 20:37 |
| TheJulia | I think mgoddard already poked at it a couple days ago | 20:37 |
| TheJulia | well, poked at the api | 20:37 |
| jroll | I poked at it a ton to write the client. not the latest version of the series, but it hasn't changed significantly. | 20:37 |
| jroll | I can do some more and lay a vote on it today | 20:37 |
| * jroll has been busy with other things today so will be working late | 20:38 | |
| *** akhilaki_ has joined #openstack-ironic | 20:38 | |
| *** akhilaki__ has joined #openstack-ironic | 20:38 | |
| rloo | i think we're good then. mark has some simple tempest tests, he just updated that and i'll review today unless i forget to do it. | 20:38 |
| TheJulia | given the node object upgrade/downgrade I'm not _as_ worried about about rolling upgrades | 20:39 |
| TheJulia | rloo: lets wf+1, and if it rolling upgrades testings has issues, we can revert or fix in place | 20:39 |
| *** jkilpatr has joined #openstack-ironic | 20:39 | |
| TheJulia | It will likely be sometime tomorrow before the api could merge anyway | 20:39 |
| rloo | TheJulia: ok | 20:39 |
| rloo | TheJulia: you want to pull the trigger? | 20:40 |
| TheJulia | sure | 20:40 |
| TheJulia | checking one thing and will context swifth to that | 20:40 |
| jroll | woohoo | 20:40 |
| jroll | will update the client shortly | 20:40 |
| rloo | TheJulia: I checked traits vs rescue, wrt conflicts in versions. Only the REST API version conflicts. Rescue's node & RPC versions have already merged, and traits has rebased on top of those. | 20:44 |
| TheJulia | \o/ | 20:44 |
| TheJulia | We might need to prod https://review.openstack.org/#/c/532268/12 again | 20:46 |
| patchbot | patch 532268 - ironic - Add RPC object for traits | 20:46 |
| TheJulia | since it failed, I rechecked it a little while ago | 20:46 |
| TheJulia | but it has wf+1 for now, so if the universe aligns and is happy, it will merge | 20:46 |
| mgoddard | woop, thanks TheJulia & rloo :D | 20:49 |
| jroll | \o/ | 20:50 |
| *** cdearborn_ has joined #openstack-ironic | 20:50 | |
| rloo | jlvillal: if you are around and have your handy script to recheck ^^ https://review.openstack.org/#/c/532268/ | 20:51 |
| patchbot | patch 532268 - ironic - Add RPC object for traits | 20:51 |
| jlvillal | rloo, Sure! | 20:51 |
| rloo | thx jlvillal! | 20:52 |
| jlvillal | rloo, running now in a tmux session. | 20:52 |
| rloo | jlvillal: :) | 20:53 |
| TheJulia | I'm stepping away for a little while and then need to run some city near palm springs to go initiate a wire transfer | 20:53 |
| TheJulia | I'll check back in periodically | 20:54 |
| rloo | TheJulia: thx | 20:54 |
| *** dsneddon has quit IRC | 20:58 | |
| *** dsneddon has joined #openstack-ironic | 20:58 | |
| *** vsaienk0 has joined #openstack-ironic | 20:59 | |
| *** cdearborn has quit IRC | 21:01 | |
| *** Goneri has quit IRC | 21:06 | |
| *** penick has quit IRC | 21:09 | |
| *** penick has joined #openstack-ironic | 21:10 | |
| *** dsneddon has quit IRC | 21:12 | |
| *** dsneddon has joined #openstack-ironic | 21:13 | |
| *** vsaienk0 has quit IRC | 21:13 | |
| *** jcoufal has quit IRC | 21:14 | |
| *** saop has joined #openstack-ironic | 21:14 | |
| saop | TheJulia, Hello | 21:14 |
| TheJulia | Greetings saop | 21:14 |
| saop | TheJulia, I want to raise https://review.openstack.org/#/c/468288/ this patch for FFE. Could you be core-reviewer for this? | 21:15 |
| patchbot | patch 468288 - ironic - Implementation for UEFI iSCSI boot for ILO | 21:15 |
| *** vsaienk0 has joined #openstack-ironic | 21:15 | |
| TheJulia | saop: happy to | 21:16 |
| saop | TheJulia, Thanks | 21:16 |
| TheJulia | btw, I comments to it about an hour ago | 21:16 |
| saop | TheJulia, I will soon update your comments :) | 21:16 |
| TheJulia | Thanks | 21:16 |
| *** penick has quit IRC | 21:16 | |
| *** saop has quit IRC | 21:17 | |
| TheJulia | note to self: the cats really don't like it when fighter jets buzz the RV park | 21:19 |
| *** penick has joined #openstack-ironic | 21:24 | |
| *** penick has quit IRC | 21:27 | |
| *** penick has joined #openstack-ironic | 21:29 | |
| jroll | rloo: hey, quick question before I upload the other changes https://review.openstack.org/#/c/532622/5/ironicclient/osc/v1/baremetal_node.py@1602 | 21:33 |
| patchbot | patch 532622 - python-ironicclient - Traits support | 21:33 |
| rloo | which one am i looking at? | 21:34 |
| rloo | L1602? | 21:34 |
| jroll | yes | 21:35 |
| *** chas has joined #openstack-ironic | 21:35 | |
| rloo | jroll: damn. it is a 'list'. | 21:35 |
| jroll | heh, both options are a reasonable way to render a list | 21:36 |
| rloo | jroll: i wonder if we made things complicated. what does listing tags return? now i need to think about it. | 21:36 |
| jroll | we allow up to 50, so I think the comma-separated thing could look weird at that point | 21:37 |
| rloo | jroll: strictly speaking, i think 'list' means list each one separately. | 21:37 |
| rloo | jroll: i need to look at our existing commands and see what we do | 21:37 |
| jroll | rloo: we don't have any other commands that return a list of strings | 21:37 |
| jroll | or even s/commands/api endpoints/ | 21:37 |
| *** dprince has quit IRC | 21:38 | |
| *** MaxPC has quit IRC | 21:38 | |
| jroll | tags would, but it isn't done | 21:38 |
| rloo | jroll: but we have commands that return lists. i think they all return 'lists', right? | 21:38 |
| jroll | yes, list of objects that have one or more keys | 21:38 |
| rloo | jroll: we have commands that return list of drivers. that is a list, right? | 21:38 |
| jroll | e.g. list vifs is [{'id': 'foo'}, ...], so we label the column id | 21:38 |
| jroll | driver list has two columns | 21:39 |
| rloo | jroll: i think the idea with vifs is that at some point, there could be more than just the 'id' as pieces of info about a vif. | 21:39 |
| *** chas has quit IRC | 21:39 | |
| jroll | driver list: https://gist.github.com/jimrollenhagen/dadf410a44625c1feeb6063a5800265c | 21:39 |
| rloo | jroll: with traits, what do we return in the API response? I've already forgotten. | 21:39 |
| jroll | rloo: {'traits': ['foo', 'bar']} | 21:40 |
| jroll | rloo: everything else is a list of dicts, in the API. {'drivers': [{'name': 'foo', 'hosts': []}, ...]} | 21:41 |
| jroll | etc. | 21:41 |
| rloo | jroll: ok, we need to look at OSC and see what it returns | 21:41 |
| jroll | rloo: what do you mean? what OSC returns for what? | 21:41 |
| rloo | jroll: for lists of things | 21:41 |
| *** penick has quit IRC | 21:41 | |
| rloo | jroll: this is the client response that we're discussing right? | 21:41 |
| jroll | rloo: I wrote it in my comment | 21:41 |
| rloo | jroll: right, we need to pick one of those two choices, right? | 21:42 |
| jroll | yes. | 21:42 |
| jroll | rloo: I guess you want me to find another service that returns a list of strings from its API, and see what it does in osc? | 21:42 |
| rloo | jroll: yup | 21:43 |
| TheJulia | on a plus side, things are looking a little better for more recent changes. https://review.openstack.org/#/c/536928/ seems to be the last job that is full of red | 21:43 |
| patchbot | patch 536928 - ironic - Deprecate classic drivers | 21:43 |
| rloo | jroll: although i was going to look, so better if you volunteer :) | 21:43 |
| jroll | rloo: okay, I'm gonna go read API docs for a while then | 21:43 |
| jroll | s/read/skim/ | 21:43 |
| jroll | or rather, figure out how to make nova tags work | 21:43 |
| *** penick has joined #openstack-ironic | 21:43 | |
| rloo | jroll: thx! just add a comment saying you chose option X because it is consistent with whatever. | 21:43 |
| rloo | TheJulia: good news! we can hope :) | 21:44 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Fix traits in Node._convert_to_version when pinned https://review.openstack.org/536978 | 21:44 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add traits field to node notifications https://review.openstack.org/536979 | 21:44 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add documentation for node traits https://review.openstack.org/536980 | 21:44 |
| mgoddard | rloo, TheJulia, jroll: Mixed news | 21:46 |
| mgoddard | ^^^ new patches for docs & notifications | 21:46 |
| mgoddard | testing with pinned services has revealed some issues | 21:46 |
| TheJulia | sigh | 21:47 |
| mgoddard | one of which is easily fixed in the first of the three patches | 21:47 |
| *** fragatina has joined #openstack-ironic | 21:47 | |
| rloo | mgoddard: that isn't good. | 21:47 |
| rloo | mgoddard: just when i thought i was done with reviewing traits patches :) | 21:47 |
| *** jjohnson2 has quit IRC | 21:48 | |
| mgoddard | rloo: never! | 21:48 |
| * TheJulia goes and removes magical flags | 21:49 | |
| TheJulia | okay, wf flags removed from rpc changes | 21:49 |
| jroll | :( | 21:51 |
| mgoddard | TheJulia: thanks :( | 21:52 |
| rloo | mgoddard: is https://review.openstack.org/#/c/536978/ based on top of your other patches? | 21:52 |
| patchbot | patch 536978 - ironic - Fix traits in Node._convert_to_version when pinned | 21:52 |
| jroll | rloo: sooo, nova instance tags aren't in osc :/ | 21:52 |
| jroll | rloo: I don't see any other tags either, except images, which are only listed as part of the image object | 21:53 |
| jroll | think we're on our own here | 21:53 |
| mgoddard | rloo: yes. Is that going to allow us to avoid a rev of the others? | 21:53 |
| rloo | mgoddard: well, we need that change in https://review.openstack.org/#/c/532268 | 21:53 |
| patchbot | patch 532268 - ironic - Add RPC object for traits | 21:53 |
| *** baha has quit IRC | 21:53 | |
| *** trown|ruck is now known as trown|outtypewww | 21:54 | |
| rloo | mgoddard: sorry, we shouldn't land 532268 w/o that. given that it hasn't merged yet. if our testing was really robust, it should fail w/o that fix, but i suspect it doesn't. | 21:55 |
| rloo | jroll: give me 10? min to think about it | 21:55 |
| jroll | rloo: sure, no problem | 21:55 |
| jroll | rloo: I'm going to upload the rest in case you agree with me :) | 21:55 |
| rloo | jroll: sure. it is cheap to do revisions :D | 21:56 |
| mgoddard | rloo: ok, I'll apply it to 532268 | 21:57 |
| TheJulia | k | 21:57 |
| rloo | thx mgoddard. if you can do it today, we can +2 (maybe +A) and see how it goes. | 21:58 |
| openstackgerrit | Jim Rollenhagen proposed openstack/python-ironicclient master: Traits support https://review.openstack.org/532622 | 21:58 |
| *** vsaienk0 has quit IRC | 22:00 | |
| *** Goneri has joined #openstack-ironic | 22:03 | |
| *** priteau has quit IRC | 22:03 | |
| *** crushil_ has quit IRC | 22:06 | |
| TheJulia | mgoddard: you indicated that there was a plurality to the issues... beyond the first. | 22:07 |
| TheJulia | Is it just the one issue? | 22:08 |
| rloo | jroll: went with your option #1 | 22:09 |
| mgoddard | TheJulia: yes, there's a second that I'm currently looking into | 22:09 |
| jroll | rloo: ++, thanks | 22:09 |
| rloo | jroll: thank YOU! :) | 22:10 |
| TheJulia | mgoddard: ack, thanks! | 22:11 |
| *** rcernin has joined #openstack-ironic | 22:15 | |
| mgoddard | rloo: my second issue is here: https://review.openstack.org/#/c/532268/12/ironic/objects/node.py@403 | 22:16 |
| patchbot | patch 532268 - ironic - Add RPC object for traits | 22:16 |
| *** penick has quit IRC | 22:16 | |
| rloo | mgoddard: so traits was in updates? | 22:17 |
| mgoddard | rloo: when the services are pinned, we can be passed traits to node.save() | 22:17 |
| rloo | mgoddard: oh, yes. if we are saving in the new conductor | 22:17 |
| mgoddard | rloo: yes, I think they are added in _convert_to_version | 22:17 |
| rloo | mgoddard: thinking. | 22:18 |
| rloo | mgoddard: do we check in the API? I think so | 22:18 |
| rloo | mgoddard: so we can delete that if statement | 22:18 |
| rloo | mgoddard: or change it to check the value of traits in updates. | 22:19 |
| rloo | mgoddard: i think there is a similar check in the db layer | 22:20 |
| *** milan has joined #openstack-ironic | 22:21 | |
| *** penick has joined #openstack-ironic | 22:21 | |
| rloo | mgoddard: i can't keep track of the changes/ versions. I suppose we can do a similar check to L344 in https://review.openstack.org/#/c/532268/12/ironic/objects/node.py | 22:24 |
| patchbot | patch 532268 - ironic - Add RPC object for traits | 22:24 |
| *** akhilaki__ has quit IRC | 22:24 | |
| *** akhilaki_ has quit IRC | 22:24 | |
| rloo | mgoddard: if you want to raise an exception. i am thinking that if we check at API level, just remove it. | 22:26 |
| mgoddard | rloo: the test was with both services new, and pinned to pike | 22:28 |
| mgoddard | we do check in the API | 22:28 |
| mgoddard | I don't see a check in the DB layer | 22:28 |
| mgoddard | relaxing the check to 'is not None' is looking promising | 22:28 |
| rloo | mgoddard: yeah, sorry, i thought it was in the db layer but it wasn't. i was thinking of the create() case at the object layer | 22:29 |
| *** penick has quit IRC | 22:29 | |
| mgoddard | rloo: I think I'm going to go with an 'is not None' check. Should have a patch up shortly | 22:32 |
| mgoddard | rloo: then bedtime :) | 22:32 |
| rloo | mgoddard: thx! | 22:33 |
| rloo | mgoddard: what if it is [] | 22:33 |
| rloo | mgoddard: you allow empty trait lists in the create() | 22:34 |
| rloo | mgoddard: can probably use same code in a helper method | 22:35 |
| mgoddard | rloo: convert_to_version sets it to None. An empty list in the updates dict could be interpreted as the new value | 22:35 |
| mgoddard | I guess None could also | 22:35 |
| rloo | mgoddard: we check at the API layer so it shouldn't happen anyway | 22:35 |
| mgoddard | rloo: right | 22:36 |
| *** fragatina has quit IRC | 22:50 | |
| *** fragatina has joined #openstack-ironic | 22:50 | |
| rpioso | jroll: May a dynamic driver have a management interface that is None? | 22:51 |
| rpioso | jroll: My read of the code is that is can't, but I would like to confirm that. | 22:51 |
| jroll | rpioso: I don't believe so, no, but not 100% sure | 22:52 |
| jroll | actually, not sure any interface may be None for a driver | 22:52 |
| rloo | rpioso: i think management is a core interface. sec. | 22:52 |
| jroll | rather, they would be no-* or noop or whatever | 22:52 |
| rloo | rpioso: oh, management is not a core, it is a standard interface. | 22:53 |
| rloo | rpioso: what jroll sez. | 22:53 |
| rpioso | rloo: Right | 22:53 |
| jroll | I guess fake would be the closest: https://github.com/openstack/ironic/blob/master/setup.cfg#L120 | 22:53 |
| rpioso | jroll: https://github.com/openstack/ironic/blob/master/ironic/common/driver_factory.py#L295 | 22:54 |
| rpioso | jroll: And https://github.com/openstack/ironic/blob/master/ironic/common/driver_factory.py#L214 | 22:55 |
| jroll | oh my, this is lots to page into my brain | 22:55 |
| jroll | rpioso: it feels like there's a higher level question you're trying to answer :) | 22:56 |
| rpioso | jroll: I'm reviewing https://review.openstack.org/#/c/526773 | 22:57 |
| patchbot | patch 526773 - ironic - boot mode support at ManagementInterface | 22:57 |
| jroll | ah | 22:57 |
| rpioso | jroll: There's a review comment exchange about whether or not a dynamic driver can have no management interface. | 22:58 |
| jroll | so you're wondering if we need to check task.driver.management | 22:58 |
| rpioso | jroll: Yes, for dynamic drivers. | 22:58 |
| *** milan has quit IRC | 22:58 | |
| jroll | rpioso: yeah, got it. dtantsur|afk says there "with hardware types interfaces cannot be None." I'm pretty sure he's right, and he would know better than anyone | 23:00 |
| jroll | but there's a chance he's wrong, I guess | 23:00 |
| jroll | er, no, he's right, you're right looking at https://github.com/openstack/ironic/blob/master/ironic/common/driver_factory.py#L214 | 23:00 |
| rpioso | jroll: So the TODO that etingof added in response to dtantsur|afk's review comment needs to be tweaked. | 23:01 |
| jroll | https://youtu.be/pDxoj-tDDIU?t=127 | 23:02 |
| jroll | rpioso: yeah, I'd agree | 23:02 |
| rpioso | jroll: Cool. I figured he and you would know :) | 23:02 |
| rpioso | jroll: lol | 23:03 |
| rpioso | jroll: C'mon, admit it. It never left :) | 23:03 |
| jroll | rpioso: oh no, it did. I totally purged my brain of the driver internals. | 23:04 |
| jroll | (with good reason) | 23:04 |
| rpioso | jroll: Thank you for confirming. And my apologies for bringing back those memories. | 23:04 |
| jroll | heh, not a problem | 23:04 |
| *** cjloader has quit IRC | 23:05 | |
| *** ijw has quit IRC | 23:06 | |
| *** ijw has joined #openstack-ironic | 23:06 | |
| *** masber has quit IRC | 23:14 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add RPC object for traits https://review.openstack.org/532268 | 23:17 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add RPC API and conductor manager for traits https://review.openstack.org/535296 | 23:17 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: API: Node Traits API https://review.openstack.org/532269 | 23:17 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add node traits to API reference https://review.openstack.org/536384 | 23:17 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add traits field to node notifications https://review.openstack.org/536979 | 23:17 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add documentation for node traits https://review.openstack.org/536980 | 23:17 |
| mgoddard | rloo, TheJulia: let's try that again | 23:18 |
| mgoddard | pinning is looking much happier now | 23:19 |
| mgoddard | rloo, TheJulia: I'm heading to bed now. Thanks for your patience! | 23:20 |
| mgoddard | Good luck little patches... | 23:20 |
| rloo | thx mgoddard! | 23:20 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Flat networks use node.uuid when binding ports. https://review.openstack.org/536040 | 23:22 |
| *** chas has joined #openstack-ironic | 23:23 | |
| *** chas has quit IRC | 23:27 | |
| rpioso | mgoddard: Good nite! | 23:33 |
| *** rpioso is now known as rpioso|afk | 23:35 | |
| *** cdearborn_ has quit IRC | 23:42 | |
| *** oanson has quit IRC | 23:42 | |
| *** oanson has joined #openstack-ironic | 23:43 | |
| *** UForgotten has quit IRC | 23:55 | |
| *** UForgotten has joined #openstack-ironic | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!