-
Notifications
You must be signed in to change notification settings - Fork 54
feat!: deprecate cloudsec resources #671
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@airadier @mateobur @tembleking - looks like I need your reviews as codeowner |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.