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.

Sign up for GitHub

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

Jump to bottom

Status check error message improved. #2319

Merged
merged 1 commit into from
Feb 26, 2024
Merged

Status check error message improved. #2319

merged 1 commit into from
Feb 26, 2024

Conversation

viktor-kurchenko
Copy link
Contributor

@viktor-kurchenko viktor-kurchenko commented Feb 20, 2024 •
edited

Currently, cilium status might fail with the following output (which is not informative):

status check failed

New output example:

status check failed: [the server could not find the requested resource (get ciliumendpoints.cilium.io)]

@viktor-kurchenko
Status check error message improved.
cf38b52 
Signed-off-by: viktor-kurchenko <viktor.kurchenko@isovalent.com>
@viktor-kurchenko viktor-kurchenko requested a review from a team as a code owner February 20, 2024 09:05
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Feb 23, 2024
@michi-covalent michi-covalent merged commit 61d51ad into main Feb 26, 2024
13 checks passed
@michi-covalent michi-covalent deleted the pr/vk/status/error branch February 26, 2024 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@asauber asauber asauber approved these changes

Assignees
No one assigned
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@viktor-kurchenko @asauber @michi-covalent