-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[ST] Update upgrade/downgrade files after 0.39.0 release and enable KRaftStrimziDowngradeST #9506
Conversation
/azp run upgrade |
Azure Pipelines successfully started running 1 pipeline(s). |
systemtest/src/test/java/io/strimzi/systemtest/upgrade/AbstractUpgradeST.java
Show resolved
Hide resolved
systemtest/src/test/java/io/strimzi/systemtest/upgrade/kraft/AbstractKRaftUpgradeST.java
Show resolved
Hide resolved
systemtest/src/test/java/io/strimzi/systemtest/upgrade/regular/StrimziDowngradeST.java
Outdated
Show resolved
Hide resolved
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.
LGTM !
systemtest/src/main/java/io/strimzi/systemtest/utils/StUtils.java
Outdated
Show resolved
Hide resolved
/azp run upgrade |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run upgrade |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run upgrade |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run upgrade |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run upgrade |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run upgrade |
Azure Pipelines successfully started running 1 pipeline(s). |
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.
Good job! 👍 very nice and clean PR :))
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.
lgtm
…ngradeST Signed-off-by: Lukas Kral <lukywill16@gmail.com> test fixups Signed-off-by: Lukas Kral <lukywill16@gmail.com> checkstyle Signed-off-by: Lukas Kral <lukywill16@gmail.com> remove the random part Signed-off-by: Lukas Kral <lukywill16@gmail.com> fixups to tests, Henrich's comment, common method for removing KafkaTopics + some helper methods Signed-off-by: Lukas Kral <lukywill16@gmail.com> different approach in isUnidirectionalTopicOperatorUsed Signed-off-by: Lukas Kral <lukywill16@gmail.com> remove building of connect image for upgrade, correct check for expected kafkatopics Signed-off-by: Lukas Kral <lukywill16@gmail.com> empty connectImage parameter in azp and specify namespace for continuous clients Signed-off-by: Lukas Kral <lukywill16@gmail.com> remove connectImage from parameters and add it to default_variables.yaml Signed-off-by: Lukas Kral <lukywill16@gmail.com>
/azp run upgrade |
Azure Pipelines successfully started running 1 pipeline(s). |
Type of change
Description
This PR updates upgrade/downgrade files after the 0.39.0 release. It also enables
KRaftStrimziDowngradeST
, which was blocked until new version of Strimzi, which supports KRaft to KRaft upgrade/(safe) downgrade is not released.Checklist