- 
                Notifications
    You must be signed in to change notification settings 
- Fork 542
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
          
     Open
      
      
            nmalocic
  wants to merge
  61
  commits into
  dapr:main
  
    
      
        
          
  
    
      Choose a base branch
      
     
    
      
        
      
      
        
          
          
        
        
          
            
              
              
              
  
           
        
        
          
            
              
              
           
        
       
     
  
        
          
            
          
            
          
        
       
    
      
from
nmalocic:3318-RavenDB-state-store-new
  
      
      
   
  
    
  
  
  
 
  
      
    base: main
Could not load branches
            
              
  
    Branch not found: {{ refName }}
  
            
                
      Loading
              
            Could not load tags
            
            
              Nothing to show
            
              
  
            
                
      Loading
              
            Are you sure you want to change the base?
            Some commits from the old base branch may be removed from the timeline,
            and old review comments may become outdated.
          
          
      
        
          +1,083
        
        
          −0
        
        
          
        
      
    
  
  
     Open
                    Changes from all commits
      Commits
    
    
            Show all changes
          
          
            61 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 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=${RAVENDBLICENSE} | ||
| - RAVEN_DATABASE=testdapr | ||
| - RAVEN_Setup_Mode=None | ||
| - RAVEN_License_Eula_Accepted=true | ||
| - RAVEN_Security_UnsecuredAccessAllowed=PrivateNetwork | ||
| restart: unless-stopped | 
  
    
      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
    
  
  
    
              
  
    
      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 | ||
| title: "RavenDB" | ||
| urls: | ||
| - title: Reference | ||
| url: https://docs.dapr.io/reference/components-reference/supported-state-stores/setup-ravendb/ | ||
| 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.
        
    
  
  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.