-
Notifications
You must be signed in to change notification settings - Fork 98
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
Add an all-in-one chart that deploys both the operator and an agent #91
Conversation
Codecov Report
@@ Coverage Diff @@
## master #91 +/- ##
=======================================
Coverage 57.19% 57.19%
=======================================
Files 31 31
Lines 4175 4175
=======================================
Hits 2388 2388
Misses 1606 1606
Partials 181 181
Continue to review full report at Codecov.
|
docs/getting_started.md
Outdated
@@ -24,140 +28,36 @@ To use the Datadog Operator, deploy it in your Kubernetes cluster. Then create a | |||
4. Create the namespace: | |||
|
|||
```shell | |||
kubectl create ns $DD_NAMESPACE | |||
kubectl create namespace $DD_NAMESPACE |
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.
@L3n41c could we remove step 3 and 4 and use the default namespace and a default deployment name? it would simplify the basic deployment even further
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.
Hi @cohenyair !
Thanks you for the feedback.
I’ve simplified the instructions by letting the agent being installed in the current namespace.
Can you PTAL ?
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.
That's great! LGTM
057e93a
to
ee28ee9
Compare
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.
small feedback
…aic/chart_all-in-one
ecab5d8
to
ef836e0
Compare
What does this PR do?
Add an all-in-one chart that deploys both the operator and an agent
Motivation
Simplify on-boarding
Additional Notes
Anything else we should know when reviewing?