crossplane-contrib / provider-jet-datadog

@crossplane provider to drive @datadog configuration control plane way
Apache License 2.0
6 stars 8 forks source link

feat: added authnmapping resource #17

Closed Kerwood closed 9 months ago

Kerwood commented 2 years ago

Signed-off-by: Patrick Kerwood patrick@kerwood.dk

Description of your changes

Added support for SAML group mappings.

This PR is currently a draft because of a bug in the DataDog Go library described here: https://github.com/DataDog/terraform-provider-datadog/issues/1335 A fix to this would be to upgrade the datadog Terraform provider from 3.12.0 to 3.14.0, but that is not possible because of the synthetics resource having a sensitive integer field which is currently not supported be terrajet. An approval of this PR will fix that. https://github.com/crossplane/terrajet/pull/290

I have:

How has this code been tested

With love ❤️.. And a local k8s cluster creating the group mapping in DD.

ytsarev commented 1 year ago

@Kerwood sorry, I am not actively monitoring this repo. Could you resolve the conflicts? After the force-push could you please ping me for the merge ? :)