Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-02-10
14:13:33 openstackgerrit garyk proposed openstack/nova master: Better handle live migration abort https://review.openstack.org/635440
14:26:58 openstackgerrit Hamdy Khader proposed openstack/os-vif master: Add VIFPortProfileSmartNIC profile https://review.openstack.org/636061
14:30:44 openstackgerrit Hamdy Khader proposed openstack/os-vif master: Add VIFPortProfileSmartNIC profile https://review.openstack.org/636061
14:47:11 openstackgerrit Hamdy Khader proposed openstack/os-vif master: Add VIFPortProfileSmartNIC profile https://review.openstack.org/636061
20:36:21 openstackgerrit Matt Riedemann proposed openstack/nova master: Change InstanceFaultRollback handling in _error_out_instance_on_exception https://review.openstack.org/633212
20:36:21 openstackgerrit Matt Riedemann proposed openstack/nova master: Raise InstanceFaultRollback for UnableToMigrateToSelf from _prep_resize https://review.openstack.org/633227
20:51:29 openstackgerrit Matt Riedemann proposed openstack/nova master: Refactor bdm handling in ServersController.create method https://review.openstack.org/633535
20:51:29 openstackgerrit Matt Riedemann proposed openstack/nova master: Cleanup return_reservation_id in ServersController.create https://review.openstack.org/633560
20:51:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Refactor "networks" processing in ServersController.create https://review.openstack.org/633594
#openstack-nova - 2019-02-11
01:14:46 openstackgerrit Merged openstack/nova stable/rocky: Fix InstanceMapping to always default queued_for_delete=False https://review.openstack.org/635534
01:51:21 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Don't change volume status to `in-use` when attach to shelved vm. https://review.openstack.org/627096
07:59:51 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support volume-backed server rebuild in compute https://review.openstack.org/625893
08:54:27 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support volume-backed server rebuild in compute https://review.openstack.org/625893
09:02:17 openstackgerrit yenai proposed openstack/nova master: Ignore VolumeAttachmentNotFound exception in compute.manager https://review.openstack.org/632657
09:16:45 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Bump API microversion to support rebuild bfv instances https://review.openstack.org/627518
10:56:33 openstackgerrit Lee Yarwood proposed openstack/nova master: fixtures: Return a mocked class instead of method within fake_imagebackend https://review.openstack.org/619804
10:57:00 openstackgerrit Lee Yarwood proposed openstack/nova stable/queens: libvirt: Add workaround to cleanup instance dir when using rbd https://review.openstack.org/628726
10:57:06 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Don't emit warning when ironic properties are zero https://review.openstack.org/608573
11:23:30 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: Reject migration requests when source is down https://review.openstack.org/623489
12:49:09 cdent I seem to remember this being talked about before: is there a way to weigh the "busy-ness" of an AZ or aggregate? Is the cpuweigher a vague proxy for that?
13:10:39 gibi bauzas: hi! could you help me pushing through two qeens/pike backports?
13:10:49 gibi bauzas: https://review.openstack.org/#/c/623355/1 and https://review.openstack.org/#/c/635897/1,
13:11:27 cdent welcome back gibi !
13:11:39 gibi cdent: thanks :)
13:30:26 bauzas gibi: hola
13:30:29 bauzas gibi: sure, lemme look
13:30:35 gibi bauzas: thanks a lot
13:31:00 bauzas gibi: FWIW, I'm also thinking about your comments like https://review.openstack.org/#/c/599208/12/nova/virt/libvirt/driver.py@6718
13:33:06 gibi bauzas: so if the TODO means that the operator today can change the config that nova does not handle then I think we have a problem
13:34:52 bauzas gibi: yeah but the problem is that we don't know yet if we need to create a new child RP
13:35:04 bauzas we only know about the root RP
13:35:15 bauzas and we don't persist the existing type
13:35:53 bauzas oh and FWIW, it doesn't work also atm
13:37:04 gibi bauzas: so, what will happen if the admin changes the enabled_vgpu_types ?
13:43:19 bauzas gibi: for the moment, we create an inventory by calling the number of left mdevs
13:43:38 bauzas gibi: if the operator changes the type, the total would be modified
13:44:07 gibi bauzas: so eventually the gpu inventory will be in sync with the reality?
13:44:21 bauzas if we have allocations, I guess placement verifies that the new total is more than the allocations value
13:44:36 bauzas gibi: every 60 secs, ye
13:45:37 gibi bauzas: so with type X the hypervisor reports 10 available mdev, then a server is created that use up 1 of that 10. Then the operator changes the config to type Y and restart the compute
13:45:51 gibi bauzas: from type Y the hypervisor report 5 mdevs available
13:46:03 bauzas right
13:46:10 gibi bauzas: so the compute changes the inventory from 10 to 5 which still fits, as only 1 allocated
13:46:15 bauzas yup
13:46:34 bauzas that said, I have an open question
13:46:47 bauzas what if we have 6 allocs for a type with 10 total
13:47:04 bauzas and then operator uses type Y which only has 5 as a total
13:47:11 gibi I think placement will not allow changing the total to 5 then
13:47:18 bauzas ok, then we're good
13:47:20 gibi but easxy to try
13:47:21 bauzas actually wait
13:47:31 bauzas if you have one pGPU
13:47:35 bauzas with multiple types
13:48:08 bauzas when you first create a mdev, then all of the other types but the one used by the mdev are having total = 0
13:48:47 bauzas so, if the operator wants to change a type for the same pGPU when there are instances, then it's litterally not possible to get more instances
13:49:01 bauzas because the total will be the number of existing mdevs
13:49:19 bauzas unless the operators changes it back to the original type
13:49:26 bauzas so we don't have a problem atm
13:49:29 gibi bauzas: so the operator needs to delete / migrate away the instances first
13:49:42 gibi then change the tyee
13:49:43 bauzas yeah, or change the type back to the original
13:49:43 gibi type
13:49:53 gibi bauzas: OK, that makes it even stricter
13:50:01 gibi bauzas: and here strict is good, I think
13:50:02 bauzas no, he can change without needing to delete
13:50:23 gibi bauzas: yeah, if he want to change back then he can without delete
13:50:26 bauzas that's just only if he really wants to change from one type to the other
13:50:32 gibi bauzas: yeas
13:50:32 bauzas yeah
13:50:35 gibi cool
13:50:50 gibi maybe we want to document this behavior somehow for the deployer
13:50:51 bauzas now, with the change, lemme think
13:50:53 gibi to avoud surprise
13:51:00 bauzas gibi: we did it already
13:51:07 gibi bauzas: then I rest my case
13:51:09 gibi :)
13:51:46 bauzas oh wait, no actually
13:51:47 bauzas https://docs.openstack.org/nova/latest/admin/virtual-gpu.html
13:52:05 bauzas I could add a note in https://docs.openstack.org/nova/latest/admin/virtual-gpu.html#enable-gpu-types-compute
13:52:54 gibi bauzas: making that note solves my comment in your patch
13:53:14 bauzas gibi: that said, I need to think on how it works with nested RPs
13:53:42 bauzas for the moment, it does work because allocations are made against the same RP
13:53:50 bauzas oh wait
13:54:05 bauzas even if people change types, they won't change the pGPU
13:54:22 bauzas so allocations will still be made against the right pGPUI
13:54:42 bauzas actually, something is missing :(
13:54:51 bauzas holy shit
13:54:59 bauzas we have an allocation against one pGPU
13:55:17 fried_rice n-sch meeting in 5 minutes in #openstack-meeting-alt
13:55:19 bauzas but then libvirt creates the mdev (or uses some mdev) from any pGPU
13:55:26 bauzas which is a problem
13:55:33 bauzas for correctly counting allocations
13:55:47 bauzas shit shit shit
13:55:53 bauzas gibi: thanks for making me think of it
13:59:43 gibi bauzas: you are welcome :)
14:12:24 openstackgerrit Hamdy Khader proposed openstack/os-vif master: Add VIFPortProfileSmartNIC profile https://review.openstack.org/636061
14:21:02 openstackgerrit Surya Seetharaman proposed openstack/nova-specs master: Support server power state update through external event https://review.openstack.org/636132
14:28:37 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix deps for api-samples tox env https://review.openstack.org/633620
14:28:38 openstackgerrit Matt Riedemann proposed openstack/nova master: Make VolumeAttachmentsSampleV249 test other methods https://review.openstack.org/633621
14:28:38 openstackgerrit Matt Riedemann proposed openstack/nova master: Add microversion to expose virtual device tags https://review.openstack.org/631948

Earlier   Later