-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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.
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
Ble-WiFi commissioning test in CI #32889
Open
jlatusek
wants to merge
62
commits into
project-chip:master
Choose a base branch
from
jlatusek:ubuntu-qemu
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
5f1fc49
Add qemu testing to CI
jlatusek d18e47c
Simple script to proxy messages between D-Bus buses
arkq b7fd335
Fix typing for old python
jlatusek 553c84e
Fix D-Bus error forwarding
arkq 7e1bc8c
Remove map file
jlatusek ae25b21
Restyled by isort
restyled-commits ad08cd2
Move BLE-WiFi testing requirements to other file
jlatusek 88a573c
Use code-wifi pairing method
jlatusek 5deffd0
Merge remote-tracking branch 'upstream/master' into ubuntu-qemu
jlatusek 410a523
Add support for WiFi interface created by mac80211_hwsim driver
jlatusek f0763e3
Do not run test script when connected by ssh
jlatusek 635042d
Add dry run suport
jlatusek fdc9a59
Make shutdown more gracefully
jlatusek fc93c24
Fix code-wifi pairing
jlatusek 44bc1cb
Merge remote-tracking branch 'upstream/master' into ubuntu-qemu
jlatusek d8bb72c
Get wlan name from config
jlatusek 2685ef2
Cleanup code
jlatusek 6e05a6f
Merge remote-tracking branch 'upstream/master' into ubuntu-qemu
jlatusek 8b9305e
Bump version file
jlatusek 391f3b9
Merge remote-tracking branch 'upstream/master' into ubuntu-qemu
jlatusek 6a1f93a
Use specific qemu img version
jlatusek bf02c8b
Update variable names, remove only one site of network interface
jlatusek a2bcbfc
Bump CI images versions
jlatusek cd83087
Bump CI images version
jlatusek d11a7c1
Upgrade pip
jlatusek 99b4808
Update all python packages
jlatusek 882ba5b
Remove bugprone-casting-through-void check from .clang-tidy
jlatusek 8a97fb8
Merge remote-tracking branch 'upstream/master' into bump_CI_images
jlatusek 6bf6bf3
Add manual constrainst to fix proto conflict issue
jlatusek c17dce5
Update telink requirements
jlatusek cfd4d91
Use version of python generated by script
jlatusek e3c969a
Update images to latest one
jlatusek 958e508
Merge branch 'master' of https://github.com/project-chip/connectedhom…
jlatusek 9d24ced
Update darwin version
jlatusek 2831fb4
Update python version for macos
jlatusek 3cbca97
Hardcode darwin python version
jlatusek d0c654d
Change darwin architecture
jlatusek 1646412
Update bitarray
jlatusek 0837128
Merge remote-tracking branch 'upstream/master' into bump_CI_images
jlatusek c000b55
Revert mac to old version but update python only
jlatusek 88f8d6e
Update python version on macos
jlatusek 372d435
Merge branch 'macos-python-update' into bump_CI_images
jlatusek 1cedc8f
Update python version on macos
jlatusek d210082
Update system python
jlatusek d37d4a6
Update license
jlatusek 0eacd5b
restyle
jlatusek caa2acc
Merge branch 'macos-python-update' into bump_CI_images
jlatusek 2e0d4fc
Merge remote-tracking branch 'upstream/master' into bump_CI_images
jlatusek ba648c6
Merge branch 'bump_CI_images' into ubuntu-qemu
jlatusek 1310792
Merge remote-tracking branch 'upstream/master' into ubuntu-qemu
jlatusek ea9fdac
Merge branch 'master' of https://github.com/project-chip/connectedhom…
jlatusek dbab8f0
Restore changes
jlatusek 3fb707b
Restore original requirements
jlatusek 66e5529
Add qemu-ubuntu config removed during merge
jlatusek 8f6dc17
Merge branch 'master' of https://github.com/project-chip/connectedhom…
jlatusek 27d2bf8
Merge remote-tracking branch 'upstream/master' into ubuntu-qemu
jlatusek 40d4f8f
Restore submodules
jlatusek 3a5c39d
Update qemu image
jlatusek 13f2d73
Remove ble_wifi arg from shutdown namespace
jlatusek 04b100b
Merge remote-tracking branch 'upstream/master' into ubuntu-qemu
jlatusek 361b07f
Zap regenerate
jlatusek 18d38f5
Fix script issue
jlatusek File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -97,3 +97,68 @@ jobs: | |||||
--target tizen-arm-tests-no-ble-no-thread \ | ||||||
build | ||||||
" | ||||||
|
||||||
qemu-linux: | ||||||
name: ubuntu | ||||||
|
||||||
runs-on: ubuntu-latest | ||||||
if: github.actor != 'restyled-io[bot]' | ||||||
|
||||||
container: | ||||||
image: ghcr.io/project-chip/chip-build-linux-qemu:74 | ||||||
volumes: | ||||||
- "/tmp/log_output:/tmp/test_logs" | ||||||
# Required for using KVM | ||||||
options: --privileged | ||||||
|
||||||
steps: | ||||||
- name: Checkout | ||||||
uses: actions/checkout@v4 | ||||||
- name: Checkout submodules & Bootstrap | ||||||
uses: ./.github/actions/checkout-submodules-and-bootstrap | ||||||
with: | ||||||
platform: linux | ||||||
|
||||||
- name: Build Apps | ||||||
run: | | ||||||
scripts/run_in_build_env.sh './scripts/build_python.sh --install_virtual_env out/venv' | ||||||
./scripts/run_in_build_env.sh \ | ||||||
"./scripts/build/build_examples.py \ | ||||||
--target linux-x64-chip-tool \ | ||||||
--target linux-x64-all-clusters \ | ||||||
build \ | ||||||
--copy-artifacts-to objdir-clone \ | ||||||
" | ||||||
# There is no enough space for running the test withouth cleaning the environment | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
- name: Clean up | ||||||
run: | | ||||||
rm -rf out/*/obj | ||||||
rm -rf out/*/lib | ||||||
rm -rf out/*/*.map | ||||||
rm -rf $PW_ENVIRONMENT_ROOT | ||||||
git clean -fdx --exclude out | ||||||
# Without all required apps paths provided as argument, script starts to search for them in the current directory and it takes a lot of time. | ||||||
- name: Run ble commission test using the python parser sending commands to chip-tool | ||||||
run: | | ||||||
./scripts/run_in_vm.sh \ | ||||||
"scripts/run_in_build_env.sh 'pip3 install -r scripts/setup/requirements.ble-wifi-testing.txt' && \ | ||||||
./scripts/run_in_build_env.sh \ | ||||||
\"./scripts/tests/run_test_suite.py \ | ||||||
--runner chip_tool_python \ | ||||||
--target TestCommissionerNodeId \ | ||||||
--chip-tool ./out/linux-x64-chip-tool/chip-tool \ | ||||||
run \ | ||||||
--iterations 1 \ | ||||||
--test-timeout-seconds 120 \ | ||||||
--all-clusters-app ./out/linux-x64-all-clusters/chip-all-clusters-app \ | ||||||
--lock-app ./out/linux-x64-lock/chip-lock-app \ | ||||||
--ota-provider-app ./out/linux-x64-ota-provider/chip-ota-provider-app \ | ||||||
--ota-requestor-app ./out/linux-x64-ota-requestor/chip-ota-requestor-app \ | ||||||
--tv-app ./out/linux-x64-tv-app/chip-tv-app \ | ||||||
--bridge-app ./out/linux-x64-bridge/chip-bridge-app \ | ||||||
--lit-icd-app ./out/linux-x64-lit-icd/lit-icd-app \ | ||||||
--microwave-oven-app .out/linux-x64-microwave-oven/chip-microwave-oven-app \ | ||||||
--rvc-app .out/linux-x64-rvc/chip-rvc-app \ | ||||||
--ble-wifi \ | ||||||
\" \ | ||||||
" |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -85,3 +85,5 @@ examples/*/esp32/dependencies.lock | |
# jupyter temporary files | ||
.ipynb_checkpoints | ||
|
||
/runner.sh | ||
/runner_status |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -22,7 +22,7 @@ | |
_PORT = 9000 | ||
|
||
if sys.platform == 'linux': | ||
_IP = '10.10.10.5' | ||
_IP = "10.10.12.5" | ||
|
||
|
||
def _make_url(): | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,36 @@ | ||
#!/usr/bin/env bash | ||
# | ||
# Copyright (c) 2024 Project CHIP Authors | ||
# | ||
# Licensed under the Apache License, Version 2.0 (the "License"); | ||
# you may not use this file except in compliance with the License. | ||
# You may obtain a copy of the License at | ||
# | ||
# http://www.apache.org/licenses/LICENSE-2.0 | ||
# | ||
# Unless required by applicable law or agreed to in writing, software | ||
# distributed under the License is distributed on an "AS IS" BASIS, | ||
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
# See the License for the specific language governing permissions and | ||
# limitations under the License. | ||
# | ||
|
||
# This script executes the command given as an argument after | ||
# activating the given python virtual environment | ||
set -e | ||
|
||
PROJECT_PATH=$(dirname "$(dirname "$(realpath "$0")")") | ||
|
||
echo "$@" >"$PROJECT_PATH/runner.sh" | ||
chmod +x "$PROJECT_PATH/runner.sh" | ||
|
||
echo "CMD:" | ||
cat "$PROJECT_PATH/runner.sh" | ||
|
||
"$PROJECT_PATH/integrations/docker/images/stage-2/chip-build-linux-qemu/run-img.sh" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is pretty specific for a "run_in_vm" script. The script needs a better name. |
||
|
||
if [ -f "$PROJECT_PATH/runner_status" ]; then | ||
exit "$(cat "$PROJECT_PATH/runner_status")" | ||
else | ||
exit 1 | ||
fi |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
PyGObject |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -28,7 +28,7 @@ | |
PORT = 9000 | ||
|
||
if sys.platform == 'linux': | ||
IP = '10.10.10.5' | ||
IP = "10.10.12.5" | ||
|
||
|
||
class AppsRegister: | ||
|
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this change?