~rjarry/dlrepo: job: rm redundant code in _link_to_product()

~rjarry/dlrepo

bcfcf656ac9e9d2d16c07d3280483b1a5426904e — Julien Floret 1 year, 2 months ago 696dc9c

job: rm redundant code in _link_to_product()

The method iterates over the job formats to symlink each format into
the product folder. It then attempts to symlink the "container" format
explicitly. The latter is redundant since containers are already taken
care of in the loop.

Fixes: bd1c23893882 ("server: add filesystem api")
Signed-off-by: Julien Floret <julien.floret@6wind.com>
Acked-by: Thomas Faivre <thomas.faivre@6wind.com>

patch browse .tar.gz

1 files changed, 0 insertions(+), 3 deletions(-)

M dlrepo/fs/job.py

M dlrepo/fs/job.py => dlrepo/fs/job.py +0 -3

@@ 110,9 110,6 @@ class Job(SubDir):
         self.update_symlink(version.path(), self._product_link_path())
         for fmt in self.get_formats():
             self.update_symlink(fmt.path(), version.path() / fmt.name)
-        container = self.path() / "container"
-        if container.is_dir():
-            self.update_symlink(container, version.path() / "container")
 
     def _cleanup_product_tree(self):
         link = self._product_link_path()