-
Notifications
You must be signed in to change notification settings - Fork 34
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
[3.8] Re-enable tests on IBM platforms due to removal of zookeeper #2030
Conversation
Will need to backport into main and most likely 3.15 branch afterwards. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dale-fu please address formatting issues
let's try again @dale-fu |
I'm so confused, all I did was remove DisabledIfSystemProperty(), I don't even think I touched the imports. Looking at the log, it seems like the imports are already alphabetical on OperatorOpenShiftAmqStreamsKafkaStreamIT.java. |
You have removed |
I'm an idiot, I was looking at it the wrong way. Clearly nothing to do with alphabetical ordering. |
No worry, validation exception is not good, but that's simply how it is. |
Summary
Due to #1854, we can re-enable the tests that were using the zookeeper image on IBM platforms. The previously skipped tests passing on latest 3.8 release.
Example test runs:
Please select the relevant options.
run tests
phrase in comment)Checklist: