Skip to content

fix(hive): Fix TLS problems by switching back to Java 11#622

Merged
sbernauer merged 2 commits intomainfrom
fix/hive
Apr 16, 2024
Merged

fix(hive): Fix TLS problems by switching back to Java 11#622
sbernauer merged 2 commits intomainfrom
fix/hive

Conversation

@sbernauer
Copy link
Copy Markdown
Member

@sbernauer sbernauer commented Apr 15, 2024

Description

Should fix broken tests introduced in #581

To sum this up for the suits:

  • We used to ship the pre-built Hive binary in a container with java 11 runtime and everything worked (this was presumably built on java 8 )
  • After building ourselves from source (with java 8 ) we switched the java runtime in the container from 11 to 8 and some weird tls stuff broker
  • Switching back to 11 in the container fixes it

The log of the failing TLS access to MinIO is attached

hive-metastore-default-0.log

See https://stackable-workspace.slack.com/archives/C031A5BEFS7/p1713185172557459 for details

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
- [x] Changes are OpenShift compatible
- [x] All added packages (via microdnf or otherwise) have a comment on why they are added
- [x] Things not downloaded from Red Hat repositories should be mirrored in the Stackable repository and downloaded from there
- [x] All packages should have (if available) signatures/hashes verified
- [ ] Does your change affect an SBOM? Make sure to update all SBOMs
- [x] Add an entry to the CHANGELOG.md file
- [ ] 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>.

@sbernauer sbernauer self-assigned this Apr 15, 2024
Copy link
Copy Markdown
Member

@Maleware Maleware left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks again!

@sbernauer sbernauer added this pull request to the merge queue Apr 16, 2024
Merged via the queue into main with commit b27df0e Apr 16, 2024
@sbernauer sbernauer deleted the fix/hive branch April 16, 2024 10:08
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