freemanboss[m] | <Richard-Akin> "masghar: yeah, I haven't been..." <- I think you're missing things out that's why. | 05:25 |
---|---|---|
freemanboss[m] | You can either choose to run ironic using bifrost in a test mode or prod mode (and this is when you've your physical server that you want to run it on). For now we are advised to run it in a test mode to understand how it works better and experiment with it | 05:25 |
Amarachi_O | Good morning Ironic! | 06:08 |
freemanboss[m] | Good morning 🌄 | 06:43 |
opendevreview | Merged openstack/ironic stable/2025.1: [Trivial] Fix typo of exception error message https://review.opendev.org/c/openstack/ironic/+/945740 | 06:55 |
rpittau | good morning ironic! happy friday! o/ | 08:26 |
opendevreview | Riccardo Pittau proposed openstack/ironic stable/2025.1: [2025.1 only] update devstack config https://review.opendev.org/c/openstack/ironic/+/945793 | 08:59 |
ayo_ | Good morning rpittau :) | 09:01 |
vsaienko | TheJulia, JayF please add to your review queue https://review.opendev.org/c/openstack/networking-generic-switch/+/945708 - adds a new job to test portgroups, and dependent patches https://review.opendev.org/c/openstack/ironic/+/940611 allows to setup environment with PG and https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/940678 adds validation for network data related to PG | 09:01 |
opendevreview | Harald JensĂĄs proposed openstack/sushy-tools master: os-vmedia: Add option to delay rebuild on eject https://review.opendev.org/c/openstack/sushy-tools/+/945800 | 10:06 |
opendevreview | Merged openstack/ironic stable/2024.1: Make floppy images more floppy https://review.opendev.org/c/openstack/ironic/+/945046 | 10:09 |
dtantsur | TheJulia, JayF, let's imagine I find someone to work on standalone switch management. Do you think we can give them enough space and provide enough guidance to make it happen in the near future? | 11:27 |
dtantsur | I'm trying to stay honest with expecations | 11:27 |
opendevreview | Doug Goldstein proposed openstack/networking-baremetal master: fix spelling mistakes https://review.opendev.org/c/openstack/networking-baremetal/+/945818 | 12:22 |
opendevreview | Doug Goldstein proposed openstack/networking-baremetal master: fix awkward logic that the linter didn't like https://review.opendev.org/c/openstack/networking-baremetal/+/945819 | 12:22 |
opendevreview | Doug Goldstein proposed openstack/networking-baremetal master: fix sphinx-lint issues https://review.opendev.org/c/openstack/networking-baremetal/+/945820 | 12:22 |
opendevreview | Doug Goldstein proposed openstack/networking-baremetal master: add pre-commit and adjust tox to utilize it https://review.opendev.org/c/openstack/networking-baremetal/+/945821 | 12:22 |
freemanboss[m] | <rpittau> "good morning ironic! happy..." <- Happy Friday | 12:43 |
JayF | dtantsur: I will be taking a 6-week sabbatical in the middle of this cycle. So I am keeping my commitments the cycle to a minimum | 13:09 |
JayF | Tbh, even if not that I'm not sure I would be the best person to help someone down that path anyway | 13:09 |
dtantsur | noted, sounds fun :) | 13:09 |
dtantsur | Yeah, I'm looking less for personal commitments more for your takes on whether our community can absorb that | 13:09 |
TheJulia | dtantsur: I'm semi-hoping we can frame things sufficient I might be able to spend time on it, but dunno. I don't know about anything at this point given the broad uncertinty | 13:19 |
dtantsur | right | 13:23 |
TheJulia | Well, I do know immediate demands, and the only way I ensure appropriate tie-in is to be involved upfront | 13:25 |
TheJulia | anyway, first meeting of the day in a few minutes, coffeeeee to finish | 13:25 |
opendevreview | Stephen Finucane proposed openstack/ironic master: conf: Add '[api] response_validation' option https://review.opendev.org/c/openstack/ironic/+/945515 | 13:57 |
opendevreview | Stephen Finucane proposed openstack/ironic master: api: Add schema for allocations API (responses) https://review.opendev.org/c/openstack/ironic/+/928921 | 13:57 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Do not silence the actual error in prepare_instance_boot https://review.opendev.org/c/openstack/ironic/+/945844 | 14:52 |
dtantsur | did we have any reasons not to ^^^? | 14:52 |
dtantsur | we've just wasted plenty of time because of it | 14:52 |
rpittau | I can't think of any | 15:04 |
Uggla | Hello ironic team, I have not seen any common topic between nova-ironic for the PTG. So I guess no cross team session is needed, but I'd like to confirm. | 15:05 |
rpittau | Uggla: hi! o/ we've discussed about topics during the last weeks and so far we have nothing with nova, I guess we'll ahve another discussion on Monday during the meeting to finalize the list, and then provide a better answer | 15:11 |
JayF | We're working on some network enhancement specs which may need some changes in the ironic nova virt driver; but I doubt we'll have details in time for a PTG cross-team | 15:12 |
JayF | probably something for next PTG if it's beefy enough for a PTG session | 15:12 |
Uggla | rpittau, good. Just let me know if you need one. Same on us, the list of topics is not closed yet so I'll warn here if needed. | 15:13 |
rpittau | perfect, thanks Uggla | 15:14 |
Uggla | rpittau, you're welcome. | 15:14 |
TheJulia | JayF: I glanced at the nova.virt.ironic driver code and pattern, the existing pattern matches what we would expect with the current idea on the table | 15:33 |
TheJulia | (so as long as nova posts the flavor or requested flavor traits to the driver, we're good | 15:33 |
TheJulia | ) | 15:33 |
TheJulia | (I didn't double check that, since I think you indicated it did happen now) | 15:36 |
cardoe | Uggla: so I've got some items around improving the nova-ironic interaction. | 15:53 |
cardoe | The error reporting is sub-optimal and that's what I think we could (and should) improve on. | 15:53 |
cardoe | Uggla: so like first patch I've got is... https://review.opendev.org/c/openstack/nova/+/942019 which needs to be backported now to 2025.1, 2024.2, 2024.1 | 15:54 |
cardoe | I forget the bug off hand but nearly all build errors end up wrapped up in a block device error message even if it was a neutron network issue. So you've got to read the call stack to figure it out. But last error won't show that. | 15:57 |
cardoe | There's some in flight stuff to expose trunk ports in the metadata as well. | 15:58 |
Uggla | cardoe, ok but I guess it does not required a cross team session. Am i wrong ? | 16:11 |
cardoe | No cross session necessary. I just figured I'd highlight there's some items around error handling / reporting that's rough right now. | 16:39 |
JayF | TheJulia: I think the question was around ordering, and if we would need to change ordering around in how nova handles e.g. configdrive generation (or, as you proposed, move most of that code into ironic) | 16:47 |
JayF | TheJulia: So I don't think we'll need surgery in the nova ironic driver, but we might need a patch or three | 16:47 |
opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent master: WIP Implement manual cleaning for ContainerHardwareManager https://review.opendev.org/c/openstack/ironic-python-agent/+/945862 | 17:03 |
ayo_ | freemanboss[m]: after deploying, my state has been waiting for a call back for quite a while, I want to know if yours did the same | 17:16 |
freemanboss[m] | ayo_: Call back? I don't understand | 17:17 |
ayo_ | After enrolling and deploying | 17:17 |
JayF | provision_state: wait call-back | 17:17 |
ayo_ | I believe it’s supposed to show successful if the deployment worked | 17:18 |
ayo_ | My issue is my provisioning state has been on wait call-back for a while | 17:19 |
ayo_ | I just want to know if it normally takes a while before it works | 17:20 |
ayo_ | It’s been 10 minutes now | 17:20 |
ayo_ | Oops | 17:21 |
ayo_ | Never mind | 17:21 |
ayo_ | Just showed active, thank you | 17:21 |
ayo_ | I’m still a bit confused on the enroll and deploy line | 17:23 |
ayo_ | JayF: if you don’t mind me asking | 17:23 |
freemanboss[m] | <ayo_> "My issue is my provisioning..." <- Just wait to see the final results. | 17:24 |
freemanboss[m] | Immediately after the deployment is run it will be on deploying and I didn't check back on the end status | 17:24 |
freemanboss[m] | Since you mentioned it now I'll have to take note on the end status. But if it's on anything wait just wait a while if it'll change | 17:24 |
freemanboss[m] | ayo_: Great you're welcome! | 17:25 |
freemanboss[m] | ayo_: What's the confusion? You can ask if I may be of help | 17:25 |
freemanboss[m] | * Just wait to see the final results. | 17:26 |
freemanboss[m] | Immediately after the deployment is run it will be on deploying and I didn't check back on the end status | 17:26 |
freemanboss[m] | Since you mentioned it now I'll have to take note on the end status. But if it's on anything wait just wait a while it'll change | 17:26 |
ayo_ | I understand using the “enroll” alongside the .json package problvides detail regarding a bare metal server | 17:26 |
freemanboss[m] | ayo_: Yes? | 17:27 |
ayo_ | The deploy stage attaches OS on the enrolled server | 17:27 |
ayo_ | I still don’t understand where the details on the OS is predefined | 17:28 |
ayo_ | freemanboss[m]: ? | 17:30 |
freemanboss[m] | <ayo_> "I still don’t understand where..." <- To understand things better one has to go deep into the code and it's a matter of time. My PC is off I'd have like to explore more so that I can explain this better. But so far actually I've what I've observed too | 17:33 |
freemanboss[m] | Since it's a test mode it provisioned 2 servers (testvm1 and testvm2) from the actual server you're running Bifrost/ironic on. It then made ssh connections to the 2 provisioned servers and installed the OS which I think it's Ubuntu therein. | 17:36 |
freemanboss[m] | <ayo_> "I still don’t understand where..." <- I think it's defined in the inventory file. Can you send it here? | 17:36 |
ayo_ | Inventory.json file? | 17:43 |
opendevreview | Julia Kreger proposed openstack/ironic master: devstack: network simulator support for sonic https://review.opendev.org/c/openstack/ironic/+/945726 | 18:00 |
freemanboss[m] | <ayo_> "Inventory.json file?" <- Yes | 18:33 |
freemanboss[m] | Or nodes.py | 18:33 |
freemanboss[m] | Nodes json | 18:33 |
opendevreview | Merged openstack/networking-generic-switch master: docs: add additional context around Cisco nxos usage https://review.opendev.org/c/openstack/networking-generic-switch/+/945145 | 18:42 |
opendevreview | Julia Kreger proposed openstack/ironic master: devstack: network simulator support for sonic https://review.opendev.org/c/openstack/ironic/+/945726 | 19:01 |
opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Adding notes for SONiC switches https://review.opendev.org/c/openstack/networking-generic-switch/+/945887 | 19:01 |
TheJulia | JayF: metadata generation is done after vif attachments get sent from what I saw, which is also done after instance_info is supposed to be populated, so as long as we wire things together that vif attachment handling logic triggers dynamic portgroup assembly, then metadata and config drive should all be good | 19:31 |
cardoe | Yeah it’s definitely done after vif attachment cause that’s how MAC addresses are correct. We should update the networking doc. | 19:41 |
opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent master: WIP Implement manual cleaning for ContainerHardwareManager https://review.opendev.org/c/openstack/ironic-python-agent/+/945862 | 19:43 |
TheJulia | cardoe: I'm not sure why you note update our networking doc, is there a doc which is wrong, the thread of discussion was related to dynamic grouping | 20:31 |
cardoe | I was saying if there’s confusion. | 20:32 |
cardoe | Sorry. Didn’t come across correctly. | 20:33 |
TheJulia | no worries | 20:40 |
kulsooms | freebossman, ps_adavize, queensly: Hi everyone, if any of you have come across and resolved this error, please let me know https://imgur.com/a/lOzDPgw | 22:10 |
opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent master: WIP Implement manual cleaning for ContainerHardwareManager https://review.opendev.org/c/openstack/ironic-python-agent/+/945862 | 22:40 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!