[nailgun] Verify a vCenter credentials consistency

Bug #1406231 reported by Andrey Danin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Andriy Popovych
6.0.x
Won't Fix
Medium
Andriy Popovych
6.1.x
Fix Released
Medium
Andriy Popovych

Bug Description

Affects: Fuel 6.0 and 6.1
vCenter credentials are verified only at the Web UI side. If a user modifies credentials via CLI
they wouldn't be verified.

I propose to implement such verification as a part of CheckBeforeDeploymentTask [1].

The next cases should be implemented:

1) If 'libvirt_type==vcenter' [2]:L651 then at least 'host_ip' [2]:L409, 'vc_user' [2]:L424,
   'vc_password' [2]:L433, 'cluster' [2]:L442 provided and their regexes are valid.

2) If 'volumes_vmdk==true' [2]:L861 and at least one node with 'cinder' role assigned
   then at least 'host_ip' [2]:L409, 'vc_user' [2]:L424, 'vc_password' [2]:L433
   provided and their regexes are valid.

3) If 'images_vcenter==true' [2]:L891 then at least 'vc_host' [2]:L899, 'vc_user' [2]:L911,
   'vc_password' [2]:L921, 'vc_datastore' [2]:L931, 'vc_datacenter' [2]:L941 provided and
   their regexes are valid.

[1] https://github.com/stackforge/fuel-web/blob/master/nailgun/nailgun/task/task.py#L580
[2] https://github.com/stackforge/fuel-web/blob/master/nailgun/nailgun/fixtures/openstack.yaml

Changed in fuel:
assignee: Fuel Partner Integration Team (fuel-partner) → Andriy Popovych (popovych-andrey)
summary: - erify a vCenter credentials consistency at the Nailgun side
+ verify a vCenter credentials consistency at the Nailgun side
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

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

summary: - verify a vCenter credentials consistency at the Nailgun side
+ [nailgun] verify a vCenter credentials consistency
summary: - [nailgun] verify a vCenter credentials consistency
+ [nailgun] Verify a vCenter credentials consistency
Revision history for this message
Ryan Moe (rmoe) wrote :

Medium priority bugs are not backported to stable releases.

Revision history for this message
Andriy Popovych (popovych-andrey) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-web (master)

Change abandoned by Andriy Popovych (<email address hidden>) on branch: master
Review: https://review.openstack.org/146235
Reason: It's implementation deprecated.
New https://review.openstack.org/160474

Revision history for this message
Andriy Popovych (popovych-andrey) wrote :
Revision history for this message
Olesia Tsvigun (otsvigun) wrote :

Verified at ISO 310
Host operating system for OpenStack (CentOS or Ubuntu)

api: '1.0'
astute_sha: d96a80b63198a578b2c159edbd76048819039eb0
auth_required: true
build_id: 2015-04-13_22-54-31
build_number: '310'
feature_groups:
- mirantis
fuellib_sha: 8b80657e9ceed8d59c2dff1c11e1481c7e69380e
fuelmain_sha: 335d3ed09ed79bd37e1f7a90442c4831c8845582
nailgun_sha: d22c074dec091e5ddd8ea3003c37665058303cd5
openstack_version: 2014.2-6.1
ostf_sha: c2a76a60ec4ebbd78e508216c2e12787bf25e423
production: docker
python-fuelclient_sha: 9208ff4a08dcb674ce2df132399a5aa3ddfac21c
release: '6.1'
release_versions:
  2014.2-6.1:
    VERSION:
      api: '1.0'
      astute_sha: d96a80b63198a578b2c159edbd76048819039eb0
      build_id: 2015-04-13_22-54-31
      build_number: '310'
      feature_groups:
      - mirantis
      fuellib_sha: 8b80657e9ceed8d59c2dff1c11e1481c7e69380e
      fuelmain_sha: 335d3ed09ed79bd37e1f7a90442c4831c8845582
      nailgun_sha: d22c074dec091e5ddd8ea3003c37665058303cd5
      openstack_version: 2014.2-6.1
      ostf_sha: c2a76a60ec4ebbd78e508216c2e12787bf25e423
      production: docker
      python-fuelclient_sha: 9208ff4a08dcb674ce2df132399a5aa3ddfac21c
      release: '6.1'

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.