feat!: deprecate cloudsec resources#671
Conversation
|
@airadier @mateobur @tembleking - looks like I need your reviews as codeowner |
tembleking
left a comment
There was a problem hiding this comment.
Blocking the merge of this PR since this is removing a resource without a previous deprecation and replacement.
There was a problem hiding this comment.
We cannot just remove resources from the provider, because those may be in use by some customer. First you need to mark it as deprecated pointing to a replacement and then ensure no one else is using it.
There was a problem hiding this comment.
You can use the field DeprecationMessage within schema.Resource like:
DeprecationMessage: "This resource is deprecated. Please use X resource instead.",That will mark the resource as deprecated.
|
Deprecation of This PR deprecates the
Impact and Solution for Potential Issues: For any really old customer installations that attempt to upgrade their provider and encounter issues due to this deprecation, the expectation is to follow the documentation provided in the PS support steps: Terraform Sysdig Provider – Deprecation of sysdig_secure_cloud_account resource (v2.x Breaking Change). The recommended solutions are:
Given that current snippets point to older versions, users upgrading will be moving up two major versions, making breaking changes expected. The ultimate solution for these users is to migrate to the new onboarding process, as the deprecated methods will no longer be supported. |
This pull request deprecates the resource responsible for making
cloudseccalls in the Sysdig Terraform provider.Specifically, it removes the
sysdig_secure_cloud_accountresource and its related implementation, tests, and documentation. The changes include deleting the resource file, its test file, and the corresponding documentation, as well as cleaning up references in the provider’s registration and internal client code. This is part of a broader effort to phase out legacycloudsecresource management from the provider.