Skip to content

docs(duckdb): add S3-compatible endpoint secret example - #5934

Draft
goanpeca wants to merge 1 commit into
SQLMesh:mainfrom
goanpeca:docs/duckdb-s3-compatible-endpoint
Draft

docs(duckdb): add S3-compatible endpoint secret example#5934
goanpeca wants to merge 1 commit into
SQLMesh:mainfrom
goanpeca:docs/duckdb-s3-compatible-endpoint

Conversation

@goanpeca

Copy link
Copy Markdown

Description

The DuckDB connection's secrets option passes every field straight through to CREATE OR REPLACE SECRET, so DuckDB's endpoint and url_style S3 secret parameters already work today without any code change. The two existing examples only show the Amazon S3 defaults, so nothing on the page tells users that pointing an s3 secret at a different S3 host is supported. The parameter list is only reachable via the DuckDB link at the bottom of the section.

This adds a third short example under "Cloud service authentication", alongside the list-format and dictionary-format examples, showing an s3 secret with a custom endpoint (and a commented url_style for providers that need path-style addressing).

Docs only, no code changes.

Test Plan

  • Built the docs with mkdocs build using docs/requirements.txt and confirmed the new tabbed section renders and appears in the page table of contents.
  • Parsed the new YAML block with yaml.safe_load to confirm it is valid.
  • Confirmed DuckDB accepts endpoint and url_style in the exact CREATE OR REPLACE SECRET (type 's3', ...) statement that BaseDuckDBConnectionConfig._cursor_init generates, so the documented fields are real rather than aspirational.

Checklist

  • I have run make style and fixed any issues
  • I have added tests for my changes (if applicable)
  • All existing tests pass (make fast-test)
  • My commits are signed off (git commit -s) per the DCO

Signed-off-by: Gonzalo Peña-Castellanos <goanpeca@gmail.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.

1 participant