-
Notifications
You must be signed in to change notification settings - Fork 440
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
Tolerations support in otelcol CRD #302
Tolerations support in otelcol CRD #302
Conversation
…l collectors on tainted nodes for both deployments and daemonsets. Signed-off-by: Vineeth Pothulapati <vineethpothulapati@outlook.com>
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, just need some input validation and a more complete godoc for the new field.
Tests are missing as well. |
… that are relevant only for specifc mode of deployments
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, but there's an import order issue. Because of that, I requested two other minor changes. Other than that, looks good to me!
pkg/collector/deployment_test.go
Outdated
@@ -17,6 +17,8 @@ package collector_test | |||
import ( | |||
"testing" | |||
|
|||
v1 "k8s.io/api/core/v1" |
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.
The import order is odd here.
Could you please check the test failure? |
Head branch was pushed to by a user without write access
@jpkrohling it looks like mergify task post all checks is failing. :) |
Signed-off-by: Vineeth Pothulapati <vineethpothulapati@outlook.com>
Signed-off-by: Vineeth Pothulapati <vineethpothulapati@outlook.com>
add support for tolerations in otelcol CRD, This helps in running Otel collectors on tainted nodes for both deployments and daemon sets.
fixes: #186