Skip to content

Comments

security(deps): bump helm and opentelemetry to fix HIGH vulnerabilities#1587

Merged
cicoyle merged 5 commits intodapr:masterfrom
fyzanshaik:security/fix-helm-otel-vulnerabilities
Feb 10, 2026
Merged

security(deps): bump helm and opentelemetry to fix HIGH vulnerabilities#1587
cicoyle merged 5 commits intodapr:masterfrom
fyzanshaik:security/fix-helm-otel-vulnerabilities

Conversation

@fyzanshaik
Copy link
Contributor

@fyzanshaik fyzanshaik commented Feb 9, 2026

Bump helm.sh/helm/v3 from v3.17.4 to v3.18.5 to fix 5 HIGH severity vulnerabilities:

  • SNYK-GOLANG-HELMSHHELMV3PKGREPO-11799541
  • SNYK-GOLANG-HELMSHHELMV3PKGLINTRULES-11799538
  • SNYK-GOLANG-HELMSHHELMV3PKGDOWNLOADER-10734107
  • SNYK-GOLANG-HELMSHHELMV3PKGCHARTUTIL-11799535
  • SNYK-GOLANG-HELMSHHELMV3PKGCHARTUTIL-11799536

Bump go.opentelemetry.io/otel/sdk from v1.39.0 to v1.40.0 to fix:

  • SNYK-GOLANG-GOOPENTELEMETRYIOOTELSDKRESOURCE-15182758

In addition to the dependency updates, this PR hardens two flaky E2E test paths observed in CI:

  • tests/e2e/standalone/scheduler_test.go (TestSchedulerDeleteAll): replaced brittle fixed-count assertions with eventual, filter-based behavior checks.
  • tests/e2e/common/common.go (StatusTestOnInstallUpgrade): made dapr status -k parsing resilient to multi-word status values and increased retry window for upgrade convergence.

These updates address remaining HIGH severity vulnerabilities not covered by PR #1586, reducing total HIGH vulnerabilities from 6 to 0, while improving E2E stability for this PR path.

Validation performed:

  • go test -c ./tests/e2e/common
  • go test -c -tags e2e ./tests/e2e/standalone

Please reference the issue this PR will close: #[issue number]

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

Bump helm.sh/helm/v3 from v3.17.4 to v3.18.5 to fix 5 HIGH severity vulnerabilities:
- SNYK-GOLANG-HELMSHHELMV3PKGREPO-11799541
- SNYK-GOLANG-HELMSHHELMV3PKGLINTRULES-11799538
- SNYK-GOLANG-HELMSHHELMV3PKGDOWNLOADER-10734107
- SNYK-GOLANG-HELMSHHELMV3PKGCHARTUTIL-11799535
- SNYK-GOLANG-HELMSHHELMV3PKGCHARTUTIL-11799536

Bump go.opentelemetry.io/otel/sdk from v1.39.0 to v1.40.0 to fix:
- SNYK-GOLANG-GOOPENTELEMETRYIOOTELSDKRESOURCE-15182758

These updates address remaining HIGH severity vulnerabilities not covered
by PR dapr#1586, reducing total HIGH vulnerabilities from 6 to 0.

Signed-off-by: fyzanshaik <fyzan.shaik@gmail.com>
@fyzanshaik fyzanshaik requested review from a team as code owners February 9, 2026 20:22
@fyzanshaik fyzanshaik mentioned this pull request Feb 9, 2026
3 tasks
@fyzanshaik
Copy link
Contributor Author

looking into this

Signed-off-by: fyzanshaik <fyzan.shaik@gmail.com>
@fyzanshaik fyzanshaik force-pushed the security/fix-helm-otel-vulnerabilities branch from 9daca3b to 039b5ac Compare February 9, 2026 21:53
@fyzanshaik
Copy link
Contributor Author

@JoshVanL please approve CI

JoshVanL
JoshVanL previously approved these changes Feb 10, 2026
Signed-off-by: fyzanshaik <fyzan.shaik@gmail.com>
@fyzanshaik
Copy link
Contributor Author

fyzanshaik commented Feb 10, 2026

@JoshVanL One note: the Linux self-hosted E2E failure was a flaky port collision (GRPCPort 3510 not available), not caused by the Helm/Otel bumps. I pushed f77deb1 to use a dynamic gRPC port in the scheduler test path (test-scheduler.yaml + tests/apps/scheduler/app.go). Could you re-run checks? Just this one test failing..

This reverts commit f77deb1.

Signed-off-by: fyzanshaik <fyzan.shaik@gmail.com>
@fyzanshaik
Copy link
Contributor Author

I’m not sure why this specific test is still failing, but the rest of the suite is passing consistently on this PR. I’ve reverted the extra test-only mitigation so this PR stays focused on the dependency security updates, and I’d appreeciate any help that can fix this? I am fine with anyone taking over too @JoshVanL 🙏

@JoshVanL
Copy link
Contributor

Thanks @fyzanshaik - the tests seem quite unhappy with the scheduler test changes

Signed-off-by: joshvanl <me@joshvanl.dev>
@cicoyle cicoyle merged commit 9ee60e3 into dapr:master Feb 10, 2026
27 checks passed
JoshVanL pushed a commit to JoshVanL/dapr-cli that referenced this pull request Feb 10, 2026
…ulnerabilities

security(deps): bump helm and opentelemetry to fix HIGH vulnerabilities
JoshVanL pushed a commit to JoshVanL/dapr-cli that referenced this pull request Feb 10, 2026
…ulnerabilities

security(deps): bump helm and opentelemetry to fix HIGH vulnerabilities

Signed-off-by: joshvanl <me@joshvanl.dev>
JoshVanL added a commit that referenced this pull request Feb 10, 2026
…rabilities (#1588)

security(deps): bump helm and opentelemetry to fix HIGH vulnerabilities

Signed-off-by: joshvanl <me@joshvanl.dev>
Co-authored-by: Cassie Coyle <cassie@diagrid.io>
@nelson-parente nelson-parente added this to the v1.17 milestone Feb 12, 2026
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.

4 participants