meshery/meshery

Replace raw error with ErrBuildOCIImg in ExportModel handler

Open

#19,424 opened on 2026年5月15日

GitHub で見る
 (1 comment) (0 reactions) (0 assignees)HTML (10,013 stars) (3,101 forks)batch import
area/ciarea/devopshelp wanted

説明

Current Behavior

Current Behavior
In ExportModel when BuildImage fails, a raw Go error is logged instead of a structured meshkit error

Desired Behavior

Use the existing ErrBuildOCIImg (already defined in error.go:610 used in meshery_pattern_handler.go:934):

h.log.Error(ErrBuildOCIImg(err))
http.Error(rw, ErrBuildOCIImg(err).Error(), http.StatusInternalServerError)

Implementation

Acceptance Tests

Contributor Guides and Handbook

コントリビューターガイド

技術スタック
go
領域
backendapi
Issue 種別
refactor
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
under 1 hour
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Go basicsUnderstanding of meshkit errors
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
90
調査方針
Locate the ExportModel handler in the codebase, likely in a file related to model export. Find where BuildImage fails and replace the raw error logging with h.log.Error(ErrBuildOCIImg(err)) and http.Error(rw, ErrBuildOCIImg(err).Error(), http.StatusInternalServerError). Reference the existing usage in meshery pattern handler.go:934 for exact pattern.