Merge "Replace mknod() with chmod()"

2 files changed, 4 insertions, 4 deletions

diff --git a/ironic/drivers/modules/console_utils.py b/ironic/drivers/modules/console_utils.py
index abc3ab496..bf60e9cc7 100644
--- a/ironic/drivers/modules/console_utils.py
+++ b/ironic/drivers/modules/console_utils.py
@@ -90,8 +90,8 @@ def make_persistent_password_file(path, password):
 
     try:
         utils.delete_if_exists(path)
-        os.mknod(path, 0o600)
         with open(path, 'wb') as file:
+            os.chmod(path, 0o600)
             file.write(password)
         return path
     except Exception as e:
diff --git a/ironic/tests/drivers/test_console_utils.py b/ironic/tests/drivers/test_console_utils.py
index 9f9bc4b1d..fb2ecdf73 100644
--- a/ironic/tests/drivers/test_console_utils.py
+++ b/ironic/tests/drivers/test_console_utils.py
@@ -121,9 +121,9 @@ class ConsoleUtilsTestCase(base.TestCase):
         # delete the file
         os.unlink(filepath)
 
-    @mock.patch.object(os, 'mknod', autospec=True)
-    def test_make_persistent_password_file_fail(self, mock_mknod):
-        mock_mknod.side_effect = IOError()
+    @mock.patch.object(os, 'chmod', autospec=True)
+    def test_make_persistent_password_file_fail(self, mock_chmod):
+        mock_chmod.side_effect = IOError()
         filepath = '%(tempdir)s/%(node_uuid)s' % {
                 'tempdir': tempfile.gettempdir(),
                 'node_uuid': self.info['uuid']}