Changing AAP mode from active-standby to active-active does not make the route to be ECMP route

[Vedamurthy Joshi]

R2.21.2 Build 41

20.1.1.3 and 20.1.1.4 were given allowed address pair IP of 20.1.1.10
Since the default AAP mode is active-standby, manually updated the mode to be active-active for both the ports.

It was seen that even though agent got the updated config, but the route for 20.1.1.10/32 did not become a ECMP route.

Naveen is aware of the issue.

>>> from vnc_api.vnc_api import VncApi
>>>
>>> vh=VncApi(username='admin',password='contrail123',tenant_name='admin',api_server_host='127.0.0.1',api_server_port='8082')
>>>
>>>
>>> obj= vh.virtual_machine_interface_read(id = '1baafb25-7e7e-4e9b-b542-0c271a4e091a')
>>> obj1 = obj.get_virtual_machine_interface_allowed_address_pairs()
>>> obj2=obj1.get_allowed_address_pair()[0]
>>>
>>> obj2.set_address_mode('active-active')
>>> obj1.set_allowed_address_pair([obj2])
>>> obj.set_virtual_machine_interface_allowed_address_pairs(obj1)
>>> vh.virtual_machine_interface_update(obj)

root@nodec7:/var/log/contrail# neutron port-show 8b98eae5-ad33-4817-b4c9-3a1e156f86fc
+-----------------------+---------------------------------------------------------------------------------+
| Field | Value |
+-----------------------+---------------------------------------------------------------------------------+
| admin_state_up | True |
| allowed_address_pairs | {"ip_address": "20.1.1.10", "mac_address": ""} |
| binding:vif_details | {"port_filter": true} |
| binding:vif_type | vrouter |
| device_id | b0358795-a930-4fa6-90d0-7a40ce6648eb |
| device_owner | compute:None |
| fixed_ips | {"subnet_id": "b6090cda-c5b7-4e35-949a-18ef74773f9e", "ip_address": "20.1.1.3"} |
| id | 8b98eae5-ad33-4817-b4c9-3a1e156f86fc |
| mac_address | 02:8b:98:ea:e5:ad |
| name | 8b98eae5-ad33-4817-b4c9-3a1e156f86fc |
| network_id | e609fc12-9145-4c85-9799-4b9b564db293 |
| security_groups | dbd01ad3-80b6-4197-80e1-0ec036097c8e |
| status | ACTIVE |
| tenant_id | a8f9eafe3f37432daafef6d799006d6a |
+-----------------------+---------------------------------------------------------------------------------+
root@nodec7:/var/log/contrail# neutron port-show 1baafb25-7e7e-4e9b-b542-0c271a4e091a
+-----------------------+---------------------------------------------------------------------------------+
| Field | Value |
+-----------------------+---------------------------------------------------------------------------------+
| admin_state_up | True |
| allowed_address_pairs | {"ip_address": "20.1.1.10", "mac_address": ""} |
| binding:vif_details | {"port_filter": true} |
| binding:vif_type | vrouter |
| device_id | 8f1043dc-3d60-4193-b4f6-7a7adb0a6ace |
| device_owner | compute:None |
| fixed_ips | {"subnet_id": "b6090cda-c5b7-4e35-949a-18ef74773f9e", "ip_address": "20.1.1.4"} |
| id | 1baafb25-7e7e-4e9b-b542-0c271a4e091a |
| mac_address | 02:1b:aa:fb:25:7e |
| name | 1baafb25-7e7e-4e9b-b542-0c271a4e091a |
| network_id | e609fc12-9145-4c85-9799-4b9b564db293 |
| security_groups | dbd01ad3-80b6-4197-80e1-0ec036097c8e |
| status | ACTIVE |
| tenant_id | a8f9eafe3f37432daafef6d799006d6a |
+-----------------------+---------------------------------------------------------------------------------+
root@nodec7:/var/log/contrail#

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/22152
Submitter: RAVI KIRAN (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/22152
Committed: http://github.org/Juniper/contrail-controller/commit/4b447b983139d6e34efc6e4f2c6920093f515595
Submitter: Zuul
Branch: master

commit 4b447b983139d6e34efc6e4f2c6920093f515595
Author: Ravi BK <email address hidden>
Date: Tue Jul 19 20:25:04 2016 +0530

AapMode change for existing AAP to be honoured.

RootCause: If Allowed-Address Pair Mode is changed from
active-standby to active-active or vice-versa, change is notified
to Agent operDB, but Agent is not handling the change if AAP is
already installed. Only change in ip-address/prefix-len of AAP is
processed if it is already installed.

Fix: Process the change and modify the route to ECMP when AAP
mode is changed to active-active from default.

Change-Id: I5d32ff5878fb51957e074417c907a0677c59e9d8
Closes-Bug: #1591886

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/22335
Submitter: RAVI KIRAN (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/22341
Submitter: RAVI KIRAN (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/22342
Submitter: RAVI KIRAN (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/22342
Committed: http://github.org/Juniper/contrail-controller/commit/f7ff805607e93ffb65cc633f33c1036201214589
Submitter: Zuul
Branch: R3.0

commit f7ff805607e93ffb65cc633f33c1036201214589
Author: Ravi BK <email address hidden>
Date: Tue Jul 19 20:25:04 2016 +0530

AapMode change for existing AAP to be honoured.

RootCause: If Allowed-Address Pair Mode is changed from
active-standby to active-active or vice-versa, change is notified
to Agent operDB, but Agent is not handling the change if AAP is
already installed. Only change in ip-address/prefix-len of AAP is
processed if it is already installed.

Fix: Process the change and modify the route to ECMP when AAP
mode is changed to active-active from default.

Change-Id: I5d32ff5878fb51957e074417c907a0677c59e9d8
Closes-Bug: #1591886

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/22341
Committed: http://github.org/Juniper/contrail-controller/commit/bf987c735fb1a15ce7778608165eabdee04d2abc
Submitter: Zuul
Branch: R3.1

commit bf987c735fb1a15ce7778608165eabdee04d2abc
Author: Ravi BK <email address hidden>
Date: Tue Jul 19 20:25:04 2016 +0530

AapMode change for existing AAP to be honoured.

RootCause: If Allowed-Address Pair Mode is changed from
active-standby to active-active or vice-versa, change is notified
to Agent operDB, but Agent is not handling the change if AAP is
already installed. Only change in ip-address/prefix-len of AAP is
processed if it is already installed.

Fix: Process the change and modify the route to ECMP when AAP
mode is changed to active-active from default.

Change-Id: I5d32ff5878fb51957e074417c907a0677c59e9d8
Closes-Bug: #1591886

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/24967
Submitter: RAVI KIRAN (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/24967
Committed: http://github.org/Juniper/contrail-controller/commit/29629e8d01e9aa98f91ca330227c7e516c4a0c81
Submitter: Zuul
Branch: R3.0.2.x

commit 29629e8d01e9aa98f91ca330227c7e516c4a0c81
Author: Ravi BK <email address hidden>
Date: Tue Jul 19 20:25:04 2016 +0530

AapMode change for existing AAP to be honoured.

RootCause: If Allowed-Address Pair Mode is changed from
active-standby to active-active or vice-versa, change is notified
to Agent operDB, but Agent is not handling the change if AAP is
already installed. Only change in ip-address/prefix-len of AAP is
processed if it is already installed.

Fix: Process the change and modify the route to ECMP when AAP
mode is changed to active-active from default.

Change-Id: I5d32ff5878fb51957e074417c907a0677c59e9d8
Closes-Bug: #1591886