diff options
-rw-r--r-- | ironic/common/utils.py | 2 | ||||
-rw-r--r-- | ironic/drivers/modules/deploy_utils.py | 9 | ||||
-rw-r--r-- | ironic/tests/unit/drivers/test_deploy_utils.py | 12 |
3 files changed, 12 insertions, 11 deletions
diff --git a/ironic/common/utils.py b/ironic/common/utils.py index f286fe7fa..8df146ec4 100644 --- a/ironic/common/utils.py +++ b/ironic/common/utils.py @@ -552,7 +552,7 @@ def dd(src, dst, *args): """ LOG.debug("Starting dd process.") execute('dd', 'if=%s' % src, 'of=%s' % dst, *args, - run_as_root=True, check_exit_code=[0]) + use_standard_locale=True, run_as_root=True, check_exit_code=[0]) def is_http_url(url): diff --git a/ironic/drivers/modules/deploy_utils.py b/ironic/drivers/modules/deploy_utils.py index 0799cbe43..840ec65aa 100644 --- a/ironic/drivers/modules/deploy_utils.py +++ b/ironic/drivers/modules/deploy_utils.py @@ -481,9 +481,7 @@ def destroy_disk_metadata(dev, node_uuid): LOG.debug("Start destroy disk metadata for node %(node)s.", {'node': node_uuid}) try: - utils.execute('dd', 'if=/dev/zero', 'of=%s' % dev, - 'bs=512', 'count=36', run_as_root=True, - check_exit_code=[0]) + utils.dd('/dev/zero', dev, 'bs=512', 'count=36') except processutils.ProcessExecutionError as err: with excutils.save_and_reraise_exception(): LOG.error(_LE("Failed to erase beginning of disk for node " @@ -506,9 +504,8 @@ def destroy_disk_metadata(dev, node_uuid): else: seek_value = block_sz - 36 try: - utils.execute('dd', 'if=/dev/zero', 'of=%s' % dev, - 'bs=512', 'count=36', 'seek=%d' % seek_value, - run_as_root=True, check_exit_code=[0]) + utils.dd('/dev/zero', dev, 'bs=512', 'count=36', + 'seek=%d' % seek_value) except processutils.ProcessExecutionError as err: with excutils.save_and_reraise_exception(): LOG.error(_LE("Failed to erase the end of the disk on node " diff --git a/ironic/tests/unit/drivers/test_deploy_utils.py b/ironic/tests/unit/drivers/test_deploy_utils.py index 33ccb2221..d43e7e716 100644 --- a/ironic/tests/unit/drivers/test_deploy_utils.py +++ b/ironic/tests/unit/drivers/test_deploy_utils.py @@ -1416,11 +1416,13 @@ class DestroyMetaDataTestCase(tests_base.TestCase): mock_gz.return_value = 64 expected_calls = [mock.call('dd', 'if=/dev/zero', 'of=fake-dev', 'bs=512', 'count=36', run_as_root=True, - check_exit_code=[0]), + check_exit_code=[0], + use_standard_locale=True), mock.call('dd', 'if=/dev/zero', 'of=fake-dev', 'bs=512', 'count=36', 'seek=28', run_as_root=True, - check_exit_code=[0])] + check_exit_code=[0], + use_standard_locale=True)] utils.destroy_disk_metadata(self.dev, self.node_uuid) mock_exec.assert_has_calls(expected_calls) self.assertTrue(mock_gz.called) @@ -1430,7 +1432,8 @@ class DestroyMetaDataTestCase(tests_base.TestCase): expected_call = [mock.call('dd', 'if=/dev/zero', 'of=fake-dev', 'bs=512', 'count=36', run_as_root=True, - check_exit_code=[0])] + check_exit_code=[0], + use_standard_locale=True)] self.assertRaises(processutils.ProcessExecutionError, utils.destroy_disk_metadata, self.dev, @@ -1442,7 +1445,8 @@ class DestroyMetaDataTestCase(tests_base.TestCase): expected_call = [mock.call('dd', 'if=/dev/zero', 'of=fake-dev', 'bs=512', 'count=36', run_as_root=True, - check_exit_code=[0])] + check_exit_code=[0], + use_standard_locale=True)] self.assertRaises(processutils.ProcessExecutionError, utils.destroy_disk_metadata, self.dev, |