Stability test library: add features and fixed bugs #13452
Conversation
|
⚪ Test history | Ya make output | Test bloat
⚪ Test history | Ya make output | Test bloat | Test bloat
⚪ Test history | Ya make output | Test bloat | Test bloat | Test bloat
🟢 |
|
⚪ Test history | Ya make output | Test bloat
⚪ Test history | Ya make output | Test bloat | Test bloat
⚪ Test history | Ya make output | Test bloat | Test bloat | Test bloat
🟢 |
naspirato
changed the title
|
⚪ Test history | Ya make output | Test bloat
⚪ Test history | Ya make output | Test bloat | Test bloat
⚪ Test history | Ya make output | Test bloat | Test bloat | Test bloat
🟢 |
|
⚪ Test history | Ya make output | Test bloat
⚪ Test history | Ya make output | Test bloat | Test bloat
⚪ Test history | Ya make output | Test bloat | Test bloat | Test bloat
🟢 |
| # How to test stability | ||
| 1) build ydbd (not stripped) | ||
| ``` | ||
| ./ya make --build=profile -DCFLAGS=-fno-omit-frame-pointer --thinlto ydb/apps/ydbd |
There was a problem hiding this comment.
А откуда это взято? Я бы просто ya make -r ydb/apps/ydbd предложил собирать
| ``` | ||
| 2) build library | ||
| ``` | ||
| ./ya make /ydb/tests/stability/library |
There was a problem hiding this comment.
Тоже предложил бы собирать как ya make -r
| ``` | ||
| 3) deploy ydb to test specific build version | ||
| ``` | ||
| cd /ydb/tests/stability/library; ./library deploy_ydb --cluster_path=<path/to/>cluster.yaml --ydbd_path=<path/to/>ydb/apps/ydbd/ydbd |
There was a problem hiding this comment.
/ydb/tests/stability/library -- первый слеш лишний
| ``` | ||
| 3) deploy ydb to test specific build version | ||
| ``` | ||
| cd /ydb/tests/stability/library; ./library deploy_ydb --cluster_path=<path/to/>cluster.yaml --ydbd_path=<path/to/>ydb/apps/ydbd/ydbd |
There was a problem hiding this comment.
<path/to/>ydb/apps/ydbd/ydbd -- вот эта конструкция выглядит на мой вкус странно, предлагаю хотя бы так
repo_root/ydb/apps/ydbd/ydbd
или
<repo_root>/ydb/apps/ydbd/ydbd
Смущают слеши в треугольных скобках
| ./library deploy_tools --cluster_path=<path/to/>cluster.yaml --ydbd_path=<path/to/>ydb/apps/ydbd/ydbd | ||
| ``` | ||
| 5) start workload: | ||
| - `start_all_workloads` - it start all listed bellow worloads |
There was a problem hiding this comment.
worloads -> workloads
| ``` | ||
| 5) start workload: | ||
| - `start_all_workloads` - it start all listed bellow worloads | ||
| - `start_workload_simple_queue_row` - create |
There was a problem hiding this comment.
- create -- что то неподчищенное как будто
| ./library deploy_tools --cluster_path=<path/to/>cluster.yaml --ydbd_path=<path/to/>ydb/apps/ydbd/ydbd | ||
| ``` | ||
| 5) start workload: | ||
| - `start_all_workloads` - it start all listed bellow worloads |
There was a problem hiding this comment.
Предлагаю убрать слово it возможно
| ``` | ||
| ./library start_all_workloads --cluster_path=<path/to/>cluster.yaml --ydbd_path=<path/to/>ydb/apps/ydbd/ydbd | ||
| ``` | ||
| to stop workload, use command `stop_workloads` - stops all worloads |
There was a problem hiding this comment.
- stops all worloads как будто лишнее, предложение же с этого началось
| to stop, use the command `stop_nemesis` | ||
|
|
||
|
|
||
| 7) Check states |
There was a problem hiding this comment.
У тебя выше пункты с маленькой буквы, здесь с большой начались
Предлагаю везде с большой сделать
| 7) Check states | ||
| 1) yq to get all node hosts | ||
| ``` | ||
| sudo wget https://github.com/mikefarah/yq/releases/latest/download/yq_linux_amd64 -O /usr/local/bin/yq |
There was a problem hiding this comment.
Какая то стремноватая тема качать бинарь с гитхаба и руками его ставить в систему
Я предлагаю утилиту доработать чтобы она это в питоне делала
Changelog entry
Changelog category
Additional information
...