Neutron should not try to bind port on compute with hypervisor_type ironic

[Vasyl Saienko]

Neutron tries to bind port on compute where instance is launched. It doesn't make sense when hypervisor_type is ironic, since VM does not live on hypervisor in this case. Furthermore it leads to failed provisioning of baremetal node, when neutron is not configured on ironic compute node.

Setup:
node-1: controller
node-2: ironic-compute without neutron

neutron-server.log: http://paste.openstack.org/show/445388/

[Vasyl Saienko (vsaienko)]

upstream bug: https://bugs.launchpad.net/neutron/+bug/1492254

[Vitaly Sedelnik (vsedelnik)]

Confirmed for 8.0 and Incomplete for 7.0 because of unclear status of Ironic support in 7.0.

Ironic team - please update the status to Confirmed for 7.0 if Ironic plugin will support MOS 7.0 and reassign to Neutron team. Otherwise set the status to Invalid, thanks.

[Pavlo Shchelokovskyy (pshchelo)]

we are currently solving this problem in plugin by installing neutron agents on Ironic node

as plugin will not be official (we do not plan to apply for certification) setting this as Invalid for 7.0.

[Roman Podoliaka (rpodolyaka)]

Decreasing to Medium, since Ironic is not top-priority for 8.0

[Serge Kovaleff (serge-kovaleff)]

Roman, could we increase the priority?

The Ironic integration is done. This bug is essential for Ironic proper functioning.

[Andrey Shestakov (ashestakov)]

Logs:
http://paste.openstack.org/show/481171/

[Fuel Devops McRobotson (fuel-devops-robot)]

Fix proposed to branch: openstack-ci/fuel-8.0/liberty
Change author: Pavlo Shchelokovskyy <email address hidden>
Review: https://review.fuel-infra.org/14558

[Fuel Devops McRobotson (fuel-devops-robot)]

Change abandoned by Pavlo Shchelokovskyy <email address hidden> on branch: openstack-ci/fuel-8.0/liberty
Review: https://review.fuel-infra.org/14558
Reason: re-proposing as https://review.fuel-infra.org/#/c/15525/ due to peculiarities of our CI choosing what patches to build package with

[Fuel Devops McRobotson (fuel-devops-robot)]

Reviewed: https://review.fuel-infra.org/15524
Submitter: Pkgs Jenkins <email address hidden>
Branch: openstack-ci/fuel-8.0/liberty

Commit: 11d14f95caff74e28ae64b87c67a52c8907cb25a
Author: Jim Rollenhagen <email address hidden>
Date: Thu Dec 31 13:39:01 2015

Allow virt driver to define binding:host_id

This patch allows the virt driver to define the host_id field, with
instance.host as the base implementation. This is passed into the
networks API and sent to Neutron with the port-create calls.

Ironic ports will need to be created with an empty host_id field in
order to do late binding of the Neutron ports. For backwards
compatibility, the old behavior should be used when
node.network_provider is 'none'.

Related-Bug: #1492272
Implements: blueprint ironic-networks-support
Change-Id: I113b04203e8b4eb10158589be7b6f76f292884b5
(cherry picked from commit 0e6e28aed74e8029e394bb16298436a239c7192e)

[Fuel Devops McRobotson (fuel-devops-robot)]

Reviewed: https://review.fuel-infra.org/15525
Submitter: Pkgs Jenkins <email address hidden>
Branch: openstack-ci/fuel-8.0/liberty

Commit: 21e9962584ea088bdb61010635b2a91f7a9d6b2a
Author: Pavlo Shchelokovskyy <email address hidden>
Date: Mon Jan 4 11:32:32 2016

Never bind ports to Ironic computes

Binding ports precludes ironic-computes from functioning
since we use specific compute host name instead of standard one.

Besides, this step is not really needed in case of nova-computes
that use ironic virt-driver as the baremetal instance is not
actually placed on this compute.

Closes-Bug: #1492272
Change-Id: Ie5261ffb0625bd26c2c4bfab009610912ca9d05c

[Kyrylo Romanenko (kromanenko)]

Verified on ISO 529
VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "8.0"
  api: "1.0"
  build_number: "529"
  build_id: "529"
  fuel-nailgun_sha: "baec8643ca624e52b37873f2dbd511c135d236d9"
  python-fuelclient_sha: "4f234669cfe88a9406f4e438b1e1f74f1ef484a5"
  fuel-agent_sha: "658be72c4b42d3e1436b86ac4567ab914bfb451b"
  fuel-nailgun-agent_sha: "b2bb466fd5bd92da614cdbd819d6999c510ebfb1"
  astute_sha: "b81577a5b7857c4be8748492bae1dec2fa89b446"
  fuel-library_sha: "e2d79330d5d708796330fac67722c21f85569b87"
  fuel-ostf_sha: "3bc76a63a9e7d195ff34eadc29552f4235fa6c52"
  fuel-mirror_sha: "fb45b80d7bee5899d931f926e5c9512e2b442749"
  fuelmenu_sha: "e071216cb214e34b4d861478033425ee6a54a3be"
  shotgun_sha: "63645dea384a37dde5c01d4f8905566978e5d906"
  network-checker_sha: "a43cf96cd9532f10794dce736350bf5bed350e9d"
  fuel-upgrade_sha: "616a7490ec7199f69759e97e42f9b97dfc87e85b"
  fuelmain_sha: "a365f05b903368225da3fea9aa42afc1d50dc9b4"

