-
Notifications
You must be signed in to change notification settings - Fork 209
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
Add actor testcontainer tests #1192
base: master
Are you sure you want to change the base?
Conversation
@salaboy I have currently the problem that the test fails with the following exception when trying to connect to the Dapr gRPC port.
Not sure why it uses port 50001, because this is the internal port. I think normally it should use the mapped port. Maybe you can have a look, if the test is not correct configured? |
@akkie thanks a lot of this.. give me some time to look into this.. check the DCO, we need that to approve the PR. Click on the Details link to see the steps to fix it. |
@salaboy Is it OK if I do a force push regarding the update of the DCO? |
@akkie yeah.. that is your fork.. so it is ok |
Signed-off-by: Christian Kaps <[email protected]>
ee83cff
to
bec4bd2
Compare
@akkie would you mind adding me as a collaborator to your fork? I am working on a fix, but I would love to push to your fork.. if not I can send you a patch to apply to your fork with the fix |
Done |
@akkie I will push two commits to your fork, I am stuck with a new error now.. but at least the connection is working now. |
Now I am stuck with this:
But the connection is working as far as I can tell. |
This is strange.. because it looks like we are hitting this: dapr/dapr#6783 @artursouza do you know if these examples are executed as part of the tests? |
@salaboy This is exactly the message we get with our testcontainer setup:
We use .NET and not Java. |
Ok so I think we are up to something here
- Blog: http://salaboy.com <http://salaboy.wordpress.com>
- Github user: http://github.com/salaboy
- Twitter: http://twitter.com/salaboy
- Mauricio "Salaboy" Salatino -
…On Wed, 22 Jan 2025 at 20:11, Christian Kaps ***@***.***> wrote:
@salaboy <https://github.com/salaboy> This is exactly the message we get
with our testcontainer setup:
´´´
fails to send binding event to http app channel, status code: 500 body:
Dapr.DaprApiException: error invoke actor method: did not find address for
actor
´´´
We use .NET and not Java.
—
Reply to this email directly, view it on GitHub
<#1192 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACCMXUKSIIUSSF7NQM2YJT2L73O5AVCNFSM6AAAAABVI4UL56VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMBYGE3TANRUHE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Description
This PR adds testcontainer based integration tests for actors. This is to make sure that actors really work with testcontainers.
Issue reference
This PR was crated based on a Discord discussion with @salaboy to check if actors works with testcontainers.