Skip to content
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

[APIT-2476] Terraform provider does not work well when deploying a Flink Model/Statement that uses sql.secrets.* #530

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

tmalikconfluent
Copy link
Contributor

@tmalikconfluent tmalikconfluent commented Jan 22, 2025

Release Notes

New Features

  • [Briefly describe new features introduced in this PR].

Bug Fixes

  • Support sensitive properties for Flink statements.

Examples

  • [Briefly describe any Terraform configuration example updates in this PR].

Checklist

  • I can successfully build and use a custom Terraform provider binary for Confluent.
  • I have verified my PR with real Confluent Cloud resources in a pre-prod or production environment, or both.
  • I have attached manual Terraform verification results or screenshots in the Test & Review section below.
  • I have included appropriate Terraform acceptance or unit tests for any new resource, data source, or functionality.
  • I confirm that this PR introduces no breaking changes or backward compatibility issues.
  • I have updated the corresponding documentation and include relevant examples for this PR.
  • I have indicated the potential customer impact if something goes wrong in the Blast Radius section below.
  • I have put checkmarks below confirming that the feature associated with this PR is enabled in:
    • Confluent Cloud prod
    • Confluent Cloud stag
    • Check this box if the feature is enabled for certain organizations only

What

Support sensitive properties for Flink statements.

Blast Radius

All customers using flink statement, specially sensitive properties

References

#397

Test & Review

Added integration tests.
Manual testing Doc: https://docs.google.com/document/d/1RCqKlMJ7Oy_h5dyeeNHKy4iN2nRSNu_22I504xpKGMg/edit?tab=t.0

tmalikconfluent and others added 3 commits January 22, 2025 10:56
…ink Model/Statement that uses sql.secrets.* (#503)

* [APIT-2476] Terraform provider does not work well when deploying a Flink Model/Statement that uses sql.secrets.*

* Nit
@tmalikconfluent tmalikconfluent changed the title [APIT-2476] Terraform provider does not work well when deploying a Fl… [APIT-2476] Terraform provider does not work well when deploying a Flink Model/Statement that uses sql.secrets.* Jan 23, 2025
@tmalikconfluent tmalikconfluent marked this pull request as ready for review January 23, 2025 20:35
@tmalikconfluent tmalikconfluent requested a review from a team as a code owner January 23, 2025 20:35
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