a few fixes for run_performance_tests.py flame graph generators

tools/run_tests/performance/process_local_perf_flamegraphs.sh

@@ -13,6 +13,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+set -ex
+
 mkdir -p "$OUTPUT_DIR"
 
 PERF_DATA_FILE="${PERF_BASE_NAME}-perf.data"
@@ -22,4 +24,5 @@ PERF_SCRIPT_OUTPUT="${PERF_BASE_NAME}-out.perf"
 echo "running perf script on $PERF_DATA_FILE"
 perf script -i "$PERF_DATA_FILE" > "$PERF_SCRIPT_OUTPUT"
 
+# use https://github.com/brendangregg/FlameGraph
 ~/FlameGraph/stackcollapse-perf.pl "$PERF_SCRIPT_OUTPUT" | ~/FlameGraph/flamegraph.pl > "${OUTPUT_DIR}/${OUTPUT_FILENAME}.svg"

tools/run_tests/performance/process_remote_perf_flamegraphs.sh

@@ -13,6 +13,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+set -ex
+
 mkdir -p "$OUTPUT_DIR"
 
 PERF_DATA_FILE="${PERF_BASE_NAME}-perf.data"
@@ -27,4 +29,5 @@ scp "$USER_AT_HOST:~/performance_workspace/grpc/$PERF_SCRIPT_OUTPUT.gz" .
 
 gzip -d -f "$PERF_SCRIPT_OUTPUT.gz"
 
+# use https://github.com/brendangregg/FlameGraph
 ~/FlameGraph/stackcollapse-perf.pl --kernel "$PERF_SCRIPT_OUTPUT" | ~/FlameGraph/flamegraph.pl --color=java --hash > "${OUTPUT_DIR}/${OUTPUT_FILENAME}.svg"

tools/run_tests/run_performance_tests.py

@@ -484,7 +484,7 @@ def run_collect_perf_profile_jobs(hosts_and_base_names, scenario_name,
     failures, _ = jobset.run(
         perf_report_jobs, newline_on_success=True, maxjobs=1)
     jobset.message(
-        'END', 'Collecting perf reports from qps workers', do_newline=True)
+        'SUCCESS', 'Collecting perf reports from qps workers', do_newline=True)
     return failures