Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-08
21:17:08 mriedem yeah i don't think we need a blueprint either
21:17:10 mriedem just reno and doc updates
21:17:56 melwitt ++
21:18:38 melwitt looks like the doc update would be in glance though
21:18:51 mriedem for the image properties one yeah,
21:18:58 mriedem but we should also document the flavor extra spec in our docs
21:19:23 melwitt yeah -- just not sure it has to fall on their change though, since it's currently undocumented even for the libvirt driver
21:20:03 mriedem melwitt: ok, i'll do them a solid and push a patch below theirs that adds the flavor extra spec doc, and they can then update it
21:20:33 melwitt sounds super
21:32:47 openstackgerrit Merged openstack/nova master: Remove stale pip-missing-reqs tox test https://review.openstack.org/565535
21:40:55 openstackgerrit Matt Riedemann proposed openstack/nova master: doc: Don't confuse CPU pinning/NUMA as Hyper-V only https://review.openstack.org/561949
21:40:56 mriedem stephenfin: we need to get -W working in the docs builds again ^
21:50:59 openstackgerrit Matt Riedemann proposed openstack/nova master: Use .. deprecated:: theme for deprecations https://review.openstack.org/565822
21:51:01 openstackgerrit Merged openstack/nova master: placement: Object changes for granular https://review.openstack.org/564351
21:51:26 sean-k-mooney mriedem: huh when did they make that a config option on the host instead of using the there custom extra spec option
21:51:58 mriedem sean-k-mooney: context?
21:52:10 sean-k-mooney hyperv numa affinity
21:52:24 mriedem that's a question for claudiu
21:52:25 mriedem or git blame
21:53:21 sean-k-mooney hyperv used (like 4/5 releases ago) have a addtion hw:somting extra spec that allowed you to set the numa affinity of the guest memory
21:54:24 sean-k-mooney i went looking for it simi recently but could not find it so i assumed it was removed but i guest its now a host config option
21:55:22 mriedem dansmith: i left comments in https://review.openstack.org/#/c/509002/ but didn't want to -1
21:57:07 sean-k-mooney mriedem: ah it was hw:numa_mempolicy=preferred|strict http://git.openstack.org/cgit/openstack/nova-specs/tree/specs/liberty/approved/hyper-v-vnuma-enable.rst#n78
22:01:10 sean-k-mooney ah it looks like they did not complete the spec until ocata and that extra spec got lost along the way. that is a shame i wanted to add that to the libvirt driver at some point.
22:04:19 dansmith mriedem: ack, thanks
22:09:58 arvindn05 ...
22:09:58 arvindn05 "trait:CUSTOM_TRUSTED": "required",
22:09:58 arvindn05 "image_id": "39f216fd-ebba-4a6a-b741-c83cd02a00ec",
22:09:58 arvindn05 "volume_image_metadata": {
22:09:58 arvindn05 mriedem: FYI tested with image backed volume the image meta data needs to be part of the volume_image_metadata...so will need to modify the fixture..will write the funtional test with that in mind :)
22:09:59 arvindn05 }
22:17:03 openstackgerrit Matt Riedemann proposed openstack/nova master: vmware:Validation video ram against the max video ram allowed. https://review.openstack.org/564193
22:17:04 openstackgerrit Matt Riedemann proposed openstack/nova master: Add docs for hw_video:ram_max_mb flavor extra spec https://review.openstack.org/567070
22:29:08 openstackgerrit Merged openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642
22:38:36 melwitt mriedem: I dunno if you saw my reply yesterday about the stable releases. I was thinking maybe we just do the releases after we get the regression fixes we know about in? what was your thought on it? do you think we should go ahead and flush the stable review queues too?
23:09:21 openstackgerrit Merged openstack/nova master: Handle PortNotFoundClient exception when getting ports https://review.openstack.org/565183
23:12:16 melwitt interesting, this old experimental queue job failing because it wants to install the liberty cloud archive which isn't supported on xenial http://logs.openstack.org/37/554437/18/experimental/legacy-tempest-dsvm-nova-libvirt-kvm-apr/e50a428/logs/devstacklog.txt.gz#_2018-05-08_22_58_11_828
23:18:54 openstackgerrit Merged openstack/python-novaclient master: Fix the policy argument in server-group-create https://review.openstack.org/564699
23:47:36 openstackgerrit Merged openstack/nova master: placement: Granular GET /allocation_candidates https://review.openstack.org/517757
23:47:44 openstackgerrit Merged openstack/nova master: doc: Don't confuse CPU pinning/NUMA as Hyper-V only https://review.openstack.org/561949
23:47:52 openstackgerrit Merged openstack/nova master: Use .. deprecated:: theme for deprecations https://review.openstack.org/565822
23:58:39 mriedem melwitt: didn't see it, but these are the regressions i had in mind: https://review.openstack.org/#/c/564461/ https://review.openstack.org/#/c/566367/ https://review.openstack.org/#/c/563213/
#openstack-nova - 2018-05-09
00:08:13 openstackgerrit Artom Lifshitz proposed openstack/nova master: Service version check for NUMA live migration https://review.openstack.org/566723
00:38:00 openstackgerrit Artom Lifshitz proposed openstack/nova master: Service version check for NUMA live migration https://review.openstack.org/566723
00:52:27 openstackgerrit fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181
01:18:44 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: Service version check for NUMA live migration https://review.openstack.org/566723
01:22:47 openstackgerrit Merged openstack/nova master: Convert fping API to processutils.execute() https://review.openstack.org/564144
02:35:14 takashin
02:44:21 openstackgerrit fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181
04:11:02 openstackgerrit Merged openstack/nova master: Add docs for hw_video:ram_max_mb flavor extra spec https://review.openstack.org/567070
04:44:55 openstackgerrit Jake Yip proposed openstack/nova master: Add --before to nova-manage db archive_deleted_rows https://review.openstack.org/556751
04:46:32 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113
04:50:10 openstackgerrit Michael Still proposed openstack/nova master: Move image conversion to privsep. https://review.openstack.org/554437
04:50:11 openstackgerrit Michael Still proposed openstack/nova master: We don't need utils.trycmd any more. https://review.openstack.org/554439
04:50:12 openstackgerrit Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438
04:51:41 openstackgerrit Merged openstack/nova master: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/564964
04:54:36 openstackgerrit Michael Still proposed openstack/nova master: Move IVS VIF plugging into the new utilities module. https://review.openstack.org/557552
04:54:37 openstackgerrit Michael Still proposed openstack/nova master: Move create_tap_dev to the new utility module. https://review.openstack.org/557835
04:54:39 openstackgerrit Michael Still proposed openstack/nova master: Move set_vf_interface_vlan to the new utility module. https://review.openstack.org/558487
04:57:56 openstackgerrit Merged openstack/nova master: Convert xenapi's xvp console to processutils. https://review.openstack.org/564145
05:04:35 openstackgerrit Takahito Hirose proposed openstack/python-novaclient master: api_version decorator becomes an error in Python 3.5.0. https://review.openstack.org/564702
05:56:48 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113
07:01:43 openstackgerrit Merged openstack/nova master: Remove IronicHostManager and baremetal scheduling options https://review.openstack.org/565805
07:23:54 openstackgerrit fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181
07:52:17 openstackgerrit sahid proposed openstack/nova master: compute: introduce cpu_shared_set option https://review.openstack.org/561912
07:52:18 openstackgerrit sahid proposed openstack/nova master: libvirt: always pass emulator threads policy https://review.openstack.org/561913
07:52:19 openstackgerrit sahid proposed openstack/nova master: libvirt: place emulator threads on CONF.cpu_shared_set https://review.openstack.org/510897
08:20:03 openstackgerrit Jan Gutter proposed openstack/nova master: Add plumbing for Contrail/Tungsten Fabric offloads https://review.openstack.org/567147
08:22:15 openstackgerrit Jan Gutter proposed openstack/nova-specs master: Spec to implement vRouter HW offloads https://review.openstack.org/567148
08:26:50 stephenfin jaypipes: By mechanisms, I was referring to the various ways of a network can be configured and segmented, either L2 or L3. Local means connectivity is only on a single host https://gist.github.com/bonclay7/458b353f997e05db374d
08:26:52 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113
08:26:53 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Add traits check in nested provider cadidates https://review.openstack.org/567150
08:26:54 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Fix broken allocation candidates with traits https://review.openstack.org/567151
08:28:04 stephenfin jaypipes: Also https://ask.openstack.org/en/question/51388/
08:28:21 tetsuro ^ jauypipes, efried, bauzas: started nested allocation candidates branch in a different way.
08:31:34 kashyap stephenfin: Hey, when you can, see what's wrong with my test here: https://review.openstack.org/#/c/565242/5/nova/tests/unit/virt/libvirt/test_driver.py
08:33:27 openstackgerrit Merged openstack/nova master: Remove unused methods in nova/compute/utils.py https://review.openstack.org/566855
08:36:11 kashyap Merci!
08:36:38 kashyap stephenfin: It is mostly just me not being good with the unit tests. The change is quite simple, if you see the commit message.
08:36:44 kashyap (And the driver.py code)
08:49:58 openstackgerrit Merged openstack/nova master: Convert certificate generation to processutils. https://review.openstack.org/564146
09:09:23 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Add tests for sharing rps in test_common_rc() https://review.openstack.org/566842
09:11:03 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Add tests for alloc cands with poor local disk https://review.openstack.org/566842
09:11:28 stephenfin kashyap: Done
09:11:44 kashyap stephenfin: Thanks; will look
09:12:46 kashyap stephenfin: About actually being supported by the processor or not change. I first thought to remove it completely because, regardless of it, it's useless
09:12:49 kashyap But ...
09:13:11 kashyap Since the first change is to deprecate the support, I should retain it
09:15:33 stephenfin kashyap: No, you can remove it but call that out and fix the test accordingly
09:15:46 stephenfin No point keeping dead code around
09:16:04 kashyap stephenfin: Right. So the plan is: Deprecate it in Rocky; delete support for it in Stein.
09:16:22 kashyap I have it locally as you indicated
09:24:50 openstackgerrit lei zhang proposed openstack/python-novaclient master: Fix the incorrect cirros default password https://review.openstack.org/567170
09:28:25 openstackgerrit jichenjc proposed openstack/nova master: [Doc]Link policies file into api https://review.openstack.org/561503
09:30:27 kashyap stephenfin: With the 'assert_has_calls', you meant:
09:30:28 kashyap ])
09:30:28 kashyap mock.call(...),
09:30:28 kashyap mock_warn.assert_has_calls([
09:30:33 kashyap ?
09:30:36 stephenfin yup

Earlier   Later