Skip to content

Conversation

@jobselko
Copy link
Contributor

@jobselko jobselko commented May 11, 2025

This PR extends the repair_metadata endpoint for repositories to also support on-demand content if the associated remote supports the PyPI json version API.

For on-demand content, we expect that:

  1. PythonPackageContent always has correct name and version
  2. RemoteArtifact always has correct sha256

Closes #849

@jobselko jobselko self-assigned this May 11, 2025
@jobselko jobselko force-pushed the repair_on_demand_content branch 4 times, most recently from fc1102d to 99180e4 Compare May 16, 2025 18:54
@jobselko jobselko force-pushed the repair_on_demand_content branch 8 times, most recently from 9e27aaf to 04c2e4f Compare May 22, 2025 11:33
@jobselko jobselko force-pushed the repair_on_demand_content branch 2 times, most recently from 9875d50 to 925e392 Compare May 22, 2025 11:38
@jobselko jobselko changed the title DRAFT: Add on_demand content to repair_metadata Support on-demand content in repair_metadata May 22, 2025
@jobselko jobselko marked this pull request as ready for review May 22, 2025 11:59
@jobselko jobselko requested a review from gerrod3 May 22, 2025 12:25
@jobselko jobselko force-pushed the repair_on_demand_content branch 2 times, most recently from 2b14e83 to 79d7acf Compare May 22, 2025 12:51
@jobselko jobselko force-pushed the repair_on_demand_content branch from 79d7acf to b2b74fa Compare May 22, 2025 12:53
@github-actions github-actions bot removed the no-issue label May 22, 2025
@jobselko jobselko force-pushed the repair_on_demand_content branch 2 times, most recently from 42e9834 to 0b91e31 Compare May 22, 2025 13:42
@jobselko jobselko marked this pull request as draft May 22, 2025 18:03
@jobselko jobselko force-pushed the repair_on_demand_content branch 2 times, most recently from 7c3cb5c to 95f6e5f Compare May 25, 2025 21:44
@jobselko jobselko marked this pull request as ready for review May 26, 2025 11:43
Copy link
Contributor

@gerrod3 gerrod3 left a comment

Choose a reason for hiding this comment

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

Good work on the logic, the grouping of the packages by name-version and then further grouping by shared url of the RAs looks right (it is definitely a bit complex!)

@jobselko jobselko force-pushed the repair_on_demand_content branch from 95f6e5f to b0b12d9 Compare June 4, 2025 13:36
@github-actions github-actions bot added multi-commit Add to bypass single commit lint check no-changelog labels Jun 4, 2025
@jobselko jobselko force-pushed the repair_on_demand_content branch from b0b12d9 to 308b353 Compare June 4, 2025 13:46
@jobselko jobselko force-pushed the repair_on_demand_content branch from 308b353 to 1cdff86 Compare June 4, 2025 13:53
@jobselko jobselko requested a review from gerrod3 June 4, 2025 18:58
@jobselko jobselko force-pushed the repair_on_demand_content branch 2 times, most recently from ab59d2d to ecb804f Compare June 9, 2025 15:01
@jobselko jobselko force-pushed the repair_on_demand_content branch from ecb804f to 154f96a Compare June 9, 2025 16:49
@jobselko jobselko requested a review from gerrod3 June 9, 2025 17:22
Copy link
Contributor

@gerrod3 gerrod3 left a comment

Choose a reason for hiding this comment

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

Looks good, thank you!

@jobselko jobselko merged commit 56c8a1c into pulp:main Jun 10, 2025
12 checks passed
@jobselko jobselko deleted the repair_on_demand_content branch July 23, 2025 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

multi-commit Add to bypass single commit lint check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support on-demand content in repair_metadata endpoint

2 participants