-
Notifications
You must be signed in to change notification settings - Fork 556
state: add RavenDB support #3702
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.
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
Merged
Merged
Changes from 11 commits
Commits
Show all changes
103 commits
Select commit
Hold shift + click to select a range
1e5c883
3318: Added files related to RavenDB implementation from old branch f…
nmalocicvega 20751eb
Fixed lint issues
nmalocicvega 813f72f
Fixed go mod so that conformance test can run
nmalocicvega 2c82f28
Added raven to github pipeline tests
nmalocicvega 53fef16
Add docker setup befor conformance test running
nmalocicvega 768a625
Changed extension ffrom yaml to yml
nmalocicvega aa20d7b
Update state/ravendb/metadata.yaml
nmalocic a1fa966
Update state/ravendb/metadata.yaml
nmalocic 358113c
Update state/ravendb/metadata.yaml
nmalocic 3ef8df4
Update state/ravendb/ravendb.go
nmalocic 62b075c
Merge branch 'main' into 3318-RavenDB-state-store-new
elena-kolevska 8aae575
Merge branch 'main' into 3318-RavenDB-state-store-new
cicoyle 3003286
Fix comments from the PR
nmalocicvega f2e6e41
Fix linting for test file
nmalocicvega 91d15de
Merge branch 'main' into 3318-RavenDB-state-store-new
msfussell 36315dc
3318: update referance to fix conflict
nmalocicvega a431b0b
Merge branch 'main' into 3318-RavenDB-state-store-new
nmalocic ba751c7
Adding aditional test as per comments
nmalocicvega 9c893ea
Merge branch 'main' into 3318-RavenDB-state-store-new
nmalocic e4db81a
runing go mod tidy
nmalocicvega abbfd98
Merge branch 'main' into 3318-RavenDB-state-store-new
nmalocicvega 277c176
Merge branch 'main' into 3318-RavenDB-state-store-new
nmalocicvega 4398cff
Merge branch 'main' into 3318-RavenDB-state-store-new
cicoyle c985605
Merge branch 'main' into 3318-RavenDB-state-store-new
cicoyle 4bbe532
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 0120009
Merge branch 'main' into 3318-RavenDB-state-store-new
nmalocic b53fa33
Read license from environemnt
nmalocicvega 5c74fe5
Add license to infrastructre docker compose
nmalocicvega be15d17
Merge branch 'main' into 3318-RavenDB-state-store-new
nmalocic 60a96f2
Merge branch 'main' into 3318-RavenDB-state-store-new
yaron2 9f20115
Merge branch 'main' into 3318-RavenDB-state-store-new
nmalocicvega 9b591b6
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 95abdbe
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 3b0f134
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot e53e0b4
Setup env variable from secret
nmalocicvega a4b52f8
Merge branch '3318-RavenDB-state-store-new' of github.com-n-personal:…
nmalocicvega eae0f77
Update .github/scripts/test-info.mjs
nmalocic 8058c19
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 1f504d5
Change way how Ravendb license is exposed to pipeline
nmalocicvega fdd78b5
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 6d6126c
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot bd5858b
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 7576537
3318: Fix test for deleting non existing key
nmalocicvega 4ef65c7
Merge branch '3318-RavenDB-state-store-new' of github.com-n-personal:…
nmalocicvega eae9409
Merge branch 'main' into 3318-RavenDB-state-store-new
nmalocicvega e90f51d
Merge branch 'main' into 3318-RavenDB-state-store-new
nmalocicvega 44bfb68
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot d154dc3
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot e1cab7e
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 27013c8
Merge branch 'main' into 3318-RavenDB-state-store-new
yaron2 2d64f55
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 701d8f0
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 6cf4257
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot fb1e34c
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot e9bf228
fix(ci): add missing matrix property
mikeee 1e6c718
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 984a62f
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 4be9b6f
Merge branch 'main' into 3318-RavenDB-state-store-new
mikeee 4d0a907
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 26a6c5f
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot f2ae375
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 9cb1cbb
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 026dbb2
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot d6d4b9e
Merge branch 'main' into 3318-RavenDB-state-store-new
mikeee a792da9
Merge branch 'main' into 3318-RavenDB-state-store-new
mikeee b8ac123
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot e57396e
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 9c8133a
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 6594658
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 3a4e69c
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot c18c52c
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot d7bd477
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 9d3bfee
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 4bda165
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot b270c9f
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 86ca49f
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot b02f323
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 87443d5
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 0d297f3
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot ed58fb1
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot f4c37cc
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot dbe76a7
Merge branch 'main' into 3318-RavenDB-state-store-new
mikeee c284dee
Use hardcoded dev license to check if pipline wil work, we should rev…
nmalocicvega 6091df4
Merge branch '3318-RavenDB-state-store-new' of github.com-n-personal:…
nmalocicvega 2411030
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 86a4a86
Revert license to be pulled from env since hardcoded worked, added de…
nmalocicvega f2cff15
Merge branch '3318-RavenDB-state-store-new' of github.com-n-personal:…
nmalocicvega 55ec1e0
Merge branch 'main' into 3318-RavenDB-state-store-new
mikeee 70640a4
Add better logging when license is not valid to docker image
nmalocicvega a705e9d
Merge branch '3318-RavenDB-state-store-new' of github.com-n-personal:…
nmalocicvega 417e678
Hardcode ravenlicense until pipeline is fixed
nmalocicvega 7e87dfa
Merge branch 'main' into 3318-RavenDB-state-store-new
nmalocicvega a8d6754
Change how license is saved in yaml files, remove env from pipeline
nmalocicvega 29112b6
Code cleanup based on comments from PR
nmalocicvega 1919cb7
Merge branch 'main' into 3318-RavenDB-state-store-new
nmalocicvega c12721b
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 17b4ee1
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 7b13c32
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 6370a26
Merge branch 'main' into 3318-RavenDB-state-store-new
dapr-bot 461b8b1
Apply suggestion from @elena-kolevska
nmalocic 2918861
Apply suggestions from code review
nmalocic 1a4b7f4
chore: update state/ravendb/metadata.yaml
mikeee f515fbd
Inculde TTL in metadata as per codereview
nmalocicvega File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| services: | ||
| ravendb: | ||
| image: ravendb/ravendb | ||
| container_name: RavenDb | ||
| ports: | ||
| - "8080:8080" | ||
| environment: | ||
| - RAVEN_LICENSE={"Id":"1bcf6b3f-cd52-488c-9aa2-c982d6e57c78","Name":"RavenDB","Keys":["CtmBDhwRJAJIDOHv+Pev+B8o4","A5l3K4n1z2FbtgHsKPl1QkQNx","LPqr9nkPEq5MUCHQZ0XkkP5BY","Czc9DkF33Y8mNFemJZl8tkGm6","DY2Q6YyYtEt/jQayDOUVUtUlY","FC4inkD4zkb3Ii2WEbyv5RRew","jzoKxeOVYXQsToAMe6xG9ABYE","DNy4wBSYoSQMqKywtLi8wJzEy","MzQVFjc4OTo7PD0+nwIfIJ8CI","CCfAiEgnwIjIJ8CJCCfAiUgnw","ImIJ8CJyCfAiggnwIpIJ8CKiC","fAisgnwIsIJ8CLSCfAi4gnwIv","IJ8CMCCfAzZAAZ8CQiBDJEQJY","htbnwRBYBhb"]} | ||
| - RAVEN_DATABASE=testdapr | ||
| - RAVEN_Setup_Mode=None | ||
| - RAVEN_License_Eula_Accepted=true | ||
| - RAVEN_Security_UnsecuredAccessAllowed=PrivateNetwork | ||
| restart: unless-stopped | ||
nmalocic marked this conversation as resolved.
Show resolved
Hide resolved
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,51 @@ | ||
| # yaml-language-server: $schema=../../component-metadata-schema.json | ||
| schemaVersion: v1 | ||
| type: state | ||
| name: ravendb | ||
| version: v1 | ||
| status: development-only | ||
mikeee marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| title: "RavenDB" | ||
| urls: | ||
| - title: Reference | ||
| url: | ||
| capabilities: | ||
| - crud | ||
| authenticationProfiles: | ||
| - title: "No authentication" | ||
| description: | | ||
| No authentication. Connect via connection string | ||
| metadata: | ||
| - name : connectionString | ||
| required: true | ||
| description: | | ||
| Connection string to ravenDB cluster | ||
| example: '"http://live-test.ravendb.net"' | ||
| metadata: | ||
| - name: DatabaseName | ||
| description: | | ||
| The name of the database to use. | ||
| default: '"daprStore"' | ||
| example: '"daprStore"' | ||
| - name: ServerURL | ||
| description: | | ||
| Url to ravendb cluster | ||
| default: '"127.0.0.1"' | ||
| example: '"http://live-test.ravendb.net"' | ||
| - name: CertPath | ||
| description: | | ||
| Path to the certificate for secure connection | ||
| example: "/path/to/cert" | ||
| - name: KeyPath | ||
| description: | | ||
| Path to the key for secure connection | ||
| example: "/path/to/key" | ||
| - name: EnableTTL | ||
| description: | | ||
| Boolean value that enables or disables RaveDB TTL functionality | ||
| example: "true" | ||
| default: "true" | ||
| - name: TTLFrequency | ||
| description: | | ||
| Sets RavenDB frequency on running the background expiration task and deleting records | ||
| example: "15" | ||
| default: "60" | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.