| *** manuvakery1 is now known as manumoh | 02:19 | |
| opendevreview | Adeola Adesina proposed openstack/bifrost master: Add notes to provide more clarity for bifrost installation https://review.opendev.org/c/openstack/bifrost/+/946603 | 05:36 |
|---|---|---|
| rpittau | good morning ironic! o/ | 06:55 |
| rpittau | freemanboss[m]: I saw the bug you added in bifrost, looking forward to see the patch :) | 06:59 |
| AmarachiOrdor[m] | Good Morning rpittau | 07:00 |
| queensly[m] | Good morning | 07:01 |
| queensly[m] | rpittau: I'm working on this bug https://bugs.launchpad.net/ironic/+bug/2072353 and thinking of merging. Is it a good idea to merge? also if a new rst is created to merge, what happens to the two rst files. | 07:04 |
| opendevreview | Nicolas Belouin proposed openstack/ironic-python-agent master: netutils: Use ethtool ioctl to get permanent mac address https://review.opendev.org/c/openstack/ironic-python-agent/+/946562 | 07:08 |
| freemanboss[m] | <rpittau> "Freeman Boss: I saw the bug..." <- Please how do I go about adding the patches? | 07:11 |
| freemanboss[m] | I don't know the way forward please | 07:11 |
| freemanboss[m] | rpittau: how do I go about adding the patches? | 07:12 |
| freemanboss[m] | I don't know the way forward please | 07:12 |
| queensly[m] | Freeman Boss: check out this guide. https://docs.opendev.org/opendev/infra-manual/latest/developers.html#gerrit-ssh-host-keys | 07:13 |
| queensly[m] | Have you created a Gerrit account already? | 07:14 |
| freemanboss[m] | queensly: I've gone through it and yet to really understand what next after creating bugs? | 07:14 |
| freemanboss[m] | I'll assign the bug and clone the repo? | 07:15 |
| masghar | freemanboss: That is correct | 07:15 |
| masghar | You'd assign the bug to yourself, clone the repo, put your changes on a branch locally, and then submit your patch using git | 07:16 |
| freemanboss[m] | masghar: alright I'll proceed with that soon. | 07:16 |
| freemanboss[m] | masghar: alright thanks. | 07:16 |
| freemanboss[m] | masghar: so each docs page has its corresponding file in the bifrost repo? | 07:18 |
| AmarachiOrdor[m] | masghar: please I added this patch but I am yet to get any review | 07:19 |
| AmarachiOrdor[m] | https://review.opendev.org/c/openstack/bifrost/+/946352 | 07:19 |
| AmarachiOrdor[m] | Please would you mind reviewing it | 07:19 |
| rpittau | AmarachiOrdor[m]: I reviewed it yesterday IIRC | 07:22 |
| queensly[m] | Yes Freeman Boss | 07:25 |
| AmarachiOrdor[m] | Oh sorry for the mis understanding @rpittau I didn't see any comment on the patch | 07:25 |
| opendevreview | Adeola Adesina proposed openstack/bifrost master: Add notes to provide more clarity for bifrost installation https://review.opendev.org/c/openstack/bifrost/+/946603 | 07:25 |
| AmarachiOrdor[m] | s/@// | 07:25 |
| opendevreview | Vasyl Saienko proposed openstack/networking-generic-switch master: WIP: Add support of OVS VTEP devices https://review.opendev.org/c/openstack/networking-generic-switch/+/946558 | 07:35 |
| opendevreview | Queensly Kyerewaa Acheampongmaa proposed openstack/ironic master: Update documentation to retain relevant terms https://review.opendev.org/c/openstack/ironic/+/946504 | 07:44 |
| opendevreview | Merged openstack/ironic master: Remove compatibility for oslo.versionedobjects < 0.10.0 https://review.opendev.org/c/openstack/ironic/+/946496 | 07:58 |
| opendevreview | Adeola Adesina proposed openstack/bifrost master: Add notes to provide more clarity for bifrost installation https://review.opendev.org/c/openstack/bifrost/+/946603 | 08:07 |
| abongale[m] | Morning Ironic | 08:57 |
| abongale | Morning Ironic Team | 09:21 |
| queensly[m] | Good morning | 09:22 |
| abongale | I’m planning to take on the task of adding Tempest tests for the inspection rules in Ironic. | 09:28 |
| abongale | As far as know we have merged inspection rules into ironic. What changes we need to make in terms of testing? do we need to add more tempest test to Ironic test? any idea? | 09:28 |
| abongale | https://bugs.launchpad.net/ironic/+bug/2105478 | 09:28 |
| masghar | abongale: cid (and others) will be able to shed more light on this | 10:41 |
| cid | Hey abongale, thanks for picking that up. So you will be adding tests to the Ironic Tempest Plugin (https://github.com/openstack/ironic-tempest-plugin) instead. | 10:58 |
| opendevreview | Queensly Kyerewaa Acheampongmaa proposed openstack/ironic master: Renamed conceptual sections in Ironic documentation https://review.opendev.org/c/openstack/ironic/+/946615 | 10:58 |
| opendevreview | Queensly Kyerewaa Acheampongmaa proposed openstack/ironic master: Renamed conceptual sections in Ironic documentation https://review.opendev.org/c/openstack/ironic/+/946615 | 11:00 |
| cid | abongale. The existing unit tests in Ironic are good reference points and you can also look at existing tests in the https://github.com/openstack/ironic-tempest-plugin/tree/master/ironic_tempest_plugin/tests/api/admin since I think that's where you'll be adding the additional tests in a new test_inspection_rules.py | 11:01 |
| abongale | Hi @cid, I will take a look. Currently going through Inspection docs, I will possibly reaching out again before adding adding new test . thank you. | 11:08 |
| cid | Good stuff, ++ | 11:11 |
| cid | queensly[m], <...2072353 and thinking of merging...> it's not very clear what you mean by merging it. Is the patch already up? | 11:17 |
| queensly[m] | The bug's description talks about combining with previous entry.... (full message at <https://matrix.org/oftc/media/v1/media/download/AWDSgZrsweKLOeI1WtpjbiXoEOyCcAPAY1n7mJMDqLLir7JqwEhNKIVieCp6vbAOmVysJsnnJycre4TZlniIp75CeWXT0fFgAG1hdHJpeC5vcmcvSFpyb1hDZWNma3VYbmZPT3ZnbWpMeWFE>) | 11:23 |
| abongale | @queensly[m]: just started going through docs. While adding new tests if required I will have consent/ discussion regarding it. Just to be clear | 11:23 |
| iurygregory | quite long message, for some reason is not showing up properly .-. | 11:24 |
| queensly[m] | abongale: are you referring to what cid said regarding what I shared? | 11:27 |
| abongale | @queensly[m] yeah, my bad I messed up reading from my mobile. ignore it. | 11:31 |
| cid | queensly[m], Oh, that makes sense; you meant it literally. So you can make the changes, push it to upstream and the correctness could be decided on a code review (?). | 11:35 |
| queensly[m] | iurygregory: I'm working on the bug https://bugs.launchpad.net/ironic/+bug/2072353 and it requires that the get_started.rst and architecture.rst have the heading renamed to "Overview of Ironic". So I have submitted a patch for it. | 11:35 |
| iurygregory | ack | 11:36 |
| queensly[m] | cid thank you. I have done that and awaiting review. | 11:37 |
| cid | ack | 11:41 |
| cardoe | There's https://review.opendev.org/c/openstack/networking-baremetal/+/945818 series which adds pre-commit which I believe finishes off the Epoxy goal for the set of repos we picked. | 12:43 |
| cardoe | https://review.opendev.org/c/openstack/ironic/+/944868 here was the WIP around dataclasses for inspection data. Did we want a spec for that? | 12:45 |
| vsaienko | cardoe, TheJulia, JayF: folks I've reported a bug about VTEP integration and tried to describe how its broken and why it does not work https://bugs.launchpad.net/ironic/+bug/2106460 please check we will need to decide what to do next. | 12:50 |
| freemanboss[m] | cid: on this bug https://bugs.launchpad.net/bugs/2106443 the `introspection` isn't present in the baremetal verb | 12:56 |
| freemanboss[m] | Whenever it's called it shows the help manual of the baremetal API. I then noticed the right one is allocation not introspection. It could be a mistake or it's outdated. Thank you | 12:59 |
| cid | freemanboss[m], I get that. Could it be `baremetal inspection` instead? The real bug is that absent verb is mentioned in the doc as the right path. But I don't think the right verb is `allocation` either, or is that what worked for you? | 13:03 |
| cid | I set the bug as invalid so that you don't go ahead with the fix yet, until it's clear what the right change is, if that makes sense. | 13:04 |
| cid | rpittau, might be of help here ^^ | 13:04 |
| freemanboss[m] | cid: yes the allocation worked for me. | 13:06 |
| freemanboss[m] | cid: alright I'll be expecting what rpittau: have to say about it. Thank you | 13:06 |
| cardoe | vsaienko: They're talking about you on the neutron call if you wanna join it. https://meetpad.opendev.org/apr2025-ptg-neutron | 13:18 |
| cardoe | They're wanting to understand your level of involvement and talking about giving you reviewer | 13:18 |
| rpittau | freemanboss[m], cid: the correct command is baremetal introspection and it's included in the python-ironic-inspector-client | 13:28 |
| JayF | rpittau: is that still true with integrated inspector? | 13:29 |
| rpittau | JayF: the subcommand is in the inspctor client, it has not been moved AFAIK | 13:29 |
| dtantsur | we need to drop mentions of `baremetal introspection list` from the docs | 13:30 |
| rpittau | heh yeah | 13:30 |
| kubajj | JayF, TheJulia: was just preparing an upgrade to Caracal and noticed the change https://review.opendev.org/c/openstack/ironic/+/903179 from my predecessor - would it be suitable to have it locked by a config option or was it other option you had in mind? | 13:32 |
| kubajj | just want to make sure I understand before I start rebasing it | 13:32 |
| shermanm | is there interest in extending networking-generic-switch to drive switches via http-based-apis, and not just ssh/netmiko? I've got some folks looking at adding support for the cumulus/nvidia api, and I was wondering if implementing a `http_devices` class alongside `netmiko_devices` made some kind of sense | 13:44 |
| * dtantsur has nothing against it | 13:44 | |
| shermanm | mostly asking cause I know there's been lots of talk about "where stuff should live" regarding switch configuration :) I've got a stub I can drop on opendev as a WIP | 13:46 |
| rpittau | shermanm: sounds like a good idea :) | 13:53 |
| opendevreview | Michael Sherman proposed openstack/networking-generic-switch master: add rest api interface for cumulus nvue https://review.opendev.org/c/openstack/networking-generic-switch/+/946630 | 13:55 |
| AmarachiOrdor[m] | Hey everyone! Just wanted to let you know the Ironic mentors sent an update via email.... (full message at <https://matrix.org/oftc/media/v1/media/download/AVOAWij7xqPPSmb_V-3wb98dk5_EznY0NqofWsdWOH3kv25FmsnULgCd-BLK9sCbt_NIdrA6vxK2FOENNAf5y4RCeWXcqAuAAG1hdHJpeC5vcmcvUkFYT2pGSENUS0RDdUF2eEtoSEpHZ21G>) | 13:57 |
| shermanm | besides "configure this switch", we've also got a use-case where openstack+ironic is living inside a SDN system operated by someone else, and it's easier for them to provide an http API endpoint with permissions, rather than driving something over SSH, or going through the effort of writing a whole ML2 driver just for it | 13:59 |
| rpittau | PTG second day starting now https://ptg.opendev.org/ptg.html | 13:59 |
| freemanboss[m] | Amarachi Ordor: thank you. I'll check it out too | 14:05 |
| cardoe | shermanm: So I think that would make sense. I think we'll want to really step up a level by using napalm/nornir (I forget the right one) so you can pick that your backend is "ssh" and it'll use netmiko | 14:40 |
| cardoe | JayF: wanted to revisit the prometheus exporter for a sec with you... you had said we blend hardware events and service events... is that in ironic-prometheus-exporter or was that in openstack-exporter? | 14:41 |
| JayF | please not during ptg, I don't have enough brainpower | 14:42 |
| cardoe | just dropping it here so I don't forget. | 14:42 |
| dtantsur | cardoe: it was about the Ironic notification system, which feeds ironic-prometheus-exporter | 15:03 |
| dtantsur | the problem IIRC is that we only have one notification bus, so everything goes there | 15:03 |
| cardoe | ah should we toss a topic about that? | 15:04 |
| opendevreview | Vasyl Saienko proposed openstack/networking-generic-switch master: Add devices API versioning https://review.opendev.org/c/openstack/networking-generic-switch/+/946639 | 15:13 |
| * freemanboss[m] uploaded an image: (97KiB) < https://matrix.org/oftc/media/v1/media/download/AX_uGPhz9kQdUsItvJT-ZgQMlL8bRaheqKuQoDkUKt1rZrTtGjYMXCg4AGMBFtacqyHx0kh3yGOKjPBSJXTlerJCeWXhA9dAAG1hdHJpeC5vcmcvclh1UVVRZkRQUGZkRkt2V2pqdUZZdnNp > | 15:14 | |
| TheJulia | might make sense to delineate it. I guess the current shape is also modeled around trying to get them all sent along since there was the classic metrics work which was done as well which blurred that line out | 15:14 |
| freemanboss[m] | rpittau: this is a reference image of how the introspection verb of baremetal failed | 15:14 |
| rpittau | freemanboss[m]: I think we mentioned already that the introspection sub command comes from the inspector-client, which is not installed by default | 15:16 |
| rpittau | the way forward would be removing the baremetal inspection mention in all documentation as it's deprecated and not used anymore | 15:16 |
| freemanboss[m] | Oh alright I understand better now | 15:17 |
| freemanboss[m] | Alright I will work on that way forward | 15:17 |
| vsaienko | shermanm: I think its a good idea to add api based switches into n-g-s it was designed for this, and there is example of VTEP capable switch https://review.opendev.org/c/openstack/networking-generic-switch/+/946558 | 15:51 |
| rpittau | cardoe: I think you crashed :) | 15:52 |
| * TheJulia falls over dead from board meeting | 15:54 | |
| JayF | Anyone interested in an ironic review party @ top of hour (10a PDT) | 16:14 |
| satoshi | I will join | 16:19 |
| * TheJulia checks schedule | 16:24 | |
| iurygregory | CFP for OIS Europe is open | 16:30 |
| *** diablo_rojo_phone is now known as Guest13175 | 16:32 | |
| * cid My IRC just disconnected for the first time in forever. Notes I will like to party too. | 16:36 | |
| TheJulia | Yes, submit to the CFP | 16:40 |
| TheJulia | It will be in France. no details on recommended hotels | 16:40 |
| * TheJulia asked this morning, and the answer was "working on it, soon!" | 16:40 | |
| freemanboss[m] | JayF: today right? I hope I can join also? | 16:40 |
| JayF | everyone is welcome, in 20 minutes | 16:41 |
| JayF | I'll toss a meetpad link in when the time comes | 16:42 |
| NobodyCam | Good Morning Ironic Folks | 16:58 |
| TheJulia | good morning NobodyCam | 16:58 |
| TheJulia | JayF: meetpad for this reviewjam? | 16:59 |
| JayF | yeah I'll be there in a moment | 16:59 |
| JayF | you can make one if you wanna start now :) | 16:59 |
| TheJulia | I'll make one | 16:59 |
| TheJulia | https://meetpad.opendev.org/apr2025-ptg-ironic | 16:59 |
| TheJulia | We can just reuse the PTG one | 16:59 |
| NobodyCam | O/ howdy all | 16:59 |
| opendevreview | Vasyl Saienko proposed openstack/networking-generic-switch master: Add devices API versioning https://review.opendev.org/c/openstack/networking-generic-switch/+/946639 | 17:02 |
| *** Guest13175 is now known as help | 17:03 | |
| *** help is now known as Guest13179 | 17:04 | |
| *** Guest13179 is now known as diablo_rojo_phone | 17:06 | |
| JayF | https://review.opendev.org/q/status:open+and+((project:%5Eopenstack/ironi.*)+or+(project:%5Eopenstack/sush.*)+or+(project:openstack/virtualpdu)+or+(project:openstack/virtualbmc)+or+(project:openstack/metalsmith)+or+(project:openstack/molteniron)+or+(project:openstack/networking-baremetal)+or+(project:openstack/networking-generic-switch)+or+(project:%5Eopenstack/python-ironi.*)+or+(project:openstack/tenks))+and+is:mergeable+and | 17:06 |
| JayF | +-reviewedby:self+and+-owner:self | 17:06 |
| JayF | https://review.opendev.org/q/status:open+and+((project:%5Eopenstack/ironi.*)+or+(project:%5Eopenstack/sush.*)+or+(project:openstack/virtualpdu)+or+(project:openstack/virtualbmc)+or+(project:openstack/metalsmith)+or+(project:openstack/molteniron)+or+(project:openstack/networking-baremetal)+or+(project:openstack/networking-generic-switch)+or+(project:%5Eopenstack/python-ironi.*)+or+(project:openstack/tenks))+and+is:mergeable | 17:06 |
| JayF | is the review link I'm about to share | 17:06 |
| cardoe | What's going on? | 17:06 |
| tkajinam | huge query magic !! | 17:06 |
| JayF | cardoe: review jam in https://meetpad.opendev.org/apr2025-ptg-ironic | 17:13 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: provide host_id to neutron early on https://review.opendev.org/c/openstack/ironic/+/946378 | 17:14 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Patch configdrive metadata https://review.opendev.org/c/openstack/ironic/+/946677 | 17:14 |
| opendevreview | Merged openstack/ironic master: Update documentation to retain relevant terms https://review.opendev.org/c/openstack/ironic/+/946504 | 18:06 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: trivial: follow-up on ftp enable/disable flag https://review.opendev.org/c/openstack/ironic/+/942357 | 18:46 |
| *** jamesdenton_ is now known as jamesdenton | 18:52 | |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix default IRONIC_DEFAULT_TRAITS setting https://review.opendev.org/c/openstack/ironic/+/942647 | 18:53 |
| opendevreview | Adeola Adesina proposed openstack/bifrost master: Add notes to provide more clarity for bifrost installation https://review.opendev.org/c/openstack/bifrost/+/946603 | 19:01 |
| opendevreview | Merged openstack/sushy-tools master: Pass image ID to _attempt_delete_image_local_file https://review.opendev.org/c/openstack/sushy-tools/+/946326 | 19:02 |
| opendevreview | Merged openstack/sushy-tools master: os-vmedia: Ensure delay-eject is cleared on insert https://review.opendev.org/c/openstack/sushy-tools/+/946503 | 19:04 |
| opendevreview | Julia Kreger proposed openstack/ironic master: ci: restrict multinode job to multinode tests https://review.opendev.org/c/openstack/ironic/+/942223 | 19:15 |
| JayF | successful review jam! https://usercontent.irccloud-cdn.com/file/PmcADCkB/open-the-gate.png | 19:20 |
| opendevreview | Merged openstack/ironic-python-agent master: Ironic Config Intergration for IPA https://review.opendev.org/c/openstack/ironic-python-agent/+/944951 | 19:34 |
| opendevreview | Merged openstack/ironic-python-agent master: Implement manual cleaning for ContainerHardwareManager https://review.opendev.org/c/openstack/ironic-python-agent/+/945862 | 19:34 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: remove duplicated rbac jobs https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/946582 | 19:37 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Remove legacy standalone jobs from test execution https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/946592 | 19:37 |
| opendevreview | Verification of a change to openstack/ironic-tempest-plugin master failed: CI: Increment jobs for 2025.1 release https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/946593 | 19:37 |
| opendevreview | Merged openstack/ironic master: Add image build method for verified OS. https://review.opendev.org/c/openstack/ironic/+/946318 | 19:41 |
| opendevreview | Merged openstack/ironic-python-agent master: Run UT of metricis_lib https://review.opendev.org/c/openstack/ironic-python-agent/+/946229 | 20:02 |
| TheJulia | Hey, anyone used Octavia + Ironic Baremetal Instances deployed via Nova ? | 20:05 |
| JayF | rm_work: ^ did we have that working at yahoo? | 20:12 |
| JayF | TheJulia: | 20:12 |
| JayF | Adam wouldbe a good person to ask | 20:12 |
| TheJulia | the going suspicion that it might just work with Amphora, but unknown with OVN | 20:13 |
| TheJulia | so ???? | 20:13 |
| JayF | if we ran it at the big purp, it was 100% amphorae based | 20:13 |
| johnsom | Ironic Baremetal instances as backend servers should just work | 20:13 |
| johnsom | Can't tell you on the OVN side however, there a bunch of gotchas with OVN and physical networks. | 20:14 |
| TheJulia | Yeah, that is kind of my feeling as well | 20:15 |
| johnsom | Heck, you can add openstack.org IPs to an amphora load balancer and as long as there is a route, it will work. | 20:16 |
| TheJulia | In that case, I would semi-expect ovn to work, but likely need to be explicitly validated | 20:16 |
| johnsom | +1 for testing when using the OVN provider | 20:17 |
| * TheJulia gives eavesdrop a minute or two to write updated logs | 20:18 | |
| cardoe | So kinda works. | 20:21 |
| cardoe | They need to fix the HA_Chassis_Group thing in Neutron | 20:21 |
| cardoe | Cause HA is dorked up | 20:21 |
| cardoe | But Octavia + OVN seems to work | 20:21 |
| TheJulia | So, I think that has been a known thing | 20:22 |
| cardoe | Yeah it is | 20:22 |
| cardoe | Just saying you don't have any kind of HA. That one OVN / neutron network node is gonna be your single point of failure for the load balancer | 20:22 |
| cardoe | TheJulia: keekz's is literally putting together that into our deployed case. | 20:23 |
| cardoe | I've done it by some hand wave. | 20:24 |
| johnsom | Octavia has native SR-IOV now as well, so if you want to bypass OVN for your baremetal instances, you have an option. | 20:26 |
| cardoe | Not understanding how that plays together? | 20:28 |
| johnsom | With SR-IOV the load balanced traffic never goes through OVN | 20:29 |
| johnsom | So, you can keep the traffic on your hardware switches | 20:29 |
| opendevreview | Merged openstack/ironic-python-agent master: Remove agent_token_required upgrade knob https://review.opendev.org/c/openstack/ironic-python-agent/+/941566 | 20:38 |
| TheJulia | errr. launchpad. I guess I can do that tomorrow :) | 20:43 |
| opendevreview | Merged openstack/ironic master: devstack: network simulator support for sonic https://review.opendev.org/c/openstack/ironic/+/945726 | 20:49 |
| opendevreview | Merged 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 | 20:49 |
| opendevreview | Merged openstack/ironic master: refactor: split glance handling into it's own method https://review.opendev.org/c/openstack/ironic/+/941010 | 20:49 |
| opendevreview | Merged openstack/ironic master: Don't log PXE when your not doing PXE. https://review.opendev.org/c/openstack/ironic/+/942135 | 20:49 |
| cardoe | So that glance split, wondering if the other glance parameters like what Anaconda uses should have been included in there. | 20:51 |
| cardoe | johnsom: I guess I'm not understanding how SR-IOV plays into that. Since I thought it was just for partitioning a NIC | 20:52 |
| johnsom | cardoe Yes, and Octavia Amphora now support connecting those SR-IOV virtual functions directly into the load balancer. Amphora VM->VF->hardware switch, no OVN in the path to your baremetal instances. | 20:55 |
| cardoe | ohhh now I follow. I was thinking of the OVN Octavia without amphora | 20:56 |
| johnsom | Oh, no, the OVN provider driver doesn't support it. | 20:56 |
| cardoe | https://docs.openstack.org/ovn-octavia-provider/latest/admin/driver.html | 20:58 |
| johnsom | Yeah, I don't think that has been kept up to date. We do maintain this however: https://docs.openstack.org/octavia/latest/user/feature-classification/index.html | 21:00 |
| cardoe | So you're saying Octavia doesn't support OVN without a VM? | 21:03 |
| johnsom | Sorry I may have confused you. There are two provider drivers for Octavia maintained by OpenStack: The Amphora driver (VMs) and the OVN-provider. The Amphora driver does not have a dependency on the ML2 used (linux bridge, OVS, OVN, etc.) it will work with all of them. The OVN-provider requires an ML2 of OVN. The Amphora driver supports SR-IOV VFs, the OVN-provider does not. | 21:06 |
| cardoe | ah | 21:21 |
| opendevreview | Verification of a change to openstack/ironic master failed: trivial: follow-up on ftp enable/disable flag https://review.opendev.org/c/openstack/ironic/+/942357 | 21:46 |
| TheJulia | dtantsur: https://bugs.launchpad.net/ironic/+bug/2106550 following up for when braincells are handy :) | 22:32 |
| opendevreview | Merged openstack/ironic master: Fix default IRONIC_DEFAULT_TRAITS setting https://review.opendev.org/c/openstack/ironic/+/942647 | 22:36 |
| opendevreview | Merged openstack/ironic master: ci: restrict multinode job to multinode tests https://review.opendev.org/c/openstack/ironic/+/942223 | 22:51 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: WIP: initial ssh socat interface ideas https://review.opendev.org/c/openstack/ironic-specs/+/946723 | 23:25 |
| TheJulia | JayF: I guess I'm sort of not firmly sold on much beyond managing some base configuration, but that being said locking the sshd to a container is likely a good idea while also restricting it's shape and operation | 23:33 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
