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

feat: support admissionController.AgentCommunicationMode #460

Merged
merged 3 commits into from Apr 5, 2022

Conversation

ahmed-mez
Copy link
Contributor

@ahmed-mez ahmed-mez commented Mar 22, 2022

What does this PR do?

  • Add admissionController.AgentCommunicationMode to configure the admission controller injection mode
  • Auto-configure DD_ADMISSION_CONTROLLER_INJECT_CONFIG_LOCAL_SERVICE_NAME

Motivation

DataDog/datadog-agent#11373

Additional Notes

Related to DataDog/helm-charts#575

Describe your test plan

Same as DataDog/datadog-agent#11373 but with the operator + make sure the service mode works as expected.

@ahmed-mez ahmed-mez added this to the v0.8.0 milestone Mar 22, 2022
@ahmed-mez ahmed-mez requested review from a team as code owners March 22, 2022 14:27
@ahmed-mez ahmed-mez changed the title feat: support admissionController.configInjectionMode feat: support admissionController.AgentCommunicationMode Mar 22, 2022
@codecov-commenter
Copy link

codecov-commenter commented Mar 22, 2022

Codecov Report

Merging #460 (7087581) into main (22cf913) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #460   +/-   ##
=======================================
  Coverage   60.60%   60.60%           
=======================================
  Files           3        3           
  Lines         132      132           
=======================================
  Hits           80       80           
  Misses         40       40           
  Partials       12       12           
Flag Coverage Δ
unittests 60.60% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


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 22cf913...7087581. Read the comment docs.

config/crd/bases/v1beta1/datadoghq.com_datadogagents.yaml Outdated Show resolved Hide resolved
docs/configuration.md Outdated Show resolved Hide resolved
@ahmed-mez ahmed-mez requested a review from alai97 March 23, 2022 09:21
Copy link
Contributor

@alai97 alai97 left a comment

Choose a reason for hiding this comment

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

Looks good for docs!

@ahmed-mez ahmed-mez merged commit b1dc945 into main Apr 5, 2022
@ahmed-mez ahmed-mez deleted the ahmed/adm-injection-mode branch April 5, 2022 12:35
@khewonc khewonc mentioned this pull request May 17, 2022
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