Skip to content

feat: update Dragonfly Helm chart to v1.3.37 with OTLP HTTP/GRPC support#391

Merged
Liam-Zhao merged 1 commit into
mainfrom
feature/tracing-http
Jun 19, 2025
Merged

feat: update Dragonfly Helm chart to v1.3.37 with OTLP HTTP/GRPC support#391
Liam-Zhao merged 1 commit into
mainfrom
feature/tracing-http

Conversation

@gaius-qi

Copy link
Copy Markdown
Member

Description

This pull request updates the Dragonfly Helm chart to include new versions of its components and introduces support for OpenTelemetry (OTel) tracing protocols. The most significant changes involve version bumps for various components and the addition of tracing configurations.

Version Updates:

  • Updated version to 1.3.37 and appVersion to 2.2.4-rc.0 in charts/dragonfly/Chart.yaml.
  • Updated image tags for manager, scheduler, client, seed-client, and dfinit components to align with the new release versions in charts/dragonfly/Chart.yaml, charts/dragonfly/README.md, and charts/dragonfly/values.yaml. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]

OpenTelemetry (OTel) Support:

  • Added support for HTTP and gRPC protocols for OpenTelemetry tracing in charts/dragonfly/Chart.yaml.
  • Introduced detailed tracing configuration options for manager, scheduler, seedClient, and client components in charts/dragonfly/values.yaml. This includes specifying protocols (http, https, grpc), endpoints, and headers for tracing logs. [1] [2] [3] [4]

Cleanup:

  • Removed unused tracing configuration fields (addr and headers) from charts/dragonfly/README.md for the manager and scheduler components. [1] [2]

Related Issue

Motivation and Context

@gaius-qi gaius-qi self-assigned this Jun 19, 2025
@gaius-qi gaius-qi added the enhancement New feature or request label Jun 19, 2025

@Liam-Zhao Liam-Zhao left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@Liam-Zhao Liam-Zhao enabled auto-merge (squash) June 19, 2025 12:54

@chlins chlins left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@Liam-Zhao Liam-Zhao merged commit 6fc6c79 into main Jun 19, 2025
11 of 13 checks passed
@Liam-Zhao Liam-Zhao deleted the feature/tracing-http branch June 19, 2025 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants