Skip to content
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

Fix CLC config #64

Merged
merged 3 commits into from
Apr 6, 2020
Merged

Fix CLC config #64

merged 3 commits into from
Apr 6, 2020

Conversation

ahmed-mez
Copy link
Contributor

What does this PR do?

  • Get rid of clusterChecksRunnerEnabled
  • Enable endpoint checks - be aligned with the helm chart

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

@ahmed-mez ahmed-mez requested a review from a team as a code owner April 3, 2020 09:50
@ahmed-mez ahmed-mez force-pushed the ahmed-mez/enable-endpointchecks branch from 53b28e1 to 25315f4 Compare April 3, 2020 10:19
@ahmed-mez ahmed-mez force-pushed the ahmed-mez/enable-endpointchecks branch 2 times, most recently from 80ab2c2 to 5469f62 Compare April 3, 2020 10:56
…datadog-operator into ahmed-mez/enable-endpointchecks
@ahmed-mez ahmed-mez force-pushed the ahmed-mez/enable-endpointchecks branch from 5469f62 to f32d7c4 Compare April 3, 2020 11:11
@codecov-io
Copy link

codecov-io commented Apr 3, 2020

Codecov Report

Merging #64 into master will increase coverage by 0.21%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #64      +/-   ##
==========================================
+ Coverage   57.00%   57.21%   +0.21%     
==========================================
  Files          31       31              
  Lines        4112     4121       +9     
==========================================
+ Hits         2344     2358      +14     
+ Misses       1585     1581       -4     
+ Partials      183      182       -1     
Flag Coverage Δ
#unittests 57.21% <100.00%> (+0.21%) ⬆️
Impacted Files Coverage Δ
pkg/controller/datadogagent/clusteragent.go 73.84% <100.00%> (ø)
pkg/controller/datadogagent/clusterchecksrunner.go 70.65% <100.00%> (+0.32%) ⬆️
pkg/controller/datadogagent/utils.go 83.00% <100.00%> (+0.66%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 44a2148...f32d7c4. Read the comment docs.

@ahmed-mez ahmed-mez added component/controller enhancement New feature or request labels Apr 3, 2020
@@ -1662,6 +1665,7 @@ func TestReconcileDatadogAgent_Reconcile(t *testing.T) {
Labels: map[string]string{"label-foo-key": "label-bar-value"},
Status: &datadoghqv1alpha1.DatadogAgentStatus{},
ClusterAgentEnabled: true,
ClusterChecksEnabled: true,
ClusterChecksRunnerEnabled: true,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we need te remove this field.

@ahmed-mez ahmed-mez merged commit d5889a1 into master Apr 6, 2020
@ahmed-mez ahmed-mez deleted the ahmed-mez/enable-endpointchecks branch April 6, 2020 16:53
@ahmed-mez ahmed-mez mentioned this pull request Apr 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants