Mocking utils.check_deploy_backups in OvercloudDeploy tests
The `check_deploy_backups` makes calls to `stat` which can potentially
lead to I/O error in testing scenarios when the relevant file is
removed after path retrieval but before `stat` is called.
This can lead to tox test failure, in both CI and local environments.
Mocking the utils.check_deploy_backups in tests where it is indirectly
called should alleviate the problem.
Long term solution would be mocking all built in functions performing I/O calls by default.
Reviewed: https:/ /review. opendev. org/c/openstack /python- tripleoclient/ +/839712 /opendev. org/openstack/ python- tripleoclient/ commit/ 1a59023772d23c3 c870621f0265753 6a4da57e42
Committed: https:/
Submitter: "Zuul (22348)"
Branch: master
commit 1a59023772d23c3 c870621f0265753 6a4da57e42
Author: Jiri Podivin <email address hidden>
Date: Thu Apr 28 10:51:16 2022 +0200
Mocking utils.check_ deploy_ backups in OvercloudDeploy tests
The `check_ deploy_ backups` makes calls to `stat` which can potentially
lead to I/O error in testing scenarios when the relevant file is
removed after path retrieval but before `stat` is called.
This can lead to tox test failure, in both CI and local environments. deploy_ backups in tests where it is indirectly
Mocking the utils.check_
called should alleviate the problem.
Long term solution would be mocking all built in functions performing I/O calls by default.
Closes-Bug: #1969425
Signed-off-by: Jiri Podivin <email address hidden> 6d476e3383ee264 cea6fd93c24
Change-Id: I96d4bfce84ffe3