Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Dockerized action runners and script updates #3027

Draft
wants to merge 26 commits into
base: dev
Choose a base branch
from
Draft

Dockerized action runners and script updates #3027

wants to merge 26 commits into from

Conversation

doomwastaken
Copy link
Member

@doomwastaken doomwastaken commented Aug 31, 2023 •
edited
Loading

What's new

  • All test actions (unit, updater and integration) should be able to run on any dockerized runner
  • Moved all scripts/testing into scripts/testops.py
  • Automatic device detection
  • New device reset scenario on any failure (flash under reset)

Verification

  • Unit, Updater and Integration test runs pass
  • Runners should be able to stop for maintenance and restart after
  • Multiple runs could be going simultaneously, up to number of physical devices connected
  • Flipper and STLink should be selected automatically

Checklist (For Reviewer)

  • PR has description of feature/bug or link to Confluence/Jira task
  • Description contains actions to verify feature/bugfix
  • I've built this code, uploaded it to the device and verified feature/bugfix

@github-actions
Copy link

github-actions bot commented Aug 31, 2023 •
edited
Loading

Compiled firmware for commit 1ae55cfc:

@github-actions
Copy link

github-actions bot commented Sep 11, 2023 •
edited
Loading

Compiled f7 firmware for commit 94642bdd:

doomwastaken and others added 19 commits September 11, 2023 07:04
removed wrong status checks
2323f5e
removed unnecessary reboot
916edf9
debug output for port list
eac1c57
enabled debug for workflows, expanded debug com port listing, enabled…
590efd5 
… sim-links ssearch
changed debug level
8049e2c
fixed port list with symlink 0 and 1
f517b77
fixed port list with symlink 0 and 1
ff4a1aa
Merge branch 'dev' into doom/dockerized_action_runners
9c53f50
Merge remote-tracking branch 'origin/dev' into doom/dockerized_action…
2fda912 
…_runners
Merge branch 'dev' into doom/dockerized_action_runners
8c374d5
reverted unit and updater workflow, reversed scripts/testing files, m…
cd11240 
…oved new functionality into test workflow
@doomwastaken
Merge branch 'dev' into doom/dockerized_action_runners
06de003
code formatting for python
bb2e78a
removed debug outputs
b7b448a
Merge branch 'dev' into doom/dockerized_action_runners
d659e8a
updated resourse management, modified new build changes, merged dev
708cd03
added proper trace for alternative workflow
531b5c9
@doomwastaken
Merge branch 'dev' into doom/dockerized_action_runners
2d4298d
removed redundant port selection in test workflow
94642bd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skotopes skotopes Awaiting requested review from skotopes skotopes will be requested when the pull request is marked ready for review skotopes is a code owner

@DrZlo13 DrZlo13 Awaiting requested review from DrZlo13 DrZlo13 will be requested when the pull request is marked ready for review DrZlo13 is a code owner

@hedger hedger Awaiting requested review from hedger hedger will be requested when the pull request is marked ready for review hedger is a code owner

@gsurkov gsurkov Awaiting requested review from gsurkov gsurkov will be requested when the pull request is marked ready for review gsurkov is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@doomwastaken