Skip to content

Commit 0efbfef

Browse files
authored
fix(ReplicationBudgetManager): add HostTag to budget manager (#7459)
<!-- Describe what has changed in this PR --> **What changed?** Added HostTag to budget manager <!-- Tell your future self why have you made these changes --> **Why?** It improves visibility to budget manager usage since the budget manager is created by host <!-- How have you verified this change? Tested locally? Added a unit test? Checked in staging env? --> **How did you test it?** <!-- Assuming the worst case, what can be broken when deploying this change to production? --> **Potential risks** <!-- Is it notable for release? e.g. schema updates, configuration or data migration required? If so, please mention it, and also update CHANGELOG.md --> **Release notes** <!-- Is there any documentation updates should be made for config, https://cadenceworkflow.io/docs/operation-guide/setup/ ? If so, please open an PR in https://github.com/cadence-workflow/cadence-docs --> **Documentation Changes** Signed-off-by: fimanishi <[email protected]>
1 parent 5bf8efe commit 0efbfef

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

service/history/handler/handler.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,7 @@ func (h *handlerImpl) Start() {
143143
h.config.ReplicationBudgetManagerMaxSizeCount,
144144
cache.AdmissionOptimistic,
145145
0,
146-
h.GetMetricsClient().Scope(metrics.ReplicatorCacheManagerScope),
146+
h.GetMetricsClient().Scope(metrics.ReplicatorCacheManagerScope, metrics.HostTag(h.config.HostName)),
147147
h.GetLogger(),
148148
h.config.ReplicationBudgetManagerSoftCapThreshold,
149149
)

0 commit comments

Comments
 (0)