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

BGDIINF_SB-1261: Fixed dockerrun target #11

Merged
merged 1 commit into from
Aug 17, 2020
Merged

Conversation

ltshb
Copy link
Contributor

@ltshb ltshb commented Aug 17, 2020

The dockerrun make target was done to run a local docker image on port
8080. The port 8080 is already used by the BIT proxy on the developper
machines and there were not target for building a local image.

Now there is a target to build local image that can be started by
dockerrun.

The default exported port by the local docker image is 5000 and can be
changed by an environment variable. This variable needs to be set prior
to deployment in order to use another port, but as deployment will be
automated it is not an issue to have to set an environment variable
before it.

The dockerrun make target was done to run a local docker image on port
8080. The port 8080 is already used by the BIT proxy on the developper
machines and there were not target for building a local image.

Now there is a target to build local image that can be started by
dockerrun.

The default exported port by the local docker image is 5000 and can be
changed by an environment variable. This variable needs to be set prior
to deployment in order to use another port, but as deployment will be
automated it is not an issue to have to  set an environment variable
before it.
@ltshb ltshb merged commit 6158517 into master Aug 17, 2020
@ltshb ltshb deleted the bugfix_docker_local_build branch August 17, 2020 07:56
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.

1 participant