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

HPCC-22879 Add parameters to Dockerfiles #18

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

xwang2713
Copy link
Member

Add support for two stages Docker image build
Add support for HPCC Systems cluster configuration on Kubernetes

To build HPCC Systems Docker images with a new version using "--build-arg version="

@gfortil please review the code
@g-pan please help review the README.md:
top level README.md
clienttools/ce/README.md
dev/README.md
platform/ce/README.md

Copy link
Member

@g-pan g-pan left a comment

Choose a reason for hiding this comment

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

See comments in line.

README.md Outdated
@@ -1,87 +1,55 @@
# HPCC Systems Platform Community Version
platform/
- ca/base/<version>/Dockerfile: HPCC Systems Platform Community version prequisites (Ubuntu 18.04)
Copy link
Member

Choose a reason for hiding this comment

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

the -version- tag ONLY displays in this raw mode, it does not display in rich text display... use < (& l t ; and & g t ; ) and > or perhaps change to version

README.md Outdated
Docker Hub: hpccsystems/hpcc-base
- ce/Dockerfile HPCC Systems Platform Community version Docker build file with version as input argument.
Docker Hub: hpccsystems/platform
- ln/base/<version>/Dockerfile: HPCC Systems Platform Internal version prequisites (CentOS 7)
Copy link
Member

Choose a reason for hiding this comment

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

see previous comment -version- does not like > < in .md file


plugins/
- ce/Dockerfile: HPCC Systems Plugins Community version Docker build file (based Platform Docker image) with version as input argument
Copy link
Member

Choose a reason for hiding this comment

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

change to : (based on Platform Docker image)

To start Docker in interactive mode and map ECLWatch port 8010 to host 8010:
```sh
sudo docker run -t -i -p 8010:8010 hpccsystems/hpcc /bin/bash
## How to buid
Copy link
Member

Choose a reason for hiding this comment

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

s/b: How to build

README.md Outdated
sudo docker run -d -p 8020:8010 hpccsystems/hpcc
```
# Development and custom Build
dev/<version>
Copy link
Member

Choose a reason for hiding this comment

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

watch for < and >

Copy link
Member

Choose a reason for hiding this comment

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

that is hiding the contents on the viewable page.

README.md Outdated
```
# Development and custom Build
dev/<version>
<bionic|disco|eoan|el7|gcc7|gcc8|gcc9>/
Copy link
Member

Choose a reason for hiding this comment

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

<bionic is OK, but it might be good to put some space here as < bionic | disco | eoan | etc | etc more readable

sudo docker run -t -i hpccsystems/platform /bin/bash
```

# How is this image build
Copy link
Member

Choose a reason for hiding this comment

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

s/b: built

sudo docker run -d hpccsystems/clienttools
```

# How is this image build
Copy link
Member

Choose a reason for hiding this comment

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

built

@xwang2713 xwang2713 force-pushed the Docker-22879-new-layout branch 2 times, most recently from b882dbe to cc2e2d0 Compare October 1, 2019 12:51
Add support for two stages Docker image build
Add support for HPCC Systems cluster configuration on Kubernetes
Add platform/ce/Dockerfile-local
@xwang2713 xwang2713 force-pushed the Docker-22879-new-layout branch 2 times, most recently from 5fbf921 to b184bdf Compare October 1, 2019 18:16
@xwang2713
Copy link
Member Author

Update as mentioned by Greg.

@xwang2713 xwang2713 force-pushed the Docker-22879-new-layout branch 2 times, most recently from 08df937 to 19c2c50 Compare October 4, 2019 17:25
Add support for two stages Docker image build
Add support for HPCC Systems cluster configuration on Kubernetes
Add platform/ce/Dockerfile-local
Add PUSH.md
@xwang2713 xwang2713 force-pushed the Docker-22879-new-layout branch from 19c2c50 to 90bc161 Compare October 8, 2019 17:01
@xwang2713
Copy link
Member Author

@richardkchapman this is ready to merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants