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

Add yaml-mapper tool for migrating from helm to operator #1567

Closed
wants to merge 4 commits into from

Conversation

swang392
Copy link
Contributor

@swang392 swang392 commented Oct 21, 2024

What this PR does / why we need it:

Add yaml-mapper tool to helm charts to help users switch from installing the Datadog Agent with helm charts to using Operator. The yaml mapper script will take a values.yaml file used for helm charts and convert it into one that is compatible with the operator. (part of agent innovation week)

Special notes for your reviewer:

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Chart Version bumped
  • Documentation has been updated with helm-docs (run: .github/helm-docs.sh)
  • CHANGELOG.md has been updated
  • Variables are documented in the README.md
  • For Datadog Operator chart or value changes update the test baselines (run: make update-test-baselines)

@swang392 swang392 added the enhancement New feature or request label Oct 21, 2024
tools/yaml-mapper/main.go Outdated Show resolved Hide resolved
tools/yaml-mapper/main.go Outdated Show resolved Hide resolved
tools/yaml-mapper/go.mod Outdated Show resolved Hide resolved
tools/yaml-mapper/go.mod Outdated Show resolved Hide resolved

toolchain go1.22.7

require helm.sh/helm/v3 v3.14.2

Choose a reason for hiding this comment

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

🟠 Library Vulnerability

helm.sh/helm/v3 → 3.14.2

Helm shows secrets in clear text (...read more)

An issue was discovered in Cloud Native Computing Foundation (CNCF) Helm. It displays values of secrets when the --dry-run flag is used. This is a security concern in some use cases, such as a --dry-run call by a CI/CD tool. NOTE: the vendor's position is that this behavior was introduced intentionally, and cannot be removed without breaking backwards compatibility (some users may be relying on these values).

View in Datadog  Leave us feedback  Documentation

github.com/xeipuuv/gojsonpointer v0.0.0-20190905194746-02993c407bfb // indirect
github.com/xeipuuv/gojsonreference v0.0.0-20180127040603-bd5ef7bd5415 // indirect
github.com/xeipuuv/gojsonschema v1.2.0 // indirect
golang.org/x/net v0.17.0 // indirect

Choose a reason for hiding this comment

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

🟠 Library Vulnerability

golang.org/x/net → 0.17.0

View all suggested fixes
Suggested change
golang.org/x/net v0.17.0 // indirect
golang.org/x/net vv0.30.1-0.20241011223655-5716b9813d2c// indirect
Suggested change
golang.org/x/net v0.17.0 // indirect
golang.org/x/net vv0.23.0// indirect
net/http, x/net/http2: close connections when receiving too many headers (...read more)

An attacker may cause an HTTP/2 endpoint to read arbitrary amounts of header data by sending an excessive number of CONTINUATION frames. Maintaining HPACK state requires parsing and processing all HEADERS and CONTINUATION frames on a connection. When a request's headers exceed MaxHeaderBytes, no memory is allocated to store the excess headers, but they are still parsed. This permits an attacker to cause an HTTP/2 endpoint to read arbitrary amounts of header data, all associated with a request which is going to be rejected. These headers can include Huffman-encoded data which is significantly more expensive for the receiver to decode than for an attacker to send. The fix sets a limit on the amount of excess header frames we will process before closing a connection.

View in Datadog  Leave us feedback  Documentation

@github-actions github-actions bot added the chart/datadog This issue or pull request is related to the datadog chart label Oct 21, 2024
@swang392 swang392 closed this Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart/datadog This issue or pull request is related to the datadog chart enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant