Skip to content

Rollback on failed OXP POST#524

Open
congwang09 wants to merge 13 commits intomainfrom
rollback-on-failed-oxp-post
Open

Rollback on failed OXP POST#524
congwang09 wants to merge 13 commits intomainfrom
rollback-on-failed-oxp-post

Conversation

@congwang09
Copy link
Copy Markdown
Contributor

@congwang09 congwang09 commented Apr 9, 2026

Resolves: #519
This PR should merge after atlanticwave-sdx/sdx-lc#204 and atlanticwave-sdx/pce#304

If OXP POST request failed, or timeout, roll back partially provisioned connection request

@congwang09 congwang09 marked this pull request as draft April 9, 2026 15:07
@coveralls
Copy link
Copy Markdown

coveralls commented Apr 9, 2026

Coverage Report for CI Build 24919728481

Coverage decreased (-2.1%) to 50.725%

Details

  • Coverage decreased (-2.1%) from the base build.
  • Patch coverage: 209 uncovered changes across 5 files (89 of 298 lines covered, 29.87%).
  • 14 coverage regressions across 4 files.

Uncovered Changes

File Changed Covered %
sdx_controller/handlers/connection_handler.py 139 48 34.53%
sdx_controller/handlers/lc_message_handler.py 87 4 4.6%
sdx_controller/controllers/l2vpn_controller.py 24 6 25.0%
sdx_controller/init.py 47 31 65.96%
sdx_controller/messaging/rpc_queue_consumer.py 1 0 0.0%

Coverage Regressions

14 previously-covered lines in 4 files lost coverage.

File Lines Losing Coverage Coverage
sdx_controller/handlers/connection_handler.py 5 57.55%
sdx_controller/controllers/l2vpn_controller.py 4 47.34%
sdx_controller/utils/db_utils.py 4 77.27%
sdx_controller/handlers/lc_message_handler.py 1 9.89%

Coverage Stats

Coverage Status
Relevant Lines: 2691
Covered Lines: 1365
Line Coverage: 50.72%
Coverage Strength: 1.01 hits per line

💛 - Coveralls

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.

Incorrect behavior for Use Case 4: OXP failed to install L2VPN

3 participants