From: dave kormann Date: Wed, 7 Aug 2019 14:59:54 +0000 (-0400) Subject: FIX: CatFile bmc privilege argument X-Git-Url: https://gerrit.akraino.org/r/gitweb?p=ta%2Fremote-installer.git;a=commitdiff_plain;h=f38c513dbecca841726958c420cd211234eec865 FIX: CatFile bmc privilege argument 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 Change-Id: Ie12e04ddd0c1386d2ce204c5aa2ac46da0a59166 --- diff --git a/src/remoteinstaller/installer/catfile.py b/src/remoteinstaller/installer/catfile.py index e5d2147..9e2010e 100644 --- a/src/remoteinstaller/installer/catfile.py +++ b/src/remoteinstaller/installer/catfile.py @@ -21,7 +21,7 @@ class CatFileException(Exception): 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 @@ -128,7 +128,7 @@ def main(): 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__":