Skip to content

fix(deps): upgrade sdk to 0.30.0 in service - #3859

Merged
elizabethhealy merged 1 commit into
mainfrom
upgrade-sdk-in-service-to-0.30.0
Aug 11, 2026
Merged

fix(deps): upgrade sdk to 0.30.0 in service#3859
elizabethhealy merged 1 commit into
mainfrom
upgrade-sdk-in-service-to-0.30.0

Conversation

@elizabethhealy

@elizabethhealy elizabethhealy commented Aug 11, 2026

Copy link
Copy Markdown
Member

Proposed Changes

  • upgrade sdk

Checklist

  • I have added or updated unit tests
  • I have added or updated integration tests (if appropriate)
  • I have added or updated documentation

Testing Instructions

Summary by CodeRabbit

  • Chores
    • Updated an underlying platform component to the latest supported version.
    • No direct end-user features or behavior changes are included.

@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 8d3b2bdb-b14f-4396-b602-5f50e0368646

📥 Commits

Reviewing files that changed from the base of the PR and between d1833ad and bc30853.

⛔ Files ignored due to path filters (1)
  • service/go.sum is excluded by !**/*.sum
📒 Files selected for processing (1)
  • service/go.mod

📝 Walkthrough

Walkthrough

The service module updates its direct github.com/opentdf/platform/sdk dependency from v0.29.0 to v0.30.0.

Changes

SDK dependency upgrade

Layer / File(s) Summary
Update service SDK requirement
service/go.mod
The service module requirement changes from SDK v0.29.0 to v0.30.0.

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

Possibly related PRs

Suggested labels: comp:sdk

Suggested reviewers: jakedoublev, c-r33d

Poem

A rabbit checks the module line,
“The SDK hops forward, version fine!”
From thirty-nine? No, thirty-oh,
A tidy change makes services grow.
Hop hop! the lockstep path is clear.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the SDK dependency upgrade from the changeset.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch upgrade-sdk-in-service-to-0.30.0

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.

@elizabethhealy
elizabethhealy marked this pull request as ready for review August 11, 2026 17:46
@elizabethhealy
elizabethhealy requested review from a team as code owners August 11, 2026 17:46
@github-actions

Copy link
Copy Markdown
Contributor
Benchmark results, click to expand

Benchmark authorization.GetDecisions Results:

Metric Value
Approved Decision Requests 1000
Denied Decision Requests 0
Total Time 299.088106ms

Benchmark authorization.v2.GetMultiResourceDecision Results:

Metric Value
Approved Decision Requests 1000
Denied Decision Requests 0
Total Time 165.45893ms

Benchmark Statistics

Name № Requests Avg Duration Min Duration Max Duration

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 436.333241ms
Throughput 229.18 requests/second

TDF3 Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 40.930622896s
Average Latency 408.516983ms
Throughput 122.16 requests/second

@elizabethhealy
elizabethhealy added this pull request to the merge queue Aug 11, 2026
Merged via the queue into main with commit 3514816 Aug 11, 2026
49 of 50 checks passed
@elizabethhealy
elizabethhealy deleted the upgrade-sdk-in-service-to-0.30.0 branch August 11, 2026 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants