-
Notifications
You must be signed in to change notification settings - Fork 522
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
Patching tests to pass (C# timing, Python uvicorn) #933
Merged
Conversation
This file contains 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
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
msfussell
approved these changes
Sep 22, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approve
msfussell
added a commit
that referenced
this pull request
Sep 25, 2023
* Add holopin.yml config (#923) Signed-off-by: Marc Duiker <marcduiker@users.noreply.github.com> * Updating Readme's to default to `dapr run -f .` (#874) * Updating Invoke readme in C# to favor dapr run -f Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> * Adding back single app run steps and tests, but to the end. Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Update service_invocation/csharp/http/README.md Co-authored-by: Mark Fussell <markfussell@gmail.com> Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com> * Updated and hardened test Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Invoke * and partial PubSub with multi-run files & readmes & tests Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Multi run done for Pubsub C#, Python and Node (http, sdk) Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> * Added Pubsub support for Go, Java Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> * updating readmes with correct appid Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> * Updating tests for PubSub and Service invoke to Paas with multiapp run Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * More fine tuning of pubsub tests (timeouts, step ends) Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Fixing Python FastAPI and C# Pubsub tests Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> --------- Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com> Co-authored-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> Co-authored-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> Co-authored-by: Mark Fussell <markfussell@gmail.com> * Patching tests to pass (C# timing, Python uvicorn) (#933) Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com> --------- Signed-off-by: Marc Duiker <marcduiker@users.noreply.github.com> Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com> Co-authored-by: Marc Duiker <marcduiker@users.noreply.github.com> Co-authored-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> Co-authored-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> Co-authored-by: Mark Fussell <markfussell@gmail.com>
msfussell
added a commit
that referenced
this pull request
Oct 13, 2023
* update dapr runtime to use v1.12.0-rc.1 Signed-off-by: Cassandra Coyle <cassie@diagrid.io> * update to latest rc for runtime and cli Signed-off-by: Cassandra Coyle <cassie@diagrid.io> * try sleeping Signed-off-by: Cassandra Coyle <cassie@diagrid.io> * pin redis version Signed-off-by: Cassandra Coyle <cassie@diagrid.io> * rm sleep Signed-off-by: Cassandra Coyle <cassie@diagrid.io> * rm deprecated unsubscribe call Signed-off-by: Cassandra Coyle <cassie@diagrid.io> * Update configuration/go/sdk/order-processor/app.go Co-authored-by: Josh van Leeuwen <me@joshvanl.dev> Signed-off-by: Cassie Coyle <cassie.i.coyle@gmail.com> * Statemanagement multiapp run Signed-off-by: rabollin <rabollin@microsoft.com> * message sequence Signed-off-by: rabollin <rabollin@microsoft.com> * message sequence Signed-off-by: rabollin <rabollin@microsoft.com> * Multi-run support for workflow Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Removing component folder from run file to match readme Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * aligning state store names on `statestore` Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Adding multi-app run for python and C# Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * pr comments Signed-off-by: rabollin <rabollin@microsoft.com> * pr comments Signed-off-by: rabollin <rabollin@microsoft.com> * sleep value increase Signed-off-by: rabollin <rabollin@microsoft.com> * sleep seconds increase Signed-off-by: rabollin <rabollin@microsoft.com> * Result correction Signed-off-by: rabollin <rabollin@microsoft.com> * Result correction Signed-off-by: rabollin <rabollin@microsoft.com> * java sdk dapr run output format fixing Signed-off-by: rabollin <rabollin@microsoft.com> * Syncing Release 1.12 with master and multi-run changes (#936) * Add holopin.yml config (#923) Signed-off-by: Marc Duiker <marcduiker@users.noreply.github.com> * Updating Readme's to default to `dapr run -f .` (#874) * Updating Invoke readme in C# to favor dapr run -f Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> * Adding back single app run steps and tests, but to the end. Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Update service_invocation/csharp/http/README.md Co-authored-by: Mark Fussell <markfussell@gmail.com> Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com> * Updated and hardened test Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Invoke * and partial PubSub with multi-run files & readmes & tests Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Multi run done for Pubsub C#, Python and Node (http, sdk) Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> * Added Pubsub support for Go, Java Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> * updating readmes with correct appid Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> * Updating tests for PubSub and Service invoke to Paas with multiapp run Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * More fine tuning of pubsub tests (timeouts, step ends) Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Fixing Python FastAPI and C# Pubsub tests Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> --------- Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com> Co-authored-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> Co-authored-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> Co-authored-by: Mark Fussell <markfussell@gmail.com> * Patching tests to pass (C# timing, Python uvicorn) (#933) Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com> --------- Signed-off-by: Marc Duiker <marcduiker@users.noreply.github.com> Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com> Co-authored-by: Marc Duiker <marcduiker@users.noreply.github.com> Co-authored-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> Co-authored-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> Co-authored-by: Mark Fussell <markfussell@gmail.com> * Fixing MMD tests with tweaks Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * setting version for kafka (#939) * fixing version for kafka Signed-off-by: Pravin Pushkar <ppushkar@microsoft.com> * validate data with some initial values instead of larger Signed-off-by: Pravin Pushkar <ppushkar@microsoft.com> --------- Signed-off-by: Pravin Pushkar <ppushkar@microsoft.com> * Updating test env to Dapr runtime 1.12 RC4 and CLI RC2 Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Update README.md Fixing typo. should be singular Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com> * Update README.md Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com> * add java workflow quickstart Signed-off-by: Sky Ao <aoxiaojian@gmail.com> * Updating runtime for tests to RC5 Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Fixes port 6003 conflict between Go Pub-sub and Go bindings. Timing issue Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * add hello-kubernetes multi-app run k8s template Signed-off-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com> * address review comments. Signed-off-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com> * update cli to rc.4 (#943) * update cli to rc.3 Signed-off-by: Cassandra Coyle <cassie@diagrid.io> * update cli to rc.4 Signed-off-by: Cassandra Coyle <cassie@diagrid.io> --------- Signed-off-by: Cassandra Coyle <cassie@diagrid.io> Signed-off-by: Cassie Coyle <cassie.i.coyle@gmail.com> * Update CLI to rc4 Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Updating .NET SDK to 1.12.* Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Dapr Golang SDK update to v1.9.0-rc-1 Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Javascript SDK to 3.1.2 Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Release 1.12 (#945) * Dapr Golang SDK update to v1.9.0-rc-1 Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Javascript SDK to 3.1.2 Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> --------- Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> Co-authored-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Java SDK update to <version>1.10.0-rc-1</version> Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Updated Java SDK to RC, and donated multi-app-run support too :) Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> * Updating tests to use dapr/dapr 1.12.0-rc.6 Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> * sdk version changes Signed-off-by: rabollin <rabollin@microsoft.com> * CLI and Runtime version changes Signed-off-by: rabollin <rabollin@microsoft.com> * go.sum refresh Signed-off-by: rabollin <rabollin@microsoft.com> * Updating with js-sdk 3.2.0 Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> --------- Signed-off-by: Cassandra Coyle <cassie@diagrid.io> Signed-off-by: Cassie Coyle <cassie.i.coyle@gmail.com> Signed-off-by: rabollin <rabollin@microsoft.com> Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> Signed-off-by: Marc Duiker <marcduiker@users.noreply.github.com> Signed-off-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com> Signed-off-by: Pravin Pushkar <ppushkar@microsoft.com> Signed-off-by: Sky Ao <aoxiaojian@gmail.com> Signed-off-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com> Co-authored-by: Cassandra Coyle <cassie@diagrid.io> Co-authored-by: Cassie Coyle <cassie.i.coyle@gmail.com> Co-authored-by: Josh van Leeuwen <me@joshvanl.dev> Co-authored-by: rabollin <rabollin@microsoft.com> Co-authored-by: Paul Yuknewicz <paulyuk@Pauls-MBP-2.lan> Co-authored-by: Marc Duiker <marcduiker@users.noreply.github.com> Co-authored-by: Paul Yuknewicz <paulyuk@Pauls-MacBook-Pro-2.local> Co-authored-by: Mark Fussell <markfussell@gmail.com> Co-authored-by: Pravin Pushkar <ppushkar@microsoft.com> Co-authored-by: Sky Ao <aoxiaojian@gmail.com> Co-authored-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Patching tests to pass (C# timing, Python uvicorn)
Issue reference
We strive to have all PRs 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: #[issue number]
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: