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

Enable management policies #28

Closed
MrVinkel opened this issue Aug 30, 2023 · 3 comments · Fixed by #29
Closed

Enable management policies #28

MrVinkel opened this issue Aug 30, 2023 · 3 comments · Fixed by #29
Labels
enhancement New feature or request

Comments

@MrVinkel
Copy link

What problem are you facing?

I'm trying to use the initProvider and managementPolicies alpha features, but they are currently not supported by the pagerduty provider. I know it is an alpha feature, but as far as I can see it will promoted to a beta feature in crossplane v1.14.

Currently it is very confusing because the pagerduty provider logs that the feature is enable at startup, but then logs it is not enabled when trying to use it:

2023-08-29T10:22:07Z    INFO    provider-pagerduty    Alpha feature enabled    {"flag": "EnableAlphaManagementPolicies"} 
2023-08-29T10:22:08Z    DEBUG    events    `spec.managementPolicies` is set to a non-default value but the feature is not enabled: [Observe Create] 

Are there any plans to support this feature?

How could PagerDuty Provider help solve your problem?

The Upjet generator already supports management policies

@MrVinkel MrVinkel added the enhancement New feature or request label Aug 30, 2023
@haarchri
Copy link
Member

let me check this - we added latest upjet version for managementPolicies and the feature flag in v0.5.0

@haarchri
Copy link
Member

@MrVinkel can you try v0.6.1 ?

@MrVinkel
Copy link
Author

I have tested it and it works now. Thank you for the fast response :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants