From d8ab7d88978c44f4b494fade21140ef336c8c83d Mon Sep 17 00:00:00 2001 From: Felipe Monteiro Date: Wed, 28 Sep 2016 17:45:37 -0400 Subject: [PATCH] Fix copying file before checking it exists in hot package. Hot Package does shutil.copy before checking that the file exists, which causes an IOError to be thrown. Change-Id: I11bbb468f53253107ff84e5c8e0d340d6102c4bf Closes-Bug: #1628688 --- murano/packages/hot_package.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/murano/packages/hot_package.py b/murano/packages/hot_package.py index ff32a0f3..7ce9bba9 100644 --- a/murano/packages/hot_package.py +++ b/murano/packages/hot_package.py @@ -77,11 +77,12 @@ class HotPackage(package_base.PackageBase): def _translate_class(self): template_file = os.path.join(self._source_directory, 'template.yaml') - shutil.copy(template_file, self.get_resource(self.full_name)) if not os.path.isfile(template_file): raise exceptions.PackageClassLoadError( self.full_name, 'File with class definition not found') + + shutil.copy(template_file, self.get_resource(self.full_name)) with open(template_file) as stream: hot = yaml.safe_load(stream) if 'resources' not in hot: