Project

Profile

Help

Story #8205

Updated by bmbouter over 2 years ago

## The Two Related Problem 

 1. The 'deprecated' data is mutable. 
 2. The 'deprecated' data is not correctly included in import/export data 

 ## Solution 

 From Once a high level, make the [`AnsibleCollectionDeprecated`](https://github.com/pulp/pulp_ansible/blob/4b3d63f14dd85b9bfd689074defb99a23ee72f31/pulp_ansible/app/models.py#L281) content. 

 ### Modeling 

 Here are the rough steps involved: 
 1. switch [the existing AnsibleCollectionDeprecated](https://github.com/pulp/pulp_ansible/blob/4b3d63f14dd85b9bfd689074defb99a23ee72f31/pulp_ansible/app/models.py#L281) solution to inherit from Content 
 2. Remove the [`repository_version` FK](https://github.com/pulp/pulp_ansible/blob/4b3d63f14dd85b9bfd689074defb99a23ee72f31/pulp_ansible/app/models.py#L286-L288) entirely. 
 3. Change the [`collection`](https://github.com/pulp/pulp_ansible/blob/4b3d63f14dd85b9bfd689074defb99a23ee72f31/pulp_ansible/app/models.py#L289) field to #8204 has been created then AnsibleCollectionDeprecated resource should be a string with the collection name instead 

 ### Sync Implementation 

 The sync code will continue to parse the deprecated data in FirstStage in the same way, but instead of [saving/managing them in first-stage](https://github.com/pulp/pulp_ansible/blob/master/pulp_ansible/app/tasks/collections.py#L140-L153), have the stages pipeline handle created so that it all by making `AnsibleCollectionDeprecated` in-memory and associating with a declarative content object. 

 ### The Galaxy API viewset 

 [Port this portion](https://github.com/pulp/pulp_ansible/blob/8cd963d769e5b25f957d241571e21c371264a8f4/pulp_ansible/app/galaxy/v3/views.py#L172) to look for the objects is included in the set of content of the RepositoryVersion. 

 ### The viewset for set/unset deprecation handling 

 [Port this portion](https://github.com/pulp/pulp_ansible/blob/8cd963d769e5b25f957d241571e21c371264a8f4/pulp_ansible/app/galaxy/v3/views.py#L298-L306) to generate a task that creates a RepositoryVersion 

 ### Additional work needed to get this working with Import/Export 

 @davidd? export/import

Back