-
Notifications
You must be signed in to change notification settings - Fork 13
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 extendeddaemonset-check helper #53
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.
This pull request does not contain a valid label. Please add one of the following labels: bug, enhancement, documentation
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.
This pull request contains a valid label.
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.
Just left a few cosmetics comments. Mostly about dead or useless things.
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.
This pull request contains a valid label.
* add new cmd binary: `check-eds` * add new `app-example` chart * update ci to deliver the `datadog/extendeddaemonset-check` container image
811b929
to
c74f099
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.
This pull request contains a valid label.
Codecov Report
@@ Coverage Diff @@
## master #53 +/- ##
==========================================
- Coverage 37.46% 36.73% -0.74%
==========================================
Files 29 29
Lines 1500 1530 +30
==========================================
Hits 562 562
- Misses 865 895 +30
Partials 73 73
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
What does this PR do?
Add extendeddaemonset-check helper
check-eds
which can check when a EDS update is finished.app-example
chart: example of an application using theExtendedDaemonset
.datadog/extendeddaemonset-check
container image.Motivation
Helm3 is not able to know when the deployment of an application deployed with an
ExtendedDaemonset
is done.The new
datadog/extendeddaemonset-check
can be used as helm3 test.Additional Notes
N/A
Describe your test plan
To test it, please follow the instruction in the
charts/app-example/README.md