Need to add validation for the fields count, interval and timeout on LBaaS health check monitor

Bug #1750738 reported by musharani
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
Fix Committed
Medium
Suresh Akula

Bug Description

In LBaaS health check monitor page validation is missing for the text fields Health Check Interval, Retry count before mark down, Timeout. Because it is not giving any error message if I give 'ee' in that fields.

Need to add validation for HTTP status code as well if we choose protocol as http

Tags: ui
Revision history for this message
musharani (musharani) wrote :
musharani (musharani)
description: updated
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/39994
Submitter: Suresh Akula (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/39994
Committed: http://github.com/Juniper/contrail-web-controller/commit/8489bf93d2bfd1eeabaedfac7a806849a34bb54a
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 8489bf93d2bfd1eeabaedfac7a806849a34bb54a
Author: Suresh Akula <email address hidden>
Date: Wed Feb 21 22:42:12 2018 -0800

1. Fixed Pool Member and Health Monitor Validation on LBaaS Wizard
2. Fixed Edit Pool Member Validation Weight option.
3. Fixed Health Monitor edit page and Wizard for HTTP Status code, it's free text and can accept list/range of HTTP status codes
4. Fixed if Pool Member added than IP- Address is required field.

Closes-Bug: #1750738

Change-Id: I2808c48918d6baf68984d2564c1b4370ef872c28

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/40053
Submitter: Suresh Akula (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/40053
Committed: http://github.com/Juniper/contrail-web-controller/commit/b17cf4f0b0ffd925a45abb528b21dad4555c610d
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit b17cf4f0b0ffd925a45abb528b21dad4555c610d
Author: Suresh Akula <email address hidden>
Date: Fri Feb 23 12:07:29 2018 -0800

non-editable load-balancer admin state changed to editable.
Fixed: added save admin-state to LB edit operation.

Change-Id: I18a0fc3148941f645b862f3b9276cc12a713b281
Closes-Bug: #1750738

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.