| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Extend Fake System Driver for Interaction with Fake IPA via REST API https://review.opendev.org/c/openstack/sushy-tools/+/923111 | 05:23 |
|---|---|---|
| rpittau | good morning ironic! o/ | 06:58 |
| sylvr | morning! | 06:59 |
| Sandzwerg[m] | morning | 08:24 |
| adam-metal3 | Hello Ironic, it is probably a very sily question but as far as I understand it is not possible to disable the automatic restarting of the node after the deployment steps have completed right? I am talking about regular no deploymetn workflow not live ISO . | 09:16 |
| adam-metal3 | *regular deployment not live ISO workflow | 09:17 |
| opendevreview | Merged openstack/ironic master: Update the redfish interoperability profile https://review.opendev.org/c/openstack/ironic/+/920574 | 09:19 |
| masghar | Good morning! (almost afternoon but morning in my book =D) | 09:26 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Extend Fake System Driver for Interaction with Fake IPA via REST API https://review.opendev.org/c/openstack/sushy-tools/+/923111 | 09:50 |
| opendevreview | Jacob Anders proposed openstack/sushy-tools master: Replace hardcoded BiosVersion with an updatable field https://review.opendev.org/c/openstack/sushy-tools/+/909487 | 10:22 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Extend Fake System Driver for Interaction with Fake IPA via REST API https://review.opendev.org/c/openstack/sushy-tools/+/923111 | 10:42 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Extend Fake System Driver for Interaction with Fake IPA via REST API https://review.opendev.org/c/openstack/sushy-tools/+/923111 | 10:44 |
| opendevreview | cid proposed openstack/ironic master: Deprecated field in Redfish Driver https://review.opendev.org/c/openstack/ironic/+/922951 | 10:56 |
| iurygregory | good morning Ironic | 11:08 |
| dtantsur | adam-metal3: please elaborate. You want to leave IPA running? | 11:20 |
| adam-metal3 | I want to execute an in-band custom deployment step and then I don't want to reboot, I wouldn't leave IPA running the custom step would deploy a daemon that would do some modifications to the node, delay the reboot but eventually it would reboot | 11:24 |
| dtantsur | adam-metal3: okay, so you have a custom deploy step, and it takes time. What's the problem with just making Ironic wait for the step to be over? | 11:25 |
| adam-metal3 | these modifications that the daemon would do are coming from som proprietary stuff I don't even know what that is but I would basically hand over the controll to this daemon, so IPA would be finished with all of it's task technically | 11:26 |
| dtantsur | adam-metal3: we have a very similar mode operation where the IPA hands over its job to a different service via dbus. Then it just loops waiting for the service to exit, then Ironic does its routine clean-up. Would that work for you? | 11:27 |
| dtantsur | adam-metal3: this https://github.com/openshift/ironic-agent-image/blob/main/hardware_manager/ironic_coreos_install.py#L143-L160 | 11:27 |
| dtantsur | Now, there is a special step called "hold", but that's not exactly what you're asking for, and probably also not what you really need. | 11:28 |
| adam-metal3 | Yeah I have checked the core os stuff but in my case there could be a situation where I would never reboot, there is a possibility that this daemon would do a systemd soft reboot rather instead of a regural reboot thus the whole systemd userspace would go away | 11:30 |
| adam-metal3 | but the kernel would stay running and a new systemd session would be re-initialized | 11:32 |
| dtantsur | adam-metal3: I don't think we have anything like that out of box, unfortunately. | 11:53 |
| adam-metal3 | I would be happy to somehow implement support for this, but could you please point me tha the function call where in case of custom agent deploy process Ironic asks for a reboot because I have a hard time finding it. I was tracing the code in Ironic and I see that in case of using IPA with cutom agent deploy I arrive to this condition https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/modules/agen | 12:00 |
| adam-metal3 | t.py#L258 and in case of NO fast track there is a power action but in case of no fast track I don't see, I might very well be looking at the wron place but I just can't find where exactly does Ironic initiats the reboot agter deployment | 12:00 |
| adam-metal3 | *point me to the function call* sorry typing in HexChat is a bit messy | 12:01 |
| adam-metal3 | so in case of no fast track I see the power action in case of NO fast track but where fast track is enabled I don't see the power action | 12:03 |
| TheJulia | good morning | 12:32 |
| dtantsur | adam-metal3: it has two parts. The power off happens in this deploy step https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/modules/agent_base.py#L1257-L1339 | 12:35 |
| iurygregory | good morning TheJulia =) | 12:35 |
| dtantsur | adam-metal3: power on here https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/modules/agent_base.py#L1176-L1200 | 12:35 |
| adam-metal3 | dtantsur, thanks! | 12:37 |
| dtantsur | adam-metal3: what you want is definitely doable with a custom deploy interface | 12:37 |
| TheJulia | I'm likely not going to be around for this weeks meeting. I'm dealing with some sickness and I have to run into town which is best when it is cooler outside because a pesky nail in a tire | 13:23 |
| iurygregory | take care TheJulia | 13:24 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Upgrade check on removed or deprecated hardware types and interfaces https://review.opendev.org/c/openstack/ironic/+/923164 | 13:44 |
| dtantsur | TheJulia: something I promised you quite long ago ^^^ | 13:44 |
| TheJulia | awesome, I'll try to take a look in the next day or so | 13:45 |
| TheJulia | Thanks! | 13:45 |
| dtantsur | no hurry, just ticking boxes on my TODO list | 13:45 |
| cardoe | Did I get my timing wrong. Is the Ironic meeting not now/ | 14:13 |
| cardoe | I did get my punctuation wrong... more coffee! | 14:13 |
| TheJulia | coffee++ | 14:14 |
| TheJulia | in like 45 minutes | 14:14 |
| cardoe | Well I'll just share a bit early. I did a really crappy write up of what I'm working on here... https://rackerlabs.github.io/understack/vision/ essentially TheJulia leaves DCIM/IPAM out of Project Mercury. But I'm saying let's make an opinionated deployment and use Nautobot to provide that functionality. | 14:44 |
| cardoe | Ultimately the design goal is for large scale, multiple data centers but want to scale down to small footprints as well. | 14:46 |
| cardoe | The reason for opinionated is just to have a reference deployment that others can use and copy from. | 14:46 |
| rpittau | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Jul 1 15:00:04 2024 UTC and is due to finish in 60 minutes. The chair is rpittau. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
| dtantsur | o/ | 15:00 |
| rpittau | Hello everyone! | 15:00 |
| rpittau | Welcome to our weekly meeting! | 15:00 |
| rpittau | The meeting agenda can be found here: | 15:00 |
| rpittau | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_July_01.2C_2024 | 15:00 |
| cid | o/ | 15:00 |
| masghar | o/ | 15:00 |
| rpittau | mmm I was hoping to have more cores today, let's wait a minute or two | 15:01 |
| iurygregory | o/ | 15:01 |
| rpittau | #topic Announcements/Reminders | 15:02 |
| rpittau | #info Standing reminder to review patches tagged ironic-week-prio and to hashtag any patches ready for review with ironic-week-prio | 15:02 |
| rpittau | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:02 |
| rpittau | I think the list is in good shape but we may need to go back to some changes from beginning of June | 15:03 |
| rpittau | #info 2024.2 Dalmatian Release Schedule | 15:04 |
| rpittau | #link https://releases.openstack.org/dalmatian/schedule.html | 15:04 |
| rpittau | we're in milestone 2 meaning we'll see some releases this week | 15:04 |
| rpittau | #info The nickname for the 2025.1 "E" release will be Epoxy | 15:04 |
| rpittau | #info the next OpenInfra PTG which will take place October 21-25, 2024 virtually! Registration is now open! | 15:05 |
| rpittau | #link https://ptg.openinfra.dev/ | 15:05 |
| rpittau | Team Survey deadline is August 30th, I will take care of it :) | 15:05 |
| rpittau | anything else to announce/remind of ? | 15:05 |
| rpittau | okey dokey, moving on | 15:06 |
| rpittau | #topic Review Ironic CI status | 15:06 |
| rpittau | haven't seen anything too broken last week | 15:06 |
| rpittau | anything to mention about CI status? | 15:07 |
| rpittau | onward! | 15:08 |
| rpittau | #topic Discussions | 15:08 |
| rpittau | I have 2 things for today, let's see | 15:08 |
| rpittau | #info any non-libraries we want to release? | 15:08 |
| dtantsur | CI was briefly broken by devstack, but it's okay now | 15:08 |
| rpittau | thanks dtantsur :) | 15:09 |
| dtantsur | non-libraries? like ironic itself you mean? | 15:09 |
| rpittau | yes | 15:09 |
| rpittau | we have released recently ironic, ipa and inspector bugfix | 15:09 |
| rpittau | but we also have networking-baremetal and networking-generic-switch | 15:10 |
| rpittau | I can have a look but just wondering if anyone already had an idea | 15:10 |
| dtantsur | I don't really. I haven't seen many changes there recently, but dunno when the last release was. | 15:10 |
| rpittau | ok, I have it in my todo list for this week anyway because of the milestone | 15:11 |
| rpittau | library releases will be automatically proposed by the release team | 15:11 |
| dtantsur | then we have sushy-tools but it should probably wait for Jacob's patches | 15:12 |
| rpittau | yep | 15:12 |
| rpittau | I guess we're good | 15:13 |
| rpittau | the second thing I had is | 15:14 |
| rpittau | #info pysnmp-lextudio 6.x migration | 15:14 |
| iurygregory | sushy-tools is independent release no? | 15:14 |
| dtantsur | right | 15:14 |
| rpittau | true | 15:14 |
| rpittau | than we don't care and we can release when we want | 15:14 |
| rpittau | thanks iurygregory for reminding that :) | 15:15 |
| iurygregory | np =) | 15:15 |
| rpittau | going back to pysnmp-lextudio | 15:15 |
| rpittau | the upgrade cause a huge breakage and we can't move forward until we adapt virtualpdu to asyncio | 15:15 |
| rpittau | I left 2 messages in the original issue | 15:16 |
| rpittau | #link pysnmp-lextudio 6.x migration | 15:16 |
| rpittau | whoops | 15:16 |
| rpittau | #link https://github.com/lextudio/pysnmp/issues/49 | 15:16 |
| iurygregory | just wondering, but if we adapt virtualpdu, could it cause problems since ironic doesn't use asyncio? | 15:16 |
| rpittau | well we need to do both | 15:17 |
| rpittau | but virtualpdu seems like a bigger change | 15:17 |
| iurygregory | got it | 15:17 |
| rpittau | this is an example of what's happening | 15:17 |
| rpittau | #link https://review.opendev.org/c/openstack/virtualpdu/+/922158 | 15:17 |
| rpittau | looks like an issue with python threading | 15:17 |
| rpittau | if anyone has time to have a look at it feel free to hijack my patch | 15:18 |
| rpittau | if we don't get answers from the pysnmp maintainer we may need to find yet another alternative | 15:18 |
| rpittau | any questions/doubts/comments? | 15:19 |
| cid | Nop | 15:20 |
| rpittau | ok, moving on | 15:20 |
| rpittau | #topic Bug Deputy Updates | 15:20 |
| rpittau | JayF was deputy last week but I don't see him around | 15:21 |
| cid | Yea, he's not on seat | 15:21 |
| rpittau | alright | 15:21 |
| rpittau | we can skip I guess, I haven't seen too much movement on bugs last week | 15:21 |
| rpittau | any volunteer as bug deputy for this week? :) | 15:22 |
| cid | Am I eligible? | 15:22 |
| rpittau | that's a good question :) | 15:23 |
| rpittau | we never talked about non-core being bug deputies | 15:23 |
| dtantsur | We'd need to add you to some lp group | 15:23 |
| rpittau | the problem is that you won't have the rights on lp | 15:23 |
| rpittau | I can do it this week and we can discuss it | 15:23 |
| dtantsur | Other than that, I personally don't see a problem with any permanent contributor taking this role | 15:23 |
| rpittau | me neither | 15:23 |
| cid | ++ | 15:24 |
| rpittau | cid: will you be able to do it next week ? | 15:24 |
| cid | I would love to | 15:24 |
| rpittau | great! | 15:25 |
| rpittau | I'll sign you up for next week then so we can see if we need to sort things out for lp | 15:25 |
| cid | Noted | 15:25 |
| rpittau | cid: thanks :) | 15:25 |
| cid | No probs =) | 15:25 |
| rpittau | I don't see new RFEs to talk about so I think we can go to | 15:26 |
| rpittau | #topic Open Discussion | 15:26 |
| rpittau | anyone has anything would like to talk about? :) | 15:26 |
| rpittau | (ironic related... mostly) | 15:26 |
| rpittau | looks like we're good :) | 15:27 |
| rpittau | thanks everyone! | 15:27 |
| rpittau | #endmeeting | 15:27 |
| opendevmeet | Meeting ended Mon Jul 1 15:27:45 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:27 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-07-01-15.00.html | 15:27 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-07-01-15.00.txt | 15:27 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-07-01-15.00.log.html | 15:27 |
| rpittau | good night! o/ | 15:53 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-inspector master: Deprecate the project https://review.opendev.org/c/openstack/ironic-inspector/+/923170 | 16:29 |
| dtantsur | boooooom! | 16:29 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Inspector migration: mention switching over to ironicclient https://review.opendev.org/c/openstack/ironic/+/923175 | 16:36 |
| opendevreview | Dmitry Tantsur proposed openstack/python-ironic-inspector-client master: Deprecate the project https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/923176 | 16:42 |
| opendevreview | Merged openstack/python-ironicclient master: Client support port name https://review.opendev.org/c/openstack/python-ironicclient/+/896067 | 16:43 |
| cid | 🫡 | 16:43 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add tox env for local dev; update documentation https://review.opendev.org/c/openstack/ironic/+/922348 | 18:28 |
| JayF | I'm sorry, I took today off impromptu and forgot that I owed a bug deputy report. I did triage all bugs that were filed recent ish that showed up in the dashboard. | 19:22 |
| JayF | I even fixed one that had been reported where we were still linking to our storyboard on opendev.org/openstack/$repo | 19:22 |
| JayF | Nothing too much of note. If so I would have brought it up in channel 😄 | 19:23 |
| JayF | cid: with the permission of rpittau I should be able to grant the permissions in Launchpad. We can take care of this tomorrow. | 19:23 |
| cid | ++ | 19:24 |
| cid | o/ | 20:28 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!