Issue #7834: Updated deprecation status is not included in sync - Ansible Plugin - Pulp

Send by e-mail

Issue #7834

Updated deprecation status is not included in sync

Added by awcrosby 4 months ago. Updated 4 months ago.

Status:

CLOSED - CURRENTRELEASE

Priority:

Normal

Assignee:

Sprint/Milestone:

0.5.2

Start date:

Due date:

Estimated time:

Severity:

3. High

Platform Release:

OS:

Triaged:

Yes

Groomed:

Sprint Candidate:

Tags:

Sprint:

Sprint 86

Quarter:

Q4-2020

Description

This was seen on testing with galaxy_ng: In 4.2.0 GA, updated deprecation status is not sync’d from cloud to on-prem hub

Steps to recreate:

Ensure the synctest repo in ci.cloud.redhat.com has some collections deprecated, others not
For the first time, sync the repo from cloud to on-prem hub
Confirm deprecation status is as expected on each collection in on-prem hub
Change deprecation status on one of the synced collections on cloud
Sync again
Observe that the collection on-prem does not have the deprecation status changed as expected

Associated revisions

Revision eae4deed View on GitHub
Added by Fabricio Aguiar 4 months ago

Ensure deprecation status is synced even when no content changes

https://pulp.plan.io/issues/7834 closes #7834

Revision a3e7827c View on GitHub
Added by Fabricio Aguiar 4 months ago

Ensure deprecation status is synced even when no content changes

https://pulp.plan.io/issues/7834 closes #7834

History

#1 Updated by fao89 4 months ago

Project changed from Pulp to Ansible Plugin

#2 Updated by fao89 4 months ago

Triaged changed from No to Yes
Sprint set to Sprint 86
Quarter set to Q4-2020

#3 Updated by fao89 4 months ago

https://github.com/pulp/pulp_ansible/blob/master/pulp_ansible/app/models.py#L237-L265

https://github.com/pulp/pulpcore/blob/master/pulpcore/app/models/repository.py#L786-L820

If the deprecation was the only thing that changed from the last version, the new version will be deleted, therefore the deprecation status won't get updated

#4 Updated by fao89 4 months ago

Sprint/Milestone set to 0.5.2

#5 Updated by fao89 4 months ago

Test scenario:

sync
modify deprecation status from one collection
sync again
verify the deprecation corresponds with the remote source
verify a new repository version was created

Question: should deprecation status be synced only on mirror mode?

#6 Updated by bmbouter 4 months ago

The syncing in these examples: from where? Since we can only "set" deprecated on the local pulp system it would need to be a pulp-to-pulp sync, but I suspect these are from actual remote sources like galaxy.

I expected sync to occur on all sync types, mirror and otherwise.

#7 Updated by bmbouter 4 months ago

I think the current bug comes in because since Pulp has already saved the content locally and the changing of deprecated state info occurs at save time it ends up not updated the changed deprecated state for content it already saved into that repo. Basically it's a re-sync bug due to the setting of deprecated data being set only on newly saved units here.