Skip to content

[STREAM-1661] Expose current alerting leader status rather than a snapshot of the value#357

Merged
hjon merged 1 commit into
developfrom
STREAM-1661
May 28, 2026
Merged

[STREAM-1661] Expose current alerting leader status rather than a snapshot of the value#357
hjon merged 1 commit into
developfrom
STREAM-1661

Conversation

@hjon
Copy link
Copy Markdown
Collaborator

@hjon hjon commented May 28, 2026

No description provided.

@codacy-production
Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 4 duplication

Metric Results
Duplication 4

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

Comment thread src/alerting-leader.ts
off: this.off.bind(this),
claimAlertingLeader: this.claimAlertingLeader.bind(this),
getLeaderStatus: () => { return this.leaderStatus; },
leaderStatus: this.leaderStatus
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have no good explanation for why I thought this would use an up-to-date value. But our clients weren't actually using this before. They need it now.

Copy link
Copy Markdown
Collaborator

@patrickfeeney03 patrickfeeney03 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@hjon hjon merged commit 0c0e53c into develop May 28, 2026
5 checks passed
@hjon hjon deleted the STREAM-1661 branch May 28, 2026 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants