| *** hshiina has joined #openstack-ironic | 00:00 | |
| *** zhurong has joined #openstack-ironic | 00:01 | |
| *** vsaienk0 has joined #openstack-ironic | 00:02 | |
| *** penick has joined #openstack-ironic | 00:07 | |
| *** rbudden has quit IRC | 00:12 | |
| *** vsaienk0 has quit IRC | 00:12 | |
| *** vsaienk0 has joined #openstack-ironic | 00:13 | |
| *** akhilaki_ has joined #openstack-ironic | 00:16 | |
| *** akhilaki has joined #openstack-ironic | 00:16 | |
| *** zhurong has quit IRC | 00:20 | |
| *** oanson has quit IRC | 00:22 | |
| *** oanson has joined #openstack-ironic | 00:24 | |
| *** yangyapeng has quit IRC | 00:25 | |
| *** yangyapeng has joined #openstack-ironic | 00:25 | |
| *** yangyapeng has quit IRC | 00:30 | |
| *** penick has quit IRC | 00:30 | |
| *** sasha2 has quit IRC | 00:32 | |
| *** penick has joined #openstack-ironic | 00:32 | |
| *** rbudden has joined #openstack-ironic | 00:42 | |
| *** liuzz has joined #openstack-ironic | 00:42 | |
| *** vsaienk0 has quit IRC | 00:48 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/bifrost master: Updated from global requirements https://review.openstack.org/537028 | 00:49 |
|---|---|---|
| *** vsaienk0 has joined #openstack-ironic | 00:55 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/537051 | 00:55 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/537052 | 00:55 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib master: Updated from global requirements https://review.openstack.org/537053 | 00:55 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements https://review.openstack.org/537054 | 00:56 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Updated from global requirements https://review.openstack.org/534935 | 00:56 |
| *** penick has quit IRC | 00:58 | |
| *** r-daneel has quit IRC | 01:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/networking-baremetal master: Updated from global requirements https://review.openstack.org/537078 | 01:05 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/networking-generic-switch master: Updated from global requirements https://review.openstack.org/537080 | 01:06 |
| *** phuongnh has joined #openstack-ironic | 01:11 | |
| *** chas has joined #openstack-ironic | 01:11 | |
| *** tiendc has joined #openstack-ironic | 01:12 | |
| *** akhilaki_ has quit IRC | 01:14 | |
| *** crushil has joined #openstack-ironic | 01:15 | |
| *** akhilaki has quit IRC | 01:16 | |
| *** chas has quit IRC | 01:16 | |
| *** yangyapeng has joined #openstack-ironic | 01:17 | |
| *** yangyapeng has quit IRC | 01:26 | |
| *** yangyapeng has joined #openstack-ironic | 01:26 | |
| *** early has quit IRC | 01:27 | |
| *** rbudden has quit IRC | 01:27 | |
| *** cjloader has joined #openstack-ironic | 01:29 | |
| *** early has joined #openstack-ironic | 01:30 | |
| *** Goneri has quit IRC | 01:30 | |
| *** oanson has quit IRC | 01:38 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/virtualbmc master: Updated from global requirements https://review.openstack.org/537200 | 01:41 |
| *** oanson has joined #openstack-ironic | 01:41 | |
| *** hieulq_ has quit IRC | 01:42 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Updated from global requirements https://review.openstack.org/534935 | 01:56 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: DNM: test rescue mode https://review.openstack.org/533187 | 01:57 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: DNM: standalone test rescue mode https://review.openstack.org/534751 | 01:58 |
| *** hieulq_ has joined #openstack-ironic | 01:59 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: DNM: standalone test rescue mode https://review.openstack.org/534751 | 02:01 |
| openstackgerrit | Jim Rollenhagen proposed openstack/python-ironicclient master: Traits support https://review.openstack.org/532622 | 02:06 |
| *** gcb has joined #openstack-ironic | 02:09 | |
| *** vsaienk0 has quit IRC | 02:09 | |
| openstackgerrit | Merged openstack/networking-generic-switch master: Reraise NGS exceptions seen during netmiko config https://review.openstack.org/530532 | 02:11 |
| *** ricardoas has joined #openstack-ironic | 02:12 | |
| *** ricardoas-away has quit IRC | 02:15 | |
| *** rloo has quit IRC | 02:18 | |
| *** cjloader has quit IRC | 02:18 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client master: Updated from global requirements https://review.openstack.org/534285 | 02:19 |
| *** harlowja has quit IRC | 02:20 | |
| *** vsaienk0 has joined #openstack-ironic | 02:21 | |
| 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 | 02:25 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/sushy-tools master: Updated from global requirements https://review.openstack.org/534059 | 02:25 |
| *** yolanda has quit IRC | 02:30 | |
| *** nicodemos has quit IRC | 02:39 | |
| *** ricardoas has quit IRC | 02:41 | |
| *** nicodemos has joined #openstack-ironic | 02:43 | |
| *** ricardoas has joined #openstack-ironic | 02:46 | |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Add support for RESCUE and UNRESCUE provision states https://review.openstack.org/408341 | 02:49 |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Add rescue_interface to node and driver https://review.openstack.org/517302 | 02:49 |
| openstackgerrit | yuan liang proposed openstack/ironic-python-agent master: Execute error in _detect_cna_card https://review.openstack.org/534690 | 02:54 |
| openstackgerrit | yuan liang proposed openstack/ironic-python-agent master: Execute error in _detect_cna_card https://review.openstack.org/534690 | 02:57 |
| *** chas has joined #openstack-ironic | 02:59 | |
| *** chas has quit IRC | 03:04 | |
| *** cjloader_ has joined #openstack-ironic | 03:08 | |
| *** crushil has quit IRC | 03:09 | |
| *** vsaienk0 has quit IRC | 03:11 | |
| *** tuanla____ has joined #openstack-ironic | 03:14 | |
| tuanla____ | Morning ironic | 03:14 |
| *** crushil_ has joined #openstack-ironic | 03:15 | |
| *** crushil__ has joined #openstack-ironic | 03:24 | |
| *** cjloader_ has quit IRC | 03:27 | |
| *** crushil_ has quit IRC | 03:28 | |
| *** fragatina has quit IRC | 03:32 | |
| *** jlabarre has quit IRC | 03:42 | |
| *** hshiina has quit IRC | 03:42 | |
| *** vsaienk0 has joined #openstack-ironic | 03:51 | |
| *** hshiina has joined #openstack-ironic | 03:54 | |
| *** nicodemos has quit IRC | 03:54 | |
| *** nicodemos has joined #openstack-ironic | 03:54 | |
| *** Goneri has joined #openstack-ironic | 03:58 | |
| *** chas has joined #openstack-ironic | 03:59 | |
| *** vsaienk0 has quit IRC | 04:00 | |
| *** tiendc has quit IRC | 04:00 | |
| *** tuanla____ has quit IRC | 04:00 | |
| *** tiendc has joined #openstack-ironic | 04:01 | |
| *** tuanla____ has joined #openstack-ironic | 04:01 | |
| *** crushil__ has quit IRC | 04:03 | |
| *** chas has quit IRC | 04:03 | |
| *** jaianshu has joined #openstack-ironic | 04:11 | |
| *** nicodemos has quit IRC | 04:13 | |
| *** ricardoas has quit IRC | 04:13 | |
| *** Goneri has quit IRC | 04:14 | |
| *** nicodemos has joined #openstack-ironic | 04:16 | |
| *** ricardoas has joined #openstack-ironic | 04:16 | |
| *** akhilaki_ has joined #openstack-ironic | 04:21 | |
| *** akhilaki has joined #openstack-ironic | 04:21 | |
| *** nicodemos has quit IRC | 04:33 | |
| *** tuanla____ has quit IRC | 04:33 | |
| *** tuanla____ has joined #openstack-ironic | 04:34 | |
| *** ricardoas has quit IRC | 04:35 | |
| *** nicodemos has joined #openstack-ironic | 04:38 | |
| *** vsaienk0 has joined #openstack-ironic | 04:40 | |
| *** ricardoas has joined #openstack-ironic | 04:41 | |
| *** masber has joined #openstack-ironic | 04:42 | |
| *** dpawar has joined #openstack-ironic | 04:49 | |
| *** chas has joined #openstack-ironic | 04:50 | |
| *** chas has quit IRC | 04:54 | |
| *** akhilaki_ has quit IRC | 04:54 | |
| *** akhilaki has quit IRC | 04:54 | |
| *** ijw has quit IRC | 05:09 | |
| *** vsaienk0 has quit IRC | 05:11 | |
| *** harlowja has joined #openstack-ironic | 05:19 | |
| *** links has joined #openstack-ironic | 05:32 | |
| *** vsaienk0 has joined #openstack-ironic | 05:36 | |
| *** vsaienk0 has quit IRC | 05:58 | |
| *** rcernin_ has joined #openstack-ironic | 06:03 | |
| *** rcernin has quit IRC | 06:03 | |
| *** vsaienk0 has joined #openstack-ironic | 06:06 | |
| *** fragatina has joined #openstack-ironic | 06:13 | |
| *** fragatina has quit IRC | 06:14 | |
| *** fragatina has joined #openstack-ironic | 06:14 | |
| *** rwsu has joined #openstack-ironic | 06:21 | |
| *** gcb has quit IRC | 06:22 | |
| *** ijw has joined #openstack-ironic | 06:26 | |
| *** ijw has quit IRC | 06:33 | |
| *** marios has joined #openstack-ironic | 06:37 | |
| *** threestrands has joined #openstack-ironic | 06:39 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Agent rescue implementation https://review.openstack.org/400437 | 06:40 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add API methods for [un]rescue https://review.openstack.org/350831 | 06:40 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add rescue interface field to node-related notifications https://review.openstack.org/502889 | 06:40 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Update description for config params of 'rescue' interface https://review.openstack.org/517919 | 06:40 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add documentation for rescue interface https://review.openstack.org/419606 | 06:40 |
| *** vsaienk0 has quit IRC | 06:41 | |
| *** vsaienk0 has joined #openstack-ironic | 06:42 | |
| *** hjensas has quit IRC | 06:46 | |
| *** threestrands has quit IRC | 06:49 | |
| openstackgerrit | Tapio Tallgren proposed openstack/bifrost master: Rewrite iptables rules with Ansible module https://review.openstack.org/536911 | 06:52 |
| *** makowals has joined #openstack-ironic | 06:53 | |
| openstackgerrit | Gao Fei proposed openstack/ironic master: Replace curly quotes with straight quotes https://review.openstack.org/537277 | 07:04 |
| *** mjura has joined #openstack-ironic | 07:08 | |
| *** pcaruana has joined #openstack-ironic | 07:10 | |
| *** rcernin has joined #openstack-ironic | 07:12 | |
| *** rcernin_ has quit IRC | 07:12 | |
| openstackgerrit | Kaifeng Wang proposed openstack/python-ironicclient master: Can not set portgroup mode as a number https://review.openstack.org/537283 | 07:17 |
| *** pcaruana has quit IRC | 07:21 | |
| *** pcaruana has joined #openstack-ironic | 07:22 | |
| openstackgerrit | yuan liang proposed openstack/ironic-python-agent master: Execute error in _detect_cna_card https://review.openstack.org/534690 | 07:25 |
| *** vsaienk0 has quit IRC | 07:27 | |
| *** vsaienk0 has joined #openstack-ironic | 07:29 | |
| *** e0ne has joined #openstack-ironic | 07:30 | |
| *** jaganathan has quit IRC | 07:31 | |
| *** stendulker has joined #openstack-ironic | 07:36 | |
| *** chas has joined #openstack-ironic | 07:36 | |
| *** rwsu has quit IRC | 07:41 | |
| *** jaganathan has joined #openstack-ironic | 07:44 | |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Flat networks use node.uuid when binding ports. https://review.openstack.org/536040 | 07:45 |
| *** jtomasek has quit IRC | 07:48 | |
| *** jtomasek has joined #openstack-ironic | 07:49 | |
| *** hjensas has joined #openstack-ironic | 07:52 | |
| *** trown|outtypewww has quit IRC | 07:53 | |
| *** trown has joined #openstack-ironic | 07:53 | |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Devstack - use neutron segments (routed provider networks) https://review.openstack.org/531275 | 07:55 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Wait for ironic-neutron-agent to report state https://review.openstack.org/531637 | 07:55 |
| *** e0ne has quit IRC | 08:01 | |
| *** pcaruana has quit IRC | 08:01 | |
| *** e0ne has joined #openstack-ironic | 08:01 | |
| *** skazi has quit IRC | 08:02 | |
| openstackgerrit | Tapio Tallgren proposed openstack/bifrost master: Rewrite iptables rules with Ansible module https://review.openstack.org/536911 | 08:07 |
| *** skazi has joined #openstack-ironic | 08:12 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 08:23 | |
| *** tesseract has joined #openstack-ironic | 08:27 | |
| *** vsaienk0 has quit IRC | 08:30 | |
| *** vsaienk0 has joined #openstack-ironic | 08:35 | |
| *** pcaruana has joined #openstack-ironic | 08:39 | |
| *** rcernin has quit IRC | 08:44 | |
| pas-ha | morning ironic ^) | 08:44 |
| *** pmannidi has quit IRC | 08:45 | |
| *** tesseract has quit IRC | 08:54 | |
| pas-ha | dtantsur|afk: unit tests on inspector/master are failing. | 08:54 |
| pas-ha | dtantsur|afk: reason - fresh bump of u-c of 'construct' lib. latest (2.8.22) version has backward-incompatible changes to exceptions presented (FieldError -> FormatFieldError and StreamError). | 08:56 |
| pas-ha | dtantsur|afk: what do we do? file a bug for construct hoping they fix it in next release and block 2.8.22 in g-r, or update our code and update g-r to be >= 2.8.22? | 08:57 |
| pas-ha | seems only inspector is actually dependent on it.. http://codesearch.openstack.org/?q=construct&i=nope&files=%5Erequirements.txt&repos= | 08:58 |
| pas-ha | so I'd go with option 2 (fix code and update g-r to require at latest 2.8.22) | 08:59 |
| *** jaianshu_ has joined #openstack-ironic | 09:03 | |
| *** tesseract has joined #openstack-ironic | 09:03 | |
| *** jaianshu has quit IRC | 09:05 | |
| openstackgerrit | ya.wang proposed openstack/ironic master: Add timeout for all HTTP requests via the requests library. https://review.openstack.org/537311 | 09:07 |
| *** harlowja has quit IRC | 09:07 | |
| *** etingof has quit IRC | 09:14 | |
| *** milan_ has joined #openstack-ironic | 09:15 | |
| pas-ha | dtantsur|afk: the breaking commit in 'construct' is https://github.com/construct/construct/commit/812753a79ad8590a563c9881c5ffd8c2125e04e2 | 09:25 |
| *** chas has quit IRC | 09:35 | |
| *** derekh has joined #openstack-ironic | 09:39 | |
| *** yangyapeng has quit IRC | 09:50 | |
| *** absubram has joined #openstack-ironic | 09:50 | |
| *** etingof has joined #openstack-ironic | 09:50 | |
| *** yangyapeng has joined #openstack-ironic | 09:51 | |
| *** amarao has joined #openstack-ironic | 09:52 | |
| *** absubram has quit IRC | 09:54 | |
| *** yangyapeng has quit IRC | 09:55 | |
| *** absubram has joined #openstack-ironic | 09:56 | |
| *** tuanla____ has quit IRC | 09:58 | |
| *** dtantsur|afk is now known as dtantsur | 10:19 | |
| dtantsur | morning ironic | 10:19 |
| dtantsur | pas-ha: I think we should at least start with the former | 10:19 |
| dtantsur | because then the stable branches are also broken, and we cannot bump requirements there | 10:20 |
| sambetts|afk | stable branches should be protected by upper-constraints | 10:20 |
| *** sambetts|afk is now known as sambetts | 10:20 | |
| dtantsur | pas-ha: in any case, could you please talk to the requirements/release team? | 10:20 |
| dtantsur | sambetts: only in our CI, not downstream | 10:20 |
| dtantsur | we hope that downstreams follow u-c, but who knows? | 10:21 |
| sambetts | they should do buts thats another story :/ looking at the construct thing this change was only in a patch release 2.8.21 to 2.8.22 so should not be breaking from their side | 10:24 |
| sambetts | so yeah I think they've just made a bad release and we should update g-r to block it | 10:24 |
| dtantsur | pas-ha: could you please talk to upstream then? | 10:26 |
| sambetts | wow... https://github.com/construct/construct/issues/428 apprently the dev for construct doesn't believe in backwards compatiblity or sem-ver... | 10:28 |
| *** jaianshu__ has joined #openstack-ironic | 10:29 | |
| dtantsur | oh damn, how did we end up using it?.. | 10:29 |
| dtantsur | we really need some rating system for dependencies. "1 star - does not work; 2 stars - works, but the author is immature and have no clue how real world works" | 10:30 |
| dtantsur | etc | 10:30 |
| sambetts | not sure.. but basically the attitude seems to be pin and you won't get broken | 10:30 |
| *** jaianshu_ has quit IRC | 10:32 | |
| *** lucas-afk is now known as lucasagomes | 10:33 | |
| dtantsur | sambetts: we need to plan getting away from this crap | 10:33 |
| *** rcernin has joined #openstack-ironic | 10:34 | |
| *** yangyapeng has joined #openstack-ironic | 10:39 | |
| sambetts | yeah... may be able to use the native python struct lib ... just looking | 10:39 |
| *** jaganathan has quit IRC | 10:39 | |
| *** phuongnh has quit IRC | 10:40 | |
| dtantsur | even if it's less convenient, at least it won't get broken at any moment with a resolution "lol your problems dudes" :) | 10:42 |
| dtantsur | sambetts: lemme know if you think it's doable to rewrite the same code in python or a saner library | 10:43 |
| *** jaganathan has joined #openstack-ironic | 10:51 | |
| *** AlexeyAbashkin has quit IRC | 10:53 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 10:54 | |
| *** yangyapeng has quit IRC | 10:58 | |
| *** liuzz has quit IRC | 10:59 | |
| *** yangyapeng has joined #openstack-ironic | 10:59 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: Implementation for UEFI iSCSI boot for ILO https://review.openstack.org/468288 | 11:05 |
| *** jaganathan has quit IRC | 11:08 | |
| *** absubram has quit IRC | 11:11 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Switch the CI to hardware types https://review.openstack.org/536875 | 11:11 |
| openstackgerrit | paresh sao proposed openstack/ironic master: Implementation for UEFI iSCSI boot for ILO https://review.openstack.org/468288 | 11:13 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Switch the CI to hardware types https://review.openstack.org/536875 | 11:15 |
| *** gmonteiro has joined #openstack-ironic | 11:17 | |
| *** racedo has joined #openstack-ironic | 11:17 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Clean up CI playbooks https://review.openstack.org/535896 | 11:17 |
| gmonteiro | Good morning Ironic | 11:17 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Deprecate classic drivers https://review.openstack.org/536928 | 11:18 |
| *** dtantsur is now known as dtantsur|brb | 11:19 | |
| *** sapd_ has quit IRC | 11:19 | |
| *** sapd_ has joined #openstack-ironic | 11:19 | |
| *** jaganathan has joined #openstack-ironic | 11:21 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Migrate python-oneviewclient validations to oneview hardware type https://review.openstack.org/524599 | 11:32 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add a timeout for powering on/off a node on oneview https://review.openstack.org/524624 | 11:32 |
| *** jkilpatr has quit IRC | 11:33 | |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Flat networks use node.uuid when binding ports. https://review.openstack.org/536040 | 11:36 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Devstack - use neutron segments (routed provider networks) https://review.openstack.org/531275 | 11:36 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Wait for ironic-neutron-agent to report state https://review.openstack.org/531637 | 11:36 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Add documentation for baremetal mech https://review.openstack.org/537353 | 11:36 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: [WIP] Add baremetal mechanism driver to gate jobs https://review.openstack.org/537354 | 11:36 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add baremetal neutron agent https://review.openstack.org/456235 | 11:38 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: start_flag = True, only first time, or conf change https://review.openstack.org/533707 | 11:38 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Switch from MechanismDriver to SimpleAgentMechanismDriverBase https://review.openstack.org/521838 | 11:38 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Use reporting_interval option from neutron https://review.openstack.org/536792 | 11:38 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Devstack - Add ironic-neutron-agent https://review.openstack.org/530117 | 11:38 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add support to bind type vlan networks https://review.openstack.org/532349 | 11:38 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add dsvm job https://review.openstack.org/530409 | 11:38 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Docs - Add multi-tenant devstack - n-g-s vlans https://review.openstack.org/534459 | 11:38 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Make the agent distributed using hashring and notifications https://review.openstack.org/524709 | 11:38 |
| openstackgerrit | Merged openstack/ironic master: Add RPC object for traits https://review.openstack.org/532268 | 11:38 |
| openstackgerrit | Merged openstack/networking-generic-switch master: Updated from global requirements https://review.openstack.org/537080 | 11:38 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Flat networks use node.uuid when binding ports. https://review.openstack.org/536040 | 11:43 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Add documentation for baremetal mech https://review.openstack.org/537353 | 11:43 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Devstack - use neutron segments (routed provider networks) https://review.openstack.org/531275 | 11:43 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Wait for ironic-neutron-agent to report state https://review.openstack.org/531637 | 11:43 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: [WIP] Add baremetal mechanism driver to gate jobs https://review.openstack.org/537354 | 11:43 |
| *** stendulker has quit IRC | 11:45 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Updated from global requirements https://review.openstack.org/537163 | 11:51 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Updated from global requirements https://review.openstack.org/534285 | 11:51 |
| *** jkilpatr has joined #openstack-ironic | 11:55 | |
| *** jaianshu__ has quit IRC | 11:56 | |
| openstackgerrit | Merged openstack/virtualbmc master: Updated from global requirements https://review.openstack.org/537200 | 11:59 |
| openstackgerrit | Merged openstack/sushy-tools master: Updated from global requirements https://review.openstack.org/534059 | 12:06 |
| openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/532436 | 12:06 |
| *** zhurong has joined #openstack-ironic | 12:10 | |
| *** vsaienk0 has quit IRC | 12:15 | |
| openstackgerrit | Gustavo Diniz Monteiro proposed openstack/ironic master: Remove python-oneviewclient from oneview hardware type. https://review.openstack.org/524729 | 12:47 |
| *** vsaienk0 has joined #openstack-ironic | 12:48 | |
| *** gmonteiro has quit IRC | 12:53 | |
| *** dpawar has quit IRC | 13:01 | |
| jroll | morning ironic | 13:05 |
| jroll | I have a thing this morning, I won't be around until 1700 utc or later, jfyi. happy to help with anything that needs some work once I'm back | 13:05 |
| pas-ha | dtantsur|brb: sambetts filed an issue with construct any way https://github.com/construct/construct/issues/453 | 13:09 |
| pas-ha | so, should we add <2.8.22 to g-r for now? | 13:10 |
| *** trown is now known as trown|ruck | 13:15 | |
| pas-ha | the construct author proposed to pull 2.8.22 out from PyPI, and move this change to a next major 2.9 release | 13:18 |
| pas-ha | I think we could add <2.9 and pin ===2.8.21 then (until we move out of consuming this lib) | 13:18 |
| *** tiendc has quit IRC | 13:21 | |
| *** hjensas has quit IRC | 13:23 | |
| pas-ha | dtantsur|brb: sambetts patch to requirements https://review.openstack.org/537378 | 13:25 |
| patchbot | patch 537378 - requirements - Block construct >= 2.9 and pin to 2.8.21 | 13:25 |
| *** MattMan has quit IRC | 13:26 | |
| *** MattMan has joined #openstack-ironic | 13:26 | |
| *** SerenaFeng has joined #openstack-ironic | 13:33 | |
| *** makowals has quit IRC | 13:34 | |
| *** makowals has joined #openstack-ironic | 13:35 | |
| *** makowals has quit IRC | 13:43 | |
| *** makowals has joined #openstack-ironic | 13:45 | |
| *** rloo has joined #openstack-ironic | 13:46 | |
| *** AlexeyAbashkin has quit IRC | 13:47 | |
| *** rcernin has quit IRC | 13:47 | |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: [WIP] Add baremetal mechanism driver to gate jobs https://review.openstack.org/537354 | 13:48 |
| *** hjensas has joined #openstack-ironic | 13:49 | |
| *** hjensas has quit IRC | 13:49 | |
| *** hjensas has joined #openstack-ironic | 13:49 | |
| *** jjohnson2 has joined #openstack-ironic | 13:50 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add traits field to node notifications https://review.openstack.org/536979 | 13:52 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add documentation for node traits https://review.openstack.org/536980 | 13:52 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Fix nits found in node traits https://review.openstack.org/537386 | 13:52 |
| rloo | good morning ironickers, mgoddard, dtantsur|brb, sambetts, pas-ha, TheJulia, and everyone else | 13:54 |
| *** dtantsur|brb is now known as dtantsur | 13:54 | |
| dtantsur | morning rloo | 13:54 |
| rloo | sambetts, dtantsur|brb: wrt construct -- would be worth opening a bug so we don't forget the reason why we want to get off it. that link was great (or not great) | 13:54 |
| dtantsur | pas-ha: great that they collaborate, at least. and thanks for tracking it. I think we should update g-r to exclude 2.8.22 and cap 2.9 for now | 13:55 |
| mgoddard | morning rloo & co | 13:56 |
| *** sasha2 has joined #openstack-ironic | 13:58 | |
| TheJulia | good morning | 13:58 |
| dtantsur | morning TheJulia, mgoddard | 13:59 |
| *** weshay|rover is now known as weshay|mtg | 14:00 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 14:00 | |
| *** Goneri has joined #openstack-ironic | 14:03 | |
| TheJulia | rloo: replied to your comments on python-ironicclient version list | 14:06 |
| rloo | TheJulia: thx | 14:06 |
| dtantsur | let's try to merge these two patches today, if possible | 14:06 |
| dtantsur | I'll get some food and will be available for review/discussion | 14:06 |
| *** krtaylor has quit IRC | 14:07 | |
| rloo | TheJulia: also, let me know about https://review.openstack.org/#/c/531029/, while i look at your comments in the other one | 14:08 |
| patchbot | patch 531029 - python-ironicclient - Facilitate latest Rest API use | 14:08 |
| *** bfournie has quit IRC | 14:08 | |
| *** bfournie has joined #openstack-ironic | 14:08 | |
| *** krtaylor has joined #openstack-ironic | 14:09 | |
| *** SerenaFeng has quit IRC | 14:10 | |
| rloo | TheJulia, dtantsur: I guess it is worth discussing. I am sorry about bikeshedding, but we're exposing 'negotiate' so I think it should be clear what that means and/or whether we want to expose that term to our users. | 14:12 |
| rloo | TheJulia, dtantsur: I don't actually see any negotiation, the client decides based on what the server supports and what the user indicated. | 14:12 |
| *** bfournie has quit IRC | 14:13 | |
| *** links has quit IRC | 14:15 | |
| rloo | TheJulia, dtantsur: how does novaclient (or any other OpenStack client) handle this? Are there any that do? | 14:16 |
| *** MattMan has quit IRC | 14:16 | |
| *** MattMan has joined #openstack-ironic | 14:16 | |
| *** hshiina has quit IRC | 14:17 | |
| *** zhurong has quit IRC | 14:18 | |
| *** cdearborn has joined #openstack-ironic | 14:18 | |
| TheJulia | rloo: most just default to latest avialable | 14:18 |
| TheJulia | we're the only client that put an explicit pin lock in, at least the last time I looked through clients | 14:19 |
| rloo | TheJulia: latest available supported by server? even if the client request is a lower version? | 14:19 |
| TheJulia | rloo: I'm not sure what you mean by "I don't actually see any negotiation"? The bulk of the logic was already there | 14:20 |
| TheJulia | rloo: it is an override at that point to what the server can support | 14:20 |
| *** gmonteiro has joined #openstack-ironic | 14:21 | |
| TheJulia | rloo: posted replies to latest | 14:22 |
| dtantsur | TheJulia, rloo, I feel like "negotiation" is a more or less established word for it | 14:22 |
| rloo | TheJulia: i didn't test your patches, just looked at them. So is the flow something like 1. user makes request, client issues request to server with specified (if any) version, server response is error -- cannot handle version or request, (negotiation started) client determines a better version, resends request? | 14:22 |
| dtantsur | rloo: the essence is: the client has some supported versions, the server has some supported versions, they negotiate on something that fits both the best | 14:23 |
| TheJulia | rloo: incorrect for latest or a list, we explicitly go ask the server up front in those cases | 14:23 |
| rloo | TheJulia, dtantsur: apologies, but I don't understand and it isn't documented -- we really should have had a spec for this. | 14:23 |
| TheJulia | but only when there is a request. | 14:23 |
| TheJulia | from there, then the rest of the negotiation logic applies | 14:23 |
| dtantsur | rloo: this is an attempt of formalizing it: https://review.openstack.org/532814 | 14:23 |
| patchbot | patch 532814 - api-wg - Add guideline on exposing microversions in SDKs | 14:23 |
| rloo | dtantsur: thx, let me read it | 14:24 |
| dtantsur | suggestions like "let us define the word 'negotiate' formally" are welcome :) | 14:24 |
| TheJulia | rloo: with that being said, also trying to get parts of that done as I have time/braincells in small chunks. The minimum viable we need, if nova will ever allow it to land, is a list of valid versions. | 14:24 |
| rloo | TheJulia, dtantsur: I understand that there is a need for this, i just want to make sure it is clear what the need/requirements are, and that these patches address them. but i cannot do that by just looking at the patches themselves. I am missing context. And I know, the deadline is tomorrow :-( | 14:25 |
| dtantsur | so, the API-SIG spec may provide some answers to "why" and "how", I hope | 14:26 |
| *** crushil has joined #openstack-ironic | 14:27 | |
| openstackgerrit | Merged openstack/ironic master: Migrate oneview hardware type to use python-hpOneView https://review.openstack.org/524310 | 14:29 |
| *** baha has joined #openstack-ironic | 14:31 | |
| *** cjloader has joined #openstack-ironic | 14:31 | |
| *** rbudden has joined #openstack-ironic | 14:31 | |
| *** jlabarre has joined #openstack-ironic | 14:31 | |
| TheJulia | rloo: greatest reason for this, have a facility to allow nova-compute to be upgraded in grenade tests.... | 14:31 |
| *** yolanda has joined #openstack-ironic | 14:32 | |
| rloo | TheJulia: :) I really blame it on the grenade infrastructure anyway! | 14:32 |
| *** rloo has quit IRC | 14:32 | |
| dtantsur | a few years pass, and maybe we'll be finally allowed to use ansible playbooks instead... | 14:33 |
| *** rloo has joined #openstack-ironic | 14:33 | |
| *** AlexeyAbashkin has quit IRC | 14:33 | |
| *** makowals has quit IRC | 14:33 | |
| *** cjloader has quit IRC | 14:34 | |
| *** lucasagomes is now known as lucas-hungry | 14:35 | |
| TheJulia | maybe, but even then we need high order execution control, and there will still always be concerns over their users peception of order where as our users already know, the only reason problems crop up is becaues of singular hard pins and functionality is tied to the microvesion | 14:36 |
| *** makowals has joined #openstack-ironic | 14:36 | |
| dtantsur | small update: I've just realized I won't make it for the next meeting - my train arrives after it starts | 14:37 |
| *** gmonteiro has left #openstack-ironic | 14:40 | |
| TheJulia | so either \o/ or /o\ | 14:40 |
| *** gmonteiro has joined #openstack-ironic | 14:40 | |
| *** gmonteiro has left #openstack-ironic | 14:40 | |
| *** dpawar has joined #openstack-ironic | 14:41 | |
| *** crushil has quit IRC | 14:42 | |
| *** bfournie has joined #openstack-ironic | 14:43 | |
| *** serlex has joined #openstack-ironic | 14:46 | |
| * dtantsur looks at zuul backlog in despair | 14:48 | |
| dtantsur | the day before feature freeze, isn't it? | 14:48 |
| rloo | dtantsur: yup :-( | 14:49 |
| *** jcoufal has joined #openstack-ironic | 14:50 | |
| rloo | dtantsur: ok, read the spec. i don't understand something, well, maybe more than one thing, but it has me stuck. L165 in https://review.openstack.org/#/c/532814/5/guidelines/sdk-exposing-microversions.rst | 14:50 |
| patchbot | patch 532814 - api-wg - Add guideline on exposing microversions in SDKs | 14:50 |
| *** jcoufal has quit IRC | 14:51 | |
| rloo | dtantsur: if 1.2 was specified, and it is supported, why would there be an exception? | 14:51 |
| dtantsur | rloo: the exception will be if the version is not supported | 14:51 |
| rloo | dtantsur: or rather, if 1.2 was specified and it was not supported, why would the exit msg be 'at least 1.2 is required'? | 14:51 |
| rloo | dtantsur: maybe that is just a typo thing. all you know is that 1.2 is not supported, right? | 14:52 |
| dtantsur | rloo: we also assume that versions don't have gaps. so if 1.2 is not supported, 1.3 etc are not either | 14:52 |
| dtantsur | "at least" means 1.3 would work too | 14:52 |
| dtantsur | but we don't have 1.2, 1.3 or any newer, only e.g. 1.1 | 14:53 |
| rloo | dtantsur: how do you know that though? that you don't have 1.3 or any newer? | 14:53 |
| rloo | dtantsur: you just know that 1.2 isn't supported | 14:53 |
| rloo | dtantsur: what if the min of the server is 1.5 | 14:53 |
| dtantsur | rloo: fair enough, this may be an unfortunate wording | 14:53 |
| dtantsur | but you get the idea: 1.2 is not there | 14:54 |
| rloo | dtantsur: ok, so the only thing we can say is that 1.2 is not supported, right? | 14:54 |
| dtantsur | rloo: yep. I'll update it in the next update of that spec | 14:54 |
| edleafe | dtantsur: I think it's safe to assume (and document) that versions should always be increasing | 14:54 |
| rloo | edleafe: that's true. but all you (I) know in that example, is that 1.2 is not supported. | 14:55 |
| edleafe | IOW, you should never have "we support 1.1, 1.3, 1.4..." and thus have 1.2 fail | 14:55 |
| rloo | edleafe: oh, that's interesting, cuz i was wondering about that. | 14:56 |
| edleafe | rloo: what I'm saying is you also know that any 1.x version greater than 1.2 is also not supported | 14:56 |
| rloo | edleafe: in the eg style of our requirements, where we can skip one or two or... :) | 14:56 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Switch the CI to hardware types https://review.openstack.org/536875 | 14:57 |
| rloo | edleafe: well, unless the server's min is 1.4, then 1.2 isn't supported but 1.4 is | 14:57 |
| edleafe | Now you can have 1.2 not supported, but support 2.0, 2.1, 3.0, etc | 14:57 |
| edleafe | rloo: ah, I was assuming that we already knew the min | 14:57 |
| rloo | edleafe: i am assuming nothing. except that i can read :) | 14:57 |
| edleafe | rloo: I can't always assume that for myself :) | 14:57 |
| rloo | edleafe: :D | 14:58 |
| *** jcoufal has joined #openstack-ironic | 14:58 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Clean up CI playbooks https://review.openstack.org/535896 | 14:58 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Deprecate classic drivers https://review.openstack.org/536928 | 14:58 |
| rloo | dtantsur: ok, now i'm at the 'list of versions', L212 | 14:59 |
| *** AlexeyAbashkin has joined #openstack-ironic | 15:00 | |
| rloo | dtantsur: if negotiated is 1.42 it barks. if it isn't 1.42 -- what version will be used, 1.0? | 15:00 |
| *** crushil has joined #openstack-ironic | 15:00 | |
| dtantsur | rloo: yes, meow executes with 1.0 then | 15:00 |
| *** dpawar has quit IRC | 15:00 | |
| rloo | dtantsur: oooohhhhh. | 15:01 |
| * dtantsur likes discussing microversions in terms of cute animals | 15:01 | |
| rloo | dtantsur: and if at L208, 1.0 is not supported but 1.42 is, would there be an exception? | 15:01 |
| dtantsur | rloo: maybe it will be easier if instead of 1.0 and 1.42 you think in terms of "pike" and "queens"? | 15:01 |
| *** mjturek has joined #openstack-ironic | 15:01 | |
| dtantsur | rloo: no, no exception, because 1.42 can be picked | 15:01 |
| rloo | dtantsur: it is ok, numbers or releases, but lets use versions since we aren't going actually support named versions right (please don't say yes) | 15:02 |
| dtantsur | lol | 15:02 |
| dtantsur | no, we aren't | 15:02 |
| rloo | dtantsur: ok, so L213 will always be true and L216 will never be executed. | 15:02 |
| dtantsur | rloo: if 1.42 is present - yes | 15:02 |
| rloo | dtantsur: so the list thing is used instead of the hardcoded, to get exactly one version. i mean, the hardcoded could be done, calling code could just loop through each version when calling catsclient.v1.get_client() | 15:04 |
| rloo | dtantsur: sorry, not to get exactly one version. to specify the only versions to be used. | 15:04 |
| *** dpawar has joined #openstack-ironic | 15:04 | |
| rloo | dtantsur: the list one is going to be hairy i think. to implement correctly. for all methods. | 15:05 |
| dtantsur | rloo: yeah, you can emulate the list of versions by looping through them from the highest to the lowest | 15:05 |
| dtantsur | rloo: not particularly. keep in mind that the negotiation is only done once | 15:05 |
| rloo | dtantsur: if your list has 10 versions, 1.2, 1.3, 1.4, ..., 1.18, 1.20, you get a client that can use all those versions. if .negotiated_api_version == 1.20, cat.bark(), else: cat.meow() uses which version? 1.18? | 15:07 |
| *** etingof has quit IRC | 15:07 | |
| TheJulia | rloo: supporting named versions in the api construct would be really mean for any 3rd party re-implementation of the api..... | 15:07 |
| dtantsur | rloo: you don't know without checking. the highest of these that the server supports. | 15:08 |
| rloo | TheJulia: i do NOT want to support that :) | 15:08 |
| TheJulia | dtantsur: actually, the negotiation, in theory, can be re-performed... I think | 15:09 |
| dtantsur | rloo: why? this is the most sane semantic IMO | 15:09 |
| TheJulia | The code path is at least friendly to it if memory serves | 15:09 |
| dtantsur | and this is exactly what nova needs of us | 15:10 |
| rloo | dtantsur: it seems to me that once a version is decided or negotiated (from list of versions), we're just going to use that version under the hood for all future calls. So in L213 to 217 -- client is going to use whatever negotiated version it has already decided. | 15:10 |
| dtantsur | rloo: yep | 15:10 |
| dtantsur | this is common for all approaches, except for specifying a version per method | 15:10 |
| *** yangyapeng has quit IRC | 15:11 | |
| rloo | dtantsur: ok, then that is really a fast way of looping via the hard-coded way of getting a client. | 15:11 |
| dtantsur | rloo: essentially | 15:11 |
| *** yangyapeng has joined #openstack-ironic | 15:12 | |
| rloo | dtantsur: so we'd have to modify the nova code to say if version 1.foo is supported, use configdrive. if version 1.bar is supported, go on vacation. etc. | 15:12 |
| TheJulia | rloo: exactly | 15:12 |
| dtantsur | rloo: right, this is the goal of this work | 15:12 |
| rloo | dtantsur: so we don't really need to land the 'list of versions' patch in this client. | 15:13 |
| dtantsur | rloo: why? | 15:13 |
| rloo | dtantsur: but we do need to land the 'beautiful client, what version have you decided to use' | 15:13 |
| TheJulia | rloo: actually, we do, because we need to define possible versions to use, since we can't constrain without a max_version and latest | 15:13 |
| rloo | dtantsur: cuz we can loop on the hardcoded version thing, to get the actual versoin. | 15:14 |
| dtantsur | rloo: yes, and we can use python-requests and through ironicclient away | 15:14 |
| dtantsur | but we try to help people doing things that they'll have to do every time | 15:14 |
| rloo | dtantsur: oh, i am not saying not to do it. just that it isn't urgent do to it for tomorrow. | 15:14 |
| dtantsur | rloo: if we don't want to switch nova now - yes | 15:15 |
| dtantsur | but then the first patch is not urgent either.. | 15:15 |
| rloo | dtantsur: we could still switch to nova, as long as we support the hardcoded approach. in nova code, we can get client by trying the versions of interest. | 15:15 |
| rloo | dtantsur: and provide a means to know what version the client is actually going to use | 15:16 |
| dtantsur | rloo: yep, and we can do it already, there is a patch by jroll. but it's ugly and error-prone, so we want to hide it inside of ironicclient | 15:16 |
| *** yangyapeng has quit IRC | 15:17 | |
| *** lucas-hungry is now known as lucas-brb | 15:17 | |
| rloo | dtantsur: are you talking about the patch that jroll submitted this week? | 15:17 |
| dtantsur | yep | 15:17 |
| *** e0ne has quit IRC | 15:18 | |
| *** e0ne has joined #openstack-ironic | 15:19 | |
| rloo | dtantsur: you mean we'd replace the _find_client_with_version() code in https://review.openstack.org/#/c/535786/5/nova/virt/ironic/client_wrapper.py? | 15:19 |
| patchbot | patch 535786 - nova - Ironic: negotiate microversion to allow downgrade ... | 15:19 |
| *** e0ne has quit IRC | 15:20 | |
| TheJulia | so, actually you can reach all the way into the http_client object directly and pull out os_ironic_api_version that has been re-negotiated, but I don't think there is any way to know that "yes, this has negotiated" at present | 15:21 |
| *** dpawar has quit IRC | 15:21 | |
| *** dpawar has joined #openstack-ironic | 15:22 | |
| TheJulia | I'll put it this way, the less code in nova, the better, since they really, to my perception, were downright hostile about us landing negotiation logic to make things better and allow us to just upgrade nova with ironic half-upgraded | 15:22 |
| rloo | TheJulia: if I understand it, we'd change _find_client_with_version (or not have it), by setting kwargs['os_ironic_api_version'] = ['1.34', '1.35']. | 15:23 |
| TheJulia | correct, and we can then just microversion guard the logic with the explicit version needed | 15:23 |
| rloo | TheJulia: i read Dan's response, and it makes sense. | 15:23 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Switch the CI to hardware types https://review.openstack.org/537415 | 15:23 |
| *** rpioso|afk is now known as rpioso | 15:24 | |
| -openstackstatus- NOTICE: gerrit has been suffering from a full disk, some mails may have been lost in the last couple of hours. we will now restart gerrit to address ongoing slowness, too | 15:24 | |
| rloo | TheJulia: wrt microversion guard, that's what jroll's patch does. we'd have to do that for any future stuff. which is ok with me. | 15:24 |
| rpioso | Good morning, ironicers. | 15:24 |
| TheJulia | rloo: was there one posted to the patch? because I only read the responses in irc | 15:24 |
| rloo | TheJulia: yup. | 15:24 |
| rloo | TheJulia: I didn't read/look at irc discussion | 15:25 |
| *** dpawar has quit IRC | 15:26 | |
| *** hjensas has quit IRC | 15:27 | |
| TheJulia | rloo: and we already have to touch the nova client code every time, but which is cleaner, a new version to a list, or exposed retry logic..... I think doing it in the library makes far more sense for a consumer. Granted, I have strong feelings in this regard because we were using the hard pin in shade and that contributed to my work on it stalling because I was in a catch 22 kind of situation with the | 15:27 |
| TheJulia | static pin. | 15:27 |
| TheJulia | the other advantage of handling it in the client is then we can properly downgrade if we hit an un-upgraded api server after the fact.... instead of possibly dying hard. | 15:29 |
| rloo | TheJulia: the 'exposed retry logic' that I am thinking of, if we don't land the version-list-part, is that the code loops through the list of versions from highest to lowest, to get the supported version. it isn't that difficult. | 15:29 |
| dtantsur | 20 lines of code vs 2, something like that | 15:29 |
| rloo | TheJulia: i am not disagreeing that we shouldn't support it in the client code. i am just saying that it isn't urgent to do for tomorrow. if the patch doesn't merge. | 15:29 |
| *** dprince has joined #openstack-ironic | 15:30 | |
| TheJulia | I would like it to merge, I'd like to fix the code in the nova virt driver, if nova doesn't want us to, then really it is all kind of pointless | 15:30 |
| *** cjloader has joined #openstack-ironic | 15:31 | |
| rloo | TheJulia: the changes in the client to 'get the version' is needed for tomorrow. i think with that, nova code can be changed. | 15:31 |
| TheJulia | The bottom line is not doing so, is just pushing a problem back which ultimately doesn't help anyone | 15:31 |
| TheJulia | rloo: no, it can't, because we still have an overriding static pin inside of our code | 15:31 |
| TheJulia | we _HAVE_ to try and negotiate on something that is not 1.9 | 15:32 |
| rloo | TheJulia: huh? well, probably not worth spending time in this discussion. i suspect it would be more fruitful to work on each patch to see how far we get. | 15:32 |
| TheJulia | and have the ability to fallback | 15:32 |
| *** Guest28399 is now known as mgagne | 15:33 | |
| *** mgagne has joined #openstack-ironic | 15:33 | |
| rloo | TheJulia: i don't see how we cannot do the above, w/o providing support for the user to specify a list of versions | 15:33 |
| *** cjloader has quit IRC | 15:33 | |
| rloo | TheJulia: it seems to me that if we land https://review.openstack.org/#/c/531029 we are good. | 15:33 |
| patchbot | patch 531029 - python-ironicclient - Facilitate latest Rest API use | 15:33 |
| rloo | TheJulia: but i could be wrong. I only started looking at this yesterday and you've been working on it for awhile. | 15:34 |
| *** cjloader has joined #openstack-ironic | 15:34 | |
| TheJulia | rloo: dmitry has comments early on in that patchset that might be impacting my position that we need both | 15:34 |
| TheJulia | rloo: I did also change the logic a little to fail out on maximum locally known, which might make it useful. | 15:35 |
| TheJulia | the downside of supporting latest is what if behavior changes in 1.39, and python-ironicclient is accidently upgraded | 15:36 |
| dtantsur | this ^^^ | 15:36 |
| TheJulia | if the client is locked in to support 1.33, 1.34, 1.35, then 1.39 is never possible | 15:36 |
| *** yolanda has quit IRC | 15:37 | |
| *** etingof has joined #openstack-ironic | 15:48 | |
| *** crushil has quit IRC | 15:50 | |
| *** SerenaFeng has joined #openstack-ironic | 15:50 | |
| *** crushil has joined #openstack-ironic | 15:51 | |
| TheJulia | looks like the oneview change that was in the gate is going to have to be rechecked | 15:51 |
| TheJulia | merge_failure | 15:51 |
| TheJulia | likely gerrit going down... maybe | 15:52 |
| *** e0ne has joined #openstack-ironic | 15:52 | |
| *** yangyapeng has joined #openstack-ironic | 15:53 | |
| mgoddard | hi dtantsur: remember this: https://git.openstack.org/cgit/openstack/nova/commit/?id=c92337bdf80fea4c0a8ebb433bacec4cc07f7a94 | 15:59 |
| mgoddard | dtantsur: oh, and this: v | 16:00 |
| mgoddard | https://git.openstack.org/cgit/openstack/nova/commit/?id=d25feca90ec4bad6ec9ececedced63b9f00b4c87 | 16:00 |
| dtantsur | mgoddard: hi, I do | 16:00 |
| mgoddard | dtantsur: currently seeing warnings because the node is reporting no inventory, so compute can't create allocations against the resource provider in placement | 16:01 |
| mgoddard | seen that before? | 16:01 |
| *** mjura has quit IRC | 16:01 | |
| dtantsur | not sure | 16:02 |
| dtantsur | on a meeting now, mind pasting your warnings? | 16:02 |
| NobodyCam | Good Morning Ironic'ers | 16:03 |
| mgoddard | dtantsur: http://paste.openstack.org/show/652292/ | 16:03 |
| mgoddard | dtantsur: it's not urgent | 16:03 |
| *** marios has quit IRC | 16:05 | |
| *** AlexeyAbashkin has quit IRC | 16:05 | |
| *** marios has joined #openstack-ironic | 16:05 | |
| rloo | good morning NobodyCam | 16:07 |
| NobodyCam | Moring rloo :) | 16:07 |
| rloo | dtantsur, TheJulia: I +2'd the first patch, 531029, but to be honest, i am uncomfortable giving it a +A. | 16:07 |
| *** mjturek has quit IRC | 16:07 | |
| rloo | dtantsur: so you can do the honours :) | 16:08 |
| dtantsur | heh :) will do, lemme finish with a meeting | 16:08 |
| *** mjturek has joined #openstack-ironic | 16:17 | |
| *** lucas-brb is now known as lucasagomes | 16:23 | |
| *** jaganathan has quit IRC | 16:35 | |
| *** pcaruana has quit IRC | 16:35 | |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Devstack - Add ironic-neutron-agent https://review.openstack.org/530117 | 16:38 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add support to bind type vlan networks https://review.openstack.org/532349 | 16:38 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add dsvm job https://review.openstack.org/530409 | 16:38 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Docs - Add multi-tenant devstack - n-g-s vlans https://review.openstack.org/534459 | 16:38 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Make the agent distributed using hashring and notifications https://review.openstack.org/524709 | 16:38 |
| *** e0ne has quit IRC | 16:40 | |
| *** e0ne has joined #openstack-ironic | 16:40 | |
| *** SerenaFeng has quit IRC | 16:40 | |
| *** radez has quit IRC | 16:44 | |
| *** hjensas has joined #openstack-ironic | 16:47 | |
| *** radez has joined #openstack-ironic | 16:49 | |
| *** jcoufal has quit IRC | 17:02 | |
| *** jcoufal has joined #openstack-ironic | 17:04 | |
| *** serlex has quit IRC | 17:05 | |
| *** akhilaki has joined #openstack-ironic | 17:05 | |
| *** akhilaki_ has joined #openstack-ironic | 17:05 | |
| *** jlabarre has quit IRC | 17:08 | |
| *** etingof has quit IRC | 17:13 | |
| *** hjensas has quit IRC | 17:14 | |
| TheJulia | rechecked 535296 since zuul never posted results and one of the jobs failed :( | 17:26 |
| TheJulia | one of the jobs that likes timing out failed.... | 17:26 |
| mgoddard | :( thanks TheJulia | 17:27 |
| openstackgerrit | Merged openstack/python-ironicclient master: Facilitate latest Rest API use https://review.openstack.org/531029 | 17:28 |
| *** yolanda has joined #openstack-ironic | 17:35 | |
| *** etingof has joined #openstack-ironic | 17:36 | |
| *** trown|ruck is now known as trown|outtypewww | 17:40 | |
| *** trown|outtypewww is now known as trown|lunch | 17:40 | |
| mjturek | dtantsur: fyi - I think I actually have the intermittent failure for building lshw fixed. I was doing 'touch lshw.1' when make builds the file to 'src/lshw.1' https://review.openstack.org/#/c/501799/ could you re-review? | 17:46 |
| patchbot | patch 501799 - ironic-python-agent - Use lshw in place of dmidecode for the default har... | 17:46 |
| dtantsur | mjturek: will do | 17:46 |
| mjturek | thx! | 17:46 |
| *** milan_ has quit IRC | 17:50 | |
| *** mjturek has quit IRC | 17:52 | |
| *** mjturek has joined #openstack-ironic | 17:52 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: [WIP][DNM] Some beginnings towards removing construct from inspector https://review.openstack.org/537472 | 17:57 |
| sambetts | dtantsur: ^^ | 17:58 |
| dtantsur | \o/ | 17:58 |
| dtantsur | will check after dinner | 17:58 |
| sambetts | dtantsur: seems like its not too difficult to recreate what we need from construct, but I've not been through all the cases we currently handle yet | 17:59 |
| dtantsur | cool | 18:00 |
| *** amarao has quit IRC | 18:03 | |
| *** derekh is now known as derekh_afk | 18:07 | |
| *** sambetts is now known as sambetts|afk | 18:08 | |
| *** hjensas has joined #openstack-ironic | 18:08 | |
| *** tesseract has quit IRC | 18:08 | |
| *** jlabarre has joined #openstack-ironic | 18:15 | |
| bfournie | dtantsur: just saw this ^, it looks like construct 2.8.22 is causing problems with CI? | 18:18 |
| *** vsaienk0 has quit IRC | 18:19 | |
| *** e0ne has quit IRC | 18:28 | |
| pas-ha | bfournie: yes, patch to requirements is https://review.openstack.org/#/c/537378/ | 18:31 |
| patchbot | patch 537378 - requirements - Block construct >2.8.21 | 18:31 |
| *** cjloader has quit IRC | 18:32 | |
| bfournie | pas-ha: thanks, is it worth fixing the exceptions for the issue introduced with 2.8.22? | 18:32 |
| *** cjloader has joined #openstack-ironic | 18:32 | |
| *** fragatina has quit IRC | 18:32 | |
| pas-ha | either we fix those - and still make a patch to requirements to require >=2.8.22, or cap it in requirements at 2.8.21 while sambetts is working on getting rid of it completely | 18:33 |
| pas-ha | inspector seems to be the only project in openstack/* depending o it anyway | 18:33 |
| bfournie | pas-ha: k, yeah I think it is, getting rid of it entirely by not be that easy though | 18:34 |
| * pas-ha -> home, have a good day|night all | 18:34 | |
| bfournie | pas-ha: good night | 18:34 |
| *** cjloader has quit IRC | 18:38 | |
| *** cjloader has joined #openstack-ironic | 18:38 | |
| *** mjturek has quit IRC | 18:52 | |
| *** marios has quit IRC | 18:52 | |
| *** mjturek has joined #openstack-ironic | 18:53 | |
| *** trown|lunch is now known as trown | 18:54 | |
| dtantsur | folks, can someone review https://review.openstack.org/#/c/460564/ please? this is loosely related to routed networks | 18:58 |
| patchbot | patch 460564 - ironic - Allow setting {provisioning,cleaning,rescuing}_net... | 18:58 |
| dtantsur | and it has been around since April, I've just updated it recently | 18:59 |
| dtantsur | wow this is weird http://logs.openstack.org/75/536875/5/check/ironic-tempest-dsvm-pxe_ipmitool-postgres/d72ee02/logs/devstacklog.txt.gz#_2018-01-24_16_27_06_388 | 19:00 |
| dtantsur | I wonder if we broken PostgreSQL with the recent traits patch | 19:01 |
| dtantsur | mgoddard: ^^^ | 19:01 |
| *** jtomasek has quit IRC | 19:05 | |
| *** lucasagomes is now known as lucas-afk | 19:10 | |
| *** nicodemos has quit IRC | 19:13 | |
| mgoddard | dtantsur: hmm, that does look traity | 19:19 |
| mgoddard | dtantsur: I'd guess it's this: https://review.openstack.org/#/c/532268/13/ironic/db/sqlalchemy/api.py@466 | 19:20 |
| patchbot | patch 532268 - ironic - Add RPC object for traits (MERGED) | 19:20 |
| rloo | mgoddard: this is where it seems to start failing: https://review.openstack.org/#/c/532268/8 | 19:21 |
| patchbot | patch 532268 - ironic - Add RPC object for traits (MERGED) | 19:21 |
| mgoddard | rloo, dtantsur: we should be able to do the joined query separate from the update. I'll investigate | 19:21 |
| rloo | thx mgoddard. sorry if this is going to be another long day for you. | 19:21 |
| dtantsur | thanks! | 19:21 |
| rloo | dtantsur: wrt your stuff, i'm guessing you're going to take off soon. and you're not around tomorrow? | 19:22 |
| dtantsur | rloo: yep, not much wifi coverage | 19:22 |
| rloo | dtantsur: if so, then we work on the classic driver migration stuff next week after you're back? | 19:22 |
| *** vsaienk0 has joined #openstack-ironic | 19:22 | |
| *** jkilpatr has quit IRC | 19:22 | |
| dtantsur | rloo: there are things to review, like https://review.openstack.org/536875 and the next patches | 19:23 |
| patchbot | patch 536875 - ironic - Switch the CI to hardware types | 19:23 |
| rloo | dtantsur: damn. do they have the same topic? | 19:23 |
| dtantsur | rloo: I will try addressing comments tomorrow and on Friday in the evening | 19:23 |
| dtantsur | rloo: did I ever mention that I'm horrible at gerrit topics? :) | 19:23 |
| rloo | dtantsur: https://review.openstack.org/#/q/topic:bug/1690185+(status:open+OR+status:merged) | 19:23 |
| rloo | dtantsur: 6 more patches? | 19:24 |
| dtantsur | oh, I'm not that bad :) | 19:24 |
| *** sasha2 has quit IRC | 19:24 | |
| dtantsur | yep, these 6 are all ready to review, I *think* the CI failures are unrelated | 19:24 |
| dtantsur | well, one needs an update, I'll do it around right now | 19:24 |
| rloo | dtantsur: ok, on my list to review them all, this week or Monday the latest. | 19:24 |
| dtantsur | thanks! | 19:25 |
| bfournie | dtantsur: posting fix for the ironic-inspector construct exception issue shortly, I don't think we need any workarounds or need to remove construct | 19:25 |
| *** sasha2 has joined #openstack-ironic | 19:25 | |
| bfournie | dtantsur: is there an upstream bug for that? I assume no | 19:26 |
| *** pcaruana has joined #openstack-ironic | 19:27 | |
| *** pcaruana has quit IRC | 19:27 | |
| dtantsur | bfournie: I'm not sure personally | 19:28 |
| *** e0ne has joined #openstack-ironic | 19:28 | |
| bfournie | dtantsur: adding one... | 19:28 |
| *** jlabarre has quit IRC | 19:30 | |
| *** muttley has joined #openstack-ironic | 19:32 | |
| *** vsaienk0 has quit IRC | 19:32 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 19:34 | |
| *** e0ne_ has joined #openstack-ironic | 19:35 | |
| *** jlabarre has joined #openstack-ironic | 19:36 | |
| *** jkilpatr has joined #openstack-ironic | 19:37 | |
| *** e0ne has quit IRC | 19:38 | |
| *** harlowja has joined #openstack-ironic | 19:38 | |
| *** AlexeyAbashkin has quit IRC | 19:38 | |
| *** fragatina has joined #openstack-ironic | 19:40 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector master: Remove use of construct lib FieldError exception https://review.openstack.org/537627 | 19:41 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Deprecate classic drivers https://review.openstack.org/536928 | 19:45 |
| dtantsur | okay, that should be it ^^^ | 19:46 |
| *** dtantsur is now known as dtantsur|afk | 19:46 | |
| dtantsur|afk | o/ | 19:46 |
| *** muttley has quit IRC | 19:48 | |
| *** athomas has quit IRC | 19:50 | |
| *** etingof has quit IRC | 19:53 | |
| *** etingof has joined #openstack-ironic | 20:07 | |
| rpioso | Should the base class definition of a new, optional method return a value or raise an UnsupportedDriverExtension exception? | 20:09 |
| rpioso | s/class/hardware interface class/ | 20:10 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Fix node update with PostgreSQL https://review.openstack.org/537637 | 20:13 |
| mgoddard | rloo: hopefully that should fix our postgres issue ^^^ | 20:14 |
| rloo | mgoddard: thx, will look in a few minutes. | 20:15 |
| rloo | mgoddard: i reviewed the client patch and it needs an update. if you feel like it. i think we can wait for jroll to update it later/tomorrow though. | 20:15 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-lib master: Add missing 'self' parameter to class methods https://review.openstack.org/537640 | 20:19 |
| *** e0ne_ has quit IRC | 20:19 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Allow API user to define list of versions https://review.openstack.org/531271 | 20:20 |
| TheJulia | is there a legitimate reason we are running cleaning in a multinode job? | 20:22 |
| TheJulia | I think we need to consider extending some timeouts at least, since the last attempt to land rpc for traits blew out because the nodes were still cleaning | 20:23 |
| TheJulia | rloo: ^^^ ? | 20:23 |
| TheJulia | jroll: if you happen to remember | 20:23 |
| rloo | TheJulia: i think we just need to run cleaning in *a* test | 20:24 |
| TheJulia | We have toooooo many long running jobs :( and on multinode... it is insane | 20:24 |
| TheJulia | because that spins.... ?5? vms per node | 20:24 |
| rloo | TheJulia: my guess is the multinode test is testing the same stuff the main grenade test. it should be sufficient to test cleaning in the single-node grenade test | 20:24 |
| rloo | TheJulia: i mean, if you're paranoid and a tester, you want to test EVERYTHING! | 20:25 |
| rloo | TheJulia: cuz who knows what might break. it's a black box. | 20:25 |
| rloo | TheJulia: you know, what if a node is being cleaned, the conductor gets shut down/upgraded, and another conductor inherits the node being cleaned and ... well, i know, we're not explicitly testing that :) | 20:26 |
| TheJulia | rloo: it is cleaning that is happening as part of initial stand-up | 20:27 |
| rloo | TheJulia: having said that, what is cleaning doing? I've already forgotten. We should just mock something, add a cleaning thing in ipa that just waits 1 minute or something before finishing. | 20:27 |
| rloo | TheJulia: we could just turn off cleaning via the configs, for multinode grenade. it is non voting now anyway, right? | 20:28 |
| TheJulia | this is not multinode grenade | 20:28 |
| TheJulia | it is just multinode | 20:28 |
| rloo | TheJulia: oh, sorry. i missed that. | 20:28 |
| TheJulia | no, my fault, I only said multinode | 20:28 |
| mjturek | I hate to ask, but should I be asking for a feature freeze exception on this patch? Is it too late? https://review.openstack.org/#/c/501799/ | 20:29 |
| patchbot | patch 501799 - ironic-python-agent - Use lshw in place of dmidecode for the default har... | 20:29 |
| rloo | mjturek: is it a feature or a bug? | 20:29 |
| rloo | mjturek: looks like a bug. no need for ffe | 20:29 |
| mjturek | rloo: is the rule basically, if it's an RFE, needs a FFE? | 20:30 |
| rloo | mjturek: yup. FFE = feature freeze extension. For features. RFE = request for Feature Enhancement I think. confusing... | 20:30 |
| mjturek | and thank you :) | 20:30 |
| * TheJulia considers it a bug/defficency for non-x86 | 20:30 | |
| jroll | rloo: TheJulia: I'm finally back, what can I help out with? | 20:30 |
| TheJulia | liquor? lots of liquor? | 20:31 |
| mjturek | awesome, thank you! | 20:31 |
| rloo | jroll: answer TheJulia's question about cleaning ^^ | 20:31 |
| * jroll fetches the bourbon | 20:31 | |
| rloo | jroll: update your traits client patch | 20:31 |
| *** hw_wutianwei has quit IRC | 20:31 | |
| rloo | jroll: review https://review.openstack.org/#/c/537637/. traits broke postgres | 20:31 |
| jroll | TheJulia: I don't know why we currently run cleaning | 20:31 |
| patchbot | patch 537637 - ironic - Fix node update with PostgreSQL | 20:31 |
| jroll | lol postgres | 20:32 |
| rloo | jroll: let me know when you'd like more stuff to do. and welcome back :D | 20:32 |
| *** hw_wutianwei has joined #openstack-ironic | 20:32 | |
| jroll | :P | 20:32 |
| TheJulia | omfg, we only don't clean on standalone runs | 20:33 |
| rloo | TheJulia: for now, i see two choices, turn of cleaning, or extend timeout. maybe something to discuss (again) at ptg.... sigh. | 20:33 |
| TheJulia | how much disk time are we wasting!?!?!?!?! | 20:33 |
| rloo | TheJulia: wow, are we that thorough? | 20:33 |
| rloo | my positive spin on that... :-( | 20:33 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Be more sane about cleaning https://review.openstack.org/537650 | 20:45 |
| *** jtomasek has joined #openstack-ironic | 20:45 | |
| TheJulia | so the main grenade job also runs 7 VMs and cleaning enabled :\ | 20:46 |
| TheJulia | jroll: rloo: jlvillal: and anyone else https://review.openstack.org/537650 | 20:47 |
| patchbot | patch 537650 - ironic - Be more sane about cleaning | 20:47 |
| TheJulia | I pretty much think that just needs to be landed at this point, there is no way we're going to handle a busy gate without cutting at least some fat off of our jobs | 20:49 |
| rloo | TheJulia: saner is always better! looking... | 20:50 |
| * jroll tries to remember the benefit of master multinode | 20:53 | |
| *** jlabarre has quit IRC | 20:55 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Be more sane about cleaning https://review.openstack.org/537650 | 20:58 |
| TheJulia | jroll: replied | 20:58 |
| jroll | TheJulia: yeah fair | 20:59 |
| * jroll is +1 either way | 20:59 | |
| *** e0ne has joined #openstack-ironic | 21:00 | |
| TheJulia | rloo: jlvillal updated commit message, please review, if so I'll rebase and re-approve one of our troublesome changes on top of it | 21:01 |
| rloo | TheJulia: i just reviewed | 21:01 |
| rloo | TheJulia: do you want to fix that one thing (which is more important that the commit, heh). or i can just +2. let me know. | 21:02 |
| TheJulia | I can fix the text later :) | 21:03 |
| TheJulia | I just want to get things stacked and running so hopefully things land more reliable | 21:03 |
| TheJulia | reliably | 21:03 |
| rloo | TheJulia: ok | 21:03 |
| *** jtomasek has quit IRC | 21:03 | |
| rloo | TheJulia: heh. looks like i ended up voting on ps2 anyway :) | 21:04 |
| rloo | jlvillal: if you want to use your recheck powers, 535296. | 21:07 |
| jlvillal | rloo, Sure :) | 21:07 |
| TheJulia | wait | 21:07 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: Restores sushy session functionality. https://review.openstack.org/535910 | 21:08 |
| rloo | jlvillal: ^^ TheJulia: did you want to rebase that? | 21:08 |
| TheJulia | yeah, we rebase it on top of the test changes | 21:08 |
| TheJulia | since it has hit cleaning timeout and overall job timeout twice on that | 21:09 |
| rloo | TheJulia: ok, actually i think it is fine for jlvillal to add to his recheck thing. | 21:09 |
| TheJulia | but then we might for for hours with a bad run due to recheck | 21:09 |
| jlvillal | TheJulia, rloo: Rebasing won't impact the 'recheck' script. It will only recheck if there is a failure. | 21:10 |
| jlvillal | And someone already did a recheck on it. | 21:10 |
| jlvillal | Getting details for patch: 535296 | 21:10 |
| jlvillal | https://review.openstack.org/#/c/535296/ Add RPC API and conductor manager for traits | 21:10 |
| jlvillal | Zuul voted: -2 at 2018-01-23 22:22:03-08:00 localtime | 21:10 |
| jlvillal | Check to see if we should post 'recheck' message ... | 21:10 |
| jlvillal | Build Failed date: 2018-01-23 22:22:03-08:00 localtime | 21:10 |
| patchbot | patch 535296 - ironic - Add RPC API and conductor manager for traits | 21:10 |
| jlvillal | recheck date: 2018-01-24 09:25:15-08:00 localtime | 21:10 |
| jlvillal | 'recheck' message already posted at: 2018-01-24 03:57:38-08:00, 2018-01-24 09:25:15-08:00 | 21:10 |
| TheJulia | yeah, and it is running, will fail shortly. If we rebase, then the new config will be in place | 21:10 |
| *** amarao has joined #openstack-ironic | 21:11 | |
| TheJulia | looks like you two want to wait for results to come back before approving | 21:12 |
| TheJulia | so then lets just recheck 535296 again by hand..... | 21:12 |
| TheJulia | because we won't have results for at least four hours | 21:12 |
| rloo | TheJulia: OH. right, if we rebase, we'll have to wait for that to merge first. So yeah, let's keep rechecking until then. | 21:12 |
| jlvillal | TheJulia, I think should do the rebase most likely. The patch has already failed in zuul. | 21:13 |
| rloo | TheJulia: i wanted to see the results first, but I guess it doesn't matter. it'll merge if they pass, and we aren't that worried about non-voting jobs. | 21:13 |
| jlvillal | http://zuul.openstack.org/ | 21:13 |
| jlvillal | Patch failed on multinode job | 21:13 |
| jlvillal | https://review.openstack.org/535296 | 21:14 |
| patchbot | patch 535296 - ironic - Add RPC API and conductor manager for traits | 21:14 |
| rloo | heh, TheJulia: you probably don't want that topic for https://review.openstack.org/#/c/537650/ | 21:16 |
| patchbot | patch 537650 - ironic - Be more sane about cleaning | 21:16 |
| TheJulia | heh | 21:18 |
| TheJulia | yay for git review | 21:18 |
| *** e0ne has quit IRC | 21:20 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add RPC API and conductor manager for traits https://review.openstack.org/535296 | 21:23 |
| * TheJulia crosses fingers | 21:23 | |
| *** amarao has quit IRC | 21:30 | |
| *** dprince has quit IRC | 21:30 | |
| *** weshay|mtg is now known as weshay|rover | 21:34 | |
| *** trown is now known as trown|ruck | 21:34 | |
| *** threestrands has joined #openstack-ironic | 21:41 | |
| *** threestrands_ has joined #openstack-ironic | 21:44 | |
| *** threestrands_ has quit IRC | 21:44 | |
| *** threestrands_ has joined #openstack-ironic | 21:44 | |
| *** threestrands has quit IRC | 21:47 | |
| *** trown|ruck is now known as trown|outtypewww | 22:03 | |
| *** baha has quit IRC | 22:04 | |
| *** jjohnson2 has quit IRC | 22:08 | |
| *** rmcall has quit IRC | 22:09 | |
| *** Goneri has quit IRC | 22:11 | |
| *** akhilaki_ has quit IRC | 22:13 | |
| *** akhilaki has quit IRC | 22:13 | |
| *** rcernin has joined #openstack-ironic | 22:14 | |
| *** yangyapeng has quit IRC | 22:16 | |
| *** bfournie has quit IRC | 22:19 | |
| *** cdearborn has quit IRC | 22:25 | |
| *** crushil has quit IRC | 22:26 | |
| *** rcernin has quit IRC | 22:29 | |
| *** jrist has quit IRC | 22:29 | |
| *** jrist has joined #openstack-ironic | 22:30 | |
| *** rcernin has joined #openstack-ironic | 22:31 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Implementation for UEFI iSCSI boot for ILO https://review.openstack.org/468288 | 22:52 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Implementation for UEFI iSCSI boot for ILO https://review.openstack.org/468288 | 22:58 |
| *** yangyapeng has joined #openstack-ironic | 23:01 | |
| *** sasha2 has quit IRC | 23:16 | |
| *** derekh_afk has quit IRC | 23:33 | |
| *** rpioso is now known as rpioso|afk | 23:42 | |
| openstackgerrit | Merged openstack/ironic-lib master: Updated from global requirements https://review.openstack.org/537053 | 23:54 |
| *** rbudden has quit IRC | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!