| *** ijw has joined #openstack-ironic | 00:00 | |
| *** ijw has quit IRC | 00:00 | |
| *** jkilpatr has joined #openstack-ironic | 00:01 | |
| *** ijw has joined #openstack-ironic | 00:01 | |
| *** ijw has quit IRC | 00:01 | |
| *** ijw_ has joined #openstack-ironic | 00:01 | |
| *** lennyb has joined #openstack-ironic | 00:02 | |
| *** ijw_ has quit IRC | 00:06 | |
| *** masber has joined #openstack-ironic | 00:12 | |
| *** ijw has joined #openstack-ironic | 00:14 | |
| *** ijw has quit IRC | 00:19 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528024 | 00:22 |
|---|---|---|
| *** absubram has quit IRC | 00:27 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent stable/pike: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528136 | 00:30 |
| *** crushil_ has joined #openstack-ironic | 00:35 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 00:45 | |
| *** AlexeyAbashkin has quit IRC | 00:50 | |
| *** zhangfei has joined #openstack-ironic | 00:57 | |
| *** jkilpatr has quit IRC | 00:58 | |
| *** TuanLA has joined #openstack-ironic | 00:58 | |
| *** zhangfei has quit IRC | 01:04 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Do not try unmounting the EFI partition if it was not mounted https://review.openstack.org/521114 | 01:05 |
| *** zshi has quit IRC | 01:05 | |
| *** zshi has joined #openstack-ironic | 01:06 | |
| *** phuongnh has joined #openstack-ironic | 01:09 | |
| *** yangyapeng has joined #openstack-ironic | 01:10 | |
| *** penick has quit IRC | 01:10 | |
| *** rpioso is now known as rpioso|afk | 01:13 | |
| *** TuanLA_ has joined #openstack-ironic | 01:15 | |
| *** zhangfei has joined #openstack-ironic | 01:16 | |
| *** TuanLA has quit IRC | 01:18 | |
| *** crushil_ has quit IRC | 01:18 | |
| *** absubram has joined #openstack-ironic | 01:20 | |
| *** r-daneel has quit IRC | 01:26 | |
| *** jaypipes has quit IRC | 01:33 | |
| *** oomichi_afk is now known as oomichi | 01:36 | |
| *** linkmark has quit IRC | 01:40 | |
| *** Sukhdev has quit IRC | 01:44 | |
| *** snasir has quit IRC | 01:48 | |
| *** snasir has joined #openstack-ironic | 01:48 | |
| *** fragatina has joined #openstack-ironic | 01:53 | |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add baremetal neutron agent https://review.openstack.org/456235 | 01:54 |
| *** zhangfei has quit IRC | 01:57 | |
| *** hshiina has joined #openstack-ironic | 02:01 | |
| *** hw_wutianwei has joined #openstack-ironic | 02:09 | |
| *** masber has quit IRC | 02:10 | |
| *** zhangfei has joined #openstack-ironic | 02:14 | |
| *** rbudden has quit IRC | 02:36 | |
| *** fragatina has quit IRC | 02:41 | |
| *** harlowja has joined #openstack-ironic | 02:41 | |
| *** harlowja has quit IRC | 02:42 | |
| *** harlowja has joined #openstack-ironic | 02:43 | |
| *** rbudden has joined #openstack-ironic | 02:44 | |
| *** namnh has joined #openstack-ironic | 02:45 | |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Add DSVM jobs to the gate for ironic-tempest-plugin https://review.openstack.org/528070 | 02:49 |
| *** ijw has joined #openstack-ironic | 02:56 | |
| *** ijw has quit IRC | 02:57 | |
| *** ijw has joined #openstack-ironic | 02:57 | |
| *** ijw has quit IRC | 02:58 | |
| *** ijw has joined #openstack-ironic | 02:58 | |
| *** Gr3nth has joined #openstack-ironic | 03:02 | |
| *** Grenth has quit IRC | 03:02 | |
| *** ijw has quit IRC | 03:03 | |
| *** ijw has joined #openstack-ironic | 03:04 | |
| *** crushil_ has joined #openstack-ironic | 03:05 | |
| *** ijw has quit IRC | 03:08 | |
| *** harlowja has quit IRC | 03:15 | |
| *** jappleii__ has joined #openstack-ironic | 03:17 | |
| *** jappleii__ has quit IRC | 03:19 | |
| *** jappleii__ has joined #openstack-ironic | 03:19 | |
| *** threestrands_ has quit IRC | 03:20 | |
| *** yangyapeng has quit IRC | 03:24 | |
| *** hshiina has quit IRC | 03:32 | |
| *** absubram has quit IRC | 03:33 | |
| *** rbudden has quit IRC | 03:36 | |
| *** masber has joined #openstack-ironic | 03:38 | |
| *** hshiina has joined #openstack-ironic | 03:39 | |
| openstackgerrit | wangzhengwei proposed openstack/ironic master: The automatic startup situation of tftpd-hpa service will affect xinetd service. I suggest that after the service is installed, we ensure that this service does not start automatically. https://review.openstack.org/528153 | 03:43 |
| *** zhangfei has quit IRC | 03:46 | |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add baremetal neutron agent https://review.openstack.org/456235 | 03:50 |
| *** r-daneel has joined #openstack-ironic | 03:52 | |
| *** jappleii__ has quit IRC | 03:56 | |
| *** zhangfei has joined #openstack-ironic | 03:58 | |
| *** udesale has joined #openstack-ironic | 04:07 | |
| *** r-daneel has quit IRC | 04:07 | |
| *** rmcall has joined #openstack-ironic | 04:15 | |
| *** MattMan has quit IRC | 04:27 | |
| *** adisky__ has joined #openstack-ironic | 04:27 | |
| *** dmellado has quit IRC | 04:27 | |
| *** MattMan has joined #openstack-ironic | 04:27 | |
| *** penick has joined #openstack-ironic | 04:30 | |
| *** mallik has joined #openstack-ironic | 04:31 | |
| mallik | Good morning Ironic!!! | 04:32 |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:32 | |
| Nisha_Agarwal | mallik, hi | 04:32 |
| *** Nisha_Agarwal has quit IRC | 04:33 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:34 | |
| *** dmellado has joined #openstack-ironic | 04:35 | |
| *** harlowja has joined #openstack-ironic | 04:36 | |
| *** tzumainn has quit IRC | 04:39 | |
| *** harlowja has quit IRC | 04:45 | |
| *** penick has quit IRC | 04:47 | |
| *** harlowja has joined #openstack-ironic | 05:21 | |
| *** harlowja has quit IRC | 05:22 | |
| *** harlowja has joined #openstack-ironic | 05:24 | |
| *** harlowja has quit IRC | 05:29 | |
| *** harlowja has joined #openstack-ironic | 05:38 | |
| *** amotoki has joined #openstack-ironic | 05:39 | |
| *** marios has joined #openstack-ironic | 05:47 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-specs master: Provides detailed conductor information https://review.openstack.org/528158 | 05:49 |
| *** absubram has joined #openstack-ironic | 05:57 | |
| *** absubram has quit IRC | 06:02 | |
| *** sasha2 has quit IRC | 06:04 | |
| *** udesale has quit IRC | 06:04 | |
| *** udesale has joined #openstack-ironic | 06:06 | |
| *** absubram has joined #openstack-ironic | 06:07 | |
| *** jaganathan has joined #openstack-ironic | 06:13 | |
| *** links has joined #openstack-ironic | 06:18 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic-specs master: External display drivers https://review.openstack.org/306078 | 06:28 |
| *** harlowja has quit IRC | 06:32 | |
| *** rcernin has quit IRC | 06:47 | |
| *** mjura has joined #openstack-ironic | 06:59 | |
| *** dpawar has joined #openstack-ironic | 07:06 | |
| *** vsaienk0 has joined #openstack-ironic | 07:09 | |
| *** kiseok7 has quit IRC | 07:17 | |
| *** udesale has quit IRC | 07:24 | |
| *** absubram has quit IRC | 07:25 | |
| *** stendulker has joined #openstack-ironic | 07:32 | |
| *** absubram has joined #openstack-ironic | 07:35 | |
| openstackgerrit | zuozongming proposed openstack/ironic master: Fixed keystone misspelling https://review.openstack.org/528184 | 07:38 |
| *** makowals has quit IRC | 07:38 | |
| *** udesale has joined #openstack-ironic | 07:51 | |
| *** makowals has joined #openstack-ironic | 07:56 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic-specs master: External display drivers https://review.openstack.org/306078 | 07:57 |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:59 | |
| *** crushil_ has quit IRC | 08:10 | |
| *** mgoddard has joined #openstack-ironic | 08:13 | |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Add support for RESCUE and UNRESCUE provision states https://review.openstack.org/408341 | 08:17 |
| *** mgoddard has quit IRC | 08:24 | |
| *** ccamacho has joined #openstack-ironic | 08:31 | |
| *** jtomasek has joined #openstack-ironic | 08:37 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Do not try unmounting the EFI partition if it was not mounted https://review.openstack.org/521114 | 08:40 |
| *** trungnv has quit IRC | 08:47 | |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic master: Follow up Add additional capabilities discovery for iRMC driver https://review.openstack.org/524137 | 08:52 |
| openstackgerrit | Mark Goddard proposed openstack/networking-generic-switch master: Generalise Dell Force10 support in README https://review.openstack.org/528206 | 08:53 |
| *** charcol has joined #openstack-ironic | 08:56 | |
| *** hw_wutianwei has quit IRC | 08:59 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic-specs master: Add service management API for ironic https://review.openstack.org/471217 | 08:59 |
| *** mgoddard has joined #openstack-ironic | 09:00 | |
| *** lucas-afk is now known as lucasagomes | 09:07 | |
| *** hshiina has quit IRC | 09:07 | |
| *** amarao has joined #openstack-ironic | 09:09 | |
| *** dougsz has joined #openstack-ironic | 09:14 | |
| *** TuanLA__ has joined #openstack-ironic | 09:17 | |
| *** TuanLA_ has quit IRC | 09:20 | |
| *** e0ne has joined #openstack-ironic | 09:43 | |
| *** derekh has joined #openstack-ironic | 09:50 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: boot mode support at ManagementInterface https://review.openstack.org/526773 | 09:53 |
| *** Nisha_ has joined #openstack-ironic | 10:01 | |
| *** Nisha_Agarwal has quit IRC | 10:03 | |
| *** absubram has quit IRC | 10:05 | |
| Tengu | hello Ironic :). Small question: apparently, we have some issues with Supermicro BMC that stops responding Ironic probes, and thus nodes are set to Maintenance mode. Is it something already known? | 10:10 |
| *** therve has joined #openstack-ironic | 10:12 | |
| therve | Hi all | 10:12 |
| therve | I was slightly annoyed by conductor using a lot of memory without what it seems a good reason | 10:13 |
| therve | I looked up, and it seems a huge chunk is used by u | 10:13 |
| therve | I looked up, and it seems a huge chunk is used by UcsSdk | 10:13 |
| *** skazi has quit IRC | 10:14 | |
| therve | Which of course I don't use and don't care about | 10:14 |
| therve | Would it be worth to try to not those modules when we don't use them? | 10:14 |
| *** TuanLA has joined #openstack-ironic | 10:15 | |
| *** TuanLA__ has quit IRC | 10:16 | |
| pas-ha | therve: hm.. are those drivers enabled in ironic.conf/[DEFAULT]/{enabled_drivers|enabled_hardware_types}? | 10:19 |
| therve | pas-ha, No | 10:19 |
| therve | pas-ha, stevedore doesn't really help in that case | 10:20 |
| therve | But even if it did, the way modules are loaded currently wouldn't work | 10:20 |
| therve | As everything is imported in ironic.drivers.pxe | 10:20 |
| pas-ha | have you tried using the ipmi hardware_type instead? (and disable the pxe_impitool in config)? | 10:21 |
| pas-ha | those classic drivers are deprecated anyway... | 10:22 |
| therve | It doesn't matter. It's imported anyway. | 10:23 |
| *** skazi has joined #openstack-ironic | 10:25 | |
| pas-ha | and do you have UcsSdk installed? | 10:27 |
| therve | Yeah I do | 10:27 |
| pas-ha | tried uninstalling it? :) it seems that some packaging is overzealous re what support libs to install for ironic then | 10:28 |
| therve | I mean, sure that's a solution | 10:28 |
| pas-ha | I know this is a workaround surely | 10:29 |
| pas-ha | not a solution | 10:29 |
| therve | I was mostly wondering if you were interested in a solution :) | 10:29 |
| *** fragatina has joined #openstack-ironic | 10:29 | |
| therve | It probably needs a stevedore patch, and a small refactoring in ironic to not import everything | 10:29 |
| pas-ha | yep, some more lazy loading would help | 10:30 |
| therve | It's mostly annoying for ucs because that module looks like a pile of garbage | 10:30 |
| therve | But all the drivers do that | 10:31 |
| *** namnh has quit IRC | 10:31 | |
| pas-ha | and I think we are interested - I remember of a similar but more severe problem when conductor would not even start due to some bug in cims (or whatever) lib although the driver was not enabled, just having it installed broke ironic.. :( | 10:32 |
| therve | pas-ha, Do you know if we can move things without breaking compatibility ? | 10:33 |
| pas-ha | a quick hack would be for drivers to check on __init__ if themselves (or their required/supported interfaces for the HW type ) are indeed enabled on config | 10:33 |
| pas-ha | wdym? | 10:34 |
| therve | Ie put PXEAndUcsDriver in ironic.drivers.pxe_ucs | 10:34 |
| *** phuongnh has quit IRC | 10:34 | |
| pas-ha | would it really help? the entrypoint would still be there | 10:34 |
| therve | Right, then we need a patch in stevedore to skip loading properly | 10:35 |
| therve | Because right now the check happens after loading, but it could be before | 10:35 |
| *** TuanLA has quit IRC | 10:39 | |
| *** dmellado has quit IRC | 10:39 | |
| *** TuanLA has joined #openstack-ironic | 10:40 | |
| openstackgerrit | Merged openstack/networking-generic-switch master: Add support for trunk ports on Dell switches https://review.openstack.org/524138 | 10:42 |
| openstackgerrit | Merged openstack/networking-generic-switch master: Generalise Dell Force10 support in README https://review.openstack.org/528206 | 10:43 |
| *** dmellado has joined #openstack-ironic | 10:44 | |
| *** TuanLA has quit IRC | 10:50 | |
| *** TuanLA has joined #openstack-ironic | 10:51 | |
| ricardoas | morning, Ironic! | 10:56 |
| *** yuanying has quit IRC | 11:00 | |
| openstackgerrit | Merged openstack/bifrost stable/ocata: Fix debian IPA build https://review.openstack.org/527277 | 11:01 |
| openstackgerrit | Merged openstack/bifrost master: Make ipxe boot script templatable https://review.openstack.org/451883 | 11:01 |
| *** fellypefca-afk is now known as fellypefca | 11:12 | |
| *** marios has quit IRC | 11:12 | |
| *** stendulker_ has joined #openstack-ironic | 11:17 | |
| *** sambetts|afk is now known as sambetts | 11:18 | |
| *** stendulker has quit IRC | 11:21 | |
| *** udesale has quit IRC | 11:22 | |
| *** lucasagomes is now known as lucas-bbl | 11:23 | |
| *** stendulker_ has quit IRC | 11:24 | |
| sambetts | pas-ha, therve: stevedore won't init the driver class unless its in the enabled_drivers, but in drivers/pxe.py it imports the drivers/modules/ucs/* files, which then import the SDK if its installed, so the only way to avoid this happening would be to move the import of /modules/ucs inside the driver class object so that they aren't imported unless the class is initilised, | 11:25 |
| sambetts | the UCSSdk is deprecated and has been replaced, and we are working to patch our drivers to use the new SDK, but its been a significant amount of work as we didn't write the original driver | 11:27 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Node traits: Add DB API & model https://review.openstack.org/528238 | 11:29 |
| mgoddard | Merry christmas ironic: ^^^ first node traits patch | 11:30 |
| sambetts | \o/ | 11:30 |
| mgoddard | Shameless copy of tags :) | 11:30 |
| sambetts | pas-ha, therve: please report a bug in launchpad, so we can track the issue with the UcsSDK hopefully close it out when we replace the SDK with the new one | 11:33 |
| *** dsneddon has quit IRC | 11:33 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: [WIP][DNM] Experiment with batch through iptables-restore https://review.openstack.org/527943 | 11:35 |
| *** dtantsur|afk is now known as dtantsur | 11:38 | |
| dtantsur | morning ironic | 11:38 |
| dtantsur | wow, I woke up at noon Oo | 11:38 |
| *** jkilpatr has joined #openstack-ironic | 11:38 | |
| ricardoas | good morning dtantsur... or afternoon | 11:40 |
| dtantsur | hard to tell :) morning ricardoas | 11:40 |
| fellypefca | morning dtantsur :) | 11:42 |
| dtantsur | hi fellypefca | 11:43 |
| *** jtomasek has quit IRC | 11:44 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/pike: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528088 | 11:48 |
| *** jkilpatr has quit IRC | 11:51 | |
| *** jkilpatr has joined #openstack-ironic | 11:52 | |
| *** milan_ has joined #openstack-ironic | 11:59 | |
| *** jkilpatr has quit IRC | 12:07 | |
| *** TuanLA has quit IRC | 12:08 | |
| *** marios has joined #openstack-ironic | 12:10 | |
| *** serlex has joined #openstack-ironic | 12:11 | |
| *** TuanLA has joined #openstack-ironic | 12:11 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/virtualbmc master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528243 | 12:18 |
| *** fragatina has quit IRC | 12:20 | |
| *** mallik has quit IRC | 12:26 | |
| *** TuanLA has quit IRC | 12:27 | |
| *** milan_ has quit IRC | 12:29 | |
| *** catintheroof has joined #openstack-ironic | 12:31 | |
| openstackgerrit | Merged openstack/python-ironicclient stable/pike: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528116 | 12:33 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client stable/pike: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528126 | 12:36 |
| *** yolanda__ has joined #openstack-ironic | 12:36 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient stable/ocata: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528247 | 12:37 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client stable/ocata: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528248 | 12:38 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib stable/ocata: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528249 | 12:38 |
| chandankumar | dtantsur: i have one question regarding ironic tempest plugin work | 12:39 |
| dtantsur | chandankumar: hi, sure | 12:39 |
| *** jaganathan has quit IRC | 12:41 | |
| chandankumar | dtantsur: since we are splitting the plugin from Queens release, as per the above reviews, we are consuming the plugin for stable releases also, but in RDO side, we cannot import a new package for stable release so that is there plan to consume the new tempest plugin for stable releases? | 12:41 |
| dtantsur | chandankumar: I'm not the best person to ask it, to be honest. this question is not ironic-specific and has to be solved for all projects. | 12:41 |
| dtantsur | but I think RDO should package the plugin repo for Ocata - Queens | 12:42 |
| therve | sambetts, Right, driver init is indeed delayed with config, but module loading is not | 12:42 |
| therve | sambetts, But I think we could tweak stevedore (or use differently) to not import when not needed | 12:42 |
| openstackgerrit | Merged openstack/sushy stable/pike: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528121 | 12:42 |
| chandankumar | dtantsur: for earlier stable releases we have created sub packages tempest plugin in RDO | 12:43 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/ocata: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528251 | 12:44 |
| dtantsur | chandankumar: I guess you can continue shipping it. I think we won't remove the tempest plugin code from stable branches, only from master | 12:45 |
| dtantsur | chandankumar: though keep in mind that the tempest plugin code from stable branches is never run in the CI | 12:45 |
| sambetts | therve: drivers/pxe.py will be imported if any driver from it is loaded, so I don't think its a stevedore issue, e.g. stevedore already works that way and won't import pxe.py if you don't enable any drivers from pxe.py, so if we moved each driver to their own driver file it wouldn't import it unless you enabled that driver | 12:46 |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Add rescue_interface to node and driver https://review.openstack.org/517302 | 12:47 |
| therve | sambetts, I'm not sure about that one. | 12:47 |
| therve | I think it imports, the check is after the extension is created | 12:47 |
| dtantsur | sambetts: hey, do you have a minute to look why ironic-inspector stable/ocata is failing with DHCP timeouts? | 12:47 |
| dtantsur | I wonder if we miss some fix that landed in pike/master | 12:47 |
| *** absubram has joined #openstack-ironic | 12:48 | |
| openstackgerrit | Merged openstack/ironic-lib stable/pike: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528109 | 12:48 |
| sambetts | dtantsur: sure I'll take a look, is there a test patch up already?? | 12:49 |
| dtantsur | sambetts: can it be https://github.com/openstack/ironic-inspector/commit/db529f3f3f25f0d6f675ea3143ff24d6056245ec#diff-a7accb299ab3c3b5058dc9165dc0ee3d by the way? | 12:49 |
| dtantsur | an example of failure is https://review.openstack.org/#/c/527994/ | 12:49 |
| patchbot | patch 527994 - ironic-inspector (stable/ocata) - devstack: set [service_available]ironic-inspector ... | 12:49 |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Add rescue_interface to node and driver https://review.openstack.org/517302 | 12:50 |
| therve | sambetts, https://github.com/openstack/stevedore/blob/master/stevedore/enabled.py#L81 ext is created, and as invoke_on_load is True the object is created already | 12:50 |
| sambetts | therve: if the entrypoint passed into the load function doesn't point to a class in pxe.py then it wouldn't even know pxe.py existed | 12:52 |
| sambetts | so it couldn't import it | 12:52 |
| therve | I'm not sure what you're arguing here | 12:53 |
| therve | Those entrypoints exist | 12:53 |
| sambetts | they exist but we only pass the entrypoints from the config file into stevedore to load | 12:53 |
| sambetts | from the enabled_drivers config option | 12:54 |
| sambetts | stevedore doesn't try to load them all | 12:54 |
| sambetts | only the ones we tell it too | 12:54 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/ocata: Use IRONIC_VM_NETWORK_BRIDGE https://review.openstack.org/528255 | 12:54 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/ocata: [Devstack]: pick correct MTU for br-inspector veth https://review.openstack.org/528256 | 12:54 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/ocata: Don't set MTU for inspector veth iface on hardware https://review.openstack.org/528257 | 12:54 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/ocata: Connect brbm-inspector and brbm when needed https://review.openstack.org/528258 | 12:54 |
| therve | Again, I don't think that's the case | 12:54 |
| therve | The entrypoints come from setup.cfg | 12:55 |
| dtantsur | sambetts: let's see if all this ^^ fixes the issue | 12:55 |
| therve | We always ask for all | 12:55 |
| sambetts | then that is just messy :/ | 12:57 |
| therve | Yes :) | 12:57 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: devstack: add support for setting rescue interface https://review.openstack.org/524118 | 12:58 |
| openstackgerrit | Dmitry Tantsur proposed openstack/virtualbmc master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528243 | 12:59 |
| *** serlex has quit IRC | 12:59 | |
| sambetts | dtantsur: I expect those might do the trick (although has the patch for IRONIC_VM_NETWORK_BRIDGE been backported to Ironic) | 13:00 |
| dtantsur | sambetts: well, I see IRONIC_VM_NETWORK_BRIDGE in stable/ocata | 13:00 |
| sambetts | dtantsur: yeah just checked the branch too | 13:00 |
| dtantsur | sambetts: the only problem is that I'll probably have to squash the first two patches.. but let's wait for the CI | 13:01 |
| sambetts | +1 lets see how it goes | 13:01 |
| *** fragatina has joined #openstack-ironic | 13:03 | |
| *** absubram has quit IRC | 13:04 | |
| *** charcol has quit IRC | 13:04 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent stable/ocata: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528262 | 13:04 |
| *** charcol has joined #openstack-ironic | 13:05 | |
| dtantsur | pas-ha: hi! do you want to update ironic-staging-drivers with ironic-tempest-plugin? | 13:06 |
| pas-ha | yep, will do | 13:08 |
| *** jcoufal has joined #openstack-ironic | 13:11 | |
| dtantsur | thanks | 13:15 |
| *** trown|outtypewww is now known as trown | 13:20 | |
| *** trown is now known as trown|ruck | 13:20 | |
| chandankumar | dtantsur: thanks :-) | 13:23 |
| *** melwitt has quit IRC | 13:23 | |
| therve | sambetts, pas-ha opened https://bugs.launchpad.net/ironic/+bug/1738411 | 13:24 |
| openstack | Launchpad bug 1738411 in Ironic "Conductor loads all driver python modules" [Undecided,New] | 13:24 |
| therve | Though I found https://bugs.launchpad.net/ironic/+bug/1697657 in the way :) | 13:25 |
| openstack | Launchpad bug 1697657 in Ironic "Ironic drivers are accidentally being loaded even when unconfigured" [Undecided,Opinion] | 13:25 |
| *** melwitt has joined #openstack-ironic | 13:25 | |
| *** melwitt is now known as Guest75547 | 13:25 | |
| sambetts | therve: hehe I forgot I openned that, I guess it must be related | 13:26 |
| therve | Yeah I believe so too | 13:27 |
| *** jcoufal has quit IRC | 13:27 | |
| openstackgerrit | Merged openstack/ironic stable/pike: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528080 | 13:28 |
| sambetts | therve: related code is here https://github.com/openstack/ironic/blob/master/ironic/common/driver_factory.py#L501 in ironic | 13:30 |
| therve | sambetts, Yeah I hacked around it to try to find the issue :) | 13:31 |
| *** jcoufal has joined #openstack-ironic | 13:31 | |
| sambetts | therve: I feel like we might need to replace that line with this https://docs.openstack.org/stevedore/latest/reference/index.html#namedextensionmanager | 13:33 |
| sambetts | therve: so that we only passed the names from config file to stevedore, instead of telling it to load all the extensions from the entrypoint | 13:34 |
| therve | Hum | 13:34 |
| sambetts | it seems to be built to do what we want: https://github.com/openstack/stevedore/blob/master/stevedore/named.py#L151-L153 | 13:35 |
| therve | sambetts, Yes that does look much much better | 13:35 |
| TheJulia | Good morning everyone | 13:37 |
| *** edleafe is now known as figleaf | 13:38 | |
| TheJulia | Is everyone glad it is friday? | 13:40 |
| sambetts | TheJulia: TGIF! | 13:40 |
| *** bfournie has joined #openstack-ironic | 13:41 | |
| dtantsur | morning TheJulia, yay, Friday \o/ \o/ \o/ | 13:43 |
| dtantsur | and, as you can imagine, there are quite a few tempest-related patches waiting for review ;) | 13:44 |
| TheJulia | woot! | 13:44 |
| dtantsur | sambetts: see, https://review.openstack.org/#/c/528258/ passed! | 13:44 |
| patchbot | patch 528258 - ironic-inspector (stable/ocata) - Connect brbm-inspector and brbm when needed | 13:44 |
| TheJulia | hopefully none for deleting the intree branch until we get a green light for 3rd party CI? | 13:44 |
| dtantsur | meaning that one of the patches in this chain does fix the CI | 13:44 |
| sambetts | dtantsur: yup! | 13:45 |
| TheJulia | s/intree branch/intree folder/ | 13:45 |
| dtantsur | TheJulia: we're not close to it yet, no ocata patches have merged | 13:45 |
| dtantsur | but good call | 13:45 |
| * TheJulia looks at the coffee bug and needs much... much more | 13:45 | |
| TheJulia | lol | 13:45 |
| TheJulia | coffee bug | 13:45 |
| TheJulia | lol | 13:45 |
| dtantsur | :D | 13:46 |
| sambetts | its a bug if the mug is empty | 13:46 |
| * dtantsur wants coffee too | 13:46 | |
| TheJulia | If I ever find a nice place to settle down, I'm sooooo getting an automated espresso machine | 13:48 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/ocata: [Devstack]: pick correct MTU for br-inspector veth https://review.openstack.org/528256 | 13:51 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/ocata: Don't set MTU for inspector veth iface on hardware https://review.openstack.org/528257 | 13:51 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/ocata: Connect brbm-inspector and brbm when needed https://review.openstack.org/528258 | 13:51 |
| TheJulia | Hmmm either my laptop or review needs coffee as well | 13:51 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/ocata: devstack: set [service_available]ironic-inspector = True https://review.openstack.org/527994 | 13:53 |
| TheJulia | dtantsur: holding on stable/ocata approval until I see tests pass | 13:53 |
| dtantsur | TheJulia: sure | 13:53 |
| TheJulia | (not that I doubt they will, just... being slightly cautious) | 13:54 |
| dtantsur | well, you never know | 13:55 |
| TheJulia | yup | 13:55 |
| TheJulia | the osc-plugin check command, I'm guessing has some invalid ansible that is causing the check to fail, luckilly it is a nv job | 13:56 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/ocata: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528276 | 13:56 |
| dtantsur | okay, this must be the last ^^^ | 13:57 |
| *** dpawar has quit IRC | 13:58 | |
| openstackgerrit | Thomas Herve proposed openstack/ironic master: Use NamedExtensionManager for drivers https://review.openstack.org/528277 | 14:02 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client stable/ocata: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528248 | 14:04 |
| *** Goneri has joined #openstack-ironic | 14:05 | |
| *** sasha2 has joined #openstack-ironic | 14:07 | |
| *** vsaienk0 has quit IRC | 14:08 | |
| TheJulia | yeah, looks like one of the jobs on the ironic gate for ocata blew up because of the setting patch not having landed yet iinspector ocata. I've got the patch on ironic blocked until we can recheck | 14:16 |
| TheJulia | otherwise, at a glance, it looks like I can just go ahead and land the entire stack. | 14:17 |
| dtantsur | TheJulia: re ironic, we can add a Depends-On on the fixes, like I did for ironic-inspector-client | 14:18 |
| TheJulia | yeah | 14:18 |
| TheJulia | That does make more sense actually. I'll do it in a moment | 14:18 |
| *** SerenaFeng has joined #openstack-ironic | 14:20 | |
| *** baoli has joined #openstack-ironic | 14:21 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/ocata: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528251 | 14:22 |
| dtantsur | thanks TheJulia | 14:23 |
| *** charcol has quit IRC | 14:25 | |
| *** tzumainn has joined #openstack-ironic | 14:25 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Follow up Add additional capabilities discovery for iRMC driver https://review.openstack.org/524137 | 14:26 |
| *** jaypipes has joined #openstack-ironic | 14:28 | |
| TheJulia | no problem dtantsur :) | 14:28 |
| *** jaypipes is now known as leakypipes | 14:28 | |
| *** dpawar has joined #openstack-ironic | 14:29 | |
| *** links has quit IRC | 14:31 | |
| TheJulia | looks like your squashed changes to stable/ocata on inspector are good, at least the first tempest test passed | 14:37 |
| dtantsur | nice! | 14:38 |
| TheJulia | heh | 14:38 |
| TheJulia | make that two passed | 14:38 |
| * dtantsur will get coffee and jump on hardware types migration now | 14:39 | |
| TheJulia | I guess since it is friday, we're not going to see a ruby today. | 14:39 |
| jroll | morning y'all | 14:41 |
| TheJulia | good morning jroll | 14:41 |
| dtantsur | TheJulia: heh. well, backports from stable team members can be approved with 1x +2 ;) | 14:41 |
| dtantsur | meaning, https://review.openstack.org/#/c/528256/ is good to go! | 14:41 |
| patchbot | patch 528256 - ironic-inspector (stable/ocata) - [Devstack]: pick correct MTU for br-inspector veth | 14:41 |
| dtantsur | morning jroll | 14:41 |
| TheJulia | dtantsur: yeah, but you swashed them :) | 14:42 |
| dtantsur | TheJulia: it's not me, it's 'git rebase -i' :D | 14:42 |
| * dtantsur fetches coffee for real now | 14:42 | |
| TheJulia | done with note for historical tracking purposes :) | 14:43 |
| dtantsur | thnx | 14:44 |
| *** lucas-bbl is now known as lucasagomes | 14:46 | |
| *** rbudden has joined #openstack-ironic | 14:47 | |
| *** openstackgerrit has quit IRC | 14:48 | |
| *** mjturek has joined #openstack-ironic | 14:48 | |
| *** jtomasek has joined #openstack-ironic | 14:50 | |
| *** jtomasek has quit IRC | 14:54 | |
| jlvillal | Good morning Ironic | 14:54 |
| jlvillal | dtantsur, Have you done https://review.openstack.org/#/c/527923/ for stable/ocata yet? | 14:55 |
| patchbot | patch 527923 - ironic-inspector - devstack: set [service_available]ironic-inspector ... (MERGED) | 14:55 |
| jlvillal | I think we need that | 14:55 |
| *** openstackgerrit has joined #openstack-ironic | 14:55 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: boot mode support at ManagementInterface https://review.openstack.org/526773 | 14:55 |
| TheJulia | jlvillal: I believe I already approved his cherry-pick for it | 14:56 |
| *** rcernin has joined #openstack-ironic | 14:57 | |
| openstackgerrit | Merged openstack/ironic master: Follow up Add additional capabilities discovery for iRMC driver https://review.openstack.org/524137 | 14:57 |
| TheJulia | jlvillal: https://review.openstack.org/#/c/527994/2 | 14:57 |
| patchbot | patch 527994 - ironic-inspector (stable/ocata) - devstack: set [service_available]ironic-inspector ... | 14:57 |
| jlvillal | TheJulia, Cool :) | 14:57 |
| jlvillal | And even better if got stable/ocata working for inspector :) | 14:58 |
| TheJulia | yup, it looks good | 14:59 |
| TheJulia | jlvillal: I'm keeping an eye on 528251, once the jobs for inspector complete, I'll +A it | 15:00 |
| jlvillal | TheJulia, Thanks | 15:00 |
| *** crushil_ has joined #openstack-ironic | 15:02 | |
| *** jcoufal_ has joined #openstack-ironic | 15:02 | |
| TheJulia | five revs in the gate queue right now | 15:02 |
| *** jcoufal has quit IRC | 15:04 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: DNM/WIP: I think we only need to define it once... https://review.openstack.org/528303 | 15:04 |
| *** SerenaFeng has quit IRC | 15:06 | |
| dtantsur | morning jroll, we're moving well :) | 15:07 |
| dtantsur | oops jlvillal | 15:07 |
| dtantsur | (autocomplete fail) | 15:07 |
| jlvillal | dtantsur, :) | 15:07 |
| *** SerenaFeng has joined #openstack-ironic | 15:08 | |
| dtantsur | jlvillal: I've posted a list of potential follow-ups in the bottom of the etherpad, please check | 15:08 |
| *** Goneri has quit IRC | 15:08 | |
| jlvillal | dtantsur, Looks like we are close to being able to remove the tempest code. Probably can do it from ironic-inspector soon | 15:08 |
| dtantsur | jlvillal: I wonder if we need to notify the mailing list; e.g. I'm not sure how it will affect packagers /cc chandankumar | 15:09 |
| jlvillal | dtantsur, I don't know... | 15:09 |
| dtantsur | jlvillal: https://review.openstack.org/#/c/528243/ is on master branch, so you can approve it :) | 15:10 |
| patchbot | patch 528243 - virtualbmc - Use the tempest plugin from openstack/ironic-tempe... | 15:10 |
| jlvillal | dtantsur, I was just starting to look at that ;) | 15:11 |
| dtantsur | cool | 15:11 |
| *** harlowja has joined #openstack-ironic | 15:11 | |
| * jlvillal grumbles about dtantsur's commit message... | 15:11 | |
| dtantsur | oh c'mon :) | 15:12 |
| jlvillal | Use more than one line!!!! :P | 15:12 |
| TheJulia | jlvillal: dtantsur: I'd say we need to wait until after the weekly meeting next week so there is at least some third party CI visibility. Sadly I ran across a random patch yesterday that had completely failed all third party CI systems but for totally unrelated and some known issues :( | 15:12 |
| *** harlowja has quit IRC | 15:13 | |
| *** livelace-link has joined #openstack-ironic | 15:13 | |
| openstackgerrit | Merged openstack/ironic-python-agent stable/pike: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528136 | 15:13 |
| jlvillal | dtantsur, +A | 15:13 |
| *** Goneri has joined #openstack-ironic | 15:14 | |
| dtantsur | jlvillal: I only see a +2, you were probably holding your nose too tight :D | 15:14 |
| jlvillal | heh | 15:14 |
| jlvillal | dtantsur, Okay really this time | 15:14 |
| dtantsur | thanks :) | 15:14 |
| *** livelace-link has quit IRC | 15:16 | |
| *** livelace-link has joined #openstack-ironic | 15:19 | |
| jlvillal | dtantsur, I'm working on an email. | 15:21 |
| dtantsur | cool | 15:21 |
| jlvillal | dtantsur, Do you think we can say we will remove the plugin code on Monday? | 15:22 |
| jlvillal | Or when? | 15:22 |
| jlvillal | At least our planned date to do it. | 15:22 |
| jlvillal | dtantsur, I'm going with Monday as the planned date :) | 15:25 |
| dtantsur | jlvillal: probably Tuesday? TheJulia suggested after the meeting | 15:25 |
| jlvillal | dtantsur, Okay | 15:25 |
| TheJulia | I think tuesday works quite well | 15:26 |
| jlvillal | dtantsur, TheJulia: http://lists.openstack.org/pipermail/openstack-dev/2017-December/125642.html | 15:28 |
| *** zhangfei has quit IRC | 15:28 | |
| dtantsur | thanks! | 15:29 |
| TheJulia | I'm not so sure about backporting deletions since that impacts packagers and what they might expect where as the new plugins just went live and they won't have time to package that or potentially not be aware of it | 15:30 |
| jlvillal | TheJulia, I'm not sure what packagers do with the tempest code. Tempest is an un-branched project. Really we should have probably been deleting the plugin code when we did a release... But we didn't. | 15:32 |
| TheJulia | jlvillal: agreed, they should be pulling from master state, but they might not be doing that on downstream ci | 15:32 |
| jlvillal | So I don't know. I'm not 100% opposed to leaving it in there if people think we should. | 15:32 |
| *** jcoufal_ has quit IRC | 15:33 | |
| jlvillal | TheJulia, We will help them find that bug then ;) heh | 15:33 |
| TheJulia | heh | 15:33 |
| sambetts | I would almost prefer that we forced downstream CIs to run the right version of tempest and the tempest plugin | 15:33 |
| sambetts | otherwise CI's are uncomparable | 15:33 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: reloads mutable config values on SIGHUP https://review.openstack.org/474331 | 15:33 |
| TheJulia | well, it is point in time reference, it might just be wrong | 15:33 |
| jlvillal | WTF?!?!?! | 15:34 |
| * TheJulia tries to remember why we needed point in time tempest versions when at HP | 15:34 | |
| jlvillal | The rloo character is supposed to on their weekend!!! | 15:34 |
| TheJulia | lol | 15:34 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs master: Amend future of classic drivers with automated migration https://review.openstack.org/528308 | 15:35 |
| dtantsur | TheJulia: based on midcycle discussions ^^^ | 15:35 |
| dtantsur | jlvillal: firm -2 to removign code in old branches | 15:35 |
| *** jcoufal has joined #openstack-ironic | 15:36 | |
| jlvillal | Okay | 15:36 |
| dtantsur | at least RDO does ship it (because "distributions should ship master tempest" is mostly a dream) | 15:36 |
| TheJulia | dtantsur: awesome | 15:36 |
| dtantsur | jlvillal: the work we're doing will probably help them stop this practice, but it will take time | 15:36 |
| *** dpawar has quit IRC | 15:36 | |
| TheJulia | well... shipping it is not really even it's purpose | 15:36 |
| TheJulia | but yeah | 15:36 |
| sambetts | also from now on I assume RDO won't package it versioned with Ironic? | 15:37 |
| jlvillal | 4 less patches to do :) | 15:37 |
| dtantsur | :) | 15:37 |
| dtantsur | sambetts: not sure I get the question. RDO will version is as we version it. | 15:37 |
| dtantsur | and now that it's separate, we can ask them to bump versions (modulo dependencies, of course) | 15:38 |
| sambetts | dtantsur: I mean it won't be packged into the ironic package, so it doesn't have to be versioned with ironic | 15:38 |
| dtantsur | correct | 15:38 |
| dtantsur | previously it was simply impossible to ship the master version of our tempest plugin with a non-master version of ironic. at least with the way RPMs are built. | 15:38 |
| sambetts | so then shipping master / latest release of tempest, so be the behaviour going forward right? | 15:39 |
| dtantsur | weeelllll... ideally | 15:39 |
| jlvillal | TheJulia, https://review.openstack.org/#/c/528248/ if you have time | 15:39 |
| patchbot | patch 528248 - python-ironic-inspector-client (stable/ocata) - Use the tempest plugin from openstack/ironic-tempe... | 15:39 |
| dtantsur | sambetts: that requires all projects switch to branchless plugins, to begin with | 15:39 |
| sambetts | dtantsur: yeah, we need to get there | 15:39 |
| dtantsur | that's a goal for Queens :) | 15:40 |
| pas-ha | dtantsur: re staging drivers - it seems we have a problem now that the ansible driver in staging is clashing with ansible driver in the ironic itself, at least for conf options being registered.. should we may be just remove it from staging?... | 15:41 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: added redfish boot mode management https://review.openstack.org/528314 | 15:41 |
| *** rcernin has quit IRC | 15:41 | |
| dtantsur | pas-ha: oh.. yes, we can remove it. and remember the next time to use staging_ prefix for config groups | 15:42 |
| pas-ha | yep... indeed that would've avoided such clash | 15:42 |
| TheJulia | jlvillal: approved, I wanted to look at the playbook first to see what was going on | 15:43 |
| * TheJulia approves some remaining items | 15:45 | |
| dtantsur | TheJulia: note that Depends-On cross-repo works a bit weird: it does not catch when dependency merge | 15:45 |
| dtantsur | so you'll need to remove W+1 and add it back.. | 15:46 |
| jlvillal | TheJulia, Thanks | 15:49 |
| *** vsaienk0 has joined #openstack-ironic | 15:50 | |
| *** derekh has quit IRC | 15:50 | |
| jlvillal | I have to admit I'm impressed with how many patches were done and approved for the tempest work in about 24-48 hours. | 15:52 |
| jlvillal | 20+ patches at my rough count | 15:52 |
| dtantsur | oh yeah | 15:53 |
| *** makowals has quit IRC | 15:55 | |
| *** amarao has quit IRC | 15:56 | |
| TheJulia | dtantsur: it used to work just fine at times :( | 15:57 |
| * TheJulia sighs | 15:57 | |
| dtantsur | really? well, then maybe it has been fixed | 15:58 |
| *** dsneddon has joined #openstack-ironic | 15:58 | |
| TheJulia | i'll dbl check in a moment | 15:58 |
| openstackgerrit | Merged openstack/ironic-inspector stable/pike: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528088 | 16:01 |
| openstackgerrit | Merged openstack/ironic-inspector stable/ocata: [Devstack]: pick correct MTU for br-inspector veth https://review.openstack.org/528256 | 16:01 |
| openstackgerrit | Merged openstack/ironic-inspector stable/ocata: devstack: set [service_available]ironic-inspector = True https://review.openstack.org/527994 | 16:01 |
| openstackgerrit | Merged openstack/ironic-inspector stable/ocata: Don't set MTU for inspector veth iface on hardware https://review.openstack.org/528257 | 16:01 |
| dtantsur | nice! | 16:01 |
| TheJulia | dtantsur: it failed :\ | 16:03 |
| *** vsaienk0 has quit IRC | 16:04 | |
| *** e0ne has quit IRC | 16:05 | |
| * TheJulia blinks | 16:08 | |
| TheJulia | reports killed but looks like it wasn't that long into the job :\ | 16:08 |
| dtantsur | weird indeed | 16:10 |
| TheJulia | it must have taken a REALLY long time to get started, fired off a recheck, the devstack log shows success so it was one of those fun timing instances | 16:12 |
| TheJulia | SWEET http://logs.openstack.org/76/528276/1/gate/ironic-inspector-tox-func/d395de2/job-output.txt.gz#_2017-12-15_15_17_53_923191 | 16:14 |
| TheJulia | oh, apparently it did legitmately time out | 16:15 |
| TheJulia | I can't count apparently | 16:15 |
| *** Guest75547 is now known as melwitt | 16:18 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: zuul: Remove duplicated TEMPEST_PLUGIN entry https://review.openstack.org/528303 | 16:19 |
| jlvillal | ^^^ Is just some house-keeping | 16:20 |
| *** crushil_ has quit IRC | 16:24 | |
| *** yangyapeng has joined #openstack-ironic | 16:24 | |
| *** crushil_ has joined #openstack-ironic | 16:24 | |
| *** mjura has quit IRC | 16:26 | |
| *** SerenaFeng has quit IRC | 16:29 | |
| hjensas | sambetts: CI job for baremetal-networking. Got any pointers for me on where to start? I guess looking at existing jobs would be a good start, but this is all new to me so I don't even know where they are. | 16:31 |
| openstackgerrit | Merged openstack/python-ironicclient stable/ocata: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528247 | 16:32 |
| *** dsneddon has quit IRC | 16:36 | |
| sambetts | hjensas: the existing jobs are all based on devstack, so your best place to start is to look at how our devstack setup works regarding setting up a baremetal simulation and connecting it into neutron | 16:37 |
| jlvillal | hjensas, If you look in openstack/ironic/zuul.d/ and openstack/ironic/playbooks/ that will be a good start. I think. | 16:38 |
| jlvillal | hjensas, I would look at the multi-tenant job in particular. And also look at openstack/ironic/devstack/lib/ironic | 16:38 |
| hjensas | sambetts: I've been using that locally to test, but single node only and I did'nt dig into the details of networking and vbmc I guess. | 16:38 |
| sambetts | hjensas: and we need to research to see if neutron has any devstack flags for enabling L3 networks | 16:39 |
| sambetts | because if they do it might already have the bulk of what we need to fake an L3 environment | 16:39 |
| *** AlexeyAbashkin has quit IRC | 16:41 | |
| hjensas | jlvillal: thanks! | 16:42 |
| jlvillal | hjensas, vsaienk0 and vdrok might also be good resources. | 16:42 |
| *** dsneddon has joined #openstack-ironic | 16:43 | |
| openstackgerrit | Merged openstack/ironic-python-agent stable/ocata: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528262 | 16:43 |
| jlvillal | Woot! Another one :) | 16:43 |
| hjensas | sambetts: I see, I will start with digging into the devstack stuff and look for clues in neutron, ask mlavalle I guess... :) | 16:44 |
| *** etingof has quit IRC | 16:45 | |
| jlvillal | dtantsur, TheJulia: I've updated https://etherpad.openstack.org/p/ironic-tempest-plugin-migration with what I think is the current state of the patches. Basically adding a strike-thru on the ones that are merged. | 16:45 |
| dtantsur | thanks, was about to ask you :) | 16:45 |
| *** d0ugal has quit IRC | 16:45 | |
| jlvillal | We have about five patches left to merge for today. The other two will be done on Tuesday. | 16:46 |
| *** etingof has joined #openstack-ironic | 16:46 | |
| openstackgerrit | Merged openstack/virtualbmc master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528243 | 16:47 |
| *** serlex has joined #openstack-ironic | 16:48 | |
| dtantsur | and one more :) | 16:48 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Switch non-vendor parts admin guide to hardware types https://review.openstack.org/528337 | 16:48 |
| jlvillal | Updated etherpad again | 16:50 |
| dtantsur | jlvillal: put my +2 on the removal patches, so that they can be merged when we are ready | 16:52 |
| jlvillal | dtantsur, Thanks | 16:53 |
| *** serlex has quit IRC | 17:01 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-inspector master: zuul: Remove duplicated TEMPEST_PLUGINS entry https://review.openstack.org/528346 | 17:04 |
| *** d0ugal has joined #openstack-ironic | 17:04 | |
| dtantsur | a very productive Friday, now it's time to wrap it up :) | 17:04 |
| dtantsur | see you on Tuesday! | 17:04 |
| jlvillal | dtantsur, Ciao | 17:05 |
| *** dtantsur is now known as dtantsur|afk | 17:05 | |
| *** absubram has joined #openstack-ironic | 17:06 | |
| *** charcol has joined #openstack-ironic | 17:07 | |
| *** absubram has quit IRC | 17:07 | |
| *** charcol has quit IRC | 17:09 | |
| *** charcol has joined #openstack-ironic | 17:09 | |
| NobodyCam | Good Morning Ironic'ers | 17:13 |
| *** pprokop has quit IRC | 17:14 | |
| *** penick has joined #openstack-ironic | 17:15 | |
| TheJulia | good morning NobodyCam | 17:19 |
| NobodyCam | Good Morning TheJulia :) | 17:21 |
| NobodyCam | happy friday | 17:22 |
| NobodyCam | omg I totally forgot.... | 17:22 |
| NobodyCam | TGIF! | 17:22 |
| *** crushil_ has quit IRC | 17:23 | |
| *** tzumainn has quit IRC | 17:34 | |
| *** mjturek has quit IRC | 17:35 | |
| *** lucasagomes is now known as lucas-afk | 17:35 | |
| sambetts | NobodyCam: TGIF!! | 17:40 |
| sambetts | I'm heading oof | 17:40 |
| sambetts | off* | 17:40 |
| sambetts | Night all o/ | 17:40 |
| *** sambetts is now known as sambetts|afk | 17:40 | |
| NobodyCam | Have a great weekend sambetts|afk | 17:40 |
| *** dprince has joined #openstack-ironic | 17:41 | |
| *** mgoddard has quit IRC | 17:43 | |
| *** marios has quit IRC | 17:44 | |
| *** charcol has quit IRC | 17:46 | |
| *** baoli has quit IRC | 17:48 | |
| *** crushil_ has joined #openstack-ironic | 17:50 | |
| openstackgerrit | Merged openstack/ironic-inspector stable/ocata: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528276 | 17:51 |
| *** mjturek has joined #openstack-ironic | 17:54 | |
| *** dougsz has quit IRC | 17:55 | |
| TheJulia | The fun of setting up new travel profiles :( | 17:58 |
| *** rmcall has quit IRC | 18:00 | |
| *** baoli has joined #openstack-ironic | 18:03 | |
| *** aarefiev has quit IRC | 18:04 | |
| openstackgerrit | Merged openstack/ironic-lib stable/ocata: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528249 | 18:05 |
| *** catintheroof has quit IRC | 18:07 | |
| *** catinthe_ has joined #openstack-ironic | 18:07 | |
| jlvillal | TheJulia, Two left to go in the tempest work. I did a recheck on https://review.openstack.org/528248 because it had a depends-on and didn't automatically go to the merge gate. | 18:15 |
| patchbot | patch 528248 - python-ironic-inspector-client (stable/ocata) - Use the tempest plugin from openstack/ironic-tempe... | 18:15 |
| TheJulia | gah | 18:15 |
| *** ram5391 has joined #openstack-ironic | 18:16 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 18:16 | |
| *** ram5391 has quit IRC | 18:17 | |
| *** AlexeyAbashkin has quit IRC | 18:21 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 18:22 | |
| *** AlexeyAbashkin has quit IRC | 18:26 | |
| *** trown|ruck is now known as trown|lunch | 18:29 | |
| *** penick has quit IRC | 18:30 | |
| *** penick has joined #openstack-ironic | 18:32 | |
| NobodyCam | TheJulia: this must me some strange new definition _FUN_ I was previously unaware of! | 18:35 |
| *** rmcall has joined #openstack-ironic | 18:36 | |
| *** penick has quit IRC | 18:40 | |
| *** Sukhdev has joined #openstack-ironic | 18:41 | |
| *** baoli has quit IRC | 18:42 | |
| *** Sukhdev has quit IRC | 18:42 | |
| *** ccamacho has quit IRC | 18:44 | |
| jroll | what isn't fun about CWT? | 18:44 |
| *** baoli has joined #openstack-ironic | 18:46 | |
| *** e0ne has joined #openstack-ironic | 19:00 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client stable/ocata: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528248 | 19:00 |
| TheJulia | It is not cwt.... | 19:01 |
| jlvillal | One more to go :) | 19:01 |
| *** crushil_ has quit IRC | 19:08 | |
| *** crushil_ has joined #openstack-ironic | 19:09 | |
| *** ricardoas is now known as ricardoas-afk | 19:19 | |
| *** catinthe_ has quit IRC | 19:21 | |
| *** e0ne has quit IRC | 19:22 | |
| *** dprince has quit IRC | 19:23 | |
| *** dprince has joined #openstack-ironic | 19:25 | |
| *** trown|lunch is now known as trown | 19:29 | |
| *** jcoufal has quit IRC | 19:30 | |
| *** trown is now known as trown|ruck | 19:34 | |
| *** racedo_ has quit IRC | 19:44 | |
| openstackgerrit | Merged openstack/ironic stable/ocata: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528251 | 19:56 |
| *** Goneri has quit IRC | 19:58 | |
| TheJulia | \o/ | 20:09 |
| *** exodusftw has quit IRC | 20:09 | |
| *** mgoddard has joined #openstack-ironic | 20:09 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-specs master: Move traits from "high" to "essential" priority https://review.openstack.org/528386 | 20:11 |
| jlvillal | Nice last one landed :) | 20:11 |
| jlvillal | mgoddard: Question on the traits patch. Can I assume that there will be other patches needed? Not just https://review.openstack.org/#/c/528238/ | 20:14 |
| patchbot | patch 528238 - ironic - Node traits: Add DB API & model | 20:14 |
| jlvillal | mgoddard: And thanks for working on it :) | 20:14 |
| *** exodusftw has joined #openstack-ironic | 20:15 | |
| mgoddard | jlvillal: yeah plenty more patches to come - that's just a first stab at the DB layer | 20:16 |
| jlvillal | mgoddard: Thanks! | 20:16 |
| *** jkilpatr has joined #openstack-ironic | 20:22 | |
| *** penick has joined #openstack-ironic | 20:23 | |
| *** penick has quit IRC | 20:31 | |
| *** mgoddard has quit IRC | 20:33 | |
| *** dprince has quit IRC | 20:36 | |
| *** e0ne has joined #openstack-ironic | 20:40 | |
| *** e0ne has quit IRC | 20:42 | |
| *** trown|ruck is now known as trown|outtypewww | 20:45 | |
| *** openstackgerrit has quit IRC | 20:48 | |
| *** penick has joined #openstack-ironic | 20:48 | |
| *** penick has quit IRC | 20:51 | |
| *** penick has joined #openstack-ironic | 20:56 | |
| *** rmcall has quit IRC | 21:16 | |
| *** Goneri has joined #openstack-ironic | 21:24 | |
| *** baoli has quit IRC | 21:27 | |
| *** openstackgerrit has joined #openstack-ironic | 21:31 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/528402 | 21:31 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib master: Updated from global requirements https://review.openstack.org/528403 | 21:31 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements https://review.openstack.org/528404 | 21:32 |
| *** rcernin has joined #openstack-ironic | 21:36 | |
| *** jlabarre has quit IRC | 21:41 | |
| *** openstackgerrit has quit IRC | 21:48 | |
| *** mjturek has quit IRC | 21:50 | |
| *** bfournie has quit IRC | 22:03 | |
| *** bfournie has joined #openstack-ironic | 22:04 | |
| *** bfournie has quit IRC | 22:08 | |
| *** jcoufal has joined #openstack-ironic | 22:17 | |
| *** crushil_ has quit IRC | 22:22 | |
| *** catintheroof has joined #openstack-ironic | 22:29 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 22:42 | |
| *** jcoufal has quit IRC | 22:44 | |
| *** AlexeyAbashkin has quit IRC | 22:47 | |
| *** penick has quit IRC | 23:07 | |
| *** zshi has quit IRC | 23:18 | |
| *** penick has joined #openstack-ironic | 23:19 | |
| *** fragatina has quit IRC | 23:26 | |
| *** fragatina has joined #openstack-ironic | 23:27 | |
| *** zshi has joined #openstack-ironic | 23:32 | |
| *** rwsu has quit IRC | 23:37 | |
| *** sasha2 has quit IRC | 23:51 | |
| *** fragatina has quit IRC | 23:54 | |
| *** adisky__ has quit IRC | 23:56 | |
| *** fragatina has joined #openstack-ironic | 23:56 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!