Skip to content

fix: set upper bound on datadog provider to < 4.0.0#104

Merged
goruha merged 1 commit into
cloudposse-terraform-components:mainfrom
wavemoran:fix/set-upper-bound-datadog-provider
Apr 7, 2026
Merged

fix: set upper bound on datadog provider to < 4.0.0#104
goruha merged 1 commit into
cloudposse-terraform-components:mainfrom
wavemoran:fix/set-upper-bound-datadog-provider

Conversation

@wavemoran

@wavemoran wavemoran commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Pins the datadog/datadog provider to >= 3.3.0, < 4.0.0 to prevent pulling v4.x which removed datadog_integration_aws_lambda_arn and datadog_integration_aws_log_collection resource types
  • Follows up on chore: remove aws provider v6 upper bound #103 which removed the AWS provider upper bound — that change allowed the latest datadog provider to be pulled, exposing this breaking change

Context

The datadog/datadog provider v4.0.0 removed several resource types used by this component:

  • datadog_integration_aws_lambda_arn
  • datadog_integration_aws_log_collection

This causes terraform plan/apply to fail with Invalid resource type errors on the v2.0.0 release.

Test plan

  • terraform init resolves datadog provider to a 3.x version
  • terraform plan no longer errors on the removed resource types

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated Datadog Terraform provider version constraint to maintain compatibility with the 3.x release line.

The datadog/datadog provider v4.0.0 removed the
`datadog_integration_aws_lambda_arn` and
`datadog_integration_aws_log_collection` resource types, breaking
this component. Pin to < 4.0.0 until the component is updated to
use the replacement resources.

Follows up on cloudposse-terraform-components#103 which removed the AWS provider upper bound.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Apr 7, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d6d3e7b5-1dde-4ae2-a30c-6b158631758b

📥 Commits

Reviewing files that changed from the base of the PR and between 60620d6 and d925332.

📒 Files selected for processing (1)
  • src/versions.tf

📝 Walkthrough

Walkthrough

Updated the Terraform datadog provider version constraint in src/versions.tf to exclude major version 4 and above, narrowing the allowed provider versions from >= 3.3.0 to >= 3.3.0, < 4.0.0.

Changes

Cohort / File(s) Summary
Datadog Provider Version Constraint
src/versions.tf
Restricted datadog provider version range to exclude 4.x versions and above, establishing an upper bound at < 4.0.0.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

needs-test, triage

Poem

🐰 A version constraint, so neat and tight,
Four-oh is barred from this Terraform sight,
Three-point-three onwards, but not beyond,
We hop along with the versions we're fond,
Stability locked, our code stays secure! 🔐

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and accurately describes the main change: setting an upper bound on the datadog provider version constraint to exclude v4.0.0 and above.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mergify mergify Bot requested review from a team April 7, 2026 17:49
@mergify mergify Bot added the triage Needs triage label Apr 7, 2026
@goruha goruha added this pull request to the merge queue Apr 7, 2026
@mergify mergify Bot removed the triage Needs triage label Apr 7, 2026
Merged via the queue into cloudposse-terraform-components:main with commit 8d91427 Apr 7, 2026
14 checks passed
@github-actions

github-actions Bot commented Apr 7, 2026

Copy link
Copy Markdown

These changes were released in v2.1.0.

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