| *** JoyAkinyi is now known as Guest29621 | 03:47 | |
| *** JoyAkinyi is now known as Guest29624 | 05:39 | |
| rpittau | good morning ironic! o/ | 06:23 |
|---|---|---|
| *** JoyAkinyi is now known as Guest29628 | 06:32 | |
| Ayo[m] | Good morning ironic!! | 06:46 |
| Ayo[m] | Good morning rpittau hroy mostepha :) | 06:48 |
| mostepha[m] | Good morning! :) | 06:49 |
| Ayo[m] | I made some patches yesterday | 06:50 |
| Ayo[m] | Can you review them for me | 06:50 |
| Ayo[m] | * I made some patches yesterday Can you review them for me mostepha | 06:50 |
| Ayo[m] | https://review.opendev.org/c/openstack/bifrost/+/964485 | 06:52 |
| Ayo[m] | https://review.opendev.org/c/openstack/bifrost/+/964485 | 06:53 |
| opendevreview | Merged openstack/ironic-python-agent master: Remove testing for pending-EOL items https://review.opendev.org/c/openstack/ironic-python-agent/+/962397 | 07:56 |
| mostepha[m] | Ayo: If you put the "ironic-week-prio" hashtag on your patches the review process might be sped up a bit :) | 09:06 |
| Ayo[m] | Ok mostepha I’ll do that | 09:48 |
| mostepha[m] | [Ayo](https://matrix.to/#/@ayobrown:matrix.org) It’s not that I don’t want to review them, I’m just not comfortable enough in Bifrost | 09:59 |
| Ayo[m] | mostepha ok, not a problem, I’ll just wait for one of my mentors then | 10:42 |
| opendevreview | Verification of a change to openstack/ironic master failed: api: Allow bios values to be None https://review.opendev.org/c/openstack/ironic/+/963376 | 10:55 |
| *** iurygregory_ is now known as iurygregory | 10:57 | |
| iurygregory | good morning ironic | 10:57 |
| opendevreview | Merged openstack/sushy master: Check required credentials in a detailed way https://review.opendev.org/c/openstack/sushy/+/963282 | 12:42 |
| iurygregory | rpittau, https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/964106 this is the patch I was talking about | 12:48 |
| rpittau | iurygregory: approved | 13:09 |
| iurygregory | tks | 13:22 |
| opendevreview | Merged openstack/ironic master: api: Allow bios values to be None https://review.opendev.org/c/openstack/ironic/+/963376 | 13:44 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: Adds new metric label for Redfish metrics https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/964106 | 13:54 |
| opendevreview | Ayo Edwin Kayode proposed openstack/bifrost master: Clarify correct usage of 'install --testenv' in bifrost-cli docs https://review.opendev.org/c/openstack/bifrost/+/964461 | 14:16 |
| cardoe | JayF: so yeah weird thing is that prepare service runs multiple times in the container | 14:53 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Deprecate tinyipa usage in CI https://review.opendev.org/c/openstack/bifrost/+/964404 | 15:15 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Deprecate tinyipa usage in CI https://review.opendev.org/c/openstack/bifrost/+/964404 | 15:18 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Deprecate tinyipa usage in CI https://review.opendev.org/c/openstack/bifrost/+/964404 | 15:21 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Deprecate tinyipa usage in CI https://review.opendev.org/c/openstack/bifrost/+/964404 | 15:23 |
| opendevreview | Harald Jensås proposed openstack/sushy-tools master: novadriver: Fix race for instance task_state https://review.opendev.org/c/openstack/sushy-tools/+/964550 | 15:44 |
| iurygregory | TheJulia JayF rpittau cardoe dtantsur cid I've sent the email to openstack-discuss with the PTG schedule o/ | 16:31 |
| cardoe | awesome. thank you for wrangling it. | 16:31 |
| TheJulia | Ack, thanks! | 16:31 |
| iurygregory | I will organize the etherpad accordingly to make things easier | 16:32 |
| opendevreview | Harald Jensås proposed openstack/sushy-tools master: novadriver: Fix race for instance task_state https://review.opendev.org/c/openstack/sushy-tools/+/964550 | 16:39 |
| *** AB is now known as Guest29665 | 17:01 | |
| cid | ack, \o. tks. | 17:08 |
| JayF | I think we are having ironic-standalone-anaconda failures b/c of bad centos images? cc clif https://usercontent.irccloud-cdn.com/file/DBTkBkBi/image.png | 17:16 |
| JayF | either that or outta ram? | 17:16 |
| JayF | https://635fcb2fbca57989d511-b59b5fe5fc805502cf7724cd208fae25.ssl.cf1.rackcdn.com/openstack/c66d2d5d8c274cdd8f38cf6c8df62dff/controller/logs/ironic-bm-logs/node-1_no_ansi_2025-10-20-19%3A16%3A25_log.txt | 17:17 |
| JayF | > [ ]CentOS-Stream-GenericCloud-9-20251020.0.x86_64.qcow22025-10-21 02:28 1.3G | 17:23 |
| JayF | timing of a new image is extremely sus | 17:23 |
| JayF | I am 90%+ confident this is a "the image changed and broke us" after some observation | 17:26 |
| JayF | I'm going to ask clif to push a change to i-t-p to make them temporarily non-voting | 17:27 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add support for multiple JSON-RPC configuration groups https://review.opendev.org/c/openstack/ironic/+/960593 | 17:36 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add force_dhcp option for managed inspection https://review.opendev.org/c/openstack/ironic/+/960824 | 17:36 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add standalone networking service for ironic https://review.opendev.org/c/openstack/ironic/+/962493 | 17:36 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: pass along physical_network to neutron from the baremetal port https://review.opendev.org/c/openstack/ironic/+/964570 | 18:05 |
| cardoe | TheJulia: finally trying to get back to that network stuff... stupid idea... ^ | 18:06 |
| cardoe | Maybe I'm complicating the spec by trying to tie in the VLAN segment matching to the VXLAN | 18:06 |
| TheJulia | Joy, okay! I have another item regarding Smartnics/dpus to put in as well. When I’m back to home. | 18:07 |
| cardoe | I'll extract out some logic into a small ML2 plugin for the other side to that the change is complete. | 18:08 |
| * TheJulia needs to go find post-dinner dessert | 18:08 | |
| cardoe | And then try and sit down with ralonsoh again./ | 18:08 |
| TheJulia | Yup | 18:11 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Deprecate tinyipa usage in CI https://review.opendev.org/c/openstack/bifrost/+/964404 | 18:45 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Deprecate tinyipa usage in CI https://review.opendev.org/c/openstack/bifrost/+/964404 | 18:46 |
| janders | TheJulia would you have few minutes to talk about https://review.opendev.org/c/openstack/ironic/+/960230? | 19:03 |
| TheJulia | janders: not at the moment, really. | 19:04 |
| janders | TheJulia ACK | 19:05 |
| TheJulia | janders: waiting for my bar receipt, what’s up? | 19:05 |
| janders | I hear your concerns about the patch, wanted to go through the context and see if we can improve or validate the logic | 19:06 |
| janders | but if you don't have bandwidth let's try figure out something quick | 19:06 |
| janders | it happens to be the first out of a chain of three patches where no 2 and no 3 are needed for some bugfixes people are chasing me about | 19:07 |
| janders | TL;DR is the reason why I put such a persistent set of checks is BMCs of servers in question are being "interesting" and going offline AFTER the Redfish Task tied to the update is marked complete | 19:08 |
| TheJulia | I mean, I don’t like it, but it is what it is and we don’t really have a better way at the moment | 19:08 |
| TheJulia | Which is why I’m not blocking it | 19:09 |
| janders | and it's kind of hard to figure out if the BMC is done doing it's thing or does it respond once and then goes incommunicado | 19:09 |
| TheJulia | At least, I don’t think I am | 19:09 |
| janders | I wanted to explain this in more detail than may be obvious in the patch thinking you may have a suggestion on taking a different approach | 19:09 |
| TheJulia | After way would be the whole deferred task idea with saving the iteration count across invocations | 19:10 |
| TheJulia | Err, not after, but a way after we have deferred tasks | 19:10 |
| janders | right | 19:10 |
| janders | but for now we don't have anything significantly less bad we can use here, right? | 19:11 |
| janders | (happy to refactor it with deferred tasks once they are a thing) | 19:11 |
| TheJulia | Not really, at least nothing I can think of at the moment | 19:12 |
| janders | OK | 19:12 |
| janders | I will think a bit more about the default config (number of retries, in-between intervals) and try to make it merge ready | 19:12 |
| janders | thank you TheJulia | 19:12 |
| TheJulia | Yeah, lower interval might make some sense, but also keep in mind request timeouts will be blocking as well | 19:16 |
| TheJulia | And could run 60 seconds | 19:17 |
| janders | my semi-educated guess is once the Task is complete it may take up to 2-3 minutes for the BMC to go down | 19:18 |
| janders | initial request will succeed more often than not, I expect the second or third to fail | 19:18 |
| janders | what would be your suggestion on the interval, given there is also the timeout? | 19:19 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix power sync timeouts after BMC firmware update https://review.opendev.org/c/openstack/ironic/+/962560 | 20:17 |
| *** AB is now known as Guest29681 | 20:18 | |
| opendevreview | Jay Faulkner proposed openstack/ironic master: [DNM/Science] More RAM for Anaconda job https://review.opendev.org/c/openstack/ironic/+/964664 | 20:58 |
| opendevreview | Jay Faulkner proposed openstack/ironic-tempest-plugin master: Change Portgroup minimum microversion to 1.26 https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/955799 | 20:59 |
| opendevreview | Jay Faulkner proposed openstack/ironic-tempest-plugin master: Change Portgroup minimum microversion to 1.26 https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/955799 | 20:59 |
| *** AB is now known as Guest29688 | 22:52 | |
| cardoe | janders: has you looked if there’s a follow on task? Some BMCs do that. | 23:57 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
