Skip to content

Record metric for service updates #5114

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

Merged
merged 2 commits into from
Sep 2, 2022

Conversation

sunzhaochang
Copy link
Contributor

Signed-off-by: sunzhaochang zhchsun1992@gmail.com

Description

Record metric for service updates

Issue reference

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:

Signed-off-by: sunzhaochang <zhchsun1992@gmail.com>
@sunzhaochang sunzhaochang requested review from a team as code owners September 1, 2022 13:00
@codecov
Copy link

codecov bot commented Sep 1, 2022

Codecov Report

Merging #5114 (3d85378) into master (2b04241) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #5114   +/-   ##
=======================================
  Coverage   65.42%   65.42%           
=======================================
  Files         114      114           
  Lines       12762    12763    +1     
=======================================
+ Hits         8349     8350    +1     
  Misses       3826     3826           
  Partials      587      587           
Impacted Files Coverage Δ
pkg/operator/handlers/dapr_handler.go 69.87% <100.00%> (+0.19%) ⬆️

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

@artursouza artursouza added this to the v1.9 milestone Sep 2, 2022
@artursouza artursouza added the automerge Allows DaprBot to automerge and update PR if all approvals are in place label Sep 2, 2022
@dapr-bot dapr-bot merged commit bb3df85 into dapr:master Sep 2, 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.

3 participants