Fix: log: missing LF after a progress bar (#1886) (#1891)

In #1020 we right pad the progress bar with space characters so that it
will not mess up with log messages printed concurrently during the
progress bar runs. However, if the temrinal width changes after the
progress bar is printed, the message line following it will be shown in
the same line with it.

This commit fixes this by adding a final turn to print the progress bar
without right padding and with a LF character.

Fixes #1886.

Author: liangxin1300

crmsh/log.py

@@ -402,8 +402,10 @@ def status_long(self, msg):
         To wait and mark something finished, start with BEGIN msg, end of END msg
         """
         self.logger.info("BEGIN %s", msg)
+        pb = ProgressBar()
         try:
-            yield ProgressBar()
+            yield pb
+            pb._end()
         except Exception:
             self.logger.error("FAIL %s", msg)
             raise
@@ -515,6 +517,24 @@ def progress(self):
         sys.stdout.write(line)
         sys.stdout.flush()
 
+    def _end(self):
+        try:
+            width, _ = os.get_terminal_size()
+        except OSError:
+            # not a terminal
+            return
+        if self._i == 0:
+            pass
+        elif self._i < width:
+            line = '\r{}\n'.format('.' * self._i)
+            sys.stdout.write(line)
+        else:
+            # the terminal is resized and narrower than the progress bar printed before
+            # just write an LF in this case
+            sys.stdout.write('\n')
+        sys.stdout.flush()
+
+
 
 def setup_logging(only_help=False):
     """