From: Taras Khlivnyak <tkhlivnyak@mirantis.com>
Date: Wed, 20 Jan 2021 15:10:14 +0000 (+0200)
Subject: Validation of auth data has been added.
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=569fe5d44b2d56a344d1df100fa00d796007c02d;p=tools%2Fsustaining.git

Validation of auth data has been added.

Change-Id: I73f910c24116fc754069aaa07fbbe4210b60ca44
---

diff --git a/os_cloud_cleaner/shell.py b/os_cloud_cleaner/shell.py
index 8554134..e7d391b 100644
--- a/os_cloud_cleaner/shell.py
+++ b/os_cloud_cleaner/shell.py
@@ -1,5 +1,6 @@
 import argparse
 import os
+import sys
 from prettytable import PrettyTable
 
 from cleaner import Cleaner
@@ -68,22 +69,27 @@ search_subparser.add_argument('--only-uuids', '-i',
 # args = parser.parse_args("search -i".split(" "))
 # args = parser.parse_args("search".split(" "))
 args = parser.parse_args()
-
-auth_url = args.os_auth_url or os.environ.get('OS_AUTH_URL')
-username = args.os_username or os.environ.get('OS_USERNAME')
-password = args.os_password or os.environ.get('OS_PASSWORD')
-project_name = args.os_project_name or os.environ.get('OS_PROJECT_NAME')
-project_domain_name = args.os_project_domain_name or os.environ.get(
+auth_data = {}
+auth_data['auth_url'] = args.os_auth_url or os.environ.get('OS_AUTH_URL')
+auth_data['username'] = args.os_username or os.environ.get('OS_USERNAME')
+auth_data['password'] = args.os_password or os.environ.get('OS_PASSWORD')
+auth_data['project_name'] = args.os_project_name or os.environ.get('OS_PROJECT_NAME')
+auth_data['project_domain_name'] = args.os_project_domain_name or os.environ.get(
     'OS_PROJECT_DOMAIN_NAME') or os.environ.get('OS_PROJECT_DOMAIN_ID')
-user_domain_name = args.os_user_domain_name or os.environ.get(
+auth_data['user_domain_name'] = args.os_user_domain_name or os.environ.get(
     'OS_USER_DOMAIN_NAME')
 
-cleaner = Cleaner(os_auth_url=auth_url,
-                  os_user=username,
-                  os_password=password,
-                  os_project_name=project_name,
-                  os_project_domain_name=project_domain_name,
-                  os_user_domain_name=user_domain_name,
+for validated_item in auth_data:
+  if not auth_data[validated_item]:
+    logger.error("Parameter {} NOT defined.".format(validated_item))
+    sys.exit(124)
+
+cleaner = Cleaner(os_auth_url=auth_data['auth_url'],
+                  os_user=auth_data['username'],
+                  os_password=auth_data['password'],
+                  os_project_name=auth_data['project_name'],
+                  os_project_domain_name=auth_data['project_domain_name'],
+                  os_user_domain_name=auth_data['user_domain_name'],
                   ignorelist_file=args.ignorelist_file)
 
 if args.action == "cleanup":