healthcheck: On success, probe is sent continuously rather than per interval seconds

Bug #1533604 reported by Senthilnathan Murugappan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
Fix Committed
High
Prabhjot Singh Sethi

Bug Description

Service health check sends continuous probe rather than per interval seconds if the probe was successful.

This can eat up the flow table and unnecessarily would hog the guest VM.

02:35:37.987681 IP (tos 0x0, ttl 63, id 58191, offset 0, flags [DF], proto TCP (6), length 52)
    109.97.15.2.33160 > 109.97.15.3.80: Flags [F.], cksum 0xf8ed (incorrect -> 0x8d60), seq 1, ack 546, win 237, options [nop,nop,TS val 34022340 ecr 854681], length 0
02:35:37.987727 IP (tos 0x0, ttl 64, id 56390, offset 0, flags [DF], proto TCP (6), length 52)
    109.97.15.3.80 > 109.97.15.2.33160: Flags [.], cksum 0xf8ed (incorrect -> 0x8c88), ack 2, win 453, options [nop,nop,TS val 854681 ecr 34022340], length 0
02:35:37.999504 IP (tos 0x0, ttl 62, id 13214, offset 0, flags [DF], proto TCP (6), length 60)
    109.97.15.2.33161 > 109.97.15.3.80: Flags [S], cksum 0xf8f5 (incorrect -> 0x114e), seq 333189208, win 29200, options [mss 1460,sackOK,TS val 34022342 ecr 0,nop,wscale 7], length 0

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

Review in progress for https://review.opencontrail.org/16267
Submitter: Prabhjot Singh Sethi (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/16267
Committed: http://github.org/Juniper/contrail-controller/commit/3f7c9d7f9c65479c257a5494097a641986aac861
Submitter: Zuul
Branch: master

commit 3f7c9d7f9c65479c257a5494097a641986aac861
Author: Prabhjot Singh Sethi <email address hidden>
Date: Wed Jan 13 22:52:04 2016 +0530

Fix script execution interval

Issue:
------
initial iteration flag was not reset, causing interval
timer never to come into picture and results in execution
of health check continously without waiting for interval

Fix:
----
reset initial flag after first execution and comment out
if condition to check change of status, to allow using
stdout for identifing pipe close and exit

Closes-Bug: 1533604
Related-Bug: 1530539
Change-Id: Id158b5dfb9a0ff7e2560b5a92f4114d38f839456

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.