| opendevreview | minwoo seo proposed openstack/ironic master: Add `api-call` action for ironic inspection rule https://review.opendev.org/c/openstack/ironic/+/946741 | 03:26 |
|---|---|---|
| opendevreview | minwoo seo proposed openstack/ironic master: Add `api-call` action for ironic inspection rule https://review.opendev.org/c/openstack/ironic/+/946741 | 03:44 |
| rpittau | good morning ironic! o/ | 06:41 |
| opendevreview | cid proposed openstack/ironic master: Log executed steps during cleaning/servicing/deploy https://review.opendev.org/c/openstack/ironic/+/952637 | 06:56 |
| queensly[m] | Good morning | 07:36 |
| *** tosky is now known as Guest18397 | 07:37 | |
| *** tosky_ is now known as tosky | 07:37 | |
| opendevreview | cid proposed openstack/ironic master: Log executed steps during cleaning/servicing/deploy https://review.opendev.org/c/openstack/ironic/+/952637 | 08:18 |
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: Add Tempest tests for inspection rules in Ironic https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/951761 | 08:29 |
| iurygregory | good morning ironic | 12:53 |
| iurygregory | rpittau, if we need to change something in a schema https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/modules/redfish/firmware_utils.py#L78 do you know if we need to bump API version? | 12:54 |
| rpittau | iurygregory: ironic API version? I don't think so | 12:55 |
| iurygregory | yeah, I was wondering if would be required | 12:56 |
| rpittau | for an upgrade no, for a new schema or interface yes | 12:56 |
| rpittau | at least that's what I would expect | 12:56 |
| iurygregory | ack | 12:56 |
| iurygregory | will wait for others to see what they think, tks! | 12:57 |
| rpittau | no problem | 12:57 |
| JayF | iurygregory: it can go either way, a microversion also gives the change more visibility in general | 14:03 |
| JayF | I would say to microvision it, but it's not a strong opinion | 14:04 |
| JayF | It can just get hard to describe to someone when something was added if you don't pop it into a new micro version | 14:04 |
| iurygregory | if we add microversion we would need to "block" nic updates in case it's using the wrong version right? | 14:08 |
| JayF | If someone's running the wrong version of ironic, it's going to be blocked anyway with a less useful error than your version is too low | 14:17 |
| JayF | Anytime our software itself can tell the operator how to fix their own situation I leaned that way, that's the main reason why I like micro versions for this stuff | 14:18 |
| JayF | I filed https://bugs.launchpad.net/sushy/+bug/2114989 -- I expect this to be a big thing and not a quick fix, borderline RFE + Spec needed | 14:47 |
| iurygregory | vendors and their interesting implementation of standards :D | 15:10 |
| iurygregory | omg, separate information in Systems<this is fine gif> | 15:10 |
| JayF | I've been told to expect this behavior from multiple vendors coming very soon. | 15:14 |
| JayF | iurygregory: btw; how is iDRAC 10 stuff going? | 15:14 |
| iurygregory | JayF, we are going to have QE testing things now to see how it goes with our product | 15:22 |
| iurygregory | probably next week, I will share updates after their result | 15:22 |
| JayF | Thanks. I won't be here to see those updates but it sounds like it's reasonable to assume when I reappear in 6 weeks it'll work :D | 15:25 |
| opendevreview | cid proposed openstack/ironic master: [WIP] Migrate conductor threads to a thread pool https://review.opendev.org/c/openstack/ironic/+/952939 | 16:59 |
| opendevreview | Merged openstack/ironic master: Add `api-call` action for ironic inspection rule https://review.opendev.org/c/openstack/ironic/+/946741 | 17:15 |
| opendevreview | Allain Legacy proposed openstack/ironic-specs master: Add support for standalone network configuration https://review.opendev.org/c/openstack/ironic-specs/+/952126 | 17:54 |
| opendevreview | Allain Legacy proposed openstack/ironic-specs master: Add support for standalone network configuration https://review.opendev.org/c/openstack/ironic-specs/+/952126 | 18:08 |
| opendevreview | Allain Legacy proposed openstack/ironic-specs master: Add support for standalone network configuration https://review.opendev.org/c/openstack/ironic-specs/+/952126 | 18:27 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/952939 unit tests clearly need some fixing ... but I'm shocked for the most part the functional tests appear OK | 19:23 |
| JayF | unit tests are a straight timeout | 19:24 |
| JayF | probably missing mocks that need fixin? | 19:24 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!