Skip to content

ExT tool update - split binary so that pod requests less permissions#304

Merged
denis-zaitsev merged 1 commit into
mainfrom
ext-split-binaries-release
Jul 9, 2026
Merged

ExT tool update - split binary so that pod requests less permissions#304
denis-zaitsev merged 1 commit into
mainfrom
ext-split-binaries-release

Conversation

@denis-zaitsev

@denis-zaitsev denis-zaitsev commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Just bumping ext version.
Only architectural changes there - we split binaries into tracing and cli to request less permissions for k8s pods.

Summary by CodeRabbit

  • Chores
    • Updated the bundled Astera extension to a newer version, which may include fixes and improvements in the built image.

@denis-zaitsev
denis-zaitsev requested review from Abdelsalam-Abbas and mag-astera and removed request for Abdelsalam-Abbas and mag-astera July 9, 2026 17:18
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Updates the default EXT_VERSION value from v0.1.4 to v0.1.8 in both Dockerfile.astera's build argument and docker/astera/install-ext.sh's script default, affecting which Astera extension version is installed when not explicitly overridden.

Changes

EXT_VERSION Version Bump

Layer / File(s) Summary
Default extension version update
Dockerfile.astera, docker/astera/install-ext.sh
EXT_VERSION default changed from v0.1.4 to v0.1.8 in both the Dockerfile build argument and the install script fallback value.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • diff-use/sampleworks#265: Introduces version-controlled EXT_VERSION behavior in install-ext.sh/Dockerfile.astera, directly related to this version bump.
  • diff-use/sampleworks#282: Updates the same EXT_VERSION default in the same files with a different version bump.

Suggested reviewers: marcuscollins, mag-astera

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title is related to the Ext version bump and the release’s split-binary permission reduction theme.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ext-split-binaries-release

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mag-astera mag-astera changed the title Bump Ext version ExT tool update - split binary so that pod requests less permissions Jul 9, 2026
@denis-zaitsev
denis-zaitsev merged commit 063f519 into main Jul 9, 2026
14 of 15 checks passed
marcuscollins pushed a commit that referenced this pull request Jul 10, 2026
…304)

Just bumping ext version.
Only architectural changes there - we split binaries into tracing and
cli to request less permissions for k8s pods.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Chores**
* Updated the bundled Astera extension to a newer version, which may
include fixes and improvements in the built image.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
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.

3 participants