Skip to content

fix: service_bd_forceful_routing with LIf instead of EPp - #440

Open
takishida wants to merge 1 commit into
v4.2.1-devfrom
service_bd_forceful_routing_with_lif
Open

fix: service_bd_forceful_routing with LIf instead of EPp#440
takishida wants to merge 1 commit into
v4.2.1-devfrom
service_bd_forceful_routing_with_lif

Conversation

@takishida

Copy link
Copy Markdown
Collaborator

PR #229 used fvRtEPpInfoToBD to detect the service BDs. However, there are some cases where service BDs do not contain fvRtEPpInfoToBD.

This PR uses fvRtLIfCtxToBD instead to cover all cases. As a result, the data output also changed as shown below.

[Check 1/1] Service Graph BD Forceful Routing...                                                                     MANUAL CHECK REQUIRED
  Bridge Domain (Tenant:BD)   Contract (Tenant:Contract)    Service Graph (Tenant:Service Graph)  Node  Connector
  -------------------------   --------------------------    ------------------------------------  ----  ---------
  common:BD_INSIDE            common:FWPBR                  common:FWPBR_CLUSTER01                N1    provider
  common:BD_OUTSIDE           common:FWPBR                  common:FWPBR_CLUSTER01                N1    consumer

  Recommended Action:
        Confirm that within these BDs there is no bridging traffic with the destination IP that doesn't belong to them.
        Please check the reference document for details.
  Reference Document: https://datacenter.github.io/ACI-Pre-Upgrade-Validation-Script/validations/#service-graph-bd-forceful-routing

@monrog2

monrog2 commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

Integration testing was run against the exact PR head SHA ebf3e0d669a6acea2754833dd73c56ee0de5d0f5 using the internal integration runner.

  • Both Python unit-test jobs passed.
  • The seven integration jobs completed with an overall failed status because the runner treats any ERROR !! result across the test inventory as a job failure. Artifact review determined the job failures were unrelated to PR fix: service_bd_forceful_routing with LIf instead of EPp #440 (baseline validation errors, authentication/connectivity issues, and transient APIC timeouts).

For Service Graph BD Forceful Routing specifically:

  • 63 executions covered affected-version scenarios: 46 returned the expected MANUAL CHECK REQUIRED, 15 passed, and 2 encountered transient APIC HTTP 503 timeouts.
  • The two timeouts occurred concurrently across multiple unrelated checks, and this check succeeded in subsequent jobs, indicating environmental APIC load rather than a deterministic regression.
  • 4,552 relation rows were parsed successfully.
  • No unformatted or unparsed DNs were reported.
  • No PR-specific functional failure was identified.

Conclusion: the integration artifacts show no failure attributable to this change. The overall pipeline remained red due to unrelated environment and baseline failures.

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.

2 participants