[Disk Manager] return detailed error with aborted status on wrong etag #2248
Conversation
| fmt.Errorf(format, args...), | ||
| &errors.ErrorDetails{ | ||
| Code: error_codes.Aborted, | ||
| Message: "data from url source changed during image creation", |
There was a problem hiding this comment.
data from url source was changed during image creation
There was a problem hiding this comment.
Я вот подумал, а не будет ли такое сообщение смущать пользователя -- не будет ли оно выглядеть так, будто это мы зачем-то изменили ему данные по url?
There was a problem hiding this comment.
можем тогда подробнее просто написать
can not create image because data from...
There was a problem hiding this comment.
но мне кажется что не должно возникать таких вопросов
|
а теста никакого на это нет? |
Есть вот такая проверка, но я не уверен, насколько она эффективно работает: https://github.com/ydb-platform/nbs/blob/main/cloud/disk_manager/internal/pkg/facade/image_service_test/image_service_test.go#L594 Подумаю, как сделать лучше |
там еще todo смущает в месте, что ты прислал. надо посмотреть что там |
Wrong ETag error occurs during creation image from url if data from source url changed during image creation. This error shoudl not be internal.