[Fuel Devops McRobotson (fuel-devops-robot)]

Fix proposed to branch: 9.0/mitaka
Change author: Pavlo Shchelokovskyy <email address hidden>
Review: https://review.fuel-infra.org/18323

[Fuel Devops McRobotson (fuel-devops-robot)]

Related fix proposed to branch: 9.0/mitaka
Change author: Jim Rollenhagen <email address hidden>
Review: https://review.fuel-infra.org/18340

[Fuel Devops McRobotson (fuel-devops-robot)]

Change abandoned by Roman Podoliaka <email address hidden> on branch: 9.0/mitaka
Review: https://review.fuel-infra.org/18340
Reason: Per Pavlo's comment we don't need this anymore.

[Fuel Devops McRobotson (fuel-devops-robot)]

Reviewed: https://review.fuel-infra.org/18323
Submitter: Pkgs Jenkins <email address hidden>
Branch: 9.0/mitaka

Commit: 8b8a816fe024d0af04815ef7b0145861789078a0
Author: Pavlo Shchelokovskyy <email address hidden>
Date: Fri Apr 1 10:11:25 2016

Never bind ports to Ironic computes

Binding ports precludes ironic-computes from functioning
since we use specific compute host name instead of standard one.

Besides, this step is not really needed in case of nova-computes
that use ironic virt-driver as the baremetal instance is not
actually placed on this compute.

Closes-Bug: #1492272
Change-Id: Ie5261ffb0625bd26c2c4bfab009610912ca9d05c

[Serge Kovaleff (serge-kovaleff)]

Release as part of 9.0 Release.

[Fuel Devops McRobotson (fuel-devops-robot)]

Fix proposed to branch: 10.0/newton
Change author: Pavlo Shchelokovskyy <email address hidden>
Review: https://review.fuel-infra.org/32479

[Fuel Devops McRobotson (fuel-devops-robot)]

Change abandoned by Pavlo Shchelokovskyy <email address hidden> on branch: 10.0/newton
Review: https://review.fuel-infra.org/32479
Reason: this patch is applied from build-spec when building package, so it should already be there

[Fuel Devops McRobotson (fuel-devops-robot)]

Fix proposed to branch: mcp/newton
Change author: Pavlo Shchelokovskyy <email address hidden>
Review: https://review.fuel-infra.org/33647

[Fuel Devops McRobotson (fuel-devops-robot)]

Fix proposed to branch: 11.0/ocata
Change author: Pavlo Shchelokovskyy <email address hidden>
Review: https://review.fuel-infra.org/34447

[Fuel Devops McRobotson (fuel-devops-robot)]

Fix proposed to branch: mcp/ocata
Change author: Pavlo Shchelokovskyy <email address hidden>
Review: https://review.fuel-infra.org/34832

[Fuel Devops McRobotson (fuel-devops-robot)]

Change abandoned by Roman Podoliaka <email address hidden> on branch: 11.0/ocata
Review: https://review.fuel-infra.org/34447
Reason: 11.0/ocata is deprecated in favor of mcp/ocata

[Fuel Devops McRobotson (fuel-devops-robot)]

Change abandoned by Roman Podoliaka <email address hidden> on branch: mcp/ocata
Review: https://review.fuel-infra.org/34832
Reason: not needed after I4d70423ca978885a982c7eb5bd1efcc024d2b777 was merged

[Fuel Devops McRobotson (fuel-devops-robot)]

Reviewed: https://review.fuel-infra.org/33647
Submitter: Pkgs Jenkins <email address hidden>
Branch: mcp/newton

Commit: 06ffc263d96ceab4a31825071d526c33c58429d5
Author: Pavlo Shchelokovskyy <email address hidden>
Date: Wed Apr 26 13:11:28 2017

Never bind ports to Ironic computes

Binding ports precludes ironic-computes from functioning
since we use specific compute host name instead of standard one.

Besides, this step is not really needed in case of nova-computes
that use ironic virt-driver as the baremetal instance is not
actually placed on this compute.

Closes-Bug: #1492272
Change-Id: Ie5261ffb0625bd26c2c4bfab009610912ca9d05c