-
Notifications
You must be signed in to change notification settings - Fork 31
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
Add initial docs on k8s healthchecks #775
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Nigel Jones <[email protected]>
Signed-off-by: Nigel Jones <[email protected]>
…r status Signed-off-by: Nigel Jones <[email protected]>
Signed-off-by: Nigel Jones <[email protected]>
Hopefully this doc contribution is useful - if it looks ok, please consider merging :-) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Useful content - just need to correct calls away from deprecated REST API calls.
### Platform | ||
This checks if the *platform* is available. | ||
|
||
#### Platform is not running |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This call is deprecated. Should be https://44623abc-eu-gb.lb.appdomain.cloud:9443/open-metadata/platform-services/users/admin/server-platform/origin
|
||
#### Platform is not running | ||
➜ ~ curl -k -X GET --connect-timeout 5 --max-time 5 "https://44623abc-eu-gb.lb.appdomain.cloud:9443/open-metadata/admin-services/users/admin/server-origin" | ||
Egeria OMAG Server Platform (version 4.1-SNAPSHOT) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure how the platform returns a version if it is not running.
Egeria OMAG Server Platform (version 4.1-SNAPSHOT) | ||
|
||
#### Platform is running | ||
➜ ~ http --verify=no --pretty=format GET "https://44623abc-eu-gb.lb.appdomain.cloud:9443/open-metadata/admin-services/users/admin/server-origin" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Again - deprecated request
This API call was introduced in [release 3.4](/release-notes/3-4.md) | ||
|
||
#### Server is not known | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This call indicates whether the server is running or not - the call to determine whether the server is known on the platform is {baseURL}/open-metadata/platform-services/users/{user}/server-platform/servers/{serverName}/is-known
``` | ||
#### No permission for api call | ||
``` | ||
➜ ~ http --verify=no --pretty=format GET "https://44623abc-eu-gb.lb.appdomain.cloud:9443/open-metadata/admin-services/users/admin/servers/cocoMDS5/instance/status" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This call is deprecated - should use the platform services request {baseURL}/open-metadata/platform-services/users/{user}/server-platform/servers/{serverName}/instance/status
} | ||
``` | ||
### Server is available | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is also a simpler call to test if the server is active ... {baseURL}/open-metadata/platform-services/users/{user}/server-platform/servers/{serverName}/is-active
|
||
For example | ||
``` | ||
curl -k -o - -X GET --connect-timeout 5 --max-time 5 "https://44623abc-eu-gb.lb.appdomain.cloud:9443/open-metadata/admin-services/users/admin/servers/cocoMDS5/instance/status" | grep '.*\"relatedHTTPCode\": 200' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update for latest request
Thanks for the review comments. will try and propose an update |
Draft doc on api healthchecks