Skip to content

[chore] Automate OBI sub-chart dependency update PRs#2301

Merged
MrAlias merged 14 commits intosignalfx:mainfrom
MrAlias:auto-up-obi
Apr 9, 2026
Merged

[chore] Automate OBI sub-chart dependency update PRs#2301
MrAlias merged 14 commits intosignalfx:mainfrom
MrAlias:auto-up-obi

Conversation

@MrAlias
Copy link
Copy Markdown
Contributor

@MrAlias MrAlias commented Mar 18, 2026

No description provided.

@MrAlias MrAlias marked this pull request as ready for review March 18, 2026 19:37
@MrAlias MrAlias requested review from a team as code owners March 18, 2026 19:37
Copilot AI review requested due to automatic review settings March 18, 2026 19:37
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR extends the existing chart-dependency auto-update automation to also handle the OpenTelemetry eBPF Instrumentation (OBI) subchart, ensuring related values.yaml defaults stay aligned when the subchart version is bumped.

Changes:

  • Add OBI-specific logic in the chart dependency update script to update .obi.image.tag when the OBI subchart version changes.
  • Extend the GitHub Actions workflow matrix to include an “obi” dependency update job.
  • Add a changelog entry documenting the automation enhancement.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
ci_scripts/update-chart-dependency.sh Adds update_obi_image_tag and wires it to run when opentelemetry-ebpf-instrumentation is updated.
.github/workflows/update_chart_dependencies.yaml Adds an OBI entry to the dependency-update matrix and updates workflow description comments.
.chloggen/autoupobi.yaml Documents the new automation behavior in the changelog system.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ci_scripts/update-chart-dependency.sh Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 18, 2026 19:43
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR extends the automated chart-dependency update workflow to also handle the OpenTelemetry eBPF Instrumentation (OBI) subchart, including keeping the parent chart’s default OBI image tag aligned with the updated subchart version.

Changes:

  • Add logic to update-chart-dependency.sh to update obi.image.tag in values.yaml when the opentelemetry-ebpf-instrumentation dependency is bumped.
  • Extend the GitHub Actions dependency-update workflow matrix to include the OBI dependency.
  • Add a changelog entry describing the automation enhancement.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
ci_scripts/update-chart-dependency.sh Adds OBI-specific post-bump logic to update values.yaml defaults.
.github/workflows/update_chart_dependencies.yaml Adds an obi matrix entry so the automation runs for the OBI dependency.
.chloggen/autoupobi.yaml Records the enhancement in the changelog system.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ci_scripts/update-chart-dependency.sh Outdated
Comment thread .chloggen/autoupobi.yaml Outdated
Copilot AI review requested due to automatic review settings March 20, 2026 14:27
@MrAlias MrAlias changed the title Automate OBI sub-chart dependency update PRs [chore] Automate OBI sub-chart dependency update PRs Mar 20, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR extends the existing “chart dependency update” automation to also cover the OpenTelemetry eBPF Instrumentation (OBI) subchart, including keeping the parent chart’s default OBI image tag in values.yaml aligned with the subchart version bump.

Changes:

  • Add update_obi_image_tag to update values.yaml’s obi.image.tag when opentelemetry-ebpf-instrumentation is bumped.
  • Extend the scheduled GitHub workflow matrix to open automated PRs for OBI dependency updates.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
ci_scripts/update-chart-dependency.sh Adds OBI-specific post-bump logic to keep obi.image.tag aligned with the new dependency version while preserving values.yaml formatting.
.github/workflows/update_chart_dependencies.yaml Adds an obi matrix entry to automate dependency update PRs for the OBI subchart.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ci_scripts/update-chart-dependency.sh Outdated
Comment thread .github/workflows/update_chart_dependencies.yaml
Copilot AI review requested due to automatic review settings April 6, 2026 19:27
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ci_scripts/update-chart-dependency.sh Outdated
Update awk script syntax.
Copilot AI review requested due to automatic review settings April 8, 2026 21:51
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ci_scripts/update-chart-dependency.sh Outdated
Comment thread ci_scripts/update-chart-dependency.sh Outdated
Comment thread ci_scripts/update-chart-dependency.sh
@MrAlias MrAlias merged commit c8cf720 into signalfx:main Apr 9, 2026
107 checks passed
@MrAlias MrAlias deleted the auto-up-obi branch April 9, 2026 21:18
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 9, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants