| opendevreview | Merged openstack/ironic master: Documentation for graphical consoles https://review.opendev.org/c/openstack/ironic/+/943122 | 00:08 |
|---|---|---|
| opendevreview | JackyChang proposed openstack/ironic-tempest-plugin master: Replaced all punctuation with an underscore in the node resource class. https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/943526 | 05:23 |
| opendevreview | Vasyl Saienko proposed openstack/ironic master: Do not review debug multinode https://review.opendev.org/c/openstack/ironic/+/942725 | 06:30 |
| *** dmellado0755393736 is now known as dmellado075539373 | 06:43 | |
| *** diablo_rojo_phone is now known as Guest10755 | 08:01 | |
| rpittau | good morning ironic! o/ | 08:04 |
| vsaienko | hello Ironic community, I may be wrong but looks like redfish whiledisk deployment is broken https://zuul.opendev.org/t/openstack/builds?job_name=ironic-standalone-redfish&skip=0 jobs constantly failing with https://paste.openstack.org/show/bAUF91sXh8P1gEq79fRI/ | 08:10 |
| opendevreview | Vasyl Saienko proposed openstack/ironic-tempest-plugin master: Mark BaremetalDriverAnsibleWholedisk as unstable https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/943529 | 08:18 |
| opendevreview | Vasyl Saienko proposed openstack/ironic-tempest-plugin master: Mark BaremetalDriverAnsibleWholedisk as unstable https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/943529 | 08:19 |
| frickler | hmm, I do not yet see how, but it looks like the error might have been introduced by https://review.opendev.org/c/openstack/ironic/+/941614 , at least that's where I first see that failure | 08:47 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-lib stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/ironic-lib/+/943531 | 08:55 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-lib stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/ironic-lib/+/943532 | 08:55 |
| opendevreview | OpenStack Release Bot proposed openstack/metalsmith stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/metalsmith/+/943535 | 08:55 |
| opendevreview | OpenStack Release Bot proposed openstack/metalsmith stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/metalsmith/+/943537 | 08:55 |
| opendevreview | OpenStack Release Bot proposed openstack/metalsmith master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/metalsmith/+/943538 | 08:55 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/943539 | 08:55 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/943540 | 08:55 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironic-inspector-client master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/943541 | 08:55 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironicclient stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/python-ironicclient/+/943542 | 08:55 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironicclient stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/python-ironicclient/+/943543 | 08:55 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironicclient master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/python-ironicclient/+/943544 | 08:55 |
| opendevreview | OpenStack Release Bot proposed openstack/sushy stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/sushy/+/943545 | 08:55 |
| opendevreview | OpenStack Release Bot proposed openstack/sushy stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/sushy/+/943546 | 08:56 |
| opendevreview | OpenStack Release Bot proposed openstack/sushy master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/sushy/+/943547 | 08:56 |
| opendevreview | Merged openstack/metalsmith master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/metalsmith/+/943538 | 09:21 |
| opendevreview | Merged openstack/ironic-lib stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/ironic-lib/+/943531 | 09:23 |
| opendevreview | Merged openstack/ironic-lib stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/ironic-lib/+/943532 | 09:23 |
| opendevreview | Merged openstack/metalsmith stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/metalsmith/+/943535 | 09:27 |
| opendevreview | Merged openstack/metalsmith stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/metalsmith/+/943537 | 09:27 |
| opendevreview | Merged openstack/sushy master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/sushy/+/943547 | 09:29 |
| opendevreview | Verification of a change to openstack/ironic-python-agent stable/2024.2 failed: Fix the way qemu-img is called with prlimits https://review.opendev.org/c/openstack/ironic-python-agent/+/943382 | 09:29 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Use bigger partition as work dir for metal3 job https://review.opendev.org/c/openstack/ironic/+/943374 | 09:35 |
| opendevreview | Merged openstack/python-ironicclient master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/python-ironicclient/+/943544 | 09:36 |
| opendevreview | Merged openstack/python-ironic-inspector-client master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/943541 | 09:37 |
| opendevreview | cid proposed openstack/ironic master: Install and run sushy-tools within a virtualenv https://review.opendev.org/c/openstack/ironic/+/943580 | 09:40 |
| cid | dtantsur, if you ever have a free cycle anytime within the week, this patch is awaiting reviews (yours specifically :D): https://review.opendev.org/c/openstack/ironic/+/939217 | 09:51 |
| cid | masghar, JayF, thanks for the reviews, I have tried addressing them. | 09:51 |
| cid | rpittau, cardoe, TheJulia, reviews welcome ^^ (I know some of us are out of office, bad timing, but hey). | 09:51 |
| opendevreview | Merged openstack/sushy stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/sushy/+/943545 | 09:53 |
| opendevreview | Merged openstack/sushy stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/sushy/+/943546 | 09:53 |
| rpittau | cid: I guess this https://review.opendev.org/c/openstack/ironic/+/942112 needs to merge first | 10:19 |
| cid | rpittau: Yes | 10:28 |
| rpittau | it looks like ironicclient tests are broken :/ | 10:42 |
| dtantsur | cid: yep, I should have time today, thank you! | 10:46 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/2024.2: More reliable TinyIPA builds with network retries https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/943380 | 10:50 |
| cid | Thanks | 10:51 |
| rpittau | I've proposed a release of ipa-builder considering all the most recent changes and that the latest one was in august 2024 https://review.opendev.org/c/openstack/releases/+/943393 | 11:05 |
| opendevreview | Riccardo Pittau proposed openstack/python-ironicclient master: Fix unit tests in test_baremetal_node https://review.opendev.org/c/openstack/python-ironicclient/+/943584 | 11:09 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/2024.1: More reliable TinyIPA builds with network retries https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/943381 | 11:09 |
| opendevreview | Riccardo Pittau proposed openstack/python-ironicclient master: Fix bare metal info order in unit tests https://review.opendev.org/c/openstack/python-ironicclient/+/943584 | 11:11 |
| opendevreview | Riccardo Pittau proposed openstack/python-ironicclient master: Fix bare metal info order in unit tests https://review.opendev.org/c/openstack/python-ironicclient/+/943584 | 11:19 |
| opendevreview | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Fix bare metal info order in unit tests https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/943588 | 11:33 |
| SvenKieske | o/ anybody happen to know if TheJulia is around today? | 11:43 |
| SvenKieske | or anyone else willing to answer complicated questions about the interaction of ironic(-conductor), metal3-operator, openstack-baremetal-operator and openshift-machine-api? :D | 11:46 |
| rpittau | SvenKieske: this is the good channel for that, but even better would be asking in Slack in the k8s group and #cluster-api-baremetal channel :) | 12:02 |
| rpittau | you can also check here the metal3 user guide for info https://book.metal3.io/introduction | 12:02 |
| rpittau | SvenKieske: btw openshift-machine-api is not part of metal3 though | 12:03 |
| SvenKieske | rpittau: yeah thanks, already reached out via slack as well. this is a red hat openstack on openshift problem, pretty bleeding edge, somehow, it seems :) | 12:04 |
| rpittau | SvenKieske: unfortunately I don't think we can provide that kind of support here, any issue with Red Hat products I believe needs to pass through the Red Hat Customer Portal | 12:09 |
| dtantsur | We can provide some general information. Many of us know about Metal3, but for RH OSO specific questions you need to catch TheJulia or hjensas (the latter probably online now) | 12:11 |
| SvenKieske | yeah sorry, I just used the channel as a hack to get some attention, maybe. will try to catch hjensas then via internal channels, thx! | 12:19 |
| * TheJulia tries to will self to hotel coffee maker | 12:45 | |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [CI] metal3 integration: reduce total time between node check https://review.opendev.org/c/openstack/ironic/+/943595 | 13:42 |
| opendevreview | Kaifeng Wang proposed openstack/ironic master: Ironic doesn't update local_link_connection when changed https://review.opendev.org/c/openstack/ironic/+/943395 | 14:07 |
| opendevreview | Jakub Darmach proposed openstack/bifrost stable/2024.1: pip: Use SETUPTOOLS_USE_STDLIB if python < 3.12 https://review.opendev.org/c/openstack/bifrost/+/943180 | 14:34 |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent-builder master: WIP Test Podman for container-based cleaning https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/943348 | 14:37 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [CI] metal3 integration: reduce total time between node check https://review.opendev.org/c/openstack/ironic/+/943595 | 16:33 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] Run metal3 integration job using UEFI boot (default) https://review.opendev.org/c/openstack/ironic/+/939694 | 16:33 |
| vsaienko | team please merge https://review.opendev.org/c/openstack/ironic/+/942340 - enables atop for devstack jobs should provide more info on what is happening on env during tests | 16:48 |
| opendevreview | Vasyl Saienko proposed openstack/ironic-tempest-plugin master: Add retries while waiting for SSH on server https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/942009 | 16:50 |
| opendevreview | Vasyl Saienko proposed openstack/ironic-tempest-plugin master: Add tests to validate trunks https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/941013 | 16:50 |
| *** awb_ is now known as awb | 18:02 | |
| opendevreview | cid proposed openstack/ironic master: Follow-up: Apply Inspection Rules https://review.opendev.org/c/openstack/ironic/+/942112 | 19:20 |
| opendevreview | cid proposed openstack/ironic master: API/Testing: Inspection rules migration https://review.opendev.org/c/openstack/ironic/+/939217 | 19:20 |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent master: WIP Add ContainerHardwareManager https://review.opendev.org/c/openstack/ironic-python-agent/+/941714 | 19:32 |
| TheJulia | stevebaker[m]: any thought on rbac role enabling write access for consoles? Read only setting being the big giant breaker/switch | 21:18 |
| stevebaker[m] | TheJulia: I was thinking that if [vnc]read_only=False then there should be mechanisms for individual nodes to be read only for lesees, controlled by owner/system. But yeah we should really add some default rules also. 'lessee' read-only by default? should system scoped reader have access at all? | 21:32 |
| TheJulia | Likely, yeah | 21:37 |
| TheJulia | Hadn’t thought about it beyond talking to JayF briefly | 21:37 |
| rm_work[m] | anyone know offhand if these sync up? `python-openstackclient>=7.4.0,<8.0.0` and `python-ironicclient>=5.10.0,<6.0.0` | 21:49 |
| stevebaker[m] | TheJulia: ok I'll do something and we can work out the details in the review. Regarding per-node read-only: should an owner/system-scoped specifically set a node property to force all users to be read-only for that node? | 21:50 |
| stevebaker[m] | in the default policy a lessee can't even enable/get a console, hmm | 22:09 |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent master: WIP Add ContainerHardwareManager https://review.opendev.org/c/openstack/ironic-python-agent/+/941714 | 22:16 |
| JayF | rm_work[m]: upper-constaints.txt in the requirements repo is probably the only place to get a "real" answer to that question beyond just seeing if it works | 22:32 |
| JayF | I stopped by and talked to one of the folks in the nix booth; they might already have some nix-adjacent tooling to do the ideal for IPA ramdisk (e.g. just take the specific files we need old-school chroot-building style) | 22:34 |
| * JayF will never give up on putting tinyipa out to pasture, the answer is somewhere | 22:35 | |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent-builder master: WIP Add Podman DIB for container-based cleaning https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/943348 | 23:08 |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent-builder master: WIP Add Podman DIB for container-based cleaning https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/943348 | 23:22 |
| TheJulia | stevebaker[m]: dunno, but specifically the configuration file should be the primary setting. I suspect making it setting per node might not be good. Owners admin/member seems to make sense for write, definitely as you’ve suggested lessee should by default read only. | 23:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!