Skip to content

fix: Correct Snowflake env var prefix and secret reference syntax#1673

Merged
lukekim merged 1 commit into
trunkfrom
fix/snowflake-env-var-prefix-and-secret-syntax
May 7, 2026
Merged

fix: Correct Snowflake env var prefix and secret reference syntax#1673
lukekim merged 1 commit into
trunkfrom
fix/snowflake-env-var-prefix-and-secret-syntax

Conversation

@claudespice
Copy link
Copy Markdown
Collaborator

Summary

  • Fix env var prefix in Snowflake connector docs: SPICE_SECRET_SNOWFLAKE_* -> SPICE_SNOWFLAKE_* (the env secret store uses SPICE_ as its prefix, not SPICE_SECRET_; confirmed in spiceai/spiceai crates/runtime-secrets/src/stores/env.rs and the spice login CLI output)
  • Fix secret reference syntax in Kubernetes spicepod YAML examples for versions 1.5.x through 1.10.x: ${snowflake.key} (dot) -> ${snowflake:key} (colon), matching the correct secret store reference syntax

Files changed

  • website/docs/components/data-connectors/snowflake.md (vNext) -- env var prefix fix
  • website/versioned_docs/version-1.5.x/components/data-connectors/snowflake.md -- env var prefix + secret syntax fix
  • website/versioned_docs/version-1.6.x/components/data-connectors/snowflake.md -- env var prefix + secret syntax fix
  • website/versioned_docs/version-1.7.x/components/data-connectors/snowflake.md -- env var prefix + secret syntax fix
  • website/versioned_docs/version-1.8.x/components/data-connectors/snowflake.md -- env var prefix + secret syntax fix
  • website/versioned_docs/version-1.9.x/components/data-connectors/snowflake.md -- env var prefix + secret syntax fix
  • website/versioned_docs/version-1.10.x/components/data-connectors/snowflake.md -- env var prefix + secret syntax fix

Test plan

  • Verify SPICE_SECRET_SNOWFLAKE_* no longer appears in any Snowflake connector doc
  • Verify ${snowflake. (dot syntax) no longer appears in any versioned Snowflake connector doc
  • Confirm the corrected env vars match what spice login snowflake writes to .env
  • Confirm the corrected secret syntax ${snowflake:key} matches the secret store reference format

The env secret store uses SPICE_ as its prefix, not SPICE_SECRET_.
Fix all Snowflake connector docs to use SPICE_SNOWFLAKE_* instead of
SPICE_SECRET_SNOWFLAKE_* for environment variable examples.

Also fix the Kubernetes spicepod YAML examples in versions 1.5.x
through 1.10.x to use colon syntax ${snowflake:key} instead of
incorrect dot syntax ${snowflake.key} for secret references.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

✅ Pull with Spice Passed

Passing checks:

  • ✅ Title meets minimum length requirement (10 characters)
  • ✅ Has at least one of the required labels: area/blog, area/docs, area/cookbook, dependencies
  • ✅ No banned labels detected
  • ✅ Has at least one assignee: claudespice

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

🔍 Pull with Spice Failed

Passing checks:

  • ✅ Title meets minimum length requirement (10 characters)
  • ✅ Has at least one of the required labels: area/blog, area/docs, area/cookbook, dependencies
  • ✅ No banned labels detected

Failed checks:

  • ❌ At least one assignee is required for this pull request.

Please address these issues and update your pull request.

1 similar comment
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

🔍 Pull with Spice Failed

Passing checks:

  • ✅ Title meets minimum length requirement (10 characters)
  • ✅ Has at least one of the required labels: area/blog, area/docs, area/cookbook, dependencies
  • ✅ No banned labels detected

Failed checks:

  • ❌ At least one assignee is required for this pull request.

Please address these issues and update your pull request.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

🚀 deployed to https://3e25188e.spiceai-org-website.pages.dev

@lukekim lukekim merged commit 971ac27 into trunk May 7, 2026
6 of 10 checks passed
@lukekim lukekim deleted the fix/snowflake-env-var-prefix-and-secret-syntax branch May 7, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants