Skip to content

Fix integration tests - Copy jmx broker config to builder image#703

Merged
maltesander merged 3 commits intomainfrom
fix/copy-jmx-config-to-builder
Jun 6, 2024
Merged

Fix integration tests - Copy jmx broker config to builder image#703
maltesander merged 3 commits intomainfrom
fix/copy-jmx-config-to-builder

Conversation

@maltesander
Copy link
Copy Markdown
Member

@maltesander maltesander commented Jun 5, 2024

Description

This adds the jmx broker.config to the kafka builder image and adds a softlink for the kcat binary to the location the operator uses.

Kafka container did not come up due to:

kafka Caused by: java.io.FileNotFoundException: /stackable/jmx/broker.yaml (No such file or directory)

Kcat sidecar did not get ready due to wrong bin path, and did not output any logs...

Definition of Done Checklist

  • Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
  • Please make sure all these things are done and tick the boxes
- [ ] Changes are OpenShift compatible
- [ ] All added packages (via microdnf or otherwise) have a comment on why they are added
- [ ] Things not downloaded from Red Hat repositories should be mirrored in the Stackable repository and downloaded from there
- [ ] All packages should have (if available) signatures/hashes verified
- [x] Add an entry to the CHANGELOG.md file
- [x] Integration tests ran successfully
TIP: Running integration tests with a new product image

The image can be built and uploaded to the kind cluster with the following commands:

bake --product <product> --image-version <stackable-image-version>
kind load docker-image <image-tagged-with-the-major-version> --name=<name-of-your-test-cluster>

See the output of bake to retrieve the image tag for <image-tagged-with-the-major-version>.

@maltesander maltesander requested a review from a team June 5, 2024 07:32
@maltesander maltesander self-assigned this Jun 5, 2024
Comment thread kafka/Dockerfile
@maltesander maltesander marked this pull request as ready for review June 5, 2024 09:20
@maltesander maltesander enabled auto-merge June 6, 2024 07:36
@maltesander maltesander added this pull request to the merge queue Jun 6, 2024
Merged via the queue into main with commit f9a531c Jun 6, 2024
@maltesander maltesander deleted the fix/copy-jmx-config-to-builder branch June 6, 2024 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants