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

[Fix] Overlapping stored flows #42

Closed
wants to merge 24 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
5dc3208
- Removed command attribute on stored_flows
viniarck Nov 4, 2021
cc48181
- Indexed stored_flows by cookie
viniarck Nov 4, 2021
4f3f4c3
Merge branch 'master' into feature/indexed_by_cookie
viniarck Nov 4, 2021
0aedf8c
Refactored check consistency methods to support indexing by cookie
viniarck Nov 5, 2021
d5ef667
- Fixed unit tests
viniarck Nov 5, 2021
b858c97
- Added lock to ensure correctnes when consistency is running
viniarck Nov 5, 2021
7704800
Fixed unit tests
viniarck Nov 5, 2021
fad0675
Fixed prior tests that used to use the flow_list
viniarck Nov 5, 2021
1610313
Linter fixes
viniarck Nov 5, 2021
830df99
Bumped version to 5.1.0
viniarck Nov 5, 2021
483828a
Merge branch 'feature/blacked' into feature/indexed_by_cookie
viniarck Nov 6, 2021
41d66d4
Merge branch 'master' into feature/indexed_by_cookie
viniarck Nov 8, 2021
75ade40
Formatted with black
viniarck Nov 8, 2021
b040b21
Reverted docstring
viniarck Nov 8, 2021
dc84085
Scoped self._storehouse_lock better
viniarck Nov 8, 2021
2676625
Simplified debug logs
viniarck Nov 8, 2021
f008ad7
Augmented _add_flow_store to overwrite overlapping flows
viniarck Nov 8, 2021
e9fd7f8
Added unit test for overlapping flows
viniarck Nov 8, 2021
ef57e42
Bumped 5.1.1
viniarck Nov 8, 2021
fdc340d
Linter fixes
viniarck Nov 8, 2021
95a5aad
Fixed fallback get dpid lookup when creating the stored_flows_list
viniarck Nov 9, 2021
8f92b93
Merge branch 'feature/indexed_by_cookie' into fix/overlapping_flows
viniarck Nov 9, 2021
22db2aa
Fixed fallback get dpid lookup when creating the stored_flows_list
viniarck Nov 9, 2021
00921af
Merge branch 'feature/indexed_by_cookie' into fix/overlapping_flows
viniarck Nov 9, 2021
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 26 additions & 0 deletions CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,32 @@ Removed
Security
========

[5.1.1] - 2021-11.08
********************

Added
=====
- Augmented ``_add_flow_store`` to overwrite overlapping flows

[5.1.0] - 2021-11.05
********************

Added
=====
- Added lock to avoid race flow mod race conditions in the consistency check

Changed
=======
- ``stored_flows`` are now indexed by cookie, issue 34
- Changed the ``flow_persistence`` data structured on storehouse
- Refactored the consistency checks methods accordingly to use cookie indexes


Deprecated
==========
- The prior ``flow_persistence`` data structure isn't supported anymore. It's required to delete the ``kytos.flow.persistence`` folder, upgrading won't be supported this time.


[4.1.2] - 2021-11.03
********************

Expand Down
2 changes: 1 addition & 1 deletion kytos.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"username": "kytos",
"name": "flow_manager",
"description": "Manage switches' flows through a REST API.",
"version": "4.1.2",
"version": "5.1.1",
"napp_dependencies": ["kytos/of_core", "kytos/storehouse"],
"license": "MIT",
"url": "https://github.com/kytos/flow_manager.git",
Expand Down
Loading