Skip to content
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

[1.11.x] health: ensure /v1/health/service/:service endpoint returns the most recent results when a filter is used with streaming #12866

Merged
merged 1 commit into from
Apr 27, 2022

Conversation

rboyer
Copy link
Member

@rboyer rboyer commented Apr 27, 2022

Backport of #12640 to 1.11.x

…the most recent results when a filter is used with streaming

Backport of #12640 to 1.11.x
@rboyer rboyer requested review from eculver and a team April 27, 2022 15:52
@rboyer rboyer self-assigned this Apr 27, 2022
@rboyer rboyer added the pr/no-changelog PR does not need a corresponding .changelog entry label Apr 27, 2022
@github-actions github-actions bot added the theme/health-checks Health Check functionality label Apr 27, 2022
Copy link
Contributor

@eculver eculver left a comment

Choose a reason for hiding this comment

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

LGTM

@rboyer rboyer merged commit 637a054 into release/1.11.x Apr 27, 2022
@rboyer rboyer deleted the 1-11-fix-blocking-query-bug branch April 27, 2022 16:12
@hc-github-team-consul-core
Copy link
Collaborator

🍒 If backport labels were added before merging, cherry-picking will start automatically.

To retroactively trigger a backport after merging, add backport labels and re-run https://circleci.com/gh/hashicorp/consul/653720.

@hc-github-team-consul-core
Copy link
Collaborator

🍒❌ Cherry pick of commit 637a054 onto release/1.10.x failed! Build Log

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/no-changelog PR does not need a corresponding .changelog entry theme/health-checks Health Check functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants