Skip to content

Commit e79ddfe

Browse files
Fix queue_time calculation: use build_start for native mode, revert queue_submit to script section [skip-ci]
1 parent 2f56e8c commit e79ddfe

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

scripts/collect_timing.sh

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,12 +20,10 @@ if [ -f results/run_start ] && [ -f results/run_end ]; then
2020
RUN_TIME=$((re - rs))
2121
fi
2222

23-
# Queue time = run_start - queue_submit
24-
if [ -f results/queue_submit ] && [ -f results/run_start ]; then
25-
qs=$(cat results/queue_submit)
26-
rs=$(cat results/run_start)
27-
QUEUE_TIME=$((rs - qs))
28-
fi
23+
# Queue time: not measurable with current Jacamar/pjsub architecture
24+
# (before_script/script all run inside the batch job, so queue_submit
25+
# is recorded after the job has already started)
26+
QUEUE_TIME=0
2927

3028
echo "BUILD_TIME=$BUILD_TIME" > results/timing.env
3129
echo "QUEUE_TIME=$QUEUE_TIME" >> results/timing.env

scripts/matrix_generate.sh

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,6 @@ ${job_prefix}_run:
125125
before_script:
126126
- mkdir -p results
127127
- echo \"Pre-created results directory on login node\"
128-
- bash scripts/record_timestamp.sh results/queue_submit
129128
script:
130129
- echo \"Starting job\"
131130
- ls -la $program_path/
@@ -178,7 +177,6 @@ ${job_prefix}_build_run:
178177
before_script:
179178
- mkdir -p results
180179
- echo \"Pre-created results directory on login node\"
181-
- bash scripts/record_timestamp.sh results/queue_submit
182180
script:
183181
- echo \"Starting build and run\"
184182
- bash scripts/record_timestamp.sh results/build_start

0 commit comments

Comments
 (0)