From: dave kormann <davek@research.att.com>
Date: Mon, 19 Oct 2020 20:20:42 +0000 (-0400)
Subject: Fix: First boot from floppy, not CD-ROM
X-Git-Url: https://gerrit.akraino.org/r/gitweb?p=ta%2Fremote-installer.git;a=commitdiff_plain;h=HEAD;hp=f67d243ed72be60eb02dacad2584eb189cf6f4b7

Fix: First boot from floppy, not CD-ROM

The remote installer sets the next boot to the floppy device, apparently
to force a boot from remote media.  On current OE19 firmware, this
appears to result in a boot from the default hard drive instead.  This
change sets the next boot to cd, which appears to work when issued
manually

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

diff --git a/remote-installer.spec b/remote-installer.spec
index e23b04c..bbbbcb4 100644
--- a/remote-installer.spec
+++ b/remote-installer.spec
@@ -14,7 +14,7 @@
 
 Name:           remote-installer
 Version:        %{_version}
-Release:        5%{?dist}
+Release:        6%{?dist}
 Summary:        Contains components for the remote-installer
 Group:          %{_platform_group}
 License:        %{_platform_licence}
diff --git a/src/remoteinstaller/installer/bmc_management/falcon.py b/src/remoteinstaller/installer/bmc_management/falcon.py
index 34b3e42..ae67d5c 100644
--- a/src/remoteinstaller/installer/bmc_management/falcon.py
+++ b/src/remoteinstaller/installer/bmc_management/falcon.py
@@ -15,7 +15,7 @@
 
 import logging
 import time
-from .bmctools import BMC
+from .bmctools import BMC, BMCException
 
 RAW_CHECK_NFS_SERVICE_STATUS = '0x32 0xd8 0x06 0x01 0x01 0x00'
 
@@ -45,9 +45,6 @@ RAW_GET_MOUNTED_IMG_COUNT = '0x32 0xd8 0x00 0x01'
 RAW_SET_IMG_NAME =  '0x32 0xd7 0x01 0x01 0x01 0x01 %s'
 RAW_STOP_REDIRECT = '0x32 0xd7 0x01 0x01 0x01 0x00 %s'
 
-class BMCException(Exception):
-    pass
-
 class FALCON(BMC):
     def __init__(self, host, user, passwd, priv_level='ADMINISTRATOR', log_path=None):
         super(FALCON, self).__init__(host, user, passwd, priv_level, log_path)
diff --git a/src/remoteinstaller/installer/bmc_management/hw17.py b/src/remoteinstaller/installer/bmc_management/hw17.py
index 869a6a2..42229fb 100644
--- a/src/remoteinstaller/installer/bmc_management/hw17.py
+++ b/src/remoteinstaller/installer/bmc_management/hw17.py
@@ -12,13 +12,10 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-from .bmctools import BMC
+from .bmctools import BMC, BMCException
 import logging
 import time
 
-class BMCException(Exception):
-    pass
-
 class HW17(BMC):
     def __init__(self, host, user, passwd, priv_level='ADMINISTRATOR', log_path=None):
         super(HW17, self).__init__(host, user, passwd, priv_level, log_path)
diff --git a/src/remoteinstaller/installer/bmc_management/oe19.py b/src/remoteinstaller/installer/bmc_management/oe19.py
index 958801c..eafd594 100644
--- a/src/remoteinstaller/installer/bmc_management/oe19.py
+++ b/src/remoteinstaller/installer/bmc_management/oe19.py
@@ -15,13 +15,10 @@
 from .or18 import OR18
 import logging
 
-class BMCException(Exception):
-    pass
-
 class OE19(OR18):
     def __init__(self, host, user, passwd, priv_level='ADMINISTRATOR', log_path=None):
         super(OE19, self).__init__(host, user, passwd, priv_level, log_path)
 
     def _set_boot_from_virtual_media(self):
-        logging.debug('Set boot from floppy (%s), and boot after that', self._host)
-        self._run_ipmitool_command('chassis bootdev floppy options=persistent')
+        logging.debug('Set boot from cdrom (%s), and boot after that', self._host)
+        self._run_ipmitool_command('chassis bootdev cdrom options=persistent')
diff --git a/src/remoteinstaller/installer/bmc_management/or18.py b/src/remoteinstaller/installer/bmc_management/or18.py
index 78606db..f2bf0ba 100644
--- a/src/remoteinstaller/installer/bmc_management/or18.py
+++ b/src/remoteinstaller/installer/bmc_management/or18.py
@@ -14,10 +14,7 @@
 
 import logging
 import time
-from .bmctools import BMC
-
-class BMCException(Exception):
-    pass
+from .bmctools import BMC, BMCException
 
 class OR18(BMC):
     def __init__(self, host, user, passwd, priv_level='ADMINISTRATOR', log_path=None):
diff --git a/src/remoteinstaller/installer/bmc_management/rm18.py b/src/remoteinstaller/installer/bmc_management/rm18.py
index 7ca7881..1c72229 100644
--- a/src/remoteinstaller/installer/bmc_management/rm18.py
+++ b/src/remoteinstaller/installer/bmc_management/rm18.py
@@ -15,9 +15,6 @@
 from .or18 import OR18
 import logging
 
-class BMCException(Exception):
-    pass
-
 class RM18(OR18):
     def __init__(self, host, user, passwd, priv_level='ADMINISTRATOR', log_path=None):
         super(RM18, self).__init__(host, user, passwd, priv_level, log_path)