Project

Profile

Help

Story #4341

As a user, I have comprehensive docs on mirroring, sync modes, and how they interact together

Added by dalley 9 months ago. Updated 6 months ago.

Status:
MODIFIED
Priority:
Normal
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Platform Release:
Blocks Release:
Backwards Incompatible:
No
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Documentation
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 52

Description

The "mirroring" docs page is currently empty. We need to fill this out with descriptions of what "mirror=True" and "mirror=False" do, and what the default sync mode is. This needs to be well explained to users, with particular emphasis given to the fact that a mirror=True sync on top of a repository with multiple different types of content will be extremely destructive, because mirror syncs don't respect content type boundaries.

https://docs.pulpproject.org/en/3.0/nightly/workflows/mirroring.html

This extends also to the interactions between immediate mode and lazy sync modes, and additive sync. There are "gotchas", like if you first perform a lazy sync and then perform an immediate, additive sync, no artifacts will be downloaded or associated with the existing content units. This is because there are no "new" units and whether the artifacts for the existing units exist or not is not checked.


Related issues

Related to Pulp - Test #4448: Test sync mirror=True CLOSED - COMPLETE Actions
Related to Pulp - Issue #4449: Default to mirror = False MODIFIED Actions

Associated revisions

Revision c6ca9f60 View on GitHub
Added by amacdona@redhat.com 6 months ago

Improve the help text for mirrored sync

Be clear that mirror=True is destructive, and remove the unnecessary
stub for mirroring.

https://pulp.plan.io/issues/4341

fixes #4341

History

#1 Updated by daviddavis 9 months ago

  • Groomed changed from No to Yes
  • Sprint Candidate changed from No to Yes

#2 Updated by dalley 8 months ago

  • Description updated (diff)

#3 Updated by kersom 8 months ago

  • Related to Test #4448: Test sync mirror=True added

#4 Updated by kersom 8 months ago

  • Related to Issue #4449: Default to mirror = False added

#5 Updated by dkliban@redhat.com 7 months ago

  • Tags Documentation added

#6 Updated by dkliban@redhat.com 7 months ago

  • Sprint set to Sprint 51

#7 Updated by amacdona@redhat.com 6 months ago

The responsibility of documenting this feature, IMO belongs to the plugins. This isn't necessarily obvious, because the field is implemented by pulpcore, and used by pulpcore-plugin in the declarative version, but plugins are not required to use the sync serializer and declarative version. It is conceivable that some plugins may not use this feature, or that it might behave differently. Fortunately, plugins do not need to do much, since this is a REST API field, if it is used, it is automatically included in the plugin REST documentation.

IMO, I don't think this needs a prose section. Instead, this story should be to remove the stub and improve the help text. Suggestion:

mirror:
boolean (Mirror)
Default: false

If True, remove content that has been removed upstream. If False, the sync will be only additive.

https://pulp-python.readthedocs.io/en/latest/restapi.html#operation/remotes_python_python_sync

#8 Updated by rchan 6 months ago

  • Sprint changed from Sprint 51 to Sprint 52

#9 Updated by amacdona@redhat.com 6 months ago

  • Status changed from NEW to POST
  • Assignee set to amacdona@redhat.com

#10 Updated by daviddavis 6 months ago

  • Sprint/Milestone set to 3.0

#11 Updated by bmbouter 6 months ago

  • Tags deleted (Pulp 3, Pulp 3 MVP)

#12 Updated by amacdona@redhat.com 6 months ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

Please register to edit this issue

Also available in: Atom PDF