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

Health check for OCI ObjectStorage that works in SE and MP #3157

Merged
merged 11 commits into from
Jul 2, 2021

Conversation

spericas
Copy link
Member

This is WIP.

Liveness health check for OCI ObjectStorage that works in SE and MP, like our other built-in health checks. Verifies that a bucket is accessible (HTTP GET) using the OCI client API. In addition to the user-specific OCI credentials, it needs the namespace and bucket name. MP and SE examples have been updated to use the new health check.

Current output for health check is shown below: `GET http://.../health/live

{
  "outcome": "UP",
  "status": "UP",
  "checks": [
    {
      "name": "objectStorage",
      "state": "UP",
      "status": "UP",
      "data": {
        "bucket": "main",
        "namespace": "idv5cxr1uqqk"
      }
    }
  ]
}

Signed-off-by: Santiago Pericasgeertsen santiago.pericasgeertsen@oracle.com

@spericas spericas self-assigned this Jun 25, 2021
@spericas spericas changed the title WIP: Health check for OCI ObjectStorage that works in SE and MP Health check for OCI ObjectStorage that works in SE and MP Jun 29, 2021
…her built-in health checks. Verifies that a bucket is accessible (HTTP GET) using the OCI client API. In addition to the user-specific OCI credentials, it needs the namespace and bucket name. MP and SE examples have been updated to use the new health check.

Signed-off-by: Santiago Pericasgeertsen <santiago.pericasgeertsen@oracle.com>
Signed-off-by: Santiago Pericasgeertsen <santiago.pericasgeertsen@oracle.com>
… and builder pattern based on Helidon conventions. Updated ObjectStore API with new request and responses for GetBucket. Updated samples for SE and MP.

Signed-off-by: Santiago Pericasgeertsen <santiago.pericasgeertsen@oracle.com>
Signed-off-by: Santiago Pericasgeertsen <santiago.pericasgeertsen@oracle.com>
Signed-off-by: Santiago Pericasgeertsen <santiago.pericasgeertsen@oracle.com>
Signed-off-by: Santiago Pericasgeertsen <santiago.pericasgeertsen@oracle.com>
Signed-off-by: Santiago Pericasgeertsen <santiago.pericasgeertsen@oracle.com>
spericas and others added 3 commits July 1, 2021 13:01
Signed-off-by: Santiago Pericasgeertsen <santiago.pericasgeertsen@oracle.com>
Signed-off-by: Tomas Langer <tomas.langer@oracle.com>
@spericas spericas merged commit fda7ba9 into helidon-io:master Jul 2, 2021
@spericas spericas mentioned this pull request Jul 21, 2021
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants