Skip to content

registry: correct meshmodel fallback to meshery/models#1041

Merged
miacycle merged 1 commit into
masterfrom
fix-meshmodel-path-meshery-models
Jun 27, 2026
Merged

registry: correct meshmodel fallback to meshery/models#1041
miacycle merged 1 commit into
masterfrom
fix-meshmodel-path-meshery-models

Conversation

@miacycle

Copy link
Copy Markdown
Member

Follow-up correction to #1040.

The meshery model data directory is relocating to the repo root as meshery/models - not meshery/server/models as the prior PR assumed. This points the ProcessRelationships fallback (registry/relationship.go:115) at ../../meshery/models.

This is the only filesystem reference to the relocated directory in MeshKit; the …/meshkit/models/meshmodel/… Go import paths are unrelated and unaffected.

The model data directory relocates to the meshery repo root as
meshery/models (not meshery/server/models). Point the ProcessRelationships
fallback at ../../meshery/models so relationship version resolution reads
from the correct location when no explicit path is supplied.

Signed-off-by: Ahmed Jamil <197998703+CodeAhmedJamil@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 27, 2026 01:33
@miacycle miacycle merged commit 644a186 into master Jun 27, 2026
2 checks passed
@miacycle miacycle deleted the fix-meshmodel-path-meshery-models branch June 27, 2026 01:33

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the default path in the ProcessRelationships function of registry/relationship.go from ../../meshery/server/models to ../../meshery/models when the path parameter is empty. There are no review comments, and I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates MeshKit’s Meshery filesystem fallback so relationship processing resolves model versions from the new canonical Meshery on-disk directory (meshery/models at repo root), correcting the prior assumption from #1040.

Changes:

  • Adjust registry.ProcessRelationships default/fallback path from ../../meshery/server/models to ../../meshery/models.

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