Skip to content

Conversation

ItalyPaleAle
Copy link
Contributor

Spin-off from #4903

It contains misc improvements to E2E test apps, including:

@ItalyPaleAle ItalyPaleAle requested review from a team as code owners August 19, 2022 00:42
@codecov
Copy link

codecov bot commented Aug 19, 2022

Codecov Report

Merging #5042 (61c74e9) into master (996cc62) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5042   +/-   ##
=======================================
  Coverage   65.45%   65.45%           
=======================================
  Files         113      113           
  Lines       12723    12723           
=======================================
  Hits         8328     8328           
  Misses       3811     3811           
  Partials      584      584           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Spin-off from dapr#4903

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
@ItalyPaleAle
Copy link
Contributor Author

/ok-to-test

@dapr-bot
Copy link
Collaborator

Congrats! All end-to-end tests have passed on linux. Thanks for your contribution!

@dapr-bot
Copy link
Collaborator

Congrats! All end-to-end tests have passed on windows. Thanks for your contribution!

@ItalyPaleAle
Copy link
Contributor Author

/ok-to-test

@dapr-bot
Copy link
Collaborator

Congrats! All end-to-end tests have passed on linux. Thanks for your contribution!

@dapr-bot
Copy link
Collaborator

End-to-end tests failed on windows. Please check the build logs

artursouza and others added 3 commits August 22, 2022 13:26
artursouza
artursouza previously approved these changes Aug 23, 2022
@artursouza artursouza added the automerge Allows DaprBot to automerge and update PR if all approvals are in place label Aug 23, 2022
dapr-bot and others added 3 commits August 22, 2022 22:58
Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
@artursouza
Copy link
Contributor

/ok-to-test

1 similar comment
@artursouza
Copy link
Contributor

/ok-to-test

@dapr-bot
Copy link
Collaborator

Congrats! All end-to-end tests have passed on linux. Thanks for your contribution!

@dapr-bot
Copy link
Collaborator

Congrats! All end-to-end tests have passed on linux. Thanks for your contribution!

@dapr-bot
Copy link
Collaborator

Congrats! All end-to-end tests have passed on windows. Thanks for your contribution!

@dapr-bot
Copy link
Collaborator

End-to-end tests failed on windows. Please check the build logs

@artursouza artursouza merged commit 8f2f1fb into dapr:master Aug 29, 2022
Taction added a commit to Taction/dapr that referenced this pull request Sep 5, 2022
* master: (134 commits)
  Record metric for service updates (dapr#5114)
  chore: remove duplicate word in comments (dapr#5099)
  Run e2e tests in previous version of K8s on AKS to see if it's breaking Windows tests (dapr#5104)
  add {namespace} option to set consumerID
  Add tracing passthrough for bindings (dapr#5049)
  Improve component init error logs and metrics (dapr#4975)
  Improvements to E2E test apps (dapr#5042)
  Fix component name for aws dynamodb (dapr#5095)
  Fix perf tests not building (dapr#5096)
  Fix missing components in Metadata API result. (dapr#5052)
  Fixed log message (dapr#5088)
  Disable IPFS binding for now (dapr#5092)
  optimize/concurrency: pubsub concurrency map writes in runtime unit test. (dapr#5086)
  Configuration API response to use dictionaries - based on PR dapr#4695 (dapr#5083)
  correct compile error
  correct checking for actors with same id
  fix type error in print
  fix typo
  add log when actor start time and end time is not expected
  Components auto-registration (dapr#5062)
  ...

Signed-off-by: zhangchao <zchao9100@gmail.com>

# Conflicts:
#	go.mod
#	pkg/grpc/api.go
#	pkg/grpc/api_test.go
#	pkg/http/api.go
#	pkg/runtime/runtime.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Allows DaprBot to automerge and update PR if all approvals are in place
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants