Add N9300 switch memory check#370
Conversation
Harinadh-Saladi
left a comment
Harinadh-Saladi
left a comment
There was a problem hiding this comment.
Pls address the comments
|
@Priyanka-Patil14 please attach test result from APIC for possible scenario's |
|
Uploaded the logs. |
|
@Priyanka-Patil14 can you please provide updated script output from APIC and error handling cases? |
MemoryCheck_Logs.txt Uploaded the logs. Please review it. |
|
Attached pytest and full script run logs as requested. |
Priyanka-Patil14
force-pushed
the
feature/n9300-switch-memory-check
branch
3 times, most recently
from
8197e54 to
d7d4289
Compare
|
Attaching the full run logs. NewValidation_APIC_FullRun_Logs.txt |
lovkeshsharma702
left a comment
lovkeshsharma702
left a comment
There was a problem hiding this comment.
intigrated test completed with no fault.
lovkeshsharma702
left a comment
lovkeshsharma702
left a comment
There was a problem hiding this comment.
intigration test passed. Logic changed, Code added in bug section with bug reference.
|
Last SR attach did have issues after upgrade, would prefer that this FLAG a FAIL_O vs just a MANUAL. do not want to give a false sense of security when this can prevent switches from coming back post-upgrade depending on the target version. |
Priyanka-Patil14
force-pushed
the
feature/n9300-switch-memory-check
branch
from
9830f7c to
6c7348b
Compare
7 participants
Summary
-This PR adds a new validation check: N9K-C93180YC-FX3 Switch Memory Less Than 32GB.
-The check is version-independent and validates that N9K-C93180YC-FX3 switches have at least 32 GB memory. This is tracked under defect CSCwm42741.
What Changed
n9k_c93180yc_fx3_switch_memory_checkinaci-preupgrade-validation-script.pydocs/docs/validations.mdtests/checks/n9k_c93180yc_fx3_switch_memory_check/Check Behavior
N/Aif no N9K-C93180YC-FX3 switches are presentFAIL_Ofor N9K-C93180YC-FX3 nodes with memory < 32 GBPASSwhen all N9K-C93180YC-FX3 nodes have >= 32 GBTest Validation
Executed:
'pytest tests/checks/n9k_c93180yc_fx3_switch_memory_check/ -q'
Result:
5 passed in 0.08sNo failures observed.