-
Notifications
You must be signed in to change notification settings - Fork 327
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(kuma-cp) Fault Injection entity #643
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also add the test case to k8s global_type_registry_test.go
@@ -93,7 +93,7 @@ var _ = Describe("kumactl install control-plane", func() { | |||
// then | |||
Expect(actual).To(MatchYAML(expected)) | |||
// and | |||
actualManifests := data.SplitYAML(data.File{Data: expected}) | |||
actualManifests := data.SplitYAML(data.File{Data: actual}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! I was having problems with this last time I edited the install script and was wondering what is going on.
don't forget about Kuma GUI but this comes with a separate PR. You can ask @bloqhead for help if needed. |
Please remember to also write documentation for this on https://github.com/Kong/kuma-website, and update the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One more thing to do in validators and we are good to go
Summary
Add
FaultInjection
entityFull changelog
*.proto
kumactl