[qa] MTU test is wrong because DF flag is not supported in busybox.

Bug #1572079 reported by Atsuko Ito
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Alexandr Kostrikov
8.0.x
Fix Released
High
Vladimir Jigulin
Mitaka
Fix Released
High
Alexandr Kostrikov
Newton
Fix Released
High
Alexandr Kostrikov

Bug Description

Currently, we're pinging with packet size 1500, 9000, 9001 - 28 and regarding overlay padding if vxlan to check MTU https://github.com/openstack/fuel-qa/blob/master/fuelweb_test/tests/test_jumbo_frames.py#L194
However, ping from busybox from Cirros doesn't support PMUTd (setting DF flag), so the test is inconsistent, because ICMP is fragmented.

Test should use PMTUd enabled ping (required Cirros rebuild) or
test should not check for 9001 byte packet (some border cases could be passed by the way).

Changed in fuel:
assignee: nobody → Vladimir Eremin (yottatsa)
status: New → In Progress
Atsuko Ito (yottatsa)
tags: added: area-qa
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (master)

Fix proposed to branch: master
Review: https://review.openstack.org/307713

Revision history for this message
Atsuko Ito (yottatsa) wrote :
Changed in fuel:
importance: Undecided → High
Changed in fuel:
milestone: none → 10.0
Atsuko Ito (yottatsa)
Changed in fuel:
assignee: Vladimir Eremin (yottatsa) → Fuel QA Team (fuel-qa)
Changed in fuel:
assignee: Fuel QA Team (fuel-qa) → Alexander Zatserklyany (zatserklyany)
Revision history for this message
Alexandr Kostrikov (akostrikov-mirantis) wrote :

Ksenia, I am reassigning that to me because I had a conversation on that topic with Vladimir Eremin and in context of the bug.

Changed in fuel:
assignee: Alexander Zatserklyany (zatserklyany) → Alexandr Kostrikov (akostrikov-mirantis)
Changed in fuel:
assignee: Alexandr Kostrikov (akostrikov-mirantis) → Vladimir Eremin (yottatsa)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (master)

Reviewed: https://review.openstack.org/307713
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=92ae4c8e4ed965af91c57e218ef7ffa7fe95db61
Submitter: Jenkins
Branch: master

commit 92ae4c8e4ed965af91c57e218ef7ffa7fe95db61
Author: Vladimir Eremin <email address hidden>
Date: Tue Apr 19 14:10:04 2016 +0300

    Quick logic fix for MTU

    Since Cirros has no proper ping utility, check that 9001 byte packet
    could pass has no sense (ICMP will be fragmented).

    Change-Id: I8501d71491ff72a56116198bbb71e6a262f49efb
    Partial-Bug: #1572079

Atsuko Ito (yottatsa)
Changed in fuel:
assignee: Vladimir Eremin (yottatsa) → Alexandr Kostrikov (akostrikov-mirantis)
Revision history for this message
Alexandr Kostrikov (akostrikov-mirantis) wrote :

Quick fix is in fuel-qa. Reassigning bug to fuel-qa

Changed in fuel:
assignee: Alexandr Kostrikov (akostrikov-mirantis) → Fuel QA Team (fuel-qa)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/308287

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (stable/mitaka)

Reviewed: https://review.openstack.org/308287
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=62506fde4bc6f66a993ce2a488ff849a6531d4e2
Submitter: Jenkins
Branch: stable/mitaka

commit 62506fde4bc6f66a993ce2a488ff849a6531d4e2
Author: Vladimir Eremin <email address hidden>
Date: Tue Apr 19 14:10:04 2016 +0300

    Quick logic fix for MTU

    Since Cirros has no proper ping utility, check that 9001 byte packet
    could pass has no sense (ICMP will be fragmented).

    Change-Id: I8501d71491ff72a56116198bbb71e6a262f49efb
    Partial-Bug: #1572079
    (cherry picked from commit 92ae4c8e4ed965af91c57e218ef7ffa7fe95db61)

tags: added: in-stable-mitaka
Changed in fuel:
assignee: Fuel QA Team (fuel-qa) → Alexandr Kostrikov (akostrikov-mirantis)
tags: added: swarm-blocker
Revision history for this message
Atsuko Ito (yottatsa) wrote :

Normal fix is not ready yet.

Revision history for this message
Nastya Urlapova (aurlapova) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (stable/8.0)

Fix proposed to branch: stable/8.0
Review: https://review.openstack.org/336483

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (stable/8.0)

Reviewed: https://review.openstack.org/336483
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=e276567c47ba7f5d32b81981ba03d2ce87fd3c8b
Submitter: Jenkins
Branch: stable/8.0

commit e276567c47ba7f5d32b81981ba03d2ce87fd3c8b
Author: Vladimir Eremin <email address hidden>
Date: Tue Apr 19 14:10:04 2016 +0300

    Quick logic fix for MTU

    Since Cirros has no proper ping utility, check that 9001 byte packet
    could pass has no sense (ICMP will be fragmented).

    Change-Id: I8501d71491ff72a56116198bbb71e6a262f49efb
    Partial-Bug: #1572079
    (cherry picked from commit 92ae4c8e4ed965af91c57e218ef7ffa7fe95db61)
    (cherry picked from commit 62506fde4bc6f66a993ce2a488ff849a6531d4e2)

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.