[promptflow][BugFix] Not clean-up log file for batch run

[zhengfeiwang]

Description

Previous PR #235 adds clean up log file logic, which will remove some system logs printed by executor. This PR adds a check first, and will not clean up for batch run, so that system/executor logs can be reserved.

All Promptflow Contribution checklist:

• The pull request does not introduce [breaking changes].
• CHANGELOG is updated for new features, bug fixes or other significant changes.
• I have read the contribution guidelines.
• Create an issue and link to the pull request to get dedicated review from promptflow team. Learn more: suggested workflow.

General Guidelines and Best Practices

• Title of the pull request is clear and informative.
• There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

• Pull request includes test coverage for the included changes.

[github-actions]

SDK CLI Global Config Test Result zhengfei/bugfix/run-log-missing

2 tests   2 ✔️  47s ⏱️
1 suites  0 💤
1 files    0 ❌

Results for commit 148740c.

♻️ This comment has been updated with latest results.

[github-actions]

SDK PFS E2E Test Result zhengfei/bugfix/run-log-missing

8 tests   8 ✔️  33s ⏱️
1 suites  0 💤
1 files    0 ❌

Results for commit 148740c.

♻️ This comment has been updated with latest results.

[github-actions]

Executor Unit Test Result zhengfei/bugfix/run-log-missing

    2 files      2 suites   1m 58s ⏱️
392 tests 391 ✔️ 1 💤 0 ❌
762 runs  759 ✔️ 3 💤 0 ❌

Results for commit 148740c.

♻️ This comment has been updated with latest results.

[github-actions]

Executor E2E Test Result zhengfei/bugfix/run-log-missing

    2 files      2 suites   7m 34s ⏱️
120 tests 118 ✔️ 2 💤 0 ❌
240 runs  234 ✔️ 6 💤 0 ❌

Results for commit 148740c.

♻️ This comment has been updated with latest results.

[github-actions]

SDK CLI Test Result zhengfei/bugfix/run-log-missing

    2 files      2 suites   30m 16s ⏱️
308 tests 295 ✔️ 13 💤 0 ❌
616 runs  590 ✔️ 26 💤 0 ❌

Results for commit 148740c.

♻️ This comment has been updated with latest results.

[github-actions]

Release Test Matrix

     20 files   -        4       20 suites   - 4   3h 19m 16s ⏱️ - 1h 41m 12s
   823 tests  -        1     771 ✔️ ±       0    16 💤 ±  0    34 ❌  -   1    2 🔥 ±0 
8 344 runs   - 1 218  8 070 ✔️  - 1 174  156 💤  - 52  104 ❌ +10  14 🔥  - 2 

For more details on these failures and errors, see this check.

Results for commit 0437471. ± Comparison against base commit 2891054.

This pull request removes 2 and adds 1 tests. Note that renamed tests count towards both.

tests.sdk_cli_test.unittests.test_run.TestRun ‑ test_invalid_yaml[tests\\test_configs\\runs/illegal/extra_field.yaml-Unknown field]
tests.sdk_cli_test.unittests.test_run.TestRun ‑ test_invalid_yaml[tests\\test_configs\\runs/illegal/non_exist_data.yaml-Can't find directory or file]

tests.sdk_cli_test.e2etests.test_flow_run.TestFlowRun ‑ test_executor_logs_in_batch_run_logs