| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-01-10 | |||
| 17:27:40 | bauzas | Placement aggregates relate to resource providers | |
| 17:27:40 | sean-k-mooney | right but this spec neverf mention compute services | |
| 17:27:43 | sean-k-mooney | only compute nodes | |
| 17:27:52 | bauzas | Nova aggregates related to nova-compute services | |
| 17:27:57 | sean-k-mooney | so if i read the spec the intent seams to be to mapp compute nodes | |
| 17:28:00 | bauzas | relate* | |
| 17:28:20 | bauzas | sean-k-mooney: the intent of that spec was to put the aggregates table to the API DB | |
| 17:28:28 | bauzas | not about desiging placement aggs | |
| 17:29:58 | sean-k-mooney | i am not talking about placment aggs at all | |
| 17:30:04 | sean-k-mooney | just nova | |
| 17:30:18 | sean-k-mooney | im also reading our newton docs https://docs.openstack.org/nova/newton/aggregates.html | |
| 17:30:26 | sean-k-mooney | whcih again talk about comptue nodes not compute services | |
| 17:30:54 | bauzas | sure | |
| 17:31:07 | bauzas | we also talk about ephemeral storage, remember ? :) | |
| 17:31:29 | bauzas | but the fact is, I know how scheduling works and I know what we return | |
| 17:31:51 | sean-k-mooney | yep but i know we have downstream docs about addign ironic does to host aggretes by uuid and i have seen that used in other place too | |
| 17:32:06 | bauzas | we return a destination which is a hostame for calling the RPC API | |
| 17:32:54 | sean-k-mooney | we return more then that | |
| 17:33:31 | sean-k-mooney | or at least we provide more then that to the filters | |
| 17:33:40 | sean-k-mooney | its not so much about what we return to the conductor | |
| 17:34:33 | opendevreview | Merged openstack/nova master: Support evacuate with PCI in placement https://review.opendev.org/c/openstack/nova/+/854615 | |
| 17:37:01 | sean-k-mooney | so lookign at the newton code it is in terms of the compute service | |
| 17:37:10 | sean-k-mooney | we do https://github.com/openstack/nova/blob/808d36475103e373f1deb3344b6829ce68d6cdd5/nova/db/sqlalchemy/api.py#L533-L542 | |
| 17:38:32 | bauzas | sean-k-mooney; I need to stop today's work :( | |
| 17:39:19 | sean-k-mooney | bauzas: that ok | |
| 17:39:41 | sean-k-mooney | anyway we need to look at the patch carfullly as it sound more liek a feature then a bug and im not sure it work fully | |
| 17:40:06 | bauzas | sean-k-mooney: I was about to say it | |
| 17:40:19 | bauzas | looks to me a new behavioural support | |
| 17:40:23 | bauzas | hence not a bug | |
| 17:40:32 | bauzas | and I'd prefer us to design it correctly | |
| 17:40:39 | bauzas | as there could be corner cases | |
| 17:41:03 | sean-k-mooney | well i think there are defintly usecase for supprotiing ironic with host aggreates | |
| 17:41:23 | sean-k-mooney | but ya i think it needs a spec espcislly with the other ironic work that is in flight | |
| 17:41:56 | bauzas | I'll quickly add a comment on that patch into that direction | |
| 17:42:15 | bauzas | I won't use the -2 hammer yet as it's late | |
| 18:19:13 | opendevreview | Kirill proposed openstack/nova-specs master: new spec: support of vnc console for ironic https://review.opendev.org/c/openstack/nova-specs/+/863773 | |
| 18:31:18 | opendevreview | Dan Smith proposed openstack/nova master: Add virt/node module for stable uuids https://review.opendev.org/c/openstack/nova/+/863915 | |
| 18:31:19 | opendevreview | Dan Smith proposed openstack/nova master: Pass service ref to init_host(), if exists https://review.opendev.org/c/openstack/nova/+/863916 | |
| 18:31:19 | opendevreview | Dan Smith proposed openstack/nova master: Add get_available_node_uuids() to virt driver https://review.opendev.org/c/openstack/nova/+/863917 | |
| 18:31:20 | opendevreview | Dan Smith proposed openstack/nova master: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | |
| 18:31:20 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Make resource tracker use UUIDs instead of names https://review.opendev.org/c/openstack/nova/+/863919 | |
| 18:31:21 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Detect host renames and abort startup https://review.opendev.org/c/openstack/nova/+/863920 | |
| 18:31:53 | dansmith | gibi: I still need to do the undelete explicit test in ^ so I marked that penultimate one as WIP with a note, I just want to get a bunch of work pushed up for insurance | |
| 18:32:04 | gibi | dansmith: ack | |
| 18:42:28 | artom | Besides the tox thing (that is now fixed?) there are no further known gate issues? | |
| 18:42:57 | sean-k-mooney | beseide the multiple tox thigns not that comes to mind | |
| 18:43:25 | sean-k-mooney | artom: are you seing somthing or wonderifn if you can recheck | |
| 18:43:32 | artom | Latter | |
| 18:43:41 | sean-k-mooney | then i belive yes | |
| 18:43:57 | artom | ✊✊ | |
| 18:44:02 | sean-k-mooney | on master at least | |
| 18:44:22 | sean-k-mooney | i dont know if all the pin patches are mergd on all the sable branches | |
| 18:45:04 | gmann | pin patches only required on python-novaclient and others are taken care by the pinning in common place https://review.opendev.org/q/I442568a5f5900e593feb2b5527109e0aa79e5aa7+status:open | |
| 18:45:58 | gmann | nova stable branches are all green https://review.opendev.org/q/topic:tox4-pin-testing | |
| 18:46:12 | sean-k-mooney | cool ill appove the client patches now so | |
| 18:46:17 | gmann | thanks | |
| 18:47:53 | opendevreview | Dan Smith proposed openstack/nova master: Add get_available_node_uuids() to virt driver https://review.opendev.org/c/openstack/nova/+/863917 | |
| 18:47:53 | opendevreview | Dan Smith proposed openstack/nova master: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | |
| 18:47:54 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Make resource tracker use UUIDs instead of names https://review.opendev.org/c/openstack/nova/+/863919 | |
| 18:47:54 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Detect host renames and abort startup https://review.opendev.org/c/openstack/nova/+/863920 | |
| 18:53:26 | Kirill_ | sean-k-mooney: could you please confirm that now everything is good) https://review.opendev.org/c/openstack/nova-specs/+/863773 | |
| 18:56:30 | opendevreview | Dan Smith proposed openstack/nova master: Add virt/node module for stable uuids https://review.opendev.org/c/openstack/nova/+/863915 | |
| 18:56:30 | opendevreview | Dan Smith proposed openstack/nova master: Pass service ref to init_host(), if exists https://review.opendev.org/c/openstack/nova/+/863916 | |
| 18:56:31 | opendevreview | Dan Smith proposed openstack/nova master: Add get_available_node_uuids() to virt driver https://review.opendev.org/c/openstack/nova/+/863917 | |
| 18:56:31 | opendevreview | Dan Smith proposed openstack/nova master: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | |
| 18:56:32 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Make resource tracker use UUIDs instead of names https://review.opendev.org/c/openstack/nova/+/863919 | |
| 18:56:32 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Detect host renames and abort startup https://review.opendev.org/c/openstack/nova/+/863920 | |
| 19:00:10 | sean-k-mooney | Kirill_: i think so but im to tired to review a spec fully this evning. skiming its in teh correct directory and since almost all the implemation is on the ironic sid eand ther eis no api changes this is likely sufficent | |
| 19:01:26 | Kirill_ | got it. thanks for your efforts! | |
| 19:03:17 | sean-k-mooney | im not sure if melwitt is around today but she might be able to provide more feedback today. otherwise ill take another look tomorow. you still need a second core to review and approvhe it in either case. | |
| 19:07:52 | opendevreview | Ghanshyam proposed openstack/osc-placement stable/zed: [stable-only] Use stable branch version of placement in functional tests https://review.opendev.org/c/openstack/osc-placement/+/869753 | |
| 19:08:40 | opendevreview | Ghanshyam proposed openstack/osc-placement stable/yoga: [stable-only] Use stable branch version of placement in functional tests https://review.opendev.org/c/openstack/osc-placement/+/869754 | |
| 19:23:49 | opendevreview | Ghanshyam proposed openstack/osc-placement master: Use pypi released version of placement in functional tests https://review.opendev.org/c/openstack/osc-placement/+/869755 | |
| 19:24:51 | opendevreview | Ghanshyam proposed openstack/osc-placement stable/zed: Use pypi released version of placement in functional tests https://review.opendev.org/c/openstack/osc-placement/+/869753 | |
| 19:25:42 | opendevreview | Dan Smith proposed openstack/nova master: Add virt/node module for stable uuids https://review.opendev.org/c/openstack/nova/+/863915 | |
| 19:25:43 | opendevreview | Dan Smith proposed openstack/nova master: Pass service ref to init_host(), if exists https://review.opendev.org/c/openstack/nova/+/863916 | |
| 19:25:43 | opendevreview | Dan Smith proposed openstack/nova master: Add get_available_node_uuids() to virt driver https://review.opendev.org/c/openstack/nova/+/863917 | |
| 19:25:44 | opendevreview | Dan Smith proposed openstack/nova master: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | |
| 19:25:44 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Make resource tracker use UUIDs instead of names https://review.opendev.org/c/openstack/nova/+/863919 | |
| 19:25:45 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Detect host renames and abort startup https://review.opendev.org/c/openstack/nova/+/863920 | |
| 19:28:23 | opendevreview | Ghanshyam proposed openstack/osc-placement stable/yoga: Use pypi released version of placement in functional tests https://review.opendev.org/c/openstack/osc-placement/+/869754 | |
| 19:58:51 | opendevreview | Ghanshyam proposed openstack/osc-placement master: Use pypi released version of placement in functional tests https://review.opendev.org/c/openstack/osc-placement/+/869755 | |
| 20:23:15 | opendevreview | Ghanshyam proposed openstack/osc-placement stable/zed: Use pypi released version of placement in functional tests https://review.opendev.org/c/openstack/osc-placement/+/869753 | |
| 20:24:17 | opendevreview | Ghanshyam proposed openstack/osc-placement stable/yoga: Use pypi released version of placement in functional tests https://review.opendev.org/c/openstack/osc-placement/+/869754 | |
| 20:40:52 | opendevreview | Merged openstack/python-novaclient stable/zed: [stable-only] Pin tox <4 https://review.opendev.org/c/openstack/python-novaclient/+/869527 | |
| 20:56:04 | opendevreview | Ghanshyam proposed openstack/osc-placement master: Use pypi released version of placement in functional tests https://review.opendev.org/c/openstack/osc-placement/+/869755 | |
| 21:17:31 | opendevreview | Ghanshyam proposed openstack/osc-placement stable/xena: Use pypi released version of placement in functional tests https://review.opendev.org/c/openstack/osc-placement/+/869768 | |
| 21:17:49 | gmann | sean-k-mooney: bauzas: gibi : these are the osc-placement gate fixes https://review.opendev.org/q/I4e3e5732411639054baaa9211a29e2e2c8210ac0 | |
| 21:39:07 | opendevreview | Merged openstack/nova stable/victoria: Adapt websocketproxy tests for SimpleHTTPServer fix https://review.opendev.org/c/openstack/nova/+/866195 | |
| 22:14:05 | dansmith | gmann: still around? | |
| #openstack-nova - 2023-01-11 | |||
| 00:43:36 | opendevreview | Merged openstack/python-novaclient stable/yoga: [stable-only] Pin tox <4 https://review.opendev.org/c/openstack/python-novaclient/+/869597 | |
| 00:54:16 | gmann | dansmith: hi | |
| 02:47:12 | opendevreview | Merged openstack/python-novaclient stable/xena: [stable-only] Pin tox <4 https://review.opendev.org/c/openstack/python-novaclient/+/869598 | |
| 05:37:52 | opendevreview | Ghanshyam proposed openstack/nova master: Enable new defaults and scope checks by default https://review.opendev.org/c/openstack/nova/+/866218 | |
| 08:30:28 | opendevreview | Kirill proposed openstack/nova-specs master: new spec: support of vnc console for ironic https://review.opendev.org/c/openstack/nova-specs/+/863773 | |
| 09:10:20 | gibi | stephenfin: if you have time I fixed your comments in https://review.opendev.org/c/openstack/nova/+/854924 | |
| 09:17:54 | gibi | and also there are some other patches in the PCI series where I lost your +2 | |
| 09:30:37 | kashyap | Morning. Can anyone remind me again if this targetted 'recheck' works: "recheck tempest-integrated-compute" | |
| 09:42:46 | gibi | kashyap: npe | |
| 09:42:48 | gibi | nope | |