| opendevreview | Adam McArthur proposed openstack/ironic-python-agent master: Adding support to view indiv. cpu-core info https://review.opendev.org/c/openstack/ironic-python-agent/+/909346 | 04:35 |
|---|---|---|
| rpittau | good morning ironic! o/ | 06:40 |
| opendevreview | Merged openstack/ironic-python-agent master: Fix versions in release notes https://review.opendev.org/c/openstack/ironic-python-agent/+/904104 | 07:04 |
| dtantsur | Can anyone spare a +W? https://review.opendev.org/c/openstack/ironic-inspector/+/921875 has 2x +2 already. | 07:51 |
| rpittau | now has 3 +2, plus the +W :) | 07:53 |
| dtantsur | thx :) | 07:54 |
| rpittau | JayF: I started working on the gentoo ipa ramdisk, I'm at a good point with the ramdisk creation, just hitting some blockers on DIB | 07:56 |
| rpittau | like https://review.opendev.org/c/openstack/diskimage-builder/+/921769 | 07:56 |
| rpittau | and we'll need some work on ipa-builder of course | 07:56 |
| rpittau | I'll publish a PoC soon | 07:56 |
| rpittau | I've created https://bugs.launchpad.net/ironic-python-agent-builder/+bug/2069266 as a tracker for that | 08:00 |
| opendevreview | Merged openstack/ironic-inspector master: Cache openstacksdk Connections to avoid leaking memory https://review.opendev.org/c/openstack/ironic-inspector/+/921875 | 09:02 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-inspector stable/2024.1: Cache openstacksdk Connections to avoid leaking memory https://review.opendev.org/c/openstack/ironic-inspector/+/921923 | 09:03 |
| dtantsur | Looking for docs reviews: https://review.opendev.org/c/openstack/ironic/+/921844 | 10:05 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Decompose the Redfish documentation https://review.opendev.org/c/openstack/ironic/+/921844 | 10:37 |
| dtantsur | rpittau: updated ^^ | 10:37 |
| rpittau | thanks, re-added +2 :) | 10:38 |
| srdemircan | Hello Everyone, I have a problem while deploying baremetal instances on Yoga release. I can successfully register and inspect(with both redfish and inspector interfaces) my baremetal node. But when I try to deploy it is failing because ironic_inspector can not find node in node_cache. /v1/continue endpoint returning HTTP 500 status code. I have noticed ironic_inspector.attributes table has BMC address and mac_addresses | 11:20 |
| srdemircan | of two interfaces while performing the inspection but somehow after inspection is finished attributes table becomes empty. | 11:20 |
| srdemircan | I can not find what I am missing. Do you have any idea? | 11:20 |
| opendevreview | Merged openstack/ironic-inspector stable/2024.1: Cache openstacksdk Connections to avoid leaking memory https://review.opendev.org/c/openstack/ironic-inspector/+/921923 | 11:27 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix and document redfish metrics https://review.opendev.org/c/openstack/ironic/+/921951 | 11:30 |
| dtantsur | iurygregory: another thing that could use your attention ^^^ | 11:30 |
| dtantsur | especially since I'm modifying the code | 11:30 |
| dtantsur | srdemircan: inspection code path is more or less supposed to fail during deployment | 11:31 |
| dtantsur | you need to see what happens next | 11:31 |
| iurygregory | good morning Ironic | 11:55 |
| iurygregory | dtantsur, ack | 11:55 |
| *** tosky_ is now known as tosky | 12:28 | |
| TheJulia | good morning | 13:08 |
| shajizad | iurygregory: just sent the vmedia implementation for review | 13:16 |
| iurygregory | shajizad, ack o/ | 13:16 |
| opendevreview | Merged openstack/ironic stable/2023.2: Add states.SERVICING and SERVICEWAIT to need_prepare_ramdisk https://review.opendev.org/c/openstack/ironic/+/915349 | 13:26 |
| pocoyo | <shajizad> "iurygregory: just sent the..." <- I added functional and unit test for the changes made by Salman Hajizada , check it out over here: https://review.opendev.org/c/openstack/openstacksdk/+/921957 | 13:57 |
| iurygregory | pocoyo, sure I will add the patch to my list to take a look o/ | 13:58 |
| rpittau | pocoyo shajizad you will also need a release note for that | 14:24 |
| shajizad | How do we add release notes? | 14:31 |
| rpittau | shajizad: https://docs.openstack.org/reno/latest/ | 14:36 |
| rpittau | the reno tool ^ | 14:36 |
| opendevreview | cid proposed openstack/ironic master: Update version change log with special treatment of .json removal https://review.opendev.org/c/openstack/ironic/+/921966 | 14:44 |
| JayF | I'm kinda shocked I'm the first person to unearth https://review.opendev.org/c/x/proliantutils/+/921967 | 14:46 |
| JayF | it's been broken in xena since we backported a patch into xena to handle retries | 14:46 |
| JayF | (which passes server_side_retries=something to self._op, and self._op doesn't accept it in proliantutils) | 14:48 |
| JayF | my downstream is testing this patch now | 14:49 |
| opendevreview | cid proposed openstack/ironic master: Update version change log with special treatment of .json removal https://review.opendev.org/c/openstack/ironic/+/921966 | 14:56 |
| rpittau | I'm shocked to see that the six library is still required by proliantutils | 14:57 |
| JayF | it's pretty much unmaintained | 15:01 |
| JayF | so I'm not surprised it's showing signs of unmaintenance lol | 15:01 |
| iurygregory | I'm not surprised also | 15:07 |
| TheJulia | Not really surprising | 15:18 |
| dtantsur | JayF: few people use the old ilo driver, I suspect even fewer use its redfish bits | 15:23 |
| JayF | yeah, I am urging downstream to change to redfish driver for ilo5 gear, but they are preferring to have it atrophy out of the environment | 15:24 |
| JayF | not the worst thing for one of us to be able to actually test and fix little things like this though | 15:24 |
| TheJulia | we've basically been telling customers if you have an ilo5, just default to redfish | 15:27 |
| rpittau | so I got a 470MB gentoo ipa ramdisk, which I need to test | 15:43 |
| rpittau | it takes some effort to build it, but it looks promising, and I'm pretty sure we can reduce its size but I doubt we can get to tinyipa size | 15:43 |
| rpittau | for comparison, tinyipa is less than 100M, CS9 is 417M | 15:43 |
| rpittau | anyway, something to think about | 15:43 |
| rpittau | good night! o/ | 15:43 |
| JayF | rpittau: can you push your work up somewhere? | 15:43 |
| JayF | rpittau: I am pretty sure I can slim it further | 15:43 |
| JayF | likely need to deselect @system and instead only pick the bits we need | 15:44 |
| rpittau | JayF: yes, I will push a very dirty PoC tomorrow, bit in a rush now :) | 15:44 |
| JayF | and maybe use a savedconfig for linux firmware to get rid of non-vm-firmwares | 15:44 |
| JayF | I added an item to next meeting agenda; Reverbverbverb has the doc audit done to the point of a final review, we'll be giving ~a week for upstream to comment before the issues out of it get filed and all. I'll have a link to the google doc out to the mailing list today or tomorrow in advance of it. | 16:55 |
| shajizad | rpittau: I added release notes, not sure if I did it correctly though. | 17:14 |
| JayF | shajizad: link to the change, I can review for accuracy if you want | 17:15 |
| JayF | Riccardo is hopefully at home eating dinner or something now :D | 17:16 |
| shajizad | Salman Hajizada proposed openstack/openstacksdk master: Add virtual media attachment and detachment support https://review.opendev.org/c/openstack/openstacksdk/+/921957 | 17:17 |
| shajizad | I also did the VIF fix yesterday: https://review.opendev.org/c/openstack/openstacksdk/+/921890 | 17:17 |
| shajizad | Thank you | 17:18 |
| JayF | only one of those files is proper -- the other two are just standard templates -- if you `git rm` the two template files and only have the good one, you may be in better shape https://usercontent.irccloud-cdn.com/file/EAdgXEC5/image.png | 17:18 |
| shajizad | Alright, I'll do that a bit later | 17:48 |
| shajizad | Thanks | 17:48 |
| * TheJulia exits the land of searching through meeting notes | 18:37 | |
| cid | o/ | 20:06 |
| JayF | o/ | 20:11 |
| *** zbitter is now known as zaneb | 21:36 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
