Skip to content

fix(core): Autobump sdk #2074

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

fix(core): Autobump sdk #2074

wants to merge 4 commits into from

Conversation

opentdf-automation[bot]
Copy link
Contributor

No description provided.

@opentdf-automation opentdf-automation bot requested review from a team as code owners April 16, 2025 21:49
@opentdf-automation opentdf-automation bot enabled auto-merge April 16, 2025 21:49
github-actions[bot]
github-actions bot previously approved these changes Apr 16, 2025
View reviewed changes
@github-actions GitHub Actions
Copy link
Contributor

Benchmark Statistics

Name № Requests Avg Duration Min Duration Max Duration
publickey 3 0.063 ms 0.036 ms 0.100 ms
rewrap-nanotdf 4722 427.002 ms 82.323 ms 999.665 ms
rewrap-tdf3 5001 454.455 ms 46.117 ms 1048.813 ms

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 357.822845ms
Throughput 279.47 requests/second

@github-actions GitHub Actions
Copy link
Contributor

Benchmark Statistics

Name № Requests Avg Duration Min Duration Max Duration
publickey 3 0.074 ms 0.041 ms 0.121 ms
rewrap-nanotdf 4832 443.506 ms 77.181 ms 1182.293 ms
rewrap-tdf3 5001 459.769 ms 40.971 ms 1291.668 ms

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 363.430218ms
Throughput 275.16 requests/second

@jakedoublev jakedoublev mentioned this pull request Apr 17, 2025
Merged
3 tasks
github-merge-queue bot pushed a commit that referenced this pull request Apr 17, 2025
@jakedoublev
fix(ci): allow SDK and other modules to differ in go version (#2077)
a47a76e 
### Proposed Changes

* SDK will not release as go version used in CI from `service` go mod is
`1.24`, causing [lint failures in SDK test
files](#2074), but SDK is on
version 1.23 so the [lint suggestions are not
possible](https://github.com/opentdf/platform/actions/runs/14519104309/job/40735372961?pr=2077)

### 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
@github-actions GitHub Actions
Copy link
Contributor

Benchmark Statistics

Name № Requests Avg Duration Min Duration Max Duration
publickey 3 0.067 ms 0.031 ms 0.107 ms
rewrap-nanotdf 4887 410.065 ms 59.984 ms 1011.202 ms
rewrap-tdf3 5001 453.844 ms 46.086 ms 1119.095 ms

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 369.527224ms
Throughput 270.62 requests/second

@github-actions GitHub Actions
Copy link
Contributor

Benchmark Statistics

Name № Requests Avg Duration Min Duration Max Duration
publickey 3 0.068 ms 0.051 ms 0.098 ms
rewrap-nanotdf 4931 412.360 ms 69.127 ms 921.725 ms
rewrap-tdf3 5001 441.546 ms 53.625 ms 1105.661 ms

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 359.215233ms
Throughput 278.38 requests/second

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jakedoublev @dmihalcik-virtru