Skip to content

Add github workflow to check for wolfboot regressions#10029

Open
danielinux wants to merge 9 commits intowolfSSL:masterfrom
danielinux:wolfboot-integration-tests
Open

Add github workflow to check for wolfboot regressions#10029
danielinux wants to merge 9 commits intowolfSSL:masterfrom
danielinux:wolfboot-integration-tests

Conversation

@danielinux
Copy link
Copy Markdown
Member

Description

A selection of tests to intercept any changes that would break wolfboot.

Copilot AI review requested due to automatic review settings March 20, 2026 15:04
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a GitHub Actions workflow to run a curated set of wolfBoot integration checks against the wolfSSL code under test, aiming to catch regressions early.

Changes:

  • Introduces a new wolfboot-integration.yml workflow with multiple jobs (keytools, Renode configs, host smoke).
  • Clones wolfBoot and links the PR’s wolfSSL workspace into wolfBoot for integration validation.
  • Uploads Renode execution logs as build artifacts for debugging.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/wolfboot-integration.yml
Comment thread .github/workflows/wolfboot-integration.yml Outdated
Comment thread .github/workflows/wolfboot-integration.yml
Comment thread .github/workflows/wolfboot-integration.yml
Comment thread .github/workflows/wolfboot-integration.yml Outdated
Comment thread .github/workflows/wolfboot-integration.yml Outdated
Copilot AI review requested due to automatic review settings March 20, 2026 17:11
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/wolfboot-integration.yml
Comment thread .github/workflows/wolfboot-integration.yml
Comment thread .github/workflows/wolfboot-integration.yml
Comment thread .github/workflows/wolfboot-integration.yml
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/wolfboot-integration.yml
@dgarske
Copy link
Copy Markdown
Member

dgarske commented Mar 21, 2026

Jenkins retest this please. History lost

@dgarske dgarske assigned danielinux and unassigned danielinux Mar 21, 2026
@dgarske
Copy link
Copy Markdown
Member

dgarske commented Mar 24, 2026

@danielinux please see copilot comment and also check CI failure (probably not related)

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/wolfboot-integration.yml Outdated
Comment thread .github/workflows/wolfboot-integration.yml
Comment thread .github/workflows/wolfboot-integration.yml Outdated
@danielinux danielinux added the Not For This Release Not for release 5.9.1 label Mar 31, 2026
@dgarske dgarske removed the Not For This Release Not for release 5.9.1 label Apr 8, 2026
@danielinux danielinux force-pushed the wolfboot-integration-tests branch from b806467 to 8b9bb6b Compare May 5, 2026 12:02
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

MemBrowse Memory Report

No memory changes detected for:

@danielinux
Copy link
Copy Markdown
Member Author

Jenkins retest this please, failure unrelated.

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