Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-02-08
23:33:24 aspiers got it
23:33:38 fried_rice I guess the existing code is mriedem's scenario 1 before the first comma.
23:36:35 aspiers I see
#openstack-nova - 2019-02-09
03:00:54 openstackgerrit Merged openstack/nova stable/queens: Null out instance.availability_zone on shelve offload https://review.openstack.org/606155
03:52:58 openstackgerrit Merged openstack/nova stable/queens: Remove stale pip-missing-reqs tox test https://review.openstack.org/628128
03:57:55 openstackgerrit Merged openstack/nova stable/queens: Removing pip-missing-reqs from default tox jobs https://review.openstack.org/628956
04:13:31 openstackgerrit Merged openstack/nova stable/queens: Don't call begin_detaching when detaching volume from shelved vm https://review.openstack.org/633962
04:13:37 openstackgerrit Merged openstack/nova stable/queens: Send context.global_id on neutron calls https://review.openstack.org/627604
04:13:44 openstackgerrit Merged openstack/nova stable/queens: Skip checking of target_dev for vhostuser https://review.openstack.org/634236
06:56:19 openstackgerrit Merged openstack/nova master: Modify InstanceMappingList.get_not_deleted_by_cell_and_project() https://review.openstack.org/635120
07:08:18 openstackgerrit Merged openstack/nova master: Plumbing for ignoring list_records_by_skipping_down_cells https://review.openstack.org/635121
07:42:49 openstackgerrit Merged openstack/nova master: Add minimum value in maximum_instance_delete_attempts https://review.openstack.org/633085
07:54:45 openstackgerrit Merged openstack/nova master: Convert CPU_TRAITS_MAPPING to use os_traits https://review.openstack.org/635919
07:54:52 openstackgerrit Merged openstack/nova master: doc: mention description field in user flavors docs https://review.openstack.org/635263
08:04:35 openstackgerrit Merged openstack/nova master: Extend RequestGroup object for mapping https://review.openstack.org/619527
13:42:43 openstackgerrit Merged openstack/nova stable/queens: Update port device_owner when unshelving https://review.openstack.org/626408
22:46:36 openstackgerrit Merged openstack/nova master: Drop nova-multiattach job https://review.openstack.org/606981
22:55:35 openstackgerrit Merged openstack/nova master: API: Remove evacuate/live-migrate 'force' parameter https://review.openstack.org/634600
#openstack-nova - 2019-02-10
08:14:13 openstackgerrit garyk proposed openstack/nova master: Better handle live migration abort https://review.openstack.org/635440
08:16:02 openstackgerrit garyk proposed openstack/nova master: Better handle live migration abort https://review.openstack.org/635440
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

Earlier   Later