Skip to content

update the curl k8s api health check code#5010

Merged
laverya merged 1 commit into
mainfrom
laverya/update-curl-health-check
Dec 21, 2023
Merged

update the curl k8s api health check code#5010
laverya merged 1 commit into
mainfrom
laverya/update-curl-health-check

Conversation

@laverya
Copy link
Copy Markdown
Contributor

@laverya laverya commented Dec 21, 2023

What this PR does / why we need it:

laverya@laverya-ubuntu18:~$ curl -k https://34.132.114.199:6443/healthz && echo "curl worked" || echo "curl failed"
curl: (16) Error in the HTTP2 framing layer
okcurl failed

the curl returns 'ok', and yet it fails because the curl also has an error.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

Does this PR require documentation?

@laverya laverya requested a review from a team as a code owner December 21, 2023 00:27
@laverya laverya added type::feature An enhancement to an existing add on or feature type::chore and removed type::feature An enhancement to an existing add on or feature labels Dec 21, 2023
@laverya laverya merged commit a0fc37f into main Dec 21, 2023
@laverya laverya deleted the laverya/update-curl-health-check branch December 21, 2023 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@ricardomaraschini ricardomaraschini ricardomaraschini approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

type::chore

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@laverya @ricardomaraschini