R4.1 Segment health check. Agent crash in HealthCheckInstanceService::DestroyInstanceTask

Bug #1732097 reported by Shashikiran H
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R4.1
Fix Committed
Critical
Ashok Singh
Trunk
Fix Committed
Critical
Ashok Singh

Bug Description

R4.1

Probable sequence of steps:
1. Change hc type from segment to local-link
2. Change it back to segment.
3. Change values of delay and timeout in segment

#0 0x00007fbd624b6c37 in raise () from /lib/x86_64-linux-gnu/libc.so.6
(gdb) bt
#0 0x00007fbd624b6c37 in raise () from /lib/x86_64-linux-gnu/libc.so.6
#1 0x00007fbd624ba028 in abort () from /lib/x86_64-linux-gnu/libc.so.6
#2 0x00007fbd624afbf6 in ?? () from /lib/x86_64-linux-gnu/libc.so.6
#3 0x00007fbd624afca2 in __assert_fail () from /lib/x86_64-linux-gnu/libc.so.6
#4 0x0000000000bc3225 in HealthCheckInstanceService::DestroyInstanceTask (this=0x7fbd4018de90) at controller/src/vnsw/agent/oper/health_check.cc:260
#5 0x0000000000bc252e in StopHealthCheckService (instance=0x7fbd4018de90, this=<optimized out>) at controller/src/vnsw/agent/oper/health_check.cc:434
#6 HealthCheckService::DeleteInstances (this=this@entry=0x7fbd48029700) at controller/src/vnsw/agent/oper/health_check.cc:629
#7 0x0000000000bc4b92 in HealthCheckService::Copy (this=this@entry=0x7fbd48029700, table=table@entry=0x7fbd54af0c60, data=0x7fbd4029a660)
    at controller/src/vnsw/agent/oper/health_check.cc:540
#8 0x0000000000bc4c83 in HealthCheckTable::OperDBOnChange (this=0x7fbd54af0c60, entry=<optimized out>, req=<optimized out>)
    at controller/src/vnsw/agent/oper/health_check.cc:683
#9 0x0000000000ba8d9a in AgentOperDBTable::OnChange (this=0x7fbd54af0c60, entry=0x7fbd48029768, req=0x7fbd40193540)
    at controller/src/vnsw/agent/oper/oper_db.h:202
#10 0x0000000001479e38 in DBTable::Input (this=0x7fbd54af0c60, tbl_partition=0x7fbd54afaf00, client=<optimized out>, req=0x7fbd40193540)
    at controller/src/db/db_table.cc:498
#11 0x000000000147993e in DBPartition::QueueRunner::Run (this=0x7fbd401e80d0) at controller/src/db/db_partition.cc:208
#12 0x00000000015cd527 in TaskImpl::execute (this=0x7fbd5baa9240) at controller/src/base/task.cc:277
#13 0x00007fbd63085b3a in ?? () from /usr/lib/libtbb.so.2
#14 0x00007fbd63081816 in ?? () from /usr/lib/libtbb.so.2
#15 0x00007fbd63080f4b in ?? () from /usr/lib/libtbb.so.2
#16 0x00007fbd6307d0ff in ?? () from /usr/lib/libtbb.so.2
#17 0x00007fbd6307d2f9 in ?? () from /usr/lib/libtbb.so.2
#18 0x00007fbd632a1184 in start_thread () from /lib/x86_64-linux-gnu/libpthread.so.0
#19 0x00007fbd6257a37d in clone () from /lib/x86_64-linux-gnu/libc.so.6

Shashikiran H (skiranh)
summary: R4.1 Segment health check. Agent crash in
+ HealthCheckInstanceService::DestroyInstanceTask
description: updated
tags: added: blocker
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R4.1

Review in progress for https://review.opencontrail.org/37530
Submitter: Hari Prasad Killi (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/37530
Committed: http://github.com/Juniper/contrail-controller/commit/fd739341408907c1c996f1c4c293642d93a87752
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit fd739341408907c1c996f1c4c293642d93a87752
Author: Hari Prasad Killi <email address hidden>
Date: Wed Nov 15 15:20:44 2017 +0530

Handle change of HC type

HC type was changed from segment to link-local. In this case, the earlier
segment instance would be deleted and a new one will be created for link local.
However the destroy was invoked after the object was modified and hence
assert of type was failing. Removing the assert as this is not required.

Change-Id: I47c40723961b540f9db4d4497731d21980af2b0f
closes-bug: #1732097

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

Review in progress for https://review.opencontrail.org/37777
Submitter: Hari Prasad Killi (<email address hidden>)

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

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

commit dc3fb428a4b1d122346823eea100c9cd79a350d4
Author: Hari Prasad Killi <email address hidden>
Date: Wed Nov 15 15:20:44 2017 +0530

Handle change of HC type

HC type was changed from segment to link-local. In this case, the earlier
segment instance would be deleted and a new one will be created for link local.
However the destroy was invoked after the object was modified and hence
assert of type was failing. Removing the assert as this is not required.

Change-Id: I47c40723961b540f9db4d4497731d21980af2b0f
closes-bug: #1732097
(cherry picked from commit fd739341408907c1c996f1c4c293642d93a87752)

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.