| rpittau | good morning ironic! o/ | 06:47 |
|---|---|---|
| rpittau | reminder: no regular meeting today, it's vPTG week! :) | 06:48 |
| rpittau | topics and schedule here https://etherpad.opendev.org/p/ironic-ptg-april-2025 | 06:48 |
| rpittau | we start at 1400 UTC | 06:48 |
| kubajj | rpittau: 🫡 | 06:50 |
| rpittau | lol | 06:54 |
| kubajj | good morning, btw :D | 06:58 |
| Ayo[m] | Good morning rpittau | 07:01 |
| AmarachiOrdor[m] | Good Morning! rpittau , good morning everyone! | 07:04 |
| freemanboss[m] | Good morning ironic | 07:43 |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent master: Implement manual cleaning for ContainerHardwareManager https://review.opendev.org/c/openstack/ironic-python-agent/+/945862 | 11:15 |
| cardoe | Morning ironic. | 12:21 |
| iurygregory | good morning ironic | 12:26 |
| cardoe | My brain is not processing the network convo from last week JayF and TheJulia. Maybe more coffee. | 12:32 |
| iurygregory | coffee++ | 12:33 |
| vsaienko_ | JayF, TheJulia: https://review.opendev.org/c/openstack/ironic/+/900568 - this patch is not needed in order to implement OVN + VTEP support with ironic, what will be the proper way to revert it by taking into account that we did microversion bumps | 12:50 |
| JayF | vsaienko_: can you explain more? | 12:51 |
| cardoe | Yeah you'll have to explain some more because that just added those fields as valid fields on local_link_connection | 12:57 |
| vsaienko_ | we already have all needed info in local_link_connection information its switch identifier and physical port. One of fields that were added is logical-switch (which is an abstraction of virtual network). This should be created dynamically since we can plug node into different virtual networks. And code that plug port into logical switch (its called bind-ls in ovn terminology) is not implemented on neutron side at all. Please chec | 13:01 |
| vsaienko_ | k https://man7.org/linux/man-pages/man7/ovn-architecture.7.html `Life Cycle of a VTEP gateway` paragraph with details on how VTEP works. I have a PoC that adds support to n-g-s will push it later today | 13:01 |
| TheJulia | And here I'm just starting to caffinate | 13:06 |
| JayF | vsaienko_: I was under the impression part of the goal of that change was to take NGS out of the picture? | 13:07 |
| vsaienko_ | its not possible | 13:07 |
| cardoe | I'm curious why you say that. | 13:08 |
| JayF | It seems like the first step would be filing a bug about this being wrong and talking about the right way, and then maybe we can get that guy from juniper to comment in there | 13:08 |
| TheJulia | cat coffee | julia | 13:10 |
| vsaienko_ | if you look closer commit that added vtep-logical-switch into neutron was implemented for l2-gtw case, in that case l2gtw creates logical-switch and binds ports | 13:15 |
| TheJulia | Lets please follow bug process since it seems like either bad or incomplete context was presented early on | 13:17 |
| JayF | Yeah I mainly just want to get documented how we missed it and see if we can at least get the original requester to put a comment in | 13:22 |
| vsaienko | https://bugs.launchpad.net/networking-ovn/+bug/1457569 https://review.opendev.org/c/openstack/networking-ovn/+/227711 | 13:22 |
| vsaienko | and btw, it was never working, https://review.opendev.org/c/openstack/ironic/+/945964 the fields were not passed to neutron at all due to keyError exception | 13:23 |
| TheJulia | I think we all realize the latter part by now | 13:54 |
| TheJulia | The question is how do we fix it and that deserves a bug to track | 13:54 |
| rpittau | ironic flamingo PTG starting now https://ptg.opendev.org/ptg.html | 14:01 |
| JayF | honestly given the previous contributed patch was said to be tested working | 14:07 |
| JayF | I'm wondering what the hell the rest of that story is | 14:07 |
| JayF | but we may never find out | 14:07 |
| TheJulia | indeed! | 14:10 |
| JayF | it makes sense if it broke from submitted patch -> commit; we knew that was possible; but vsaienk__ is suggesting the whole idea was flawed from the start | 14:12 |
| dtantsur | cardoe: (inspired by the items I see in the etherpad) do you have any interest in running (Redfish-based) inspection as a service step? | 14:16 |
| cardoe | Maybe. | 14:16 |
| opendevreview | Vasyl Saienko proposed openstack/networking-generic-switch master: WIP: Add support of OVS VTEP devicesy https://review.opendev.org/c/openstack/networking-generic-switch/+/946558 | 14:30 |
| opendevreview | Nicolas Belouin proposed openstack/ironic-python-agent master: netutils: Use ethtool ioctl to get permanent mac address https://review.opendev.org/c/openstack/ironic-python-agent/+/946562 | 14:42 |
| vsaienko | TheJulia: let me create a bug and describe in details why/how VTEP with ironic does not work and we will decide what to do next | 14:57 |
| kulsooms | rpittau, masghar: Hi mentors, please let me know if submitting a patch to the Ironic project is also required (along with the summary and report) to be accepted as an intern? | 15:00 |
| satoshi | kulsooms: The third task is highly encouraged but optional. | 15:48 |
| masghar | kulsooms: satoshi is correct, its highly encouraged, but optional :) | 15:57 |
| JayF | I know other outreachy candidates have submitted changes. | 15:57 |
| JayF | I also don't know how much that impacts decisions as I'm not a decider. | 15:57 |
| rpittau | well, it is indeed highly encouraged :D | 16:00 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent-builder master: Science: Remove ipmitool from tinyipa https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/946573 | 16:41 |
| freemanboss[m] | rpittau: masghar please I just launched this patch, help me check it out please. https://storyboard.openstack.org/#!/story/2011398 | 16:43 |
| rpittau | freemanboss[m]: thanks for that, you should report that in launchpad though https://launchpad.net/bifrost we don't use storyboard anymore | 16:45 |
| freemanboss[m] | Please I will be happy if I can get a review as soon as possible so I can know if I am on the right path cause I still have some patch I | 16:45 |
| freemanboss[m] | rpittau: oh alright. I will go with that now | 16:46 |
| freemanboss[m] | rpittau: please I am not getting the icon or link to click on to create a bug | 16:52 |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic master: WIP Add allow_ironic_project_image_access https://review.opendev.org/c/openstack/ironic/+/946575 | 16:56 |
| freemanboss[m] | https://bugs.launchpad.net/bifrost/+bug/2106416 | 17:07 |
| freemanboss[m] | rpittau: masghar this is it now | 17:07 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Stop using eventlet for notification listener https://review.opendev.org/c/openstack/networking-baremetal/+/946576 | 17:42 |
| * JayF jumping into https://meetpad.opendev.org/ironic-ptg-followup-cidocs | 17:58 | |
| iurygregory | cardoe, you added the "Firmware Updates via Push" to the ptg etherpad right? do you have links about it I can take a look? :D | 18:04 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Science: base standalone job on redfish standalone https://review.opendev.org/c/openstack/ironic-python-agent/+/946580 | 18:12 |
| JayF | ironic-tempest-ramdisk-bios-snmp-pxe | 18:34 |
| cardoe | iurygregory: I did add it. Uhhh lemme find some. | 18:37 |
| iurygregory | tks | 18:38 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: remove duplicatd rbac jobs https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/946582 | 18:51 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: PTG: Move CI job descriptions into yaml https://review.opendev.org/c/openstack/ironic/+/946584 | 19:20 |
| kulsooms | satoshi, masghar, JayF, rpittau: ok thank you | 19:22 |
| JayF | TheJulia: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/2PME5HIGOSCOWPX3EFYOYNFCGSPVKVM6/ as discussed in our breakout earlier | 19:39 |
| opendevreview | Queensly Kyerewaa Acheampongmaa proposed openstack/bifrost master: Clarify usage of master branch and SSH key types in testenv.rst https://review.opendev.org/c/openstack/bifrost/+/946116 | 19:45 |
| JayF | is it somehow possible for our hash ring to get so absolutely bamboozled that a node is perm never working with a conductor? | 20:10 |
| JayF | e.g. Apr 07 20:06:50 ip-172-31-94-37 devstack@ir-api.service[4096727]: DEBUG ironic.api.controllers.v1.node [None req-53c704b3-caff-45ce-89a9-8df66f59d0c2 None admin] Currently there is no conductor servicing node e2601873-0399-44f1-ada5-c3a6485d37c4. {{(pid=4096727) node_convert_with_links /opt/stack/ironic/ironic/api/controllers/v1/node.py:1662}} in the logs | 20:12 |
| JayF | cc satoshi | 20:12 |
| JayF | node basically goes to cleaning and just parks there indefinately | 20:12 |
| JayF | I suspect this is somewhat environmentally driven but it's too strange to not try and suss out | 20:13 |
| JayF | node.reservation in the DB is null, and a restart of the services makes no hcange | 20:15 |
| satoshi | https://review.opendev.org/c/openstack/ironic/+/946575 This is the patch I'm trying to test. | 20:16 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Added a FAQ on CI debugging, starts with network https://review.opendev.org/c/openstack/ironic/+/946589 | 20:29 |
| JayF | well, we managed to cause that issue to occur after reverting to master branch + restarting | 20:29 |
| JayF | so I don't /think/ it's related to your code, but who knows?! | 20:29 |
| JayF | (I also don't think this is indicative of any ironic break; I'm just curious how folks would dig into something like this) | 20:45 |
| TheJulia | ENOBRAIN | 21:03 |
| TheJulia | JayF: that would be a new one, knowing the algorithm model under the hood I'm not sure how that would have occured | 21:05 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Base standalone job on redfish standalone https://review.opendev.org/c/openstack/ironic-python-agent/+/946580 | 21:11 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: remove duplicated rbac jobs https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/946582 | 21:18 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: Remove legacy standalone jobs from test execution https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/946592 | 21:18 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: CI: Increment jobs for 2025.1 release https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/946593 | 21:18 |
| TheJulia | JayF: ^ | 21:18 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Fix tempest - enable neutron-trunk service plugin https://review.opendev.org/c/openstack/networking-baremetal/+/946594 | 21:18 |
| JayF | TheJulia: https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/946593/1#message-3a1e3b04d9cbe9272526e2ee346540ead42fd8a8 | 21:19 |
| TheJulia | good catch, meant to remove htem, thanks | 21:20 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: CI: Increment jobs for 2025.1 release https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/946593 | 21:21 |
| JayF | +2 | 21:22 |
| hjensas | TheJulia: ^ on the topic of what we discussed in PTG. The trunk tests added to ngs broke networking-baremetal because trunk service_plugin not enabled in neutron. (I think). If I wanted to disable it instead, could I set something like tempest_baremetal_feature_enabled_trunks_supported: False in the zuul yaml? | 21:24 |
| TheJulia | not really | 21:25 |
| TheJulia | you'd have to format it as an entire thing | 21:25 |
| TheJulia | I think the cleanest path is just "don't turn it on by default" | 21:25 |
| TheJulia | uhhh. | 21:25 |
| TheJulia | https://review.opendev.org/c/openstack/networking-generic-switch/+/946089 | 21:25 |
| hjensas | oh, yes - that would do it. | 21:27 |
| JayF | hjensas: if you are +1 to that, put the vote on there and I'll single core land it | 21:28 |
| * TheJulia wonders if there are any stray threads which still need attention today | 21:29 | |
| JayF | my brain is leaking, personally ;) | 21:29 |
| JayF | between ptg and wtf'ing at satoshi's devstack | 21:30 |
| opendevreview | Merged openstack/ironic master: Create Support Resources Page https://review.opendev.org/c/openstack/ironic/+/941293 | 21:58 |
| opendevreview | Merged openstack/ironic master: Ironic Container Config for IPA Integration https://review.opendev.org/c/openstack/ironic/+/944796 | 22:39 |
| opendevreview | Merged openstack/networking-generic-switch master: CI: Fix trunks enabled by default https://review.opendev.org/c/openstack/networking-generic-switch/+/946089 | 23:27 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!