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

chore: Update Druid product version to 31.0.1 and adjust coordinator memory limit #685

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dervoeti
Copy link
Member

@dervoeti dervoeti commented Jan 29, 2025

Description

Part of stackabletech/docker-images#965

Also bumps OPA to 1.0.0.

I had to increase the memory limit of the coordinator and middlemanager, otherwise the authorization integration test fails because the Pods get OOMKilled (happens on my local machine and on Replicated).

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

Author

Preview Give feedback

Reviewer

Preview Give feedback

Acceptance

Preview Give feedback

@NickLarsenNZ NickLarsenNZ changed the title chore: Update Druid product version to 31.0.1 and adjust coordinator memory limit chore: Update Druid product version to 31.0.1, update OPA to 1.0.0, and adjust coordinator memory limit Jan 29, 2025
@NickLarsenNZ NickLarsenNZ changed the title chore: Update Druid product version to 31.0.1, update OPA to 1.0.0, and adjust coordinator memory limit chore: Update Druid product version to 31.0.1, update OPA to 1.0.0, and adjust coordinator memory limit Jan 29, 2025
@dervoeti dervoeti force-pushed the chore/update-druid-to-31.0.1 branch 2 times, most recently from 735c50a to 15698d8 Compare January 29, 2025 20:59
@@ -45,7 +45,7 @@ metadata:
name: psql-s3-druid
spec:
image:
productVersion: 30.0.0
productVersion: 31.0.1
Copy link
Member

Choose a reason for hiding this comment

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

Do we want to put the experimental version in the examples yet?

Copy link
Member Author

Choose a reason for hiding this comment

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

31 was accidentally marked as experimental, fixed.

# To use a custom image, add a comma and the full name after the product version
# - 28.0.1,docker.stackable.tech/sandbox/druid:28.0.1-stackable0.0.0-dev
- name: druid-latest
values:
- 30.0.0
- 31.0.1
Copy link
Member

Choose a reason for hiding this comment

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

Do we want to set the experimental as the latest, or keep that at the latest stable version?

Copy link
Member Author

Choose a reason for hiding this comment

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

31 was accidentally marked as experimental, fixed.

@NickLarsenNZ
Copy link
Member

Just noting that this is blocking stackabletech/docker-images#969

@dervoeti
Copy link
Member Author

dervoeti commented Feb 5, 2025

Just noting that this is blocking stackabletech/docker-images#969

Oh, right. And since this PR, or rather all Druid 31.0.1 PRs, are blocked by this decision and it might take a while, I removed the OPA bump from this PR (was not really necessary anyway, but I thought this PR would be a quick one). I'll open a separate PR for the OPA bump.

@dervoeti dervoeti changed the title chore: Update Druid product version to 31.0.1, update OPA to 1.0.0, and adjust coordinator memory limit chore: Update Druid product version to 31.0.1 and adjust coordinator memory limit Feb 5, 2025
@dervoeti dervoeti force-pushed the chore/update-druid-to-31.0.1 branch from f274b35 to 285aa14 Compare February 10, 2025 08:03
@dervoeti dervoeti force-pushed the chore/update-druid-to-31.0.1 branch from 285aa14 to 2daed10 Compare February 10, 2025 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Development: In Review
Development

Successfully merging this pull request may close these issues.

3 participants