Skip to content

Update/Fix: fix-70296-cloudbatch - #70770

Closed
saitejabandaru-in wants to merge 2 commits into
apache:mainfrom
saitejabandaru-in:fix-70296-cloudbatch
Closed

Update/Fix: fix-70296-cloudbatch#70770
saitejabandaru-in wants to merge 2 commits into
apache:mainfrom
saitejabandaru-in:fix-70296-cloudbatch

Conversation

@saitejabandaru-in

Copy link
Copy Markdown

Automated Pull Request to address fix-70296-cloudbatch. Please review.

@boring-cyborg boring-cyborg Bot added area:providers provider:google Google (including GCP) related issues provider:standard labels Jul 30, 2026

@amoghrajesh amoghrajesh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rigger_dagrun.py carries the same stale commit from #70769, run_after isn't a template field, #70490 already handled logical_date and deliberately left run_after alone. drop that commit.

cloud_batch.py - good catch on protobuf vs Jinja, but render_template_fields() only runs at execute time on a worker, not during dag parsing/serialization. so a raw Job would now serialize as a stringified repr instead of a dict. Cannot proceed in current state.

@amoghrajesh

Copy link
Copy Markdown
Contributor

Please follow contributing guidelines of the repo and do not use AI carelessly.

@saitejabandaru-in

Copy link
Copy Markdown
Author

I sincerely apologize for the automated pull request. It was triggered by an experimental agent framework testing open source contribution pipelines, and I completely missed that it violated the DCO and AI contribution guidelines for this repository. I am closing this PR immediately and have disabled the automation. Thank you for your patience and sorry for the noise!

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 provider:standard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants