[pci]/alias option is not present in the nova configuration on the computes

Bug #1702730 reported by Sai Sindhur Malleni
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
puppet-nova
Fix Released
Medium
Oliver Walsh

Bug Description

[pci]/alias option is not being configured by the compute manifest. This option is needed in the nova configuration file on compute nodes starting with Ocata.

summary: - alias option is not present in the nova configuration on the computes
+ [pci]/alias option is not present in the nova configuration on the
+ computes
Changed in puppet-nova:
assignee: nobody → Sai Sindhur Malleni (smalleni)
status: New → In Progress
Changed in puppet-nova:
importance: Undecided → Medium
Revision history for this message
Oliver Walsh (owalsh) wrote :

IIRC the conclusion was that this isn't required and it was a doc issue instead

Changed in puppet-nova:
status: In Progress → Invalid
Revision history for this message
Johan Guldmyr (jguldmyr) wrote :

With Newton if I only have pci_alias in the api node's nova.conf, when I do a resize it says "alias not defined".

Of course it would be convenient if nova-compute asked the api node.

https://docs.openstack.org/nova/latest/admin/pci-passthrough.html

"From the Newton release, to resize guest with PCI device, configure the PCI alias on the compute node as well."

Revision history for this message
Johan Guldmyr (jguldmyr) wrote :

Not a documentation issue. Set to status new.

Changed in puppet-nova:
status: Invalid → New
Revision history for this message
Oliver Walsh (owalsh) wrote :

Confirmed this is expected behaviour with moshele on #openstack-nova IRC channel

Changed in puppet-nova:
status: New → Confirmed
Changed in puppet-nova:
assignee: Sai Sindhur Malleni (smalleni) → Oliver Walsh (owalsh)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-nova (master)

Reviewed: https://review.openstack.org/476327
Committed: https://git.openstack.org/cgit/openstack/puppet-nova/commit/?id=c1a4ab211dd2322572349719379cd13c6f2abb9a
Submitter: Jenkins
Branch: master

commit c1a4ab211dd2322572349719379cd13c6f2abb9a
Author: Sai Sindhur Malleni <email address hidden>
Date: Wed Jun 21 23:44:26 2017 -0400

    Add pci/alias to compute manifest

    Starting with Ocata pci_alias needs to be configured on compute nodes as
    well.

    This requires refactoring of the manifests as the parameter is common to
    nova::compute and nova::api.
    Common pci configuration is now done by the nova::pci class while compute
    specific configuration is done now by the nova::compute::pci class.

    Closes-Bug: #1702730
    Change-Id: Iac4f22cbe043629ae9f0477ae62cf77ad0d761f8

Changed in puppet-nova:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-nova (stable/pike)

Fix proposed to branch: stable/pike
Review: https://review.openstack.org/507435

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-nova (stable/ocata)

Fix proposed to branch: stable/ocata
Review: https://review.openstack.org/507458

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-nova (stable/ocata)

Reviewed: https://review.openstack.org/507458
Committed: https://git.openstack.org/cgit/openstack/puppet-nova/commit/?id=82966150a1f8f1e60b31102d1edb1c99de986b1f
Submitter: Jenkins
Branch: stable/ocata

commit 82966150a1f8f1e60b31102d1edb1c99de986b1f
Author: Sai Sindhur Malleni <email address hidden>
Date: Wed Jun 21 23:44:26 2017 -0400

    Add pci/alias to compute manifest

    Starting with Ocata pci_alias needs to be configured on compute nodes as
    well.

    This requires refactoring of the manifests as the parameter is common to
    nova::compute and nova::api.
    Common pci configuration is now done by the nova::pci class while compute
    specific configuration is done now by the nova::compute::pci class.

    Closes-Bug: #1702730
    Change-Id: Iac4f22cbe043629ae9f0477ae62cf77ad0d761f8
    (cherry picked from commit c1a4ab211dd2322572349719379cd13c6f2abb9a)

tags: added: in-stable-ocata
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-nova (stable/pike)

Reviewed: https://review.openstack.org/507435
Committed: https://git.openstack.org/cgit/openstack/puppet-nova/commit/?id=16e88fcf02adc9a4de3631c96cdc02ec4e80d6c7
Submitter: Jenkins
Branch: stable/pike

commit 16e88fcf02adc9a4de3631c96cdc02ec4e80d6c7
Author: Sai Sindhur Malleni <email address hidden>
Date: Wed Jun 21 23:44:26 2017 -0400

    Add pci/alias to compute manifest

    Starting with Ocata pci_alias needs to be configured on compute nodes as
    well.

    This requires refactoring of the manifests as the parameter is common to
    nova::compute and nova::api.
    Common pci configuration is now done by the nova::pci class while compute
    specific configuration is done now by the nova::compute::pci class.

    Closes-Bug: #1702730
    Change-Id: Iac4f22cbe043629ae9f0477ae62cf77ad0d761f8
    (cherry picked from commit c1a4ab211dd2322572349719379cd13c6f2abb9a)

tags: added: in-stable-pike
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-nova 11.4.0

This issue was fixed in the openstack/puppet-nova 11.4.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-nova 12.0.0

This issue was fixed in the openstack/puppet-nova 12.0.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-nova 10.5.0

This issue was fixed in the openstack/puppet-nova 10.5.0 release.

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.