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

Added status API calls (leader, peers) and node service health checks, changed tag filtering #69

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EmilienR
Copy link

Hello,

I've added status API calls (https://developer.hashicorp.com/consul/api-docs/status) to check cluster status, changed the tag filtering method (use filter query parameter instead of tag, added filter variable in QueryOptions) and added node health check API calls (https://developer.hashicorp.com/consul/api-docs/health#list-checks-for-node).

Thanks,

Emilien

@pierresouchay
Copy link
Owner

Hello @EmilienR Can you rebase to avoid conflicts?

@EmilienR
Copy link
Author

EmilienR commented Jan 5, 2023

Hello @pierresouchay,
I'm sorry I don't understand what is wrong ? I'm already based on the last master commit
Thanks

@pierresouchay
Copy link
Owner

Huh... Sorry, it seems ok.
But in order to merge, we need first to fix format (see the github actions that are failing)

@EmilienR EmilienR force-pushed the dev-status-api branch 2 times, most recently from 2085039 to 25ad96f Compare January 8, 2023 16:31
@EmilienR
Copy link
Author

EmilienR commented Jan 8, 2023

Hello,
I think it's okay now, I added new test cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants