Skip to content

Commit

Permalink
Fix AWS AppMesh issue when providing multiple backends
Browse files Browse the repository at this point in the history
Signed-off-by: Joao Carreira <[email protected]>
  • Loading branch information
jddcarreira committed Feb 25, 2021
1 parent bfb329b commit 03be6a5
Showing 1 changed file with 8 additions and 10 deletions.
18 changes: 8 additions & 10 deletions pkg/router/appmesh_v1beta2.go
Original file line number Diff line number Diff line change
Expand Up @@ -119,24 +119,22 @@ func (ar *AppMeshv1beta2Router) reconcileVirtualNode(canary *flaggerv1.Canary, n
}

backends := make([]appmeshv1.Backend, 0)
for _, b := range canary.Spec.Service.Backends {
var bk appmeshv1.Backend
if strings.HasPrefix(b, "arn:aws") {
bk = appmeshv1.Backend{
for i := range canary.Spec.Service.Backends {
if strings.HasPrefix(canary.Spec.Service.Backends[i], "arn:aws") {
backends = append(backends, appmeshv1.Backend{
VirtualService: appmeshv1.VirtualServiceBackend{
VirtualServiceARN: &b,
VirtualServiceARN: &canary.Spec.Service.Backends[i],
},
}
})
} else {
bk = appmeshv1.Backend{
backends = append(backends, appmeshv1.Backend{
VirtualService: appmeshv1.VirtualServiceBackend{
VirtualServiceRef: &appmeshv1.VirtualServiceReference{
Name: b,
Name: canary.Spec.Service.Backends[i],
},
},
}
})
}
backends = append(backends, bk)
}

if len(backends) > 0 {
Expand Down

0 comments on commit 03be6a5

Please sign in to comment.