Skip to content

Fix Google Cloud Batch error messages to include job name#69783

Open
parkhojeong wants to merge 2 commits into
apache:mainfrom
parkhojeong:google/fix-cloud-batch-job-name-message
Open

Fix Google Cloud Batch error messages to include job name#69783
parkhojeong wants to merge 2 commits into
apache:mainfrom
parkhojeong:google/fix-cloud-batch-job-name-message

Conversation

@parkhojeong

@parkhojeong parkhojeong commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Add missing f-string prefixes so Cloud Batch errors display the actual job
name instead of {job_name}.

  • f"Batch job with name {job_name} has failed its execution."
  • f"Unexpected error in the operation: Batch job with name {job_name} is being deleted."

Test: breeze run pytest providers/google/tests/unit/google/cloud/hooks/test_cloud_batch.py -xvs


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

The missing f-string prefixes caused Cloud Batch errors to display the
literal `{job_name}`.
@boring-cyborg boring-cyborg Bot added area:providers provider:google Google (including GCP) related issues labels Jul 12, 2026
@parkhojeong parkhojeong marked this pull request as ready for review July 12, 2026 18:21
@parkhojeong parkhojeong requested a review from shahar1 as a code owner July 12, 2026 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:google Google (including GCP) related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant