Skip to content

[PR #1396/0dc1ac25 backport][3.36] Fix nondeterministic simple API package ordering - #1399

Merged
gerrod3 merged 1 commit into
3.36from
patchback/backports/3.36/0dc1ac255ece225679aba91fa1f1072f7077acb5/pr-1396
Sep 28, 2026
Merged

gerrod3 merged 1 commit into
3.36from
patchback/backports/3.36/0dc1ac255ece225679aba91fa1f1072f7077acb5/pr-1396

Conversation

@patchback

@patchback patchback Bot commented Sep 28, 2026

Copy link
Copy Markdown

This is a backport of PR #1396 as merged into main (0dc1ac2).

Summary

  • Add a stable secondary ordering for project names in the simple API query.
  • Order package releases by normalized name and filename.
  • Prevent nondeterministic output when metadata names canonicalize to the same project name.

This carries over the publish.py change from TenSt/pulp_python commit ccae248a91bfc505b48e302db9829435d2b06d2a only.

Tests

  • source ~/devel/pulp/bin/activate && ruff check pulp_python/app/tasks/publish.py
  • Targeted functional test attempted through oci-env; the test reached the publication assertion path but the generated internal HTTPS content URL refused the connection (ConnectionRefusedError).

Summary by CodeRabbit

  • Improvements
    • Project names now appear in a stable, predictable order when normalized names overlap.
    • Package releases are sorted consistently by name and filename.
    • S3 storage configuration now uses the RustFS-compatible endpoint for supported deployments.
    • Build and validation workflows have been updated for more consistent package verification and current Python versions.

Fix nondeterministic simple API package ordering

(cherry picked from commit 0dc1ac2)
@coderabbitai

coderabbitai Bot commented Sep 28, 2026

Copy link
Copy Markdown

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 22fd39cb-8b52-413a-9217-a591039c7ab6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@gerrod3
gerrod3 enabled auto-merge (rebase) September 28, 2026 12:59
@gerrod3
gerrod3 merged commit a209980 into 3.36 Sep 28, 2026
15 checks passed
@gerrod3
gerrod3 deleted the patchback/backports/3.36/0dc1ac255ece225679aba91fa1f1072f7077acb5/pr-1396 branch September 28, 2026 13:08
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