| opendevreview | Merged openstack/ironic stable/2025.1: Don't fail publishing an already published image https://review.opendev.org/c/openstack/ironic/+/1003590 | 03:21 |
|---|---|---|
| zigo | Good morning Ironic. | 06:41 |
| zigo | When building 38.0.0, I get this https://paste.opendev.org/show/b5J4ipyW26Ev4wnxH250/ | 06:41 |
| zigo | Any idea why I get this "duplicate option: tls-ciphers" ? | 06:41 |
| zigo | FYI, building with older versions of libs (keystoneauth1, oslo.config, etc.) from Gazpacho doesn't lead to this error, so it is likely a change in one of the oslo libs. | 06:41 |
| zigo | (or keystoneauth1, the sdk, etc.) | 06:41 |
| dtantsur | zigo: this is new keystoneauth1 | 06:55 |
| zigo | dtantsur: Oh, so what can I do? | 06:56 |
| dtantsur | zigo: you may want to pull in https://review.opendev.org/c/openstack/ironic/+/1003617 | 06:56 |
| zigo | Ah, thanks, trying. | 06:56 |
| zigo | dtantsur: Tests are currently running, thank you ! | 07:06 |
| dtantsur | yw | 07:07 |
| opendevreview | Merged openstack/ironic bugfix/38.0: Add short timeout for image URL validation https://review.opendev.org/c/openstack/ironic/+/1003574 | 07:42 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix oslo.service version to >= 4.5.0 https://review.opendev.org/c/openstack/ironic/+/1003755 | 12:04 |
| TheJulia | good morning | 13:08 |
| *** zigo_ is now known as zigo | 14:07 | |
| iurygregory | ironic cores - https://review.opendev.org/c/openstack/releases/+/1003789 cycle highlights | 14:09 |
| iurygregory | I wasn't sure how we should mention the CVE's fixes so I just mentioned something generic | 14:10 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Switch netconf transport from paramiko to libssh https://review.opendev.org/c/openstack/networking-generic-switch/+/998062 | 14:38 |
| TheJulia | iurygregory: I guess I'm not sure I'd focus on deprecations, and maybe the longer form tone to set is "Ironic has seen a number of security related fixes and improvements go in thanks to the efforts of contributors and vulnerability reporters." Furthermore, we've also seen enhancements to node firmware, node history endpoints. Another item we've added is per-step RBAC enforcement as well extending operator. | 14:50 |
| JayF | I have language going into gerrit now fwiw | 14:52 |
| TheJulia | k | 14:53 |
| TheJulia | Sorry, I've been on a call for the past hour or so which has consumed my attention until the last little bit | 14:53 |
| iurygregory | TheJulia, ack | 14:56 |
| *** sfinucan is now known as stephenfin | 14:58 | |
| iurygregory | going to grab lunch, will update when I'm back o/ | 14:59 |
| stephenfin | Can someone tell, what's the difference between ironic-core and ironic-approvers in Gerrit? | 15:28 |
| stephenfin | *tell me | 15:28 |
| TheJulia | Approvers can workflow | 15:29 |
| TheJulia | I think, I might have that backwards | 15:30 |
| stephenfin | so there's also ironic-reviewers | 15:30 |
| TheJulia | okay, then core may be a legacy group. I personally haven;'t looked at it recently | 15:30 |
| * stephenfin looks at openstack/project-config | 15:31 | |
| stephenfin | Yeah, looks like ironic-core is an old alias that probably needs cleaning up | 15:33 |
| stephenfin | TheJulia: I assume molteniron is a dead project? | 15:34 |
| TheJulia | Yes, I thought it was nuked | 15:35 |
| stephenfin | (context of all this is that I'm making changes for a new nova-reviwers group at long last) | 15:35 |
| TheJulia | (figured that, and thanks!) | 15:35 |
| TheJulia | (also, mentally on a standards body meeting at the moment) | 15:36 |
| stephenfin | openstack/moltencore retirement patches: | 15:46 |
| stephenfin | https://review.opendev.org/c/openstack/project-config/+/1003838 | 15:46 |
| stephenfin | https://review.opendev.org/c/openstack/molteniron/+/1003841 | 15:46 |
| stephenfin | https://review.opendev.org/c/openstack/project-config/+/1003839 | 15:46 |
| stephenfin | ...and the follow-up patch to do what I actually wanted and remove ironic-core from Gerrit ACLs. That should clear up any confusion and allow for the eventual deletion (?) of the ironic-core group: | 15:47 |
| stephenfin | https://review.opendev.org/c/openstack/project-config/+/1003840 | 15:47 |
| stephenfin | I would appreciate reviews from ~~ironic-core~~ ironic-reviewers / ironic-approvers when possible 🙏 | 15:47 |
| JayF | +1/+2 across the stack, one comment on the base project-config change | 15:48 |
| JayF | but I suspect you've done retirement more than me :) | 15:48 |
| stephenfin | I think fungi told me not to remove official-openstack-repo-jobs for...some reason | 15:49 |
| stephenfin | (I just copied what I did for requestsexceptions a few months back) | 15:49 |
| JayF | ack | 15:49 |
| stephenfin | and thanks | 15:49 |
| JayF | like I said I believe you probably have a better handle on it than mE :) | 15:49 |
| JayF | I honestly am a litle embarassed we didn't do this | 15:49 |
| JayF | molteniron was tools for doing third-party-ci with ironic | 15:50 |
| JayF | it's been dead a good long while | 15:50 |
| stephenfin | good to know | 15:51 |
| stephenfin | also note that there's https://review.opendev.org/c/openstack/project-config/+/1003840 which is the one I ultimately wanted in | 15:51 |
| TheJulia | .. I thought we did ages ago, I'm super surprised as well | 15:51 |
| stephenfin | I think it makes sense, based on my understanding of "core" in ironic today | 15:51 |
| JayF | stephenfin: put a comment on it. I'm also about to check to make sure bifrost.config is in use at all | 15:55 |
| JayF | stephenfin: the real fix for us might be just aligning that with rest of Ironic and using the same config... but IDK if bifrost-core has any members not in ironic-* | 15:55 |
| stephenfin | ack, I wasn't if the approvers/reviewers split carried over to bifrost. I've made the change | 15:56 |
| TheJulia | I think it was used as a launching pad early on but likely is no longer needed. | 15:56 |
| JayF | At this point, it's pretty embedded in our ethos | 15:58 |
| JayF | and we've done a decent job of keeping people promoting through the pipelines | 15:58 |
| TheJulia | Yeah, it took it to get there | 15:58 |
| TheJulia | but yeah, way better place now then we were | 15:58 |
| JayF | if we can't talk to someone and say "please only approve things on bifrost and trivial thinsg in other repos" | 15:58 |
| JayF | I don't want them to have +2 on anything | 15:58 |
| JayF | someones' judgement should be good enough to self-guide on when to be a "core" or not, and if not, that's a sign they shouldn't have the cred IMO | 15:59 |
| stephenfin | makes sense | 16:04 |
| TheJulia | Guys, do you need anything from me in the next week? | 16:49 |
| fungi | i don't recall telling stephenfin not to remove official-openstack-repo-jobs but i don't recall a lot of things these days so i trust his memory more than mine, i must have made a compelling argument | 16:51 |
| JayF | TheJulia: if you need to GTFO; do so. If we have a hard dep on you personally it's a bug we can fix while you take care of stuff | 16:53 |
| TheJulia | JayF: more so to make sure nobody was expecting anything from me because I'm going to wind down my week soon given I have a 6AM flight out of san diego on sunday morning | 16:56 |
| JayF | yeah go get yourself some rest | 16:56 |
| JayF | that's for shanghai, yeah? | 16:56 |
| TheJulia | well, still some loose ends, but yeah | 16:56 |
| iurygregory | I've updated the cycle highlights https://review.opendev.org/c/openstack/releases/+/1003789 o/ | 17:12 |
| iurygregory | Tks JayF TheJulia and kubajj | 17:13 |
| opendevreview | Merged openstack/ironic master: Fix oslo.service version to >= 4.5.0 https://review.opendev.org/c/openstack/ironic/+/1003755 | 22:18 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!