| opendevreview | Jay Faulkner proposed openstack/ironic master: Implement periodic task for populating shards tbl https://review.opendev.org/c/openstack/ironic/+/864954 | 00:11 |
|---|---|---|
| TheJulia | sooooo... hjensas 1324 nodes/second on the chassis_uuid patch. did you benchmark it locally? | 00:51 |
| TheJulia | prior test was close to 2000/node, but that is literally the largest I've ever seen, ever and makes me think it is just ci node variability | 00:56 |
| TheJulia | another run looks like it was 1282.6/second | 00:58 |
| TheJulia | of course, that query pattern only runs nova's quiery hit | 00:58 |
| TheJulia | query hit | 00:58 |
| TheJulia | full list would be where the benefit exists | 00:58 |
| *** akahat|ruck is now known as akahat | 04:12 | |
| opendevreview | kamlesh chauvhan proposed openstack/sushy master: Retry on iDRAC SYS518 errors for all requests https://review.opendev.org/c/openstack/sushy/+/864911 | 05:52 |
| opendevreview | kamlesh chauvhan proposed openstack/sushy master: Retry on iDRAC SYS518 errors for all requests https://review.opendev.org/c/openstack/sushy/+/864911 | 05:52 |
| arne_wiebalck | Good morning, Ironic! | 07:13 |
| rpittau | good morning ironic! Happy Friday! o/ | 08:02 |
| janders | hey arne_wiebalck rpittau and Ironic o/ | 09:33 |
| hjensas | TheJulia: I had not benchmarked it locally. It is slower, I think a full list would benefit ... set the change to WIP and will investigate more. | 10:48 |
| iurygregory | good morning Ironic | 11:34 |
| arne_wiebalck | hey rpittau hjensas iurygregory o/ | 12:21 |
| dtantsur | my attempt to run the metal3 CI is now kernel panicking on inspection, and I declare it a great progress :D | 13:20 |
| TheJulia | hjensas: yeah... might as well also benchmark a full list as long as it is not going to take an absurd amount of time | 13:37 |
| TheJulia | dtantsur: that *is* progress! | 13:39 |
| hjensas | TheJulia: not sure a full list is faster either. I'd expect to see it on a basic 'node list --long' from cli, even that is marginally slower-to-no-improvment. | 13:42 |
| *** dmellado_ is now known as dmellado | 14:03 | |
| TheJulia | so i've been chatting with Kirill_ regarding getting graphical console stuffs finally working in nova... and since I looked at the nova code for this stuffs, it has been largely... completely... rewritten. | 14:19 |
| TheJulia | For security isolation I think, but the idea has floated, what if we had our own "service proxy" that could bridge the gap between nova->ironic>bmc | 14:20 |
| TheJulia | that way the nova internals don't need internal details of the BMC network | 14:20 |
| TheJulia | thoughts? | 14:20 |
| TheJulia | the challenge is basically we need to get a credential back, and nova relies upon client authentication with a token -> proxy-> to backend service utilizing it's running context | 14:25 |
| opendevreview | Merged openstack/ironic-inspector stable/ussuri: Use port.id instead of port.uuid https://review.opendev.org/c/openstack/ironic-inspector/+/863627 | 14:40 |
| dtantsur | TheJulia: no objections, but why cannot it be a part of ironic? | 14:43 |
| TheJulia | oh, I think that is where Kirill_ is kind of leaning | 14:45 |
| TheJulia | I'm encouraging prototyping and a spec (unfortunately) since this is kind of major | 14:45 |
| TheJulia | but an awesome addition and hopefully means less breakage for downstream patch consumers who have done this | 14:45 |
| TheJulia | o/ sdanni | 14:48 |
| opendevreview | Verification of a change to openstack/ironic-inspector stable/ussuri failed: Fix port id vs uuid in ValidateInterfacesHook https://review.opendev.org/c/openstack/ironic-inspector/+/863626 | 14:51 |
| sdanni | Hi TheJulia! | 14:54 |
| opendevreview | Harald Jensås proposed openstack/ironic master: Add ports statistics to tools/benchmark scripts https://review.opendev.org/c/openstack/ironic/+/864762 | 14:56 |
| dtantsur | TIL https://ipxe.org/howto/dhcpd#speeding_up_dhcp | 14:57 |
| dtantsur | anyone ever seen ^^^? | 14:57 |
| TheJulia | Neato…. No, I had not, at least not in recent memory | 15:00 |
| TheJulia | In an OpenStack context with spine/leaf stuffs… we wouldn’t be able to set that… really | 15:01 |
| TheJulia | Really | 15:01 |
| Kirill_ | thanks everyone, will be back with spec | 15:02 |
| TheJulia | sdanni: I noticed you had a patch against ipa which needs to be rebased :) | 15:03 |
| sdanni | TheJulia: will do! | 15:03 |
| TheJulia | whee... doing expenses and three difference currencies are on a receipt \o/ #success | 16:11 |
| TheJulia | okay, actually 2 | 16:14 |
| dtantsur | 2 is not bad as well (3 would be awesome) | 16:16 |
| rpittau | bye, have a great weekend! o/ | 16:40 |
| dtantsur | o/ | 16:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!