Skip to content

An example of a bundle that creates Lakebase database instance and catalog#99

Merged
anton-107 merged 4 commits intomainfrom
anton-107/lakebase-catalog-example
Aug 25, 2025
Merged

An example of a bundle that creates Lakebase database instance and catalog#99
anton-107 merged 4 commits intomainfrom
anton-107/lakebase-catalog-example

Conversation

@anton-107
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

@juliacrawf-db juliacrawf-db left a comment

Choose a reason for hiding this comment

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

A few minor comments

Huge favor, can you fix the dogfood instances in bundle-examples in this PR? We shouldn't be exposing these URLs (there may be others):
https://github.com/databricks/bundle-examples/blob/main/knowledge_base/development_cluster/databricks.yml#L8
https://github.com/databricks/bundle-examples/blob/main/knowledge_base/job_with_run_job_tasks/databricks.yml#L5

@anton-107
Copy link
Copy Markdown
Contributor Author

A few minor comments

Huge favor, can you fix the dogfood instances in bundle-examples in this PR? We shouldn't be exposing these URLs (there may be others): https://github.com/databricks/bundle-examples/blob/main/knowledge_base/development_cluster/databricks.yml#L8 https://github.com/databricks/bundle-examples/blob/main/knowledge_base/job_with_run_job_tasks/databricks.yml#L5

done in a separate PR: #101

Copy link
Copy Markdown
Contributor

@shreyas-goenka shreyas-goenka left a comment

Choose a reason for hiding this comment

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

LGTM, only one comment.

@anton-107 anton-107 merged commit 0344144 into main Aug 25, 2025
1 check passed
@anton-107 anton-107 deleted the anton-107/lakebase-catalog-example branch August 25, 2025 12:50
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.

3 participants