Skip to content

[PR #1310/c7fa1454 backport][3.19] Switch content 2.4 metadata upload test to use attrs package to avoid race condition with large sync test - #1313

Merged
gerrod3 merged 1 commit into
pulp:3.19from
gerrod3:patchback/backports/3.19/c7fa1454ac6d806502bfe46be676ef73c50369a0/pr-1310
Jul 29, 2026
Merged

[PR #1310/c7fa1454 backport][3.19] Switch content 2.4 metadata upload test to use attrs package to avoid race condition with large sync test#1313
gerrod3 merged 1 commit into
pulp:3.19from
gerrod3:patchback/backports/3.19/c7fa1454ac6d806502bfe46be676ef73c50369a0/pr-1310

Conversation

@gerrod3

@gerrod3 gerrod3 commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

This is a backport of PR #1310 as merged into main (c7fa145).

Cherry-pick conflict resolved manually: kept the PyPISimple lookup style used on 3.19 while switching the package from setuptools to attrs.

test_basic_pulp_to_pulp_sync uses the large python fixture to populate Pulp which includes setuptools. On sync we don't populate the metadata_version field since it isn't exposed in the pypi/json api, so if this test runs first then setuptools will not have the field set when the content is reused on the upload test. We should come up with a better long term solution but for now this should work.

Made with Cursor

@gerrod3
gerrod3 force-pushed the patchback/backports/3.19/c7fa1454ac6d806502bfe46be676ef73c50369a0/pr-1310 branch from 4f72d37 to bc6052f Compare July 29, 2026 16:03
@jobselko

Copy link
Copy Markdown
Member

It looks you should rebase your branch to get the twine<7 fix.

… race condition with large sync test

(cherry picked from commit c7fa145)
@gerrod3
gerrod3 force-pushed the patchback/backports/3.19/c7fa1454ac6d806502bfe46be676ef73c50369a0/pr-1310 branch from bc6052f to 39fff68 Compare July 29, 2026 17:18
@gerrod3
gerrod3 enabled auto-merge (rebase) July 29, 2026 17:22
@gerrod3
gerrod3 merged commit 5a4f2f9 into pulp:3.19 Jul 29, 2026
14 checks passed
@gerrod3
gerrod3 deleted the patchback/backports/3.19/c7fa1454ac6d806502bfe46be676ef73c50369a0/pr-1310 branch July 29, 2026 17:44
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.

2 participants