mirror of
https://github.com/fluxcd/flagger.git
synced 2026-04-15 06:57:34 +00:00
Use target port for virtual routers
AppMesh does not support port mappings
This commit is contained in:
@@ -269,7 +269,7 @@ func (ar *AppMeshv1beta2Router) reconcileVirtualRouter(canary *flaggerv1.Canary,
|
||||
Listeners: []appmeshv1.VirtualRouterListener{
|
||||
{
|
||||
PortMapping: appmeshv1.PortMapping{
|
||||
Port: appmeshv1.PortNumber(canary.Spec.Service.Port),
|
||||
Port: ar.getContainerPort(canary),
|
||||
Protocol: protocol,
|
||||
},
|
||||
},
|
||||
|
||||
@@ -52,7 +52,7 @@ func TestAppmeshv1beta2Router_Reconcile(t *testing.T) {
|
||||
vnPrimary, err := router.appmeshClient.AppmeshV1beta2().VirtualNodes("default").Get(context.TODO(), primaryName, metav1.GetOptions{})
|
||||
require.NoError(t, err)
|
||||
|
||||
primaryDNS := fmt.Sprintf("%s.%s.svc.cluster.local", primaryName, mocks.appmeshCanary.Namespace)
|
||||
primaryDNS := fmt.Sprintf("%s.%s.svc.cluster.local.", primaryName, mocks.appmeshCanary.Namespace)
|
||||
assert.Equal(t, primaryDNS, vnPrimary.Spec.ServiceDiscovery.DNS.Hostname)
|
||||
|
||||
// test backends update
|
||||
|
||||
Reference in New Issue
Block a user