Skip to content

agents: add major release deprecation and removal workflows - #18671

Open
c2thorn wants to merge 4 commits into
GoogleCloudPlatform:mainfrom
c2thorn:add-major-release-removal-workflows
Open

agents: add major release deprecation and removal workflows#18671
c2thorn wants to merge 4 commits into
GoogleCloudPlatform:mainfrom
c2thorn:add-major-release-removal-workflows

Conversation

@c2thorn

@c2thorn c2thorn commented Aug 17, 2026

Copy link
Copy Markdown
Member

This PR introduces structured workflows and a dedicated subagent for managing major release deprecations and removals in Magic Modules:

  • deprecate_resource_or_field: Handles establishing forward-compatibility and adding deprecation notices on the main branch.
  • major_release_removal: Handles executing breaking removals on a major release feature branch (FEATURE-BRANCH-major-release-X.0.0), updating the upgrade guide, and verifying downstream builds and tests.
  • removal-auditor subagent: Performs pre-flight audits of deprecation/replacement readiness on main, checks major release branch sync status, and scans all codebase layers (YAMLs, templates, samples, tests, TGC converters) to produce a comprehensive removal blast-radius report.
  • Registered workflows in .agents/WORKFLOWS.md (including GitHub label routing) and added references in .agents/knowledge/index.md.

This workflow was successfully used on the following PRs:

  1. PR #18674deprecate google_beyondcorp_app_* resources and data sources (targeting main)
  2. PR #18675beyondcorp: remove deprecated google_beyondcorp_app_* for 8.0.0 (targeting FEATURE-BRANCH-major-release-8.0.0)
  3. PR #18673vertexai: remove deprecated google_vertex_ai_schedule for 8.0.0 (targeting FEATURE-BRANCH-major-release-8.0.0)
  4. PR #18676remove deprecated interconnect_attachment_group.attachment (targeting FEATURE-BRANCH-major-release-8.0.0)
  5. PR #18679iap: remove deprecated google_iap_brand and google_iap_client (targeting FEATURE-BRANCH-major-release-8.0.0)
  6. PR #18680integrations: remove deprecated run_as_service_account (targeting FEATURE-BRANCH-major-release-8.0.0)
  7. PR #18681mlengine: remove deprecated google_ml_engine_model for 8.0.0 (targeting FEATURE-BRANCH-major-release-8.0.0)

@modular-magician

Copy link
Copy Markdown
Collaborator

Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 708e45b:

Diff report

Your PR hasn't generated any diffs, but I'll let you know if a future commit does.

@modular-magician

Copy link
Copy Markdown
Collaborator

Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 0f0850c:

Diff report

Your PR hasn't generated any diffs, but I'll let you know if a future commit does.

@c2thorn

c2thorn commented Aug 17, 2026

Copy link
Copy Markdown
Member Author

@modular-magician reassign-reviewer

@github-actions

Copy link
Copy Markdown

Googlers: For automatic test runs see go/terraform-auto-test-runs.

@rileykarson, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look.

You can help make sure that review is quick by doing a self-review and by running impacted tests locally.

@github-actions
github-actions Bot requested a review from rileykarson August 17, 2026 22:22
custom_agent:
system_prompt_sections:
- title: "Role & Persona"
content: "You are a Senior Terraform Provider Engineer specializing in major release scoping, deprecation verification, and removal dependency analysis in Magic Modules."

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Are role assignments still effective? I thought current gen models made them mostly superfluous.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

you're correct, the are largely superfluous. it was just picked up since our older subagent specs have them.

Comment thread .agents/agents/removal-auditor/config.yaml Outdated
@modular-magician

Copy link
Copy Markdown
Collaborator

Hi there, I'm the Modular magician. I've detected the following information about your changes for commit a22c2d6:

Diff report

Your PR hasn't generated any diffs, but I'll let you know if a future commit does.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants