Skip to content

Commit

Permalink
ci: use single workflow for examples and test apps
Browse files Browse the repository at this point in the history
  • Loading branch information
igrr committed Sep 27, 2024
1 parent a155997 commit 42ca438
Show file tree
Hide file tree
Showing 13 changed files with 246 additions and 201 deletions.
49 changes: 49 additions & 0 deletions .github/get_idf_build_apps_args.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,49 @@
#!/usr/bin/env python3

import argparse
import os

def main():
parser = argparse.ArgumentParser()
parser.add_argument('-v', '--verbose', action='store_true', help='Enable verbose output')
parser.add_argument('modified_files_list', type=argparse.FileType('r'), help='Input file containing list of modified files')
parser.add_argument('idf_build_apps_args', type=argparse.FileType('w'), help='Output file containing idf-build-apps arguments')
args = parser.parse_args()

modified_files = args.modified_files_list.read().splitlines()
idf_build_apps_args = []
idf_build_apps_args += [
'--modified-files',
';'.join(modified_files)
]

if args.verbose:
print('Modified files:')
for file in sorted(modified_files):
print(f' - {file}')

modified_components = set()
excluded_dirs = ['.github', 'test_app']
for file in modified_files:
toplevel = file.split('/')[0]
if toplevel in excluded_dirs:
continue
if not os.path.isdir(toplevel):
continue
modified_components.add(toplevel)

idf_build_apps_args += [
'--modified-components',
';'.join(modified_components)
]

args.idf_build_apps_args.write(' '.join(idf_build_apps_args))

if args.verbose:
print('Modified components:')
for component in sorted(modified_components):
print(f' - {component}')


if __name__ == '__main__':
main()
48 changes: 48 additions & 0 deletions .github/get_pytest_args.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,48 @@
#!/usr/bin/env python3

import argparse
import json
import glob


def main():
parser = argparse.ArgumentParser()
parser.add_argument('-v', '--verbose', action='store_true', help='Enable verbose output')
parser.add_argument('--target', type=str, required=True, help='Target to run tests for')
parser.add_argument('build_info_json', type=str, help='Input file(s) containing build info generated by idf-build-apps. Accepts globs.')
parser.add_argument('pytest_args', type=argparse.FileType('w'), help='Output file containing pytest arguments')

args = parser.parse_args()
pytest_args = []
app_json_files = glob.glob(args.build_info_json)
if args.verbose:
print(f'Found {len(app_json_files)} app_json files')
for app_json_file in app_json_files:
print(f' - {app_json_file}')
for app_json_file in app_json_files:
with open(app_json_file, 'r') as build_info_json:
if args.verbose:
print(f'Processing {app_json_file}')
for app_json_line in build_info_json.readlines():
app_json = json.loads(app_json_line)
skip = False
if app_json['target'] != args.target:
continue
if app_json['build_status'] == 'skipped':
if args.verbose:
print(f'Skipping {app_json["app_dir"]})')
pytest_args += [
'--ignore',
app_json['app_dir']
]
else:
if args.verbose:
print(f'Not skipping {app_json["app_dir"]})')


args.pytest_args.write(' '.join(pytest_args))


if __name__ == '__main__':
main()

147 changes: 147 additions & 0 deletions .github/workflows/build_and_run_apps.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,147 @@
name: Build and Run Apps

on:
schedule:
- cron: '0 0 * * *' # Once per day at midnight
pull_request:
types: [opened, reopened, synchronize]

jobs:
build:
name: Build Apps
strategy:
fail-fast: false
matrix:
idf_ver:
- "release-v5.0"
- "release-v5.1"
- "release-v5.2"
- "release-v5.3"
- "latest"
parallel_index: [1,2,3,4,5] # Update --parallel-count below when changing this
runs-on: ubuntu-22.04
container: espressif/idf:${{ matrix.idf_ver }}
steps:
- uses: actions/checkout@v4
with:
submodules: 'true'
- name: Install dependencies
shell: bash
run: |
. ${IDF_PATH}/export.sh
pip install idf-component-manager idf-build-apps --upgrade
- name: Fix git repo permissions
# Needed by the next git diff step.
# See https://github.com/actions/runner/issues/2033
if: github.event_name == 'pull_request'
run: |
build_dir=$PWD
cd /
git config --global --add safe.directory $build_dir
cd -
- name: Find files and component changed in PR
if: github.event_name == 'pull_request'
# For PRs only:
# - find the files changed in the PR
# - based on the files list, determine which components have changed
# - output both lists as a file of idf-build-apps arguments
run: |
git fetch --recurse-submodules=no origin ${{ github.base_ref }}:base_ref
git fetch --recurse-submodules=no origin pull/${{ github.event.pull_request.number }}/head:pr_ref
git diff --name-only -r base_ref pr_ref > changed_files.txt
python3 .github/get_idf_build_apps_args.py -v changed_files.txt idf_build_apps_args.txt
- name: Find apps
shell: bash
run: |
. ${IDF_PATH}/export.sh
idf-build-apps find
- name: Build apps
shell: bash
run: |
. ${IDF_PATH}/export.sh
export PEDANTIC_FLAGS="-DIDF_CI_BUILD -Werror -Werror=deprecated-declarations -Werror=unused-variable -Werror=unused-but-set-variable -Werror=unused-function"
export EXTRA_CFLAGS="${PEDANTIC_FLAGS} -Wstrict-prototypes"
export EXTRA_CXXFLAGS="${PEDANTIC_FLAGS}"
touch idf_build_apps_args.txt
idf-build-apps build --parallel-index ${{ matrix.parallel_index }} --parallel-count 5 --collect-app-info build_info_${{ matrix.idf_ver }}_${{ matrix.parallel_index }}.json $(cat idf_build_apps_args.txt)
- uses: actions/upload-artifact@v4
with:
name: app_binaries_${{ matrix.idf_ver }}_${{ matrix.parallel_index }}
path: |
*/examples/*/build_esp*/bootloader/bootloader.bin
*/examples/*/build_esp*/partition_table/partition-table.bin
*/examples/*/build_esp*/*.bin
*/examples/*/build_esp*/flasher_args.json
*/examples/*/build_esp*/config/sdkconfig.json
*/test_apps/**/build_esp*/bootloader/bootloader.bin
*/test_apps/**/build_esp*/partition_table/partition-table.bin
*/test_apps/**/build_esp*/*.bin
*/test_apps/**/build_esp*/flasher_args.json
*/test_apps/**/build_esp*/config/sdkconfig.json
build_info*.json
run-target:
name: Run apps on target
if: ${{ github.repository_owner == 'espressif' }}
needs: build
strategy:
fail-fast: false
matrix:
idf_ver:
- "release-v5.0"
- "release-v5.1"
- "release-v5.2"
- "release-v5.3"
- "latest"
runner:
- runs-on: "esp32"
marker: "generic"
target: "esp32"
- runs-on: "ESP32-ETHERNET-KIT"
marker: "ethernet"
target: "esp32"
env:
TEST_RESULT_NAME: test_results_${{ matrix.runner.target }}_${{ matrix.runner.marker }}_${{ matrix.idf_ver }}
TEST_RESULT_FILE: test_results_${{ matrix.runner.target }}_${{ matrix.runner.marker }}_${{ matrix.idf_ver }}.xml
runs-on: [self-hosted, linux, docker, "${{ matrix.runner.runs-on }}"]
container:
image: python:3.11-bookworm
options: --privileged # Privileged mode has access to serial ports
steps:
- uses: actions/checkout@v4
- uses: actions/download-artifact@v4
with:
pattern: app_binaries_${{ matrix.idf_ver }}_*
merge-multiple: true
- name: Install Python packages
env:
PIP_EXTRA_INDEX_URL: "https://dl.espressif.com/pypi/"
run: pip install --prefer-binary cryptography pytest-embedded pytest-embedded-serial-esp pytest-embedded-idf
- name: Run apps
run: |
python3 .github/get_pytest_args.py --target=${{ matrix.runner.target }} -v 'build_info*.json' pytest-args.txt
cat pytest-args.txt
pytest $(cat pytest-args.txt) --ignore-glob '*/managed_components/*' --ignore=test_app --ignore=.github --junit-xml=${{ env.TEST_RESULT_FILE }} --target=${{ matrix.runner.target }} -m ${{ matrix.runner.marker }} --build-dir=build_${{ matrix.runner.target }}
- name: Upload test results
uses: actions/upload-artifact@v4
if: always()
with:
name: ${{ env.TEST_RESULT_NAME }}
path: ${{ env.TEST_RESULT_FILE }}

publish-results:
name: Publish Test results
needs:
- run-target
if: ${{ always() && github.repository_owner == 'espressif' }} # Run even if the previous step failed
runs-on: ubuntu-22.04
steps:
- name: Download Test results
uses: actions/download-artifact@v4
with:
pattern: test_results_*
path: test_results
- name: Publish Test Results
uses: EnricoMi/publish-unit-test-result-action@v2
with:
files: test_results/**/*.xml
97 changes: 0 additions & 97 deletions .github/workflows/build_and_run_examples.yml

This file was deleted.

Loading

0 comments on commit 42ca438

Please sign in to comment.