| *** xuhaiwei has quit IRC | 00:01 | |
| *** oomichi has joined #openstack-ironic | 00:02 | |
| *** cdearborn_ has joined #openstack-ironic | 00:08 | |
| *** cdearborn__ has joined #openstack-ironic | 00:16 | |
| *** cdearborn has quit IRC | 00:18 | |
| openstackgerrit | yuan liang proposed openstack/ironic-inspector master: Add selinux check in Troubleshooting PXE boot document https://review.openstack.org/482848 | 00:21 |
|---|---|---|
| *** cdearborn_ has quit IRC | 00:29 | |
| *** zhangfei has quit IRC | 00:30 | |
| *** karimb has quit IRC | 00:31 | |
| *** masber has quit IRC | 00:33 | |
| *** masber has joined #openstack-ironic | 00:33 | |
| *** cdearborn__ has quit IRC | 00:35 | |
| *** mtanino has quit IRC | 00:35 | |
| *** krtaylor has joined #openstack-ironic | 00:40 | |
| *** zhangfei has joined #openstack-ironic | 00:43 | |
| *** bfournie has joined #openstack-ironic | 00:47 | |
| *** fragatin_ has joined #openstack-ironic | 00:51 | |
| *** fragatina has quit IRC | 00:54 | |
| *** fragati__ has joined #openstack-ironic | 00:54 | |
| *** rama_y_ has quit IRC | 00:55 | |
| *** fragatin_ has quit IRC | 00:55 | |
| *** fragatina has joined #openstack-ironic | 00:56 | |
| *** fragatina has quit IRC | 00:57 | |
| *** fragatin_ has joined #openstack-ironic | 00:57 | |
| masber | hi, would like to ask, does ironic project supports kolla project? | 00:58 |
| *** jkilpatr has quit IRC | 00:58 | |
| *** fragatin_ has quit IRC | 00:59 | |
| *** fragatina has joined #openstack-ironic | 00:59 | |
| *** fragati__ has quit IRC | 00:59 | |
| *** fragatina has quit IRC | 01:03 | |
| *** vishwanathj has joined #openstack-ironic | 01:12 | |
| *** fragatina has joined #openstack-ironic | 01:12 | |
| *** vishwana_ has quit IRC | 01:12 | |
| *** fragatina has quit IRC | 01:14 | |
| *** rloo has quit IRC | 01:16 | |
| *** crushil_ has joined #openstack-ironic | 01:23 | |
| *** crushil_ has quit IRC | 01:41 | |
| *** baoli has joined #openstack-ironic | 02:06 | |
| *** phuongnh has joined #openstack-ironic | 02:06 | |
| *** crushil has quit IRC | 02:07 | |
| *** crushil has joined #openstack-ironic | 02:07 | |
| *** mtanino_ has joined #openstack-ironic | 02:12 | |
| *** crushil has quit IRC | 02:22 | |
| *** crushil has joined #openstack-ironic | 02:22 | |
| *** crushil has quit IRC | 02:36 | |
| *** yufei has joined #openstack-ironic | 02:54 | |
| *** r-daneel has joined #openstack-ironic | 03:05 | |
| openstackgerrit | Debayan Ray proposed openstack/proliantutils master: Remove sushy manager hack https://review.openstack.org/485806 | 03:06 |
| wong_ | hello,Do you have any method or tool to know whether the deploy image have right driver and configuration? | 03:08 |
| *** baoli has quit IRC | 03:09 | |
| *** fragatina has joined #openstack-ironic | 03:12 | |
| *** Sukhdev has joined #openstack-ironic | 03:14 | |
| *** fragatina has quit IRC | 03:16 | |
| *** jaganathan has joined #openstack-ironic | 03:28 | |
| *** jaganathan has quit IRC | 03:35 | |
| openstackgerrit | Merged openstack/proliantutils master: Remove sushy manager hack https://review.openstack.org/485806 | 03:36 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Adds macs discovery https://review.openstack.org/483127 | 03:41 |
| *** gcb has quit IRC | 03:49 | |
| *** gcb has joined #openstack-ironic | 03:52 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Correct BFV PXE logic and debugging https://review.openstack.org/485812 | 03:53 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: BFV Deploy skip minor logging, logic, and test fixes https://review.openstack.org/479326 | 03:54 |
| *** fragatina has joined #openstack-ironic | 03:56 | |
| *** sasha21 has quit IRC | 03:56 | |
| *** fragatina has quit IRC | 03:56 | |
| *** fragatina has joined #openstack-ironic | 03:57 | |
| *** mtanino_ has quit IRC | 03:57 | |
| TheJulia | masber: I'm not sure I completely understand your question. As I understand it, they have support to deploy ironic, and I believe a number of their users deploy using ironic as well, but I can only go by what some of their developers say from time to time when I chat with them. | 03:58 |
| *** mtanino_ has joined #openstack-ironic | 03:59 | |
| TheJulia | wong_: I think a tool wouldn't really work since we're supporting such a wide and diverse ecosystem. Most operators seem to use the golden image concept. Validate it works in their environment and update as the know it works. | 03:59 |
| masber | TheJulia, I see, one of the developers told me Ironic is only supported on kolla-kubernetes | 04:00 |
| TheJulia | wong_: which really means tracking your hardware driver requirements and what is built into, or what os your using your deploy ramdisk with. | 04:00 |
| TheJulia | masber: i'm not sure I really get the distinction, but I mainly focus on Ironic | 04:01 |
| TheJulia | masber: oh, deploy k8s nodes with ironic? | 04:01 |
| TheJulia | masber: I _think_ that is in part due to the base OS images, but again, I don't track their work closely | 04:03 |
| yufei | :D good morning, ironicers | 04:07 |
| *** hw_wutianwei has joined #openstack-ironic | 04:08 | |
| *** mtanino_ has quit IRC | 04:10 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Correct BFV PXE logic and debugging https://review.openstack.org/485812 | 04:11 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: BFV Deploy skip minor logging, logic, and test fixes https://review.openstack.org/479326 | 04:12 |
| TheJulia | yufei: good morning, and goodnight :) | 04:12 |
| TheJulia | jlvillal: thanks for the review! :) | 04:12 |
| * TheJulia crosses her fingers and hopes that things work | 04:13 | |
| * TheJulia heads off to finally... actually sleep | 04:14 | |
| yufei | good night | 04:15 |
| *** adisky__ has joined #openstack-ironic | 04:18 | |
| openstackgerrit | Chason Chan proposed openstack/python-ironic-inspector-client master: Update the documentation link for doc migration https://review.openstack.org/485120 | 04:25 |
| *** aparnav has joined #openstack-ironic | 04:30 | |
| *** hoonetorg has joined #openstack-ironic | 04:32 | |
| *** yufei has left #openstack-ironic | 04:34 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-specs master: Hardware inspection support for standard Redfish driver https://review.openstack.org/438859 | 04:43 |
| *** hshiina has joined #openstack-ironic | 04:43 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Correct BFV PXE logic and debugging https://review.openstack.org/485812 | 04:46 |
| TheJulia | jlvillal: sigh ^^^, nova requests validates in available state | 04:47 |
| * TheJulia really goes to bed now | 04:47 | |
| *** mvpnitesh has joined #openstack-ironic | 04:52 | |
| *** syqian_ has quit IRC | 04:58 | |
| *** syqian_ has joined #openstack-ironic | 04:58 | |
| *** r-daneel has quit IRC | 05:01 | |
| *** r-daneel has joined #openstack-ironic | 05:02 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client master: Updated from global requirements https://review.openstack.org/485958 | 05:06 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient master: Updated from global requirements https://review.openstack.org/485959 | 05:07 |
| *** marios has joined #openstack-ironic | 05:14 | |
| *** fragatina has quit IRC | 05:15 | |
| *** jjung has joined #openstack-ironic | 05:16 | |
| *** marios_ has joined #openstack-ironic | 05:19 | |
| *** rcernin has quit IRC | 05:22 | |
| *** Sukhdev has quit IRC | 05:26 | |
| *** rcernin has joined #openstack-ironic | 05:36 | |
| *** marios_ has quit IRC | 05:43 | |
| *** marios has quit IRC | 05:43 | |
| *** marios has joined #openstack-ironic | 05:43 | |
| *** mjura has joined #openstack-ironic | 05:44 | |
| *** yufei has joined #openstack-ironic | 05:47 | |
| *** syqian__ has joined #openstack-ironic | 05:47 | |
| *** alex_xu has quit IRC | 05:48 | |
| *** alex_xu has joined #openstack-ironic | 05:49 | |
| *** syqian_ has quit IRC | 05:51 | |
| *** alex_xu has quit IRC | 05:56 | |
| *** alex_xu has joined #openstack-ironic | 05:56 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Discover iscsi_boot attribute https://review.openstack.org/341653 | 06:00 |
| *** ccamacho has quit IRC | 06:02 | |
| *** ccamacho has joined #openstack-ironic | 06:02 | |
| *** syqian__ has quit IRC | 06:10 | |
| openstackgerrit | paresh sao proposed openstack/proliantutils master: Redfish: Follow-up patch for set pending boot mode https://review.openstack.org/485982 | 06:14 |
| *** ChubYann has quit IRC | 06:15 | |
| *** skazi has joined #openstack-ironic | 06:18 | |
| *** ccamacho is now known as ccamacho|brb | 06:21 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Discover Storage details https://review.openstack.org/341370 | 06:22 |
| *** amarao has joined #openstack-ironic | 06:28 | |
| openstackgerrit | paresh sao proposed openstack/proliantutils master: Redfish: Follow-up patch for set pending boot mode https://review.openstack.org/485982 | 06:28 |
| *** phuongnh has quit IRC | 06:30 | |
| openstackgerrit | paresh sao proposed openstack/proliantutils master: Redfish: [Fix] Set pending boot mode json payload https://review.openstack.org/485982 | 06:31 |
| *** phuongnh has joined #openstack-ironic | 06:31 | |
| *** ralonsoh has joined #openstack-ironic | 06:32 | |
| *** jhesketh has quit IRC | 06:35 | |
| *** jhesketh has joined #openstack-ironic | 06:35 | |
| *** links has joined #openstack-ironic | 06:37 | |
| *** amarao has quit IRC | 06:37 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-specs master: Hardware inspection support for standard Redfish driver https://review.openstack.org/438859 | 06:39 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Discover Storage details https://review.openstack.org/341370 | 06:41 |
| *** jaganathan has joined #openstack-ironic | 06:45 | |
| *** ralonsoh has quit IRC | 06:45 | |
| *** ralonsoh has joined #openstack-ironic | 06:46 | |
| *** ccamacho|brb is now known as ccamacho | 06:48 | |
| *** _milan_ has joined #openstack-ironic | 06:49 | |
| *** athomas has joined #openstack-ironic | 06:49 | |
| *** ralonsoh has quit IRC | 06:57 | |
| *** rcernin has quit IRC | 06:58 | |
| *** pc_m has quit IRC | 06:58 | |
| *** ralonsoh has joined #openstack-ironic | 06:58 | |
| *** rcernin has joined #openstack-ironic | 06:58 | |
| *** yufei has quit IRC | 06:59 | |
| *** dtantsur|afk is now known as dtantsur | 07:00 | |
| *** pc_m has joined #openstack-ironic | 07:00 | |
| *** phuongnh has quit IRC | 07:03 | |
| *** yufei has joined #openstack-ironic | 07:04 | |
| *** _milan_ is now known as milan | 07:07 | |
| *** phuongnh has joined #openstack-ironic | 07:07 | |
| milan | morning Ironic! :) | 07:07 |
| *** pc_m has quit IRC | 07:07 | |
| *** pc_m has joined #openstack-ironic | 07:08 | |
| *** vgadiraj has quit IRC | 07:08 | |
| *** joanna has quit IRC | 07:08 | |
| *** mpotdar has quit IRC | 07:08 | |
| openstackgerrit | Aparna proposed openstack/proliantutils master: Redfish: Adds server capabilities https://review.openstack.org/484360 | 07:11 |
| wong_ | morning and evening :-) | 07:14 |
| dtantsur | morning/evening! | 07:15 |
| wong_ | aha | 07:15 |
| *** tesseract has joined #openstack-ironic | 07:16 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: PGSQL: create Enum before using https://review.openstack.org/472725 | 07:17 |
| milan | morning dtantsur! :) | 07:18 |
| milan | this patch was laying around for a bit (same as the bug) so decided to update the commit message to attract dear reviewers ;) ^ | 07:19 |
| *** mgoddard_ has joined #openstack-ironic | 07:22 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Log warning when API version is not specified for the ironic tool https://review.openstack.org/484866 | 07:32 |
| openstackgerrit | paresh sao proposed openstack/proliantutils master: Redfish: [Fix] Set pending boot mode json payload https://review.openstack.org/485982 | 07:36 |
| masber | good afternoon all, what is the difference between the my-image.mvlinux image generated by the command "disk-image-create ubuntu baremetal dhcp-all-interfaces grub2 -o my-image" and the coreos_production_pxe.vmlinuz generated by coreOS tools? to me they both the same (kernel disks images) so why do we need 2? | 07:37 |
| *** yufei has quit IRC | 07:41 | |
| *** yufei has joined #openstack-ironic | 07:42 | |
| *** ccamacho is now known as ccamacho|brb | 07:43 | |
| *** yufei has left #openstack-ironic | 07:45 | |
| *** phuongnh has quit IRC | 07:49 | |
| mgoddard_ | hi masber. The former image is the kernel for your machine image that will be deployed by ironic. If you specify the baremetal element, you'll get a single partition image and a kernel and ramdisk which may be used to PXE boot the machine image if the node has capabilities:local_boot=netboot. | 07:52 |
| mgoddard_ | coreos_production_pxe.vmlinuz is the kernel that does with a coreos ramdisk and contains the ironic python agent deployment agent. This is used by ironic when provisioning the node but users do not use it directly | 07:53 |
| openstackgerrit | paresh sao proposed openstack/proliantutils master: Redfish: [Fix] Set pending boot mode json payload https://review.openstack.org/485982 | 07:59 |
| *** lucas-afk is now known as lucasagomes | 08:03 | |
| *** dims has quit IRC | 08:09 | |
| *** dims has joined #openstack-ironic | 08:11 | |
| *** yufei has joined #openstack-ironic | 08:14 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Update the documentation link for doc migration https://review.openstack.org/485120 | 08:18 |
| *** dims has quit IRC | 08:21 | |
| *** dims has joined #openstack-ironic | 08:23 | |
| *** etingof has joined #openstack-ironic | 08:24 | |
| *** mvpnitesh has quit IRC | 08:29 | |
| openstackgerrit | bhavani proposed openstack/bifrost master: Handle log message interpolation by the logger in common/ https://review.openstack.org/485123 | 08:29 |
| *** karimb has joined #openstack-ironic | 08:37 | |
| openstackgerrit | howard lee proposed openstack/ironic-lib master: Update and replace http with https for doc links https://review.openstack.org/486047 | 08:40 |
| *** marios has quit IRC | 08:41 | |
| *** derekh has joined #openstack-ironic | 08:42 | |
| *** marios has joined #openstack-ironic | 08:42 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Log warning when API version is not specified for the ironic tool https://review.openstack.org/484866 | 08:50 |
| *** ccamacho|brb is now known as ccamacho | 08:51 | |
| *** skazi__ has joined #openstack-ironic | 08:57 | |
| *** skazi has quit IRC | 09:00 | |
| *** skazi has joined #openstack-ironic | 09:00 | |
| *** skazi__ has quit IRC | 09:01 | |
| openstackgerrit | D G Lee proposed openstack/ironic master: ironic-conductor can't report heartbeat to database https://review.openstack.org/475210 | 09:08 |
| openstackgerrit | Merged openstack/proliantutils master: Redfish: [Fix] Set pending boot mode json payload https://review.openstack.org/485982 | 09:12 |
| openstackgerrit | D G Lee proposed openstack/ironic master: ironic-conductor can't report heartbeat to database https://review.openstack.org/475210 | 09:13 |
| openstackgerrit | bhavani proposed openstack/bifrost master: Handle log message interpolation by the logger in common/ https://review.openstack.org/485123 | 09:18 |
| *** amarao has joined #openstack-ironic | 09:19 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Start passing portgroup information to Neutron https://review.openstack.org/446763 | 09:20 |
| dtantsur | vsaienk0: hi! please remind me: what's the use case of security groups with e.g. provisioning networks? | 09:23 |
| vsaienk0 | morning dtantsur | 09:25 |
| vsaienk0 | dtantsur: if Neutron ML2 supports security groups you need to open ports that are needed for provisioning nodes as by default all is blocked | 09:26 |
| dtantsur | aha | 09:26 |
| dtantsur | so we essentially have to use them, if they're even supported | 09:26 |
| * dtantsur wonders if at least one ML2 driver supports them in reality | 09:26 | |
| vsaienk0 | dtantsur: arista and cisco drivers should but I don't know for sure. | 09:28 |
| dtantsur | vsaienk0: I was told cisco does not. arista - dunno | 09:29 |
| dtantsur | anyway, thanks! this is what I needed to know :) | 09:29 |
| openstackgerrit | D G Lee proposed openstack/ironic master: ironic-conductor can't report heartbeat to database https://review.openstack.org/475210 | 09:29 |
| vsaienk0 | you are welcome | 09:29 |
| *** sambetts|afk is now known as sambetts | 09:33 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Documetation for 'ilo' hardware type https://review.openstack.org/476895 | 09:33 |
| sambetts | dtantsur: there are interesting probelms with supporting security groups for baremetal and it mostly comes down to the fact that neutron security groups were built around iptables which was designed to do a different job than what ACLs in TORs were built for | 09:37 |
| dtantsur | right. I've heard this from other vendors as well | 09:37 |
| dtantsur | I wonder if we should sit with neutron folks and figure out some safe subset of the security groups.. | 09:37 |
| sambetts | dtantsur: I think the main issue is that security groups support both egress and ingress rules, and most TORs only support a single direction | 09:38 |
| sambetts | for ACLs, although I never rememeber which direction | 09:38 |
| * dtantsur suspects ingress | 09:38 | |
| sambetts | yeah, protect the server from the network, not protect the network from the server | 09:38 |
| *** marios has quit IRC | 09:43 | |
| *** marios has joined #openstack-ironic | 09:43 | |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: Allow setting {provisioning,cleaning}_network in node driver_info https://review.openstack.org/460564 | 09:44 |
| *** jjung has quit IRC | 09:44 | |
| *** yufei has quit IRC | 09:48 | |
| *** amarao1 has joined #openstack-ironic | 09:54 | |
| *** stendulker has joined #openstack-ironic | 09:56 | |
| aparnav | Hi dtantsur, could you review this patch https://review.openstack.org/422572 when you find time? It is in there for quite some time. | 09:57 |
| patchbot | patch 422572 - ironic - Support hpsum based `update_firmware` for iLO drivers | 09:57 |
| dtantsur | hi, eventually. we're in the middle of release deadlines now, so no promises | 10:00 |
| dtantsur | thigns are going to improve a bit after the soft freeze, I guess | 10:00 |
| *** stendulker_ has joined #openstack-ironic | 10:00 | |
| *** jjung has joined #openstack-ironic | 10:02 | |
| *** stendulker has quit IRC | 10:04 | |
| *** slagle has quit IRC | 10:04 | |
| *** stendulker_ has quit IRC | 10:05 | |
| openstackgerrit | Merged openstack/bifrost master: Allows to setup more than one nameserver https://review.openstack.org/483998 | 10:06 |
| aparnav | dtantsur, okay thank you. I really want this patch to be in for this release. | 10:07 |
| *** vmud213 has joined #openstack-ironic | 10:13 | |
| *** jaypipes has joined #openstack-ironic | 10:13 | |
| *** MattMan_1 has quit IRC | 10:16 | |
| *** MattMan_1 has joined #openstack-ironic | 10:17 | |
| *** jkilpatr has joined #openstack-ironic | 10:27 | |
| openstackgerrit | Merged openstack/ironic master: Replace the usage of 'manager' with 'os_primary' https://review.openstack.org/475996 | 10:30 |
| *** jkilpatr has quit IRC | 10:33 | |
| openstackgerrit | Merged openstack/ironic master: Improve ports API reference https://review.openstack.org/483015 | 10:34 |
| *** wong_ has quit IRC | 10:38 | |
| *** pc_m has quit IRC | 10:41 | |
| vdrok | late morning all | 10:43 |
| *** amarao1 has quit IRC | 10:44 | |
| *** amarao1 has joined #openstack-ironic | 10:45 | |
| dtantsur | hi vdrok | 10:49 |
| *** pc_m has joined #openstack-ironic | 10:50 | |
| *** amarao1 has quit IRC | 10:51 | |
| vdrok | o/ | 10:51 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Add hardware types to support Cisco UCS Servers https://review.openstack.org/486100 | 10:53 |
| sambetts | dtantsur: ^ | 10:53 |
| dtantsur | nice! | 10:53 |
| * dtantsur stars | 10:53 | |
| * dtantsur instant -1 :D | 10:55 | |
| sambetts | \o/ | 10:55 |
| sambetts | damn knew I forgot something XD | 10:55 |
| *** rook has quit IRC | 10:57 | |
| *** dtantsur is now known as dtantsur|bbl | 10:57 | |
| *** sai_ has quit IRC | 10:57 | |
| * dtantsur|bbl -> short walk | 10:57 | |
| *** sai has joined #openstack-ironic | 10:58 | |
| *** rook has joined #openstack-ironic | 10:58 | |
| *** rook is now known as Guest11562 | 10:59 | |
| *** hw_wutianwei has quit IRC | 11:01 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Doc for disk erase support in iLO drivers https://review.openstack.org/408441 | 11:03 |
| *** jkilpatr has joined #openstack-ironic | 11:09 | |
| *** appu103 has joined #openstack-ironic | 11:11 | |
| *** aparnav has quit IRC | 11:13 | |
| *** gcb has quit IRC | 11:14 | |
| *** appu103 has quit IRC | 11:16 | |
| *** appu103 has joined #openstack-ironic | 11:16 | |
| xavierr | good morning ironic | 11:17 |
| *** ccamacho is now known as ccamacho|lunch | 11:17 | |
| xavierr | do I need to add release notes to this patch https://review.openstack.org/482209 ? I don't think it is user visible... | 11:18 |
| patchbot | patch 482209 - ironic - Removes agent mixin from oneview drivers | 11:18 |
| xavierr | also, if you have time to review it ... :) | 11:19 |
| *** appu103 has quit IRC | 11:21 | |
| *** fellypefca has joined #openstack-ironic | 11:22 | |
| fellypefca | Morning, Ironic :) TGIF o/ | 11:25 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/485177 | 11:26 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Add hardware types to support Cisco UCS Servers https://review.openstack.org/486100 | 11:46 |
| *** lucasagomes is now known as lucas-hungry | 11:47 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Adds local_gb discovery https://review.openstack.org/483179 | 11:47 |
| xavierr | sambetts: we are also proposing oneview hardware types, may you review it? https://review.openstack.org/#/c/485798/ | 11:53 |
| patchbot | patch 485798 - ironic - [WIP|DNM] Add hardware type for HPE OneView | 11:53 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Add hardware types to support Cisco UCS Servers https://review.openstack.org/486100 | 11:55 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Adds local_gb discovery https://review.openstack.org/483179 | 12:01 |
| *** fragatina has joined #openstack-ironic | 12:14 | |
| *** slagle has joined #openstack-ironic | 12:15 | |
| *** hw_wutianwei has joined #openstack-ironic | 12:18 | |
| *** baoli has joined #openstack-ironic | 12:20 | |
| *** jaganathan has quit IRC | 12:23 | |
| *** dprince has joined #openstack-ironic | 12:24 | |
| *** nicodemos has joined #openstack-ironic | 12:28 | |
| nicodemos | morning, ironic. | 12:28 |
| *** xavierr has quit IRC | 12:32 | |
| *** rbudden has joined #openstack-ironic | 12:32 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Introducing the user content https://review.openstack.org/480250 | 12:39 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Docs migration conclusion https://review.openstack.org/480251 | 12:39 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Introducing the installation content https://review.openstack.org/480249 | 12:39 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Introducing the contributor content https://review.openstack.org/480222 | 12:39 |
| milan | morning fellypefca, nicodemos, sambetts and all! :) | 12:39 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Refactor VIFPortIDMixin: rename https://review.openstack.org/486124 | 12:39 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Refactor VIFPortIDMixin: factor out common methods https://review.openstack.org/486125 | 12:39 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Refactor VIFIDPortMixin: vif_attach and vif_detach https://review.openstack.org/486126 | 12:39 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Refactor VIFPortIDMixin: port[_group]_changed https://review.openstack.org/486127 | 12:39 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add fake network interface https://review.openstack.org/486128 | 12:40 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Use fake network interface in tempest VIF attach https://review.openstack.org/486129 | 12:40 |
| *** trown|outtypewww is now known as trown | 12:40 | |
| milan | fellypefca, TGIF indeed! :D | 12:40 |
| fellypefca | milan o/ | 12:40 |
| nicodemos | hey, milan, fellypefca. TGIF for sure \o/ | 12:40 |
| milan | and morning vdrok! :) | 12:40 |
| milan | NobodyCam TGIF? :D | 12:41 |
| vdrok | milan: mornin :) | 12:41 |
| * milan getting crossed-eyes from the docs | 12:41 | |
| milan | sambetts gonna hate me for that I guess O:-) | 12:41 |
| sambetts | milan: ;) | 12:42 |
| *** ricardoas has joined #openstack-ironic | 12:43 | |
| milan | sambetts, https://review.openstack.org/#/c/480250/5//COMMIT_MSG | 12:43 |
| patchbot | patch 480250 - ironic-inspector - Introducing the user content | 12:43 |
| * milan hides | 12:43 | |
| *** lucas-hungry is now known as lucasagomes | 12:45 | |
| jlvillal | TheJulia, Congrats! I see the tempest tests passed for boot-from-volume. Nice job! :) | 12:48 |
| * TheJulia dances | 12:49 | |
| *** jrist has quit IRC | 12:49 | |
| *** edleafe is now known as figleaf | 12:50 | |
| *** vmud213 has quit IRC | 12:50 | |
| *** ccamacho|lunch is now known as ccamacho | 12:51 | |
| TheJulia | dtantsur|bbl: I saw your comments, and that was the direction I was originally heading and then came to the realization that ask of should_write_image is "should I write an image for this process", and I think that logic is fairly unique. Perhaps we could put a method in deploy_utils that other modules could re-use at some point, and change all the references to use the helper method. I also thought of the | 12:54 |
| TheJulia | case of oneview where they would be likely to do something like a oneview-cinder merged driver since oneview might have to change some information potentially in the back of the blade chassises for volume mappings to be valid, and the logic of should_write_image could then be overridden | 12:54 |
| TheJulia | jlvillal: and that run only took 25 minutes..... | 12:55 |
| jlvillal | TheJulia, If you have time maybe you can review: https://review.openstack.org/#/c/482448/ | 12:56 |
| patchbot | patch 482448 - ironic - Allow updating interfaces on a node in available s... | 12:56 |
| jlvillal | TheJulia, It is first in the list of patches | 12:56 |
| *** makowals has joined #openstack-ironic | 12:56 | |
| makowals | Hello everyone, I have a problem with debugging python agent so my question in here | 12:57 |
| makowals | Having box booted with deploy image, is there any simple way to run IPA and tell it to execute one selected clean step ? | 12:58 |
| makowals | If I do this from conductor side, I need to wait for reboot and all related stuff I'd prefer to avoid | 12:58 |
| *** xavierr has joined #openstack-ironic | 12:58 | |
| makowals | Also if I modify IPA code manually, if I reboot the node again I will lose my changes (which are just for debugging the problem= | 12:58 |
| makowals | ) | 12:58 |
| makowals | Any ideas how to proceed ? | 12:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Start passing portgroup information to Neutron https://review.openstack.org/446763 | 12:59 |
| sambetts | makowals: if you put the node into maintenance in ironic then ironic will ignore the node, you could then interact with the IPA rest API directly | 13:00 |
| *** sasha21 has joined #openstack-ironic | 13:00 | |
| *** hw_wutianwei has quit IRC | 13:00 | |
| *** hw_wutianwei1 has joined #openstack-ironic | 13:01 | |
| *** hw_wutianwei1 is now known as hw_wutianwei | 13:03 | |
| *** jrist has joined #openstack-ironic | 13:03 | |
| TheJulia | jlvillal: Since the job passed, you may want to re-evaluate your -1 on https://review.openstack.org/#/c/472740 :) | 13:04 |
| patchbot | patch 472740 - ironic - Tempest scenario test for boot-from-volume | 13:04 |
| *** jcoufal has joined #openstack-ironic | 13:04 | |
| *** cdearborn has joined #openstack-ironic | 13:04 | |
| jlvillal | TheJulia, Sure I will look | 13:05 |
| makowals | sambetts: Is there any docs about it ? In https://docs.openstack.org/ironic-python-agent/latest/ I cannot find any reference to api calls | 13:05 |
| TheJulia | jlvillal: I also rechecked the devstack patch since it blew up on the grenade run | 13:06 |
| *** adisky__ has quit IRC | 13:08 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/485405 | 13:08 |
| jroll | makowals: tl;dr something like: curl -X POST http://agent-ip:9999/v1/commands --data '{"name": "command name", "params": {"any": "params", "you": "need"}}' | 13:08 |
| sambetts | o/ jroll | 13:08 |
| ricardoas | morning, ironic! | 13:08 |
| jroll | I had a script for this but not sure if I still have access | 13:08 |
| jroll | hi sambetts :) | 13:08 |
| ricardoas | is there a documentation freeze date? | 13:09 |
| jroll | makowals: this file may be helpful: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/tests/functional/test_commands.py | 13:09 |
| TheJulia | ricardoas: not really, just before we need to cut the release. | 13:10 |
| ricardoas | TheJulia: right... just checking because we´ve been updating some of oneview docs :] | 13:13 |
| *** sauloaislan has joined #openstack-ironic | 13:13 | |
| TheJulia | ricardoas: I figured as much :) | 13:14 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: PGSQL: create Enum before using https://review.openstack.org/472725 | 13:14 |
| makowals | sambetts, jroll: When I start agent on a node which is "manageable" and "in maintenance" agent does not seem to listen on any port, it just returns "ironic_api_client [-] Failure status code: 404" | 13:16 |
| *** crushil has joined #openstack-ironic | 13:16 | |
| jroll | oh, it doesn't start up the agent API because ironic's lookup API is inaccessible. hrm | 13:17 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client master: Updated from global requirements https://review.openstack.org/485958 | 13:18 |
| jroll | makowals: I think you can work around that by running the agent with --standalone | 13:18 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Assume neutron port exists in VIF attach https://review.openstack.org/476949 | 13:19 |
| makowals | Ahh, that seems to work, thanks a lot ! | 13:19 |
| jroll | you're welcome :) | 13:19 |
| openstackgerrit | Rodrigo Farias de Macedo proposed openstack/ironic master: Migrate oneview driver to use python-hpOneView https://review.openstack.org/466872 | 13:19 |
| openstackgerrit | Merged openstack/virtualbmc master: Removes unnecessary utf-8 encoding https://review.openstack.org/485535 | 13:20 |
| openstackgerrit | Merged openstack/virtualbmc master: Drop MANIFEST.in - it's not needed by pbr https://review.openstack.org/485534 | 13:20 |
| TheJulia | jlvillal: if you wouldn't mind, adding your $0.02 to https://review.openstack.org/#/c/485812/ or the discussion I've tried to start with dmitry while he is away. I'd like to try and get the fix landed today if at all humanly possible. | 13:21 |
| patchbot | patch 485812 - ironic - Correct BFV PXE logic and debugging | 13:21 |
| *** amarao1 has joined #openstack-ironic | 13:23 | |
| *** amarao1 has quit IRC | 13:24 | |
| TheJulia | jlvillal: http://logs.openstack.org/40/472740/18/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/48d2493/console.html#_2017-07-20_23_44_25_770288 looks new Oo | 13:25 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Rolling upgrades support for create_port RPCAPI https://review.openstack.org/485773 | 13:28 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add tempest tests for physical networks https://review.openstack.org/470915 | 13:32 |
| *** dtantsur|bbl is now known as dtantsur | 13:32 | |
| dtantsur | wow, a real jroll :) morning | 13:32 |
| dtantsur | TheJulia: mmm, yeah, maybe a helper.. or a method wrapping a method | 13:33 |
| *** SerenaFeng has joined #openstack-ironic | 13:33 | |
| *** rbudden has quit IRC | 13:33 | |
| fellypefca | dtantsur Did you have some time to discuss driver composition? :) | 13:33 |
| TheJulia | I'm okay with changing it to a helper, I'm just afraid people will use it directly and copy the code, and then we may end up in another troubleshooting headache | 13:34 |
| jroll | \o dtantsur, happy friday to y'all | 13:34 |
| dtantsur | fellypefca: I hope so | 13:34 |
| TheJulia | jroll: o/ | 13:35 |
| fellypefca | dtantsur We from OneView are implementing it, and the documentation recommends to do not implement deploy.py, but we need to do some configurations in this phase. | 13:35 |
| dtantsur | fellypefca: is flashing the image really different for oneview? | 13:35 |
| dtantsur | this is subtle way to hint you that the actual difference tends to be in boot interfaces | 13:36 |
| dtantsur | (we had a similar exercise with ilo a few months ago) | 13:36 |
| jroll | hey TheJulia :) | 13:36 |
| * jroll should have known popping his head in here would result in an assault of good mornings :D | 13:37 | |
| sambetts | jroll: hows life treating you? | 13:37 |
| *** SerenaFeng has quit IRC | 13:39 | |
| fellypefca | dtantsur In the case of applying a server profile and we will need to update the ports on Ironic when using a virtual mac. Do we migrate all this to the boot interface? (this last change needs to be before add_cleaning/provisioning_network on Neutron) | 13:39 |
| jlvillal | vsaienk0, What happened to the release notes on https://review.openstack.org/#/c/446763/ ? | 13:40 |
| patchbot | patch 446763 - ironic - Start passing portgroup information to Neutron | 13:40 |
| dtantsur | fellypefca: I guess it boils down to what "applying a server profile" means for you. if it's closer to booting routine, it should go to the boot interface. if it's closer to IPA/image writing business, it can probably stay in deploy (but I doubt it) | 13:40 |
| *** SerenaFeng has joined #openstack-ironic | 13:40 | |
| *** jcoufal_ has joined #openstack-ironic | 13:41 | |
| dtantsur | TheJulia: btw should your new patch be merged into https://review.openstack.org/#/c/479326/ ? | 13:42 |
| patchbot | patch 479326 - ironic - BFV Deploy skip minor logging, logic, and test fixes | 13:42 |
| TheJulia | I'd prefer to keep them separate at this point | 13:42 |
| dtantsur | TheJulia: I guess I'm fine with the method staying as it is, but I suspect we should rename should_write_image to something more generic.. | 13:43 |
| * dtantsur tries to turn on his brain | 13:43 | |
| TheJulia | since then it will just become a bunch of general fixes, and we know the order in which the revisions work :) | 13:43 |
| *** links has quit IRC | 13:43 | |
| *** jcoufal has quit IRC | 13:43 | |
| * TheJulia hands dtantsur coffee and goes and makes more | 13:43 | |
| dtantsur | \o/ | 13:43 |
| *** zhangfei has quit IRC | 13:44 | |
| TheJulia | dtantsur: in that case, I'm much more for a deploy_utils method and replacing the existing calls to call deploy_utils where it has the logic, and the big mean nasty notes saying "here is what this can cause" | 13:44 |
| TheJulia | name suggestions, welcome ;) | 13:44 |
| openstackgerrit | Merged openstack/ironic-ui master: Add the capability to associate ports with portgroups https://review.openstack.org/480583 | 13:44 |
| dtantsur | TheJulia: so, what about maybe: storage.should_prepare_ramdisk and storage.should_write_image (the former calling the latter)? | 13:44 |
| fellypefca | dtantsur Looking for cleaning and deploy process the first thing that Ironic do is add this ports on Neutron. Since we need to update the Ironic Ports before that, we do not have a good place for this. tip for this? | 13:44 |
| dtantsur | fellypefca: could you please point me to related code? | 13:45 |
| mjturek | TheJulia: so basically, the call to should_write_image in pxe was problematic for cleaning? | 13:45 |
| mjturek | as we actually do want to send the ramdisk and kernel for cleaning? | 13:45 |
| TheJulia | dtantsur: prepare_ramdisk is really boot specific. | 13:45 |
| dtantsur | I'm just trying to invent a good name for the wrapper call (and I agree it can live in the driver interface still) | 13:46 |
| TheJulia | mjturek: yeah, it is and we do. tl;dr the task to destroy makes a copy of the volume data when it stats, that same data gets sent into the cleaning logic, even though the database records have been destroyed | 13:46 |
| fellypefca | dtantsur It's still a work in progress since we do not figure out this yet. But basically changes the Ironic's port mac to the virtual one on OneView | 13:46 |
| TheJulia | s/stats/starts/ | 13:47 |
| mjturek | got it, awesome | 13:47 |
| TheJulia | mjturek: yeah, totally awesome. Like, I'll be drunk all weekend awesome | 13:47 |
| mjturek | hahahaha | 13:47 |
| dtantsur | fellypefca: it feels to me like a new network interface, to be honest :) may be part of boot still | 13:47 |
| TheJulia | Once I got some debug logging in place it hit my like a brick wall what the problem was | 13:47 |
| * mjturek feels guilty for adding the should_write_image check :-| | 13:48 | |
| *** nmathew has joined #openstack-ironic | 13:48 | |
| TheJulia | dtantsur: I'd kind of prefer to keep it out of the driver, deploy_utils is growing on me | 13:49 |
| TheJulia | mjturek: as punishment, perhaps a name suggestion for a deploy_utils helper | 13:49 |
| TheJulia | > | 13:49 |
| TheJulia | ? :) | 13:49 |
| *** baoli has quit IRC | 13:49 | |
| dtantsur | TheJulia: also, IIRC lines 460 - 475 are moving away in https://review.openstack.org/#/c/485812/6/ironic/drivers/modules/pxe.py no? | 13:49 |
| patchbot | patch 485812 - ironic - Correct BFV PXE logic and debugging | 13:49 |
| dtantsur | I wonder how it affects this discussion | 13:49 |
| openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/485177 | 13:50 |
| dtantsur | I'm fine with the deploy_utils too, and I'd still use something like should_prepare_ramdisk.. | 13:50 |
| *** vishwanathj has quit IRC | 13:50 | |
| *** vishwanathj has joined #openstack-ironic | 13:50 | |
| * TheJulia tries to think, would the logic differ | 13:51 | |
| * sambetts really hates deploy_utils it makes stuff hard to override if required | 13:51 | |
| TheJulia | the whole thing that tosses a wrench into this is some of this stuff gets called as a result of takeover | 13:51 |
| fellypefca | dtantsur Maybe a Network Interface is a good thing to create on OneView. Thanks for the help :) | 13:52 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Start passing portgroup information to Neutron https://review.openstack.org/446763 | 13:52 |
| *** mgoddard_ has quit IRC | 13:53 | |
| TheJulia | dtantsur: I really suspect the method, if done as a separate distinctly named on the storage interface, would basically house identical logic, so it kind of seems silly | 13:54 |
| mjturek | +1 on should_prepare_ramdisk, but I'll throw should_configure_pxe into the mix as it's getting called within prepare_ramdisk | 13:54 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: iRMC: Support volume boot for iRMC virtual media boot interface https://review.openstack.org/467911 | 13:54 |
| *** rbudden has joined #openstack-ironic | 13:55 | |
| *** zhangfei has joined #openstack-ironic | 13:56 | |
| TheJulia | and just for now have it wrap should_write_image if we determine specific logic is needed? | 13:57 |
| sambetts | TheJulia, mjturek, dtantsur: this seems to be heading down the route of having conditionals around everything... would this be solved by just not calling into deploy for the case when we don't need to because the volume is already populated? | 13:57 |
| *** dansmith is now known as superdan | 13:57 | |
| dtantsur | we could avoid calling prepare_ramdisk at all, if we moved boot.ipxe creation somewhere earlier | 13:58 |
| *** racedo has quit IRC | 13:59 | |
| dtantsur | anyone wants to check the devstack bits https://review.openstack.org/#/c/466333/ ? | 13:59 |
| patchbot | patch 466333 - ironic - DevStack: Add configuration for boot-from-volume | 13:59 |
| mjturek | dtantsur: https://review.openstack.org/#/c/484032/ | 13:59 |
| patchbot | patch 484032 - ironic - [WIP] Generate iPXE boot script on start up | 13:59 |
| *** amarao1 has joined #openstack-ironic | 14:00 | |
| mjturek | I can update this today | 14:00 |
| mjturek | oh, you've seen it already | 14:00 |
| dtantsur | yep, that's why I'm talking about it :) | 14:02 |
| mjturek | :) | 14:02 |
| mjturek | working on it now | 14:02 |
| *** SerenaFeng has quit IRC | 14:02 | |
| *** Goneri has joined #openstack-ironic | 14:02 | |
| *** baoli has joined #openstack-ironic | 14:02 | |
| TheJulia | sambetts: part of this problem is the reuse of code for things like cleaning, which uses the deploy interface, which calls the boot interface. :( | 14:03 |
| *** SerenaFeng has joined #openstack-ironic | 14:03 | |
| TheJulia | but would that really allow us to remove the prepare_ramdisk check.... | 14:03 |
| * TheJulia ponders | 14:04 | |
| TheJulia | Yeah, It should allow us to | 14:04 |
| crushil | dtantsur, Morning/Afternoon. I have thought about the XClarity spec for the past couple of days. The deploy interface we're intending to implement is not Agent deploy/PXE deploy. We have a scenario in which LXCA manages the servers in the case we are using images not provided by Glance. Would this be reusable from the current deploy interfaces that Ironic provides? | 14:04 |
| sambetts | TheJulia: right, but if we for the case of boot from prepopulated volume, didn't called deploy, just directly called storage attach and boot.prepare_intstnace and power on from the conductor, then prepare_ramdisk wouldn't need to care about volumes at all | 14:06 |
| *** _milan_ has joined #openstack-ironic | 14:06 | |
| sambetts | crushil: so LXCA writes the user image to the server? | 14:07 |
| TheJulia | sambetts: then we would cascade all of the logic into the conductor.... | 14:07 |
| *** milan has quit IRC | 14:07 | |
| dtantsur | crushil: morning! but how does the image gets written? | 14:07 |
| TheJulia | crushil: And how do the nodes get cleaned? | 14:07 |
| TheJulia | crushil: and is the image source, something that ironic might want to use elsewhere? | 14:07 |
| crushil | LXCA manages the whole deploy process using an out of band IMM process | 14:08 |
| sambetts | what does the user pass into "openstack server create --image" too | 14:08 |
| sambetts | if the images aren't in glance | 14:08 |
| TheJulia | sambetts: good point | 14:08 |
| TheJulia | It could be stand-alone users... | 14:09 |
| sambetts | yeah | 14:09 |
| TheJulia | Whole baremetal, and just baremetal clouds do exist | 14:09 |
| *** jaypipes is now known as leakypipes | 14:09 | |
| TheJulia | mjturek: yeah, removing the prepare_ramdisk short circuit is really really growing on me. | 14:10 |
| TheJulia | and that is the root of the problem. | 14:10 |
| crushil | sambetts, It would be stand alone users. I have only used "openstack server create --image" too. If we want to have a corner case of providing HTTP links which can then be leveraged by lxca to manage the whole deploy process with IMM, what would that look like? | 14:10 |
| mjturek | yeaaah - I think it's the right way to go | 14:10 |
| TheJulia | mjturek: if we do that, I could just nuke all of the logic from my patch and we could just keep the logging :) | 14:11 |
| sambetts | TheJulia: regarding putting the logic into the conductor, I think this is the first time we have a case where we have two different ways for deploying one server based on user choice, e.g. BFV, or write an image to disk, the conductor is the place to make the distiction "do I need to write an image?" yes call the deploy, no do something else | 14:11 |
| mjturek | +1, logging for BFV has been a bit conservative | 14:11 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: iRMC: Support volume boot for iRMC virtual media boot interface https://review.openstack.org/467911 | 14:12 |
| *** baoli has quit IRC | 14:12 | |
| *** jcoufal has joined #openstack-ironic | 14:12 | |
| *** baoli has joined #openstack-ironic | 14:13 | |
| *** amarao2 has joined #openstack-ironic | 14:13 | |
| *** jcoufal_ has quit IRC | 14:13 | |
| *** amarao1 has quit IRC | 14:13 | |
| TheJulia | sambetts: yeah, and that whole discussion lead to the storage interface should be the decision maker since it can house all of the logic specific to the interface, since ultimately it knows and can help facilitate our regular deployment, booting from a remote volume, or attaching volumes and letting the normal process proceed. | 14:14 |
| sambetts | part of me wonders if we should have had a deploy function in the storage interface, so the conductor would do: if booting from volume: storage_interface.deploy() else image_deploy_interface.deploy() where image_deployer is our current deploy interface | 14:14 |
| crushil | sambetts, dtantsur LXCA writes the user image to the server directly once IMM has been configured. The node cleanup process and the whole deployment process is taken over by lxca. As TheJulia said, it would be the corner case for stand-alone users. | 14:14 |
| dtantsur | this sounds like a new deploy interface indeed | 14:14 |
| TheJulia | sambetts: there is a combined case, where both may be leveraged in the end, since the main check is... is there an image | 14:14 |
| TheJulia | problem with new deploy interfaces is, we've burned so much in for cleaning | 14:15 |
| TheJulia | we would almost have to be prepared to rip our cleaning to it's own interface | 14:15 |
| TheJulia | oh wait | 14:16 |
| TheJulia | we are mixing discussions | 14:16 |
| *** baoli has quit IRC | 14:16 | |
| TheJulia | doh! | 14:16 |
| dtantsur | yeah, sorry, I was responding to crushil re the new interface | 14:16 |
| TheJulia | It was aptly timed though ;) | 14:16 |
| crushil | :) | 14:17 |
| dtantsur | I guess we should give it a try, lay out the patches and see how they look | 14:17 |
| *** baoli has joined #openstack-ironic | 14:17 | |
| dtantsur | this is re BFV discussion ^^^ | 14:17 |
| TheJulia | there are repercussions we can't even begin to think of though. We would almost need to whiteboard it out, and walk through each step because we're basically talking major refactoring to do it right | 14:18 |
| TheJulia | ^^ also BFV | 14:19 |
| sambetts | dtantsur, TheJulia, mjturek: sounds like we might need a higher bandwidth channel for this | 14:19 |
| TheJulia | I kind of think it is PTG worthy at this point, that is if we have time... | 14:19 |
| *** Guest11562 is now known as rook | 14:19 | |
| TheJulia | sambetts: agreed | 14:20 |
| dtantsur | well, I would not mind finishing BFV in some usable state right now | 14:20 |
| TheJulia | My note regarding time is I've heard rumor of zuul 2.5 to zuul 3 upgrade the week prior to the PTG which means we may have to be refactoring our CI jobs during the week of the PTG. | 14:21 |
| crushil | dtantsur, Ok, do you want me to write up a new spec for the deploy interface? Or do you want me to expand more in the deploy section of the current spec? | 14:21 |
| dtantsur | crushil: please expand on this spec. add technical details on how LXCA works and what you expect as the outcome | 14:21 |
| TheJulia | dtantsur: I think it is basically there, my latest patch can just be rolled back to logging once mjturek fixes the boot.ipxe creation | 14:21 |
| dtantsur | TheJulia: yep, so let's take the very minimum, update it to a working state and look at it | 14:22 |
| dtantsur | I'm getting a bit confused by a storm of suggestions :) | 14:22 |
| crushil | dtantsur, Will do, thanks for your input. :) | 14:22 |
| TheJulia | dtantsur: ditto, we're also kind of reaching for big picture, what would be perfect without accounting for all of the cases | 14:22 |
| mjturek | dtantsur, TheJulia: should I revert this https://review.openstack.org/#/c/478690/ and then rebase https://review.openstack.org/#/c/484032 on top of the revert? | 14:24 |
| patchbot | patch 478690 - ironic - Generate iPXE boot script when deploying with boot... (MERGED) | 14:24 |
| patchbot | patch 484032 - ironic - [WIP] Generate iPXE boot script on start up | 14:24 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: iRMC: Support volume boot for iRMC virtual media boot interface https://review.openstack.org/467911 | 14:24 |
| mjturek | or just integrate th revert into 484032? I don't think we need anything from 478690 | 14:24 |
| dtantsur | mjturek: given the CI stability, I'd prefer less patches | 14:25 |
| mjturek | you got it dtantsur | 14:25 |
| TheJulia | +1 to dtantsur's statement | 14:26 |
| *** jrist has quit IRC | 14:26 | |
| *** r-daneel has quit IRC | 14:27 | |
| *** mtanino_ has joined #openstack-ironic | 14:28 | |
| *** SerenaFeng has quit IRC | 14:32 | |
| *** baoli has quit IRC | 14:33 | |
| *** hshiina has quit IRC | 14:33 | |
| TheJulia | re bfv and deploy interfaces, I added a note to the PTG etherpad and added context/questions we would have to consider for discussion as well as a link to the irc log. | 14:34 |
| dtantsur | thanks! | 14:36 |
| *** baoli has joined #openstack-ironic | 14:36 | |
| *** jrist has joined #openstack-ironic | 14:39 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Updated from global requirements https://review.openstack.org/485958 | 14:43 |
| *** jjung has quit IRC | 14:43 | |
| *** amarao2 has quit IRC | 14:43 | |
| *** [1]rpioso has joined #openstack-ironic | 14:44 | |
| *** baoli has quit IRC | 14:46 | |
| *** hw_wutianwei has quit IRC | 14:47 | |
| *** hw_wutianwei has joined #openstack-ironic | 14:47 | |
| *** cdearborn_ has joined #openstack-ironic | 14:56 | |
| makowals | Could someone explain me thing with priorities of clean steps, as maybe I'm getting something totally wrong. Isn't that I can control order by setting proper priorities inside the same "interface" ? | 15:00 |
| makowals | What I observe currently is complete randomness as from controller side they're executed asynchronously | 15:01 |
| TheJulia | It should be the behavior, but they shouldn't be executed randomly. it would really help to see logs and some example of the cleaning steps your requesting. Preferably into a bug because that is likely going to take someone a little while to wrap their head around | 15:03 |
| TheJulia | wow that first sentence is broken | 15:03 |
| * _milan_ calling it a day! CU Ironic! Have a nice weekend! :) | 15:03 | |
| TheJulia | The behavior should be ordered by priority | 15:03 |
| TheJulia | have a good weekend _milan_ | 15:03 |
| _milan_ | o/ TheJulia the same to you! :) | 15:03 |
| *** _milan_ has left #openstack-ironic | 15:04 | |
| *** nmathew has quit IRC | 15:04 | |
| makowals | TheJulia: Randomly from the controller or agent side ? Even if controller does it always in order, for the agent it may take faster/longer depends on the environment and in the end I cannot determine the order beforehand | 15:05 |
| *** hoonetorg has quit IRC | 15:05 | |
| makowals | TheJulia: Anyway, in that case (=asynchronous execution), what's the best way to define cleaning in order to always control the order (and doing it sequentially) ? Should I just create "dummy" step which will be a wrapper around the real work ? | 15:06 |
| *** dprince has quit IRC | 15:07 | |
| sambetts | makowals: how are you starting the clean steps? are you passing them to IPA all at the same time or one at a time? | 15:07 |
| makowals | I just set node's state to "provide", so it's all automatic | 15:07 |
| makowals | Priorities are 20,80,81 and apparently the one with 20 is executed as the last one | 15:08 |
| *** mjura has quit IRC | 15:08 | |
| sambetts | higher number == high priority | 15:09 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Discover iscsi_boot attribute https://review.openstack.org/341653 | 15:09 |
| sambetts | so 20 being last is correct | 15:09 |
| jroll | sambetts: is good but busybusy | 15:09 |
| sambetts | jroll: glad to here it :) | 15:10 |
| jroll | and you? | 15:10 |
| sambetts | yeah, good, been pretty busy too | 15:11 |
| *** patriciadomin has quit IRC | 15:11 | |
| makowals | ... | 15:11 |
| makowals | +1 for me for understanding this then :D | 15:12 |
| makowals | Thanks | 15:12 |
| *** cdearborn has quit IRC | 15:12 | |
| *** rcernin has quit IRC | 15:14 | |
| *** jrist has quit IRC | 15:18 | |
| sambetts | makowals: no problem, https://docs.openstack.org/ironic/latest/admin/cleaning.html#how-are-cleaning-steps-ordered , <--- all this stuff is documented here, you might not have been able to find because the google links etc haven't updated correctly yet after a recent doc refactoring | 15:18 |
| *** r-daneel has joined #openstack-ironic | 15:20 | |
| *** vishwanathj has quit IRC | 15:22 | |
| *** aarefiev is now known as aarefiev_afk | 15:23 | |
| *** vishwanathj has joined #openstack-ironic | 15:23 | |
| *** racedo has joined #openstack-ironic | 15:25 | |
| *** jcoufal_ has joined #openstack-ironic | 15:27 | |
| *** jcoufal has quit IRC | 15:28 | |
| *** ElCoyote_ has quit IRC | 15:37 | |
| *** zhangfei has quit IRC | 15:39 | |
| *** baoli has joined #openstack-ironic | 15:47 | |
| *** makowals has quit IRC | 15:51 | |
| *** jrist has joined #openstack-ironic | 15:57 | |
| *** jrist has quit IRC | 15:57 | |
| *** jrist has joined #openstack-ironic | 15:57 | |
| *** lucasagomes is now known as lucas-afk | 16:09 | |
| openstackgerrit | Michael Turek proposed openstack/ironic master: Generate iPXE boot script on start up https://review.openstack.org/484032 | 16:13 |
| mjturek | might need some testing adjustments ^ my local setup decided to die on me so kind of rolling the dice here. | 16:15 |
| *** fellypefca has quit IRC | 16:20 | |
| *** amarao has quit IRC | 16:21 | |
| *** rodrigofm has quit IRC | 16:37 | |
| *** greghaynes has quit IRC | 16:40 | |
| *** marios has quit IRC | 16:42 | |
| TheJulia | gerrit must know it is friday | 16:44 |
| *** r-daneel has quit IRC | 16:47 | |
| *** dprince has joined #openstack-ironic | 16:48 | |
| *** derekh has quit IRC | 16:54 | |
| *** athomas has quit IRC | 16:55 | |
| *** trown is now known as trown|lunch | 16:55 | |
| TheJulia | mjturek: left some feedback on the change. Something doesn't look quite right with some of the tests, maybe I'm crazy though. | 16:56 |
| *** r-daneel has joined #openstack-ironic | 16:58 | |
| *** sambetts is now known as sambetts|afk | 17:02 | |
| sambetts|afk | Night all! | 17:02 |
| sambetts|afk | cya'll Monday | 17:02 |
| openstackgerrit | Rodrigo Farias de Macedo proposed openstack/ironic master: Migrate oneview driver to use python-hpOneView https://review.openstack.org/466872 | 17:03 |
| *** anupn has quit IRC | 17:03 | |
| TheJulia | have a good weekend sambetts|afk | 17:03 |
| *** aNuposic has joined #openstack-ironic | 17:04 | |
| *** ralonsoh has quit IRC | 17:04 | |
| -openstackstatus- NOTICE: Restarting Gerrit for our weekly memory leak cleanup. | 17:05 | |
| *** racedo has quit IRC | 17:06 | |
| *** aNuposic is now known as anupn | 17:07 | |
| dtantsur | have a nice weekend | 17:11 |
| *** dtantsur is now known as dtantsur|afk | 17:11 | |
| *** racedo has joined #openstack-ironic | 17:11 | |
| *** skazi has quit IRC | 17:14 | |
| *** greghaynes has joined #openstack-ironic | 17:15 | |
| openstackgerrit | Merged openstack/ironic master: Updated from global requirements https://review.openstack.org/485405 | 17:17 |
| TheJulia | you too dtantsur|afk | 17:18 |
| *** racedo has quit IRC | 17:22 | |
| *** dyasny has quit IRC | 17:26 | |
| *** dyasny has joined #openstack-ironic | 17:27 | |
| *** jkilpatr has quit IRC | 17:36 | |
| jlvillal | TheJulia, I saw a comment about doing a revert in https://review.openstack.org/#/c/485812/ | 17:37 |
| patchbot | patch 485812 - ironic - Correct BFV PXE logic and debugging | 17:37 |
| jlvillal | TheJulia, Are we planning on reverting an already merged patch? My gut thinks a patch fixing the issue would be easier. | 17:38 |
| jlvillal | TheJulia, But I had to go to the doctor this morning so I am out of the loop on things. | 17:38 |
| *** jkilpatr has joined #openstack-ironic | 17:40 | |
| *** fragatina has quit IRC | 17:41 | |
| *** Sukhdev has joined #openstack-ironic | 17:48 | |
| *** ElCoyote_ has joined #openstack-ironic | 17:50 | |
| *** trown|lunch is now known as trown | 17:52 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Add hardware type for HPE OneView https://review.openstack.org/485798 | 17:54 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Add hardware type for HPE OneView https://review.openstack.org/485798 | 18:01 |
| *** karimb has quit IRC | 18:12 | |
| *** baoli has quit IRC | 18:13 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Remove check if enabled for OneViewInspect https://review.openstack.org/486192 | 18:20 |
| *** baoli has joined #openstack-ironic | 18:23 | |
| *** jrist has quit IRC | 18:29 | |
| *** rbudden has quit IRC | 18:33 | |
| *** baoli has quit IRC | 18:37 | |
| *** baoli has joined #openstack-ironic | 18:38 | |
| -openstackstatus- NOTICE: docs.o.o is currently broken, we're investigating | 18:39 | |
| *** baoli has quit IRC | 18:40 | |
| *** fragatina has joined #openstack-ironic | 18:40 | |
| vdrok | good night everyone! | 18:41 |
| nicodemos | night vdrok | 18:43 |
| *** jkilpatr has quit IRC | 18:47 | |
| *** baoli has joined #openstack-ironic | 18:48 | |
| *** baoli has quit IRC | 18:49 | |
| *** jkilpatr has joined #openstack-ironic | 18:50 | |
| *** rpioso has joined #openstack-ironic | 18:51 | |
| *** [1]rpioso has quit IRC | 18:51 | |
| TheJulia | jlvillal: mjturek is kind of reverting in a patch to also fix it with the boot.ipxe on startup, so we keep the number of patches running through the gate to a minimum at this point | 18:56 |
| *** baoli has joined #openstack-ironic | 18:56 | |
| *** baoli has quit IRC | 18:57 | |
| jlvillal | TheJulia, So is it a clean revert of a patch? Or a new patch which reverts some things and does other stuff? I kind of like the idea of the second due to our gate issues. | 18:57 |
| jlvillal | TheJulia, But I will trust your judgement :) | 18:58 |
| *** baoli has joined #openstack-ironic | 18:59 | |
| *** jcoufal_ has quit IRC | 19:07 | |
| *** ChubYann has joined #openstack-ironic | 19:17 | |
| *** rbudden has joined #openstack-ironic | 19:19 | |
| *** crushil has quit IRC | 19:19 | |
| TheJulia | jlvillal: everyone seems to be onboard with the second idea | 19:23 |
| TheJulia | jlvillal: take a look at https://review.openstack.org/#/c/484032/ | 19:23 |
| patchbot | patch 484032 - ironic - Generate iPXE boot script on start up | 19:23 |
| xavierr | hey mjturek, I'm having some problems deploying using a storage interface. I suspect this is the cause https://github.com/openstack/ironic/commit/2c18f22f9976870b71bfd1e471afa18fd50a7a25#diff-fac2a03a761f9a13461eaa37b85fdf1bR467 may you take a look? | 19:24 |
| jlvillal | TheJulia, Sounds good. Thanks. | 19:25 |
| xavierr | mjturek: shouln't the prepare_ramdisk be only executed in case of should_write_image == True? | 19:25 |
| xavierr | :) | 19:26 |
| *** xavierr is now known as xavierr|brb | 19:28 | |
| *** sasha21 has quit IRC | 19:32 | |
| *** sasha2 has joined #openstack-ironic | 19:37 | |
| *** patriciadomin has joined #openstack-ironic | 19:39 | |
| *** bcornec1 has joined #openstack-ironic | 19:39 | |
| *** vishwana_ has joined #openstack-ironic | 19:41 | |
| *** vishwanathj has quit IRC | 19:41 | |
| *** vishwana_ has quit IRC | 19:50 | |
| *** vishwanathj has joined #openstack-ironic | 19:50 | |
| *** [1]rpioso has joined #openstack-ironic | 19:53 | |
| *** vishwanathj has quit IRC | 19:54 | |
| *** vishwanathj has joined #openstack-ironic | 19:54 | |
| *** rpioso has quit IRC | 19:55 | |
| *** jkilpatr has quit IRC | 20:03 | |
| *** [1]rpioso is now known as rpioso | 20:05 | |
| *** sasha2 has quit IRC | 20:07 | |
| *** vishwana_ has joined #openstack-ironic | 20:07 | |
| *** vishwanathj has quit IRC | 20:07 | |
| *** sasha2 has joined #openstack-ironic | 20:12 | |
| *** bcornec1 has quit IRC | 20:17 | |
| *** boris-42____ has quit IRC | 20:28 | |
| *** hw_wutianwei has quit IRC | 20:32 | |
| *** hw_wutianwei1 has joined #openstack-ironic | 20:32 | |
| *** jrist has joined #openstack-ironic | 20:33 | |
| *** hw_wutianwei1 is now known as hw_wutianwei | 20:34 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic master: [devstack] add support for running behind tls-proxy https://review.openstack.org/469836 | 20:37 |
| *** rama_y has joined #openstack-ironic | 20:41 | |
| *** exodusftw has quit IRC | 20:41 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic master: [TEST] [DO NOT MERGE] Test tls-proxy with IPA https://review.openstack.org/469837 | 20:42 |
| *** vishwana_ has quit IRC | 20:46 | |
| *** exodusftw has joined #openstack-ironic | 20:47 | |
| *** baoli has quit IRC | 20:47 | |
| *** vishwanathj has joined #openstack-ironic | 20:47 | |
| *** bnemec has quit IRC | 20:50 | |
| *** jkilpatr has joined #openstack-ironic | 20:52 | |
| *** vishwana_ has joined #openstack-ironic | 20:59 | |
| *** vishwanathj has quit IRC | 20:59 | |
| *** baoli has joined #openstack-ironic | 21:00 | |
| *** trown is now known as trown|lunch | 21:01 | |
| *** trown|lunch is now known as trown|outtypewww | 21:01 | |
| *** marlinc has quit IRC | 21:06 | |
| *** bnemec has joined #openstack-ironic | 21:11 | |
| *** lhinds has quit IRC | 21:20 | |
| *** rnoriega has quit IRC | 21:20 | |
| *** rbudden has quit IRC | 21:21 | |
| *** dprince has quit IRC | 21:34 | |
| *** etingof has quit IRC | 21:45 | |
| *** baoli has quit IRC | 21:47 | |
| *** mtanino_ has quit IRC | 21:55 | |
| *** Goneri has quit IRC | 22:05 | |
| *** rbudden has joined #openstack-ironic | 22:11 | |
| *** Sukhdev has quit IRC | 22:17 | |
| *** tesseract has quit IRC | 22:25 | |
| *** figleaf is now known as edleafe | 22:35 | |
| *** ElCoyote_ has left #openstack-ironic | 23:01 | |
| *** Goneri has joined #openstack-ironic | 23:01 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: DNM/WIP: Test possible grenade fix https://review.openstack.org/486255 | 23:04 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: DNM/WIP: Test possible grenade fix #2 https://review.openstack.org/486256 | 23:04 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: DNM/WIP: Test possible grenade fix #3 https://review.openstack.org/486257 | 23:05 |
| *** r-daneel has quit IRC | 23:31 | |
| *** hw_wutianwei has quit IRC | 23:33 | |
| *** hw_wutianwei has joined #openstack-ironic | 23:33 | |
| *** rpioso has quit IRC | 23:34 | |
| *** cdearborn_ has quit IRC | 23:46 | |
| *** baoli has joined #openstack-ironic | 23:52 | |
| *** baoli has quit IRC | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!