-
Notifications
You must be signed in to change notification settings - Fork 2.4k
docs: add DataHub MCP server extension documentation #5769
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds comprehensive documentation for the DataHub MCP server extension, enabling users to integrate DataHub's data discovery and metadata platform capabilities with goose.
Key changes:
- Adds DataHub MCP server extension to the servers registry
- Creates tutorial documentation with configuration examples for both CLI and Desktop interfaces
- Includes prerequisites, capabilities, example usage scenarios, and troubleshooting guidance
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
documentation/static/servers.json |
Registers DataHub extension with required environment variables (DATAHUB_GMS_URL and DATAHUB_GMS_TOKEN) and minor whitespace cleanup for other entries |
documentation/docs/mcp/datahub-mcp.mdx |
Comprehensive tutorial covering DataHub integration, including installation instructions, configuration examples, use cases for data discovery/lineage/queries, and troubleshooting tips |
|
|
||
| :::note Desktop | ||
|
|
||
| The DataHub extension will search across your data catalog and return relevant datasets with their metadata, including: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If possible, an actual example response (or snippet) would go in these :::note blocks (but only h4 or lower if you include formatting)
|
Will be taking a look today! Thank you Sam! |
jjoyce0510
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Heads up -- We are just about to add a few new tools to the MCP server (e.g. acryldata/mcp-server-datahub#55) so I am making some suggestions to include coverage.
I'm also going to provide some suggestions about how to improve storytelling + use case examples..
|
One final question: Is there a place to define recipes that are shown within Goose specifically? Or does that happen automagically? |
we have a recipe cookbook here if you want the general public to access your recipes and have it listed on our site -- https://block.github.io/goose/recipes instructions to do so here: https://github.com/block/goose/blob/main/CONTRIBUTING_RECIPES.md but other recipes are typically only accessible on your local machine or if you configured/pointed them to a specific repo @jjoyce0510 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
3ff34e0 to
a45ff7b
Compare
- Add comprehensive tutorial at documentation/docs/mcp/datahub-mcp.mdx - Register extension in documentation/static/servers.json - Include CLI and Desktop configuration examples - Document prerequisites, capabilities, and troubleshooting - Add realistic scenario-based usage examples Signed-off-by: Sam Osborn <[email protected]>
a45ff7b to
111961d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
* main: docs: add DataHub MCP server extension documentation (#5769) docs: lowercase goose in remaining topics (#5861) docs: lowercase goose in getting-started and guides topics (#5857) Fix multi tool calling (#5855) fix(#5626 #5832): handle multiple content chunks & images better (#5839) chore: some old code hanging around, and mention configure cli (#5822) feat : add support for math / science symbology via katex (#5773) feat : add ability to see error message in toast (#5851)
Signed-off-by: Sam Osborn <[email protected]>
Signed-off-by: Sam Osborn <[email protected]> Signed-off-by: Douwe Osinga <[email protected]>
…ed-context-2 * upstream/main: Move recipe actions to bottom bar icon and edit goosehints to settings (block#5864) [docs] Add “Building a Social Media Agent” Blog Post (block#5844) deps: upgrade rmcp to 0.9.1 (block#5860) chore: suggest using text/markdown when fetching content (block#5854) Revert "fix: do not load active extensions when no extensions in the recipe" (block#5871) goose remote access (block#5251) docs: add DataHub MCP server extension documentation (block#5769) # Conflicts: # ui/desktop/src/components/ChatInput.tsx
* main: fix: adjust strange spacing in agent.rs (#5877) Move recipe actions to bottom bar icon and edit goosehints to settings (#5864) [docs] Add “Building a Social Media Agent” Blog Post (#5844) deps: upgrade rmcp to 0.9.1 (#5860) chore: suggest using text/markdown when fetching content (#5854) Revert "fix: do not load active extensions when no extensions in the recipe" (#5871) goose remote access (#5251) docs: add DataHub MCP server extension documentation (#5769)
Signed-off-by: Sam Osborn <[email protected]> Signed-off-by: Sai Karthik <[email protected]>
Signed-off-by: Sam Osborn <[email protected]> Signed-off-by: Blair Allan <[email protected]>
Summary
Type of Change
AI Assistance
Testing
Tested by adding extension locally, following the new docs, and queried our datahub instance successfully.