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

ST: Add system test for tiered-storage with Aiven plugin #10149

Merged
merged 8 commits into from
May 31, 2024

Conversation

Frawless
Copy link
Member

@Frawless Frawless commented May 23, 2024

Type of change

  • Enhancement / new feature

Description

This PR adds system test with tiered storage configuration for Kafka together with Aiven plugin. The tests deals with building an image with Aiven plugin inside using Kaniko or OpenShift build and also deploys Minio as S3 compatible storage.

Manually verified on OCP 4.15 and Minikube (azp setup script)

Packit changes are needed for removal the legacy jobs that are not exists anymore. It resulted into execution of the jobs that immediately failed due to missing config.

Checklist

  • Make sure all tests pass

Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
@Frawless Frawless added this to the 0.42.0 milestone May 23, 2024
Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
@Frawless
Copy link
Member Author

/packit test --labels components

@Frawless Frawless marked this pull request as ready for review May 23, 2024 12:56
@Frawless Frawless requested review from scholzj and a team May 23, 2024 12:57
Copy link
Member

@im-konge im-konge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I left some comments

Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
Copy link
Member

@see-quick see-quick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just two nits. Thanks for working on this! 👍 💯 Good Job!

@Frawless
Copy link
Member Author

/azp run regression

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Frawless
Copy link
Member Author

/packit test --labels components

@Frawless
Copy link
Member Author

TF failures are not connected to this PR. I will try to solve them in separate PR. TieredStorage test is green on AZP and on TF as well

Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
@Frawless Frawless merged commit 4179606 into strimzi:main May 31, 2024
13 checks passed
@Frawless Frawless deleted the tiered-storage-st branch May 31, 2024 07:01
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.

3 participants