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

E2E tests stabilizations #1921

Merged
merged 1 commit into from
Dec 7, 2023
Merged

E2E tests stabilizations #1921

merged 1 commit into from
Dec 7, 2023

Conversation

Crumby
Copy link
Collaborator

@Crumby Crumby commented Dec 6, 2023

Fixes several E2E tests to reduce noise in the test results:

  • TestJmxEnabled
    • Skip the test if the test is executed against Infinispan 13
  • TestUpdatePodTargetLabels
    • The test removed the 'testname' labels used to clean up resources after the execution
  • BackupRestoreTests
    • The test occasionally fails with ISPN-15173 when running against older Infinispan versions. Given the issue won't be fixed, just skip the test when the error with this message pops up
  • TestGracefulShutdownWithTwoReplicas
    • It's possible for the test to fail the PUT request with EOF right after the cluster was brought back up. The issue randomly appears only when the test is executed as part of the whole testsuite. Debugging showed up that in the occasions the EOF happens the request does not reach the cluster which suggests that there is some race condition between routing network in the OpenShift and executing the request. Fixed by retrying the request.

@ryanemerson
Copy link
Contributor

@Crumby Can you rebase on the latest main so that the CI agents can build, thanks.

@Crumby Crumby added the Preview WIP. Don't merge yet label Dec 6, 2023
@Crumby Crumby removed the Preview WIP. Don't merge yet label Dec 7, 2023
@ryanemerson ryanemerson merged commit 4f4497c into infinispan:main Dec 7, 2023
@ryanemerson
Copy link
Contributor

Thanks @Crumby

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants