Skip to content

Revert relative_path migrations - #8065

Merged
ggainey merged 1 commit into
pulp:mainfrom
mdellweg:revert_relpath
Sep 4, 2026
Merged

Revert relative_path migrations#8065
ggainey merged 1 commit into
pulp:mainfrom
mdellweg:revert_relpath

Conversation

@mdellweg

@mdellweg mdellweg commented Sep 4, 2026

Copy link
Copy Markdown
Member

📜 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

@mdellweg
mdellweg marked this pull request as ready for review September 4, 2026 15:29

@dkliban dkliban left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks like the correct way to revert the migration and ensure systems that ran it already will be in the same state as the ones that get the new version of migration 156.

@ggainey ggainey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks good - we revert the changes that bite us on large installs while leaving the change on distributions, which are several orders of magnitude less common than "pieces of content".

@ggainey
ggainey merged commit 97b394e into pulp:main Sep 4, 2026
24 of 27 checks passed
@patchback

patchback Bot commented Sep 4, 2026

Copy link
Copy Markdown

Backport to 3.117: 💚 backport PR created

✅ Backport PR branch: patchback/backports/3.117/97b394e350f565457df89b4dbee6c058c23b87c8/pr-8065

Backported as #8066

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

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.

3 participants