Added validation check for CSCwt58626 - #405
Conversation
lovkeshsharma702
left a comment
There was a problem hiding this comment.
all intigrated test passed.
monrog2
left a comment
There was a problem hiding this comment.
Requesting changes for correctness against the current CSCwt58626 CDETS data and for fail-safe handling of APIC responses. Each concern is attached inline.
|
@lovkeshsharma702 try this way |
monrog2
left a comment
There was a problem hiding this comment.
Requesting changes for the remaining CSCwt58626 correctness issues. The current CDETS release note now explicitly states the affected condition as 6.1.3f+ or 6.2.1g; the inline comments also cover preserving confirmed upgrade-failure evidence and completing inventory evaluation.
|
@monrog2 please review the PR again. commented changes were completed and tested with successful integration test. |
|
@monrog2 please review the PR. commented part is taken care of. |
monrog2
left a comment
There was a problem hiding this comment.
Requesting changes for the two remaining CSCwt58626 issues: preserve incomplete overlapping inventory evidence, and align remediation guidance with the current defect record.
|
@monrog2 lets sync up today if anymore changes neede on this PR. |
|
Integration test summary (identifying environment details omitted):
Overall clean-run timing was 12-31 seconds, averaging approximately 19 seconds. Pytest validation also passed on both supported runtimes: 940 tests passed on Python 2.7 and Python 3.8, including 18/18 parametrized InfraVLAN overlap cases on each runtime. |
Added new validation for the defect CSCwt58626.
This check will check if the InfraVLAN overlaps with any user-configured VLAN pool range in Access Policies.
Apic failure logs:
apic_failure_log.txt
apic_full_run_log.txt
pytest_full_run_log.txt