search

confluentinc / cp-demo

Confluent Platform Demo including Apache Kafka, ksqlDB, Control Center, Schema Registry, Security, Schema Linking, and Cluster Linking
Apache License 2.0
38 stars 322 forks source link

DEVX-2671: complete PR #379 #388

Closed ybyzek closed 3 years ago

ybyzek commented 3 years ago

Description

https://confluentinc.atlassian.net/browse/DEVX-2671

What behavior does this PR change, and why?

This PR ties up one loose end from https://github.com/confluentinc/cp-demo/pull/379 , which is to make mods to the docker-compose.yml file used by SBC.

First I validated that the SBC script breaks in 6.2.0-post, then validated that the PR fixes the issue

[17:17:06] ~/git/cp-demo(DEVX-2671) ✗: ./scripts/sbc/add-broker.sh 
Signature ok
subject=C = US, ST = Ca, L = PaloAlto, O = CONFLUENT, OU = TEST, CN = kafka3
Getting CA Private Key
Certificate was added to keystore
Certificate reply was installed in keystore
Certificate was added to keystore
Certificate stored in file <kafka3.der>
Importing keystore kafka.kafka3.keystore.jks to kafka3.keystore.p12...
Entry for alias kafka3 successfully imported.
Entry for alias snakeoil-caroot successfully imported.
Import command completed:  2 entries successfully imported, 0 entries failed or cancelled
openldap is up-to-date
zookeeper is up-to-date
Creating kafka3 ... done
Waiting up to 120 seconds for SBC add broker to start rebalance planning
...kafka1            | [2021-09-15 21:17:36,809] INFO Addition status for broker 3 changed to PLAN_COMPUTATION (io.confluent.databalancer.ConfluentDataBalanceEngine)
✔

Author Validation

Describe the validation already done, or needs to be done, by the PR submitter.

Reviewer Tasks

Describe the tasks/validation that the PR submitter is requesting to be done by the reviewer.