[pull] master from dapr:master #274
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
![pull[bot]](https://avatars.githubusercontent.com/in/12910?s=60&v=4){kind=small}
Codecov Report
@@ Coverage Diff @@
## master #274 +/- ##
==========================================
+ Coverage 58.17% 58.89% +0.72%
==========================================
Files 82 87 +5
Lines 7347 7708 +361
==========================================
+ Hits 4274 4540 +266
- Misses 2779 2857 +78
- Partials 294 311 +17
Continue to review full report at Codecov.
|
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## master #274 +/- ##
==========================================
+ Coverage 58.17% 64.55% +6.38%
==========================================
Files 82 161 +79
Lines 7347 17069 +9722
==========================================
+ Hits 4274 11019 +6745
- Misses 2779 5186 +2407
- Partials 294 864 +570
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Co-authored-by: Josh van Leeuwen <me@joshvanl.dev> Signed-off-by: Cassie Coyle <cassie.i.coyle@gmail.com>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Signed-off-by: Elena Kolevska <elena@kolevska.com>
* Linter: Updates test code for new linter rules Updates all test code to adhear to the knew linter rules. Branched from #8146 to ensure that no test changes were made for that PR to test that no implementation functional changes were made. Removes rule in github workflows to only test new changes. Signed-off-by: joshvanl <me@joshvanl.dev> * Review comments Signed-off-by: joshvanl <me@joshvanl.dev> * Fix app pubsub string Signed-off-by: joshvanl <me@joshvanl.dev> --------- Signed-off-by: joshvanl <me@joshvanl.dev>
…8149) * feat(outbox): use content-type in the metadata for projections item Signed-off-by: Luigi Rende <luigirende@gmail.com> * feat(outbox): add integration test for contenType attribute in metadata Signed-off-by: Luigi Rende <luigirende@gmail.com> --------- Signed-off-by: Luigi Rende <luigirende@gmail.com> Co-authored-by: Josh van Leeuwen <me@joshvanl.dev> Co-authored-by: Mike Nguyen <hey@mike.ee>
Signed-off-by: Loong <loong.dai@intel.com> Co-authored-by: Yaron Schneider <schneider.yaron@live.com>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
…sion Update the golangci-lint action
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
* conversation api: add intg tests Signed-off-by: yaron2 <schneider.yaron@live.com> * linter Signed-off-by: yaron2 <schneider.yaron@live.com> * Update tests/integration/suite/daprd/conversation/grpc/basic.go Co-authored-by: Josh van Leeuwen <me@joshvanl.dev> Signed-off-by: Yaron Schneider <schneider.yaron@live.com> * Update tests/integration/suite/daprd/conversation/grpc/basic.go Co-authored-by: Josh van Leeuwen <me@joshvanl.dev> Signed-off-by: Yaron Schneider <schneider.yaron@live.com> * Update tests/integration/suite/daprd/conversation/http/basic.go Co-authored-by: Josh van Leeuwen <me@joshvanl.dev> Signed-off-by: Yaron Schneider <schneider.yaron@live.com> * Update tests/integration/suite/daprd/conversation/grpc/basic.go Co-authored-by: Josh van Leeuwen <me@joshvanl.dev> Signed-off-by: Yaron Schneider <schneider.yaron@live.com> * review feedback Signed-off-by: yaron2 <schneider.yaron@live.com> --------- Signed-off-by: yaron2 <schneider.yaron@live.com> Signed-off-by: Yaron Schneider <schneider.yaron@live.com> Co-authored-by: Josh van Leeuwen <me@joshvanl.dev>
Signed-off-by: Anton Troshin <anton@diagrid.io>
Signed-off-by: Anton Troshin <anton@diagrid.io>
Fix CVE GO-2024-3250 bump github.com/diagridio/go-etcd-cron
…onnect-test Skip Scheduler Reconnect Test on Windows
* Add support for test framework to run with local Kind cluster Signed-off-by: Anton Troshin <anton@diagrid.io> * fix lint Signed-off-by: Anton Troshin <anton@diagrid.io> * Add README note Add env variable to override default kind cluster name for local testing Signed-off-by: Anton Troshin <anton@diagrid.io> * CR fixes Add DAPR_TEST_REGISTRY_PORT env variable Fix usage of DAPR_TEST_KIND_CLUSTER_NAME Signed-off-by: Anton Troshin <anton@diagrid.io> * lint Signed-off-by: Anton Troshin <anton@diagrid.io> * Add note to set target arch before build Signed-off-by: Anton Troshin <anton@diagrid.io> * Add windows tag to integration test Signed-off-by: Anton Troshin <anton@diagrid.io> --------- Signed-off-by: Anton Troshin <anton@diagrid.io> Co-authored-by: Josh van Leeuwen <me@joshvanl.dev> Co-authored-by: Elena Kolevska <elena-kolevska@users.noreply.github.com> Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Mustafa Arslan <mustafa.arslan1992@gmail.com> Co-authored-by: Elena Kolevska <elena-kolevska@users.noreply.github.com> Co-authored-by: Artur Souza <asouza.pro@gmail.com> Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
* chore: add logging to dapr-bot script Signed-off-by: mikeee <hey@mike.ee> * fix: console spacing Signed-off-by: Mike Nguyen <hey@mike.ee> --------- Signed-off-by: mikeee <hey@mike.ee> Signed-off-by: Mike Nguyen <hey@mike.ee> Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com> Co-authored-by: Artur Souza <asouza.pro@gmail.com>
* Resiliency: allow to ignore resiliency policy based on response http status code Signed-off-by: zhangchao <zchao9100@gmail.com> * add filter to helm Signed-off-by: zhangchao <zchao9100@gmail.com> * fix crd Signed-off-by: zhangchao <zchao9100@gmail.com> * fix codes in yaml Signed-off-by: zhangchao <zchao9100@gmail.com> * fix e2e Signed-off-by: zhangchao <zchao9100@gmail.com> * fix e2e and adds filter to input binding Signed-off-by: zhangchao <zchao9100@gmail.com> * fix e2e test Signed-off-by: zhangchao <zchao9100@gmail.com> * Update pkg/resiliency/retry.go Co-authored-by: Alessandro (Ale) Segala <43508+ItalyPaleAle@users.noreply.github.com> Signed-off-by: Taction <zchao9100@gmail.com> * Update pkg/resiliency/retry.go Co-authored-by: Alessandro (Ale) Segala <43508+ItalyPaleAle@users.noreply.github.com> Signed-off-by: Taction <zchao9100@gmail.com> * fix review: change regexp to code range Signed-off-by: zhangchao <zchao9100@gmail.com> * add code filter to grpc proxy Signed-off-by: zhangchao <zchao9100@gmail.com> * fix review and add more test case Signed-off-by: zhangchao <zchao9100@gmail.com> * add grpc proxy e2e tests Signed-off-by: zhangchao <zchao9100@gmail.com> * fix lint Signed-off-by: zhangchao <zchao9100@gmail.com> * fix unit test Signed-off-by: zhangchao <zchao9100@gmail.com> * fix review: rename ignoreOnCodes to noRetryOnCodes Signed-off-by: zhangchao <zchao9100@gmail.com> * fix lint Signed-off-by: zhangchao <zchao9100@gmail.com> * resiliency: add retry on code filter for pubsub with e2e Signed-off-by: zhangchao <zchao9100@gmail.com> * Ignore bad retry status code patterns. Signed-off-by: Artur Souza <asouza.pro@gmail.com> * integration test for retry error code filter. Signed-off-by: Artur Souza <asouza.pro@gmail.com> * onRetryCodes is now string. Removed ignoreRetryCodes Signed-off-by: Artur Souza <asouza.pro@gmail.com> * integration test for retry status code filter for gRPC. Signed-off-by: Artur Souza <asouza.pro@gmail.com> * Redesign retry logic to make it protocol specific. Signed-off-by: Artur Souza <asouza.pro@gmail.com> * remove ref to github.com/dapr/dapr/tests/integration/framework/util Signed-off-by: Artur Souza <asouza.pro@gmail.com> * Fix typo in E2e resiliency yaml. Signed-off-by: Artur Souza <asouza.pro@gmail.com> * Update pkg/resiliency/retry.go Co-authored-by: Josh van Leeuwen <me@joshvanl.dev> Signed-off-by: Artur Souza <asouza.pro@gmail.com> * Use auto-detect status code type for binding. Signed-off-by: Artur Souza <asouza.pro@gmail.com> * Unify status code error for gRPC and HTTP with auto-detect. Signed-off-by: Artur Souza <asouza.pro@gmail.com> * Update charts/dapr/crds/resiliency.yaml Co-authored-by: Anton Troshin <troll.sic@gmail.com> Signed-off-by: Artur Souza <asouza.pro@gmail.com> * Rename conditions to matching Signed-off-by: Artur Souza <asouza.pro@gmail.com> * Fix import Signed-off-by: Anton Troshin <anton@diagrid.io> * fix lint Signed-off-by: Anton Troshin <anton@diagrid.io> * Change the wording from "filter" to "match", to match the field name semantically Signed-off-by: Anton Troshin <anton@diagrid.io> * Fix resiliency match test Add invalid resiliency startup test Signed-off-by: Anton Troshin <anton@diagrid.io> * CR fixes Signed-off-by: Anton Troshin <anton@diagrid.io> * Add multiple integration test cases for HTTP and gRPC retry policy mix of statuses Signed-off-by: Anton Troshin <anton@diagrid.io> * Add gRPC proxy resiliency test Signed-off-by: Anton Troshin <anton@diagrid.io> * fix build Signed-off-by: Anton Troshin <anton@diagrid.io> * lint fix Signed-off-by: Anton Troshin <anton@diagrid.io> --------- Signed-off-by: zhangchao <zchao9100@gmail.com> Signed-off-by: Taction <zchao9100@gmail.com> Signed-off-by: Artur Souza <asouza.pro@gmail.com> Signed-off-by: Anton Troshin <anton@diagrid.io> Co-authored-by: zhangchao <zchao9100@gmail.com> Co-authored-by: Alessandro (Ale) Segala <43508+ItalyPaleAle@users.noreply.github.com> Co-authored-by: Artur Souza <asouza.pro@gmail.com> Co-authored-by: Josh van Leeuwen <me@joshvanl.dev> Co-authored-by: Yaron Schneider <schneider.yaron@live.com>
Merge 1.14 in master
Signed-off-by: Artur Souza <asouza.pro@gmail.com>
Signed-off-by: Artur Souza <asouza.pro@gmail.com>
* do not ingore dlq errors Signed-off-by: Yevgen Polyak <yevgen.polyak@gmail.com> * do not send to DQL when shutting down Signed-off-by: Yevgen Polyak <yevgen.polyak@gmail.com> --------- Signed-off-by: Yevgen Polyak <yevgen.polyak@gmail.com>
Signed-off-by: Artur Souza <asouza.pro@gmail.com>
Signed-off-by: Elena Kolevska <elena@kolevska.com> Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
* Doesn’t skip update when a previous actor hosts unregisters all actors (workflows) Signed-off-by: Elena Kolevska <elena@kolevska.com> * upsert of hosts with no actors in fsm Signed-off-by: Elena Kolevska <elena@kolevska.com> * Adds unit test for “requiresUpdateInPlacementTables” Signed-off-by: Elena Kolevska <elena@kolevska.com> * Adds unit tests for “upsert required” Signed-off-by: Elena Kolevska <elena@kolevska.com> * small test update Signed-off-by: Elena Kolevska <elena@kolevska.com> * Adds more integration tests Signed-off-by: Elena Kolevska <elena@kolevska.com> * Updates tests Signed-off-by: Elena Kolevska <elena@kolevska.com> * fix Signed-off-by: Elena Kolevska <elena@kolevska.com> * Updates comment for clarity Signed-off-by: Elena Kolevska <elena@kolevska.com> --------- Signed-off-by: Elena Kolevska <elena@kolevska.com> Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Anton Troshin <anton@diagrid.io>
…l.ha (#8304) * allow for local dapr_scheduler.ha to be used to scale outside of global.ha Signed-off-by: Cassandra Coyle <cassie@diagrid.io> * update code comment local ha -> dapr_scheduler.ha for readability Signed-off-by: Cassandra Coyle <cassie@diagrid.io> --------- Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
#8301) * Increase graceful shutdown duration for state components shutdown test Signed-off-by: Anton Troshin <anton@diagrid.io> * register cleanup check before calling cleanup Signed-off-by: Anton Troshin <anton@diagrid.io> --------- Signed-off-by: Anton Troshin <anton@diagrid.io> Co-authored-by: Artur Souza <asouza.pro@gmail.com>
* Mark Workflow API Alpha and Beta as deprecated Signed-off-by: Anton Troshin <anton@diagrid.io> * remove deprecation from Workflow Beta APIs, still in use, cause lint error staticcheck SA1019 Signed-off-by: Anton Troshin <anton@diagrid.io> --------- Signed-off-by: Anton Troshin <anton@diagrid.io> Co-authored-by: Yaron Schneider <schneider.yaron@live.com> Co-authored-by: Elena Kolevska <elena-kolevska@users.noreply.github.com>
Signed-off-by: Fabian Martinez <46371672+famarting@users.noreply.github.com>
) * Dissalow running with ephemeral storage Signed-off-by: Elena Kolevska <elena@kolevska.com> * Updates helm value docs Signed-off-by: Elena Kolevska <elena@kolevska.com> --------- Signed-off-by: Elena Kolevska <elena@kolevska.com> Co-authored-by: Artur Souza <asouza.pro@gmail.com> Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
… to {{ .Release.Service }} (#8052)
Signed-off-by: Manuel Menegazzo <65919883+m3nax@users.noreply.github.com>
Co-authored-by: Josh van Leeuwen <me@joshvanl.dev>
Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
* further fine tuning Signed-off-by: Cassandra Coyle <cassie@diagrid.io> Signed-off-by: Jake Engelberg <jake@diagrid.io> * try fixing borked e2e Signed-off-by: Cassandra Coyle <cassie@diagrid.io> Signed-off-by: Jake Engelberg <jake@diagrid.io> * try fixing standalone validation Signed-off-by: Cassandra Coyle <cassie@diagrid.io> Signed-off-by: Jake Engelberg <jake@diagrid.io> * install numpy too Signed-off-by: Cassandra Coyle <cassie@diagrid.io> Signed-off-by: Jake Engelberg <jake@diagrid.io> * update pip & install pkgs globally instead of venv Signed-off-by: Cassandra Coyle <cassie@diagrid.io> Signed-off-by: Jake Engelberg <jake@diagrid.io> * install requests Signed-off-by: Cassandra Coyle <cassie@diagrid.io> Signed-off-by: Jake Engelberg <jake@diagrid.io> * // -> # for commented line Signed-off-by: Cassandra Coyle <cassie@diagrid.io> Signed-off-by: Jake Engelberg <jake@diagrid.io> * match only installed version of powershell Signed-off-by: Cassandra Coyle <cassie@diagrid.io> Signed-off-by: Jake Engelberg <jake@diagrid.io> * wip: errorcode const consolidation Signed-off-by: Jake Engelberg <jake@diagrid.io> * wip: error code monitoring/metric recording Signed-off-by: Jake Engelberg <jake@diagrid.io> * style: error_code -> errorcode Signed-off-by: Jake Engelberg <jake@diagrid.io> * fix: error changes from refactor Signed-off-by: Jake Engelberg <jake@diagrid.io> * remove 2 unused error codes Signed-off-by: Jake Engelberg <jake@diagrid.io> * Revert "wip: error code monitoring/metric recording" This reverts commit 53a02b5. Done so to implement in a separate PR. Signed-off-by: Jake Engelberg <jake@diagrid.io> * Revert "Revert "wip: error code monitoring/metric recording"" This reverts commit 44c31be. Signed-off-by: Jake Engelberg <jake@diagrid.io> * use RecordAndGet() to record error code metric only whenever ApiError var will be logged Signed-off-by: Jake Engelberg <jake@diagrid.io> * undo invalid error code recording Signed-off-by: Jake Engelberg <jake@diagrid.io> * apply cohesive prefixes to error codes Signed-off-by: Jake Engelberg <jake@diagrid.io> * correctly apply metric recording to inline string error definitions, fix unit tests Signed-off-by: Jake Engelberg <jake@diagrid.io> * add explicit "type" field for error code metrics Signed-off-by: Jake Engelberg <jake@diagrid.io> * linting improvements Signed-off-by: Jake Engelberg <jake@diagrid.io> * errorcode recording unit test, fix: register type field on init Signed-off-by: Jake Engelberg <jake@diagrid.io> * add category to error codes for O(1) operation on recording Signed-off-by: Jake Engelberg <jake@diagrid.io> * use error code vars in tests Signed-off-by: Jake Engelberg <jake@diagrid.io> * fumpt Signed-off-by: Jake Engelberg <jake@diagrid.io> * convert last test strings, fix some integration tests Signed-off-by: Jake Engelberg <jake@diagrid.io> * fix errorcodes compile error Signed-off-by: Jake Engelberg <jake@diagrid.io> * golangci-lint fixes Signed-off-by: Jake Engelberg <jake@diagrid.io> * lint fix Signed-off-by: Jake Engelberg <jake@diagrid.io> * remove debug binary Signed-off-by: Jake Engelberg <jake@diagrid.io> * Revert "lint fix" This reverts commit 392080a. Revert "golangci-lint fixes" This reverts commit d6c6066. Revert "fix errorcodes compile error" This reverts commit 9739534. Revert "convert last test strings, fix some integration tests" This reverts commit ae39228. Revert "fumpt" This reverts commit 7f04bb3. Revert "use error code vars in tests" This reverts commit c6ae704. remove addtl err code var in test Signed-off-by: Jake Engelberg <jake@diagrid.io> * revise "key" field in metric to "category" Signed-off-by: Jake Engelberg <jake@diagrid.io> * restore original error codes + bug fixes Signed-off-by: Jake Engelberg <jake@diagrid.io> * fix configuration test Signed-off-by: Jake Engelberg <jake@diagrid.io> fix copyright headers Signed-off-by: Jake Engelberg <jake@diagrid.io> utilize todo context w/o struct member Signed-off-by: Jake Engelberg <jake@diagrid.io> nelson style improvements Signed-off-by: Jake Engelberg <jake@diagrid.io> * fix: remove doubled calls of RecordAndGet() Signed-off-by: Jake Engelberg <jake@diagrid.io> * reduce repitition in pubsub.go by recording at parent build() Signed-off-by: Jake Engelberg <jake@diagrid.io> * add specific recorder logic for Jobs API composite err codes Signed-off-by: Jake Engelberg <jake@diagrid.io> * refactor metric RecordX() funcs to reduce repetition Signed-off-by: Jake Engelberg <jake@diagrid.io> * lint fix Signed-off-by: Jake Engelberg <jake@diagrid.io> * metric naming: count -> total Signed-off-by: Jake Engelberg <jake@diagrid.io> * fix: style/naming Signed-off-by: Jake Engelberg <jake@diagrid.io> * fix: correct crypto metric recording w/ wrapper funcs Signed-off-by: Jake Engelberg <jake@diagrid.io> * fix: remove extra metric recordings Signed-off-by: Jake Engelberg <jake@diagrid.io> * integtest: error code metrics Signed-off-by: Jake Engelberg <jake@diagrid.io> * fix duplicated/missing http recordings Signed-off-by: Jake Engelberg <jake@diagrid.io> * clarify/further consolidate recording funcs Signed-off-by: Jake Engelberg <jake@diagrid.io> * further reduction of recording, requires dapr/kit PR Signed-off-by: Jake Engelberg <jake@diagrid.io> * Add metric spec plural copy, remove some comments Signed-off-by: Jake Engelberg <jake@diagrid.io> * nit improvements Signed-off-by: Jake Engelberg <jake@diagrid.io> * provide newest dapr/kit to properly record error code Signed-off-by: Jake Engelberg <jake@diagrid.io> * remove dapr/kit replace, add log for malformed error code Signed-off-by: Jake Engelberg <jake@diagrid.io> * log: error -> warn Signed-off-by: Jake Engelberg <jake@diagrid.io> --------- Signed-off-by: Cassandra Coyle <cassie@diagrid.io> Signed-off-by: Jake Engelberg <jake@diagrid.io> Co-authored-by: Cassandra Coyle <cassie@diagrid.io> Co-authored-by: Elena Kolevska <elena-kolevska@users.noreply.github.com> Co-authored-by: Yaron Schneider <schneider.yaron@live.com> Co-authored-by: Artur Souza <asouza.pro@gmail.com>
panic: sync: WaitGroup is reused before previous Wait has returned Signed-off-by: Luis Rascao <luis.rascao@gmail.com>
Signed-off-by: Artur Souza <asouza.pro@gmail.com>
Stop nightly buids and remove longhaul update.
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.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )