Earlier  
Posted Nick Remark
#openstack-nova - 2018-02-05
08:52:18 amorin I would like to know if there is any way to create a neutron client from nova, which could use the retries parameter of neutron
08:52:23 amorin on nova side: https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L137
08:52:33 amorin neutron side: https://github.com/openstack/python-neutronclient/blob/master/neutronclient/v2_0/client.py#L220
08:52:52 amorin seems that nova is not using retries at all
09:03:26 bauzas good morning folksq
09:05:01 gibi bauzas: good morning
09:05:10 amorin morning
09:17:41 openstackgerrit jichenjc proposed openstack/nova master: deprecate fping_path config option https://review.openstack.org/526602
09:43:14 openstackgerrit Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
09:48:07 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Trivial fix a missleading comment https://review.openstack.org/540796
10:03:37 openstackgerrit Merged openstack/nova master: Zuul: Remove project name https://review.openstack.org/540326
10:03:47 openstackgerrit Merged openstack/nova master: Fix SUSE Install Guide: Placement port https://review.openstack.org/538163
10:08:39 gibi the gate is super fast now :)
10:11:32 openstackgerrit Jianghua Wang proposed openstack/nova master: doc: update the GPU passthrough HPC feature entry https://review.openstack.org/539638
10:11:33 openstackgerrit Jianghua Wang proposed openstack/nova master: Provide support matrix and doc for VGPU https://review.openstack.org/539266
10:11:33 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: Provide support matrix and doc for VGPU https://review.openstack.org/540808
10:50:18 ameeda gibi: replied
11:11:48 gibi ameeda: OK, I will double check with mriedem about the double detach
11:12:27 ameeda gibi: thank you !!
11:12:55 gibi ameeda: until then, I will try to make the functional test work without the need for mocking the detach call
11:13:53 ameeda gibi: yes please , I try a lot with it with no lucks
11:16:06 ameeda gibi: I have 2 nova reviews needs for 2 +2 here https://review.openstack.org/#/c/539880/ and https://review.openstack.org/#/c/540274/
11:16:17 ameeda gibi: also I need help here https://review.openstack.org/#/c/526900/
11:16:40 ameeda sorry for inconvenience, thanks for you time !!
11:25:27 openstackgerrit Marcin Juszkiewicz proposed openstack/nova master: Make sure that we have usable input for graphical console https://review.openstack.org/538003
11:26:09 hrw morning
11:33:35 rabel hi. can we merge https://review.openstack.org/#/c/535871/ ?
11:38:58 openstackgerrit Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
12:03:04 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Add missing encryption_secret_uuid tests https://review.openstack.org/540679
12:03:21 lyarwood ops
12:03:26 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Add missing encryption_secret_uuid tests https://review.openstack.org/540679
12:03:29 lyarwood there we go
12:23:40 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: Unmap compute nodes when deleting host mapping https://review.openstack.org/527560
12:26:34 ameeda gibi: replied
12:34:50 openstackgerrit Lajos Katona proposed openstack/nova master: WIP: ServerMovingTests with custom resources https://review.openstack.org/497399
12:42:07 openstackgerrit Claudiu Belu proposed openstack/nova master: hyperv: Cleans up live migration Planned VM https://review.openstack.org/478943
12:44:16 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: Do not attempt volume swap when guest is stopped/suspended https://review.openstack.org/533733
12:45:09 hrw https://review.openstack.org/#/c/532504/ - someone will give second +2 (and +w) to stable/pike backport for arm64?
12:48:38 openstackgerrit Merged openstack/nova stable/ocata: Fix false positive server group functional tests https://review.openstack.org/536995
12:58:28 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: docs: Fix stable/pike config references https://review.openstack.org/540846
13:10:06 gibi bauzas: hi, I have a quetion in https://review.openstack.org/#/c/539266/9/doc/source/admin/virtual-gpu.rst@133
13:18:30 bauzas gibi: replying
13:19:50 bauzas gibi: replied ;)
13:21:22 gibi bauzas: thanks. so what is this '*Error* state' the doc is refer to?
13:25:28 bauzas gibi: http://paste.openstack.org/show/662486/
13:25:30 bauzas HTH
13:28:36 gibi bauzas: ahh so the suspend instance action will have the error state
13:28:45 bauzas exactly
13:29:00 bauzas hence not the VM state "ERROR"
13:29:07 gibi bauzas: thanks.
13:29:21 bauzas (not capitalized")
13:30:37 gibi bauzas: I +Wd that patch
13:30:49 bauzas cool thanks
13:31:30 gibi thank you for creating such a good doc
13:32:03 bauzas thanks
13:39:26 openstackgerrit sahid proposed openstack/nova master: hardware: only take into account small pages https://review.openstack.org/532168
13:56:48 jaypipes_ morning supernovas
13:57:03 edleafe Scheduler subteam meeting in 3 minutes in #openstack-meeting-alt
13:59:51 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: Removed unused argument from LibvirtDriver._connect_volume https://review.openstack.org/540859
13:59:51 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: Remove unused argument from LibvirtDriver._disconnect_volume https://review.openstack.org/540860
13:59:52 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: libvirt: Collocate encryptor and volume driver calls https://review.openstack.org/540861
14:00:53 edleafe Scheduler subteam meeting running NOW in #openstack-meeting-alt
14:16:04 bauzas wow, I feel the breeze of not having internal stuff to do
14:18:23 openstackgerrit Marcin Juszkiewicz proposed openstack/nova master: Make sure that we have usable input for graphical console https://review.openstack.org/538003
14:22:41 openstackgerrit Merged openstack/nova stable/pike: Fix the ocata config-reference URLs https://review.openstack.org/515410
14:22:48 openstackgerrit Merged openstack/nova master: doc: update the GPU passthrough HPC feature entry https://review.openstack.org/539638
14:23:03 openstackgerrit Merged openstack/nova master: Provide support matrix and doc for VGPU https://review.openstack.org/539266
14:27:04 ameeda jaypipes_: could you please check this https://review.openstack.org/#/c/526900/ ?
14:27:33 openstackgerrit Andrey Kurilin proposed openstack/python-novaclient master: Remove 2 redundant methods https://review.openstack.org/528325
14:36:32 gibi bauzas: If you have some free time, then there is a bugfix that needs some review https://review.openstack.org/#/c/525242/ :)
14:37:14 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: add "--until-complete" option for nova-manage db archive_deleted_rows. https://review.openstack.org/540871
14:37:19 bauzas gibi: free time is not exactly what I call as my current workload, but I'll ;)
14:38:37 gibi bauzas: Ohh then I was misled by your feeling the breeze comment above :)
14:39:07 bauzas gibi: oh, your patch is crucial https://review.openstack.org/#/c/525242/
14:39:54 mriedem lyarwood: i think it would be ok to just leave that as a separate refactor/test coverage change on top
14:40:14 mriedem don't really want to hold up https://review.openstack.org/#/c/539739/ since it's breaking cinder 3rd party CI
14:40:40 lyarwood mriedem: yup np
14:43:18 gibi mriedem: hi, I think we have some indirect disagreement in https://review.openstack.org/#/c/528385 I think the duplicate detach will cause confusions in the logs and in the notification interface
14:46:26 efried cdent: This sentence does not parse for me: "That new structure allowed for the creation a POST to /allocations that allows an atomic action to manage allocations for multiple consumers in one request, supporting race-free migration allocations."
14:51:25 bauzas gibi: mriedem: I'm a bit concerned by a bugfix having RPC versions bumped but it's understandable given the upcall issue
14:51:52 bauzas context being https://review.openstack.org/#/c/525242/14
14:51:56 gibi bauzas: First I tried without the rpc bump but that solution would create yet another cell upcall
14:52:14 bauzas yeah I know
14:52:39 bauzas FWIW, I have a long standing blueprint for passing request specs to computes
14:52:50 bauzas it's purely procedural tho
14:53:03 bauzas I'm fine with approving that change if everything else is ok
14:53:14 bauzas just thinking out loud
14:54:15 gibi yes, we need to agree on such a bump together. Especially as dansmith might plan to do a major rpc version bump before the release
14:54:46 dansmith yeah, unfortunately, that reqspec cleanup would be really nice to get into place
14:56:30 mriedem i got 2 patches in against it for queens!
14:56:31 gibi bauzas: if that helps with the current bugfix, I can offer some help with the request spec blueprint in Rocky
14:56:33 mriedem with great success
14:57:29 efried bauzas: What's a request spec, as distinct from allocation request data?
14:58:01 efried That is, what does a request spec has that an allocation request doesn't?
14:58:19 mriedem gibi: replied in https://review.openstack.org/#/c/528385/
14:58:32 mriedem efried: all sorts of stuff,
14:58:38 mriedem instance groups, scheduler hints,
14:58:41 mriedem image id
14:58:46 mriedem it's basically the instance
14:58:53 mriedem minus host stuff (and BDMs for some reason)

Earlier   Later