Skip to content

feat(ruleset): add warnings to apply - #7313

Open
jahnavimalhotra1 wants to merge 1 commit into
cloudflare:nextfrom
jahnavimalhotra1:jahnavi/ruleset-dryrun-warnings
Open

jahnavimalhotra1 wants to merge 1 commit into
cloudflare:nextfrom
jahnavimalhotra1:jahnavi/ruleset-dryrun-warnings

Conversation

@jahnavimalhotra1

Copy link
Copy Markdown
Contributor
  • I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested

Currently, warnings in Cloudflare Rulesets are not surfaced when running terraform apply. This change reports any warnings that the Rulesets API returns during the apply phase to the user.

Acceptance test run results

  • I have added or updated acceptance tests for my changes
  • I have run acceptance tests for my changes and included the results below

Steps to run acceptance tests

export TF_ACC=1
export CLOUDFLARE_API_TOKEN=<token>
export CLOUDFLARE_ACCOUNT_ID=<account id>
export CLOUDFLARE_ZONE_ID=<zone id>
go test ./internal/services/ruleset/ -run TestAccCloudflareRuleset_WarningOnUpdate -v

Test output

=== RUN   TestAccCloudflareRuleset_WarningOnUpdate
--- PASS: TestAccCloudflareRuleset_WarningOnUpdate (12.60s)
PASS

Additional context & links

@jahnavimalhotra1
jahnavimalhotra1 changed the base branch from main to next August 18, 2026 15:17
@jahnavimalhotra1
jahnavimalhotra1 requested a review from a team as a code owner August 18, 2026 15:17
@stainless-app
stainless-app Bot force-pushed the next branch 7 times, most recently from bff0e51 to c438411 Compare August 20, 2026 19:34
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.

1 participant