Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix example configuration that uses an unsupported managementPolicy #569

Merged

Conversation

mergenci
Copy link
Member

I added "Observe" management policy to "Create" and "Delete" policies, because it is required in all configurations except when pausing a resource.

v1.11 and v1.12 don't have this updated example, so they didn't change.

"Observe" management policy is required in all configurations except
when pausing a resource.

Signed-off-by: Cem Mergenci <cmergenci@gmail.com>
@netlify
Copy link

netlify bot commented Oct 16, 2023

Deploy Preview for crossplane ready!

Name Link
🔨 Latest commit ca05dc5
🔍 Latest deploy log https://app.netlify.com/sites/crossplane/deploys/652ce6492f38dc000840471b
😎 Deploy Preview https://deploy-preview-569--crossplane.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance:
Accessibility:
Best Practices:
SEO:
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@mergenci mergenci changed the title Fix example configuration that uses an unsupported managementPolicy. Fix example configuration that uses an unsupported managementPolicy Oct 16, 2023
@plumbis plumbis merged commit fc1a414 into crossplane:master Oct 16, 2023
7 checks passed
@mergenci mergenci deleted the fix-unsupported-management-policy branch October 17, 2023 10:12
plumbis pushed a commit to plumbis/crossplane-docs that referenced this pull request Oct 31, 2023
@mergenci mergenci mentioned this pull request Feb 26, 2024
6 tasks
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.

2 participants