Skip to content

Conversation

@crusaderky
Copy link
Contributor

  • Bump pixi to latest version
  • Fix issue that causes pixi 0.58 to fail when it's called from lefthook in parallel mode. I observed the same problem sporadically with past versions too.
  • Run pixi update. No noteworthy changes.
  • Misc CI tweaks

@lucascolley
Copy link
Member

Fix issue that causes pixi 0.58 to fail when it's called from lefthook in parallel mode. I observed the same problem sporadically with past versions too.

Could you say a little more about this? Where does the bug seem to be?

@crusaderky
Copy link
Contributor Author

crusaderky commented Oct 24, 2025

Fix issue that causes pixi 0.58 to fail when it's called from lefthook in parallel mode. I observed the same problem sporadically with past versions too.

Could you say a little more about this? Where does the bug seem to be?

The bug in xpx CI, and what I would argue is a missing feature in pixi, is that every time you call pixi run pixi deletes and recreates some files in the environment, even if at the end of it nothing changed.
So if you run pixi run -e lint <some task> many times in parallel, like lefthook does in main, you trigger a race condition among the multiple pixi processes, which causes them to fail, because they all delete and recreate some files even if you just entered the lint environment with pixi run -e lint lint, so in theory there should be nothing to do.

@lucascolley
Copy link
Member

Interesting. @Hofer-Julian do you know if there is an open issue for this?

Pixi's CI itself is still using this lefthook setup, so maybe there is another difference that means this isn't hit over there? What are the files created/deleted?

The disadvantage is that the duplication of lint task definitions increases maintenance burden a little.

@Hofer-Julian
Copy link

Yeah, there's this issue. Tbh I was already wondering why we don't seem to hit it with our lefthook setup:

prefix-dev/pixi#1482

@lucascolley
Copy link
Member

Haha, I already gave it a +1 😅

@Hofer-Julian
Copy link

Maybe we should add --frozen for now? I guess this would at least avoid this problem for now

@crusaderky
Copy link
Contributor Author

Changed to --frozen; it seems to do the job

@lucascolley lucascolley changed the title MAINT Pixi maintenance MAINT: Pixi maintenance Oct 24, 2025
Copy link
Member

@lucascolley lucascolley left a comment

Choose a reason for hiding this comment

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

thanks Guido, much appreciated!

@lucascolley lucascolley added the maintenance general maintenance label Oct 24, 2025
@lucascolley lucascolley added this to the 0.9.1 milestone Oct 24, 2025
@lucascolley lucascolley merged commit 4c6960c into data-apis:main Oct 24, 2025
11 checks passed
@crusaderky crusaderky deleted the pixi branch October 24, 2025 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance general maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants