Skip to content
This repository has been archived by the owner on Dec 29, 2023. It is now read-only.

moving ARG to place where they are first used to get better docker la… #16

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

moving ARG to place where they are first used to get better docker la… #16

wants to merge 1 commit into from

Conversation

giedriusramas
Copy link

Moving ARG to the places where they are first used to get better Docker layer reuse. Putting ARG at top of Dockerfile prevent layer re-use.

@ventz
Copy link
Contributor

ventz commented Apr 30, 2020

@giedriusramas Could you please open PR against: https://github.com/harvard-itsecurity/docker-misp/

This repo here is just a read-only clone copy of that repo. All of the development is done there.

Also, could you leave a text comment on top, with - even though they are listed in the README, it would be convenient for someone to see them all in one place.

# User supplied inputs - (see bellow within Dockerfile for real ARGS that you can replace)
# // DO NOT TOUCH HERE - THIS SECTION IS NOT USED - SIMPLY USED AS README //
##################################################################
# ARG MYSQL_MISP_PASSWORD=ChangeThisDefaultPassworda9564ebc3289b7a14551baf8ad5ec60a	
# ARG POSTFIX_RELAY_HOST=localhost	
# ARG MISP_FQDN=localhost	
# ARG MISP_EMAIL=admin@localhost	
# ARG MISP_GPG_PASSWORD=ChangeThisDefaultPasswordXuJBao5Q2bps89LWFqWkKgDZwAFpNHvc
##################################################################
# ^^^ DO NOT TOUCH HERE - THIS SECTION IS NOT USED - SIMPLY USED AS README ^^^

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.

2 participants