| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Retry in ProxyErro during post inspector data https://review.opendev.org/c/openstack/ironic-python-agent/+/902448 | 00:20 |
|---|---|---|
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Retry in ProxyError during post inspector data https://review.opendev.org/c/openstack/ironic-python-agent/+/902448 | 00:22 |
| rpittau | good morning ironic! o/ | 08:00 |
| rpittau | JayF: the new bugfix branches are out already since last week, I think that topic has been left from last meeting | 08:36 |
| rpittau | dtantsur: 100% in favor of the multiple callback urls in IPA | 08:41 |
| rpittau | dtantsur: for Bug 2045551 also looks ok, for the variable name following JayF suggestion I would go for something like 'generate_ipxe_fallback_script' as a direct pointer to ipxe_fallback_script variable | 08:53 |
| iurygregory | good morning Ironic | 10:57 |
| iurygregory | rpittau, dtantsur in case you have cycles https://review.opendev.org/c/openstack/ironic-python-agent/+/902448 this is for the 502 proxy error with inspector | 10:57 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Fix volume_name issue in Redfish RAID https://review.opendev.org/c/openstack/ironic/+/902649 | 12:57 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Use sha256 instead of md5 by default https://review.opendev.org/c/openstack/bifrost/+/902656 | 13:20 |
| n0ob | is some one here having experience with developing the sushy emulator? | 14:28 |
| iurygregory | you mean sushy-tools right? | 14:32 |
| * iurygregory wondering if could be other emulator outside of openstack, so he want to be sure | 14:32 | |
| masghar | If one fixes an issue in two changes, should they update the first release note, or make a new one? | 15:17 |
| JayF | So I try to think about the end state https://docs.openstack.org/releasenotes/ironic/unreleased.html which is here | 15:19 |
| JayF | combined with if it needs to be backported | 15:20 |
| JayF | if no backport, and it really is just a fix-of-a-fix, just edit the release note | 15:20 |
| JayF | only exception is when that release note crosses a release boundry | 15:20 |
| masghar | It would be a good-if-backported fix, but it doesnt cross a release boundary | 15:26 |
| masghar | I will add a new release note | 15:26 |
| masghar | Thanks Jay! | 15:27 |
| opendevreview | Merged openstack/bifrost stable/2023.1: ansible-lint: Skip key-order[play] https://review.opendev.org/c/openstack/bifrost/+/902150 | 15:31 |
| mnasiadka | Hello, is anybody familiar with the CI failures in https://review.opendev.org/c/openstack/ironic-python-agent/+/899883 ? It would be very useful to get this patch backported, but I most probably lack the background around why those node cleanings are failing ;-) | 15:31 |
| n0ob | iurygregory: yes that's rigth | 15:40 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Fix volume_name issue in Redfish RAID https://review.opendev.org/c/openstack/ironic/+/902649 | 15:42 |
| iurygregory | n0ob, cool, feel free to ask any questions regarding sushy-tools and we will try to answer them | 15:42 |
| *** dmellado206 is now known as dmellado | 15:54 | |
| n0ob | iurygregory: we are doing a project where a HW redfish api is involved. Now we want to use the sushy emulator as a backend for testing. But it is missing some features we need and would like to implement | 16:16 |
| JayF | n0ob: What's the project, just curious? | 16:18 |
| JayF | n0ob: and FWIW, sushy-tools is the emulator; sushy is a general purpose python redfish library (both projects are Ironic-maintained :D:) | 16:18 |
| n0ob | JayF: custom tool to provision servers | 16:19 |
| JayF | n0ob: any reason why Ironic or a preexisting tool doesn't do the trick for you? (can't fault me for doing product research while also helping you ;P) | 16:19 |
| JayF | n0ob: if I can't talk you into using ironic, and you need features in sushy-tools, best route is file a bug here https://bugs.launchpad.net/sushy-tools describing what you need; put a note in RFE review section here https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 16:20 |
| n0ob | JayF: I don't know if there was an evaluation of any tool before starting the project | 16:20 |
| JayF | n0ob: well I'm the team lead for this cycle in Ironic, and chair of the TC for all openstack this cycle too; happy to chat with you/your team/bosses/whoever if it's helpful in saving folks from reimplementing the world :D | 16:20 |
| JayF | it is easy to underestimate the complexity of hardware provisioning | 16:21 |
| n0ob | JayF: i came to the project ~ 4 months ago and i already see the complexity ;) | 16:21 |
| JayF | given you haven't named it, I assume it's something downstream/proprietary and not OSS? | 16:22 |
| n0ob | yeah nothing open sadly | 16:22 |
| JayF | Well, that's the process for requesting enhancements in sushy-tools | 16:23 |
| JayF | obviously you'd likely also have to implement the feature, too :) | 16:24 |
| JayF | w | 16:24 |
| JayF | I know that janders also has some tools you might find useful; some redfish schema stuff which like, pulls the schema off hardware to make it easier to troubleshoot | 16:24 |
| n0ob | JayF: yeah i am aware of the "implement it yourself" would be nice to have it upstream though | 16:24 |
| JayF | well, I'm saying implement it yourself meaning like, even if upstream, you'd likely write the code :) | 16:25 |
| JayF | we don't have a surplus of people to implement features for sushy-tools :D | 16:25 |
| JayF | but we'd be happy to help review/merge your code upstream and help keep it alongside the rest so you don't have to rebase until the heat death of the universe :D | 16:25 |
| n0ob | that sounds great! | 16:26 |
| JayF | good luck, but not too much good luck if you're our competetor :P | 16:26 |
| n0ob | JayF: i don't think so ;) | 16:27 |
| rpittau | good night! o/ | 17:08 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix volume_name issue in Redfish RAID https://review.opendev.org/c/openstack/ironic/+/902649 | 18:44 |
| JayF | Ironic'ers: we will soon have to decide how we want to manage patching on our older branches | 18:47 |
| JayF | with the change from extended maintenance to unmaintained branches, we're forking the core groups since we no longer assume the core teams are responsible for unmaintained branches | 18:47 |
| JayF | default behavior will be that unmaintained branches get an openstack-wide core group (currently populated with tonyb and elodliles), with power to merge on any unmaintained branch, but individual projects can override that with their own group (or append -- create our own group and add the global group to it) | 18:48 |
| JayF | I would propose we do just that -- create an ironic-unmaintained-cores group, add the global openstack-unmaintained-core group to it, and add any volunteer cores (or really, non-cores if we trust them) to ironic-unmaintained-core to permit us the ability to backport our own stuff, still | 18:49 |
| iurygregory | ^ makes sense to me | 18:49 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix volume_name issue in Redfish RAID https://review.opendev.org/c/openstack/ironic/+/902649 | 19:28 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Fix referencing to the raid_device var which is not set https://review.opendev.org/c/openstack/ironic-python-agent/+/900324 | 19:33 |
| iurygregory | this doesn't look good https://zuul.opendev.org/t/openstack/build/9b19b7881fef4d75b60aceb515453331/log/job-output.txt#26149 | 19:38 |
| iurygregory | I can reach the url so I will trigger recheck in the patch | 19:38 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Cleanup py27 support https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/890732 | 19:50 |
| *** zbitter is now known as zaneb | 20:23 | |
| opendevreview | Merged openstack/ironic master: Handle internal server errors while configuring secure boot https://review.opendev.org/c/openstack/ironic/+/902431 | 20:39 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Add test for dhcp-less vmedia based deployment https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/898006 | 20:39 |
| iurygregory | if anyone has cycles for review https://review.opendev.org/c/openstack/ironic-python-agent/+/902448 o/ | 21:01 |
| opendevreview | Merged openstack/ironic master: Fix volume_name issue in Redfish RAID https://review.opendev.org/c/openstack/ironic/+/902649 | 22:11 |
| JayF | Any core reviewer around with some time to review the Ironic vPTG summary slides for OpenInfra live on Thursday morning? | 23:09 |
| JayF | stevebaker[m]: you around, perchance? | 23:10 |
| stevebaker[m] | I am | 23:11 |
| JayF | DM'd you, looking for a review on a private slide deck | 23:13 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!