Skip to content
This repository has been archived by the owner on Aug 7, 2020. It is now read-only.

fixes regarding an openshift environment #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ertanden
Copy link

Fixes #24 .

Make sure all boxes are checked (add x inside the brackets) when you submit your contribution, remove this sentence before doing so.

  • I have thoroughly tested my contribution.
  • The code changes are reflected in the documentation README.md.

These changes should fix deployment of the image into an OpenShift environment (OpenShift uses a randomly generated User Id for the user that's running within a container).

It also contains some reorganization of Dockerfile so that whenever there are changes just on start.sh the build is faster. Also some fixes regarding the WORKDIR and HOME directory (/opt/emqttd) so that it's consistent everywhere.

Copy link
Collaborator

@vowstar vowstar left a comment

Choose a reason for hiding this comment

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

LGTM

@spring2maz
Copy link
Contributor

Any reason why this was not merged ? or did it fall between the chars?
I guess it now will conflict with emq30 branch. @turtleDeng

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

Successfully merging this pull request may close these issues.

4 participants