Skip to content

chore(logging): lower cache version conflict log level to debug#5630

Merged
baszalmstra merged 1 commit intoprefix-dev:mainfrom
mohitdebian:fix-cache-conflict-log
Mar 10, 2026
Merged

chore(logging): lower cache version conflict log level to debug#5630
baszalmstra merged 1 commit intoprefix-dev:mainfrom
mohitdebian:fix-cache-conflict-log

Conversation

@mohitdebian
Copy link
Contributor

Description

Reduces the log severity for cache version conflicts in source_metadata/mod.rs.

Cache conflicts can occur when multiple Pixi processes compute metadata concurrently and attempt to write to the cache. In these cases Pixi already falls back to the locally computed result, so the situation is expected under optimistic concurrency and does not indicate an error.

Previously this condition was logged using tracing::warn! in source_metadata/mod.rs, while the same condition in build_backend_metadata/mod.rs is logged with tracing::debug!.

This change aligns the log level in source_metadata with the existing behavior in build_backend_metadata, preventing unnecessary warnings in CI logs.

Fixes #5626

How Has This Been Tested?

  • Ran cargo check to ensure the project builds successfully.
  • Ran cargo fmt to ensure formatting is correct.
  • Verified that the only change is reducing the log level from warn! to debug! for WriteResult::Conflict(_) in source_metadata/mod.rs.
  • Confirmed the log level now matches the behavior in build_backend_metadata/mod.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

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.
  • I have verified that changes that would impact the JSON schema have been made in schema/model.py.

@baszalmstra baszalmstra merged commit 8cb7f3f into prefix-dev:main Mar 10, 2026
37 checks passed
@ytausch
Copy link
Contributor

ytausch commented Mar 13, 2026

In light of #5668, I think this change should be reverted. The warning indicated an actual bug in pixi. Not having the warning would have made debugging harder.

@baszalmstra
Copy link
Contributor

Jup did that in #5673

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.

Cache was updated by another process warning when using pixi build

3 participants