Skip to content

CI tweaks - #1265

Closed
kayabaNerve wants to merge 5 commits into
nextfrom
ci-tweaks
Closed

kayabaNerve wants to merge 5 commits into
nextfrom
ci-tweaks

Conversation

@kayabaNerve

Copy link
Copy Markdown
Member

No description provided.

The default is `(pasta, implicit)`, for which the host cannot connect to the
spawned containers. This attempts to fix that by solely changing the port
driver, not the network driver.
@kayabaNerve kayabaNerve added improvement This could be better CI/CD Continuous integration workflows labels Sep 15, 2026
The jobs _generally_ all pass or all fail, so running a single-pass first is
optimal to not redundantly yield the exact same error. We do still need the
full matrix, in case there is a platform-specific issue (especially with
dependencies), but this should be more polite to runners.
It isn't actively a build dependency, and it's expensive to setup
(taking 5-10 minutes in CI). This should reduce CI latency.

`test-dependencies` doesn't make sense today, to spawn a variety of external
networks' nodes in test configurations. The workflows which need such nodes now
run the action for the node they need directly.
@kayabaNerve

Copy link
Copy Markdown
Member Author

The desired effects from this were manually merged.

@kayabaNerve
kayabaNerve deleted the ci-tweaks branch September 19, 2026 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Continuous integration workflows improvement This could be better

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant