FIX: CatFile bmc privilege argument 40/1340/1
authordave kormann <dk3239@att.com>
Wed, 7 Aug 2019 14:59:54 +0000 (10:59 -0400)
committerdave kormann <davek@research.att.com>
Wed, 7 Aug 2019 15:06:26 +0000 (11:06 -0400)
A previous change added an IPMI privilege level configuration to
CatFile and erroneously supplied a default value in the initializer.
This change removes the default, as all code that uses the class
already passes a value.

Change-Id: Iec2d6633283ff67bddda5236095c324e97a1c6b8

signed-off-by: dave kormann <davek@resesearch.att.com>
Change-Id: Ie12e04ddd0c1386d2ce204c5aa2ac46da0a59166

src/remoteinstaller/installer/catfile.py

index e5d2147..9e2010e 100644 (file)
@@ -21,7 +21,7 @@ class CatFileException(Exception):
     pass
 
 class CatFile(object):
     pass
 
 class CatFile(object):
-    def __init__(self, bmc_host, bmc_user, bmc_password, bmc_priv_level='ADMINISTRATOR', login_user, login_password):
+    def __init__(self, bmc_host, bmc_user, bmc_password, bmc_priv_level, login_user, login_password):
         self._host = bmc_host
         self._user = bmc_user
         self._password = bmc_password
         self._host = bmc_host
         self._user = bmc_user
         self._password = bmc_password
@@ -128,7 +128,7 @@ def main():
 
     logging.basicConfig(level=logging.DEBUG)
 
 
     logging.basicConfig(level=logging.DEBUG)
 
-    cat_file = CatFile(args.bmc_host, args.bmc_user, args.bmc_password, args.user, bmc_priv_level, args.password)
+    cat_file = CatFile(args.bmc_host, args.bmc_user, args.bmc_password, args.bmc_priv_level, args.user, args.password)
     cat_file.cat(args.file, args.output_file, args.password)
 
 if __name__ == "__main__":
     cat_file.cat(args.file, args.output_file, args.password)
 
 if __name__ == "__main__":