Project

Profile

Help

Issue #7834

Updated deprecation status is not included in sync

Added by awcrosby 6 months ago. Updated 6 months ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
-
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
3. High
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
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:

  1. Ensure the synctest repo in ci.cloud.redhat.com has some collections deprecated, others not
  2. For the first time, sync the repo from cloud to on-prem hub
  3. Confirm deprecation status is as expected on each collection in on-prem hub
  4. Change deprecation status on one of the synced collections on cloud
  5. Sync again
  6. 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 6 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 6 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 6 months ago

  • Project changed from Pulp to Ansible Plugin

#2 Updated by fao89 6 months ago

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

#3 Updated by fao89 6 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 6 months ago

  • Sprint/Milestone set to 0.5.2

#5 Updated by fao89 6 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 6 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 6 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.

#8 Updated by fao89 6 months ago

the tests scenario I thought was not valid, since the deprecation is being changed locally, and for the reported issue it happens at server side

#10 Updated by pulpbot 6 months ago

  • Status changed from NEW to POST

#11 Updated by Anonymous 6 months ago

  • Status changed from POST to MODIFIED

#13 Updated by pulpbot 6 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF