From: Lucas Di Pentima <ldipentima@veritasgenetics.com>
Date: Tue, 26 Nov 2019 20:06:03 +0000 (-0300)
Subject: 15849: Adds a message for dry-run mode when nothing should be migrated.
X-Git-Tag: 2.0.0~100^2
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/b0f2bbde029e5402a1174423e93a3bdfb5ce9050

15849: Adds a message for dry-run mode when nothing should be migrated.

Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <ldipentima@veritasgenetics.com>
---

diff --git a/tools/vocabulary-migrate/vocabulary-migrate.py b/tools/vocabulary-migrate/vocabulary-migrate.py
index d178228c01..920344b320 100644
--- a/tools/vocabulary-migrate/vocabulary-migrate.py
+++ b/tools/vocabulary-migrate/vocabulary-migrate.py
@@ -111,8 +111,10 @@ def main(arguments=None):
                 if not args.dry_run and migrated_counter % 100 == 0:
                     logger.info('Migrating {} objects...'.format(migrated_counter))
 
-    if not args.dry_run:
-        logger.info('Done, total object migrated: {}.'.format(migrated_counter))
+    if args.dry_run and migrated_counter == 0:
+        logger.info('Nothing to do.')
+    elif not args.dry_run:
+        logger.info('Done, total objects migrated: {}.'.format(migrated_counter))
     return 0
 
 if __name__ == "__main__":