-
Notifications
You must be signed in to change notification settings - Fork 112
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
YAML Parsing Error When Importing OCI Image via URL Import #650
Comments
@leecalcote The issue occurred because the URL pointed to the GitHub page (HTML) instead of the raw file, leading to errors in parsing. Using the raw GitHub URL, such as https://github.com/souvikinator/my_configs/raw/master/apisix.tar, resolves the problem as it directly links to the file. While the import now works, there's a need for:
Switching to the raw file URL addresses the issue effectively, but improving detection and feedback mechanisms would make the process more user-friendly and reliable. @amitamrutiya FYI |
I'll create a pull request addressing the first point linking to this issue. Currently, with the URL import, if we cannot detect the file type, we set it to "unknown" and allow the content to proceed through the pipeline. Instead, we could block it at that point and provide a relevant message. According to the code, the URL import should cover the following file types:
Am I missing any other file types? |
@souvikinator |
@souvikinator fixed? |
Current Behavior
When attempting to upload an exported OCI image from a URL in Meshery, the import process fails with a YAML parsing error. This prevents the successful import of the model.
file used: https://github.com/souvikinator/my_configs/blob/master/apisix.tar
The same file imports successfully when using the file import method instead of the URL import option.
Expected Behavior
The OCI image should import successfully without encountering YAML parsing issues. Meshery should validate and process the uploaded image seamlessly.
Screenshots/Logs
Environment
Contributor Guides and Handbook
The text was updated successfully, but these errors were encountered: