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

Added more failure handlers to basic tests #495

Merged
merged 1 commit into from
Oct 8, 2024
Merged

Added more failure handlers to basic tests #495

merged 1 commit into from
Oct 8, 2024

Conversation

AverageMarcus
Copy link
Member

What this PR does

More debugging if tests fail.

Checklist

  • Update changelog in CHANGELOG.md.

Trigger e2e tests

/run cluster-test-suites

@AverageMarcus AverageMarcus requested a review from a team October 8, 2024 06:26
@AverageMarcus AverageMarcus requested a review from a team as a code owner October 8, 2024 06:26
@tinkerers-ci
Copy link

tinkerers-ci bot commented Oct 8, 2024

cluster-test-suites

Run name pr-cluster-test-suites-495-cluster-test-suitess2xsw
Commit SHA a067dfd
Result Failed ❌

📋 View full results in Tekton Dashboard

Rerun trigger:
/run cluster-test-suites

Tip

To only re-run the failed test suites you can provide a TARGET_SUITES parameter with your trigger that points to the directory path of the test suites to run, e.g. /run cluster-test-suites TARGET_SUITES=./providers/capa/standard to re-run the CAPA standard test suite. This supports multiple test suites with each path separated by a comma.

@AverageMarcus AverageMarcus force-pushed the extra-failure-handlers branch from a067dfd to c88a890 Compare October 8, 2024 06:55
@AverageMarcus
Copy link
Member Author

/run cluster-test-suites

@AverageMarcus AverageMarcus force-pushed the extra-failure-handlers branch from c88a890 to 202ebb4 Compare October 8, 2024 07:02
@AverageMarcus
Copy link
Member Author

/run cluster-test-suites

@tinkerers-ci
Copy link

tinkerers-ci bot commented Oct 8, 2024

cluster-test-suites

Run name pr-cluster-test-suites-495-cluster-test-suiteshrzzf
Commit SHA c88a890
Result Failed ❌

📋 View full results in Tekton Dashboard

Rerun trigger:
/run cluster-test-suites

Tip

To only re-run the failed test suites you can provide a TARGET_SUITES parameter with your trigger that points to the directory path of the test suites to run, e.g. /run cluster-test-suites TARGET_SUITES=./providers/capa/standard to re-run the CAPA standard test suite. This supports multiple test suites with each path separated by a comma.

@tinkerers-ci
Copy link

tinkerers-ci bot commented Oct 8, 2024

cluster-test-suites

Run name pr-cluster-test-suites-495-cluster-test-suites4j8nc
Commit SHA 202ebb4
Result Failed ❌

📋 View full results in Tekton Dashboard

Rerun trigger:
/run cluster-test-suites

Tip

To only re-run the failed test suites you can provide a TARGET_SUITES parameter with your trigger that points to the directory path of the test suites to run, e.g. /run cluster-test-suites TARGET_SUITES=./providers/capa/standard to re-run the CAPA standard test suite. This supports multiple test suites with each path separated by a comma.

@AverageMarcus
Copy link
Member Author

CAPVCD failure known flake, adding skip label

@AverageMarcus AverageMarcus added the skip/ci Instructs pr-gatekeeper to ignore any required PR checks label Oct 8, 2024
@AverageMarcus AverageMarcus merged commit 0f37dff into main Oct 8, 2024
3 of 4 checks passed
@AverageMarcus AverageMarcus deleted the extra-failure-handlers branch October 8, 2024 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@njuettner njuettner njuettner approved these changes

@fiunchinho fiunchinho fiunchinho approved these changes

Assignees
No one assigned
Labels
skip/ci Instructs pr-gatekeeper to ignore any required PR checks
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AverageMarcus @fiunchinho @njuettner