| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-04-14 | |||
| 20:44:20 | noonedeadpunk | I mean shouldn't be there a valid protection, that exact same cells are not present in DB with different UUIDs? | |
| 20:44:23 | sean-k-mooney | noonedeadpunk: checking the db content wont work i this case actully | |
| 20:44:23 | sean-k-mooney | noonedeadpunk: checking the db content wont work i this case actully | |
| 20:45:20 | sean-k-mooney | i think the cell name is really the unique thing | |
| 20:45:20 | sean-k-mooney | i think the cell name is really the unique thing | |
| 20:45:29 | sean-k-mooney | ingoring the sharding usecase | |
| 20:45:30 | sean-k-mooney | ingoring the sharding usecase | |
| 20:45:34 | sean-k-mooney | that is not currently implemetned | |
| 20:45:34 | sean-k-mooney | that is not currently implemetned | |
| 20:47:11 | sean-k-mooney | dansmith: actully i dont think we can support the sharding usecase with the current db schema without denormalising the database | |
| 20:47:11 | sean-k-mooney | dansmith: actully i dont think we can support the sharding usecase with the current db schema without denormalising the database | |
| 20:47:18 | noonedeadpunk | but at least combination of name+transport_url+database_connection should be unique IMO. And considering templates, I see the only way to do that is to check here https://opendev.org/openstack/nova/src/commit/9e5feb051c15166988e87b10132927135bd56884/nova/cmd/manage.py#L623-L624 that cell.database_connection is still in tempate format, but not transformed | |
| 20:47:18 | noonedeadpunk | but at least combination of name+transport_url+database_connection should be unique IMO. And considering templates, I see the only way to do that is to check here https://opendev.org/openstack/nova/src/commit/9e5feb051c15166988e87b10132927135bd56884/nova/cmd/manage.py#L623-L624 that cell.database_connection is still in tempate format, but not transformed | |
| 20:47:36 | sean-k-mooney | dansmith: we would realy need to have a second db table that mapped db urls to the cell id or similar | |
| 20:47:36 | sean-k-mooney | dansmith: we would realy need to have a second db table that mapped db urls to the cell id or similar | |
| 20:48:02 | noonedeadpunk | I kind of submitted bug regarding this https://bugs.launchpad.net/nova/+bug/1923899 as well | |
| 20:48:02 | noonedeadpunk | I kind of submitted bug regarding this https://bugs.launchpad.net/nova/+bug/1923899 as well | |
| 20:48:04 | openstack | Launchpad bug 1923899 in OpenStack Compute (nova) "Nova create duplicate cells when templates are used" [Undecided,New] | |
| 20:48:04 | openstack | Launchpad bug 1923899 in OpenStack Compute (nova) "Nova create duplicate cells when templates are used" [Undecided,New] | |
| 20:48:09 | dansmith | sean-k-mooney: yeah, above I said that was the original intent to be able to do that, but that we'd still have work to do to make it work | |
| 20:48:09 | dansmith | sean-k-mooney: yeah, above I said that was the original intent to be able to do that, but that we'd still have work to do to make it work | |
| 20:49:20 | noonedeadpunk | sorry for late discussion, already midnight here, so need to head out :( | |
| 20:49:20 | noonedeadpunk | sorry for late discussion, already midnight here, so need to head out :( | |
| 20:49:39 | sean-k-mooney | no worries | |
| 20:50:42 | sean-k-mooney | i think if we jsut put a unique constarint on name that would be enough. if we want to supprot sharding we would need to have a second table with something like (id, cell_id, db | |
| 20:50:42 | sean-k-mooney | i think if we jsut put a unique constarint on name that would be enough. if we want to supprot sharding we would need to have a second table with something like (id, cell_id, db | |
| 20:50:50 | noonedeadpunk | eventually what I intended to do is to rely on cell_create command return code, and if it's 2 - update_cell, if 0 - proceed with setup. And for me it's now always 0 when using templates :( | |
| 20:50:50 | noonedeadpunk | eventually what I intended to do is to rely on cell_create command return code, and if it's 2 - update_cell, if 0 - proceed with setup. And for me it's now always 0 when using templates :( | |
| 20:51:01 | sean-k-mooney | (id, cell_id, db_url, db_table) | |
| 20:51:01 | sean-k-mooney | (id, cell_id, db_url, db_table) | |
| 20:51:42 | sean-k-mooney | noonedeadpunk: ya for now i guess you have to check and not use the return code | |
| 20:51:42 | sean-k-mooney | noonedeadpunk: ya for now i guess you have to check and not use the return code | |
| 20:51:59 | sean-k-mooney | but i think the bug is more or less valid and we should be returning 2 | |
| 20:51:59 | sean-k-mooney | but i think the bug is more or less valid and we should be returning 2 | |
| 20:52:02 | noonedeadpunk | yeah:( | |
| 20:52:02 | noonedeadpunk | yeah:( | |
| 20:53:24 | sean-k-mooney | dansmith: im not sure how ye were thinking about shardin gby the way so you might not need to do it by table so maybe the other table would just be (id, cell_id, db_url) not (id, cell_id, db_url, db_table) | |
| 20:53:24 | sean-k-mooney | dansmith: im not sure how ye were thinking about shardin gby the way so you might not need to do it by table so maybe the other table would just be (id, cell_id, db_url) not (id, cell_id, db_url, db_table) | |
| 20:55:15 | sean-k-mooney | but the impotant thing is we would need to model the many to one relation ship while maintianing only on uuid per cell | |
| 20:55:15 | sean-k-mooney | but the impotant thing is we would need to model the many to one relation ship while maintianing only on uuid per cell | |
| 21:06:10 | mnaser | i've been noticing an issue where in a scenario that libvirt loses config, when using vgpu, and doing a hard reboot generates xml that _does not_ contain the vgpu | |
| 21:06:10 | mnaser | i've been noticing an issue where in a scenario that libvirt loses config, when using vgpu, and doing a hard reboot generates xml that _does not_ contain the vgpu | |
| 21:06:47 | mnaser | to reproduce: openstack server stop <instance>; virsh undefine <instance>; openstack server start <instance>; virsh dumpxml <instance> ... no mdev | |
| 21:06:47 | mnaser | to reproduce: openstack server stop <instance>; virsh undefine <instance>; openstack server start <instance>; virsh dumpxml <instance> ... no mdev | |
| 21:12:07 | mnaser | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L3453-L3456 | |
| 21:12:07 | mnaser | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L3453-L3456 | |
| 21:13:09 | mnaser | this is starting to sound like mdev state is actually conserved in libvirt domains? | |
| 21:13:09 | mnaser | this is starting to sound like mdev state is actually conserved in libvirt domains? | |
| 21:57:42 | melwitt | mnaser: this seems kinda related https://bugs.launchpad.net/nova/+bug/1900800 | |
| 21:57:42 | melwitt | mnaser: this seems kinda related https://bugs.launchpad.net/nova/+bug/1900800 | |
| 21:57:43 | openstack | Launchpad bug 1900800 in OpenStack Compute (nova) "VGPUs is not recreated on host reboot" [Low,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza) | |
| 22:01:19 | mnaser | melwitt: that’s exactly it | |
| 22:01:19 | mnaser | melwitt: that’s exactly it | |
| #openstack-nova - 2021-04-15 | |||
| 01:26:13 | openstackgerrit | norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348 | |
| 01:30:37 | guilhermesp | melwitt: and I suppose the only workaround is rebuild the instance to get the vgpu reallocated, right? | |
| 01:30:38 | guilhermesp | melwitt: and I suppose the only workaround is rebuild the instance to get the vgpu reallocated, right? | |
| 01:57:45 | openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: Support live migrate vtpm server https://review.opendev.org/c/openstack/nova-specs/+/785860 | |
| 01:57:45 | openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: Support live migrate vtpm server https://review.opendev.org/c/openstack/nova-specs/+/785860 | |
| 02:07:29 | openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: Support fuzzy querying instance by tag https://review.opendev.org/c/openstack/nova-specs/+/768853 | |
| 02:07:29 | openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: Support fuzzy querying instance by tag https://review.opendev.org/c/openstack/nova-specs/+/768853 | |
| 03:48:45 | openstackgerrit | Xinran WANG proposed openstack/nova-specs master: Repropose smartnic support spec https://review.opendev.org/c/openstack/nova-specs/+/783632 | |
| 05:20:28 | openstackgerrit | Jeffrey Zhang proposed openstack/nova master: Support inject-nmi action in watchdog https://review.opendev.org/c/openstack/nova/+/741072 | |
| 05:20:28 | openstackgerrit | Jeffrey Zhang proposed openstack/nova master: Support inject-nmi action in watchdog https://review.opendev.org/c/openstack/nova/+/741072 | |
| 07:00:28 | melwitt | guilhermesp: shelve offload/unshelve might work? my thinking is shelve offload would deallocate placement resources and then unshelve would reallocate everything again | |
| 07:00:28 | melwitt | guilhermesp: shelve offload/unshelve might work? my thinking is shelve offload would deallocate placement resources and then unshelve would reallocate everything again | |
| 07:38:08 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Replace blind retry with libvirt event waiting in detach https://review.opendev.org/c/openstack/nova/+/770246 | |
| 07:38:09 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Move the guest.get_disk test to test_guest https://review.opendev.org/c/openstack/nova/+/777151 | |
| 07:39:32 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Enable mypy on libvirt/guest.py https://review.opendev.org/c/openstack/nova/+/777155 | |
| 07:39:33 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: libvirt: Remove dead error handling code https://review.opendev.org/c/openstack/nova/+/779704 | |
| 07:39:33 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Follow up type hints for a634103 https://review.opendev.org/c/openstack/nova/+/777159 | |
| 07:49:13 | openstackgerrit | Balazs Gibizer proposed openstack/osc-placement master: Mark microversion 1.37 supported https://review.opendev.org/c/openstack/osc-placement/+/784023 | |
| 07:49:13 | openstackgerrit | Balazs Gibizer proposed openstack/osc-placement master: Mark microversion 1.37 supported https://review.opendev.org/c/openstack/osc-placement/+/784023 | |
| 07:49:27 | lyarwood | gibi: I'll start looking at the detach stuff this morning | |
| 07:49:27 | lyarwood | gibi: I'll start looking at the detach stuff this morning | |
| 07:49:37 | lyarwood | gibi: https://review.opendev.org/c/openstack/nova/+/784129/2 would you mind hitting this to unblock the actual fix on top? | |
| 07:49:37 | lyarwood | gibi: https://review.opendev.org/c/openstack/nova/+/784129/2 would you mind hitting this to unblock the actual fix on top? | |
| 07:50:39 | gibi | lyarwood: thanks. and sure I will look at the funct test | |
| 07:50:39 | gibi | lyarwood: thanks. and sure I will look at the funct test | |
| 07:50:46 | lyarwood | cool thanks | |
| 07:50:46 | lyarwood | cool thanks | |
| 08:44:16 | lyarwood | does anyone know where I can find the actual schedule for ptg? https://www.openstack.org/ptg/ just shows a static image for Monday AFAICT. | |
| 08:44:16 | lyarwood | does anyone know where I can find the actual schedule for ptg? https://www.openstack.org/ptg/ just shows a static image for Monday AFAICT. | |
| 08:44:48 | lyarwood | http://ptg.openstack.org/ptg.html ah | |
| 08:44:48 | lyarwood | http://ptg.openstack.org/ptg.html ah | |
| 08:59:04 | gibi | lyarwood: also above the static image there is a link to a pdf | |
| 08:59:04 | gibi | lyarwood: also above the static image there is a link to a pdf | |
| 09:44:02 | openstackgerrit | Merged openstack/nova master: Add regression test for bug #1922053 https://review.opendev.org/c/openstack/nova/+/784129 | |
| 09:44:02 | openstackgerrit | Merged openstack/nova master: Add regression test for bug #1922053 https://review.opendev.org/c/openstack/nova/+/784129 | |
| 09:44:03 | openstack | bug 1922053 in OpenStack Compute (nova) "Operators can force up compute services with `done` evacuation migration records still active against the host" [Medium,In progress] https://launchpad.net/bugs/1922053 - Assigned to Lee Yarwood (lyarwood) | |
| 09:44:03 | openstack | bug 1922053 in OpenStack Compute (nova) "Operators can force up compute services with `done` evacuation migration records still active against the host" [Medium,In progress] https://launchpad.net/bugs/1922053 - Assigned to Lee Yarwood (lyarwood) | |
| 09:44:43 | openstackgerrit | Merged openstack/nova master: api: Reject requests to force up computes when `done` evacuation records exist https://review.opendev.org/c/openstack/nova/+/784130 | |
| 09:44:43 | openstackgerrit | Merged openstack/nova master: api: Reject requests to force up computes when `done` evacuation records exist https://review.opendev.org/c/openstack/nova/+/784130 | |
| 09:53:58 | gibi | fyi, there is a frequent live migration failure in tempest in the new trunk port test | |
| 09:53:58 | gibi | fyi, there is a frequent live migration failure in tempest in the new trunk port test | |
| 09:54:01 | gibi | https://bugs.launchpad.net/tempest/+bug/1924258 | |
| 09:54:01 | gibi | https://bugs.launchpad.net/tempest/+bug/1924258 | |
| 09:54:02 | openstack | Launchpad bug 1924258 in tempest "test_live_migration_with_trunk fails intermittently" [Undecided,New] | |
| 09:54:02 | openstack | Launchpad bug 1924258 in tempest "test_live_migration_with_trunk fails intermittently" [Undecided,New] | |
| 09:55:55 | gibi | I notified the author of the test case and he promised to check it | |
| 09:55:55 | gibi | I notified the author of the test case and he promised to check it | |