Conversation
dboltovskyi
force-pushed
the
feature/atf_parallels
branch
from
dc6da99 to
5d7de34
Compare
README.md
Outdated
| - Run `make install` | ||
|
|
||
| **4.** Copy content of scripts repository | ||
| - Run `cp -r <path_to_sdl_atf_test_scripts>/* .` |
There was a problem hiding this comment.
@dboltovskyi maybe better to propose an alternative and to create a soft links?
README.md
Outdated
| - folder with test scripts | ||
| - [OPTIONS] - options supported by ATF: | ||
| - --sdl-core - path to SDL binaries | ||
| - --config - name of configuration |
There was a problem hiding this comment.
@dboltovskyi is it really a name? Maybe it is also a path to some config file?
README.md
Outdated
| - test set | ||
| - folder with test scripts | ||
| - [OPTIONS] - options supported by ATF: | ||
| - --sdl-core - path to SDL binaries |
There was a problem hiding this comment.
| - --sdl-core - path to SDL binaries | |
| - --sdl-core <path> - path to SDL binaries |
README.md
Outdated
| - [OPTIONS] - options supported by ATF: | ||
| - --sdl-core - path to SDL binaries | ||
| - --config - name of configuration | ||
| - --sdl-api - path to SDL APIs |
There was a problem hiding this comment.
| - --sdl-api - path to SDL APIs | |
| - --sdl-api <path> - path to SDL APIs |
README.md
Outdated
| - --sdl-core - path to SDL binaries | ||
| - --config - name of configuration | ||
| - --sdl-api - path to SDL APIs | ||
| - --report - path to report and logs |
There was a problem hiding this comment.
| - --report - path to report and logs | |
| - --report <path> - path to report and logs |
| } | ||
|
|
||
| function prepare_queue { | ||
| if [ -f "$_queue" ];then |
There was a problem hiding this comment.
@dboltovskyi probably its nice to add a description here as well
| } | ||
|
|
||
| function mktemptdir { | ||
| tmpdirname=$(mktemp --suffix=_worker --tmpdir=$_tmp_dir -d) |
tools/runners/parallels.sh
Outdated
| ############################################################# | ||
|
|
||
| function int_handler { | ||
| echo "Please, wait for subprocesses to be buried alive." |
There was a problem hiding this comment.
@dboltovskyi nice kek :) but for production its better to rephrase
tools/runners/parallels.sh
Outdated
| _tmp_workers=$(echo $_tmp_workers" $screen_basename" | xargs) | ||
| done | ||
|
|
||
| log "Workers are running. Waiting termination..." |
There was a problem hiding this comment.
| log "Workers are running. Waiting termination..." | |
| log "Workers are running. Waiting for termination..." |
| dummy() { :; } | ||
|
|
||
| kill_sdl() { | ||
| dummy |
There was a problem hiding this comment.
@dboltovskyi is this file has a role of placeholder in terms of current proposal? Maybe it will be good to leave a TODO comment here?
There was a problem hiding this comment.
This is by design since remote runner is based on common one. However all these functions implemented in common have no sense for remote mode and have to be 'disabled'.
|
@AKalinich-Luxoft Please find updates in e1484c8 - fd3133e |
* add component RC * Add HMI capabilities cache file handling functionality into module SDL * Update connecttest to handle HMI capabilities cache * Add additional checks on file presence into SDL module Co-authored-by: vjacheslav <vklepikov@luxoft.com> Co-authored-by: Dmitriy Boltovskiy <dboltovskyi@luxoft.com>
dboltovskyi
force-pushed
the
feature/atf_parallels
branch
from
ad32fd2 to
8b71811
Compare
3 participants
PR for internal review of SDL-0128 ATF Script Runner proposal