| opendevreview | Merged openstack/bifrost stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/bifrost/+/876975 | 01:23 |
|---|---|---|
| opendevreview | OpenStack Proposal Bot proposed openstack/ironic master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic/+/877066 | 03:11 |
| rpittau | good morning ironic! o/ | 08:12 |
| rpittau | happy friday! | 08:16 |
| opendevreview | Riccardo Pittau proposed openstack/sushy master: Exclude .local from flake8 tests https://review.opendev.org/c/openstack/sushy/+/877077 | 08:22 |
| rpittau | dtantsur, JayF, the metal3 ci integration is ready for reviews when you have a mooment, thanks! https://review.opendev.org/c/openstack/ironic/+/863873 | 10:42 |
| dtantsur | rpittau: I wrote a half of it, it's probably not fair for me to +2 it now :) | 10:43 |
| dtantsur | let's maybe catch iurygregory later for another review | 10:43 |
| rpittau | right! | 10:44 |
| opendevreview | Riccardo Pittau proposed openstack/sushy master: Exclude all files starting with . from flake8 tests https://review.opendev.org/c/openstack/sushy/+/877077 | 11:19 |
| iurygregory | good morning Ironic | 11:20 |
| iurygregory | looking at the patch now rpittau | 11:20 |
| rpittau | hey iurygregory :) | 11:23 |
| rpittau | ack | 11:23 |
| dtantsur | could I also get moar reviews on https://review.opendev.org/c/openstack/ironic/+/874677 and https://review.opendev.org/c/openstack/ironic/+/875915 please? | 11:31 |
| dtantsur | both are essentially moving code around | 11:32 |
| iurygregory | dtantsur, sure | 11:32 |
| iurygregory | rpittau, quick question, I notice that the after_pivoting folder doesn't contain any information, this is expected? | 11:42 |
| dtantsur | I don't think we do pivoting at this point | 11:43 |
| dtantsur | TheJulia: do we have any reasons not to restrict viewing driver_internal_info to admins only? | 11:43 |
| iurygregory | just wanted to double check, +2 | 11:44 |
| kubajj | morning everyone o/ | 11:49 |
| dtantsur | morning kubajj | 11:52 |
| dtantsur | TheJulia: aha, we already restrict driver_internal_info to SYSTEM_OR_OWNER_READER. I wonder if it should be SYSTEM_OR_OWNER_ADMIN instead, but at least lessees are ruled out. | 11:53 |
| kubajj | dtantsur: out of the storyboard https://storyboard.openstack.org/#!/story/2010275 , what should be the next step that I could look into after all the deadlines are passed? | 11:55 |
| dtantsur | kubajj: I would suggest ironicclient and openstacksdk support for the inventory API | 11:56 |
| dtantsur | (it's not in the story, feel free to create tasks there) | 11:56 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Migrate the inspector's /continue API https://review.opendev.org/c/openstack/ironic/+/875944 | 12:00 |
| rpittau | iurygregory: we don't do the pivoting for the time being | 13:03 |
| iurygregory | rpittau, ack, I've already +2 | 13:04 |
| rpittau | thanks! | 13:04 |
| iurygregory | yw | 13:04 |
| vanou | TheJulia: If possible, please give review for https://review.opendev.org/c/openstack/ironic/+/870881/6 | 13:29 |
| vanou | JayF: Do you have any feedback from OpenStack vulnerability team? | 13:29 |
| vanou | JayF: Sorry for annoying you with ^ | 13:30 |
| opendevreview | Merged openstack/metalsmith stable/zed: test_provision: don't assert provision_node call order https://review.opendev.org/c/openstack/metalsmith/+/877039 | 14:02 |
| opendevreview | Merged openstack/metalsmith stable/yoga: test_provision: don't assert provision_node call order https://review.opendev.org/c/openstack/metalsmith/+/877040 | 14:03 |
| opendevreview | Merged openstack/metalsmith stable/xena: test_provision: don't assert provision_node call order https://review.opendev.org/c/openstack/metalsmith/+/877041 | 14:03 |
| opendevreview | Merged openstack/metalsmith stable/wallaby: test_provision: don't assert provision_node call order https://review.opendev.org/c/openstack/metalsmith/+/877042 | 14:03 |
| opendevreview | Verification of a change to openstack/ironic master failed: Restructure the inspector module in preparation for its expansion https://review.opendev.org/c/openstack/ironic/+/874677 | 14:24 |
| JayF | vanou: it looks like it'll be possible, I'm just running behind because I'm at a conference. I'll pursue it more next week. | 15:56 |
| opendevreview | Merged openstack/ironic-python-agent stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/876979 | 16:03 |
| opendevreview | Merged openstack/ironic-python-agent master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/876980 | 16:03 |
| opendevreview | Merged openstack/bifrost stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/bifrost/+/876976 | 16:05 |
| rpittau | bye everyone have a great weekend! o/ | 16:05 |
| opendevreview | Verification of a change to openstack/ironic master failed: Restructure the inspector module in preparation for its expansion https://review.opendev.org/c/openstack/ironic/+/874677 | 16:31 |
| opendevreview | Merged openstack/ironic master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic/+/877066 | 16:34 |
| opendevreview | Merged openstack/ironic master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/ironic/+/877000 | 16:34 |
| JayF | In case you all were wondering what TheJulia and I are up to at SCALE -> https://opensource.com/article/23/3/cloud-hardware that's the article we wrote for opensource.com about the topic we're speaking on tomorrow | 17:11 |
| JayF | dtantsur: Does this make us the first k8s<>openstack mutual-co-CI project? | 17:21 |
| * JayF just +2A that metal3 ci | 17:22 | |
| dtantsur | \o/ | 17:22 |
| dtantsur | possibly? | 17:22 |
| JayF | dtantsur: it'd be an API-break to change that policy default. I think the impetus needs to be the other way around; is there a good reason it needs to be _ADMIN over _READER? | 17:23 |
| * JayF wonders if you stumbled onto a near-security-thing | 17:23 | |
| dtantsur | JayF: driver_internal_info exposes not versioned and often poorly reviewed internals | 17:23 |
| JayF | you only see it if you ask for the field or /details, right? | 17:24 |
| dtantsur | right | 17:24 |
| JayF | I don't see that as a compelling reason then; but I'll be interested to see waht others thing. | 17:24 |
| JayF | *think | 17:24 |
| JayF | might be worth asking on the list to increase reach to operators | 17:24 |
| TheJulia | Is the policy thing in the ci job? | 17:43 |
| dtantsur | TheJulia: no, it's re your question on the /continue_inspection patch | 17:44 |
| dtantsur | about the cached BMC addresses in driver_internal_info | 17:44 |
| TheJulia | I’m feeling under the weather, today is not time to discuss or think about such. Monday I guess | 17:45 |
| dtantsur | ack, get better! | 18:03 |
| dtantsur | does anyone remember when we removed support for classic drivers like pxe_ipmitool? | 18:16 |
| * dtantsur cannot find 12.0.0 release notes | 18:18 | |
| JayF | wow it was a ... while ago | 18:18 |
| JayF | like Ussuri? | 18:18 |
| JayF | Victoria? | 18:18 |
| dtantsur | I think longer ago.. at least I hope so | 18:20 |
| * dtantsur does not want to support classic drivers in rust-openstack lol | 18:20 | |
| JayF | I give you permission hereby to not do so. | 18:20 |
| JayF | What is rust-openstack? :D | 18:20 |
| dtantsur | https://docs.rs/openstack/latest/openstack/ | 18:21 |
| JayF | nice, SDK work is fun. | 18:21 |
| dtantsur | per release notes, it was before Rocky, but when exactly.. | 18:22 |
| dtantsur | ah, well. we have an issue in our old release notes that some first versions are not displayed | 18:22 |
| dtantsur | all signs point at 11.0.0 (Rocky) as the removal release | 18:22 |
| JayF | I honestly just remember doing a J->U upgrade, and we had to migrate drivers->hardware types | 18:22 |
| dtantsur | yep, it was 11.0.0. thank you git. | 18:23 |
| JayF | dtantsur: you are the doc-bug-finder these days, aren't ya | 18:23 |
| dtantsur | :D | 18:23 |
| JayF | can you file that or dig into it? | 18:23 |
| JayF | I do not expect myself to remember after 2.5 more days of conference | 18:23 |
| dtantsur | I can file but where? | 18:24 |
| dtantsur | (not going to dig into it a 7:30pm) | 18:24 |
| JayF | storyboard still for now | 18:24 |
| dtantsur | it's equivalent of /dev/null :) | 18:24 |
| JayF | so is my brain | 18:24 |
| JayF | so at least we get two shots | 18:24 |
| dtantsur | so yeah, I don't want to support microversions before the classic drivers removal. much easier for me. | 18:25 |
| JayF | the venn diagram of "users using rust to interact with openstack" and "users using a rocky-or-earlier" openstack HAS to be two separate circles, right? | 18:25 |
| JayF | that'd be a software stack mullet otherwise D: | 18:26 |
| dtantsur | I expect that | 18:26 |
| dtantsur | Well, I did get a bug report about Newton not so long ago.. | 18:26 |
| dtantsur | but I think ironic users are usually more fast-paced | 18:26 |
| dtantsur | JayF: https://storyboard.openstack.org/#!/story/2010640 | 18:26 |
| JayF | ty | 18:28 |
| JayF | I hope you squashed that bug report with, WTF; upgrade :D | 18:28 |
| JayF | I made the comment to someone yesterday, sometimes it feels like OpenStack is judged on the software written 3-5yrs ago instead of the software today | 18:28 |
| TheJulia | I think the last ops survey was mostly Victoria and newer in Berlin | 18:29 |
| JayF | Okay, maybe 2 years then ;) hehe | 18:33 |
| JayF | vanou: please see list; unless Ironic contributors object I will be applying for Ironic to be managed by VMT. | 18:34 |
| JayF | vanou: This will leave a gap for unofficial library projects which are not under openstack, but are closely related. VMT will responsibly disclose things to our downstream libraries if presented through those avenues; but projects in that category should consider publishing instructions on responsible disclosure (I believe github has built-in tools now) | 18:35 |
| opendevreview | Merged openstack/ironic master: Add a non-voting metal3 CI job https://review.opendev.org/c/openstack/ironic/+/863873 | 19:08 |
| dtantsur | yay, great end of week ^^^ | 19:13 |
| dtantsur | have a nice weekend folks | 19:13 |
| opendevreview | Merged openstack/ironic master: Restructure the inspector module in preparation for its expansion https://review.opendev.org/c/openstack/ironic/+/874677 | 19:16 |
| opendevreview | Verification of a change to openstack/ironic master failed: Refactoring: extract some common functions from the inspector code https://review.opendev.org/c/openstack/ironic/+/875915 | 19:27 |
| jlvillal | So if I read the docs correctly Bifrost is not expected to work in an IPv6 environment. Is that correct? And I'm okay with that as a note. | 19:32 |
| opendevreview | John L. Villalovos proposed openstack/bifrost master: chore: allow ironic-conductor to work with IPv6 disabled https://review.opendev.org/c/openstack/bifrost/+/877122 | 19:37 |
| opendevreview | John L. Villalovos proposed openstack/bifrost master: chore: provide the location of deploy_image_path if missing https://review.opendev.org/c/openstack/bifrost/+/877124 | 19:42 |
| iurygregory | jlvillal, yeah we are lacking IPv6 support on it .-. | 21:38 |
| sschmitt | This is a long shot but I have a train based ironic setup using ISCSI deploy and im getting intermittent deploy failures about the ISCSI target failing to start due to "Processes with the following PIDs are holding device /dev/sda: <PID>" any ideas where to start with this? | 21:48 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!