Skip to content

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

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

[PR #1310/c7fa1454 backport][3.27] Switch content 2.4 metadata upload test to use attrs package to avoid race condition with large sync test#1312
jobselko merged 1 commit into
pulp:3.27from
gerrod3:patchback/backports/3.27/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.27 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

… race condition with large sync test

(cherry picked from commit c7fa145)
@jobselko
jobselko merged commit 6e15ad0 into pulp:3.27 Jul 29, 2026
54 of 63 checks passed
@gerrod3
gerrod3 deleted the patchback/backports/3.27/c7fa1454ac6d806502bfe46be676ef73c50369a0/pr-1310 branch July 29, 2026 17:17
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