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

Docker image based on Alpine Linux #107

Merged
merged 31 commits into from
Oct 11, 2017
Merged

Docker image based on Alpine Linux #107

merged 31 commits into from
Oct 11, 2017

Conversation

cniweb
Copy link
Member

@cniweb cniweb commented Oct 11, 2017

No description provided.

cniweb added 30 commits June 28, 2017 19:41
Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
Echo actual `NEW_USER_ID`, instead of hardcoded/default id=9001.
Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
- Add alpine as base image and new architecture

Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
- Add alpine as base image in TravisCI build

Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
- Add alpine as base image and new architecture

Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
- Generate new Dockerfiles

Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
- Change base logic for alpine and debian images

Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
- Generate Dockerfiles

Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
- Fix CMD in Dockerfiles

Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
Fix #104

Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
- Add bash for alpine images

Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
- Add generated Dockerfiles

Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
- Add shadow package for alpine
- Add generated Dockerfiles

Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
- Change entrypoint.sh

Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
- Add generated entrypoint.sh

Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
- Fix entrypoint.sh for alpine and version 1.8.3
- Add generated entrypoint.sh

Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
- Update readme for new alpine docker images

Signed-off-by: Christian Häussler <c-n-i@web.de> (github: @cniweb)
@cniweb
Copy link
Member Author

cniweb commented Oct 11, 2017

Close #99

@cniweb cniweb requested review from martinvw and legacycode October 11, 2017 20:23
@cniweb cniweb merged commit 3a28795 into openhab:master Oct 11, 2017
@martinvw
Copy link
Member

I was only at file one with reviewing but that looked good 😉

@martinvw
Copy link
Member

martinvw commented Nov 3, 2017

@cniweb do you think all users running snapshots realize that they will not be updated anymore because they have to pull another image / tag?

It had me in doubt for a while:

martinvw@elbrus:~$ docker pull openhab/openhab:2.2.0-snapshot-amd64
2.2.0-snapshot-amd64: Pulling from openhab/openhab
Digest: sha256:92d8ad4b2947b19e82b883f8dd0f4040e16ea7475558ad74e7754dfaa29ca9df
Status: Image is up to date for openhab/openhab:2.2.0-snapshot-amd64
martinvw@elbrus:~$ docker pull openhab/openhab:2.2.0-snapshot-amd64-debian
2.2.0-snapshot-amd64-debian: Pulling from openhab/openhab
e5b356725761: Pull complete 
16fbc193406b: Pull complete 
959e4b64c293: Pull complete 
a9dc74fa018c: Pull complete 
25de107891bf: Pull complete 
66fa18b5fd7a: Pull complete 
a68a8f15bc03: Pull complete 
cabcbe7b624b: Pull complete 
Digest: sha256:d64d803ef4c7fe3ba8a6db4f674a4f9701eaf3a1de7152017fa963e68139793a
Status: Downloaded newer image for openhab/openhab:2.2.0-snapshot-amd64-debian

wdyt? Should we put out a notice somewhere or can we make we either remove the old openhab/openhab:2.2.0-snapshot-amd64 or let it somehow forward to openhab/openhab:2.2.0-snapshot-amd64-debian

@cniweb
Copy link
Member Author

cniweb commented Nov 3, 2017

@martinvw Good point, I will remove the old tags.

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