Skip to content

Commit 7d54906

Browse files
Apply suggestions from code review
Co-authored-by: Yanchao Lu <yanchaol@nvidia.com> Signed-off-by: Zhanrui Sun <184402041+ZhanruiSunCh@users.noreply.github.com>
1 parent c4369c0 commit 7d54906

1 file changed

Lines changed: 7 additions & 7 deletions

File tree

.github/workflows/bot-command.yml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -53,27 +53,27 @@ jobs:
5353
const helpMessage = "" +
5454
"## GitHub Bot Help\n\n" +
5555
"`/bot [-h] ['run', 'kill', 'skip', 'reuse-pipeline'] ...`\n\n" +
56-
"provides a user friendly way for developers to interact with a Jenkins server.\n\n" +
56+
"Provide a user friendly way for developers to interact with a Jenkins server.\n\n" +
5757
"Run `/bot [-h|--help]` to print this help message.\n\n" +
5858
"See details below for each supported subcommand.\n\n" +
5959
"<details>\n\n" +
60-
"`run [--disable-fail-fast --skip-test --stage-list \"A10-e2e-0, xxx\" --gpu-type \"A30, H100_PCIe\" --multi-gpu-test --post-merge --extra-stage \"H100_PCIe-[Post-Merge]accuracy-0, xxx\" --memory-profiling --disable-incremental-build --enable-publish-last-known-good ]`\n\n" +
60+
"`run [--disable-fail-fast --skip-test --stage-list \"A10-1, xxx\" --gpu-type \"A30, H100_PCIe\" --add-multi-gpu-test --only-multi-gpu-test --disable-multi-gpu-test --post-merge --extra-stage \"H100_PCIe-[Post-Merge]-1, xxx\"]`\n\n" +
6161
"Launch build/test pipelines. All previously running jobs will be killed.\n\n" +
6262
"`--disable-fail-fast ` *(OPTIONAL)* : Disable fail fast on build/tests/infra failures.\n\n" +
6363
"`--skip-test ` *(OPTIONAL)* : Skip all test stages, but still run build stages, package stages and sanity check stages. Note: Does **NOT** update GitHub check status.\n\n" +
64-
"`--stage-list \"A10-e2e-0, xxx\"` *(OPTIONAL)* : Only run the specified test stages. Examples: \"A10-e2e-0, xxx\". Note: Does **NOT** update GitHub check status.\n\n" +
64+
"`--stage-list \"A10-1, xxx\"` *(OPTIONAL)* : Only run the specified test stages. Examples: \"A10-1, xxx\". Note: Does **NOT** update GitHub check status.\n\n" +
6565
"`--gpu-type \"A30, H100_PCIe\"` *(OPTIONAL)* : Only run the test stages on the specified GPU types. Examples: \"A30, H100_PCIe\". Note: Does **NOT** update GitHub check status.\n\n" +
6666
"`--only-multi-gpu-test ` *(OPTIONAL)* : Only run the multi-GPU tests. Note: Does **NOT** update GitHub check status.\n\n" +
67-
"`--disable-multi-gpu-test ` *(OPTIONAL)* : Disable multi-GPU tests. Note: Does **NOT** update GitHub check status.\n\n" +
68-
"`--add-multi-gpu-test ` *(OPTIONAL)* : Force run multi-GPU tests. Will also run L0 pre-merge pipeline.\n\n" +
67+
"`--disable-multi-gpu-test ` *(OPTIONAL)* : Disable the multi-GPU tests. Note: Does **NOT** update GitHub check status.\n\n" +
68+
"`--add-multi-gpu-test ` *(OPTIONAL)* : Force run the multi-GPU tests. Will also run L0 pre-merge pipeline.\n\n" +
6969
"`--post-merge ` *(OPTIONAL)* : Run the L0 post-merge pipeline instead of the ordinary L0 pre-merge pipeline.\n\n" +
70-
"`--extra-stage \"H100_PCIe-[Post-Merge]accuracy-0, xxx\"` *(OPTIONAL)* : Run the ordinary L0 pre-merge pipeline and specified test stages. Examples: --extra-stage \"H100_PCIe-[Post-Merge]accuracy-0, xxx\".\n\n" +
70+
"`--extra-stage \"H100_PCIe-[Post-Merge]-1, xxx\"` *(OPTIONAL)* : Run the ordinary L0 pre-merge pipeline and specified test stages. Examples: --extra-stage \"H100_PCIe-[Post-Merge]-1, xxx\".\n\n" +
7171
"### kill\n\n" +
7272
"`kill `\n\n" +
7373
"Kill all running builds associated with pull request.\n\n" +
7474
"### skip\n\n" +
7575
"`skip --comment COMMENT `\n\n" +
76-
"Skip testing for latest commit on pull request.\n\n" +
76+
"Skip testing for latest commit on pull request. `--comment \"Reason for skipping build/test\"` is required. IMPORTANT NOTE: This is dangerous since lack of user care and validation can cause top of tree to break.\n\n" +
7777
"### reuse-pipeline\n\n" +
7878
"`reuse-pipeline `\n\n" +
7979
"Reuse a previous pipeline to validate current commit. This action will also kill all currently running builds associated with the pull request. IMPORTANT NOTE: This is dangerous since lack of user care and validation can cause top of tree to break.\n\n" +

0 commit comments

Comments
 (0)