[OPNFLWPLUG-649] New FRM client of Batch RPC Created: 22/Mar/16  Updated: 27/Sep/21  Resolved: 24/May/16

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

Type: Bug
Reporter: Jozef Slezák Assignee: Andrej Leitner
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: All
Platform: All

Issue Links:
Blocks
blocks OPNFLWPLUG-651 New FRM RETRY mechanism for RCP call Resolved
is blocked by OPNFLWPLUG-648 Batch RPC flows/groups/meters in one ... Resolved
External issue ID: 5575

 Description   

This issue continues https://bugs.opendaylight.org/show_bug.cgi?id=5574

New FRM:

  • calculates the diff for the Switch and calls at once new batch RPC
  • no barriers in new FRM are needed
  • does not need to call separate route RPCs

 Comments   
Comment by Anil Vishnoi [ 30/Mar/16 ]

Hi Michal,

Can you please provide some details on why we need new FRM and why can't we use the existing FRM?

Comment by Michal Rehak [ 31/Mar/16 ]

Hi Anil,
it is currently WIP and under heavy testing.

It is not meant to be a replacement of current FRM because of more restrictive behavior. In principle this alternative implementation of FRM keeps DS/config and device in sync. No other flows/groups/meters allowed. So if device gets changed via RPC and corresponding item (e.g.: flow) is not present in DS/config then as soon as the item emerges in DS/operational then it will be removed from device.

On the other hand there is support for batched changes which enables for:

  • consistent pushing of dependent groups, flows, meters
  • faster operations as the barrier usage can be optimized and still provide valid rpc outcome for all changes.
  • but in order to stay consistent all the changes have to be written atomically (at least within one node). And so this impl listens to whole node and decides the order of changes when pushing to device.
Comment by Andrej Leitner [ 12/May/16 ]

Here is the chain in gerrit: https://git.opendaylight.org/gerrit/#/q/topic:bug5575-frm

Comment by Andrej Leitner [ 24/May/16 ]

merged

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