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.

Sign up for GitHub

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

Jump to bottom

Add information about LACP connection status for LAG ports #4964

Open
niksv opened this issue Oct 20, 2022 · 0 comments · May be fixed by #5040
Open

Add information about LACP connection status for LAG ports #4964

niksv opened this issue Oct 20, 2022 · 0 comments · May be fixed by #5040
Assignees
@durgakako
Labels
feature

Comments

@niksv
Copy link
Collaborator

niksv commented Oct 20, 2022 •
edited
Loading

LAG ports has ability to reply on LACP packets
After LACP enabling kilda starts to receive LACP packets. (See LacpService in floodlight)
These packets contains information about state of LACP session.
Need to save this data in data base to have ability to see state of LACP session.

Example get LACP port output:

{
  "lacp_reply": true,
  "logical_port_number": 1234,
  "port_numbers": [
    1, 2
  ],
  
  "lacp_request_data": {
   "state": {
        "active": true,
        "shortTimeout": true,
        "aggregatable": true,
        "synchronised": true,
        "collecting": true,
        "distributing": true,
        "defaulted": false,
        "expired": false
   }
  "last_updated": "2022-10-20T15:14:18.691Z"
  }
}

Good place for storing LACP info will be ConnectedDevices topology
@niksv niksv added the feature label Oct 20, 2022
@durgakako durgakako linked a pull request Jan 16, 2023 that will close this issue
4964 add information about lacp connection status for lag ports #5040
Open
pkazlenka added a commit that referenced this issue Jan 31, 2023
@pkazlenka
#4964: Add information about lacp connection status for lag ports tests
8427127 
* Added support to traffgen to emit LACP packet
pkazlenka added a commit that referenced this issue Jan 31, 2023
@pkazlenka
#4964: Add information about lacp connection status for lag ports tests
eeb17e3 
* Added debug for endpoint
durgakako pushed a commit that referenced this issue Jan 31, 2023
#4964 LACP status - Updated based on review comments
ffeebda
pkazlenka added a commit that referenced this issue Feb 1, 2023
@pkazlenka
#4964: Add information about lacp connection status for lag ports tests
d710d73 
* Changed model
durgakako pushed a commit that referenced this issue Feb 1, 2023
#4964 LACP status - Updated based on review comments
0185f4a
pkazlenka added a commit that referenced this issue Feb 1, 2023
@pkazlenka
#4964: Modify traffexam to be able to emit LACP data units
ec5496e 
* Added new traffexam endpoint (/address/<idnr>/lacp)
pkazlenka added a commit that referenced this issue Feb 1, 2023
@pkazlenka
#4964: Modify traffexam to be able to emit LACP data units
4ac1932 
* Fixed wrong explanation in comment
pkazlenka added a commit that referenced this issue Feb 10, 2023
@pkazlenka
#4964: add information about LACP connection tests
ba9a2e8 
* Added new test for LACP connection
* New class LAG representing LAG business entity
* New experimental approach to switches picking
* Implemented new methods for new traffgen feature
pkazlenka added a commit that referenced this issue Feb 10, 2023
@pkazlenka
#4964: Add information about lacp connection status for lag ports tests
b786eea 
* Added support to traffgen to emit LACP packet
pkazlenka added a commit that referenced this issue Feb 10, 2023
@pkazlenka
#4964: Add information about lacp connection status for lag ports tests
05ca3fb 
* Added debug for endpoint
pkazlenka added a commit that referenced this issue Feb 10, 2023
@pkazlenka
#4964: Add information about lacp connection status for lag ports tests
41c0cdd 
* Changed model
pkazlenka added a commit that referenced this issue Feb 10, 2023
@pkazlenka
#4964: Modify traffexam to be able to emit LACP data units
858b771 
* Added new traffexam endpoint (/address/<idnr>/lacp)
pkazlenka added a commit that referenced this issue Feb 10, 2023
@pkazlenka
#4964: Modify traffexam to be able to emit LACP data units
5512ae4 
* Fixed wrong explanation in comment
pkazlenka added a commit that referenced this issue Feb 10, 2023
@pkazlenka
#4964: add information about LACP connection tests
b98a923 
* Added new test for LACP connection
* New class LAG representing LAG business entity
* New experimental approach to switches picking
* Implemented new methods for new traffgen feature
pkazlenka added a commit that referenced this issue Feb 13, 2023
@pkazlenka
#4964: add information about LACP connection tests
9fecc0f 
* Fixed code style errors
pkazlenka added a commit that referenced this issue Feb 14, 2023
@pkazlenka
#4964: Modify traffexam to be able to emit LACP data units
2d03ce8 
* Bumped version (dev19 -> dev20)
pkazlenka added a commit that referenced this issue Feb 15, 2023
@pkazlenka
#4964: add information about LACP connection tests
9227722 
* Fixed post-test cleanup
pkazlenka added a commit that referenced this issue Feb 16, 2023
@pkazlenka
#4964: Modify traffexam to be able to emit LACP data units
82c7c74 
* Updated LACP emitting code to be closer to other methods approach
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@durgakako durgakako

Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

4964 add information about lacp connection status for lag ports telstra/open-kilda
2 participants
@niksv @durgakako