[OPNFLWPLUG-807] Flows are not getting Programmed on DPN's when statistics are turned off(is-statistics-polling-off=true) Created: 27/Oct/16  Updated: 27/Sep/21  Resolved: 02/Nov/16

Status: Resolved
Project: OpenFlowPlugin
Component/s: General
Affects Version/s: None
Fix Version/s: None

Type: Bug
Reporter: Sumesh Kumar R Assignee: Unassigned
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: Linux
Platform: All


Attachments: File STATISTICS DISABLED LOGS.rar    
External issue ID: 7043
Priority: High

 Description   

Flows are not getting Programmed on DPN's when statistics are turned off(is-statistics-polling-off=true) whereas it works fine when the statistics are turned ON(is-statistics-polling-off=false)

Scenario Execution Step:
========================
1. Start the Controller and connect Mininet

2. Disable Statistics(is-statistics-polling-off=true)

3.Add Bulk Flow(1000 flows) Controller Node

4. Get Bulk Flows and Verify In Cluster config DS all flows are installed

5. Verify Flows In Switch , The test is failing in this step. no flows are seen
on the OVS switch.

Attached are the karaf logs



 Comments   
Comment by Sumesh Kumar R [ 27/Oct/16 ]

Attachment STATISTICS DISABLED LOGS.rar has been added with description: Karaf Logs attached

Comment by Shuva Jyoti Kar [ 27/Oct/16 ]

Works on the latest version of stable/Bo. Please test it out on latest stable/Bo.

Comment by Shuva Jyoti Kar [ 27/Oct/16 ]

(In reply to Sumesh Kumar R from comment #0)
> Created attachment 1320 [details]
> Karaf Logs attached
>
> Flows are not getting Programmed on DPN's when statistics are turned
> off(is-statistics-polling-off=true) whereas it works fine when the
> statistics are turned ON(is-statistics-polling-off=false)
>
> Scenario Execution Step:
> ========================
> 1. Start the Controller and connect Mininet
>
> 2. Disable Statistics(is-statistics-polling-off=true)
>
> 3.Add Bulk Flow(1000 flows) Controller Node
>
> 4. Get Bulk Flows and Verify In Cluster config DS all flows are installed
>
> 5. Verify Flows In Switch , The test is failing in this step. no flows are
> seen
> on the OVS switch.
>
>
> Attached are the karaf logs

Please retest and close it . Reducing the prio.

Comment by Shuva Jyoti Kar [ 27/Oct/16 ]

(In reply to Shuva Jyoti Kar from comment #1)
> Works on the latest version of stable/Bo. Please test it out on latest
> stable/Bo.

with this changeset https://git.opendaylight.org/gerrit/#/c/47418/

Comment by Shuva Jyoti Kar [ 27/Oct/16 ]

Might require the fix to be in sr1. Since with stats off functionality is broken

Comment by Sumesh Kumar R [ 27/Oct/16 ]

HI,

I had checked on the below latest build and the issue still exist.

Build Used : distribution-karaf-0.5.1-20161026.220204-606

Thanks,
Sumesh

Comment by Shuva Jyoti Kar [ 27/Oct/16 ]

(In reply to Sumesh Kumar R from comment #5)
> HI,
>
> I had checked on the below latest build and the issue still exist.
>
> Build Used : distribution-karaf-0.5.1-20161026.220204-606
>
> Thanks,
> Sumesh

yes, my bad. We would require this change-set to be merged
https://git.opendaylight.org/gerrit/#/c/47418/

Its +2'd but not merged yet. Its a functionality breakage with stats off

Comment by A H [ 27/Oct/16 ]

A patch was submitted to fix this bug in Boron SR1: https://git.opendaylight.org/gerrit/#/c/47418/

To better assess the impact of this bug and fix, could someone from your team please help us identify the following:
Severity: Could you elaborate on the severity of this bug? Is this a BLOCKER such that we cannot release Boron SR1 without it? Is there a workaround such that we can write a release note and fix in Carbon?
Testing: Could you also elaborate on the testing of this patch? How extensively has this patch been tested? Is it covered by any unit tests or system tests?
Impact: Does this fix impact any dependent projects?

Comment by Shuva Jyoti Kar [ 27/Oct/16 ]

(In reply to A H from comment #7)
> A patch was submitted to fix this bug in Boron SR1:
> https://git.opendaylight.org/gerrit/#/c/47418/
>
> To better assess the impact of this bug and fix, could someone from your
> team please help us identify the following:
> Severity: Could you elaborate on the severity of this bug? Is this a
> BLOCKER such that we cannot release Boron SR1 without it? Is there a
> workaround such that we can write a release note and fix in Carbon?
By default stats collection is on. So this issue will not occur. But with statistics turned off before a switch connection, flow provisioning is not going through. And that breaks the ofplugin functionality
> Testing: Could you also elaborate on the testing of this patch? How
> extensively has this patch been tested? Is it covered by any unit tests or
> system tests?
No. Testing done by individual testing with 2-5 flows.

> Impact: Does this fix impact any dependent projects?
No

Comment by Sumesh Kumar R [ 28/Oct/16 ]

I had verified the fix in the below build and it is working fine as expected(Flows are getting programmed on DPN’s when statistics is disabled)

Build Used: distribution-karaf-0.5.1-20161028.001556-607.zip

Thanks,
Sumesh

Comment by A H [ 01/Nov/16 ]

Has this bug been verified as fixed in the latest Boron SR1 Build 20161030?

Comment by Shuva Jyoti Kar [ 02/Nov/16 ]

(In reply to A H from comment #10)
> Has this bug been verified as fixed in the latest Boron SR1 Build 20161030?

Verified with the latest SR1 distribution Build 20161030 and it works

Generated at Wed Feb 07 20:33:26 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.