Add kuttl tests for lightspeed-app-server /metrics endpoint#22
Add kuttl tests for lightspeed-app-server /metrics endpoint#22jancervenka wants to merge 1 commit into
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: jancervenka The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Hi @jancervenka. Thanks for your PR. I'm waiting for a openstack-k8s-operators member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
jancervenka
force-pushed
the
kuttl-tests-for-metrics-endpoint
branch
from
b371d7d to
4fef44e
Compare
The kuttl test asserts that the endpoint is reachable and responds with a valid Prometheus format.
jancervenka
force-pushed
the
kuttl-tests-for-metrics-endpoint
branch
from
4fef44e to
3c85e10
Compare
|
/ok-to-test |
| - kind: ServiceAccount | ||
| name: ${SA_NAME} | ||
| namespace: ${NAMESPACE} | ||
| EOF |
There was a problem hiding this comment.
These ClusterRole and ClusterRoleBinding are beeing created but I don't see them being cleaned up at the end, so they will stay in the cluster as leftovers artifacts.
Also, I think we can create them in a declarative way ? Instead of inside the TestAssert
| echo "Response (first 500 chars):" | ||
| echo "$METRICS_OUTPUT" | head -c 500 | ||
| exit 1 | ||
| fi |
There was a problem hiding this comment.
I would to hear @lpiwowar and/or @malingatembo opinion on using kuttl tests this ^
It seems like it fits more into the new test framework than kuttl. Because this is asserting some behavior from our API endpoint rather than just asserting the state of the cluster (which I believe kuttl is better designed for).
| echo "$METRICS_OUTPUT" | head -c 500 | ||
| exit 1 | ||
| fi | ||
There was a problem hiding this comment.
Also, if we are OK with the kuttl tests running these type of tests, I think we should also check for at least one lightspeed_ specific metric here ? Making sure it's the right endpoint and not just any other prometheus endpoint
2 participants
The kuttl test asserts that the endpoint is reachable and responds with a valid Prometheus format.