Skip to content

feat: warn on stale zst shards repodata#2153

Open
mohamedawnallah wants to merge 2 commits intoconda:mainfrom
mohamedawnallah:feat/warn-stale-zst-shards
Open

feat: warn on stale zst shards repodata#2153
mohamedawnallah wants to merge 2 commits intoconda:mainfrom
mohamedawnallah:feat/warn-stale-zst-shards

Conversation

@mohamedawnallah
Copy link

@mohamedawnallah mohamedawnallah commented Mar 1, 2026

Description

In this PR, we warn when stale repodata.json.zst or repodata_shards.msgpack.zst files exist, but their outputs are disabled, preventing clients from silently consuming outdated package metadata. It also adds integration tests to verify that it has the intended behavior changes and to ensure no regressions.

Closes #1856.

Rattler Index Demo Update

Before After
before-stale-zst-shards-warning.mov
after-stale-zst-shards-warning.mov

How Has This Been Tested?

With integration tests in place in tests/integration/stale_output_tests.rs.

AI Disclosure

  • This PR contains AI-generated content.
    • I have tested any AI-generated content in my PR.
    • I take responsibility for any AI-generated content in my PR.

Tools: {Claude Code}

I used Claude Code to learn and explore relevant parts of rattler codebase. I also used it to write boilerpalate rust test code

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added sufficient tests to cover my changes.

Warn when stale `repodata.json.zst` or `repodata_shards.msgpack.zst`
files exist but their outputs are disabled, preventing clients
from silently consuming outdated package metadata.
@mohamedawnallah
Copy link
Author

Hi @baszalmstra, @wolfv I am new to the Rattler codebase, and I am genuinely interested in contributing. I have submitted that PR to resolve this existing issue #1856. I have looked at the GitHub CI tests seems they are related to the changeset introduced in this PR #2132.

Would love to receive any feedback. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

rattler-index: check whether shards / zst exist if we are not writing them

1 participant