Commit df82e52
committed
fix: Add null check before shutil.copy to prevent NoneType error
This fixes a crash when processing redis-benchmark tests where
full_result_path is None. The test execution and metrics collection
complete successfully, but the file preservation step would fail.
Changes:
- Add null check before shutil.copy(full_result_path, dest_fpath)
- Log warning message when result path is unavailable
- Prevents TypeError: stat: path should be string, not NoneType
Fixes issue observed in:
- memtier_benchmark-nokeys-pubsub-mixed-100-channels test
- Other redis-benchmark output conversion scenarios1 parent 683cda5 commit df82e52
1 file changed
+12
-5
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3101 | 3101 | | |
3102 | 3102 | | |
3103 | 3103 | | |
3104 | | - | |
3105 | | - | |
3106 | | - | |
| 3104 | + | |
| 3105 | + | |
| 3106 | + | |
| 3107 | + | |
| 3108 | + | |
| 3109 | + | |
| 3110 | + | |
| 3111 | + | |
| 3112 | + | |
| 3113 | + | |
| 3114 | + | |
| 3115 | + | |
3107 | 3116 | | |
3108 | | - | |
3109 | | - | |
3110 | 3117 | | |
3111 | 3118 | | |
3112 | 3119 | | |
| |||
0 commit comments