DevOps Community Sponsored by Cloud Posse

SweetOps

The datadog `monitors` submodule is not compatible with the latest release of the DataDog Terraform provider.
<@UCX71U7KJ>
<https://github.com/cloudposse/terraform-datadog-platform/pull/125>

Someone on the team familiar with that one will need to review it

Ah sorry. Saw you were a codeowner/automatically added to the reviewers.

<@UB2EH889X> Would prefer for us not to have to switch our TF to use an internally published fork / pin to DD provider v3 here.  Is there someone I can get this contribution in front of?

LGTM - I marked it as a major release since we're upgrading a non-backward-compatible provider change.

<@UB2EH889X>, we'll probably need to eventually split this module into individual repository modules, so that we can have individual provider management, as opposed to a major release for each subfolders update to the 4.x provider.

<https://github.com/cloudposse/terraform-datadog-platform/pull/125#issuecomment-4008513844>

for posterity

<@UB2EH889X>
<@U0210GHDJQ0>

Looks like one of the checks (test/terratest) stalled on last weeks commit.  Considering <@U0210GHDJQ0>’s comments about the other submodules, I bumped everything else up to v4 because they’re all technically compatible with v4 without any further changes.

Would really like to get this out soon as it’s blocking a few workstreams on our end. We’ll probably have to switch to our fork soon if this can’t get merged in

<@U037B7VBM6Z> looks like we won't be able to get to it this week

I appreciate the contribution, but don't want to get in your way; suggest using your fork for now.

<@UB2EH889X> How about this week? :slightly_smiling_face:

Merged. With the major label tag, I think this is a fine update with the provider change. It was my one concern.