Cluster attrubute use_vcenter value is always true

[Alexander Kislitsky]

Cluster attribute vcenter.use_vcenter is always True.

[Ihor Kalnytskyi (ikalnytskyi)]

Is this really 6.0.1 specific? if not, we need to target to 6.1 also.

[Aleksey Kasatkin (alekseyk-ru)]

This parameter is not in use, actually. Seems, it can be dropped from fixture. But its value is saved into stats snapshot. Let's drop it from snapshot then.

[Aleksey Kasatkin (alekseyk-ru)]

Igor G. , please comment.

[Igor Zinovik (izinovik)]

I think we can safely drop 'use_vcenter', it is not used in fuel-library/ puppet manifests. In manifests we rely on $::fuel_settings['libvirt_type'] = 'vcenter', not no 'use_vcenter' = true.

I checked against fuel-library/ master branch as of 15 Jan 2015.

[Igor Gajsin (igajsin)]

I agree. Commit will be sended to gerrit a little later.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/147484
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=f858b6a703a86358b202cf11e405d18ce758f0cc
Submitter: Jenkins
Branch: master

commit f858b6a703a86358b202cf11e405d18ce758f0cc
Author: Igor Gajsin <email address hidden>
Date: Thu Jan 15 15:11:06 2015 +0300

    Remove useless attribute use_vcenter from openstack.yaml

    There is parameter use_vcenter in openstack.yaml file which always equal true
    and never used. This patch just remove unused parameter.

    Change-Id: Ifa91691516140861b33bfdb2c562481061834b62
    Closes-bug: #1410163

[OpenStack Infra (hudson-openstack)]

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

[Tatyana Dubyk (tdubyk)]

Verified on 6.1 (#64 iso)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/148220
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=bdbcc92b870af37dff2466348a0f2a392b73d1b0
Submitter: Jenkins
Branch: master

commit bdbcc92b870af37dff2466348a0f2a392b73d1b0
Author: Igor Zinovik <email address hidden>
Date: Mon Jan 19 14:22:55 2015 +0300

    Zap unused variable 'use_vcenter' that is no longer exist

    - system tests rely on non-existent variable 'use_vcenter', instead of
      relying on variable that is always set to true, tests must rely on
      hypervisor type
    - commit f858b6a703a86358b202cf11e405d18ce758f0cc in fuel-web/
      repository removed 'use_vcenter', justification for this can be found
      in bug #1410163

    Change-Id: I454697d7d160be4197abb5791b609948baf009bb