Skip to content

Commit b32e4b0

Browse files
Lukas Jaroschtimebertt
authored andcommitted
feat: map flavor not found to ResourceExhausted grpc code
1 parent 5ebc47e commit b32e4b0

2 files changed

Lines changed: 13 additions & 7 deletions

File tree

pkg/driver/executor/errors.go

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,17 @@ import (
88
"fmt"
99
)
1010

11-
// NoValidHost is a part of the error message returned when there is no valid host in the zone to deploy a VM.
12-
// Matches:
13-
//
14-
// "No valid host was found."
15-
// "No valid host was found. There are not enough hosts available."
16-
const NoValidHost = "No valid host was found"
11+
const (
12+
// NoValidHost is a part of the error message returned when there is no valid host in the zone to deploy a VM.
13+
// Matches:
14+
//
15+
// "No valid host was found."
16+
// "No valid host was found. There are not enough hosts available."
17+
NoValidHost = "No valid host was found"
18+
19+
// FlavorNotFound is part of the error message returned when a flavor cannot be resolved.
20+
FlavorNotFound = "error resolving flavor"
21+
)
1722

1823
var (
1924
// ErrNotFound is returned when the requested resource could not be found.

pkg/driver/utils.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,8 @@ func mapErrorToCode(err error) codes.Code {
7272

7373
func mapErrorMessageToCode(err error) codes.Code {
7474
errorMessage := err.Error()
75-
if strings.Contains(errorMessage, executor.NoValidHost) {
75+
if strings.Contains(errorMessage, executor.NoValidHost) ||
76+
strings.Contains(errorMessage, executor.FlavorNotFound) {
7677
return codes.ResourceExhausted
7778
}
7879
return codes.Internal

0 commit comments

Comments
 (0)