| *** r-daneel has quit IRC | 00:01 | |
| *** weshay is now known as weshay_PTO | 00:04 | |
| *** exodusftw has quit IRC | 00:18 | |
| *** exodusftw has joined #openstack-ironic | 00:23 | |
| *** mrunge_ has quit IRC | 00:25 | |
| *** hw_wutianwei has quit IRC | 00:26 | |
| *** mrunge has joined #openstack-ironic | 00:26 | |
| *** hw_wutianwei has joined #openstack-ironic | 00:26 | |
| *** moshele has joined #openstack-ironic | 00:42 | |
| *** fragatin_ has quit IRC | 00:42 | |
| *** fragatina has joined #openstack-ironic | 00:42 | |
| *** mjturek has quit IRC | 00:47 | |
| *** trungnv has joined #openstack-ironic | 00:49 | |
| *** trungnv has quit IRC | 00:53 | |
| *** phuongnh has joined #openstack-ironic | 00:56 | |
| *** openstackgerrit has quit IRC | 01:03 | |
| *** hshiina has joined #openstack-ironic | 01:04 | |
| *** rloo has quit IRC | 01:09 | |
| *** yangyapeng has quit IRC | 01:12 | |
| *** yangyapeng has joined #openstack-ironic | 01:12 | |
| *** tiendc has joined #openstack-ironic | 01:16 | |
| *** moshele has quit IRC | 01:36 | |
| *** openstackgerrit has joined #openstack-ironic | 01:42 | |
| openstackgerrit | Merged openstack/ironic stable/pike: Zuul: Remove project name https://review.openstack.org/543581 | 01:42 |
|---|---|---|
| openstackgerrit | OpenStack Proposal Bot proposed openstack/bifrost stable/queens: Updated from global requirements https://review.openstack.org/543675 | 01:48 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib stable/queens: Updated from global requirements https://review.openstack.org/543682 | 01:50 |
| *** yangyapeng has quit IRC | 01:53 | |
| *** yangyapeng has joined #openstack-ironic | 01:54 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client stable/queens: Updated from global requirements https://review.openstack.org/543734 | 01:59 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/sushy stable/queens: Updated from global requirements https://review.openstack.org/543754 | 02:02 |
| *** moshele has joined #openstack-ironic | 02:09 | |
| *** Sukhdev has quit IRC | 02:10 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 02:21 | |
| *** AlexeyAbashkin has quit IRC | 02:25 | |
| *** r-daneel has joined #openstack-ironic | 02:33 | |
| *** SerenaFeng has joined #openstack-ironic | 02:35 | |
| *** links has joined #openstack-ironic | 02:35 | |
| *** crushil has joined #openstack-ironic | 02:48 | |
| *** shekar has quit IRC | 02:54 | |
| *** shekarvajjula has joined #openstack-ironic | 02:55 | |
| openstackgerrit | Merged openstack/ironic master: Stop running non-voting jobs in gate https://review.openstack.org/543543 | 03:01 |
| *** harlowja has quit IRC | 03:04 | |
| *** dpawar has joined #openstack-ironic | 03:16 | |
| *** dpawar has quit IRC | 03:17 | |
| *** SerenaFeng has quit IRC | 03:21 | |
| *** crushil has quit IRC | 03:23 | |
| *** moshele has quit IRC | 03:27 | |
| *** yangyapeng has quit IRC | 03:28 | |
| *** yangyapeng has joined #openstack-ironic | 03:29 | |
| *** dpawar has joined #openstack-ironic | 03:31 | |
| *** yangyapeng has quit IRC | 03:33 | |
| *** sapd has quit IRC | 03:42 | |
| *** stendulker has joined #openstack-ironic | 03:53 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 03:59 | |
| *** links has quit IRC | 04:05 | |
| *** liuzz has quit IRC | 04:06 | |
| *** ijw has quit IRC | 04:07 | |
| *** links has joined #openstack-ironic | 04:24 | |
| *** phuongnh has quit IRC | 04:29 | |
| *** phuongnh has joined #openstack-ironic | 04:29 | |
| *** dpawar has quit IRC | 04:29 | |
| *** crushil has joined #openstack-ironic | 04:35 | |
| *** etingof|afk has quit IRC | 04:42 | |
| *** threestrands has quit IRC | 04:50 | |
| *** etingof|afk has joined #openstack-ironic | 04:55 | |
| openstackgerrit | Merged openstack/ironic master: Add validate_rescue() method to boot interface https://review.openstack.org/540954 | 05:00 |
| *** SerenaFeng has joined #openstack-ironic | 05:08 | |
| *** SerenaFeng has quit IRC | 05:13 | |
| *** phuongnh has quit IRC | 05:24 | |
| *** phuongnh has joined #openstack-ironic | 05:25 | |
| *** fragatina has quit IRC | 05:26 | |
| *** crushil has quit IRC | 05:46 | |
| *** dpawar has joined #openstack-ironic | 05:48 | |
| *** moshele has joined #openstack-ironic | 05:49 | |
| *** harlowja has joined #openstack-ironic | 05:52 | |
| *** hieulq has joined #openstack-ironic | 06:17 | |
| *** kzaitsev_pi has quit IRC | 06:19 | |
| *** kzaitsev_pi has joined #openstack-ironic | 06:21 | |
| *** fragatina has joined #openstack-ironic | 06:27 | |
| *** parlos has joined #openstack-ironic | 06:29 | |
| *** harlowja has quit IRC | 06:33 | |
| *** vsaienk0 has joined #openstack-ironic | 06:36 | |
| *** vsaienk0 has quit IRC | 06:46 | |
| *** Nisha_Agarwal has quit IRC | 06:47 | |
| *** rbartal has joined #openstack-ironic | 06:50 | |
| *** vsaienk0 has joined #openstack-ironic | 06:52 | |
| *** mjura has joined #openstack-ironic | 06:56 | |
| *** marios has joined #openstack-ironic | 06:57 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Implements validate_rescue() for IloVirtualMediaBoot https://review.openstack.org/453012 | 06:57 |
| *** vsaienk0 has quit IRC | 07:02 | |
| *** olivierbourdon38 has quit IRC | 07:07 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Implements validate_rescue() for IloVirtualMediaBoot https://review.openstack.org/453012 | 07:12 |
| ameeda | Morning :) | 07:14 |
| ameeda | dtantsur: are you around ? | 07:15 |
| *** hieulq has quit IRC | 07:17 | |
| *** jaganathan_ has joined #openstack-ironic | 07:24 | |
| *** fragatina has quit IRC | 07:25 | |
| *** rcernin has quit IRC | 07:25 | |
| *** jaganathan has quit IRC | 07:25 | |
| *** pmannidi has quit IRC | 07:26 | |
| *** ptx0 has quit IRC | 07:28 | |
| *** ptx0 has joined #openstack-ironic | 07:28 | |
| *** zhangfei has joined #openstack-ironic | 07:31 | |
| *** hoonetorg has quit IRC | 07:34 | |
| *** jaganathan_ has quit IRC | 07:36 | |
| *** jaganathan_ has joined #openstack-ironic | 07:36 | |
| *** olivierbourdon38 has joined #openstack-ironic | 07:37 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui stable/queens: Imported Translations from Zanata https://review.openstack.org/543202 | 07:38 |
| *** rbartal has quit IRC | 07:38 | |
| *** pcaruana has joined #openstack-ironic | 07:41 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:48 | |
| *** hoonetorg has joined #openstack-ironic | 07:50 | |
| *** moshele has quit IRC | 07:51 | |
| *** d0ugal has quit IRC | 07:52 | |
| *** moshele has joined #openstack-ironic | 07:52 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 07:57 | |
| *** tiendc has quit IRC | 08:01 | |
| *** MattMan has quit IRC | 08:01 | |
| *** MattMan has joined #openstack-ironic | 08:02 | |
| *** e0ne has joined #openstack-ironic | 08:02 | |
| *** tiendc has joined #openstack-ironic | 08:02 | |
| *** d0ugal has joined #openstack-ironic | 08:02 | |
| *** jtomasek has joined #openstack-ironic | 08:04 | |
| *** olivierbourdon38 has quit IRC | 08:06 | |
| *** jaganathan__ has joined #openstack-ironic | 08:10 | |
| *** jaganathan_ has quit IRC | 08:10 | |
| *** jaganathan__ has quit IRC | 08:15 | |
| *** jaganathan__ has joined #openstack-ironic | 08:15 | |
| *** hieulq has joined #openstack-ironic | 08:18 | |
| *** rbartal has joined #openstack-ironic | 08:25 | |
| *** tesseract has joined #openstack-ironic | 08:26 | |
| *** parlos has quit IRC | 08:37 | |
| *** moshele has quit IRC | 08:49 | |
| *** vsaienk0 has joined #openstack-ironic | 08:55 | |
| *** priteau has joined #openstack-ironic | 08:59 | |
| *** Nisha_Agarwal has quit IRC | 09:02 | |
| *** vsaienk0 has quit IRC | 09:05 | |
| *** vsaienk0 has joined #openstack-ironic | 09:08 | |
| *** dtantsur|afk is now known as dtantsur | 09:09 | |
| dtantsur | morning ironic | 09:09 |
| dtantsur | ameeda: around now | 09:09 |
| dtantsur | (though will leave soon) | 09:09 |
| ameeda | dtantsur: thanks for reply, the introspect range is correct , there is work around to solve the issue, he reboot those 5 servers and boot them manually to pxe, then introspection complete successfully, I guess that boot order is wrong, but I want to make sure from that when he answered me . do you guess something else ? | 09:17 |
| *** vsaienk0 has quit IRC | 09:17 | |
| dtantsur | ameeda: it may be a boot order problem. or it may be too many servers PXE-booting in parallel | 09:18 |
| ameeda | dtantsur: how to be sure that there are no many pxe-booting ? | 09:18 |
| openstackgerrit | Ankit Kumar proposed openstack/ironic-tempest-plugin master: Added fixed_ip support for standalone-ironic https://review.openstack.org/542792 | 09:23 |
| *** dpawar has quit IRC | 09:24 | |
| dtantsur | ameeda: it's hard to tell. if nothing else help, then it makes sense to boot less nodes in parallel. but look into the boot order problem first | 09:25 |
| ameeda | dtantsur: it's okay, I still waiting to get permissions to check the boot order. Thank you for your time :) | 09:26 |
| dtantsur | you're welcome | 09:29 |
| *** hshiina has quit IRC | 09:29 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Only set switch_id in local_link_connection if it is a mac address https://review.openstack.org/542214 | 09:34 |
| openstackgerrit | Merged openstack/ironic stable/queens: Update UPPER_CONSTRAINTS_FILE for stable/queens https://review.openstack.org/542208 | 09:35 |
| openstackgerrit | Merged openstack/ironic-python-agent stable/queens: Update UPPER_CONSTRAINTS_FILE for stable/queens https://review.openstack.org/542156 | 09:35 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client stable/queens: Update UPPER_CONSTRAINTS_FILE for stable/queens https://review.openstack.org/538086 | 09:37 |
| openstackgerrit | Merged openstack/ironic-ui stable/queens: Imported Translations from Zanata https://review.openstack.org/543202 | 09:39 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Change ironic-inspector to be source installed https://review.openstack.org/543491 | 09:39 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Collect ironic-inspector logs https://review.openstack.org/542795 | 09:41 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Updated from global requirements https://review.openstack.org/541944 | 09:43 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Make Fedora26 CI job voting https://review.openstack.org/540524 | 09:44 |
| * dtantsur goes to pick his passport, brb | 09:47 | |
| *** dtantsur is now known as dtantsur|brb | 09:47 | |
| *** racedo has joined #openstack-ironic | 09:53 | |
| *** derekh has joined #openstack-ironic | 09:54 | |
| *** vsaienk0 has joined #openstack-ironic | 09:55 | |
| *** moshele has joined #openstack-ironic | 10:00 | |
| *** hieulq has quit IRC | 10:02 | |
| *** vsaienk0 has quit IRC | 10:05 | |
| *** tiendc has quit IRC | 10:10 | |
| *** phuongnh has quit IRC | 10:14 | |
| *** priteau has quit IRC | 10:17 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Implements validate_rescue() for IRMCVirtualMediaBoot https://review.openstack.org/543883 | 10:17 |
| *** links has quit IRC | 10:36 | |
| openstackgerrit | Merged openstack/bifrost master: Change ironic-inspector to be source installed https://review.openstack.org/543491 | 10:36 |
| openstackgerrit | Merged openstack/bifrost master: Updated from global requirements https://review.openstack.org/541944 | 10:36 |
| openstackgerrit | Merged openstack/bifrost master: Make Fedora26 CI job voting https://review.openstack.org/540524 | 10:38 |
| *** SerenaFeng has joined #openstack-ironic | 10:39 | |
| *** SerenaFeng has quit IRC | 10:43 | |
| skazi | @dtantsur|brb: can you explain how the rebuild patches are fixing this bug: https://bugzilla.redhat.com/show_bug.cgi?id=1433088 ? | 10:43 |
| openstack | skazi: Error: Could not parse XML returned by bugzilla.redhat.com: HTTP Error 403: Forbidden (https://bugzilla.redhat.com/show_bug.cgi?id=1433088&ctype=xml) | 10:43 |
| skazi | @dtantsur|brb: I'm looking at this code and I can't see any part which would prevent the configdrive to be stored in the DB | 10:44 |
| skazi | @dtantsur|brb: maybe I miss some important piece of this puzzle... | 10:45 |
| *** links has joined #openstack-ironic | 10:49 | |
| *** etingof|afk is now known as etingof | 10:50 | |
| *** vsaienk0 has joined #openstack-ironic | 10:56 | |
| *** dpawar has joined #openstack-ironic | 10:56 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add db migration and model for node bios table https://review.openstack.org/511162 | 11:05 |
| *** vsaienk0 has quit IRC | 11:09 | |
| *** stendulker_ has joined #openstack-ironic | 11:10 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add db migration and model for node bios table https://review.openstack.org/511162 | 11:11 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add bios_interface DB field, database migration, and object entry for node bios functionality. https://review.openstack.org/528609 | 11:12 |
| *** stendulker has quit IRC | 11:13 | |
| *** stendulker_ has quit IRC | 11:14 | |
| *** serlex has joined #openstack-ironic | 11:19 | |
| *** derekh has quit IRC | 11:23 | |
| *** derekh has joined #openstack-ironic | 11:23 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add bios_interface db field https://review.openstack.org/528609 | 11:24 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add db migration and model for node bios table https://review.openstack.org/511162 | 11:30 |
| *** milan_ has joined #openstack-ironic | 11:39 | |
| openstackgerrit | Merged openstack/ironic-inspector stable/queens: ironic_inspector: ironic: Fix 'auth_type' when 'auth_strategy' is used https://review.openstack.org/543577 | 11:41 |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: Fix for proper usage when keystone enabled https://review.openstack.org/543441 | 11:43 |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: Add way to customize the IPA version to be used https://review.openstack.org/542873 | 11:45 |
| *** olivierbourdon38 has joined #openstack-ironic | 11:47 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add db api layer for node bios operations https://review.openstack.org/511402 | 11:48 |
| *** vsaienk0 has joined #openstack-ironic | 11:58 | |
| *** robcresswell has quit IRC | 11:58 | |
| *** robcresswell has joined #openstack-ironic | 11:58 | |
| *** portdirect has quit IRC | 12:03 | |
| *** portdirect has joined #openstack-ironic | 12:03 | |
| *** vsaienk0 has quit IRC | 12:08 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add node BIOS support - RPC object https://review.openstack.org/511714 | 12:08 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add boot mode support to ManagementInterface https://review.openstack.org/526773 | 12:13 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Added redfish boot mode management https://review.openstack.org/528314 | 12:21 |
| *** dtantsur|brb is now known as dtantsur | 12:22 | |
| dtantsur | skazi: hi! the idea is to stop storing configdrive in the database at all. I'm not sure we're quite there though | 12:22 |
| *** rbartal has quit IRC | 12:26 | |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: Fix DHCP informations according to network interface settings https://review.openstack.org/542167 | 12:28 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Fix non-deployed nodes due to short nodes.instance_info https://review.openstack.org/334967 | 12:29 |
| *** jcoufal has joined #openstack-ironic | 12:30 | |
| *** hw_wutianwei has quit IRC | 12:30 | |
| *** vsaienk0 has joined #openstack-ironic | 12:32 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/queens: Change ironic-inspector to be source installed https://review.openstack.org/543919 | 12:32 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/queens: Update .gitreview for stable/queens https://review.openstack.org/542172 | 12:33 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/queens: Update UPPER_CONSTRAINTS_FILE for stable/queens https://review.openstack.org/542173 | 12:33 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/queens: Updated from global requirements https://review.openstack.org/543675 | 12:33 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Collect ironic-inspector logs https://review.openstack.org/542795 | 12:35 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Collect ironic-inspector logs https://review.openstack.org/542795 | 12:35 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add boot mode support to ManagementInterface https://review.openstack.org/526773 | 12:37 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Added redfish boot mode management https://review.openstack.org/528314 | 12:40 |
| *** rbartal has joined #openstack-ironic | 12:40 | |
| dtantsur | vsaienk0: hey, around? if yes, mind reviewing https://review.openstack.org/#/c/542214/ please? it should not require specific ironic-inspector knowledge | 12:41 |
| patchbot | patch 542214 - ironic-inspector - Only set switch_id in local_link_connection if it ... | 12:41 |
| *** vsaienk0 has quit IRC | 12:42 | |
| *** ricardoas has quit IRC | 12:42 | |
| *** ianychoi_ has joined #openstack-ironic | 12:42 | |
| *** jcoufal_ has joined #openstack-ironic | 12:43 | |
| *** ricardoas has joined #openstack-ironic | 12:44 | |
| *** ianychoi has quit IRC | 12:46 | |
| *** jcoufal has quit IRC | 12:47 | |
| dtantsur | sambetts|afk: ditto when you're no longer afk ^^^ | 12:47 |
| *** vsaienk0 has joined #openstack-ironic | 12:53 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: WIP: Add BIOSInterface to base driver class https://review.openstack.org/507793 | 12:57 |
| *** sambetts|afk is now known as sambetts | 12:57 | |
| sambetts | dtantsur: I'll take a look :) | 12:57 |
| *** rbartal has quit IRC | 12:59 | |
| dtantsur | thnx! | 12:59 |
| *** vsaienk0 has quit IRC | 13:02 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add Node BIOS support - REST API https://review.openstack.org/512579 | 13:14 |
| *** jlabarre has joined #openstack-ironic | 13:14 | |
| *** rbartal has joined #openstack-ironic | 13:15 | |
| *** pcaruana has quit IRC | 13:16 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add node BIOS support - RPC API https://review.openstack.org/512200 | 13:18 |
| *** jcoufal has joined #openstack-ironic | 13:19 | |
| *** olivierbourdon38 has quit IRC | 13:22 | |
| *** jcoufal_ has quit IRC | 13:22 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: [WIP] Retry port lists on failure in PXE filter periodic sync https://review.openstack.org/543928 | 13:27 |
| *** rloo has joined #openstack-ironic | 13:27 | |
| dtantsur | sambetts: also if you have time, mind providing some early feedback on the idea ^^^? | 13:27 |
| sambetts | dtantsur: sure | 13:27 |
| *** dprince has joined #openstack-ironic | 13:27 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Stop running standalone tests for classic drivers https://review.openstack.org/543930 | 13:30 |
| dtantsur | sambetts: just to clarify: this does not replace the ironic API fix, but rather complements it | 13:30 |
| *** oanson has quit IRC | 13:33 | |
| sambetts | dtantsur: yeah that makes sense | 13:34 |
| dtantsur | okay, I'll finish it then | 13:34 |
| *** oanson has joined #openstack-ironic | 13:35 | |
| dtantsur | sambetts: any objections to approving https://review.openstack.org/#/c/542214/ ? | 13:35 |
| patchbot | patch 542214 - ironic-inspector - Only set switch_id in local_link_connection if it ... | 13:35 |
| sambetts | dtantsur: nope, it was only that you asked vsaink0 to look but he doesn't seem to be around | 13:35 |
| dtantsur | yeah | 13:35 |
| dtantsur | morning rloo. bifrost queens needs https://review.openstack.org/#/c/543919/ first of all | 13:38 |
| patchbot | patch 543919 - bifrost (stable/queens) - Change ironic-inspector to be source installed | 13:38 |
| rloo | dtantsur: morning. #@%! ok. | 13:38 |
| dtantsur | I've rebased all other patches on top of it to make them pass | 13:38 |
| rloo | dtantsur: oh, that explains it. I can +A cuz you cherry-picked? | 13:38 |
| dtantsur | I believe so | 13:39 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add db migration and model for node bios table https://review.openstack.org/511162 | 13:39 |
| openstackgerrit | Merged openstack/bifrost master: Collect ironic-inspector logs https://review.openstack.org/542795 | 13:43 |
| dtantsur | yolanda: hi! if you have some time, maybe you could look into introducing a non-voting bifrost job to ironic and ironic-inspector? | 13:43 |
| *** dprince has quit IRC | 13:43 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add bios_interface db field https://review.openstack.org/528609 | 13:45 |
| *** ianychoi_ has quit IRC | 13:48 | |
| openstackgerrit | chandra shekar proposed openstack/ironic-specs master: Added new spec for L3 based Ironic deployment https://review.openstack.org/543936 | 13:48 |
| *** ianychoi_ has joined #openstack-ironic | 13:49 | |
| openstackgerrit | James E. Blair proposed openstack/networking-generic-switch stable/ocata: Zuul: Remove project name https://review.openstack.org/543938 | 13:49 |
| *** rloo has quit IRC | 13:49 | |
| *** fragatina has joined #openstack-ironic | 13:49 | |
| dtantsur | hjensas, sambetts, https://review.openstack.org/#/c/543936 hmm, looks related to the routed networks work | 13:50 |
| patchbot | patch 543936 - ironic-specs - Added new spec for L3 based Ironic deployment | 13:50 |
| *** rloo has joined #openstack-ironic | 13:50 | |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: Fix DHCP informations according to network interface settings https://review.openstack.org/542167 | 13:51 |
| * TheJulia raises an eyebrow | 13:53 | |
| TheJulia | Good morning everyone | 13:53 |
| etingof | o/ | 13:54 |
| TheJulia | Static l3 addressing... | 13:55 |
| dtantsur | morning TheJulia, afternoon etingof | 13:56 |
| *** zul has quit IRC | 13:56 | |
| dtantsur | TheJulia: this good use your attention: https://review.openstack.org/#/c/543555/ | 13:57 |
| patchbot | patch 543555 - ironic-tempest-plugin - Add jobs for stable/queens | 13:57 |
| dtantsur | s/good/could/ (WUT?) | 13:57 |
| *** zul has joined #openstack-ironic | 13:57 | |
| *** dprince has joined #openstack-ironic | 14:00 | |
| *** dpawar has quit IRC | 14:05 | |
| *** etingof has quit IRC | 14:05 | |
| openstackgerrit | Merged openstack/bifrost stable/queens: Change ironic-inspector to be source installed https://review.openstack.org/543919 | 14:05 |
| dtantsur | okay, this ^^^ should unblock bifrost stable/queens | 14:06 |
| dtantsur | TheJulia: should we backport it further? | 14:06 |
| TheJulia | absolutely | 14:06 |
| * dtantsur does too many pings to someone who is still having coffee | 14:06 | |
| *** zhangfei has quit IRC | 14:07 | |
| * TheJulia raises an eyebrow at gerrit not letting her do it via the website | 14:07 | |
| *** bfournie has joined #openstack-ironic | 14:09 | |
| *** bfournie1 has joined #openstack-ironic | 14:10 | |
| *** bfournie has quit IRC | 14:11 | |
| openstackgerrit | Merged openstack/bifrost stable/queens: Update .gitreview for stable/queens https://review.openstack.org/542172 | 14:11 |
| openstackgerrit | Merged openstack/bifrost stable/queens: Update UPPER_CONSTRAINTS_FILE for stable/queens https://review.openstack.org/542173 | 14:11 |
| openstackgerrit | Merged openstack/bifrost stable/queens: Updated from global requirements https://review.openstack.org/543675 | 14:11 |
| *** cdearborn has joined #openstack-ironic | 14:11 | |
| *** vsaienk0 has joined #openstack-ironic | 14:12 | |
| rloo | dtantsur: question about this backport to queens: https://review.openstack.org/#/c/540985/ | 14:15 |
| patchbot | patch 540985 - ironic-lib (stable/queens) - Use six.wraps() for Metrics so decorated methods c... | 14:15 |
| *** mjturek has joined #openstack-ironic | 14:15 | |
| rloo | dtantsur: if we backport it, woudl we push out a new version of ironic-lib? | 14:15 |
| rloo | dtantsur: and then change the upper constraints of the queens releases? | 14:15 |
| rloo | dtantsur: and if not, then why bother backporting? | 14:15 |
| dtantsur | rloo: upper-constraints will change automatically after the release IIUC | 14:16 |
| rloo | dtantsur: ok, so you think it is worth backporting that? or should i ask TheJulia. Don't know who will do the release :) | 14:17 |
| dtantsur | I think so, yes | 14:17 |
| *** Goneri has joined #openstack-ironic | 14:17 | |
| rloo | dtantsur: ok. | 14:17 |
| *** zhangfei has joined #openstack-ironic | 14:20 | |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: Fix DHCP informations according to network interface settings https://review.openstack.org/542167 | 14:22 |
| *** vsaienk0 has quit IRC | 14:22 | |
| *** vsaienk0 has joined #openstack-ironic | 14:25 | |
| *** fragatina has quit IRC | 14:31 | |
| *** openstackgerrit has quit IRC | 14:33 | |
| *** vsaienk0 has quit IRC | 14:35 | |
| *** r-daneel has quit IRC | 14:40 | |
| *** baha has joined #openstack-ironic | 14:44 | |
| rloo | mgoddard: wrt your 2 patches: https://review.openstack.org/#/c/543461/ & https://review.openstack.org/#/c/543574/, nova isn't going to land 543574 until rocky (and maybe backport? dunno?). w/o that, 543461 isn't useful. unless we json-decode at ironic end. | 14:44 |
| patchbot | patch 543461 - ironic - Validate instance_info.traits against node traits | 14:44 |
| patchbot | patch 543574 - nova - Don't JSON encode instance_info.traits for ironic | 14:44 |
| rloo | mgoddard: just wondering if we really care. or punt it all to rocky... | 14:44 |
| rloo | mgoddard: just means corner cases of flavor traits not matching node traits cuz admin changed the node traits between scheduling & deploy. right? | 14:45 |
| *** links has quit IRC | 14:47 | |
| *** openstackgerrit has joined #openstack-ironic | 14:52 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-inspector stable/queens: Only set switch_id in local_link_connection if it is a mac address https://review.openstack.org/543961 | 14:52 |
| *** mgoddard_ has joined #openstack-ironic | 14:53 | |
| mgoddard | rloo: I think we're going to need to support JSON decode of instance_info.traits, similarly to instance_info.capabilities | 14:54 |
| mgoddard | rloo: https://github.com/openstack/ironic/blob/180277a65a5ad7f9cdc55d55aee08ef25ff92f8b/ironic/drivers/modules/deploy_utils.py#L572 | 14:54 |
| TheJulia | dtantsur: I don't recall right now, ^^^ need furhter backporting? | 14:54 |
| mgoddard | rloo: given that at least one release of queens nova will use JSON instance_info.traits | 14:54 |
| mgoddard | rloo: so although it adds a little extra work, I don't think it changes whether we need to validate instance_info.traits or not | 14:54 |
| TheJulia | dtantsur: the ironic-inspector patch above | 14:54 |
| jroll | good morning everyone | 14:55 |
| TheJulia | good morning jroll | 14:55 |
| rloo | mgoddard: if your nova patch is backported to queens, then I am not sure ironic needs to support json-decode of traits. let me think | 14:55 |
| rloo | and good morning mgoddard, jroll, TheJulia, and everyone else! | 14:56 |
| dtantsur | TheJulia: the switch_id issue? yes, thanks! | 14:56 |
| dtantsur | morning jroll | 14:56 |
| jroll | \o | 14:56 |
| rloo | mgoddard: so ironic doesn't do anything with the instance_info.traits in queens, *unless* we want to validate against the node.traits | 14:57 |
| *** moshele has quit IRC | 14:58 | |
| rloo | mgoddard: and our upgrade process is ironic first, then nova. so ironic rocky will presumably do something with instance_info.traits, which it could get from nova.queens. | 14:58 |
| rloo | mgoddard: the nova/json fix will be in nova.rocky, if they backport to nova.queens, we're ok. | 14:58 |
| *** etingof has joined #openstack-ironic | 14:59 | |
| jroll | +1 what rloo said, we need to backport the nova thing | 14:59 |
| *** zhangfei has quit IRC | 15:00 | |
| rloo | jroll: do you think we need to validate instance_info.traits against node.traits in ironic.queens though? | 15:00 |
| rloo | jroll: i think in monday's meeting (was it only yesterday) mgoddard said it was a nice to have... | 15:01 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-inspector stable/pike: Only set switch_id in local_link_connection if it is a mac address https://review.openstack.org/543964 | 15:02 |
| rloo | mgoddard: let me know what you think; right now, I don't see a need to get your 543461 into queens, or to add a patch to json-decode instance_info.traits (the only reason for the latter is for our own development purposes i think) | 15:03 |
| *** vsaienk0 has joined #openstack-ironic | 15:03 | |
| mgoddard | morning rloo & all | 15:03 |
| mgoddard | rloo: correct, but the nova patch is not going to make the first queens release, so there may be some installations that end up using it | 15:03 |
| mgoddard | rloo: (or rather end up without the patch) | 15:03 |
| rloo | mgoddard: which is fine (well, sort of fine). that's what subsequent queens releases are for, to address issues in queens :) | 15:04 |
| jroll | rloo: I agree it's very nice to have, I support backporting validation to ironic queens, but not the end of the world if we don't | 15:04 |
| rloo | mgoddard: the point being that ironic.queens doesn't yet do anything with instance_info.traits, unless we do what jroll sez ^^ | 15:04 |
| jroll | we can also get it in nova before queens final, then it isn't a problem | 15:05 |
| rloo | mgoddard: and if we really think we need validation in queens, then that is a case to put to nova to have them merge that patch now, rather than later. | 15:05 |
| mgoddard_ | rloo: right, but if a user takes the first queens release and no more, then upgrades ironic to rocky, then they could get instance_info.traits in JSON format | 15:06 |
| jroll | mgoddard_: we can get it in before the first "final" queens release of nova | 15:07 |
| rloo | jroll: wrt the nova patch, it isn't clear they see it as a high priority, see https://bugs.launchpad.net/nova/+bug/1748942/comments/3 | 15:07 |
| openstack | Launchpad bug 1748942 in OpenStack Compute (nova) "Ironic virt driver should not JSON encode traits in instance_info" [Medium,In progress] - Assigned to Mark Goddard (mgoddard) | 15:07 |
| jroll | if someone runs an RC for a whole cycle, they deserve a failed build or three :P | 15:07 |
| rloo | jroll: hence my question, if we think it is a high priority, we should ask them to reconsider it. | 15:07 |
| jroll | rloo: yes, because we need queens nova to work with rocky ironic. the two options to do that are to fix this in nova queens or do compat hacks in rocky ironic | 15:08 |
| rloo | jroll: or we add a patch in ironic to support json-encoded traits temporarily. | 15:08 |
| jroll | yes, that's what I said, compat hacks | 15:08 |
| * jroll comments in the bug | 15:08 | |
| mgoddard_ | jroll: are you sure? I spoke to mriedem and he suggested otherwise. http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2018-02-12.log.html#t2018-02-12T19:30:25 | 15:10 |
| jroll | mgoddard_: right, I'm not sure it's worth holding up RC2. I believe nova's upgrade path dictates using the latest stable release before upgrading. if we believe users are sane and follow that, this is fine for 17.0.1 | 15:11 |
| rloo | i agree with jroll. (and it seems to me that in the past, they've had to backport stuff so anyone using nova must know this...) | 15:13 |
| * jroll stops commenting on the bug report as he realizes matt is just saying we should wait for rc2, not that we should not backport it | 15:14 | |
| mgoddard_ | jroll: right, although later in the conversation he said it would be post 17.0.0 | 15:15 |
| mgoddard_ | anyway, if the upgrade requires the latest stable release then we're good | 15:16 |
| jroll | mgoddard_: right right | 15:17 |
| jroll | makes sense to me | 15:17 |
| *** etingof has quit IRC | 15:17 | |
| mgoddard_ | although if we backport 543461 to queens then it seems likely we'd need a compat hack in queens at least | 15:18 |
| mgoddard_ | I was thinking that perhaps instance_info could do with some schema validation | 15:20 |
| mgoddard_ | that would have made this a lot easier to find | 15:20 |
| jroll | mgoddard_: right, I don't think we should backport that until nova has released a queens thing with the bugfix | 15:20 |
| dtantsur | ++ | 15:21 |
| dtantsur | we may even modify the release note when backporting | 15:21 |
| dtantsur | to say that nova has to be updated to $VERSION before | 15:21 |
| *** etingof has joined #openstack-ironic | 15:22 | |
| rloo | jroll, mgoddard_: i think we might want to put in a hack. at the very least, the code cannot assume that instance_info.traits is a list; we need to catch any exceptions... | 15:22 |
| rloo | ok, so now that i think about it, maybe we should put in a hack to json-decode. otherwise, it gets mucky wrt the process of upgrading, etc. easier to put in hack and remove it later. | 15:22 |
| mgoddard_ | rloo: I tend to agree. It could even be raise Exception('upgrade nova') | 15:22 |
| dtantsur | lol | 15:23 |
| jroll | idk, I'd almost just prefer to blow up the build if it isn't a list | 15:24 |
| rloo | dtantsur: heh. so, from operators point of view, we want to minimize surprises and things that are different from the norm. in this case, it is fairly trivial to do, and even if we forget to remove the hack (not that, cough, cough, it has ever happened) it won't hurt | 15:24 |
| rloo | jroll: really? | 15:24 |
| *** vsaienk0 has quit IRC | 15:24 | |
| jroll | rloo: either way we need to raise a proper error if it isn't something we recognize, to handle crazy people putting junk into instance_info.traits | 15:25 |
| rloo | jroll: yes, that is true. (although i wonder whether our code validates the existing instance_info stuff :) | 15:25 |
| jroll | I'm not opposed to trying to do json.decode() if it isn't a proper list | 15:25 |
| jroll | but, meh | 15:26 |
| mgoddard_ | the code already exists for capabilities, would just require a little copy/paste | 15:26 |
| rloo | so here's my suggestion. Add patch to try to json.decode() if it isn't a list; with a TODO to remove in S*?, backport it to queens. | 15:26 |
| mgoddard_ | and then there were two kludges... | 15:27 |
| rloo | then whenever we land the validation code, we can backport. | 15:27 |
| *** r-daneel has joined #openstack-ironic | 15:27 | |
| mgoddard_ | rloo: you mean convert in the API so that we never end up with JSON in the DB? | 15:28 |
| jroll | I wouldn't worry about the DB, we only validate at deploy time, it isn't used beyond that | 15:28 |
| jroll | or shouldn't be | 15:28 |
| * jroll thinks hm, rebuild | 15:29 | |
| rloo | mgoddard_: dang. depends on where you access the instance_info.traits. I was thinking it'd be in a similar/same place as capabilities, but your validation isn't done there. | 15:29 |
| rloo | mgoddard_: i think we probably want some helper function, that we call whenever we access instance_info.traits. with TODO all over. | 15:30 |
| mgoddard_ | something similar to parse_instance_info_capabilities would work - a validator | 15:30 |
| mgoddard_ | realised you'd beaten me to it | 15:30 |
| rloo | mgoddard_: great minds think alike. makes me wonder about not-great minds, but :) | 15:30 |
| openstackgerrit | Merged openstack/ironic-inspector master: Only set switch_id in local_link_connection if it is a mac address https://review.openstack.org/542214 | 15:30 |
| openstackgerrit | Merged openstack/python-ironicclient stable/queens: Update UPPER_CONSTRAINTS_FILE for stable/queens https://review.openstack.org/538082 | 15:30 |
| openstackgerrit | Merged openstack/ironic-lib stable/queens: Updated from global requirements https://review.openstack.org/543682 | 15:30 |
| openstackgerrit | Merged openstack/ironic-inspector stable/queens: Update UPPER_CONSTRAINTS_FILE for stable/queens https://review.openstack.org/541749 | 15:30 |
| jroll | I'm +1 on not sticking json strings into the db, fwiw | 15:31 |
| jroll | it can't be a good thing to do | 15:31 |
| rloo | jroll: ++ | 15:31 |
| openstackgerrit | Merged openstack/ironic master: Clean up RPC versions and database migrations for Rocky https://review.openstack.org/542728 | 15:31 |
| rloo | jroll: dang. as long as we don't try to access instance_info.traits *after* we remove the hack. for some long-lived instance that spans over releases :) | 15:32 |
| mgoddard_ | hmm, but if we can't guarantee that instance_info.traits in the DB is not JSON, now we need to convert in the API, and convert when accessing | 15:32 |
| mgoddard_ | argh great mind rloo | 15:33 |
| rloo | mgoddard_: i was only thinking about converting when accessing. i don't like the idea of changing/saving to db. | 15:33 |
| mgoddard_ | rloo: that seems simplest to me. I don't think the DB cares that much about nested JSON | 15:34 |
| rloo | mgoddard_: presumably (although things could change) we don't need to look at instance_info.traits after a node has been deployed. which is another reason for validating at deploy time, not at the API layer when someone tries to update the node's traits or instance_info.traits... | 15:35 |
| *** rpioso|afk is now known as rpioso | 15:35 | |
| rpioso | Good morning, ironicers. | 15:36 |
| rloo | dtantsur: should we backport this? https://review.openstack.org/#/c/540954/ | 15:37 |
| patchbot | patch 540954 - ironic - Add validate_rescue() method to boot interface (MERGED) | 15:37 |
| rloo | morning rpioso | 15:37 |
| rpioso | rloo: Hey :) | 15:38 |
| dtantsur | rloo: I've been thinking about it for some time, and I'm not decided. | 15:38 |
| rloo | dtantsur: or maybe we can't, it is a change to the interface. | 15:38 |
| dtantsur | right, this is the concern | 15:38 |
| rloo | dtantsur: i think it is fine if we don't backport. the only reason is to provide support for future RescueInterfaces, right? | 15:39 |
| dtantsur | however, we haven't officially released queens yet, so the stable branch is not quite stable yet | 15:39 |
| TheJulia | If we are going to, now is the time | 15:39 |
| rloo | dtantsur: ah, in that case, we shoudl backport | 15:39 |
| rloo | dtantsur: would be nice to add that reno to the big rescue reno in queens release though. (just saying :)) | 15:40 |
| dtantsur | similarly, for example, I'm about to introduce dependency on python-retrying to stable/queens.. | 15:42 |
| dtantsur | which will require bumping minor version of ironic-inspector | 15:43 |
| dtantsur | TheJulia: context for my ping on #openstack-release ^^^ | 15:43 |
| rloo | dtantsur: OH :-( That must be the critical bug you're trying to fix... i was going to ask you about that. | 15:43 |
| dtantsur | yep | 15:43 |
| dtantsur | essentially, ironic-inspector crashes if anything happens on the periodic port.list call | 15:44 |
| dtantsur | given that it's a call over network, it's a bit too harsh | 15:44 |
| rloo | dtantsur: yup. crashing is harsh :-( i'm guessing you might need to backport that too! | 15:44 |
| dtantsur | right. I asked about backporting a change with requirements to #openstack-release and I was told it's still fine | 15:45 |
| TheJulia | I think we can add the requirement since we will still be constrained | 15:45 |
| dtantsur | since it's in g-r already | 15:45 |
| TheJulia | if it is not in g-r... then... yeah | 15:45 |
| TheJulia | yup | 15:45 |
| * TheJulia double checks | 15:45 | |
| TheJulia | yup, it is | 15:46 |
| TheJulia | same version in pike as is current | 15:46 |
| * TheJulia waits out a grenade test | 15:47 | |
| *** vsaienk0 has joined #openstack-ironic | 15:51 | |
| *** vsaienk0 has quit IRC | 16:01 | |
| *** mjura has quit IRC | 16:03 | |
| *** openstackgerrit has quit IRC | 16:04 | |
| *** openstackgerrit has joined #openstack-ironic | 16:09 | |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Add jobs for stable/queens https://review.openstack.org/543555 | 16:09 |
| openstackgerrit | Merged openstack/ironic-lib stable/queens: Use six.wraps() for Metrics so decorated methods can be inspected https://review.openstack.org/540985 | 16:09 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Retry port lists on failure in PXE filter periodic sync https://review.openstack.org/543928 | 16:11 |
| dtantsur | okay, here is the patch ^^^ TheJulia, sambetts | 16:11 |
| *** moshele has joined #openstack-ironic | 16:11 | |
| *** VisBits has quit IRC | 16:15 | |
| *** VisBits has joined #openstack-ironic | 16:16 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Add a full set of tests for the IPMI hardware type to the standalone job https://review.openstack.org/543989 | 16:18 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Stop running standalone tests for classic drivers https://review.openstack.org/543930 | 16:18 |
| *** jcoufal_ has joined #openstack-ironic | 16:21 | |
| *** mgoddard_ has quit IRC | 16:22 | |
| *** jcoufal has quit IRC | 16:23 | |
| sambetts | dtantsur: thanks | 16:24 |
| *** vsaienk0 has joined #openstack-ironic | 16:24 | |
| openstackgerrit | Merged openstack/ironic master: Do not send sensors data for nodes in maintenance mode https://review.openstack.org/415173 | 16:26 |
| * sambetts --> away | 16:26 | |
| sambetts | night all | 16:26 |
| rloo | bye sambetts | 16:28 |
| sambetts | o/ rloo | 16:29 |
| *** sambetts is now known as sambetts|afk | 16:29 | |
| *** Sukhdev has joined #openstack-ironic | 16:30 | |
| *** milan_ has quit IRC | 16:30 | |
| *** vsaienk0 has quit IRC | 16:34 | |
| dtantsur | ImportError: /home/dtantsur/Projects/ironic/.tox/venv/lib/python2.7/site-packages/netifaces.so: undefined symbol: PyUnicodeUCS2_FromString | 16:35 |
| dtantsur | what the hell with out docs again.. | 16:35 |
| *** jcoufal_ has quit IRC | 16:38 | |
| *** etingof has quit IRC | 16:39 | |
| *** jcoufal has joined #openstack-ironic | 16:39 | |
| TheJulia | I quite literally just got that and forced a pip based re-install of netifaces to fix it | 16:39 |
| TheJulia | out of sync ubuntu packages floating around out there? | 16:41 |
| *** serlex has quit IRC | 16:43 | |
| *** moshele has quit IRC | 16:44 | |
| dtantsur | s/ubuntu/fedora/ in my case, and that's in a venv | 16:46 |
| *** pcaruana has joined #openstack-ironic | 16:47 | |
| *** olivierbourdon38 has joined #openstack-ironic | 16:49 | |
| * TheJulia blinks | 16:58 | |
| TheJulia | hit it again | 16:58 |
| rpioso | dtantsur: You review commented, "Unfortunately, we have already used the word "bios" before." at https://tinyurl.com/y8xb63xr | 16:59 |
| rpioso | dtantsur: Have we used the word "bios" in the context of boot mode? AFAIU, boot mode config is new. | 17:00 |
| TheJulia | we have | 17:00 |
| TheJulia | we offer a boot_mode parameter in instance_info and properties | 17:01 |
| rpioso | TheJulia: Do you have a pointer to some doc on that at your fingertips? | 17:02 |
| *** Sukhdev has quit IRC | 17:02 | |
| TheJulia | https://docs.openstack.org/ironic/pike/install/include/boot-mode.html | 17:03 |
| *** jcoufal_ has joined #openstack-ironic | 17:03 | |
| *** moshele has joined #openstack-ironic | 17:03 | |
| rpioso | TheJulia: ty | 17:04 |
| *** jcoufal has quit IRC | 17:05 | |
| TheJulia | dtantsur: what version of netifaces do you have? | 17:05 |
| *** moshele has quit IRC | 17:07 | |
| TheJulia | https://github.com/al45tair/netifaces/issues/2 | 17:12 |
| rpioso | TheJulia: And congrats! (I missed yesterday's meeting, because I was out on vacation.) | 17:14 |
| TheJulia | thanks! | 17:14 |
| dtantsur | TheJulia: 0.10.6, yeah | 17:19 |
| *** crushil has joined #openstack-ironic | 17:22 | |
| *** marios has quit IRC | 17:22 | |
| *** crushil has quit IRC | 17:23 | |
| *** crushil has joined #openstack-ironic | 17:23 | |
| *** moshele has joined #openstack-ironic | 17:28 | |
| dtantsur | TheJulia: do I get it right that --os-auth-plugin=none STILL does not work? | 17:33 |
| *** moshele has quit IRC | 17:34 | |
| TheJulia | hmmm | 17:34 |
| TheJulia | did you try --os-auth-plugin=noauth? | 17:35 |
| *** e0ne has quit IRC | 17:35 | |
| dtantsur | OpenStackConfigException: Missing value user-id required for auth plugin noauth | 17:36 |
| dtantsur | and with none it's | 17:36 |
| dtantsur | AttributeError: 'NoAuth' object has no attribute 'get_auth_ref' | 17:36 |
| TheJulia | hahaha | 17:36 |
| TheJulia | every cycle I swear | 17:37 |
| rloo | dtantsur: don't know if this is what you're interested in: https://bugs.launchpad.net/python-openstackclient/+bug/1724283 | 17:37 |
| openstack | Launchpad bug 1724283 in python-openstackclient "osc-lib is not able to validate a none plugin parameters in the CLI" [Undecided,In progress] - Assigned to Vladyslav Drok (vdrok) | 17:37 |
| *** AlexeyAbashkin has quit IRC | 17:38 | |
| *** vsaienk0 has joined #openstack-ironic | 17:38 | |
| dtantsur | yeah.. | 17:39 |
| dtantsur | okay, I'll document using a fake token.. again | 17:39 |
| vdrok | dtantsur: correct, still does not work | 17:40 |
| dtantsur | can we make some noise about it? <-- TheJulia it will hurt us after removal of "ironic" | 17:40 |
| vdrok | one patch left there | 17:40 |
| TheJulia | lol | 17:40 |
| rloo | sorry vdrok, i know you wanted me to review that... | 17:41 |
| vdrok | rloo: np, not sure it will make any difference, the patch is super easy | 17:42 |
| rloo | vdrok: i will look today unless i get interrupted :) | 17:42 |
| vdrok | thx! | 17:42 |
| *** devananda has joined #openstack-ironic | 17:45 | |
| *** vsaienk0 has quit IRC | 17:48 | |
| openstackgerrit | James E. Blair proposed openstack/python-ironicclient stable/pike: Zuul: Remove project name https://review.openstack.org/544024 | 17:51 |
| dtantsur | https://bugs.launchpad.net/ironic/+bug/1749256 sigh | 17:54 |
| openstack | Launchpad bug 1749256 in Ironic "FakeHardware does not support noop implementations" [Low,Triaged] - Assigned to Dmitry Tantsur (divius) | 17:54 |
| TheJulia | ugh | 17:55 |
| *** pcaruana has quit IRC | 17:55 | |
| dtantsur | spotted this while making updates to the dev quickstart | 17:55 |
| *** mgoddard_ has joined #openstack-ironic | 17:58 | |
| *** Sukhdev has joined #openstack-ironic | 18:00 | |
| *** derekh has quit IRC | 18:02 | |
| *** pcaruana has joined #openstack-ironic | 18:06 | |
| TheJulia | now lets see, grenade is spinning... next() | 18:07 |
| TheJulia | of course, I spoke too soon | 18:09 |
| *** jcoufal has joined #openstack-ironic | 18:10 | |
| *** jcoufal_ has quit IRC | 18:12 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add missing noop implementations to fake-hardware https://review.openstack.org/544032 | 18:13 |
| dtantsur | TheJulia: another candidate for queens ^^^ | 18:13 |
| rloo | vdrok: wrt https://review.openstack.org/#/c/512699/, how does one pass/specify the endpoint_override? (quite possible i have no idea what is going on) | 18:17 |
| patchbot | patch 512699 - osc-lib - Allow to use the none auth plugin | 18:17 |
| vdrok | rloo: it is only possible when constructing plugin in code | 18:17 |
| vdrok | here it is https://review.openstack.org/#/c/515730/3/keystoneauth1/noauth.py | 18:18 |
| patchbot | patch 515730 - keystoneauth - Make none auth usable in CLI (MERGED) | 18:18 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-inspector stable/ocata: Use native v3 tox jobs https://review.openstack.org/544034 | 18:18 |
| rloo | vdrok: OH. is that how the other plugins work too? that you can only specify the override when constructing the plugin? | 18:19 |
| TheJulia | dtantsur: ^^^ should (hopefully) fix the ocata gate | 18:19 |
| vdrok | rloo: yup | 18:19 |
| TheJulia | for ironic-inspector | 18:19 |
| rloo | vdrok: ok then, yay for consistency :) | 18:19 |
| vdrok | rloo: in cli, what's the point of override if you write --os-endpoint ? :) | 18:19 |
| rloo | vdrok: oh, right. | 18:19 |
| *** Sukhdev has quit IRC | 18:22 | |
| *** etingof has joined #openstack-ironic | 18:22 | |
| dtantsur | okay, I guess I'll finish updating the contributor guide tomorrow.. | 18:24 |
| *** dtantsur is now known as dtantsur|afk | 18:24 | |
| dtantsur|afk | o/ | 18:24 |
| vdrok | good night | 18:25 |
| TheJulia | goodnight dtantsur|afk | 18:26 |
| rloo | bye dtantsur|afk | 18:29 |
| *** jcoufal_ has joined #openstack-ironic | 18:35 | |
| openstackgerrit | Merged openstack/ironic master: releasing docs: document stable jobs for the tempest plugin https://review.openstack.org/543558 | 18:37 |
| *** jcoufal has quit IRC | 18:38 | |
| TheJulia | rloo: if you have time, a glance at https://review.openstack.org/#/c/543961/ would be appreciated | 18:39 |
| patchbot | patch 543961 - ironic-inspector (stable/queens) - Only set switch_id in local_link_connection if it ... | 18:39 |
| rloo | TheJulia: ok, in the next hour maybe... | 18:40 |
| TheJulia | thanks! | 18:40 |
| *** e0ne has joined #openstack-ironic | 18:40 | |
| *** tesseract has quit IRC | 18:40 | |
| *** marlinc has quit IRC | 18:41 | |
| *** Sukhdev has joined #openstack-ironic | 18:41 | |
| *** baha has quit IRC | 18:43 | |
| *** moshele has joined #openstack-ironic | 18:44 | |
| *** pcaruana has quit IRC | 18:45 | |
| *** marlinc has joined #openstack-ironic | 18:50 | |
| *** moshele has quit IRC | 18:56 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: DNM test patch for AIO setup https://review.openstack.org/535596 | 19:02 |
| *** harlowja has joined #openstack-ironic | 19:03 | |
| *** baha has joined #openstack-ironic | 19:04 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add db api layer for node bios operations https://review.openstack.org/511402 | 19:04 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add db api layer for node bios operations https://review.openstack.org/511402 | 19:05 |
| *** jcoufal_ has quit IRC | 19:06 | |
| *** jcoufal has joined #openstack-ironic | 19:07 | |
| *** Sukhdev has quit IRC | 19:07 | |
| *** moshele has joined #openstack-ironic | 19:08 | |
| *** etingof has quit IRC | 19:12 | |
| *** moshele has quit IRC | 19:12 | |
| *** etingof has joined #openstack-ironic | 19:12 | |
| rloo | TheJulia: you want me to approve the pike one too? 543964? | 19:15 |
| TheJulia | yeah, likely good to do them all at the same time | 19:15 |
| rloo | err... two cherry picks? | 19:15 |
| TheJulia | oh, I must have forgotten to remove the other line | 19:16 |
| rloo | TheJulia: does it matter? | 19:16 |
| TheJulia | no | 19:16 |
| rloo | TheJulia: ok then, here we go.... | 19:17 |
| *** ijw has joined #openstack-ironic | 19:22 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Validate instance_info.traits against node traits https://review.openstack.org/543461 | 19:27 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add db api layer for node bios operations https://review.openstack.org/511402 | 19:29 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add db api layer for node bios operations https://review.openstack.org/511402 | 19:33 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add bios_interface db field https://review.openstack.org/528609 | 19:33 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add db migration and model for node bios table https://review.openstack.org/511162 | 19:34 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add node BIOS support - RPC API https://review.openstack.org/512200 | 19:34 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add Node BIOS support - REST API https://review.openstack.org/512579 | 19:34 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: WIP: Add BIOSInterface to base driver class https://review.openstack.org/507793 | 19:34 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add node BIOS support - RPC object https://review.openstack.org/511714 | 19:35 |
| *** jcoufal_ has joined #openstack-ironic | 19:38 | |
| *** jcoufal has quit IRC | 19:40 | |
| *** crushil has quit IRC | 19:41 | |
| *** crushil has joined #openstack-ironic | 19:42 | |
| *** moshele has joined #openstack-ironic | 19:45 | |
| *** moshele has quit IRC | 19:51 | |
| rloo | TheJulia: wrt this RFE you opened: https://bugs.launchpad.net/ironic/+bug/1735478, do you think it needs a spec? | 19:54 |
| openstack | Launchpad bug 1735478 in Ironic "Cinder-less BFV support" [Wishlist,Confirmed] - Assigned to Michael Turek (mjturek) | 19:54 |
| TheJulia | no, it is a minor logic change all things considered | 19:55 |
| rloo | TheJulia: ok, the etherpad has 2? ideas. Is this Just for a noop storage interface? | 19:57 |
| rloo | TheJulia: "Another possibty - preconfigured remote disks in ironic and a SAN, nova is completely unaware." | 19:57 |
| TheJulia | oh cinderless... hmmmm so actually, two cases. the other possibiity is the minor logic change, the cinderless support would actually bejust use of noop and ensuring that we don't have anything predicated expecting that, but we build things with that in mind | 19:59 |
| *** jcoufal has joined #openstack-ironic | 20:00 | |
| TheJulia | it would really... truly be template generation | 20:00 |
| rloo | TheJulia: what's this particular RFE for? :) | 20:00 |
| *** jcoufal__ has joined #openstack-ironic | 20:02 | |
| *** jcoufal_ has quit IRC | 20:02 | |
| *** jcoufal has quit IRC | 20:05 | |
| TheJulia | both are useful in standalone use cases where people have pre-existing SANs with pre-configured luns or just generate the pxe config | 20:05 |
| TheJulia | failure to generate pxe config with a "noop" as the storage_interface would really be a bug | 20:06 |
| TheJulia | sorry, context switching a lot today | 20:06 |
| rloo | TheJulia: so... is this RFE meant to address both? | 20:08 |
| rloo | TheJulia: I'm trying to get clarification on the scope of work for that RFE | 20:08 |
| TheJulia | I understand | 20:08 |
| rloo | TheJulia: it is OK if it is 'someone needs to investigate/think about it...' :) | 20:09 |
| TheJulia | that is exactly why I created it, so we could do it as a spec for overall usage enhancements | 20:09 |
| rloo | TheJulia: ok, then it needs a spec. I'll update it in a few min with ^^. | 20:10 |
| openstackgerrit | Christopher Dearborn proposed openstack/ironic master: Added known issue to iDRAC driver docs https://review.openstack.org/544061 | 20:10 |
| *** moshele has joined #openstack-ironic | 20:14 | |
| TheJulia | ok | 20:17 |
| *** moshele has quit IRC | 20:18 | |
| *** etingof has quit IRC | 20:18 | |
| *** e0ne_ has joined #openstack-ironic | 20:22 | |
| *** e0ne has quit IRC | 20:25 | |
| *** mgoddard_ has quit IRC | 20:30 | |
| *** SerenaFeng has joined #openstack-ironic | 20:32 | |
| *** jcoufal has joined #openstack-ironic | 20:34 | |
| *** jcoufal__ has quit IRC | 20:36 | |
| *** etingof has joined #openstack-ironic | 20:43 | |
| *** jcoufal_ has joined #openstack-ironic | 20:55 | |
| *** jcoufal__ has joined #openstack-ironic | 20:56 | |
| *** jcoufal has quit IRC | 20:58 | |
| *** jcoufal has joined #openstack-ironic | 20:59 | |
| *** jcoufal_ has quit IRC | 20:59 | |
| *** jcoufal has quit IRC | 21:00 | |
| *** jcoufal__ has quit IRC | 21:02 | |
| *** Sukhdev has joined #openstack-ironic | 21:18 | |
| openstackgerrit | Merged openstack/ironic-inspector stable/queens: Only set switch_id in local_link_connection if it is a mac address https://review.openstack.org/543961 | 21:19 |
| openstackgerrit | Merged openstack/ironic-inspector stable/pike: Only set switch_id in local_link_connection if it is a mac address https://review.openstack.org/543964 | 21:19 |
| *** olivierbourdon38 has quit IRC | 21:22 | |
| *** ijw has quit IRC | 21:23 | |
| *** ijw has joined #openstack-ironic | 21:24 | |
| *** e0ne_ has quit IRC | 21:30 | |
| *** dprince has quit IRC | 21:31 | |
| *** SerenaFeng has quit IRC | 21:32 | |
| *** bfournie1 has quit IRC | 21:35 | |
| *** rbartal has quit IRC | 21:41 | |
| * TheJulia sighs about grenade | 21:43 | |
| *** openstackstatus has quit IRC | 21:43 | |
| *** Goneri has quit IRC | 21:44 | |
| *** openstackstatus has joined #openstack-ironic | 21:44 | |
| *** ChanServ sets mode: +v openstackstatus | 21:44 | |
| *** etingof has quit IRC | 21:50 | |
| *** afazekas has quit IRC | 21:50 | |
| *** afazekas has joined #openstack-ironic | 21:52 | |
| *** crushil has quit IRC | 21:52 | |
| *** rbartal has joined #openstack-ironic | 21:53 | |
| *** rcernin has joined #openstack-ironic | 22:05 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add security tips https://review.openstack.org/466729 | 22:06 |
| *** r-daneel has quit IRC | 22:07 | |
| *** crushil has joined #openstack-ironic | 22:09 | |
| *** baha has quit IRC | 22:10 | |
| *** crushil_ has joined #openstack-ironic | 22:11 | |
| *** crushil has quit IRC | 22:14 | |
| *** bfournie has joined #openstack-ironic | 22:34 | |
| * TheJulia just glares at reproduce.sh | 22:38 | |
| *** etingof has joined #openstack-ironic | 22:41 | |
| *** ijw has quit IRC | 22:57 | |
| openstackgerrit | Christopher Dearborn proposed openstack/ironic master: Added known issue to iDRAC driver docs https://review.openstack.org/544061 | 23:03 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add 'Other considerations' to security doc https://review.openstack.org/466729 | 23:03 |
| *** ricardoas has quit IRC | 23:17 | |
| *** ijw has joined #openstack-ironic | 23:24 | |
| *** cdearborn has quit IRC | 23:25 | |
| *** zshi has quit IRC | 23:30 | |
| TheJulia | rpioso: does your doc patch need to be on the queens branch? | 23:33 |
| TheJulia | err, not rpioso chris dearborn who is not here | 23:33 |
| *** Sukhdev has quit IRC | 23:42 | |
| *** ijw has quit IRC | 23:42 | |
| *** Sukhdev has joined #openstack-ironic | 23:46 | |
| *** charcol has joined #openstack-ironic | 23:46 | |
| *** Sukhdev has quit IRC | 23:48 | |
| rpioso | TheJulia: Good question! I'll defer to cdearborn. | 23:50 |
| rpioso | TheJulia: He may be afk for the night. | 23:51 |
| *** ijw has joined #openstack-ironic | 23:52 | |
| TheJulia | That is what I figured :\ | 23:54 |
| * TheJulia tries grenade... one... more... time | 23:54 | |
| JayF | If grenade is to be believed, every openstack upgrade succeeds on the 3rd or 4th try ;) | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!