Loadbalancer Monitor screen requires unneeded fields for PING and TCP monitor types

Bug #1152138 reported by Ilya Shakhat
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Low
Ilya Shakhat

Bug Description

Fields shown in Add Monitor form should depend on type of monitor. For PING and TCP types fields 'HTTP Method', 'URL', 'Expected Codes' are not needed and should be hidden. Moreover, if 'Expected Codes' is clear then the application crashes with error: "Invalid input for expected_codes. Reason: '' is not a valid input."

Tags: lbaas quantum
Ilya Shakhat (shakhat)
Changed in horizon:
assignee: nobody → Ilya Shakhat (shakhat)
Ilya Shakhat (shakhat)
Changed in horizon:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

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

Changed in horizon:
importance: Undecided → Low
milestone: none → grizzly-rc1
Changed in horizon:
milestone: grizzly-rc1 → havana-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/23815
Committed: http://github.com/openstack/horizon/commit/b4923b5deedfe880cbb12bcf49cd6bfcd06b6286
Submitter: Jenkins
Branch: master

commit b4923b5deedfe880cbb12bcf49cd6bfcd06b6286
Author: Ilya Shakhat <email address hidden>
Date: Thu Mar 7 17:16:39 2013 +0400

    Hide unnecessary fields based on Loadbalancer Monitor type

    Change-Id: I83a56c8067af8f6ec87d89192156658232ad87cf
    Fixes: bug 1152138

Changed in horizon:
status: In Progress → Fix Committed
Akihiro Motoki (amotoki)
tags: added: lbaas quantum
Thierry Carrez (ttx)
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: havana-1 → 2013.2
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.