| rpittau | good morning ironic! o/ | 07:23 |
|---|---|---|
| cid | JayF: Thanks | 07:44 |
| cid | When changes get abandoned, or goes from assigned to someone to nobody, shouldn't the status also go from "In Progress" to something else? Currently it does not do it automatically and can't also be done manually ( I couldn't). | 08:38 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy master: Name is not mandatory for Storage/StorageControllers https://review.opendev.org/c/openstack/sushy/+/925853 | 09:34 |
| dtantsur | discovered on iLO 5 ^^ | 09:34 |
| dtantsur | cid: we need to add you to some group so that you can change the status.. | 10:37 |
| dtantsur | yes, it does not happen automatically | 10:37 |
| cid | dtantsur: Oh, it makes sense that it's not automatic because that will require to know what status to return it to. By the way, only add me to such group if I'm a good fit yet, I was mostly wondering if that possibility was there at all. | 10:43 |
| dtantsur | I think you are but I'll let JayF decide | 10:43 |
| cid | ++ | 10:43 |
| iurygregory | good morning ironic | 11:07 |
| iurygregory | dtantsur, approved | 11:08 |
| dtantsur | thx | 11:09 |
| adam-metal3 | hello Ironic o/ | 11:54 |
| opendevreview | Merged openstack/sushy master: Name is not mandatory for Storage/StorageControllers https://review.opendev.org/c/openstack/sushy/+/925853 | 11:54 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy stable/2024.1: Name is not mandatory for Storage/StorageControllers https://review.opendev.org/c/openstack/sushy/+/925862 | 11:56 |
| adam-metal3 | dtantsur, rpittau, JayF, TheJulia: Thanks for the feedback regarding the encryption proposal I won't answer the comments right away I think I have a way to create something that answers all the questions e.g. LVM support, flexibility, iterative implementation etc but first I will try to create an implementation for the whole disk image workflow to see if my idea works and then I will update the proposal accordingly. | 11:58 |
| dtantsur | ++ good plan | 11:59 |
| *** tkajinam is now known as Guest2415 | 12:52 | |
| *** tkajinam is now known as Guest2416 | 13:00 | |
| TheJulia | ++ | 13:19 |
| iurygregory | so I've finished my review of https://review.opendev.org/c/openstack/ironic/+/922142 and I will +2, rpittau also provided +2, since the patch is huge I would appreciated if you could also check things JayF o/ | 13:33 |
| iurygregory | or any other core =) | 13:34 |
| opendevreview | Merged openstack/sushy stable/2024.1: Name is not mandatory for Storage/StorageControllers https://review.opendev.org/c/openstack/sushy/+/925862 | 13:37 |
| opendevreview | Merged openstack/sushy-tools master: Update irrelevant-files for sushy-jobs https://review.opendev.org/c/openstack/sushy-tools/+/925069 | 13:44 |
| opendevreview | Merged openstack/ironic master: Deprecate [agent]/manage_agent_boot https://review.opendev.org/c/openstack/ironic/+/923328 | 14:00 |
| cardoe | Someone mentioned that OpenStack overall is making strides with using an OpenAPI generator... anyone have a reference to that? | 14:03 |
| opendevreview | Merged openstack/ironic master: CI: set tftp folder permission https://review.opendev.org/c/openstack/ironic/+/923590 | 14:11 |
| TheJulia | I don't think it would be a generator, but a definition... I think | 14:17 |
| opendevreview | cid proposed openstack/ironic master: docs-audit-2024: Use gerunds for task headings https://review.opendev.org/c/openstack/ironic/+/925129 | 14:18 |
| opendevreview | cid proposed openstack/ironic master: docs-audit-2024: Use a description for concept headings https://review.opendev.org/c/openstack/ironic/+/925692 | 15:33 |
| cid | iurygregory: thanks | 15:50 |
| TheJulia | We really need to fix our teardown of devstack with the ironic plugin... | 16:01 |
| rpittau | good night! o/ | 16:02 |
| JayF | ++ | 16:03 |
| TheJulia | I filed a new bug because I switched back to doing an ipmi configuration locally, likely a quick item | 16:04 |
| opendevreview | Merged openstack/ironic master: Self-Service via Runbooks https://review.opendev.org/c/openstack/ironic/+/922142 | 18:03 |
| opendevreview | Jay Faulkner proposed openstack/python-ironicclient master: Add CLI support for runbooks https://review.opendev.org/c/openstack/python-ironicclient/+/924101 | 18:47 |
| opendevreview | cid proposed openstack/ironic master: Follow up to the runbooks change (#922142) https://review.opendev.org/c/openstack/ironic/+/925917 | 19:26 |
| cid | o/ | 19:30 |
| iurygregory | trying to remember, but for iLO6 redfish should just work right? at least I remember something about this | 20:35 |
| TheJulia | ... i... think so? | 21:20 |
| JayF | iurygregory: very yes | 22:29 |
| JayF | iurygregory: If that's ever not true in your experience, please loop me in on it | 22:29 |
| iurygregory | yeah, we don't have machines with iLO6 downstream to test, the person only said it fails to load the virtual media and he suspects the API has changed (without providing any context on what version of our product they are trying to use XD) | 22:29 |
| JayF | My downstream uses ilo6+redfish, but does not currently utilize virtual media. | 22:30 |
| JayF | However I'd be interested to see a detailed bug around that if broken | 22:30 |
| iurygregory | ++ yeah | 22:30 |
| JayF | and can perhaps help with hardware access if there's version overlap | 22:30 |
| iurygregory | I asked for logs to see if I can check the errors ironic would show etc | 22:30 |
| iurygregory | to see if they are also using latest code we had (since we had the Managers vs System modification for Vmedia) | 22:31 |
| iurygregory | and maybe they don't have the code in the version they are using etc | 22:31 |
| JayF | ah, good point | 22:32 |
| iurygregory | people always expect we can say something when they only tell us "Do we support iLO6? it fails to load the virtual media, so I suspect the API has changed" =D | 22:32 |
| JayF | They are hoping "oh, just openstack baremetal node set --driver-info actually_work=true broken-node" | 22:33 |
| JayF | which to be fair, is sometimes our answer | 22:33 |
| iurygregory | yeah! | 22:33 |
| JayF | I forget what the argument is | 22:33 |
| JayF | persistent_boot_device_settings=true or something like that | 22:34 |
| JayF | it covers up so many ills :D | 22:34 |
| TheJulia | My big worry with newer hpe gear is people order openbmc and think it works like I’ll | 22:34 |
| TheJulia | Err work like ilo | 22:35 |
| iurygregory | TheJulia, they can choose between "ilo" vs "openbmc" implementation in the gear? | 22:35 |
| iurygregory | =O | 22:35 |
| TheJulia | Yup, it is an option on that gear…. | 22:35 |
| iurygregory | interesting | 22:35 |
| iurygregory | hopefully there is way to identify things (UI or via some redfish response) in case we need to deal with something specific | 22:36 |
| TheJulia | I don’t think we ever got an answer | 22:37 |
| JayF | AIUI it's basically two entirely separate methods | 22:38 |
| JayF | I'd be surprised if they even show up as the same model in redfish | 22:38 |
| JayF | because I believe if they have an ilo, it actually has an extra board in it | 22:38 |
| iurygregory | JayF, I hope so :D | 22:38 |
| TheJulia | What the hpe folks said was, for some gear it was just different firmware | 22:38 |
| TheJulia | So it may vary a lot | 22:39 |
| iurygregory | yeah | 22:39 |
| JayF | ahh, that's not the case for all their gear, I know at least of one machine where an ilo literally involves extra hardware vs a non-ilo version | 22:40 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!