| opendevreview | Merged openstack/networking-generic-switch master: Add vlan aware VMs support https://review.opendev.org/c/openstack/networking-generic-switch/+/928490 | 01:01 |
|---|---|---|
| opendevreview | Ghanshyam proposed openstack/ironic master: Testing doc job on Ubuntu Noble https://review.opendev.org/c/openstack/ironic/+/944006 | 05:31 |
| opendevreview | Dr. Jens Harbott proposed openstack/ironic master: Advanced vmedia deployment test ops https://review.opendev.org/c/openstack/ironic/+/898010 | 07:56 |
| opendevreview | Dr. Jens Harbott proposed openstack/ironic master: Advanced vmedia deployment test ops https://review.opendev.org/c/openstack/ironic/+/898010 | 08:00 |
| rpittau | good morning ironic! o/ | 08:06 |
| frickler | fyi I've seen this apache startup failure multiple times now, never seen this in "normal" devstack, the only possible explanation I have would be a timing issue because this job disables so many of the stock devstack services. will hold a node to debug once it repeats https://zuul.opendev.org/t/openstack/build/84d05ce0c0534b10b507fd528ca728c6 | 08:34 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Adds vlan aware VMs support for Cumulus NVUE and DellOS10 https://review.opendev.org/c/openstack/networking-generic-switch/+/940966 | 08:34 |
| *** jroll02 is now known as jroll0 | 08:45 | |
| opendevreview | Dr. Jens Harbott proposed openstack/ironic master: Advanced vmedia deployment test ops https://review.opendev.org/c/openstack/ironic/+/898010 | 08:47 |
| opendevreview | cid proposed openstack/ironic master: Remove IPA build on cleanup https://review.opendev.org/c/openstack/ironic/+/943987 | 08:53 |
| opendevreview | Merged openstack/ironic-python-agent master: Doc: Fix incorrect function in example code https://review.opendev.org/c/openstack/ironic-python-agent/+/943771 | 10:23 |
| mnasiadka | TheJulia: reached Europe, can do something today/tomorrow if needed :) | 11:12 |
| opendevreview | Verification of a change to openstack/ironic master failed: fix glance metadata layout https://review.opendev.org/c/openstack/ironic/+/942496 | 11:18 |
| opendevreview | Verification of a change to openstack/ironic master failed: vncproxy: Use dedicated options for SSL https://review.opendev.org/c/openstack/ironic/+/943678 | 12:46 |
| opendevreview | Verification of a change to openstack/ironic master failed: vncproxy: Use dedicated options for SSL https://review.opendev.org/c/openstack/ironic/+/943678 | 13:10 |
| opendevreview | Verification of a change to openstack/ironic master failed: fix glance metadata layout https://review.opendev.org/c/openstack/ironic/+/942496 | 13:18 |
| opendevreview | Dr. Jens Harbott proposed openstack/ironic master: Advanced vmedia deployment test ops https://review.opendev.org/c/openstack/ironic/+/898010 | 13:19 |
| rpittau | it works! \o/ https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_7a9/943374/7/check/metal3-integration/7a98729/controller/system/df--h.txt | 13:19 |
| rpittau | sorry :D | 13:19 |
| rpittau | time for clean things up | 13:20 |
| TheJulia | good morning | 13:25 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix libvirt logs collection in metal3 job https://review.opendev.org/c/openstack/ironic/+/944039 | 13:29 |
| rpittau | good morning TheJulia :) | 13: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 | 13:31 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [CI] Use bigger partition as work dir for metal3 job https://review.opendev.org/c/openstack/ironic/+/943374 | 13:32 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [CI] Fix libvirt logs collection in metal3 job https://review.opendev.org/c/openstack/ironic/+/944039 | 13:32 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [CI] Fix libvirt logs collection in metal3 job https://review.opendev.org/c/openstack/ironic/+/944039 | 13:34 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [CI] Use bigger partition as work dir for metal3 job https://review.opendev.org/c/openstack/ironic/+/943374 | 13:34 |
| rpittau | ooook now all ready for reviews | 13:34 |
| TheJulia | good morning | 13:39 |
| TheJulia | again :) | 13:39 |
| dtantsur | good second morning TheJulia | 13:56 |
| dtantsur | is second morning something you do to justify a second breakfast? | 13:56 |
| frickler | am I missing something or are there no docs for "Runbooks must be created and associated with nodes beforehand." https://docs.openstack.org/ironic/latest/admin/cleaning.html#runbooks-for-manual-cleaning | 14:07 |
| JayF | > Runbooks are curated lists of steps that can be associated with nodes via traits | 14:16 |
| JayF | I think we have a separate runbooks page that goes into that detail | 14:16 |
| JayF | but traits are the key | 14:16 |
| JayF | https://opendev.org/openstack/ironic/commit/48f50248c25229522a98da80ac3aa0580b5b809c looks like there is no more detailed docs | 14:18 |
| JayF | yeah we need better examples for this | 14:20 |
| JayF | cid: ^ not urgent, but we should probably ensure we're more clear in the docs for runbooks about this | 14:20 |
| JayF | frickler: cid wrote that and should be able to help you get it moving, that may help inform how we can improve the docs | 14:21 |
| frickler | ty | 14:22 |
| JayF | yeah I think the algo is as simple as | 14:23 |
| JayF | the runbook name must be a node trait | 14:23 |
| TheJulia | dtantsur: just still waking up and doing a re-review of documents before an important meeting in about 40 minutes | 14:23 |
| JayF | idea being as an admin, you can set that up and let a member only run the things you want | 14:23 |
| TheJulia | so... lots of coffeeeeeee | 14:23 |
| JayF | and also you never want e.g. a dell firmware update runbook to be attempted on an hp | 14:23 |
| TheJulia | It occurs to me, we *likely* ought to consider recording the hardware vendor as a trait as well... although that gets us into that not so fun area about how there was a lack of agreement over traits and meaning of traits/resource classes | 14:25 |
| TheJulia | "The agreement to not agree" | 14:25 |
| JayF | I think with inspection rules, we give users all the tools to map how they want, but I don't feel strongly one way or the other | 14:26 |
| cid | JayF, frickler, Yeah, I will update the docs to make those very clear. | 14:31 |
| dtantsur | JayF: I think https://review.opendev.org/c/openstack/ironic/+/942112 is ready for approval | 14:39 |
| JayF | 🛬 | 14:47 |
| opendevreview | Merged openstack/networking-generic-switch master: Adds vlan aware VMs support for Cumulus NVUE and DellOS10 https://review.opendev.org/c/openstack/networking-generic-switch/+/940966 | 15:14 |
| opendevreview | Merged openstack/ironic master: [CI] metal3 integration: reduce total time between node check https://review.opendev.org/c/openstack/ironic/+/943595 | 15:20 |
| opendevreview | Takashi Kajinami proposed openstack/ironic-inspector master: Replace deprecated configure_auth_token_middleware https://review.opendev.org/c/openstack/ironic-inspector/+/935240 | 15:20 |
| opendevreview | Verification of a change to openstack/ironic master failed: Follow-up: Apply Inspection Rules https://review.opendev.org/c/openstack/ironic/+/942112 | 15:49 |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic master: Add ignore_project_check_for_admin_tasks config option https://review.opendev.org/c/openstack/ironic/+/943028 | 16:01 |
| dtantsur | JayF: remind me please, by which date should we land the API? | 16:24 |
| dtantsur | (to get it into this release, I mean) | 16:24 |
| opendevreview | Verification of a change to openstack/ironic master failed: fix glance metadata layout https://review.opendev.org/c/openstack/ironic/+/942496 | 16:24 |
| dtantsur | hmm, a couple POST_FAILUREs in a row | 16:25 |
| dtantsur | Failed to fetch https://mirror.gra1.ovh.opendev.org/ubuntu/dists/noble-updates/main/binary-amd64/Packages.gz File has unexpected size (1215823 != 1220439). Mirror sync in progress? | 16:25 |
| dtantsur | Let's hope that a mirror sync is in progress indeed | 16:25 |
| *** awb_ is now known as awb | 16:30 | |
| opendevreview | Verification of a change to openstack/ironic master failed: Follow-up: Apply Inspection Rules https://review.opendev.org/c/openstack/ironic/+/942112 | 16:39 |
| JayF | dtantsur: I'm not sure tbh; I think in the next week or two? final intermediary releases are 3/24-3/28, but I don't think we should wait that long | 16:39 |
| rpittau | probably early next week | 16:40 |
| rpittau | to have the time of the release | 16:40 |
| rpittau | we're at R-3 | 16:40 |
| TheJulia | We know from experience, we basically have to release R-3 unless the world is on fire. | 16:44 |
| TheJulia | Because branch changes get started around then as well. | 16:44 |
| TheJulia | So we can't think of "next week" as anything more then "before end of this week" | 16:45 |
| opendevreview | Merged openstack/ironic master: vncproxy: Use dedicated options for SSL https://review.opendev.org/c/openstack/ironic/+/943678 | 16:47 |
| priteau | Should https://review.opendev.org/c/openstack/networking-generic-switch/+/943061 be targeted for merge before Epoxy is branched? | 16:54 |
| priteau | I see it had two +2 votes recently, but needs merge conflict fix | 16:54 |
| JayF | it's in merge conflict, but it likely should be | 16:55 |
| JayF | it's backportable if not though | 16:55 |
| JayF | I'll rebase it after I'm at the end of another task if cid doesn't get to it first | 16:55 |
| priteau | I am happy to fix it now | 16:56 |
| JayF | please do :) | 16:56 |
| JayF | I can preserve my +2 if so | 16:56 |
| cid | pritteau, JayF, also, that patch is a bit different from the version that got approval votes, so may need review | 16:57 |
| * cid priteau :) | 16:57 | |
| opendevreview | Pierre Riteau proposed openstack/networking-generic-switch master: Fix info leakage in Netmiko connection errors https://review.opendev.org/c/openstack/networking-generic-switch/+/943061 | 16:57 |
| priteau | The merge conflict was trivial | 16:59 |
| TheJulia | okay, I'm finally done with meetings... sort of | 18:28 |
| opendevreview | Verification of a change to openstack/ironic master failed: fix glance metadata layout https://review.opendev.org/c/openstack/ironic/+/942496 | 18:52 |
| opendevreview | Merged openstack/ironic master: Follow-up: Apply Inspection Rules https://review.opendev.org/c/openstack/ironic/+/942112 | 19:21 |
| TheJulia | \o/ | 19:22 |
| JayF | we still need the api patch 🤞 | 19:24 |
| TheJulia | https://review.opendev.org/c/openstack/ironic/+/939217 ? | 19:26 |
| JayF | aye | 19:28 |
| TheJulia | Looks like the last one failed, do we know if this is random failure or if there is a specific issue which needs eyes? | 19:29 |
| JayF | it's the apache2 thing frickler mentioned earlier this morning, I believe | 19:29 |
| frickler | ah, I should have a held node for that now, then. anyone wants to have access and take a look? | 19:32 |
| JayF | I am still working thru my post-SCALE todo list, including reviews. | 19:33 |
| frickler | https://paste.opendev.org/show/bnFKEZUelv6azPRmiNIN/ | 19:37 |
| frickler | simply starting the service again works, maybe we should just add a retry in devstack? | 19:40 |
| TheJulia | frickler:... that seems like a default change in that there is suddenly no hostname?! | 19:40 |
| TheJulia | #confused | 19:40 |
| TheJulia | I feel like I'm this kernel: https://forums.fedoraforum.org/showthread.php?323033-kernel-Dazed-and | 19:41 |
| frickler | TheJulia: the message about ServerName is non fatal, it was just too many restarts too fast in a row for systemd or at least that's my understanding | 19:44 |
| TheJulia | frickler: eek :( | 20:04 |
| TheJulia | frickler: It *feels* like I've seen it fail in a failure to resolve a name if not configured though. Then again my httpd context is spread over many many many years | 20:04 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Add tests to validate trunks https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/941013 | 20:07 |
| frickler | TheJulia: simply setting ServerName in devstack might be a good option, too. will take a closer look tomorrow | 20:13 |
| TheJulia | at least ages ago, it would try to lookup the IP it chose and if it couldn't that could sometimes actually be fatal | 20:14 |
| TheJulia | but... I'm thinking of times ages ago | 20:15 |
| opendevreview | Merged openstack/ironic master: [CI] Fix libvirt logs collection in metal3 job https://review.opendev.org/c/openstack/ironic/+/944039 | 20:40 |
| opendevreview | Verification of a change to openstack/ironic master failed: [CI] Use bigger partition as work dir for metal3 job https://review.opendev.org/c/openstack/ironic/+/943374 | 20:40 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add servicing to stuck states on recovery https://review.opendev.org/c/openstack/ironic/+/941017 | 21:49 |
| JayF | ^ I fixed the minor lint issue on that and voted +2, would be a good one to get in + likely backport | 21:50 |
| opendevreview | Merged 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 | 22:01 |
| opendevreview | Steve Baker proposed openstack/ironic master: DNM Don't install graphical console deps with bindep https://review.opendev.org/c/openstack/ironic/+/943868 | 22:37 |
| opendevreview | Steve Baker proposed openstack/ironic master: DMN disable ir-novnc in ironic-standalone-redfish https://review.opendev.org/c/openstack/ironic/+/943871 | 22:37 |
| opendevreview | Steve Baker proposed openstack/ironic master: DNM install podman https://review.opendev.org/c/openstack/ironic/+/944078 | 22:37 |
| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: DNM Revert "Mark BaremetalDriverAnsibleWholedisk as unstable" https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/944079 | 22:46 |
| opendevreview | Steve Baker proposed openstack/ironic master: DNM Don't install graphical console deps with bindep https://review.opendev.org/c/openstack/ironic/+/943868 | 22:47 |
| opendevreview | Steve Baker proposed openstack/ironic master: DMN disable ir-novnc in ironic-standalone-redfish https://review.opendev.org/c/openstack/ironic/+/943871 | 22:47 |
| opendevreview | Steve Baker proposed openstack/ironic master: DNM install podman https://review.opendev.org/c/openstack/ironic/+/944078 | 22:47 |
| stevebaker[m] | sorry about the noise | 22:47 |
| JayF | oh no! You're trying to fix CI! How terrible! :D | 22:49 |
| JayF | :D | 22:49 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!