Add models for pre0migrating pulp2 repository related data #11
Conversation
pulp3_repository_version (models.OneToOneField): Pulp 3 repository version which Pulp 2 | ||
repository was migrated to | ||
""" | ||
pulp2_repo_id = models.CharField(max_length=255) # in Pulp2 it might be longer? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we use textfield then?wdyt?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it can be but it's a part of a unique index now, so it's better be limited
repository was migrated to | ||
""" | ||
pulp2_repo_id = models.CharField(max_length=255) # in Pulp2 it might be longer? | ||
pulp2_object_id = models.CharField(max_length=255) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is the reason having both object_id and repo_id? can we refer only to repo_id?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we can run a migration multiple times, it's possible that a repo with a specific repo_id can be removed and then created again and it will be a completely different repo. Migration plugin needs to distinguish between those.
closes #5306
https://pulp.plan.io/issues/5306