RHOSP:Int mirror test case failed due to pcap file check

Bug #1743635 reported by shajuvk
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R4.0
In Progress
High
Ankit Jain
R4.1
Fix Released
High
Ankit Jain
Trunk
Fix Committed
High
Ankit Jain

Bug Description

test case: test_intf_mirror_with_subintf_src_cn1vn1_dst_cn2vn2_analyzer_cn3vn3

Traceback (most recent call last):
  File "tcutils/wrappers.py", line 78, in wrapper
    result = function(self, *args, **kwargs)
  File "scripts/intf_mirror/test_intf_mirror.py", line 89, in test_intf_mirror_with_subintf_src_cn1vn1_dst_cn2vn2_analyzer_cn3vn3
    return self.verify_intf_mirroring_src_on_cn1_vn1_dst_on_cn2_vn2_analyzer_on_cn3_vn3(sub_intf=True)
  File "common/intf_mirroring/verify.py", line 69, in verify_intf_mirroring_src_on_cn1_vn1_dst_on_cn2_vn2_analyzer_on_cn3_vn3
    return self.verify_intf_mirroring(compute_nodes, [0, 1, 2], sub_intf, parent_intf, nic_mirror)
  File "common/intf_mirroring/verify.py", line 551, in verify_intf_mirroring
    analyzer_vm_ip, analyzer_fq_name, routing_instance, src_port=sport, sub_intf=sub_intf, parent_intf=parent_intf, nic_mirror=nic_mirror):
  File "common/intf_mirroring/verify.py", line 606, in _verify_intf_mirroring
    if not self.verify_port_mirroring(src_vm_fixture, dst_vm_fixture, mirror_vm_fixture, vlan=vlan, parent=parent_intf):
  File "tcutils/util.py", line 62, in f_retry
    result = f(*args, **kwargs) # first attempt
  File "common/servicechain/mirror/verify.py", line 313, in verify_port_mirroring
    pcap = self.start_tcpdump(session, tapintf, vlan=vlan)
  File "common/servicechain/mirror/verify.py", line 892, in start_tcpdump
    filt_str = ''
AssertionError: pcap file does not exist
============

need at least 5 sec delay for pacp verification:

        execute_cmd(session, cmd, self.logger)
        sleep(5)
        assert check_pcap_file_exists(session, pcap),'pcap file does not exist'
        return pcap

shajuvk (shajuvk)
tags: added: sanityblocker
information type: Proprietary → Public
Revision history for this message
shajuvk (shajuvk) wrote :

Hi Ankit,

Sanity test case failed while checking pcap. Looks like you added pacp verification recently. Could you please check it.

https://bugs.launchpad.net/juniperopenstack/+bug/1743635

Thanks,
Shaju

Revision history for this message
Ankit Jain (ankitja) wrote :
Revision history for this message
Ankit Jain (ankitja) wrote :

This commit should solve the issue.

https://review.opencontrail.org/#/c/38986/

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/39224
Submitter: Ankit Jain (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R4.1

Review in progress for https://review.opencontrail.org/39225
Submitter: Ankit Jain (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

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

commit fff3280091f03f1ebdc5e51fc4578bd7f9ce4afd
Author: ankitja <email address hidden>
Date: Thu Jan 25 20:12:09 2018 +0530

Fix for intf mirror test case (bug 1743635)

Related-bug: #1743635

Fixed couple of script issue:
1) get_svm_tapintf failed to fetch correct port uuid for parent intf when both sub intf and parent intf exist
2. Issue with parent vmi read in intf_mirroring/verify.py
3. Added warn log in check_pcap_file_exists
4. Also, some minor changes

Change-Id: I1486f40461ad451af4b66e4c769558162f4eb43d

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/39225
Committed: http://github.com/Juniper/contrail-test-ci/commit/2e17bd2be561e77142895dda4bfada449de031de
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 2e17bd2be561e77142895dda4bfada449de031de
Author: ankitja <email address hidden>
Date: Thu Jan 25 20:12:09 2018 +0530

Fix for intf mirror test case (bug 1743635)

Related-bug: #1743635

Fixed couple of script issue:
1) get_svm_tapintf failed to fetch correct port uuid for parent intf when both sub intf and parent intf exist
2. Issue with parent vmi read in intf_mirroring/verify.py
3. Added warn log in check_pcap_file_exists
4. Also, some minor changes

Change-Id: I1486f40461ad451af4b66e4c769558162f4eb43d

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R4.0

Review in progress for https://review.opencontrail.org/39342
Submitter: Ankit Jain (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/39342
Committed: http://github.com/Juniper/contrail-test-ci/commit/a507d79dacf6182fc8fb3b7552afa31006741a2b
Submitter: Zuul (<email address hidden>)
Branch: R4.0

commit a507d79dacf6182fc8fb3b7552afa31006741a2b
Author: ankitja <email address hidden>
Date: Thu Jan 25 20:12:09 2018 +0530

Fix for intf mirror test case (bug 1743635)

Related-bug: #1743635

Fixed couple of script issue:
1) get_svm_tapintf failed to fetch correct port uuid for parent intf when both sub intf and parent intf exist
2. Issue with parent vmi read in intf_mirroring/verify.py
3. Added warn log in check_pcap_file_exists
4. Also, some minor changes

Change-Id: I1486f40461ad451af4b66e4c769558162f4eb43d

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/40095
Submitter: Ankit Jain (<email address hidden>)

Jeba Paulaiyan (jebap)
tags: added: sanity
removed: sanityblocker
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.