| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-01-10 | |||
| 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: Add get_available_node_uuids() to virt driver https://review.opendev.org/c/openstack/nova/+/863917 | |
| 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: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:20 | opendevreview | Dan Smith proposed openstack/nova master: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | |
| 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: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | |
| 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:54 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Detect host renames and abort startup https://review.opendev.org/c/openstack/nova/+/863920 | |
| 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: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: Pass service ref to init_host(), if exists https://review.opendev.org/c/openstack/nova/+/863916 | |
| 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:31 | opendevreview | Dan Smith proposed openstack/nova master: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | |
| 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:32 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Detect host renames and abort startup https://review.opendev.org/c/openstack/nova/+/863920 | |
| 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 | |
| 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: Add get_available_node_uuids() to virt driver https://review.opendev.org/c/openstack/nova/+/863917 | |
| 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: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:44 | opendevreview | Dan Smith proposed openstack/nova master: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | |
| 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 | |
| 09:43:01 | gibi | there is now way to selectively recheck | |
| 09:43:06 | gibi | it is intentional | |
| 09:50:43 | kgube | gibi: Hi! Could you have another look at this: https://review.opendev.org/c/openstack/nova-specs/+/855490 | |
| 09:50:51 | kashyap | gibi: Aaah, I see. | |
| 09:51:28 | gibi | kgube: I will try | |
| 09:51:47 | kgube | thanks! | |
| 10:01:35 | gibi | kashyap: left feedback in https://review.opendev.org/c/openstack/nova/+/869587 | |
| 10:02:19 | gibi | kashyap, sean-k-mooney: I vaguely remember that you discussed removing the this check ^^ before. What was the outcome of that? | |
| 10:02:48 | auniyal | Hi gibi | |
| 10:02:55 | auniyal | can you please review these also | |
| 10:02:55 | gibi | auniyal: hi! | |
| 10:02:57 | auniyal | https://review.opendev.org/c/openstack/nova/+/864006 | |
| 10:02:59 | kashyap | gibi: Thanks! Will check in abit | |
| 10:03:04 | auniyal | https://review.opendev.org/c/openstack/nova/+/864006 | |
| 10:03:13 | auniyal | https://review.opendev.org/c/openinfra/openstack-map/+/866568 | |
| 10:09:25 | gibi | bauzas: do you want to check https://review.opendev.org/c/openstack/nova-specs/+/855490 (Use extend volume completion action) or I can +A it? | |
| 10:09:48 | gibi | you have review prio +1 on it | |
| 10:09:51 | gibi | hence my question | |
| 10:11:26 | opendevreview | Lukas Piwowarski proposed openstack/nova stable/zed: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/nova/+/869804 | |
| 10:14:00 | bauzas | gibi: sorry, discussing with Uggla but sure I can do it | |
| 10:15:59 | gibi | bauzas: it is more like do you want to? we have 2 +2s on it | |
| 10:16:36 | kashyap | gibi: Responded. Thanks for the review. I hope I have answered at least 60% of your questions :) | |
| 10:40:43 | pslestang | Hello all, dansmith will you have some time to review https://review.opendev.org/c/openstack/nova/+/867832, I pushed an other patchset after your +2 review | |