datadog-operator icon indicating copy to clipboard operation
datadog-operator copied to clipboard

[override] Implement component overrides

Open davidor opened this issue 1 year ago • 1 comments

What does this PR do?

This implements the overrides defined in a v2alpha1.DatadogAgentComponentOverride object.

The overrides are mostly about the pod template specs, but they also affects deployments, and the required components.

The overrides at container level were already implemented in a previous PR: https://github.com/DataDog/datadog-operator/pull/584

Describe your test plan

Define a datadog agent object with overrides and check that everything works as expected.

davidor avatar Aug 09 '22 11:08 davidor

Codecov Report

Merging #592 (bdc93c0) into main (235ea45) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #592   +/-   ##
=======================================
  Coverage   58.51%   58.51%           
=======================================
  Files           3        3           
  Lines         135      135           
=======================================
  Hits           79       79           
  Misses         43       43           
  Partials       13       13           
Flag Coverage Δ
unittests 58.51% <ø> (ø)

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 235ea45...bdc93c0. Read the comment docs.

codecov-commenter avatar Aug 09 '22 11:08 codecov-commenter

Rebased on top of main to fix conflicts.

davidor avatar Aug 30 '22 16:08 davidor

As discussed offline, we'll fix the issues with the config file overrides in a future PR.

davidor avatar Aug 30 '22 16:08 davidor