confluentinc / terraform-provider-confluent

Terraform Provider for Confluent
Apache License 2.0
31 stars 64 forks source link

[CLI-3313] Resume Flink statement along with principal and/or compute_pool ID update #488

Closed channingdong closed 1 day ago

channingdong commented 6 days ago

Summary:

This PR supports the feature of resume the Flink statement with a different principal/compute_pool.

Namely when stopped parameter has change from true -> false, we can update the principal and/or compute_pool in place and avoid recreating the Flink statement resource.

Code changes:

Reference:

This PR also updates the Flink statement status type according to the latest DML flow here: https://confluentinc.atlassian.net/wiki/spaces/FLINK/pages/3420389383/Refine+the+Customer+Facing+Statement+Status#Finite-State-Machine-Definitions-and-Logic-for-Flink-in-Confluent-Cloud

The whole verification for Terraform is done and documented in the docs below.

Testing Done:

https://docs.google.com/document/d/1dutVZmbEwJBBqMzx57uCXqllV1SEr2vxnjUrtTPCwBk/edit?usp=sharing

confluent-cla-assistant[bot] commented 6 days ago

:tada: All Contributor License Agreements have been signed. Ready to merge.
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.