Skip to content

Add entry for hoverfly mcp server #38

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

kapishmalik
Copy link

Add entry for hoverfly mcp server

@kapishmalik kapishmalik requested a review from a team as a code owner July 2, 2025 22:21
@kapishmalik
Copy link
Author

Hey just wanted to check @nunocoracao by when we can expect PR to be merged.

@ipedrazas
Copy link
Contributor

Thank you for your contribution.
Could you please add this config to your server.yaml file to mount the volume for your MCP Server:

run:
  volumes:
    - '{{hoverfly-mcp-server.data}}:/opt/hoverfly-mcp/simulation-data'
config:
  description: The MCP server is allowed to access this path
  parameters:
    type: object
    properties:
      data:
        type: string
        default:
          $HOME/data
    required:
      - data

@kapishmalik
Copy link
Author

@ipedrazas I have incorporated suggested change in server.yaml file.

@ipedrazas
Copy link
Contributor

Excellent, could you remove the highlighted: true attribute? We no longer use it.

@kapishmalik
Copy link
Author

@ipedrazas I am done with suggested change.

@kapishmalik
Copy link
Author

Hey @ipedrazas it will be great if we can wrap this up.

@kapishmalik
Copy link
Author

Hey @ipedrazas Can we merge this PR if everything looks fine.

@ipedrazas
Copy link
Contributor

The icon field is missing. If you run task validate -- --name hoverfly-mcp-server you will see that it returns an error. (It also has highlighted a little bug in the validate command, which I will fix straight away 🙈)

@kapishmalik
Copy link
Author

@ipedrazas I have made necessary change. Could you please check now.

@kapishmalik
Copy link
Author

@ipedrazas could you please check this. I am waiting from long time to get this merged.

@cmrigney
Copy link
Contributor

Hi @kapishmalik, I don't believe you pushed the change to add the icon. Can you do so and we can review agian?

@kapishmalik
Copy link
Author

Not sure how it got missed, pushed now. @ipedrazas @cmrigney Could you please check.

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.

4 participants