Skip to content
This repository was archived by the owner on May 30, 2025. It is now read-only.

Add Support for Observe Only Resources #503

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add Support for Observe Only Resources #503

wants to merge 3 commits into from

Conversation

turkenh
Copy link
Contributor

@turkenh turkenh commented Apr 14, 2023 •
edited
Loading

Description of your changes

This PR adds a flag for Management Policies (a.k.a.) Observe Only resources.

Please note, this PR does not cover representing the full state at status.atProvider which is being tracked with a separate issue: #504

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

  1. Run provider with --enable-management-policies
  2. Create a resource with spec.managementPolicy: ObserveOnly
  3. Verify that it becomes ready & synced after observing.

@turkenh turkenh requested review from negz and Feggah April 14, 2023 07:07
This was referenced Apr 14, 2023
Closed
Open
@chlunde chlunde mentioned this pull request Jun 23, 2023
Merged
2 tasks
@tennix tennix mentioned this pull request Apr 18, 2025
Merged
2 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@negz negz Awaiting requested review from negz

@Feggah Feggah Awaiting requested review from Feggah

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@turkenh