Skip to content

Fixes and improvements for GHA - #3

Merged
sivel merged 1 commit into
mainfrom
gha-fixes
Mar 5, 2026
Merged

sivel merged 1 commit into
mainfrom
gha-fixes

Conversation

@sivel

@sivel sivel commented Mar 5, 2026

Copy link
Copy Markdown
Member
  • Use custom runner for guaranteed capacity
  • ensure build.py failure results in job failure
  • dump setup_collections container logs if the process fails
  • increase setup_collections timeout
  • no tty interaction

* Use custom runner for guaranteed capacity
* ensure build.py failure results in job failure
* dump setup_collections container logs if the process fails
* increase setup_collections timeout
* no tty interaction
@sivel
sivel merged commit 31eb608 into main Mar 5, 2026
3 checks passed
@sivel
sivel deleted the gha-fixes branch March 5, 2026 21:04
include:
- runner: ubuntu-24.04
- runner: ubuntu-24.04-arm64
- runner: ubuntu-24.04-arm64-core

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@sivel what's the difference? Can I use this in ansible/pylibssh?

- name: Build container
id: build
run: |
set -o pipefail

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@sivel FYI, it's possible to just do shell: bash -eEuxo pipefail {0} so that it's a bit separated from the script contents... Could make it a job-default this way too.

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.

2 participants