github/flagger
1
0
Fork 0
mirror of https://github.com/fluxcd/flagger.git synced 2026-04-15 06:57:34 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #831 from jddcarreira/fixMultipleAppMeshBackends

Browse Source 
Fix AWS AppMesh issue when providing multiple backends
This commit is contained in:
Stefan Prodan
2021-02-26 11:13:57 +02:00
committed by GitHub
parent 138c42ec3a 03be6a58ac
commit dfdd8cffd7
1 changed files with 8 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/router/appmesh_v1beta2.go
+8 -10
View File
@@ -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 {