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

Reduce CI duration by modifying e2e test execution strategy #6416

Closed
wants to merge 1 commit into from

Conversation

zzzk1
Copy link
Contributor

@zzzk1 zzzk1 commented Dec 26, 2024

Which problem is this PR solving?

Description of the changes

  • old version of the backend storage integration tests runs once daily.

How was this change tested?

  • integration tests

Checklist

@zzzk1 zzzk1 requested a review from a team as a code owner December 26, 2024 15:08
@zzzk1 zzzk1 requested a review from jkowall December 26, 2024 15:08
@dosubot dosubot bot added the area/storage label Dec 26, 2024
@yurishkuro
Copy link
Member

I don't understand this PR.

@zzzk1
Copy link
Contributor Author

zzzk1 commented Dec 26, 2024

I don't understand this PR.

I wish the other version tests would run on the nightly build, but I don't know how to conduct the tests.

@zzzk1 zzzk1 force-pushed the reduce-ci-e2e-test-time branch from afe25c2 to 00464f1 Compare December 26, 2024 16:18
@zzzk1
Copy link
Contributor Author

zzzk1 commented Dec 26, 2024

@yurishkuro I apologize for the confusion. I am new to writing pipeline scripts, so could you please provide me with more context? thanks 😊

@yurishkuro
Copy link
Member

This is not solving any useful problem. We do want all e2e tests to run on PRs. But we don't want to run tests against Kafka v2 because it's EOL.

@yurishkuro
Copy link
Member

If you don't understand the problem described in the ticket, the best course is to ask clarifying questions, not trying to implement random changes and asking "is this it?"

@zzzk1 zzzk1 closed this Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document and update integration test with the versions of Kafka Jaeger supports
2 participants