Skip to content

Conversation

gojimmypi
Copy link
Contributor

@gojimmypi gojimmypi commented Aug 12, 2025

Description

This is part 1 of 3 improving Arduino examples and related GitHub workflows.

See also wolfssl-examples #519 and Arduino-wolfSSL #20

Enclosed:

Arduino CI Build 1: https://github.com/$REPO_OWNER/wolfssl/.github/workflows/arduino.yml

  • Builds Arduino library from local clone of wolfssl master branch.
  • Fetches examples from https://github.com/$REPO_OWNER/wolfssl-examples.

The other 2 parts (wolfssl-examples and Arduino-wolfssl repos)

wolfssl-examples

See wolfSSL/wolfssl-examples#519

Arduino CI Build 2: https://github.com/$REPO_OWNER/wolfssl-examples/.github/workflows/arduino-release.yml
- Tests examples based on latest published release of Arduino library, NOT latest on wolfssl github.
- Should be identical to Arduino CI Build 3 in every way but wolfssl install.
- Copies only compile script from wolfssl-examples
- Builds local examples
- No other repos used

Arduino CI Build 3: https://github.com/$REPO_OWNER/wolfssl-examples/.github/workflows/arduino.yml

  • Fetches current wolfSSL from https://github.com/$REPO_OWNER/wolfssl
  • Creates an updated Arduino library
  • Compiles local examples
  • Contains the source of compile-all-examples.sh and respective board-list.txt

Arduino-wolfssl

Arduino CI Build 4: https://github.com/$REPO_OWNER/Arduino-wolfssl/.github/workflows/arduino.yml
- Assembles and installs an updated Arduino wolfssl library from LOCAL wolfssl master source
- Copies only compile script copied from wolfssl-examples
- Builds local examples
- No other repos used

Fixes zd#

Testing

How did you test?

Added new Arduino workflow

See also my dev examples:

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@gojimmypi gojimmypi self-assigned this Aug 12, 2025
@gojimmypi gojimmypi requested a review from dgarske August 12, 2025 20:45
@gojimmypi gojimmypi requested review from wolfSSL-Bot and removed request for dgarske August 12, 2025 21:01
@gojimmypi gojimmypi assigned wolfSSL-Bot and unassigned dgarske Aug 12, 2025
@gojimmypi gojimmypi force-pushed the pr-arduino-testing branch 3 times, most recently from 14e27b4 to 3898d89 Compare August 13, 2025 03:14
@gojimmypi gojimmypi closed this Aug 13, 2025
@anhu anhu reopened this Aug 13, 2025
@gojimmypi
Copy link
Contributor Author

I don't know how this ended up closed. I must have tripped on a keyboard shortcut?

@gojimmypi
Copy link
Contributor Author

Jenkins retest this please

For AgentOfflineException: Unable to create live FilePath for wolf-linux-cloud-node-[n]; wolf-linux-cloud-node-[n] was marked offline: Connection was broken

@gojimmypi gojimmypi requested a review from dgarske August 15, 2025 20:43
@gojimmypi
Copy link
Contributor Author

Jenkins retest this please

For Cannot contact wolf-linux-cloud-node-[n]: java.lang.InterruptedException

@gojimmypi
Copy link
Contributor Author

Jenkins retest this please

For (I don't know what happened) but

tail: cannot open '/var/lib/jenkins/workspace/wolfSSL/PRB-140-3-tests/wolfssl-fips/test-#2.log' for reading: No such file or directory

@gojimmypi
Copy link
Contributor Author

Jenkins retest this please

For Unable to create live FilePath for wolf-linux-cloud-node-[n]; wolf-linux-cloud-node-[n] was marked offline: Connection was broken

Copy link
Contributor

@dgarske dgarske left a comment

Choose a reason for hiding this comment

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

[check-source-text] [4 of 52] [62adf7ab70]
    configure...   real 0m7.391s  user 0m4.763s  sys 0m3.368s
sort: /tmp/wolfssl_test_workdir.3524983/wolfssl/.wolfssl_known_macro_extras:624: disorder: WIFIESPAT
/tmp/wolfssl_test_workdir.3524983/wolfssl/.wolfssl_known_macro_extras is not in C lexical order and/or has repeats.
unneeded entries in .wolfssl_known_macro_extras:
WC_16BIT_CPU
WOLFSSL_DTLS
WOLFSSL_DTLS13
WOLFSSL_NO_TLS12
WOLFSSL_NO_TLS13
WOLFSSL_TLS13
    [skipping "codespell" subtest in check-source-text -- codespell is missing or too old]
overlong lines added:
examples/configs/user_settings_arduino.h:100 #if defined(ESP8266) || defined(__SAM3X8E__) || defined(ARDUINO_AVR_ETHERNET) || \

@dgarske dgarske removed their assignment Sep 2, 2025
@gojimmypi gojimmypi force-pushed the pr-arduino-testing branch 3 times, most recently from f7ba73c to f49a0d9 Compare September 2, 2025 18:36
@gojimmypi
Copy link
Contributor Author

Jenkins retest this please.

for "java.io.StreamCorruptedException: invalid stream header: "

@dgarske - I've updated this PR:

  • Removed stray .wolfssl_known_macro_extras entries and resorted.
  • Removed gojimmypi interim branch references in the new Arduino workflow now that these are merged:
  • Set cURL fetch to use master branch when fetching compile script and board_list.txt from examples repo. (e.g f49a0d9)
  • Added if: github.repository_owner == 'wolfssl' to the new Arduino workflow here.
  • Shortened long line in examples/configs/user_settings_arduino.h
  • Sync with upstream & force push into single commit.

@dgarske dgarske merged commit 095fd88 into wolfSSL:master Sep 3, 2025
398 of 401 checks passed
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