Skip to content

Commit 2a1b38c

Browse files
Lukas Jaroschtimebertt
authored andcommitted
feat: map flavor not found to ResourceExhausted grpc code
1 parent 6c1d40c commit 2a1b38c

File tree

2 files changed

+13
-7
lines changed

2 files changed

+13
-7
lines changed

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
@@ -68,7 +68,8 @@ func mapErrorToCode(err error) codes.Code {
6868

6969
func mapErrorMessageToCode(err error) codes.Code {
7070
errorMessage := err.Error()
71-
if strings.Contains(errorMessage, executor.NoValidHost) {
71+
if strings.Contains(errorMessage, executor.NoValidHost) ||
72+
strings.Contains(errorMessage, executor.FlavorNotFound) {
7273
return codes.ResourceExhausted
7374
}
7475
return codes.Internal

0 commit comments

Comments
 (0)