| opendevreview | melanie witt proposed openstack/nova master: Add [workarounds]unified_limits_permissive_enforcement https://review.opendev.org/c/openstack/nova/+/924025 | 03:27 |
|---|---|---|
| opendevreview | melanie witt proposed openstack/nova master: Add [workarounds]unified_limits_permissive_enforcement https://review.opendev.org/c/openstack/nova/+/924025 | 06:53 |
| *** bauzas_ is now known as bauzas | 06:56 | |
| gibi | artom: sean-k-mooney: bauzas: you probably remember when we looked at why unshelving to an ironic host fails with the cryptic error as Destination.node is not set while unshelving to a libvirt host works even if the compute api only sets Destination.host there too. The answer is in https://bugs.launchpad.net/nova/+bug/2075504/comments/3 | 08:25 |
| gibi | basically the scheduler fills Destination.node based on .host if the mapping is 1:1 | 08:26 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Fix PCI passthrough cleanup on reschedule https://review.opendev.org/c/openstack/nova/+/926407 | 08:45 |
| sean-k-mooney[m] | i didnt think shelve was supported for ironic | 10:00 |
| sean-k-mooney[m] | i know unselve could work but i would expect that to not be supproted as it would require chaning the flavor | 10:01 |
| sean-k-mooney[m] | since ironic nodes should bnot have cpu, ram or disk | 10:01 |
| sean-k-mooney[m] | and should be using a flavor with only a custom resource class | 10:05 |
| sean-k-mooney[m] | so placement should retrun no allocation for the host | 10:05 |
| gibi | sean-k-mooney: yes, it is not supported but it fails in a very stange way | 10:09 |
| gibi | so I have that cosmetic bug open to fail it in a more clear way | 10:09 |
| gibi | and while we discussed that we had an open question how don't unshelve to libvirt node does not fail with the same reason, the missing Destination.node field | 10:10 |
| sean-k-mooney[m] | there is also a tempet/job config bug. there is a temest setting for pining which host is used for the source and dest node for migrations | 10:10 |
| gibi | and yesterday I found the reason | 10:10 |
| sean-k-mooney[m] | we either are not setting it and shelve tests dont respect it or we need to add that to the job config | 10:10 |
| gibi | sean-k-mooney: yeah, also James is fixing the host selection logic for unshelve to host | 10:10 |
| gibi | in tempest | 10:11 |
| sean-k-mooney[m] | that or we need to add a shelve sepcific config for that | 10:11 |
| sean-k-mooney[m] | ok | 10:11 |
| sean-k-mooney[m] | we should proably return an exception in the 1:many case | 10:11 |
| gibi | yeah that would probably make sense otherwise the scheduler blows up later anyway | 10:12 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Fix PCI passthrough cleanup on reschedule https://review.opendev.org/c/openstack/nova/+/926407 | 10:13 |
| *** bauzas_ is now known as bauzas | 10:32 | |
| opendevreview | Takashi Natsume proposed openstack/nova-specs master: Create specs directory for 2025.1 Epoxy https://review.opendev.org/c/openstack/nova-specs/+/924227 | 12:54 |
| opendevreview | sean mooney proposed openstack/nova master: split monkey_patching form import https://review.opendev.org/c/openstack/nova/+/922425 | 13:31 |
| opendevreview | sean mooney proposed openstack/nova master: refactor cmd dir for non eventlet console_scripts https://review.opendev.org/c/openstack/nova/+/904424 | 13:31 |
| opendevreview | sean mooney proposed openstack/nova master: Remove nova debugger funcitonality https://review.opendev.org/c/openstack/nova/+/922496 | 13:31 |
| opendevreview | sean mooney proposed openstack/nova master: Add IO Thread Pool Executor https://review.opendev.org/c/openstack/nova/+/922497 | 13:31 |
| opendevreview | sean mooney proposed openstack/nova master: remove eventlet.tpool https://review.opendev.org/c/openstack/nova/+/905287 | 13:31 |
| opendevreview | sean mooney proposed openstack/nova master: use futureist for scatter_gather https://review.opendev.org/c/openstack/nova/+/905284 | 13:31 |
| opendevreview | sean mooney proposed openstack/nova master: use futerist for RBD calls https://review.opendev.org/c/openstack/nova/+/917962 | 13:31 |
| sean-k-mooney | stephenfin: gibi: ^ rebased and adress the current out standing issues if you have time to review | 13:31 |
| stephenfin | aclk | 13:32 |
| opendevreview | sean mooney proposed openstack/nova master: add initial healthcheck support https://review.opendev.org/c/openstack/nova/+/825015 | 13:55 |
| opendevreview | sean mooney proposed openstack/nova master: add healthcheck manager to manager base https://review.opendev.org/c/openstack/nova/+/827844 | 13:55 |
| opendevreview | sean mooney proposed openstack/nova master: add healthcheck tracker to nova context https://review.opendev.org/c/openstack/nova/+/829468 | 13:55 |
| opendevreview | sean mooney proposed openstack/nova master: add healthcheck endpoint to proxy commands https://review.opendev.org/c/openstack/nova/+/830703 | 13:55 |
| opendevreview | sean mooney proposed openstack/nova master: [WIP] add iniital health check to compute agent https://review.opendev.org/c/openstack/nova/+/906911 | 13:55 |
| opendevreview | sean mooney proposed openstack/nova master: [WIP] add libvirt connection healtcheck https://review.opendev.org/c/openstack/nova/+/907424 | 13:55 |
| sean-k-mooney | thats ^ just a rebase i need to go though them and see what review feedback is present | 13:55 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Fix PCI passthrough cleanup on reschedule https://review.opendev.org/c/openstack/nova/+/926407 | 14:10 |
| opendevreview | Zhang Hua proposed openstack/nova master: BlockDeviceMapping object has no attribute copy https://review.opendev.org/c/openstack/nova/+/920374 | 14:34 |
| opendevreview | Arnaud Morin proposed openstack/nova master: libvirt: Log exception if failure checking block job https://review.opendev.org/c/openstack/nova/+/907720 | 15:07 |
| gibi | artom: sean-k-mooney: the running CI looking green on https://review.opendev.org/c/openstack/nova/+/926407 and I fixed up all the comments. | 16:27 |
| gibi | that is my last contribution for this week and next week I'm off. | 16:27 |
| gibi | see you on the 26th | 16:27 |
| sean-k-mooney | gibi: thanks gibi im pretty happy with that patch | 17:19 |
| sean-k-mooney | ill see if i can chase other to review while your on pto, enjoy | 17:19 |
| *** bauzas_ is now known as bauzas | 18:55 | |
| *** bauzas_ is now known as bauzas | 20:12 | |
| opendevreview | Merged openstack/nova master: api: Add response body validation helper https://review.opendev.org/c/openstack/nova/+/915741 | 22:03 |
| *** bauzas_ is now known as bauzas | 22:04 | |
| opendevreview | Merged openstack/nova master: hardware: Correct log https://review.opendev.org/c/openstack/nova/+/926223 | 22:35 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!