meshery/meshery

Replace raw error with ErrBuildOCIImg in ExportModel handler

Open

#19 424 ouverte le 15 mai 2026

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)HTML (10 013 stars) (3 101 forks)batch import
area/ciarea/devopshelp wanted

Description

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

Guide contributeur

Stack technique
go
Domaine
backendapi
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Go basicsUnderstanding of meshkit errors
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90
Direction de recherche
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.