| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-01 | |||
| 05:48:58 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing remote console policies https://review.opendev.org/716482 | |
| 05:49:30 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in remote consoles policy https://review.opendev.org/716483 | |
| 05:49:43 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in remote console policies https://review.opendev.org/716484 | |
| 06:04:26 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing rescue policies https://review.opendev.org/716486 | |
| 06:08:06 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in rescue server policy https://review.opendev.org/716488 | |
| 06:37:50 | openstackgerrit | Merged openstack/nova master: Add new default roles in os-hypervisors policies https://review.opendev.org/715071 | |
| 06:38:29 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in rescue server policies https://review.opendev.org/716496 | |
| 07:01:42 | brinzhang_ | sean-k-mooney: gibi: what do you think of this releasenote change? https://review.opendev.org/#/c/716185/2/releasenotes/notes/accelerator-requests-6c9a6fef77ab776a.yaml@11 I am ok add this in releasenote for nova-cyborg interaction feature | |
| 07:02:47 | brinzhang_ | sean-k-mooney: gibi: do we need add these in the API guide? that the user can know what action they can do while the instance has some accelerators. | |
| 07:02:54 | brinzhang_ | s/while/if | |
| 07:17:07 | gibi | brinzhang_: do we have api guide for cyborg interaction? | |
| 07:17:36 | gibi | brinzhang_: at least I don't find it | |
| 07:17:51 | gibi | brinzhang_: but sure an api guide would be good to have | |
| 07:18:18 | brinzhang_ | gibi: that also what I want to see too. | |
| 07:18:43 | gibi | something similar to https://github.com/openstack/nova/blob/master/api-guide/source/port_with_resource_request.rst | |
| 07:19:53 | gibi | brinzhang_: could you please note this api-guide request in the reno review | |
| 07:19:56 | gibi | ? | |
| 07:20:48 | brinzhang_ | gibi: I will asked Sundar in wechat, wait later :) | |
| 07:21:23 | brinzhang_ | gibi: I left some comments in https://review.opendev.org/#/c/693828/24, can you see fast? | |
| 07:22:38 | gibi | brinzhang_: will do in a couple of minutes (on a daily standup meeting now) | |
| 07:23:06 | brinzhang_ | gibi: thanks^^ | |
| 08:31:59 | gibi | brinzhang_: replied in https://review.opendev.org/#/c/693828 | |
| 08:32:26 | openstackgerrit | jayaditya gupta proposed openstack/nova master: Support for nova-manage placement heal_allocations --cell https://review.opendev.org/714459 | |
| 08:39:16 | brinzhang_ | gibi: I will looked into, thanks | |
| 09:04:50 | gibi | brinzhang_: also feedback on https://review.opendev.org/#/c/711194 | |
| 09:04:56 | jkulik | Could someone have a look at this, please? https://review.opendev.org/#/c/713863/ | |
| 09:08:12 | gibi | jkulik: could you please file a bug | |
| 09:08:40 | jkulik | gibi: if that helps, sure | |
| 09:08:51 | gibi | jkulik: also, is this something that worked well in the past and then broken by a certain release? | |
| 09:09:03 | jkulik | nope. never worked properly for us | |
| 09:09:07 | jkulik | we're on queens ;) | |
| 09:09:32 | gibi | jkulik: yes it helps as it feels like a bug, and bugs tend to be backported to stable branches. All these tracking is easier if there is a bug that connect the patches on different branches | |
| 09:09:55 | jkulik | ok, got it. will create a bug and mention it in the commit-message | |
| 09:09:59 | jkulik | thank you | |
| 09:10:39 | gibi | jkulik: I see. If you can write a sort reproduction steps in the bug report then I will try to repoduce it in my devstack to confirm the bug | |
| 09:15:09 | brinzhang_ | gibi: with https://review.opendev.org/#/c/71114 I also have some doubt, hope dansmith can have some good idea. | |
| 09:15:54 | brinzhang_ | gibi: A question, I will update the novalcinet to support v2.85, do I need to add all request parameter to the volume-update CLI? | |
| 09:16:41 | brinzhang_ | gibi: We just can update the delete flag, how about just only update the delete_on_termination to the volume-update CLI to support update volume? | |
| 09:17:25 | gibi | brinzhang_: I think it is OK to update the CLI to only offer d-o-t update | |
| 09:17:39 | gibi | that is the main use case for the end user anyhow | |
| 09:17:42 | brinzhang_ | other parameters, e.g. device, id, serverId, tag, that should keep same value, so I think we dont add that in the CLI | |
| 09:17:55 | gibi | I agree | |
| 09:18:02 | brinzhang_ | gibi: ok, I will do by this way, thanks | |
| 09:21:17 | bauzas | gibi: around for a question about reshapes ? | |
| 09:21:27 | bauzas | and your vgpu func test ? | |
| 09:25:14 | gibi | bauzas: sure, hit me | |
| 09:27:17 | bauzas | gibi: okay, so I wonder why you mocked https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/test_reshape.py#L96-L101 | |
| 09:27:40 | bauzas | gibi: you started two instances without having any supported types | |
| 09:27:53 | bauzas | gibi: so the instances wouldn't have any VGPU allocations | |
| 09:29:49 | bauzas | oh, but you created some VGPU inventories... | |
| 09:29:52 | gibi | bauzas: before that there is a placement update | |
| 09:29:55 | gibi | yeah | |
| 09:29:59 | gibi | that | |
| 09:30:37 | gibi | I guess this is a simplification | |
| 09:30:39 | bauzas | mmm ok | |
| 09:30:53 | gibi | in a good test the virt driver should report the new vgpu inventory | |
| 09:31:07 | gibi | I don't remember why I don't added a new fake driver that does that | |
| 09:31:13 | gibi | I guess there was complications | |
| 09:31:14 | bauzas | what I honestly wonder is whether your test is valid for a reshape | |
| 09:31:29 | bauzas | because when you restart nova-compute, you get a reshape, right? | |
| 09:31:50 | bauzas | in this case, when allocating an instance for this node, you get the nested RPs | |
| 09:32:45 | bauzas | I mean, we only have VGPU inventories on the root RP for nodes that were not restarted | |
| 09:32:57 | bauzas | (or <Stein) | |
| 09:34:55 | gibi | this test tries to create instances with old type of allocation then hit the reshape by restarting the compute | |
| 09:35:27 | bauzas | I see | |
| 09:35:33 | gibi | if the virt driver report the vgpu at fist start then with the current code reshape is triggered, so I had to simulate and old tree somehow | |
| 09:35:42 | gibi | then trigger a reshape | |
| 09:36:21 | gibi | see https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/test_reshape.py#L53 | |
| 09:36:21 | openstackgerrit | Merged openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999 | |
| 09:36:28 | bauzas | gibi: but you agree with the fact it wouldn't be possible to create an instance having old types of allocations with Stein or Ussuri ? | |
| 09:37:07 | gibi | I agree. But to test reshap in >=Stein I have to simualte an old tree | |
| 09:37:11 | gibi | hence the hack in the test | |
| 09:37:18 | bauzas | I mean, the _allocate_mdev() code in libvirt that's eventually called when creating an instance would be different between Rocky and Stein | |
| 09:37:56 | gibi | that is a valid point, I haven't simulated the Rocky _allocate_mdev code | |
| 09:38:19 | bauzas | I mean, I wrote https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L7011-L7013 | |
| 09:38:25 | gibi | I guess creating a proper test is only possible a grenade job that upgrades from Rocky to Stein | |
| 09:39:06 | bauzas | but I think it wasn't needed | |
| 09:39:12 | bauzas | yeah maybe | |
| 09:39:27 | bauzas | anyway, I need to think about what do do | |
| 09:39:49 | bauzas | your test is nice and i don't want to remove it | |
| 09:40:24 | tobias-urdin | continue troubleshooting https://bugs.launchpad.net/oslo.config/+bug/1869929 seems like some issue with a nova dependency or import order issues, have a complete reproduce and a solution, but the solution is to remove the @db.select_db_reader_mode decorator from two functions in nova/objects/service.py | |
| 09:40:25 | gibi | yeah. feel free to remove the func test if it does not make sense any more. The hack there is definitely partially simulate Rocky state | |
| 09:40:26 | openstack | Launchpad bug 1869929 in oslo.config "RuntimeError: maximum recursion depth exceeded while calling a Python object" [Undecided,New] | |
| 09:41:00 | bauzas | gibi: well, I just feel we would just need to create a new instance without asking for VGPUs and then creating the VGPU allocations directly | |
| 09:41:04 | gibi | bauzas: if you remove the func test from master, we will still have it on stable branches where this test is more relevant | |
| 09:41:06 | bauzas | (for the func test) | |
| 09:41:22 | tobias-urdin | ^ if anybody knows import could potentially cause it, trying to track down changes to imports between the versions, seems like nova.cells.rpcapi changed | |
| 09:42:22 | gibi | bauzas: but then mdev assignments will be missing, isn't it? | |
| 09:42:42 | bauzas | ah shit you're right, indeed | |
| 09:42:47 | gibi | bauzas: so that will be partial from a different perspective | |
| 09:42:53 | bauzas | yeah | |
| 09:43:03 | bauzas | anyway, I'll think about it | |
| 09:43:11 | gibi | bauzas: cool. let me know if I can help | |
| 09:43:22 | bauzas | we still need to support the reshape for Ussuri like we said | |
| 09:43:54 | bauzas | so maybe a comment telling it's not really possible to have no parent UUID would be simple | |
| 09:50:14 | openstackgerrit | Johannes Kulik proposed openstack/nova master: Don't recompute weighers' minval/maxval attributes https://review.opendev.org/713863 | |
| 09:50:56 | jkulik | gibi, bug for ^ is here https://bugs.launchpad.net/nova/+bug/1870096 | |
| 09:50:57 | openstack | Launchpad bug 1870096 in OpenStack Compute (nova) "soft-affinity weight not normalized base on server group's maximum" [Undecided,In progress] - Assigned to Johannes Kulik (jkulik) | |
| 09:51:06 | gibi | jkulik: ack, I will check | |
| 09:51:09 | gibi | thanks | |