Skip to content

Replicas should atomically update their distributions#7492

Open
dralley wants to merge 2 commits intopulp:mainfrom
dralley:atomic-replication
Open

Replicas should atomically update their distributions#7492
dralley wants to merge 2 commits intopulp:mainfrom
dralley:atomic-replication

Conversation

@dralley
Copy link
Copy Markdown
Contributor

@dralley dralley commented Mar 18, 2026

It's not ideal for distributions to pick up replica changes at random time intervals as various tasks complete, ideally the entire replica is presented as updated at once (or with the smallest possible window).

closes #7333

📜 Checklist

  • Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
  • A changelog entry or entries has been added for any significant changes
  • Follows the Pulp policy on AI Usage
  • (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

@dralley dralley force-pushed the atomic-replication branch 3 times, most recently from 95d534b to 8c05e06 Compare March 24, 2026 03:20
@dralley dralley force-pushed the atomic-replication branch 7 times, most recently from 2ca9bc6 to 88a62e8 Compare March 31, 2026 06:39
It's not ideal for distributions to pick up replica changes at random
time intervals as various tasks complete, ideally the entire replica is
presented as updated at once (or with the smallest possible window).

closes pulp#7333
Assisted-By: claude-opus-4.6
@dralley dralley force-pushed the atomic-replication branch from 88a62e8 to a92f9e2 Compare March 31, 2026 12:25
@dralley dralley marked this pull request as ready for review March 31, 2026 12:44
elif upstream_distribution.get("repository_version"):
# Extract repository href from repository_version href
repo_href = upstream_distribution["repository_version"].rsplit("versions/", 1)[0]
manifest = self.repository_ctx_cls(self.pulp_ctx, repo_href).entity["manifest"]
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is tricky, it implies that other plugins may also have compatibility issues without modification

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe pulp_python may also need a tweak

@dralley dralley requested a review from gerrod3 March 31, 2026 12:49
Generated-By: claude-opus-4.6
@dralley
Copy link
Copy Markdown
Contributor Author

dralley commented Mar 31, 2026

Pulp Python: pulp/pulp_python#1174

Pulp Container: pulp/pulp_container#2301

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Distribution updates during replication should be atomic

1 participant