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

docs: deprecate some old filter parameters #9262

Merged
merged 1 commit into from Dec 15, 2020

Conversation

dnephin
Copy link
Contributor

@dnephin dnephin commented Nov 23, 2020

There are a few HTTP API endpoints under Health and Catalog that have query parameters for filtering results on specific fields. This filtering can now be done with the filter query parameter, so we should deprecate the older parameters and remove them in a future version of Consul.

/health/state/:state has a node-meta query parameter, but that one can't be replaced by filter because the response doesn't include the NodeMeta. The current implementation queries additional tables to get the metadata required for filtering.

@dnephin dnephin added type/docs Documentation needs to be created/updated/clarified type/docs-cherrypick labels Nov 23, 2020
@dnephin dnephin force-pushed the dnephin/docs-deprecate-old-filters branch from 62d87e5 to ce3fe97 Compare November 23, 2020 22:53
@dnephin dnephin changed the title WIP: docs: deprecate some old filter parameters docs: deprecate some old filter parameters Nov 23, 2020
@dnephin dnephin requested review from kyhavlov and a team November 23, 2020 22:54
@dnephin dnephin marked this pull request as ready for review November 23, 2020 22:54
@dnephin dnephin force-pushed the dnephin/docs-deprecate-old-filters branch from ce3fe97 to 597c848 Compare November 23, 2020 23:00
The filtering can be done with the general purpose `filter` query parameter.
@dnephin dnephin merged commit c5b74fc into master Dec 15, 2020
@dnephin dnephin deleted the dnephin/docs-deprecate-old-filters branch December 15, 2020 22:11
@hashicorp-ci
Copy link
Contributor

🍒 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/299035.

@hashicorp-ci
Copy link
Contributor

🍒✅ Cherry pick of commit c5b74fc onto stable-website succeeded!

hashicorp-ci pushed a commit that referenced this pull request Dec 15, 2020
…lters

docs: deprecate some old filter parameters
@hashicorp-ci
Copy link
Contributor

🍒✅ Cherry pick of commit c5b74fc onto release/1.9.x succeeded!

hashicorp-ci pushed a commit that referenced this pull request Dec 15, 2020
…lters

docs: deprecate some old filter parameters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/docs Documentation needs to be created/updated/clarified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants