Added security context, network policy and tests #39
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: Paolo Patierno <ppatierno@live.com>
Signed-off-by: Paolo Patierno <ppatierno@live.com>
ppatierno
changed the title
kyguy
approved these changes
Apr 23, 2020
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.
Signed-off-by: Paolo Patierno ppatierno@live.com
Type of change
Description
Fixes some Jakub's comments like security context and network policy.
Checklist
Please go through this checklist and make sure all applicable tasks have been done
./design