[CI][Bench] Run nightly benchmarks after the nightly image was pushed

[lukaszstolarczuk]

It's better to use the actually last nightly build in benchmarks.

[lukaszstolarczuk]

Anyone remember if this was on purpose, to run before the SYCL nightly workflow, not after?

A few days back we hit an issue, which was fixed by updating dockers... and they were not used in benchmarks' workflow, as it used the old nightly docker....

[sarnex]

no idea if was on purpose but imo the new behavior is correct, any benchmarks of the nightly should be benchmarking that nightly

[aelovikov-intel]

Can you use a trigger like this

llvm/.github/workflows/sycl-linux-precommit-aws.yml

Lines 13 to 17 in 7aa73a8

	on:
	workflow_run:
	workflows: [SYCL Pre Commit on Linux]
	types:
	- completed

instead of relying on the scheduling delay?

[sarnex]

Can you use a trigger like this instead of relying on the scheduling delay?

While maybe less clean I think the scheduling delay is safer, in theory there could be a race condition if the images are a little slow to be available for download after being uploaded, a 3 hour delay should prevent that

[aelovikov-intel]

Can you use a trigger like this instead of relying on the scheduling delay?

While maybe less clean I think the scheduling delay is safer, in theory there could be a race condition if the images are a little slow to be available for download after being uploaded, a 3 hour delay should prevent that

I'm relying on it being available here

llvm/.github/workflows/sycl-prebuilt-e2e-container.yml

Line 99 in e502f46

image: ghcr.io/${{ github.repository }}/sycl_prebuilt_tests:${{ inputs.ref || github.ref_name }}

but that only runs manually and very rarely. Haven't had issues, but maybe I was just lucky. Google search's AI says

In the context of GitHub Actions workflows, one job can immediately use the image pushed by a previous job, provided the correct permissions and naming conventions are used.

but I didn't verify that's not a hallucination.

To sum up, @lukaszstolarczuk , your call. Both Nick and I are fine with either approach.