| opendevreview | Merged openstack/ironic stable/2025.2: Fix async periodic on redfish for servicewait https://review.opendev.org/c/openstack/ironic/+/978125 | 00:18 |
|---|---|---|
| opendevreview | Merged openstack/networking-generic-switch master: vxlan: Arista EOS and vendor neutral SONiC https://review.opendev.org/c/openstack/networking-generic-switch/+/972763 | 00:19 |
| opendevreview | Merged openstack/ironic master: Various docstring fixes https://review.opendev.org/c/openstack/ironic/+/977993 | 02:02 |
| cardoe | TheJulia: the rest of your changes are merged conflicted. | 02:19 |
| TheJulia | Nooooooooooo | 02:19 |
| TheJulia | I’ll fix them after the game | 02:19 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Remove flavor dependency from InspectorBasicTest https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/978106 | 02:45 |
| opendevreview | Merged openstack/networking-baremetal master: Add a reconciler to fix bug 1995078 https://review.opendev.org/c/openstack/networking-baremetal/+/975333 | 04:17 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: follow-up on trunk reconciler https://review.opendev.org/c/openstack/networking-baremetal/+/976977 | 05:12 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: Add Junos, Cumulus NVUE, and denote Dell OS10 as unsupported https://review.opendev.org/c/openstack/networking-generic-switch/+/972764 | 05:14 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: OVS testing patch for 'vxlan' binding model https://review.opendev.org/c/openstack/networking-generic-switch/+/972765 | 05:15 |
| TheJulia | I'll clean up the docs one in the morning with coffee, but ngs was basically just gerrit confusion | 05:17 |
| TheJulia | shermanm: w/r/t the multiple connection restriction on your patch, any reason for it? I guess mentally I already started to trade concurrency issues for folks using sonic, but you indicate its mostly os10 in this case? | 05:21 |
| rpittau | good morning ironic! happy friday ! o/ | 07:46 |
| rpittau | TheJulia: re ngs/n-b release: sounds good, I was going to ask, I'll check what's left today:) | 07:47 |
| rpittau | TheJulia: I've reviewd what I think was in the prio list for ngs and nb, please doublecheck if there's anything left when you got a moment :) | 09:02 |
| opendevreview | Merged openstack/ironic stable/2025.1: ci: Pin `setuptools` to a range that still ships https://review.opendev.org/c/openstack/ironic/+/977317 | 09:16 |
| opendevreview | Merged openstack/python-ironicclient stable/2025.2: Fix parallel initial version negotiation https://review.opendev.org/c/openstack/python-ironicclient/+/977406 | 09:31 |
| opendevreview | Merged openstack/networking-generic-switch master: vxlan: Add Junos, Cumulus NVUE, and denote Dell OS10 as unsupported https://review.opendev.org/c/openstack/networking-generic-switch/+/972764 | 09:35 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: Migrate ci jobs to noble https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/977966 | 09:57 |
| opendevreview | Seunghun Lee proposed openstack/ironic stable/2025.1: Improve image download performance https://review.opendev.org/c/openstack/ironic/+/978179 | 10:11 |
| rpittau | cardoe: hey any reason why the ironic-week-prio tag was removed by https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/975891 ? | 10:25 |
| opendevreview | Merged openstack/networking-generic-switch master: vxlan: OVS testing patch for 'vxlan' binding model https://review.opendev.org/c/openstack/networking-generic-switch/+/972765 | 10:29 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Dial back the multi-tenancy testing https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/978041 | 10:32 |
| opendevreview | Merged openstack/ironic master: clean up exception handling so type hinting is correct https://review.opendev.org/c/openstack/ironic/+/978020 | 10:35 |
| opendevreview | Merged openstack/ironic master: Update ovn documentation based upon VXLAN discoveries https://review.opendev.org/c/openstack/ironic/+/977546 | 10:35 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: Refactor redfish parser to eliminate code duplication https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/975891 | 12:01 |
| opendevreview | Christian Berendt proposed openstack/ironic stable/2024.2: Set columns in bios_settings table to bigint https://review.opendev.org/c/openstack/ironic/+/964771 | 12:12 |
| rpittau | please review https://review.opendev.org/c/openstack/bifrost/+/977955 when you get a chance, fix for bifrost CI | 14:00 |
| *** dking is now known as Guest3956 | 14:15 | |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: follow-up on trunk reconciler https://review.opendev.org/c/openstack/networking-baremetal/+/976977 | 14:23 |
| TheJulia | rpittau: ^^^ is one, working on the last one (docs) | 14:27 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: docs: Reorder docs to be more sane https://review.opendev.org/c/openstack/networking-generic-switch/+/978015 | 14:32 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Detect device configuration errors on Dell Force10 and OS10 https://review.opendev.org/c/openstack/networking-generic-switch/+/868318 | 14:38 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: follow-up on trunk reconciler https://review.opendev.org/c/openstack/networking-baremetal/+/976977 | 14:48 |
| *** Guest3956 is now known as dking | 14:50 | |
| Sandzwerg[m] | Ohai Ironic, I'm currently in the process of switching from the (very) old policies to the new RBAC ones. And we have a cloud_baremetal_admin. Basically the people like me who can do everything with a node and we'd like to keep it, at least for now. So from my understanding I want that people who have this role to be basically system scoped. For all other things I hope the default RBAC rules should work. How would such a rule look | 15:25 |
| Sandzwerg[m] | like? | 15:25 |
| JayF | well, you can take different approaches | 15:46 |
| JayF | Are you single-tenant cloud? | 15:46 |
| JayF | In a single tenant cloud, I think a pattern of setting node.owner to that project and giving folks project-scoped admin is likely better than giving system-scoped admin out | 15:46 |
| Sandzwerg[m] | Nope, but we as cloud operators are basically the only ones who change anything at node level. And so far we haven't looked into the whole owner/leesee thing. | 15:48 |
| Sandzwerg[m] | hmm so make a single project owner? If that is the way to go that would work as well. Then I can just run the default polciy file? | 15:49 |
| opendevreview | Merged openstack/networking-baremetal master: CI: implement vxlan job https://review.opendev.org/c/openstack/networking-baremetal/+/977081 | 15:51 |
| TheJulia | That seems like a good way to handle it | 15:53 |
| TheJulia | Yes!!!! It merged | 15:53 |
| JayF | Exactly. Than you just have project admins | 15:56 |
| JayF | if those folks are enrolling nodes, there's a toggle to enable that | 15:56 |
| JayF | so a project admin can do node creates and it automatically gets the node.owner=that_project | 15:57 |
| Sandzwerg[m] | Yeah that would be useful Is that a setting in the policy or in the ironic config? | 15:58 |
| JayF | in the ironic config | 15:59 |
| JayF | #project_admin_can_manage_own_nodes = true | 15:59 |
| JayF | default of true is surprising to me, in a good way | 15:59 |
| JayF | basically your choice is embrace Ironic's use of RBAC (good, what we've talked about here) or try to pretend scoping was never added (use system admin everywhere in Ironic at your own risk) | 16:00 |
| Sandzwerg[m] | hmkay then for viewers I would need to make sure that certain nodes get assigned as leesee to the corresponding project so users could see the nodes? That might be a bit of work as right now we don't care about a direct owner/leesee <-> node connection. As long as they have quota and there isa free node they can spawn an instance. And if the one they are using breaks but another one is free they can use that one | 16:04 |
| TheJulia | Not every endpoint supports non system scoped access, but it all seemed logical to us at the time. If we need to change something or there is feedback, by all means let us know | 16:04 |
| TheJulia | Yeah, the lesser can be added automatically so as long as the user has reader privs they can see the node in the list | 16:05 |
| cardoe | Sounds like Sandzwerg[m] is trying to do the same auth stuff I am? | 16:06 |
| cardoe | Cause project_admin_can_manage_own_nodes=true would be my mode. | 16:06 |
| JayF | you only need lessee if you have >1 project | 16:07 |
| JayF | if you are single-project, just use project reader :) | 16:07 |
| JayF | if you have >1 project, using node.owner for the cloud-management team and automatic lessee to set node.lessee to the project that provisioned it (cleared on deprovision) | 16:07 |
| JayF | then you can tweak for both owner and lessee case separately | 16:07 |
| Sandzwerg[m] | Yeah I'm not in general against RBAC but this wasn't touched or thought about for a long time. The policy file is still json instead of yaml. And now I need to think about what we want or need to change as I got a big warning that the old rules will break with the next upgrade (currently on 2023.1) and we'd like to update soonish | 16:08 |
| cardoe | https://review.opendev.org/c/openstack/ironic/+/977497 could I get eyes on that? the relationship chain above it is already +2'd and +W'd. That was +2'd and +W'd but when I re-enabled pep8 I needed to fix it up. | 16:08 |
| JayF | I would strongly suggest starting with a point of evaluating our post-migration defaults | 16:09 |
| JayF | and trying to fit the business use case into that rather than trying to emulate your previous workflows | 16:09 |
| Sandzwerg[m] | Oh, the lessee get's automatically set on deployment? I was under the impression I had to set that manually. But if that is set automatically it would make everything much easier | 16:09 |
| JayF | automated_lessee_source | 16:09 |
| JayF | we can set it from metadata from nova or from keystone project metadata on the token for ironic-direct provisioning | 16:10 |
| * JayF also points at runbooks to complete the feature-triangle | 16:10 | |
| Sandzwerg[m] | Oh yeah I started with that, and it seems we are more or less using the default rules except for our "cloud_baremetal_admin" and "cloud_baremetal_viewer" which can only be handed out by cloud admins but give full rights/full view (except for secrets) | 16:10 |
| JayF | yeah that's system (scope) admin/reader | 16:11 |
| JayF | but those are ... big roles to give out | 16:11 |
| JayF | so I'd try to reduce/remove the need for that big of a grant | 16:11 |
| cardoe | JayF: speaking of runbooks... didn't you make it so that automated cleaning can inject extra steps from runbooks if something matches? | 16:11 |
| Sandzwerg[m] | Hmm, I heard a bit about runbooks but haven't looked to deep into them. I thought they were a bit like manual workflows(or what was the correct term) for cleaning? What role do they play here? | 16:12 |
| cardoe | clif: if ya peek at https://review.opendev.org/c/openstack/ironic/+/977995 there's a question on the autodoc_mock_imports and if we can resolve that then your whole docs chain can land as well. | 16:12 |
| JayF | cardoe: automated cleaning via runbook | 16:12 |
| JayF | cardoe: you basically opt-in to having a runbook be your automated cleaning instead of implicit steps | 16:13 |
| cardoe | ah so I gotta define all the steps. | 16:13 |
| JayF | https://docs.openstack.org/ironic/latest/admin/cleaning.html#cleaning-steps | 16:13 |
| Sandzwerg[m] | > so I'd try to reduce/remove the need for that big of a grant | 16:15 |
| Sandzwerg[m] | yeah, think that that is the idea. Might think about this a bit more. But switching to owner/lessee with the default policy then sounds like the way forward | 16:15 |
| * TheJulia goes and has vxlan celebration breakfast | 16:18 | |
| cardoe | TheJulia: no no. You've got 5 opened ones. 2 can be merged and have a +2 from me. 3 of them either have merge conflicts or failed tests. No breakfast for you until it's all green! | 16:22 |
| * cardoe adjusts his evil hat. | 16:22 | |
| cardoe | https://review.opendev.org/q/topic:%22ironic-vxlan%22+status:open | 16:22 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2024.2: ci: Pin `setuptools` to a range that still ships https://review.opendev.org/c/openstack/ironic/+/978234 | 16:25 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add osprofiler.opts as an import https://review.opendev.org/c/openstack/ironic/+/977994 | 16:27 |
| opendevreview | Clif Houck proposed openstack/ironic master: Suppress autodoc warnings https://review.opendev.org/c/openstack/ironic/+/977995 | 16:27 |
| opendevreview | Clif Houck proposed openstack/ironic master: Fix documentation related build warnings throughout Ironic's docs https://review.opendev.org/c/openstack/ironic/+/977996 | 16:27 |
| opendevreview | Clif Houck proposed openstack/ironic master: Treat warnings as errors in sphinx-build for docs tox target https://review.opendev.org/c/openstack/ironic/+/977997 | 16:27 |
| opendevreview | Clif Houck proposed openstack/ironic master: Documentation for Trait Based Networking https://review.opendev.org/c/openstack/ironic/+/975686 | 16:27 |
| dtantsur | JayF: good point on https://review.opendev.org/c/openstack/ironic/+/973279/4/ironic/conductor/deployments.py but this aspect was cargo-culted from cleaning https://opendev.org/openstack/ironic/src/commit/b789679abdf89b8c3fa85709590d7002be242bc0/ironic/conductor/cleaning.py#L466 | 16:30 |
| dtantsur | so I guess we need to agree to change both | 16:30 |
| dtantsur | pardon, all three: https://opendev.org/openstack/ironic/src/commit/b789679abdf89b8c3fa85709590d7002be242bc0/ironic/conductor/servicing.py#L419 | 16:30 |
| JayF | In that case, we should ignore my comment there and change all together if we think it's the right move | 16:30 |
| dtantsur | JayF: I'm fine doing that, I'm also fine changing my code and filing a bug for cleaning+servicing. | 16:31 |
| JayF | The more I think about it, the more I think it's the right move, btu it'd be cleaner to ahve that be a separate unit oof work | 16:31 |
| JayF | I have to run to the doc for a surgery follow-up, feel free to merge over my objection there since we've chatted about it | 16:31 |
| dtantsur | Good luck! I don't think anyone would merge over your -1 :) | 16:32 |
| dtantsur | but it can wait until next week for sure | 16:32 |
| * dtantsur adds to the Monday meeting | 16:33 | |
| shermanm | TheJulia the reason I limited it to just concurrency=1 was because I didn't implement a connection pool yet for caching multiple. For dell os10 in particular, and I believe also for os9, the switch allows you to make multiple concurrent ssh connections, but will fail or corrupt config if you try to use them, so they only actually "support" a single connection at a time. | 16:39 |
| * dtantsur facepalm | 16:39 | |
| shermanm | I don't yet have any switches to test against for the multiple connections case, but I don't see any reason why a pool of cached connections of size=ngs_max_connections wouldn't work | 16:39 |
| shermanm | also while testing this I found that ngs totally ignores ngs_max_connections if the coordination backend is unset. (and made https://bugs.launchpad.net/networking-generic-switch/+bug/2142786) | 16:41 |
| shermanm | something in neutron might yell if you leave it blank, but ngs-stress at least just ignores it silently | 16:41 |
| opendevreview | Clif Houck proposed openstack/ironic master: Import and set_defaults osprofiler.opts https://review.opendev.org/c/openstack/ironic/+/977994 | 16:42 |
| opendevreview | Clif Houck proposed openstack/ironic master: Suppress autodoc warnings https://review.opendev.org/c/openstack/ironic/+/977995 | 16:42 |
| opendevreview | Clif Houck proposed openstack/ironic master: Fix documentation related build warnings throughout Ironic's docs https://review.opendev.org/c/openstack/ironic/+/977996 | 16:42 |
| opendevreview | Clif Houck proposed openstack/ironic master: Treat warnings as errors in sphinx-build for docs tox target https://review.opendev.org/c/openstack/ironic/+/977997 | 16:42 |
| opendevreview | Clif Houck proposed openstack/ironic master: Documentation for Trait Based Networking https://review.opendev.org/c/openstack/ironic/+/975686 | 16:42 |
| clif | cardoe: I think I've addressed things | 16:42 |
| cardoe | cool. Just trying to help ya get it landed. | 16:47 |
| clif | ty! | 16:47 |
| clif | I appreciate it | 16:47 |
| TheJulia | cardoe: two of those can definitely wait ;) I’ll fix the new merge conflicts after breakfast | 16:49 |
| cardoe | I'll allow it if ya re +W https://review.opendev.org/c/openstack/ironic/+/977497 now that I fixed that pep8 issue. | 16:50 |
| TheJulia | Done, time for coffee! | 16:53 |
| TheJulia | https://usercontent.irccloud-cdn.com/file/3XNSPMLI/1772211244.JPG | 16:54 |
| TheJulia | shermanm: interesting, true for os9 afaik. I think 10 is supposed to be better, but yeah. Are you going to continue to revise ? Think it makes a lot of sense to try and pool to an extent, anything we can do to speed performance. | 17:02 |
| shermanm | I can revise, I'd mostly wanted to get comments before I went further. In some ways you don't really need this change if you have multiple connections available to use. | 17:05 |
| shermanm | for pooling, it wasn't clear to me what the best way to proceed was. the cached connections are local to the instance of ngs/neutron-server, but the "max connections" and locking is supposed to be across (potentially) multiple instances, right? | 17:05 |
| shermanm | I can revise, I'd mostly wanted to get comments before I went further. In some ways you don't really need this change if you have multiple connections available to use. | 17:05 |
| shermanm | for pooling, it wasn't clear to me what the best way to proceed was. the cached connections are local to the instance of ngs/neutron-server, but the "max connections" and locking is supposed to be across (potentially) multiple instances, right? | 17:05 |
| TheJulia | It’s across the process. Yeah, need to think about tha | 17:15 |
| TheJulia | That :( | 17:15 |
| shermanm | I guess the simple approach is that each process caches up to ngs_max_connections sessions in a local pool, but the distributed lock prevents more than ngs_max_connections from getting "used" in total, so we'd always have "max cached connections >= max used connections". | 17:26 |
| shermanm | keep the two mechanisms completely orthogonal | 17:26 |
| shermanm | I guess the simple approach is that each process caches up to ngs_max_connections sessions in a local pool, but the distributed lock prevents more than ngs_max_connections from getting "used" in total, so we'd always have "max cached connections >= max used connections". | 17:26 |
| shermanm | keep the two mechanisms completely orthogonal | 17:26 |
| shermanm | (I need to get used to the keybindings here 🤦♂️) | 17:27 |
| TheJulia | Yeah, likely best to do that, document but not try to account for everything in code. | 17:30 |
| dtantsur | JFYI metal3 is getting rid of the iRMC driver in https://github.com/metal3-io/baremetal-operator/pull/3034 and https://github.com/metal3-io/ironic-image/pull/928 | 17:40 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2024.2: ci: Pin `setuptools` to a range that still ships https://review.opendev.org/c/openstack/ironic/+/978234 | 17:41 |
| opendevreview | Merged openstack/ironic stable/2025.1: Fix Redfish boot device setting for full boot request vendors https://review.opendev.org/c/openstack/ironic/+/976755 | 17:43 |
| opendevreview | Merged openstack/ironic stable/2025.1: Improve image download performance https://review.opendev.org/c/openstack/ironic/+/978179 | 17:44 |
| opendevreview | Merged openstack/ironic master: split off helper func in inspection_rules for testing https://review.opendev.org/c/openstack/ironic/+/977497 | 18:06 |
| opendevreview | Julia Kreger proposed openstack/ironic master: vxlan: Add docs to Ironic to cover the VXLAN use case https://review.opendev.org/c/openstack/ironic/+/977102 | 18:17 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: trunk-reconciler: Dial back logging https://review.opendev.org/c/openstack/networking-baremetal/+/977074 | 18:18 |
| TheJulia | cardoe: there you go :) | 18:18 |
| opendevreview | Merged openstack/ironic master: fix inspection rules loaded from a file to not fail in logging https://review.opendev.org/c/openstack/ironic/+/978116 | 18:24 |
| opendevreview | Merged openstack/ironic master: add helper tool for testing inspection rules https://review.opendev.org/c/openstack/ironic/+/973381 | 18:24 |
| cardoe | only 12 patches in ironic-week-prio :) | 18:40 |
| cardoe | TheJulia: https://review.opendev.org/c/openstack/networking-generic-switch/+/868318 that test failure is a bit concerning? | 18:41 |
| TheJulia | Going to computer to look | 18:42 |
| TheJulia | its not the job itself, its a union of chagnes, I'll propose a job fix | 18:47 |
| opendevreview | Julia Kreger proposed openstack/ironic master: ci: unrestrict runs on ironic-tempest-ipa-wholedisk-direct-multinode https://review.opendev.org/c/openstack/ironic/+/978252 | 18:51 |
| TheJulia | cardoe: ^ | 18:51 |
| cardoe | +2'd | 18:53 |
| cardoe | All but 2 patches in ironic-week-prio also have at least 1 core giving them a +2 | 18:54 |
| JayF | dtantsur: flipped my vote there; but we still should improve the docs if we can | 18:55 |
| cardoe | JayF: i’m trying to make that tag more useful for us | 19:14 |
| JayF | If we really wanna fix the tag, we rename it (as much as that would suck) | 19:14 |
| JayF | it's the ultimate contributor decoder ring lol | 19:14 |
| TheJulia | we'll need to get https://review.opendev.org/c/openstack/ironic/+/978252 merged soon() | 19:20 |
| TheJulia | might need to kick some rechecks once merged too, its all good | 19:20 |
| TheJulia | only 5 needing workflows... | 19:22 |
| cardoe | JayF: what would you rename it to? | 19:24 |
| TheJulia | who is asking for it to be renamed? | 19:24 |
| JayF | probably a review-priority vote system like other projects use | 19:24 |
| * TheJulia is sooooo very confused | 19:24 | |
| JayF | TheJulia: I am after telling a new contributor for the 100th time about it, and it's undiscoverable | 19:24 |
| JayF | I'm not saying we should do it, because the disruption would be painful, just that it's a decoder ring and not very discoverable :( | 19:25 |
| JayF | I don't like the status quo but not sure it's worth the disruption to fix it. A lot of that going around | 19:25 |
| TheJulia | Yeah, except review priority can't be set by other users afaik, and also, most other projects don't seem to actaully really use it to marshall folks. At least, I've seen it used and never seen people respond to them so *shrugs* | 19:25 |
| cardoe | Other projects actually are looking at our hash tag cause the review priority system is funky. | 19:25 |
| JayF | I think that's more of a community reflection than method :) (you can amke review-priority votable by anyone) | 19:26 |
| JayF | oh lol | 19:26 |
| TheJulia | I did notice some other projects got hashtag added and didn't use it, but then are like "wait, what, oh wow!" | 19:26 |
| rm_work[m] | cardoe: Ah ... I didn't see any doc for it, but that is my problem, SSH ports blocked :( | 19:27 |
| TheJulia | when they click where its used on ironic or on something in another repo | 19:27 |
| TheJulia | rm_work[m]: yeah, if you just set it up on your client end, it should just work, but I didn't use it since I worked at HP/HPE (or... was it IBM) where outbound ssh when on the VPN was blocked | 19:27 |
| cardoe | rm_work[m]: I’ll look if I still have it configured when I get back to the computer | 19:27 |
| TheJulia | Does anyone know when Iury is back? | 19:29 |
| TheJulia | Asking because of https://review.opendev.org/c/openstack/ironic/+/975521 | 19:29 |
| cardoe | There’s a couple others that I’ve got questions about for him as well. | 19:34 |
| rm_work[m] | cardoe: Ah I figured it out, you need to go into your settings on the review site and generate an HTTP password | 19:39 |
| rm_work[m] | that was the part I was missing, was trying to use my launchpad stuff :D | 19:39 |
| rm_work[m] | now I can do these backports... | 19:39 |
| TheJulia | Woot | 19:41 |
| opendevreview | Merged openstack/ironic master: Import and set_defaults osprofiler.opts https://review.opendev.org/c/openstack/ironic/+/977994 | 20:19 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: inspection rules tester returns non-zero code on failure https://review.opendev.org/c/openstack/ironic/+/978260 | 20:24 |
| cardoe | Something somebody internally asked me for for scripting purposes. | 20:24 |
| opendevreview | Merged openstack/bifrost master: Replace passlib with direct bcrypt for htpasswd generation https://review.opendev.org/c/openstack/bifrost/+/977955 | 20:28 |
| opendevreview | Merged openstack/ironic master: fix up the behavior of serial number reading on iDRAC https://review.opendev.org/c/openstack/ironic/+/978112 | 20:36 |
| opendevreview | Merged openstack/ironic master: Suppress autodoc warnings https://review.opendev.org/c/openstack/ironic/+/977995 | 20:36 |
| opendevreview | Merged openstack/ironic master: Fix documentation related build warnings throughout Ironic's docs https://review.opendev.org/c/openstack/ironic/+/977996 | 20:36 |
| opendevreview | Merged openstack/ironic master: Treat warnings as errors in sphinx-build for docs tox target https://review.opendev.org/c/openstack/ironic/+/977997 | 20:36 |
| opendevreview | Merged openstack/ironic master: Fix vMedia insertion on Cisco C845A M8 and similar OpenBMC systems https://review.opendev.org/c/openstack/ironic/+/977605 | 20:37 |
| cardoe | hrm.. Zuul isn't running https://review.opendev.org/c/openstack/ironic/+/978234 | 20:39 |
| cardoe | JayF: if ya get a sec for TheJulia... https://review.opendev.org/c/openstack/ironic/+/978252 | 20:39 |
| JayF | +2a | 20:40 |
| cardoe | You wanna +W the callback states since you flipped your comment on it? Or still wait for Monday? | 20:42 |
| JayF | I'll give a chance for the doc to get updated there, I flipped to a +1 because we still don't warn the operator about the identified footgun | 20:43 |
| JayF | so like, we can land it with a promise of a follow up, or quickfix + land monday morning | 20:43 |
| opendevreview | Merged openstack/ironic master: ci: unrestrict runs on ironic-tempest-ipa-wholedisk-direct-multinode https://review.opendev.org/c/openstack/ironic/+/978252 | 21:57 |
| opendevreview | Merged openstack/ironic master: Documentation for Trait Based Networking https://review.opendev.org/c/openstack/ironic/+/975686 | 21:57 |
| opendevreview | Verification of a change to openstack/ironic master failed: Redfish: guard retrieve_nic_components against MissingAttributeError https://review.opendev.org/c/openstack/ironic/+/969788 | 22:03 |
| opendevreview | Merged openstack/ironic stable/2024.2: ci: Pin `setuptools` to a range that still ships https://review.opendev.org/c/openstack/ironic/+/978234 | 22:22 |
| opendevreview | Verification of a change to openstack/ironic master failed: Redfish: guard retrieve_nic_components against MissingAttributeError https://review.opendev.org/c/openstack/ironic/+/969788 | 22:42 |
| opendevreview | Merged openstack/sushy-tools master: Fake backend: allow disabling persistent memoization https://review.opendev.org/c/openstack/sushy-tools/+/972228 | 23:24 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.2: fix inspection rules loaded from a file to not fail in logging https://review.opendev.org/c/openstack/ironic/+/978280 | 23:35 |
| opendevreview | Merged openstack/ironic master: Fix TODO: Do not assume DEPLOYWAIT https://review.opendev.org/c/openstack/ironic/+/976564 | 23:49 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
