Skip to content
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

Validate CosmosDB state store for Actor #2243

Closed
artursouza opened this issue Oct 12, 2020 · 4 comments
Closed

Validate CosmosDB state store for Actor #2243

artursouza opened this issue Oct 12, 2020 · 4 comments
Assignees
Labels
area/runtime/actor kind/bug Something isn't working P1 size/S 1 week of work triaged/resolved Indicates that this issue has been triaged

Comments

@artursouza
Copy link
Member

artursouza commented Oct 12, 2020

In what area(s)?

/area runtime

/area operator
/area placement
/area docs
/area test-and-release

What version of Dapr?

0.1.x
0.2.x
Output of git describe --dirty
0.10

Expected Behavior

CosmosDB can be used to save state in Actord

Actual Behavior

Example from Java SDK did not work when switched to CosmosDB

Steps to Reproduce the Problem

Use Actor's example from Java SDK
Configure statestore to use CosmosDB

Release Note

RELEASE NOTE: Validated CosmosDB state store for actors

@artursouza artursouza added the kind/bug Something isn't working label Oct 12, 2020
@artursouza artursouza added this to To do in 1.0.0 Milestone 1 via automation Oct 12, 2020
@artursouza artursouza added P1 area/runtime/actor size/S 1 week of work triaged/resolved Indicates that this issue has been triaged labels Oct 12, 2020
@yaron2
Copy link
Member

yaron2 commented Oct 12, 2020

@KaiWalter, are you using Dapr and CosmosDB for actors? did you notice any issues with Dapr => v0.10?

@KaiWalter
Copy link
Contributor

@yaron2 we noticed no regressions - neither 0.10 nor 0.11 - and we're running our tests really regularly

@artursouza artursouza added this to To do in 1.0.0 Milestone Oct 27, 2020
@artursouza artursouza moved this from To do to Planned in 1.0.0 Milestone 1 Nov 3, 2020
@artursouza artursouza moved this from Planned to To do in 1.0.0 Milestone 1 Nov 3, 2020
@artursouza artursouza self-assigned this Nov 4, 2020
@artursouza artursouza moved this from To do to Planned in 1.0.0 Milestone 1 Nov 4, 2020
@artursouza
Copy link
Member Author

I replayed the example from the Java SDK using cosmosdb and I got the following error:

== APP == io.dapr.exceptions.DaprException: ERR_ACTOR_STATE_TRANSACTION_SAVE: BadRequest, Message: {"Errors":["Encountered exception while executing function. Exception = Error: {\"Errors\":[\"Requests originating from scripts cannot reference partition keys other than the one for which client request was submitted.\"]}\r\nStack trace: Error: {\"Errors\":[\"Requests originating from scripts cannot reference partition keys other than the one for which client request was submitted.\"]}\n   at callback (script.js:51:18)\n   at Anonymous function (script.js:759:29)"]}

The reminders to get persisted in the Collection but the actor state do not.

@artursouza artursouza assigned yaron2 and unassigned artursouza Nov 6, 2020
@yaron2 yaron2 closed this as completed Nov 6, 2020
1.0.0 Milestone 1 automation moved this from Planned to Review Nov 6, 2020
1.0.0 Milestone automation moved this from To do to Done Nov 6, 2020
@artursouza artursouza moved this from Review to Done in 1.0.0 Milestone 1 Nov 9, 2020
@artursouza artursouza moved this from Done to Release in 1.0.0 Milestone 1 Nov 11, 2020
@artursouza
Copy link
Member Author

artursouza commented Nov 23, 2020

Partition key in the container must be partitionKey.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/runtime/actor kind/bug Something isn't working P1 size/S 1 week of work triaged/resolved Indicates that this issue has been triaged
Projects
No open projects
Development

No branches or pull requests

3 participants