-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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: Adding Apache Kafka Kubernetes Add-on #1090
Conversation
…riment from project]
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.
LGTM! I have fixed few formatting issues and added doc
Let's wait for one more approval @bryantbiggs , @askulkarni2
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.
Looks good - let's only add values that are non-default to make version changes easier for users
We should also get this added to the GitOps addon repo
@AlyIbrahim Would you be able to raise another PR for ArgoCD deployment? Here is the repo location. Please reach out if you need help. |
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.
LGTM!
@vara-bonthu Yep, will check that |
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.
LGTM
Co-authored-by: Vara Bonthu <vara.bonthu@gmail.com>
Co-authored-by: Vara Bonthu <vara.bonthu@gmail.com>
Co-authored-by: Vara Bonthu <vara.bonthu@gmail.com>
What does this PR do?
🛑 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted.
Consult the CONTRIBUTING guide for submitting pull-requests.
Add Strmizi Operator as a Kubernetes Add-on to be used by DoEKS for Apache Kafka on EKS
Motivation
More
pre-commit run -a
with this PRNote: Not all the PRs require a new example and/or doc page. In general:
docs/add-ons/*
is required for new a new addonFor Moderators
Additional Notes