[RFE] Port NUMA affinity policy

Bug #1886798 reported by Rodolfo Alonso
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Wishlist
Rodolfo Alonso

Bug Description

The aim of this RFE is to add a new parameter to the port object. This new parameter, "numa_affinity_policy", specifies the NUMA affinity policy for a single port.

This parameter will be passed to Nova, along with the port resource request, when a new server is created; that implies the server creation command contains a pre-created port.

Nova will use this information to schedule the VM and the port filtering with NUMATopologyFilter.

The parameter, for backwards compatibility, will be "None" by default and no information will be provided in the port resource request.

Tags: rfe-approved
Changed in neutron:
assignee: nobody → Rodolfo Alonso (rodolfo-alonso-hernandez)
importance: Undecided → Wishlist
summary: - [RFE] Port NUMA node affinity
+ [RFE] Port NUMA node policy affinity
summary: - [RFE] Port NUMA node policy affinity
+ [RFE] Port NUMA affinity policy
description: updated
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron-specs (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/740011

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron-lib (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/740058

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.opendev.org/740067

Changed in neutron:
status: New → In Progress
tags: added: rfe-triaged
Revision history for this message
Rodolfo Alonso (rodolfo-alonso-hernandez) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/740269

Revision history for this message
Slawek Kaplonski (slaweq) wrote :

We discussed that RFE on our last drivers meeting on 10.07.2020 and we agreed to approve it. Lets discuss about details in the review of the spec.

tags: added: rfe-approved
removed: rfe-triaged
Changed in neutron:
milestone: none → next
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron-specs (master)

Reviewed: https://review.opendev.org/740011
Committed: https://git.openstack.org/cgit/openstack/neutron-specs/commit/?id=a6fa3a5925206107dd2b834694ad4b0450a5077d
Submitter: Zuul
Branch: master

commit a6fa3a5925206107dd2b834694ad4b0450a5077d
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Wed Jul 8 14:47:24 2020 +0000

    Port NUMA affinity policy

    Change-Id: I308fbd407a56ef2984ac81a0ad9175c98077bfda
    Related-Bug: #1886798

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron-specs (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/744116

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (master)

Change abandoned by Rodolfo Alonso Hernandez (<email address hidden>) on branch: master
Review: https://review.opendev.org/740269
Reason: This feature is not going to provide the NUMA policy affinity through the port resource request but using a specific extension. We don't need this refactor.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron-lib (master)

Reviewed: https://review.opendev.org/740058
Committed: https://git.openstack.org/cgit/openstack/neutron-lib/commit/?id=a3961d7f7ab9e29ca7e4cdd5e646cb2ab812a88a
Submitter: Zuul
Branch: master

commit a3961d7f7ab9e29ca7e4cdd5e646cb2ab812a88a
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Wed Jul 8 16:02:29 2020 +0000

    New api-def: port-numa-affinity-policy

    New API definition: ``port-numa-affinity-policy``. The new field
    added to ``ports`` API, ``numa_affinity_policy``, represents the NUMA
    affinity policy requested by this port during the virtual machine
    scheduling.

    Values: "None", "required", "preferred", "legacy".

    Change-Id: I5402a8ad462a10c51dc251cb4561b64dbf0b8744
    Related-Bug: #1886798

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron-specs (master)

Reviewed: https://review.opendev.org/744116
Committed: https://git.openstack.org/cgit/openstack/neutron-specs/commit/?id=8922ff334ec1b065b4e2e379033616414b93768d
Submitter: Zuul
Branch: master

commit 8922ff334ec1b065b4e2e379033616414b93768d
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Fri Jul 31 09:56:21 2020 +0000

    Port NUMA affinity policy, revisit

    Since the conversation reported in [1], the information to be reported
    to Nova will not be stored in the port "resource_request" and the
    NUMA affinity policies will not be considered as resource provider
    traits.

    NUMA is going to be modeled in Placement soon, but until this feature
    does not land in a stable version, Neutron will only provide this
    information via an extension.

    Related-Bug: #1886798

    [1]https://review.opendev.org/#/c/739961/

    Change-Id: I2a96e5ba394f4476121cbc916634e1c4d4b4212f

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.opendev.org/740067
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=a217a5d29094bd68196bff54f8055defdc9ffe13
Submitter: Zuul
Branch: master

commit a217a5d29094bd68196bff54f8055defdc9ffe13
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Wed Jul 8 16:13:51 2020 +0000

    Add port NUMA affinity policy

    Added a new port extension: NUMA affinity policy. This extension adds
    the "numa_affinity_policy" parameter to the "port" API and specifies
    the NUMA affinity policy per port.

    This parameter is passed to Nova when a virtual machine is created.
    Nova will use this information to schedule the virtual machine.

    For backwards compatibility, this parameter will be "None" by default.

    Depends-On: https://review.opendev.org/#/c/740058/
    Closes-Bug: #1886798

    Change-Id: Ie3d68c098ddb727ab8333aa1de4064e67a4f00a7

Changed in neutron:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 18.0.0.0rc1

This issue was fixed in the openstack/neutron 18.0.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron-lib (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron-lib/+/832832

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron-lib (master)

Reviewed: https://review.opendev.org/c/openstack/neutron-lib/+/832832
Committed: https://opendev.org/openstack/neutron-lib/commit/096fb92766007a6cdb01d94d26217c54c1039b30
Submitter: "Zuul (22348)"
Branch: master

commit 096fb92766007a6cdb01d94d26217c54c1039b30
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Fri Mar 4 22:28:54 2022 +0000

    Rehome ``NumaAffinityPoliciesEnumField``

    Change-Id: Ifa72655b272bb466fff5fd9c84f6b833a673e86c
    Related-Bug: #1886798

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron-lib (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron-lib/+/833941

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron-lib (master)

Reviewed: https://review.opendev.org/c/openstack/neutron-lib/+/833941
Committed: https://opendev.org/openstack/neutron-lib/commit/5ef0d26ad372031c4118304f29bebb963a7981f4
Submitter: "Zuul (22348)"
Branch: master

commit 5ef0d26ad372031c4118304f29bebb963a7981f4
Author: Brian Haley <email address hidden>
Date: Tue Mar 15 20:34:25 2022 -0400

    Add NumaAffinityPoliciesEnumField class unit tests

    Prior change rehomed this class here, add some basic
    unit tests to cover it.

    Change-Id: Ide8229f101456bc448f4acc78ff74b879809261a
    Related-Bug: #1886798

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.