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

Adding Dapr configuration support to Testcontainers #1148

Merged
merged 8 commits into from
Oct 17, 2024

Conversation

artur-ciocanu
Copy link
Contributor

Description

This is a tiny PR that adds support for Dapr Configuration to Testcontainers. The PR has been picked up from this one: #1126

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #1125

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

@artur-ciocanu artur-ciocanu requested review from a team as code owners October 17, 2024 13:50
Signed-off-by: Artur Ciocanu <[email protected]>
@artur-ciocanu
Copy link
Contributor Author

artur-ciocanu commented Oct 17, 2024

@artursouza @cicoyle @salaboy whenever you have a few minutes could you please take a look. Thank you!

Artur Ciocanu added 6 commits October 17, 2024 20:14
Signed-off-by: Artur Ciocanu <[email protected]>
Signed-off-by: Artur Ciocanu <[email protected]>
Signed-off-by: Artur Ciocanu <[email protected]>
@artur-ciocanu
Copy link
Contributor Author

@artursouza or @cicoyle I would really appreciate if anyone of you could re-trigger the build. One of the build steps died with "Could not initialize Dapr". I have never seen an issue like this, but I think a new build won't hurt.

Thank you!

@artursouza artursouza merged commit b1196d3 into dapr:master Oct 17, 2024
7 checks passed
@artur-ciocanu artur-ciocanu deleted the gh-1125 branch October 18, 2024 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for Dapr Configuration on DaprContainer (testcontainers)
2 participants