Fix annotating CI run when product tests fail

[nineinchnick]

Description

The product tests job test step should have an ID used to check if it failed, so its test reports can be processed and annotations can be generated.

Additional context and related issues

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text:

[wendigo]

@nineinchnick there is no annotation here

[nineinchnick]

I think this is because the environment failed to start, and it didn't even run any tests. If tempto was not started, it couldn't have generated any testng-report.xml file:

2024-09-19T10:17:41.626Z	INFO	main	io.trino.tests.product.launcher.env.Environment	Environment 'multinode-sqlserver' failed to start in attempt(s): 6: java.lang.RuntimeException: java.util.concurrent.ExecutionException: org.testcontainers.containers.ContainerLaunchException: Container startup failed for image mcr.microsoft.com/mssql/server:2017-latest
2024-09-19T10:17:41.626Z	WARN	main	io.trino.tests.product.launcher.cli.TestRun	Failed to execute tests
java.lang.RuntimeException: java.util.concurrent.ExecutionException: org.testcontainers.containers.ContainerLaunchException: Container startup failed for image mcr.microsoft.com/mssql/server:2017-latest

This should not block us from merging this PR.