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

Pull v0.7 to main #373

Merged
merged 25 commits into from
Sep 14, 2021
Merged

Pull v0.7 to main #373

merged 25 commits into from
Sep 14, 2021

Conversation

clamoriniere
Copy link
Collaborator

What does this PR do?

A brief description of the change being made with this pull request.

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

Describe your test plan

Write there any instructions and details you may have to test your PR.

@clamoriniere clamoriniere requested review from a team as code owners September 13, 2021 21:17
Copy link

@github-actions github-actions bot left a 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

@codecov-commenter
Copy link

Codecov Report

Merging #373 (1848dd9) into main (2031a6e) will increase coverage by 25.94%.
The diff coverage is 73.52%.

Impacted file tree graph

@@             Coverage Diff             @@
##             main     #373       +/-   ##
===========================================
+ Coverage   39.15%   65.10%   +25.94%     
===========================================
  Files          65       63        -2     
  Lines       11465     6926     -4539     
===========================================
+ Hits         4489     4509       +20     
+ Misses       6657     2098     -4559     
  Partials      319      319               
Flag Coverage Δ
unittests 65.10% <73.52%> (+25.94%) ⬆️

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

Impacted Files Coverage Δ
api/v1alpha1/datadogagent_types.go 100.00% <ø> (ø)
controllers/datadogagent_controller.go 60.60% <ø> (ø)
pkg/defaulting/images.go 100.00% <ø> (ø)
api/v1alpha1/datadogagent_default.go 80.56% <18.18%> (-0.65%) ⬇️
controllers/datadogagent/orchestrator.go 78.15% <100.00%> (+3.15%) ⬆️
controllers/datadogagent/utils.go 84.30% <100.00%> (+0.03%) ⬆️
api/v1alpha1/zz_generated.deepcopy.go

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 2031a6e...1848dd9. Read the comment docs.

Copy link
Contributor

@kayayarai kayayarai left a comment

Choose a reason for hiding this comment

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

LGTM from Documentation perspective

@@ -410,7 +410,7 @@ type LogCollectionConfig struct {
// Defaults to `/var/log/containers`
//
// +optional
ContainerSymlinksPath *string `json:"ContainerSymlinksPath,omitempty"`
ContainerSymlinksPath *string `json:"containerSymlinksPath,omitempty"`
Copy link
Contributor

Choose a reason for hiding this comment

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

I will rebase this one

@@ -722,6 +722,9 @@ type ConfigDirSpec struct {
// ConfigMapName name of a ConfigMap used to mount a directory.
ConfigMapName string `json:"configMapName,omitempty"`
// items mapping between configMap data key and file path mount.
// +optional
// +listType=map
// +listMapKey=key
Items []corev1.KeyToPath `json:"items,omitempty"`
Copy link
Contributor

Choose a reason for hiding this comment

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

and this one

@clamoriniere clamoriniere merged commit a266e68 into main Sep 14, 2021
@clamoriniere clamoriniere deleted the clamoriniere/pull-v0.7-main branch September 14, 2021 07:30
@khewonc khewonc mentioned this pull request May 17, 2022
mftoure pushed a commit that referenced this pull request Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants