-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Update datadog-operator chart to v1.4.1 and datadog-crds chart to v1.3.0 #1284
Update datadog-operator chart to v1.4.1 and datadog-crds chart to v1.3.0 #1284
Conversation
08fbbaf
to
dd8de8b
Compare
digest: sha256:86af5339c70e484401b7d340b2380802c93f946ceb0441d7348f74811917be8b | ||
generated: "2024-01-04T12:16:01.40644-05:00" |
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.
This got updated because I updated and re-built the operator chart dependencies. Should I copy the sha256 and timestamp from the datadog-operator-v1.4.1.
tag? I'm concerned this change might trigger a conflicting tag or override the datadog-operator-v1.4.1
chart tag.
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.
In the add-on environment chart dependencies should be packaged together with the wrapper, they aren't pulled from the Helm repo. So this should be fine. As long as chart can be rendered and installed properly we should be good.
@@ -1,7 +1,7 @@ | |||
apiVersion: v2 | |||
name: datadog-operator | |||
version: 1.0.5 | |||
appVersion: 1.0.3 | |||
version: 1.4.1 |
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.
This needs to be update in the operator-eks-addon
chart and it's version bumped.
12fb2ae
to
7f26cb8
Compare
…gs, and regenerate wrapper baseline
ca5aa7b
to
dfad543
Compare
chart to v0.6.1
What this PR does / why we need it:
Update datadog-crds chart to v1.4.1 and datadog-operator chart to v1.3.0 for new operator-eks-addon wrapper chart release.
Which issue this PR fixes
(optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged)Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
.github/helm-docs.sh
)CHANGELOG.md
has been updatedREADME.md
make update-test-baselines
)