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.

Sign up for GitHub

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

Jump to bottom

Add de ability to use environment variables in Dockerfile #114

Closed
2 tasks done
manuasir opened this issue Feb 8, 2019 · 1 comment
Closed
2 tasks done

Add de ability to use environment variables in Dockerfile #114

manuasir opened this issue Feb 8, 2019 · 1 comment
Assignees
@SitoRBJ
Labels
operations Top priority issues.
Projects
v3.9.x
Milestone
6th week sprint

Comments

@manuasir
Copy link
Contributor

manuasir commented Feb 8, 2019 •
edited

Hi team, it'd be needed to use environment variables within the Dockerfile, so we can build two different images from the same Dockerfile.

  • Research the functionality from the Docker side.
  • Test the solution by building different images from the same Dockerfile.

Cheers
@manuasir manuasir added this to the 6th week sprint milestone Feb 8, 2019
@manuasir manuasir added this to To do in v3.9.x via automation Feb 8, 2019
@manuasir manuasir added the operations Top priority issues. label Feb 8, 2019
@manuasir manuasir moved this from To do to In progress in v3.9.x Feb 8, 2019
@SitoRBJ SitoRBJ mentioned this issue Feb 8, 2019
Merged
@SitoRBJ
Copy link
Contributor

SitoRBJ commented Feb 8, 2019

Hello team,

The PR #117 solves this issue.

Best regards,

Alfonso Ruiz-Bravo

@manuasir manuasir closed this as completed Feb 8, 2019
v3.9.x automation moved this from In progress to Done Feb 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SitoRBJ SitoRBJ

Labels
operations Top priority issues.
Projects
No open projects
v3.9.x
  
Done
Milestone
6th week sprint
Development

No branches or pull requests
2 participants
@SitoRBJ @manuasir