Skip to content

feat(policy): DSPX-902 NDR service crud protos only (1/2) #2092

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 29 commits into
base: main
Choose a base branch
from
Open

feat(policy): DSPX-902 NDR service crud protos only (1/2) #2092

wants to merge 29 commits into from

Conversation

ryanulit
Copy link
Contributor

Proposed Changes

  • service CRUD proto method definitions and generated files only

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

ryanulit added 29 commits April 15, 2025 09:03
@ryanulit
add protos and generate
9592e69
@ryanulit
fix missed rename
d47cdc4
@ryanulit
add reg res db client methods except list and initial test setup
4c57dfa
@ryanulit
add list sql queries and client methods
62ddb7a
@ryanulit
add integration tests for create db client methods
ba348b5
@ryanulit
Merge branch 'main' into DSPX-901-ndr-database-crud
ff8596e
@ryanulit
add delete integration tests
093c19d
@ryanulit
add update integration tests
5dd05d2
@ryanulit
Merge branch 'main' into DSPX-901-ndr-database-crud
e13fc7d
@ryanulit
fix proto lint errors
569d0fb
@ryanulit
more lint fixes
1f8bd21
@ryanulit
Merge branch 'main' into DSPX-901-ndr-database-crud
dd571fb
@ryanulit
add get integration tests
f1101fc
@ryanulit
Merge branch 'main' into DSPX-901-ndr-database-crud
c73118a
@ryanulit
Merge branch 'main' into DSPX-901-ndr-database-crud
56d6e5a
@ryanulit
lint fix
9647b24
@ryanulit
add list sql filter and fix bugs
dd17373
@ryanulit
refactor integration tests to not modify fixture data
e05a007
@ryanulit
add list integration tests
6acd110
@ryanulit
fix lint issues
5fff148
@ryanulit
Merge branch 'main' into DSPX-901-ndr-database-crud
25e55a1
@ryanulit
add service proto and proto tests
e2b17db
@ryanulit
implement service methods
58f47c9
@ryanulit
Merge branch 'main' into DSPX-902-ndr-service-crud
8234149
@ryanulit
Merge branch 'main' into DSPX-902-ndr-service-crud
92114a9
@ryanulit
use pointer receiver and update gets to use request msg correcly
9244101
@ryanulit
Merge branch 'main' into DSPX-902-ndr-service-crud
4cbe1cb
@ryanulit
fix failing services tests
3ef2569
@ryanulit
break out proto only changes from branch of service crud PR
ee28e63
@ryanulit ryanulit changed the title feat(policy): DSPX-902 NDR service crud (1/2 protos only) feat(policy): DSPX-902 NDR service crud protos only (1/2) Apr 22, 2025
@ryanulit ryanulit marked this pull request as ready for review April 22, 2025 20:04
@ryanulit ryanulit requested review from a team as code owners April 22, 2025 20:04
@ryanulit ryanulit enabled auto-merge April 22, 2025 20:06
@github-actions GitHub Actions
Copy link
Contributor

Standard Benchmark Skipped or Failed

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 374.40634ms
Throughput 267.09 requests/second

@ryanulit ryanulit added this pull request to the merge queue Apr 22, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jakedoublev jakedoublev jakedoublev approved these changes

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
@ryanulit @jakedoublev