Remove unnecessary check in storm plugin

Bug #1400822 reported by Telles Mota Vidal Nóbrega
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Sahara
Fix Released
Low
Telles Mota Vidal Nóbrega

Bug Description

The check for the port is not necessary in this context

def _set_cluster_info(self, cluster):
        st_master = utils.get_instance(cluster, "nimbus")
        info = {}

        if st_master:
            port = "8080"

            if port is not None:
                info['Strom'] = {
                    'Web UI': 'http://%s:%s' % (st_master.management_ip, port)
                }
        ctx = context.ctx()
        conductor.cluster_update(ctx, cluster, {'info': info})

Changed in sahara:
status: New → Confirmed
assignee: nobody → Telles Mota Vidal Nóbrega (tellesmvn)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to sahara (master)

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

Changed in sahara:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to sahara (master)

Reviewed: https://review.openstack.org/140424
Committed: https://git.openstack.org/cgit/openstack/sahara/commit/?id=49400a30da4c1accb8693fa07fe7fc6ab52733c5
Submitter: Jenkins
Branch: master

commit 49400a30da4c1accb8693fa07fe7fc6ab52733c5
Author: Telles Nobrega <email address hidden>
Date: Tue Dec 9 15:18:36 2014 -0300

    Removing unecessary check

    The check for the port is not necessary in this context.
    This patch removes it

    Change-Id: Ie2b8e69ba7d017d66f214727dc06ded64dcb9215
    Closes-bug: #1400822

Changed in sahara:
status: In Progress → Fix Committed
Changed in sahara:
importance: Undecided → Low
milestone: none → kilo-1
Thierry Carrez (ttx)
Changed in sahara:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in sahara:
milestone: kilo-1 → 2015.1.0
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.