Skip to content

Conversation

@kidehen
Copy link

@kidehen kidehen commented May 29, 2025

Description

Added the OpenLink generic MCP servers for the following to the Community Servers collection, integrated in alphabetical order.

Moved new additions to the Community section and ordered alphabetically.

Server Details

Motivation and Context

List enrichment.

How Has This Been Tested?

Via our normal product QA and release process across all major DBMS platforms that we support e.g.. Oracle, Microsoft SQL Server / Sybase, Informix, Progress 4GL / OpenEdge, DB2, MySQL, and PostgreSQL. This also includes our ODBC to JDBC and JDBC to ODBC bridges.

Breaking Changes

This is simply a list addition. Existing users of these MCP servers will not need to change anything.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • [X ] Documentation update

Checklist

  • I have read the MCP Protocol Documentation
  • My changes follows MCP security best practices
  • I have updated the server's README accordingly
  • I have tested this with an LLM client
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have documented all environment variables and configuration options

kidehen added 2 commits May 29, 2025 09:11
Moved new additions to the Community section and ordered alphabetically.
Typo fix in generic pyODBC mcp server url
Copy link
Member

@olaservo olaservo left a comment

Choose a reason for hiding this comment

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

Hi, thanks for the PR. Could you remove the other unrelated changes? Thanks!

@olaservo olaservo added the waiting for submitter Waiting for the submitter to provide more info label Jul 18, 2025
@kidehen
Copy link
Author

kidehen commented Jul 18, 2025

This approach isn't going to work, since by the time you get to review my PR the sorting order would have changed. How do I simply make a submission of our MCP servers for you to incorporate?

@olaservo
Copy link
Member

Hi @kidehen, we've been experimenting with bypassing the need to sort and alphabetize in each PR by aggregating multiple additions like this one: #2370 (Doing it one by one was leading to more chances for conflicts, etc.)

We hadn't updated the PR template yet since we were still refining the process and seeing some PRs getting missed, but we will clarify that in the template for future PRs. Thanks!

@kidehen
Copy link
Author

kidehen commented Jul 22, 2025

Hi @kidehen, we've been experimenting with bypassing the need to sort and alphabetize in each PR by aggregating multiple additions like this one: #2370 (Doing it one by one was leading to more chances for conflicts, etc.)

We hadn't updated the PR template yet since we were still refining the process and seeing some PRs getting missed, but we will clarify that in the template for future PRs. Thanks!

Okay, thanks for the reply. At this juncture, what's the best cause of action for adding our MCP servers?

@TallTed
Copy link
Contributor

TallTed commented Jul 22, 2025

@kidehen @olaservo — I think all merge conflicts are resolved and this is ready to merge into https://github.com/modelcontextprotocol/servers/tree/main.

Please let me know if there are remaining issues.

@olaservo
Copy link
Member

olaservo commented Aug 7, 2025

Thanks for your contribution to the servers list. This has been merged in this combined PR: #2475

This is a new process we're trying out, so if you see any issues feel free to re-open the PR and tag me.

@olaservo olaservo closed this Aug 7, 2025
@TallTed TallTed deleted the fix/20250727-TallTed-sync-to-modelcontextprotocol_main branch August 8, 2025 15:23
@TallTed TallTed restored the fix/20250727-TallTed-sync-to-modelcontextprotocol_main branch August 8, 2025 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting for submitter Waiting for the submitter to provide more info

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants