Skip to content

Oxp conn status change - #516

Merged
congwang09 merged 9 commits into
mainfrom
oxp-conn-status-change
Mar 26, 2026
Merged

Oxp conn status change#516
congwang09 merged 9 commits into
mainfrom
oxp-conn-status-change

Conversation

@congwang09

@congwang09 congwang09 commented Feb 18, 2026

Copy link
Copy Markdown
Contributor

Resolves: #487, #486
Depends on: atlanticwave-sdx/sdx-lc#203

Handle status change when LC sends status update to sdx controller.

@coveralls

coveralls commented Feb 18, 2026

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 23596626709

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 20 (0.0%) changed or added relevant lines in 1 file are covered.
  • 100 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.5%) to 53.373%

Changes Missing Coverage Covered Lines Changed/Added Lines %
sdx_controller/handlers/lc_message_handler.py 0 20 0.0%
Files with Coverage Reduction New Missed Lines %
sdx_controller/controllers/l2vpn_controller.py 42 52.08%
sdx_controller/handlers/connection_handler.py 58 66.0%
Totals Coverage Status
Change from base Build 21918320104: -0.5%
Covered Lines: 1266
Relevant Lines: 2372

💛 - Coveralls

Comment thread sdx_controller/handlers/lc_message_handler.py Outdated
Comment thread sdx_controller/handlers/lc_message_handler.py Outdated
Comment thread sdx_controller/handlers/lc_message_handler.py
@congwang09
congwang09 requested a review from italovalcy March 26, 2026 17:41

@italovalcy italovalcy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@congwang09
congwang09 merged commit 765380d into main Mar 26, 2026
4 checks passed
@congwang09
congwang09 deleted the oxp-conn-status-change branch March 26, 2026 18:55
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.

Pull L2VPN's status from OXP, update sdx controller if status changes

4 participants