R4.1-144: Sanity: Fix contrail-status display of kakfa service: when confluent-kafka is deployed by systemd

Bug #1779060 reported by Ankit Jain
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
New
Undecided
Unassigned
R4.1
Fix Committed
High
Ankit Jain

Bug Description

Kafka need to renamed as per the following mail from Jack:

An issue with contrail-status display script is the reason for the behavior you have observed.

Please see the note below:

I have logged into ssh root@10.204.216.105 (nodem16) and validated a few things.

· Kafka is installed and functional. You can validate this by invoking the following
o apt list --installed | grep kafka
· Kafka installation is working as expected; a couple of ways to validated that are listed below:
o Check contrail-status on analytics node (collector and alarmgen show active status)
· Furthermore, you can manually check that Kafak is functional as follows:
o List kafka topics: /usr/bin/kafka-topics --list --zookeeper localhost:2182
o Create a kafka topic: /usr/bin/kafka-topics --create --zookeeper localhost:2182 --replication-factor 1 --partitions 1 --topic test123
o Write to the kafka topic: /usr/bin/kafka-console-producer --broker-list localhost:9092 --topic test123
o Read from the kafka topic: /usr/bin/kafka-console-consumer --bootstrap-server localhost:9092 --topic test123 --from-beginning
· You can start and stop kafka service by invoking the following
o service confluent-kafka start (it is no longer service kafka stop)
o service confluent-kafka stop
· However, contrail-status command doesn’t work (this is the behavior you observed)
o I manually edited the file (on the analyticsdb docker) as follows:
§ Replace kafka with confluent-kafka in /usr/bin/contrail-status.
§ things start to look good
§ (this behavior is specific to systemd driven services)
· I posted the following review for R4.1 and this should fix the issue.
o https://review.opencontrail.org/#/c/44185/

Thanks,
Jack

Tags: sanity
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R4.1

Review in progress for https://review.opencontrail.org/44187
Submitter: Ankit Jain (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/44187
Committed: http://github.com/Juniper/contrail-test-ci/commit/16b28e1fcbf9f95e8e65872588bf0a46b9c7a045
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 16b28e1fcbf9f95e8e65872588bf0a46b9c7a045
Author: ankitja <email address hidden>
Date: Thu Jun 28 11:16:33 2018 +0530

Renaming kafka service to confluent-kafka

Change-Id: I0bfdb598fc844dca02520eeb8039c48db0169212
Closes-bug: #1779060

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

Review in progress for https://review.opencontrail.org/44229
Submitter: Jack Jonnalagadda (<email address hidden>)

summary: - R4.1-144: Sanity: Renaming kafka to confluent-kafka in test scripts
+ R4.1-144: Sanity: Fix contrail-status status display of kakfa: when
+ confluent-kafka is deployed by systemd
summary: - R4.1-144: Sanity: Fix contrail-status status display of kakfa: when
+ R4.1-144: Sanity: Fix contrail-status display of kakfa service: when
confluent-kafka is deployed by systemd
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/44228
Submitter: Jack Jonnalagadda (<email address hidden>)

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

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

commit 93ca28beed2d81b1caf5bce635a5ce5ec35e000b
Author: Jackjvs <email address hidden>
Date: Thu Jun 28 19:32:51 2018 -0700

Upgrade kafka to confluent-kafka-2.11-1.1.1-1
Closes-Bug: #1779060

Fix the contrail-status display for systemd deployments of
confluenet-kafka

The validated output format:
confluent-kafka-2.11/now 1.1.1-1 all [installed,local]
root@nodem16(analyticsdb):/# contrail-status
== Contrail Database ==
contrail-database: active

kafka: active
contrail-database-nodemgr: active
========Run time service failures=============
/var/crashes/core.auplink.28219.nodem16.1530177245
root@nodem16(analyticsdb):/#

Change-Id: Idfc6c0429d4a898260f0304c13572b73d7208f15

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

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

commit ddcb2f138fe18fddc93d111911b8c01930ece065
Author: Jack Jonnalagadda <email address hidden>
Date: Thu Jun 28 18:37:31 2018 -0800

Revert "Renaming kafka service to confluent-kafka"
Closes-bug: #1779060

This reverts commit 16b28e1fcbf9f95e8e65872588bf0a46b9c7a045.

Change-Id: Ib3584fe344ebad3ff8aa207547cb443ee3fc780d

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.