Skip to content

Migrate Bitnami Zookeeper and Kafka Docker Images#6210

Merged
KarstenSchnitter merged 1 commit into
opensearch-project:mainfrom
KarstenSchnitter:main
Oct 30, 2025
Merged

Migrate Bitnami Zookeeper and Kafka Docker Images#6210
KarstenSchnitter merged 1 commit into
opensearch-project:mainfrom
KarstenSchnitter:main

Conversation

@KarstenSchnitter

Copy link
Copy Markdown
Collaborator

Description

Broadcom discontinued support of non-secured Docker images used in the integration tests. They were moved to the bitnamilegacy repository. This PR changes the images accordingly.

Eventually the images should be replaced by supported images as there is no further updates on the images. The images were not changed to avoid conflicts that might arise using other images.

Issues Resolved

#6057

Check List

  • New functionality includes testing.
  • New functionality has a documentation issue. Please link to it in this PR.
    • New functionality has javadoc added
  • Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Broadcom discontinued support of non-secured Docker images
used in the integration tests. They were moved to the
bitnamilegacy repository. This PR changes the images accordingly.

Eventually the images should be replaced by supported images
as there is no further updates on the images. The images were not
changed to avoid conflicts that might arise using other images.

Signed-off-by: Karsten Schnitter <k.schnitter@sap.com>
@KarstenSchnitter

Copy link
Copy Markdown
Collaborator Author

The failing tests are still using the old docker-compose.yml files, not the ones provided with this change. Please advise, what I can do to change that.

@dlvenable

Copy link
Copy Markdown
Member

@KarstenSchnitter , I believe that these tests don't use the updated ones until after it is merged on main. So the only way to test is either merge to main or test on the main branch of a fork.

@KarstenSchnitter

Copy link
Copy Markdown
Collaborator Author

Let's merge and see, if it helps other PRs.

@KarstenSchnitter KarstenSchnitter merged commit bf1de9e into opensearch-project:main Oct 30, 2025
45 of 50 checks passed
@KarstenSchnitter

Copy link
Copy Markdown
Collaborator Author

Validated with #6211, that Kafka and Zookeeper Docker images can be pulled once more. Still I recommend to change the images to supported versions.

@dlvenable

Copy link
Copy Markdown
Member

Validated with #6211, that Kafka and Zookeeper Docker images can be pulled once more. Still I recommend to change the images to supported versions.

Thanks for checking that!

Yes, I agree, we should move to the recommended/supported versions. Is there a GitHub issue for that?

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.

2 participants