Add ssh.pipefail config and use it in sql:dump --gzip (#3818)

weitzman · web-flow · commit bc4ca9b71ecd · 2018-12-09T08:16:28.000-05:00
diff --git a/examples/example.drush.yml b/examples/example.drush.yml
@@ -151,6 +151,8 @@ ssh:
   # password authentication, and is included here, so you may add additional
   # parameters without losing the default configuration.
   options: '-o PasswordAuthentication=no'
+  # This string is valid for Bash shell. Override in case you need something different. See https://github.com/drush-ops/drush/issues/3816.
+  pipefail: 'set -o pipefail; '
 
 notify:
   # Notify when command takes more than 30 seconds.
diff --git a/src/Sql/SqlBase.php b/src/Sql/SqlBase.php
@@ -162,7 +162,9 @@ public function dump()
         $cmd = $this->dumpCmd($table_selection);
         // Gzip the output from dump command(s) if requested.
         if ($this->getOption('gzip')) {
-            $cmd .= ' | gzip -f';
+            // See https://github.com/drush-ops/drush/issues/3816.
+            $pipefail = $this->getConfig()->get('ssh.pipefail', 'set -o pipefail;');
+            $cmd = "$pipefail $cmd | gzip -f";
             $file_suffix .= '.gz';
         }
         if ($file) {
