Skip to content

CI: upload sshd-test logs on failure - #1324

Open
ejohnstown wants to merge 1 commit into
wolfSSL:masterfrom
ejohnstown:sshd-test-failure-logs
Open

ejohnstown wants to merge 1 commit into
wolfSSL:masterfrom
ejohnstown:sshd-test-failure-logs

Conversation

@ejohnstown

Copy link
Copy Markdown
Contributor

When a step fails, the wolfSSHd Test workflow now uploads its logs as artifacts kept for five days, so an intermittent failure leaves its diagnostics behind.

  • make check: test-suite.log, the per-test logs, and config.log, uploaded before later steps reconfigure
  • wolfSSHd suite: the daemon's root-owned log.txt and the failing test's stdout.txt

The wolfSSHd Test workflow now keeps its logs as five-day artifacts when
a step fails, so an intermittent failure leaves its diagnostics behind.

- make check: test-suite.log, the per-test logs, and config.log
- wolfSSHd suite: the daemon's root-owned log.txt and the failing
  test's stdout.txt
Copilot AI balanced review requested due to automatic review settings October 16, 2026 21:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fenrir Automated Review — PR #1324

Scan targets checked: wolfssh-pr-general
Review type: general (no scan target covers the changed files)
Coverage: 1 of 1 in-scope changed file(s) opened by the reviewer

Fenrir result: Approved ✅

No new issues found in the changed files.

Advisory only — this automated result does not count as a GitHub approval.

Review tier: Lite

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.

4 participants