[Ironic] test deployment of ironic nodes on fuel CI

Bug #1524773 reported by Vasyl Saienko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Vasyl Saienko
8.0.x
Fix Released
High
Vasyl Saienko
Mitaka
Fix Released
High
Vasyl Saienko

Bug Description

Fuel CI doesn't allow SSH connection to Hypervisor from test environment due to security reasons. But tcp connection to Libvirt socket is ok. At the moment ironic drivers support only SSH transport.
We need to have an Ironic driver that will allow to connect to Hypervisor libvirt socket via tcp (with/without SASL).
Also libvirt tcp socket should be opened from test environment on Hypervisor.

Vasyl Saienko (vsaienko)
Changed in fuel:
milestone: none → 8.0
Changed in fuel:
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-agent (master)

Fix proposed to branch: master
Review: https://review.openstack.org/255852

Changed in fuel:
assignee: nobody → Vasyl Saienko (vsaienko)
status: Confirmed → In Progress
tags: added: ironic
tags: added: enhancement
Changed in fuel:
importance: Undecided → Wishlist
Dmitry Pyzhov (dpyzhov)
tags: added: area-python feature
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-agent (master)

Reviewed: https://review.openstack.org/255852
Committed: https://git.openstack.org/cgit/openstack/fuel-agent/commit/?id=6ab2827fd336c73dbd25eae623075e49dbe269bb
Submitter: Jenkins
Branch: master

commit 6ab2827fd336c73dbd25eae623075e49dbe269bb
Author: vsaienko <email address hidden>
Date: Thu Dec 10 14:47:52 2015 +0200

    Add fuel_libvirt ironic driver

    This driver allows to perform Power/Management operation
    with VM via Libvirt API.

    Change-Id: Ia82f7fb45d2592d96ddcc423771bbd14df1e4e28
    Partial-Bug: #1524773

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to fuel-infra/puppet-manifests (master)

Related fix proposed to branch: master
Change author: Igor Shishkin <email address hidden>
Review: https://review.fuel-infra.org/14872

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to fuel-infra/puppet-manifests (master)

Reviewed: https://review.fuel-infra.org/14872
Submitter: Alexander Charykov <email address hidden>
Branch: master

Commit: 165f10e045037bf54a640232da4d3035b2e2e6eb
Author: Igor Shishkin <email address hidden>
Date: Mon Dec 21 11:55:38 2015

Moving out listen_addr from libvirt hardcode

It's required to make it possbile to specify it in hiera.

Change-Id: I470ef420c2ee70e398991c3158e2695da2fa6607
Related-Bug: #1524773
Signed-off-by: Igor Shishkin <email address hidden>

Revision history for this message
Vasyl Saienko (vsaienko) wrote :

It looks that ports are still closed:
vsaienko@srv49-bud:~$ python
Python 2.7.6 (default, Jun 22 2015, 17:58:13)
[GCC 4.8.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>>import libvirt
>>> conn = libvirt.open('qemu+tcp://10.109.1.1/system')
libvirt: XML-RPC error : unable to connect to server at '10.109.1.1:16509': Connection refused
Traceback (most recent call last):
 File "<stdin>", line 1, in <module>
 File "/usr/lib/python2.7/dist-packages/libvirt.py", line 252, in open
   if ret is None:raise libvirtError('virConnectOpen() failed')
libvirt.libvirtError: unable to connect to server at '10.109.1.1:16509': Connection refused

​[5:16]
vsaienko@srv49-bud:~$ netstat -an |grep 16509
tcp 0 0 127.0.0.1:16509 0.0.0.0:* LISTEN

​[5:16]
vsaienko@srv49-bud:~$ cat /etc/libvirt/libvirtd.conf |grep listen_addr
#listen_addr = "192.168.0.1"
listen_addr = "127.0.0.1"

Changed in fuel:
assignee: Vasyl Saienko (vsaienko) → Igor Shishkin (teran)
Revision history for this message
Igor Shishkin (teran) wrote :

@Vasyl, please create separate bug assigned to fuel-devops contains exact actions on our side: "Run puppet on nodes in <particular CI>" for instance.
This bug is not relevant for our particular task left.

Changed in fuel:
assignee: Igor Shishkin (teran) → Vasyl Saienko (vsaienko)
Revision history for this message
Vasyl Saienko (vsaienko) wrote :

Igor, not sure that it is correct flow, but ok. I've opened https://bugs.launchpad.net/fuel/+bug/1529240.

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

Reviewed: https://review.openstack.org/260600
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=199f69c0b607c8aef619d9ef592dd9a344c6d030
Submitter: Jenkins
Branch: master

commit 199f69c0b607c8aef619d9ef592dd9a344c6d030
Author: vsaienko <email address hidden>
Date: Tue Dec 22 17:34:25 2015 +0200

    Add fuel_libvirt to ironic enabled_drivers

    Enable fuel_libvirt driver.

    Change-Id: I270c97dd60e2ed1447c6e2a7b3c0b21512bc3434
    Depends-On: https://review.fuel-infra.org/14570/
    Related-Bug: #1524773

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-library (stable/8.0)

Related fix proposed to branch: stable/8.0
Review: https://review.openstack.org/261960

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-library (stable/8.0)

Reviewed: https://review.openstack.org/261960
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=9128aa803f97a8322b0904bb4feb9a455b09b60a
Submitter: Jenkins
Branch: stable/8.0

commit 9128aa803f97a8322b0904bb4feb9a455b09b60a
Author: vsaienko <email address hidden>
Date: Tue Dec 22 17:34:25 2015 +0200

    Add fuel_libvirt to ironic enabled_drivers

    Enable fuel_libvirt driver.

    Change-Id: I270c97dd60e2ed1447c6e2a7b3c0b21512bc3434
    Depends-On: https://review.fuel-infra.org/14570/
    Related-Bug: #1524773
    (cherry picked from commit 199f69c0b607c8aef619d9ef592dd9a344c6d030)

Revision history for this message
Vasyl Saienko (vsaienko) wrote :

Rising to high, deployment step should be covered by CI tests.

Changed in fuel:
importance: Wishlist → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-qa (stable/8.0)

Related fix proposed to branch: stable/8.0
Review: https://review.openstack.org/265354

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

Related fix proposed to branch: master
Review: https://review.openstack.org/265355

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-qa (stable/8.0)

Change abandoned by Vasyl Saienko (<email address hidden>) on branch: stable/8.0
Review: https://review.openstack.org/265354

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

Reviewed: https://review.openstack.org/265355
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=689b7c409df2236027ae16c1a05bf75090f45899
Submitter: Jenkins
Branch: master

commit 689b7c409df2236027ae16c1a05bf75090f45899
Author: vsaienko <email address hidden>
Date: Fri Jan 8 08:12:41 2016 +0000

    Introduce deployment test for Ironic

    Depends-On: I15dbde05327d45f96894a9fa31e2e685cfef1033
    Related-Bug: #1524773
    Change-Id: Ife957ca0fcb3780b228a0bcafbd7cf5d5985a2b3

tags: added: area-ironic
removed: area-python ironic
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to fuel-infra/jenkins-jobs (master)

Fix proposed to branch: master
Change author: Vasyl Saienko <email address hidden>
Review: https://review.fuel-infra.org/16348

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to fuel-infra/jenkins-jobs (master)

Related fix proposed to branch: master
Change author: Vasyl Saienko <email address hidden>
Review: https://review.fuel-infra.org/16350

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote :

Related fix proposed to branch: master
Change author: Vasyl Saienko <email address hidden>
Review: https://review.fuel-infra.org/16403

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote :

Related fix proposed to branch: master
Change author: Vasyl Saienko <email address hidden>
Review: https://review.fuel-infra.org/16404

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to fuel-infra/jenkins-jobs (master)

Reviewed: https://review.fuel-infra.org/16348
Submitter: Dmitry Kaigarodsev <email address hidden>
Branch: master

Commit: ddf20920ef81fa1c5811d4d6e39d0e74c03554f0
Author: Vasyl Saienko <email address hidden>
Date: Mon Jan 25 15:43:03 2016

Update ironic swarm job

Add testing for ironic provisioning with swift
Increase SLAVE_NODE_MEMORY to 4096

Change-Id: I86e1d1e5ba5205f01f13622b380f079cf5d797e5
Related-Bug: #1524773

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote :

Reviewed: https://review.fuel-infra.org/16350
Submitter: Dmitry Kaigarodsev <email address hidden>
Branch: master

Commit: 4daa58deeb330beebea8f0bcee929f1eced22a77
Author: Vasyl Saienko <email address hidden>
Date: Mon Jan 25 15:45:36 2016

Add ironic_deploy_ceph swarm job

Add testing for ironic provisioning with ceph

Change-Id: I918322735ee1a63f1ececcefad499a8d1fca1953
Related-Bug: #1524773

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote :

Reviewed: https://review.fuel-infra.org/16403
Submitter: Dmitry Kaigarodsev <email address hidden>
Branch: master

Commit: 23435fcd9042ba801c1c04fb8b33ce573115769c
Author: Vasyl Saienko <email address hidden>
Date: Mon Jan 25 15:48:33 2016

Update ironic swarm job 8.0

Add testing for ironic provisioning with swift
Increase SLAVE_NODE_MEMORY to 4096

Change-Id: I4a7209434ea3824b0afce3059fe75cfac0c862c3
Related-Bug: #1524773

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote :

Reviewed: https://review.fuel-infra.org/16404
Submitter: Dmitry Kaigarodsev <email address hidden>
Branch: master

Commit: 8651bd0c30d700e670bf4d5b4bdc28d1736195a3
Author: Vasyl Saienko <email address hidden>
Date: Mon Jan 25 15:51:36 2016

Add ironic_deploy_ceph swarm job 8.0

Add testing for ironic provisioning with ceph

Change-Id: Ic3ffd999d934c9a8f45dcb8e2ca05a04294feeb5
Related-Bug: #1524773

Revision history for this message
Vasyl Saienko (vsaienko) wrote :

swarm jobs has been added. Libivirt socket was opened on jenkins slaves.

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.