Skip to content
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

fix: match pattern file with Meshery Design schema #175

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

weilirs
Copy link

@weilirs weilirs commented Jan 9, 2025

Description
#173 (comment)

This PR fixes this issue

Notes for Reviewers
Followed the instructions from Pranav, exported a design file with a cilium component from Kanvas and replaced the content of it with the existing deploy.yaml
https://layer5io.slack.com/archives/C017NA80S66/p1736126652434049?thread_ts=1736126645.139429&cid=C017NA80S66

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.

@weilirs how was this design created?

Change the title from "Untitled Design".

When I import this design into a Meshery Server, its validation fails with a missing property.

@leecalcote
Copy link
Member

As you work through this, please confirm that the design is deployable.

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.

[CI] TestAdapters workflow is failing
2 participants