Skip to content

Conversation

mkumykov
Copy link
Contributor

@mkumykov mkumykov commented Dec 9, 2019

For SAP, layer matching for base container image layers modified to use SHA256 ids.
Project structure removal is modified to allow multiple versions of a container image to co-exist nicely

@mkumykov mkumykov requested a review from gsnyder2007 December 9, 2019 20:09
Copy link
Contributor

@gsnyder2007 gsnyder2007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm assuming that the same change you made should be applied in the else clause.

@@ -845,7 +845,7 @@ def get_or_create_project_version(self, project_name, version_name, parameters =
if project:
version = self.get_version_by_name(project, version_name)
if not version:
self.create_project_version(project, version_name, parameters)
self.create_project_version(project, version_name, parameters=parameters)
version = self.get_version_by_name(project, version_name)
else:
self.create_project(project_name, version_name, parameters)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this also be changed? parameters=parameters?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that is correct

@gsnyder2007 gsnyder2007 merged commit 1c04abf into master Dec 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants