[release test] assign job url when runner exists

[aslonnie]

rather than exit early. this makes sure that the job url and job ids are all assigned even for release test jobs that failed.

[gemini-code-assist]

Code Review

This pull request refactors run_release_test_anyscale to retrieve job information (URL, ID, logs) even when an exception occurs during the test run. This is a valuable change for improving the debuggability of failed tests. The accompanying test updates correctly validate this new behavior. My main feedback is to add error handling around the new block that retrieves job information, as it could fail and cause an unhandled exception, which would be counterproductive to the goal of this PR.

[gemini-code-assist]

While moving this logic out of the main try block is a good idea to ensure job details are captured on failure, this new block is not protected by a try...except. If any of the runner methods (job_url(), job_id(), or get_last_logs()) raise an exception, it will be unhandled and crash the program. This would prevent any further cleanup or reporting. It's safer to wrap this in its own try...except block to gracefully handle potential errors during job info retrieval.

Suggested change

	result.job_url = runner.job_url()
	result.job_id = runner.job_id()
	if result.job_id:
	result.last_logs = runner.get_last_logs()
	try:
	result.job_url = runner.job_url()
	result.job_id = runner.job_id()
	if result.job_id:
	result.last_logs = runner.get_last_logs()
	except Exception as e:
	logger.warning(f'Failed to get job info from runner: {e}')