Skip to content

[New Resource] Support for Assured Workloads - #12457

Open
acpana wants to merge 2 commits into
GoogleCloudPlatform:masterfrom
acpana:issue-6650-iPuO
Open

[New Resource] Support for Assured Workloads#12457
acpana wants to merge 2 commits into
GoogleCloudPlatform:masterfrom
acpana:issue-6650-iPuO

Conversation

@acpana

@acpana acpana commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

This PR adds support for the AssuredWorkloadsWorkload resource in Config Connector using the direct controller pattern.

It includes:

  • Direct controller implementation for AssuredWorkloadsWorkload.
  • Updated KRM types and regenerated mappers.
  • MockGCP service implementation for Assured Workloads.
  • E2E test fixture for basic workload creation and update.

Fixes #6650.

Overseer powered by gemini-3-flash-preview.

@google-oss-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from acpana. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@acpana

acpana commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator Author

I have addressed the issues and cleaned up the PR history.

Key changes:

  • Rebased on master to clean up the diff (removed unrelated generated files).
  • Fixed the AssuredWorkloadsWorkload implementation which was incomplete and used non-existent helper functions.
  • Added missing OrganizationRef and Location fields to the Spec.
  • Implemented AssuredWorkloadsWorkloadIdentity and GetIdentity method.
  • Refactored the controller to use standard direct controller patterns (common.CompareProtoMessage, common.BasicDiff).
  • Implemented AdapterForURL and fixed Export in the controller.
  • Fixed a bug in MockGCP normalization for Assured Workloads.

(This comment was generated by Overseer)

@acpana
acpana marked this pull request as ready for review August 19, 2026 00:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[New Resource] Support for Assured Workloads

1 participant