Skip to content

Add Snowflake post-start hints#227

Merged
carole-lavillonniere merged 1 commit intomainfrom
drg-805-improve-snowflake-cli-custom-port-and-status-ux-2
May 6, 2026
Merged

Add Snowflake post-start hints#227
carole-lavillonniere merged 1 commit intomainfrom
drg-805-improve-snowflake-cli-custom-port-and-status-ux-2

Conversation

@carole-lavillonniere
Copy link
Copy Markdown
Collaborator

@carole-lavillonniere carole-lavillonniere commented May 6, 2026

Motivation

Snowflake users see no post-start hints today (endpoint, tip). The bare • Endpoint: line printed for AWS is misleading for Snowflake — Snowflake clients need the snowflake.<host> form to route through the gateway.

Changes

  • New internal/emulator/snowflake package with Endpoint(resolvedHost) that builds http://snowflake.<host>; returns "" when the host is an IP (subdomain on an IP is invalid).
  • For Snowflake, replace • Endpoint: with • Snowflake endpoint: http://snowflake.<host>; fall back to the bare endpoint when the snowflake-prefixed form is invalid (DNS rebind / IP host).
  • Show a randomized > Tip: line for Snowflake too (previously AWS only).
  • runPostStartSetups now emits post-start pointers for every unique emulator type, not only types with a registered setup function — so Snowflake gets pointers without needing a setup hook.

Towards DRG-805

image

@carole-lavillonniere carole-lavillonniere changed the title Add Snowflake-specific runtime hints Add Snowflake post-start hints May 6, 2026
@carole-lavillonniere carole-lavillonniere marked this pull request as ready for review May 6, 2026 10:04
Copy link
Copy Markdown
Contributor

@anisaoshafi anisaoshafi left a comment

Choose a reason for hiding this comment

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

Tested and looks good 🚀

@carole-lavillonniere carole-lavillonniere merged commit 2be87e4 into main May 6, 2026
12 of 13 checks passed
@carole-lavillonniere carole-lavillonniere deleted the drg-805-improve-snowflake-cli-custom-port-and-status-ux-2 branch May 6, 2026 10:52
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