-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: Add Hive 4.0.0 #818
feat: Add Hive 4.0.0 #818
Conversation
In terms of patches: 07-patch-updates.patch => Later :) |
Built the image locally and used operator stackabletech/hive-operator#508 . Integrationtests all work fine. I tried the e2e demo which also worked (data in superset etc), but the Hive pod throws connection errors continuously like
Full logs hive.log Edit: K8s healtch check (kerberos)... |
Awesome thanks! Just for the record: This is a know problem of the k8s healthcheck not using Kerberos |
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!. Integration tests and e2e demo works. We will merge as is(?) and probably mark this as experimental first, check the CVE scans for the new image and eventually to dependency bumps?
Yes! :) |
Yeah that should have been "as discussed" and not a question :D |
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!
Description
Part of stackabletech/issues#620
During this is noticed a minor problem with
schemaTool.sh
and created apache/hive#5419 (already merged :) )We did not bump misc dependencies as Hive 3.1.3 is doing (see https://stackable-workspace.slack.com/archives/C031A5BEFS7/p1725275085248519 for details) to keep this within the defined time box. We can always do that later on based on a CVE scan.
Definition of Done Checklist
Tasks
TIP: Running integration tests with a new product image
The image can be built and uploaded to the kind cluster with the following commands:
See the output of
bake
to retrieve the image tag for<image-tagged-with-the-major-version>
.