Skip to content

Separate workflow approval from build job in QNX workflow#80

Merged
ltekieli merged 2 commits intomainfrom
lt_qnx_workflow_approval_merge
Apr 10, 2026
Merged

Separate workflow approval from build job in QNX workflow#80
ltekieli merged 2 commits intomainfrom
lt_qnx_workflow_approval_merge

Conversation

@ltekieli
Copy link
Copy Markdown
Member

Extract approval into a dedicated job and add merge_group trigger so the build job can be skipped for merge queue events while still requiring manual approval on pull requests.

Extract approval into a dedicated job and add merge_group trigger
so the build job can be skipped for merge queue events while still
requiring manual approval on pull requests.
@ltekieli ltekieli force-pushed the lt_qnx_workflow_approval_merge branch from 7979ddb to 9af1712 Compare April 10, 2026 10:50
@ltekieli ltekieli temporarily deployed to workflow-approval April 10, 2026 10:51 — with GitHub Actions Inactive
@ltekieli ltekieli temporarily deployed to workflow-approval April 10, 2026 11:07 — with GitHub Actions Inactive
@ltekieli ltekieli closed this Apr 10, 2026
@github-project-automation github-project-automation Bot moved this from Backlog to Done in TST - Testing Community Apr 10, 2026
@ltekieli ltekieli reopened this Apr 10, 2026
@ltekieli ltekieli temporarily deployed to workflow-approval April 10, 2026 11:11 — with GitHub Actions Inactive
@ltekieli ltekieli force-pushed the lt_qnx_workflow_approval_merge branch from 4798cb6 to 43cb110 Compare April 10, 2026 11:23
@ltekieli ltekieli temporarily deployed to workflow-approval April 10, 2026 11:23 — with GitHub Actions Inactive
The post-exit drain loop broke after a single 0.1s select check even
when data was still in transit through the SSH transport, causing
truncation and a -1 exit code for large outputs. Now the drain reads
data inline and loops until true EOF.

Also reduce the large-stdout test from 5000x65B to 1000x256B iterations
and raise max_exec_time from 30s to 60s so it completes within the
timeout on emulated QNX QEMU targets.
@ltekieli ltekieli force-pushed the lt_qnx_workflow_approval_merge branch from 43cb110 to ebe4983 Compare April 10, 2026 11:26
@ltekieli ltekieli temporarily deployed to workflow-approval April 10, 2026 11:26 — with GitHub Actions Inactive
@ltekieli ltekieli enabled auto-merge April 10, 2026 11:39
@antonkri antonkri self-requested a review April 10, 2026 11:56
@ltekieli ltekieli added this pull request to the merge queue Apr 10, 2026
Merged via the queue into main with commit e3bdccf Apr 10, 2026
5 checks passed
@ltekieli ltekieli deleted the lt_qnx_workflow_approval_merge branch April 10, 2026 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants