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

feat: Secrets datasource #3131

Merged
merged 93 commits into from
Oct 18, 2024
Merged

feat: Secrets datasource #3131

merged 93 commits into from
Oct 18, 2024

Conversation

sfc-gh-fbudzynski
Copy link
Collaborator

@sfc-gh-fbudzynski sfc-gh-fbudzynski commented Oct 11, 2024

Changes

  • added datasource for secrets
  • generated documentation for secrets
  • ComposeCheckDestroy function

Test Plan

  • acceptance tests

References

https://docs.snowflake.com/en/sql-reference/sql/create-secret

Copy link

Integration tests failure for 7724d5785ea3fe3a4cf6610ae59597112027e893

pkg/acceptance/check_destroy.go Outdated Show resolved Hide resolved
Copy link

Integration tests success for 734ec99d23ddf7f5da174543ab6234bfea31d05a

@sfc-gh-asawicki sfc-gh-asawicki self-requested a review October 15, 2024 20:45
pkg/acceptance/check_destroy.go Outdated Show resolved Hide resolved
pkg/datasources/secrets.go Outdated Show resolved Hide resolved
pkg/datasources/secrets_acceptance_test.go Outdated Show resolved Hide resolved
Copy link

Integration tests failure for cc046488247c0636794286175152f1b8c1032159

Copy link

Integration tests failure for 3ac5fbd1fb829934a6043610b6cd2a851491d4e0

Base automatically changed from secret-resource to main October 17, 2024 07:23
Copy link

Integration tests failure for da6223fd0636a8acd9ce0a8d2c2eeb0f4df2d0cd

Copy link

Integration tests failure for b0aed22446e1470900ad3141bd4664ec1b7b5b92

Copy link

Integration tests failure for 429071a31afc12af7ae5a99f41a3eebd8a658def

),
},
// In Account
{
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nit: adjust the comment to highlight the filter is account + prefix - or remove prefix from this test, leaving only account param.

@sfc-gh-asawicki sfc-gh-asawicki self-requested a review October 18, 2024 15:06
@sfc-gh-fbudzynski sfc-gh-fbudzynski merged commit 8110138 into main Oct 18, 2024
8 of 9 checks passed
@sfc-gh-fbudzynski sfc-gh-fbudzynski deleted the secrets-datasource branch October 18, 2024 16:11
sfc-gh-jmichalak pushed a commit that referenced this pull request Nov 8, 2024
##
[0.98.0](v0.97.0...v0.98.0)
(2024-11-08)

Feature scope readiness for V1:
[link](https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/v1-preparations/ESSENTIAL_GA_OBJECTS.MD)
([Roadmap
reference](https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/ROADMAP.md#wrap-up-the-functional-scope)).
:exclamation: Migration guide: [v0.97.0 ->
v0.98.0](https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/MIGRATION_GUIDE.md#v0970--v0980)

### 🎉 What's new
- New resources:
- authentication_policy
([#3098](#3098)),
references
[#2880](#2880)
- external_volume
([#3106](#3106)),
partially references
[#2980](#2980)
- stream_on_directory_table
([#3129](#3129))
- stream_on_view
([#3150](#3150))
- primary_connection, secondary_connection
([#3162](#3162))
- secret_with_basic_authentication, secret_with_generic_string,
secret_with_oauth_authorization_code_grant,
secret_with_oauth_client_credentials
([#3110](#3110)),
([#3141](#3141))
- New data sources:
- connections
([#3155](#3155)),
([#3173](#3173))
- secrets
([#3131](#3131))
- Reworked:
- provider configuration hierarchy
([#3166](#3166)),
references
[#1881](#1881),
[#2145](#2145),
[#2925](#2925),
[#2983](#2983),
[#3104](#3104)
- provider configuration fields
([#3152](#3152))
streams data source
([#3151](#3151))
- SDK upgrades:
- Upgrade tag SDK
([#3126](#3126))
- Recreate streams when they are stale
([#3129](#3129))
### 🔧  Misc
- Add object renaming research summary
([#3172](#3172))
- Test support for object renaming
([#3130](#3130)),
([#3147](#3147)),
([#3154](#3154))
- Add tests to issue
[#3117](#3117)
([#3133](#3133))
- New roadmap entry
([#3158](#3158))
- Test more authentication methods
([#3178](#3178))
- Minor fixes
([#3174](#3174))
### 🐛  Bug fixes
- Apply various fixes
([#3176](#3176)),
this addresses BCR 2024_08, references
[#2717](#2717),
[#3005](#3005),
[#3125](#3125),
[#3127](#3127),
[#3153](#3153)
- Connection and secret data sources tests
([#3177](#3177))
- Fix grant import docs
([#3183](#3183)),
resolves
[#3179](https://github.com/Snowflake-Labs/terraform-provider-snowflake/discussions/3179)
- Fix user resource import
([#3181](#3181))
- Handle external type changes in stream resources
([#3164](#3164))
- Do not use OR REPLACE on initial creation in resources with
copy_grants
([#3129](#3129))
- Address issue
[#2201](#2201)
by introducing new stream resources

Co-authored-by: snowflake-release-please[bot] <105954990+snowflake-release-please[bot]@users.noreply.github.com>
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.

4 participants