missing option 'use_vcenter' in ComputeGroup

Bug #1575646 reported by Oleksandr Kyrylchuk
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Low
Oleksandr Kyrylchuk

Bug Description

Option 'use_vcenter' is not declared in ComputeGroup explicitly. Proof: https://github.com/openstack/fuel-ostf/blob/master/fuel_health/config.py#L109-#L209

This option is just initialized in _parse_cluster_attributes: https://github.com/openstack/fuel-ostf/blob/master/fuel_health/config.py#L641

It would be more correct if this option was presented in ComputeGroup.

Tags: area-ostf
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-ostf (master)

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

Changed in fuel:
assignee: nobody → Oleksandr Kyrylchuk (okyrylchuk)
status: New → In Progress
Changed in fuel:
importance: Undecided → Low
milestone: none → 10.0
tags: added: area-ostf
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-ostf (master)

Reviewed: https://review.openstack.org/310455
Committed: https://git.openstack.org/cgit/openstack/fuel-ostf/commit/?id=02035661eace89613bd19a42113a2fcdbf9a1789
Submitter: Jenkins
Branch: master

commit 02035661eace89613bd19a42113a2fcdbf9a1789
Author: Oleksandr Kyrylchuk <email address hidden>
Date: Wed Apr 27 15:48:15 2016 +0300

    Added 'use_vcenter' to the list of options

    Reasons:
        - 'use_vcenter' is not declared in option list of ComputeGroup
          but assigned later in _parse_cluster_attributes()
    Changes:
        - Added option 'use_vcenter' to ComputeGroup

    Change-Id: I066b59b46413701233c9723a60645edfe1f1f7fb
    Closes-Bug: 1575646

Changed in fuel:
status: In Progress → Fix Committed
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.