Skip to content

[v9.0] Fix/diracx ci gubbins solve group sed v9 - #8850

Merged
aldbr merged 2 commits into
DIRACGrid:rel-v9r0from
fstagni:fix/diracx-ci-gubbins-solve-group-sed-v9
Oct 22, 2026
Merged

aldbr merged 2 commits into
DIRACGrid:rel-v9r0from
fstagni:fix/diracx-ci-gubbins-solve-group-sed-v9

Conversation

@fstagni

@fstagni fstagni commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

backport of #8843

fstagni and others added 2 commits October 22, 2026 13:38
The 'Apply workarounds' step of the DiracX Unit Tests job used
'/solve-group = "gubbins"/d' to break the gubbins environments out of
their shared solve group. In diracx's pixi.toml every gubbins
environment is defined as a single-line inline table ending with
', solve-group = "gubbins"}', so the sed deleted the whole environment
definitions. The pytest-gubbins-* meta-task then depended on tasks in
now-undefined environments, which newer pixi versions reject with:

  task 'pytest-gubbins-all-one-by-one' depends on task
  'pytest-gubbins-core' in environment 'gubbins-core', which is not
  defined

Strip only the ', solve-group = "gubbins"' clause instead, keeping all
environments defined while still decoupling them from the shared group.
pixi 0.82.1 errors on tasks that depend on undefined environments
(prefix-dev/pixi#7284). The workaround intentionally deletes the gubbins
environments (unused by this job), so also delete the
pytest-gubbins-all-one-by-one task that still references them.

Assisted-by: Claude:claude-opus-5-5
@fstagni
fstagni requested a review from atsareg as a code owner October 22, 2026 11:40
@fstagni fstagni changed the title Fix/diracx ci gubbins solve group sed v9 [v9.0] Fix/diracx ci gubbins solve group sed v9 Oct 22, 2026
@aldbr
aldbr enabled auto-merge October 22, 2026 15:39
@aldbr
aldbr disabled auto-merge October 22, 2026 15:42
@aldbr
aldbr merged commit a18222d into DIRACGrid:rel-v9r0 Oct 22, 2026
32 of 34 checks passed
@DIRACGridBot DIRACGridBot added the sweep:ignore Prevent sweeping from being ran for this PR label Oct 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sweep:ignore Prevent sweeping from being ran for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants