Skip to content

Snowflake Apps: resolve deploy defaults from config table#2834

Draft
sfc-gh-gbloom wants to merge 18 commits intomainfrom
gbloom/snow-apps-remote-config
Draft

Snowflake Apps: resolve deploy defaults from config table#2834
sfc-gh-gbloom wants to merge 18 commits intomainfrom
gbloom/snow-apps-remote-config

Conversation

@sfc-gh-gbloom
Copy link
Copy Markdown
Contributor

@sfc-gh-gbloom sfc-gh-gbloom commented Mar 26, 2026

Pre-review checklist

  • I've confirmed that instructions included in README.md are still correct after my changes in the codebase.
  • I've added or updated automated unit tests to verify correctness of my new code.
  • I've added or updated integration tests to verify correctness of my new code.
  • I've confirmed that my changes are working by executing CLI's commands manually on MacOS.
  • I've confirmed that my changes are working by executing CLI's commands manually on Windows.
  • I've confirmed that my changes are up-to-date with the target branch.
  • I've described my changes in the release notes.
  • I've described my changes in the section below.
  • I've described my changes in the documentation.

Changes description

snow __app deploy now reads fallback defaults from SNOW_APPS.CONFIG.APP_DEFAULTS (keyed by role) when values like warehouse, compute_pool, and eai aren't set in snowflake.yml. It's valid for the table to be missing, empty, or inaccessible.

Resolution order:

  1. snowflake.yml
  2. config table
  3. built-in defaults

@sfc-gh-gbloom sfc-gh-gbloom marked this pull request as ready for review March 26, 2026 14:05
@sfc-gh-gbloom sfc-gh-gbloom requested a review from a team as a code owner March 26, 2026 14:05
@sfc-gh-gbloom sfc-gh-gbloom marked this pull request as draft March 26, 2026 15:20
@sfc-gh-gbloom sfc-gh-gbloom force-pushed the gbloom/snow-apps-remote-config branch from 7907b9e to 1bcf2f7 Compare March 27, 2026 14:00
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