Re-enable hostap tests and remove some flaky tests#9841
Re-enable hostap tests and remove some flaky tests#9841dgarske merged 1 commit intowolfSSL:masterfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Re-enables the hostap/wpa-supplicant GitHub Actions workflow on push/PR and adjusts cipher suite parsing behavior to mark default suites as explicitly set, while also removing a subset of flaky hostap tests.
Changes:
- Set
suites->setSuiteswhen falling back to the wolfSSL default cipher list. - Re-enable the
hostap-vmworkflow forpushandpull_requestevents and expand the certificate regeneration step. - Remove several flaky tests from pinned hostap config test lists.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/internal.c | Marks the default cipher suite list as “set” when ParseCipherList falls back to wolfSSL defaults. |
| .github/workflows/hostap-vm.yml | Re-enables CI triggers and updates the order/steps for regeneration of hostap auth_serv certs. |
| .github/workflows/hostap-files/configs/b607d2723e927a3446d89aed813f1aa6068186bb/tests | Removes specific flaky tests from this pinned hostap test list. |
| .github/workflows/hostap-files/configs/07c9f183ea744ac04585fb6dd10220c75a5e2e74/tests | Removes specific flaky tests from this pinned hostap test list. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Jenkins retest this please - M1 never ran |
|
Jenkins retest this please - history lost. The M1 failure in unit test doesn't appear to be related. |
Set suites->setSuites to 1 in ParseCipherList function
d717013 to
4498e12
Compare
|
@julek-wolfssl not sure why but your PR is failing https://cloud.wolfssl-test.com/jenkins/job/PRB-multi-test-script/10005/pipeline-overview/ |
|
Merging with multi-test issues (not related) |
This reverts commit 4498e12. see wolfSSL#9841 and wolfSSL#10052.
This reverts commit 4498e12. see wolfSSL#9841 and wolfSSL#10052.
Set suites->setSuites to 1 in ParseCipherList function