Skip to content

[WIP] Bump HBase to 2.1.10 - #3

Draft
nvtkaszpir wants to merge 3 commits into
Gradiant:hbase-2.0.1from
nvtkaszpir:2.1.10
Draft

[WIP] Bump HBase to 2.1.10#3
nvtkaszpir wants to merge 3 commits into
Gradiant:hbase-2.0.1from
nvtkaszpir:2.1.10

Conversation

@nvtkaszpir

Copy link
Copy Markdown

2.1.10 is minimal maintained HBase version.

Also had to add HADOOP_VERSION because COPY --from was not following symlink. (another option is to try with hardlinks)

@nvtkaszpir nvtkaszpir changed the title Bump HBase to 2.1.10 [WIP] Bump HBase to 2.1.10 Apr 27, 2020
@nvtkaszpir
nvtkaszpir marked this pull request as draft April 27, 2020 08:54
Comment thread hbase-base/Dockerfile Outdated
@@ -1,9 +1,9 @@
FROM gradiant/hadoop-base:2.7.7 as hadoop
FROM quay.io/kaszpir/dockerized-hadoop:3.1.3 as hadoop

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We must not hardcode an external registry in the Dockerfile.
We can use docker build args for that:

ARG hadoop_image="gradiant/hdfs:3.1.2"
FROM $hadoop_image as hadoop
...

Then just set hadoop_image variable in your build environment (for example):

docker build -t quay.io/kaszpir/hbse:2.1.10 --build-arg hadoop_image=quay.io/kaszpir/dockerized-hadoop:3.1.3 .

Also if you need hadoop 3.1.x versions, we have created dockerhub gradiant/hdfs repo with 2.7.7, 2.9.2, 3.1.2 and 3.2.1 images from current gradiant/dockerized-hadoop.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

yeah I know, that's why it's WIP/Draft.
I'm experimenting with some other options, but ARG looks the best.
Was trying to avoid any external Makefiles and so on because I wanted to use ofifical quay.io building image feature from Dockerfile.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants