-
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
fix(hbase): remove dependency on htrace #820
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Nick <[email protected]>
Co-authored-by: Nick <[email protected]>
Re: the hadolint failure, I stumbled across this today: https://www.docker.com/blog/introduction-to-heredocs-in-dockerfiles/ Maybe we need to update the syntax for the Dockerfile for hadolint to accept the heredoc? Although, there is already an issue for hadolint: hadolint/hadolint#806 |
Converted to draft again, since:
|
@@ -36,7 +36,7 @@ | |||
"java-base": "11", | |||
"java-devel": "11", | |||
"async_profiler": "2.9", | |||
"phoenix": "5.3.0-4afe457", | |||
"phoenix": "5.3.0-ca21a87dd6", |
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.
Can we add a comment next to this to show where it comes from?
Also, common short-hash is 7 characters long (unless there is a new recommendation to avoid collisions)?
"phoenix": "5.3.0-ca21a87dd6", | |
"phoenix": "5.3.0-ca21a87", # This is currently manually built and uploaded to Nexus |
Description
Part of #816
Relevant comment: #816 (comment)
Images are building successfully: https://github.com/stackabletech/docker-images/actions/runs/10664458546
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>
.