Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bsn_flow_checksum extension #193

Merged
merged 4 commits into from
Feb 6, 2014

Conversation

rlane
Copy link
Contributor

@rlane rlane commented Feb 5, 2014

Reviewer: @andi-bigswitch

The bsn_flow_checksum messages are largely copied from bsn_gentable.

Also changes the behavior of gentable delete if the entry does not exist.

@andi-bigswitch
Copy link
Contributor

Looks good - thanks!

ACK

@bsn-abat
Copy link

bsn-abat commented Feb 6, 2014

ABAT: START_MERGE: Log file at http://sbs2/abat/2014.02.06.0311-m.loxigen.master/abat.log

bsn-abat pushed a commit to floodlight/loxigen-artifacts that referenced this pull request Feb 6, 2014
Loxigen Head commit floodlight/loxigen@165e6de
commit 165e6de3454070fdb52461f6ee78faf5e357855e
Merge: 51cdfba 67dec2d
Author: abat <abat@bigswitch.com>
Date:   Wed Feb 5 19:11:36 2014 -0800

    Merge into master from pull request #193:
    bsn_flow_checksum extension (floodlight/loxigen#193)

commit 67dec2d815b203c1083e08da49f468d990f12364
Author: Rich Lane <rlane@bigswitch.com>
Date:   Wed Feb 5 14:00:53 2014 -0800

    loci: remove dead code

commit 013dea0f195ef17b9f3cf1ea47619d908cf47e13
Author: Rich Lane <rlane@bigswitch.com>
Date:   Wed Feb 5 13:44:13 2014 -0800

    loci: support bsn_flow_checksum

commit 7b49e3b9fc36c765fe10b81eff44db298d972e82
Author: Rich Lane <rlane@bigswitch.com>
Date:   Wed Feb 5 13:44:03 2014 -0800

    openflow_input: add bsn_flow_checksum

commit 45e341fd4638b4da6742ae8975d9362cd18db3e4
Author: Rich Lane <rlane@bigswitch.com>
Date:   Wed Feb 5 13:28:58 2014 -0800

    openflow_input: change gentable delete error case

    It's easier for the controller if deleting a nonexistent entry is not an error.
@bsn-abat
Copy link

bsn-abat commented Feb 6, 2014

ABAT: ACCEPT: Successfully merged

In case you want to see the build log, check out :
Log file at http://sbs2/abat/2014.02.06.0311-m.loxigen.master/abat.log
/cc

bsn-abat pushed a commit that referenced this pull request Feb 6, 2014
bsn_flow_checksum extension (#193)
@bsn-abat bsn-abat merged commit 67dec2d into floodlight:master Feb 6, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants