R5.0-micro-services - test automation - contrail internal and external vip are no longer supported and needed.

Bug #1777039 reported by Ritam Gangopadhyay on 2018-06-15
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R5.0
Fix Committed
High
Ritam Gangopadhyay
Trunk
Fix Committed
High
Ritam Gangopadhyay

Bug Description

          Today we don’t have anything as contrail_internal/contrail_external_vip. Now, the contrail test init code initializes contrail_internal_vip like this:-
[ritam@nodem3 contrail-test]$ sed -n 821p common/contrail_test_init.py
        self.contrail_internal_vip = self.contrail_external_vip = self.api_server_ip
[ritam@nodem3 contrail-test]$

          Since we don’t have anything as contrail_in/external_vip, so self.api_server_ip takes effect and is populated as contrail_internal_vip and we see the 10.204.217.103 IP which is actually nodem14.

[ritam@nodem3 contrail-test]$ sed -n 291,295p common/contrail_test_init.py
        self.contrail_internal_vip = read_config_option(self.config, 'HA',
                                          'contrail_internal_vip', None)
        self.contrail_external_vip = read_config_option(self.config, 'HA',
                                          'contrail_external_vip',
                                          self.contrail_internal_vip)
[ritam@nodem3 contrail-test]$

           API server IP is in turn populated from the variable CONFIG_API_VIP which is set to m14 IP in all.yaml (https://github.com/Juniper/contrail-tools/blob/4ea7c2fd1a5b875b2addcc5297f7ee0789a5ff6e/yamls/new_regr_cluster.yaml#L83) as proposed that it should be one of the controller IP till CONFIG_API_VIP is fully implemented for contrail HA (for neutron and heat plugins etc.)

[ritam@nodem3 contrail-test]$ sed -n 817p common/contrail_test_init.py
        self.api_server_ip = contrail_configs.get('CONFIG_API_VIP')
[ritam@nodem3 contrail-test]$

            I can change CONFIG_API_VIP to the openstack VIP we are using in all.yaml, but I am not sure of the impact it might have on provisioning. Let me try it once and get back to you, if we can fix this for test code from provisioning configuration end.

Regards,
Ritam.

**********************************************************
**********************************************************
**********************************************************
**********************************************************

Removing contrail_internal and external vip from test code.

Changed in juniperopenstack:
milestone: r5.0.1 → r5.1.0

Review in progress for https://review.opencontrail.org/43864
Submitter: Ritam Gangopadhyay (<email address hidden>)

Review in progress for https://review.opencontrail.org/43954
Submitter: Ritam Gangopadhyay (<email address hidden>)

Review in progress for https://review.opencontrail.org/43864
Submitter: Ritam Gangopadhyay (<email address hidden>)

Review in progress for https://review.opencontrail.org/43954
Submitter: Ritam Gangopadhyay (<email address hidden>)

Reviewed: https://review.opencontrail.org/43864
Committed: http://github.com/Juniper/contrail-test/commit/ac73c49850a3e8c66270402336ff21a421fd98f6
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit ac73c49850a3e8c66270402336ff21a421fd98f6
Author: Ritam Gangopadhyay <email address hidden>
Date: Tue Jun 19 15:20:09 2018 +0530

contrail internal and external vip are no longer supported and needed.

Closes-Bug: 1777039

Change-Id: I64dfd652887fca5841e3d1566c90a07496ffde5d

Reviewed: https://review.opencontrail.org/43954
Committed: http://github.com/Juniper/contrail-test/commit/5e60b4f11f409a37c33d935909be31ff01640ee6
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0

commit 5e60b4f11f409a37c33d935909be31ff01640ee6
Author: Ritam Gangopadhyay <email address hidden>
Date: Tue Jun 19 15:20:09 2018 +0530

contrail internal and external vip are no longer supported and needed.

Closes-Bug: 1777039

Change-Id: I64dfd652887fca5841e3d1566c90a07496ffde5d

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers