Securing communication between Cruise Control and Kafka #37
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
Moved metrics topic to Cruise Control configuration Hiding truststore and keystore passwords in the log Signed-off-by: Paolo Patierno <ppatierno@live.com>
kyguy
changed the title
ppatierno
approved these changes
Apr 22, 2020
kyguy
added a commit
that referenced
this pull request
Apr 22, 2020
* Adding TLS communication between CC and Kafka Signed-off-by: Kyle Liberti <kliberti@redhat.com> * Adding TLS communication between metric reporter and Kafka Signed-off-by: Kyle Liberti <kliberti@redhat.com> * Fixed metrics report to use TLS hostname verification (#38) Moved metrics topic to Cruise Control configuration Hiding truststore and keystore passwords in the log Signed-off-by: Paolo Patierno <ppatierno@live.com> Co-authored-by: Paolo Patierno <ppatierno@live.com> Signed-off-by: Kyle Liberti <kliberti@redhat.com>
ppatierno
added a commit
that referenced
this pull request
Apr 26, 2020
* Added Cruise Control System Tests
- Added deployment with CC to Kafka ST resources
- Added pod name and deployment resources
- Added CC deployment system test
Signed-off-by: Thomas Cooper <tcooper@redhat.com>
* Cruise Control Deployment
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Add broker capacity estimation and configuration
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Add k8s memory parsing to return different byte multiples
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Refactor Capacity constructor
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Regenerate helm charts
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Addressing some comments
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Addressing more comments
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Addressing more comments (refactoring capacity API, checkOwnerReferences)
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Addressing comments ( capacity properties as strings/validate units in schema)
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Tightening capacity regex; fixing docs
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Another doc fix
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Add missing goal; bump CC version to 2.0.100
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Fix logging and typos
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Addressing comments ( Update allowed disk capacity notation + refactoring)
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Fixing more typos
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Fixing another doc issue
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Securing communication between Cruise Control and Kafka (#37)
* Adding TLS communication between CC and Kafka
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Adding TLS communication between metric reporter and Kafka
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Fixed metrics report to use TLS hostname verification (#38)
Moved metrics topic to Cruise Control configuration
Hiding truststore and keystore passwords in the log
Signed-off-by: Paolo Patierno <ppatierno@live.com>
Co-authored-by: Paolo Patierno <ppatierno@live.com>
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Removing CC replicas, refactor Dockerfile + CC dependencies
Signed-off-by: Kyle Liberti <kliberti@redhat.com>
* Added security context, network policy and tests (#39)
* Added security context and tests
Signed-off-by: Paolo Patierno <ppatierno@live.com>
* Added network policy for the cruise control REST API port
Signed-off-by: Paolo Patierno <ppatierno@live.com>
* Added CruiseControl ST to regression tests
Signed-off-by: Paolo Patierno <ppatierno@live.com>
* Fixed NPE on Cruise Control network policy
Signed-off-by: Paolo Patierno <ppatierno@live.com>
Co-authored-by: Thomas Cooper <tcooper@redhat.com>
Co-authored-by: Paolo Patierno <ppatierno@live.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Type of change
Description
Secures the communication between Cruise Control and Kafka.
Checklist
Please go through this checklist and make sure all applicable tasks have been done
./design