Skip to content

Conversation

@ssicard
Copy link
Contributor

@ssicard ssicard commented Nov 12, 2025

This adds migration tests for the logpush_job resource. Below is a more granular listing of all changes:

  • adds zone level resources to logpush_job sweep
  • adds migration tests for both account-level logpush jobs and zone-level

Note: resource.go was updated for this resource to account for less options for kind (no instant-logs supported)

Acceptance test run results

  • I have run acceptance tests for my changes and included the results below
=== RUN   TestMigrateCloudflareLogpushJob_Migration_Basic_MultiVersion
=== RUN   TestMigrateCloudflareLogpushJob_Migration_Basic_MultiVersion/from_v4_52_1
--- PASS: TestMigrateCloudflareLogpushJob_Migration_Basic_MultiVersion (18.01s)
    --- PASS: TestMigrateCloudflareLogpushJob_Migration_Basic_MultiVersion/from_v4_52_1 (18.01s)
=== RUN   TestMigrateCloudflareLogpushJob_Migration_OutputOptions
--- PASS: TestMigrateCloudflareLogpushJob_Migration_OutputOptions (15.43s)
=== RUN   TestMigrateCloudflareLogpushJob_Migration_InstantLogs
--- PASS: TestMigrateCloudflareLogpushJob_Migration_InstantLogs (16.20s)
PASS
ok      github.com/cloudflare/terraform-provider-cloudflare/internal/services/logpush_job       51.040s

Additional context & links

stainless-app bot and others added 9 commits November 11, 2025 21:26
* feat: add terraform config for universal_ssl_setting

- Add id_property: zone_id to enable ImportState functionality
- Add method: upsert for proper create/update behavior
- Configure edit method with explicit type and terraform properties

This enables the universal_ssl_setting resource to be fully manageable
in Terraform with import support and proper drift detection.
…nes, Streams, Sinks routes

* feat(pipelines): Configure SDKs/Terraform to use the new Pipelines, Streams, Sinks routes
@ssicard ssicard marked this pull request as ready for review November 12, 2025 20:15
@ssicard ssicard self-assigned this Nov 12, 2025
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updates are being made in resource.go to account for an update in kind in logpush_job resource. instant-logs was a valid kind option in v4, but is not valid in v5. The API was adding instant-logs back to the state even though it was not valid, so we explicitly ignore that as a value.

musa-cf and others added 2 commits November 12, 2025 14:23
… Application (#6453)

Adds a new app type for the session duration compatible app types for the zero type access application resource. The newly supported type is proxy_endpoint.
@ssicard ssicard force-pushed the ssicard/logpush-job branch from 51ed600 to 00a15e7 Compare November 12, 2025 21:28
@ssicard
Copy link
Contributor Author

ssicard commented Nov 21, 2025

Closing this PR in favor of #6483 as this got muddled with a lot of other next changes.

@ssicard ssicard closed this Nov 21, 2025
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.