Skip to content
This repository was archived by the owner on Sep 7, 2020. It is now read-only.

Commit cdb4ff1

Browse files
rmelottemergify[bot]
authored andcommitted
tests: boardfarm: use device_get_info for check_status
With #1486, the first device to start has been changed to be the agent. Since the agent cannot be operational unless there is a controller already running on the same network, something else has to be used for "check_status". For now, use netcat to check that "device_get_info" works. Signed-off-by: Raphaël Mélotte <raphael.melotte@mind.be>
1 parent 21f905d commit cdb4ff1

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

tests/boardfarm_plugins/boardfarm_prplmesh/devices/prplmesh_docker.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,8 @@ def check_status(self):
7777
It is used by boardfarm to indicate that spawned device instance is ready for test
7878
and also after test - to insure that device still operational.
7979
"""
80-
self._run_shell_cmd(os.path.join(rootdir, "tools", "docker", "test.sh"),
81-
["-v", "-n", self.docker_name])
80+
self._run_shell_cmd("printf",
81+
["device_get_info", "|", "nc", "-w", "1", "controller-rme", "8002"])
8282

8383
def isalive(self):
8484
"""Method required by boardfarm.

tools/docker/boardfarm-ci/Dockerfile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ RUN apt-get update \
1010
gnupg \
1111
gnupg-agent \
1212
libsnmp-dev \
13+
netcat \
1314
software-properties-common \
1415
wireshark-common \
1516
&& rm -rf /var/lib/apt/lists/*

0 commit comments

Comments
 (0)