Skip to content

Conversation

@ritesh-karankal
Copy link
Contributor

Notes for Reviewers

This PR fixes #

Signed commits

  • Yes, I signed my commits.

Copy link
Member

@leecalcote leecalcote left a comment

Choose a reason for hiding this comment

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

Good job on these questions, @ritesh-karankal 👍

- id: "a"
text: "Meshery Server will refuse to start"
- id: "b"
text: "It may lead to ambiguous routing of gRPC calls to the wrong instance"
Copy link
Member

Choose a reason for hiding this comment

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

This leads to ambiguity of users, but not for Meshery Server. Each adapter connects and registers its callback interface on a dedicated port. Each registered adapter must use a unique port, not currently used by another adapter.

…rtified-meshery-contributor/meshery-extensibility-bank.md
- id: "d"
text: "To deploy Meshery's custom controllers and manage MeshSync."
- id: "q19"
text: "What is the name of the built-in, default **Local Provider** in Meshery?"
Copy link
Member

Choose a reason for hiding this comment

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

Too difficult and subject to change. The answer used to be "None". A different question on the difference between local and remote providers is ideal, though.

- id: "d"
text: "LocalHost"
- id: "q20"
text: "Which environment variable is used to enforce the use of a specific Remote Provider in a Meshery instance?"
Copy link
Member

Choose a reason for hiding this comment

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

This could be a little difficult. It might be better to ask a question like:

"Can a single Meshery Server deployment support use of multiple Providers? - yes.

And another:

"Can a Meshery Server deployment be configured to only allow use of a pre-selected Provider?"

Yes.

ritesh-karankal and others added 10 commits October 14, 2025 03:09
…rtified-meshery-contributor/meshery-extensibility-bank.md

Co-authored-by: Lee Calcote <[email protected]>
…rtified-meshery-contributor/meshery-extensibility-bank.md

Co-authored-by: Lee Calcote <[email protected]>
…rtified-meshery-contributor/meshery-extensibility-bank.md

Co-authored-by: Lee Calcote <[email protected]>
…rtified-meshery-contributor/meshery-extensibility-bank.md

Co-authored-by: Lee Calcote <[email protected]>
Signed-off-by: Ritesh Karankal <[email protected]>
… annotations for Font-Awesome

Signed-off-by: Lee Calcote <[email protected]>
@leecalcote leecalcote merged commit 47214b9 into meshery-extensions:master Oct 14, 2025
1 check passed
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.

2 participants