Skip to content

Comments

[DNM] Investigate tls kuttl suite failures#826

Closed
vyzigold wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
vyzigold:investigate-nad-kuttl-issue
Closed

[DNM] Investigate tls kuttl suite failures#826
vyzigold wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
vyzigold:investigate-nad-kuttl-issue

Conversation

@vyzigold
Copy link
Contributor

@vyzigold vyzigold commented Jan 5, 2026

Seems like the tls kuttl-test suite often fails on the NAD metric-storage assertion. I'm trying to add further assertions to determine what exactly fails and why.

@openshift-ci openshift-ci bot requested review from frenzyfriday and stuggi January 5, 2026 10:57
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 5, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: vyzigold

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved label Jan 5, 2026
Seems like the tls kuttl-test suite often fails on the NAD
metric-storage assertion. I'm trying to add further assertions to
determine what exactly fails and why.
@vyzigold vyzigold force-pushed the investigate-nad-kuttl-issue branch from 05f2f9b to f54143f Compare January 5, 2026 13:58
@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 5, 2026

/retest

1 similar comment
@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 6, 2026

/retest

@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 6, 2026

/test telemetry-operator-build-deploy-kuttl

7 similar comments
@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 6, 2026

/test telemetry-operator-build-deploy-kuttl

@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 6, 2026

/test telemetry-operator-build-deploy-kuttl

@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 6, 2026

/test telemetry-operator-build-deploy-kuttl

@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 6, 2026

/test telemetry-operator-build-deploy-kuttl

@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 6, 2026

/test telemetry-operator-build-deploy-kuttl

@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 7, 2026

/test telemetry-operator-build-deploy-kuttl

@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 7, 2026

/test telemetry-operator-build-deploy-kuttl

@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 7, 2026

/test telemetry-operator-build-deploy

@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 8, 2026

/test telemetry-operator-build-deploy-kuttl

@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 8, 2026

/retest

@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 8, 2026

/test telemetry-operator-build-deploy-kuttl

@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 8, 2026

/retest

@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 8, 2026

/test images

@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 8, 2026

/test telemetry-operator-build-deploy-kuttl

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 8, 2026

@vyzigold: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/telemetry-operator-build-deploy-kuttl f54143f link true /test telemetry-operator-build-deploy-kuttl

Full PR test history. Your PR dashboard.

Details

Instructions 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. I understand the commands that are listed here.

@jlarriba
Copy link
Collaborator

jlarriba commented Jan 9, 2026

Maybe we should add some more timeout before assertion on the kuttl tests? Having to retest this number of times does not seem right, even if it will finally work out.

@vyzigold
Copy link
Contributor Author

vyzigold commented Jan 9, 2026

I was trying to trigger a specific failure I saw back on Monday an 3 separate PRs and try to get some additional information. But the tests actually worked surprisingly well, until it whole started failing to pull some image. If you look at the history https://prow.ci.openshift.org/pr-history/?org=openstack-k8s-operators&repo=telemetry-operator&pr=826 The first failure was because of my mistake. Then the test succeeded 6 times in a row, then they started hitting the image pull issue (which I guess is specific to this PR?). I was already thinking about just abandoning this effort, since the NAD issue didn't reappear. So I'm closing the PR.

@vyzigold vyzigold closed this Jan 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants