瀏覽代碼

FD3-657 Se actualiza readme y la forma de recibir el nuevo parámetro en el
comando.

Maximiliano Schvindt 6 年之前
父節點
當前提交
1ceed9f939
共有 2 個文件被更改,包括 4 次插入4 次删除
  1. 2 2
      tools/readme.md
  2. 2 2
      tools/src/Release.php

+ 2 - 2
tools/readme.md

@@ -58,7 +58,7 @@
     # para más información de los parámetros ejecutar "docker build --help"
 
 ####. Crear los archivos necesarios para la instalción. Dentro del directorio actual se crea un nuevo directorio con el nombre de la empresa.
-    docker run -it -v /opt/flowdat:/opt/flowdat -v /var/run/docker.sock:/tmp/docker.sock dind make:install /opt/flowdat --client=$CLIENT --add-nginx-links=false
+    docker run -it -v /opt/flowdat:/opt/flowdat -v /var/run/docker.sock:/tmp/docker.sock dind make:install /opt/flowdat --client=$CLIENT --add-nginx-links
     # docker run: ejecuta un comando sobre el contenedor.
     # -it: significa que voy a tener un tty interativo.
     # -v $(pwd):$(pwd): monta como un volumen el directorio actual, en el contenedor bajo el mismo directorio.
@@ -66,7 +66,7 @@
     # dind: es el nombre que le pusimos anteriormente al contenedor.
     # make:install: es el comando que se ejecuta dentro del docker para crear los archivos.
     # $CLIENT: nombre de la empresa que estoy instalando.
-    # --add-nginx-links=true: permite crear links nginx en los servicios de los módulos web de FD3, sirve para instalaciones en modo desarrollo.
+    # --add-nginx-links: permite crear links nginx en los servicios de los módulos web de FD3, sirve para instalaciones en modo desarrollo. Si no se agrega, por defecto no los crea.
     # --domain=DOMINIO-INSTALACION: es el nombre del dominio que se utilizará. El dominio final quedaría base.DOMINIO-INSTALACION.flowdat.com, ftth.DOMINIO-INSTALACION.flowdat.com, etc.
     # para mas informacion ejecutar "docker run -it -v $(pwd):$(pwd) -v /var/run/docker.sock:/tmp/docker.sock dind make:install --help"
 

+ 2 - 2
tools/src/Release.php

@@ -260,7 +260,7 @@ class Release extends Command
             ->addOption('docker-tag', null, InputOption::VALUE_REQUIRED, 'Docker tag to be used. Ej. v0.1.1', "latest")
             ->addOption('all-ref', null, InputOption::VALUE_REQUIRED, 'Git reference (branch or tag) to clone all app. Ej. 0.1.1', "")
             ->addOption('general-version', null, InputOption::VALUE_REQUIRED, 'Set options all-ref and docker-tag with this value. Ej. 0.1.1', "")
-            ->addOption('add-nginx-links', null, InputOption::VALUE_REQUIRED, 'Add NGINX links between dockers', false);
+            ->addOption('add-nginx-links', null, InputOption::VALUE_NONE, 'Add NGINX links between dockers', null);
     }
 
     protected function execute(InputInterface $input, OutputInterface $output)
@@ -323,7 +323,7 @@ class Release extends Command
 	        $this->_ansible_vars["API_CIDR"] = "172.20.0.0/24";
             $this->_ansible_vars["MYSQL_ROOT_PASSWORD"] = $this->_mysql_root_pass;
             
-            $this->_add_nginx_links = $input->getOption('add-nginx-links');
+            $this->_add_nginx_links = (boolean)$input->getOption('add-nginx-links');
 
             $dObj = new DevOps\FileSystem(realpath($this->directory));
             $dObj->dirExists()->realpath();