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

Bump strimzi kafka testcontainer to 0.107.0 #1207

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

jedla97
Copy link
Member

@jedla97 jedla97 commented Jul 18, 2024

Summary

Fixing #1186

This is the draft to check if the new strimzi fix work. Will change this after they release full version.

Please check the relevant options

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Dependency update
  • Refactoring
  • Release (follows conventions described in the RELEASE.md)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • This change requires a documentation update
  • This change requires execution against OCP (use run tests phrase in comment)

Checklist:

  • Example scenarios has been updated / added
  • Methods and classes used in PR scenarios are meaningful
  • Commits are well encapsulated and follow the best practices

@jedla97 jedla97 changed the title Bump strimzi kafka testcontainer to 0.107.0-rc1 [DON'T MERGE] Bump strimzi kafka testcontainer to 0.107.0-rc1 Jul 18, 2024
@jedla97
Copy link
Member Author

jedla97 commented Jul 19, 2024

Run CI 3 times and kafka error not appear any time so it should be fixed. Now waiting for 0.107.0 release which should be at the start of the next week

@jedla97
Copy link
Member Author

jedla97 commented Jul 24, 2024

run tests

@jedla97 jedla97 changed the title [DON'T MERGE] Bump strimzi kafka testcontainer to 0.107.0-rc1 Bump strimzi kafka testcontainer to 0.107.0 Jul 24, 2024
@jedla97 jedla97 marked this pull request as ready for review July 24, 2024 11:49
@jedla97
Copy link
Member Author

jedla97 commented Jul 24, 2024

Openshift native failure is not related to this bump

@jedla97 jedla97 requested a review from fedinskiy July 24, 2024 14:11
@jedla97
Copy link
Member Author

jedla97 commented Jul 24, 2024

Maybe this should be also backported?

@rsvoboda
Copy link
Member

I don't think we have Kafka with KRaft in 3.8 branch. @fedinskiy can you confirm please?

@jedla97
Copy link
Member Author

jedla97 commented Jul 24, 2024

@rsvoboda Looking at PRs and it was probably backported 3.8 #1190 and 3.2 #1189

@fedinskiy
Copy link
Contributor

@rsvoboda we have Kafka with Kraft in all 3.x branches.

@jedla97 could you create backport PRs as well? We also should update the TS to 0.107.0, since the upstream Quarkus brings in an older version

@fedinskiy fedinskiy added triage/backport-1.3? Backport to 1.3 branch may be required triage/backport-1.4? Quarkus 3.8 stream labels Jul 24, 2024
@fedinskiy fedinskiy merged commit edb221e into quarkus-qe:main Jul 24, 2024
9 of 10 checks passed
@gtroitsk
Copy link
Member

gtroitsk commented Sep 9, 2024

I remove backport labels as changes were already backported

@gtroitsk gtroitsk removed triage/backport-1.3? Backport to 1.3 branch may be required triage/backport-1.4? Quarkus 3.8 stream labels Sep 9, 2024
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.

4 participants