434-dynamically-calculate-the-required-free-space-in-switch-bootflash by lovkeshsharma702 · Pull Request #441 · datacenter/ACI-Pre-Upgrade-Validation-Script

lovkeshsharma702 · 2026-09-09T12:22:21Z

cover dynamic space check.

Testing logs when fabric on 5.2.8h.
-----------

fab1-apic1# python aci-preupgrade-validation-script.py -d switch_bootflash_usage_check -t "6.1(5e)"     
    ==== 2026-09-09T02-15-28+0000, Script Version v4.2.0  ====

!!!! Check https://github.com/datacenter/ACI-Pre-Upgrade-Validation-Script for Latest Release !!!!

To use a non-default Login Domain, enter apic#DOMAIN\\USERNAME
Enter username for APIC login          : admin
Enter password for corresponding User  : 

Gathering Node Information...

Current APIC Version...5.2(8h)
Lowest Switch Version...5.2(8h)

Target APIC version is overridden to 6.1(5e)

Collecting VPC Node IDs...

Progress: |████████████████████████████████████████████████████████████████████████████████████████████████████| 1/1 checks completed


=== Check Result (failed only) ===

[Check 1/1] Switch Node /bootflash usage...                                                                       FAIL - UPGRADE FAILURE!!
  Pod-ID  Node-ID  Avail (MB)  Required (MB)
  ------  -------  ----------  -------------
  1       105      5437.03     6518.78

  Recommended Action: Insufficient free space to download and extract the target image! Contact Cisco TAC for Support
  Reference Document: https://datacenter.github.io/ACI-Pre-Upgrade-Validation-Script/validations/#switch-node-bootflash-usage



=== Summary Result ===

PASS                        :  0
FAIL - OUTAGE WARNING!!     :  0
FAIL - UPGRADE FAILURE!!    :  1
MANUAL CHECK REQUIRED       :  0
POST UPGRADE CHECK REQUIRED :  0
N/A                         :  0
ERROR !!                    :  0
TOTAL                       :  1

    Pre-Upgrade Check Complete.
    Next Steps: Address all checks flagged as FAIL, ERROR or MANUAL CHECK REQUIRED

    Result output and debug info saved to below bundle for later reference.
    Attach this bundle to Cisco TAC SRs opened to address the flagged checks.

      Result Bundle: /data/techsupport/preupgrade_validator_2026-09-09T02-15-28+0000.tgz

==== Script Version v4.2.0 FIN ====
fab1-apic1# python aci-preupgrade-validation-script.py -d switch_bootflash_usage_check -t "5.3(2b)"          
    ==== 2026-09-09T02-16-32+0000, Script Version v4.2.0  ====

!!!! Check https://github.com/datacenter/ACI-Pre-Upgrade-Validation-Script for Latest Release !!!!

To use a non-default Login Domain, enter apic#DOMAIN\\USERNAME
Enter username for APIC login          : admin
Enter password for corresponding User  : 

Gathering Node Information...

Current APIC Version...5.2(8h)
Lowest Switch Version...5.2(8h)

Target APIC version is overridden to 5.3(2b)

Collecting VPC Node IDs...

Progress: |████████████████████████████████████████████████████████████████████████████████████████████████████| 1/1 checks completed


=== Check Result (failed only) ===


=== Summary Result ===

PASS                        :  1
FAIL - OUTAGE WARNING!!     :  0
FAIL - UPGRADE FAILURE!!    :  0
MANUAL CHECK REQUIRED       :  0
POST UPGRADE CHECK REQUIRED :  0
N/A                         :  0
ERROR !!                    :  0
TOTAL                       :  1

    Pre-Upgrade Check Complete.
    Next Steps: Address all checks flagged as FAIL, ERROR or MANUAL CHECK REQUIRED

    Result output and debug info saved to below bundle for later reference.
    Attach this bundle to Cisco TAC SRs opened to address the flagged checks.

      Result Bundle: /data/techsupport/preupgrade_validator_2026-09-09T02-16-32+0000.tgz

==== Script Version v4.2.0 FIN ====
fab1-apic1# python aci-preupgrade-validation-script.py -d switch_bootflash_usage_check -t "6.0(9e)"          
    ==== 2026-09-09T02-17-06+0000, Script Version v4.2.0  ====

!!!! Check https://github.com/datacenter/ACI-Pre-Upgrade-Validation-Script for Latest Release !!!!

To use a non-default Login Domain, enter apic#DOMAIN\\USERNAME
Enter username for APIC login          : admin
Enter password for corresponding User  : 

Gathering Node Information...

Current APIC Version...5.2(8h)
Lowest Switch Version...5.2(8h)

Target APIC version is overridden to 6.0(9e)

Collecting VPC Node IDs...

Progress: |████████████████████████████████████████████████████████████████████████████████████████████████████| 1/1 checks completed


=== Check Result (failed only) ===

[Check 1/1] Switch Node /bootflash usage...                                                                       FAIL - UPGRADE FAILURE!!
  Pod-ID  Node-ID  Avail (MB)  Required (MB)
  ------  -------  ----------  -------------
  1       105      5437.03     5989.18

  Recommended Action: Insufficient free space to download and extract the target image! Contact Cisco TAC for Support
  Reference Document: https://datacenter.github.io/ACI-Pre-Upgrade-Validation-Script/validations/#switch-node-bootflash-usage



=== Summary Result ===

PASS                        :  0
FAIL - OUTAGE WARNING!!     :  0
FAIL - UPGRADE FAILURE!!    :  1
MANUAL CHECK REQUIRED       :  0
POST UPGRADE CHECK REQUIRED :  0
N/A                         :  0
ERROR !!                    :  0
TOTAL                       :  1

    Pre-Upgrade Check Complete.
    Next Steps: Address all checks flagged as FAIL, ERROR or MANUAL CHECK REQUIRED

    Result output and debug info saved to below bundle for later reference.
    Attach this bundle to Cisco TAC SRs opened to address the flagged checks.

      Result Bundle: /data/techsupport/preupgrade_validator_2026-09-09T02-17-06+0000.tgz

==== Script Version v4.2.0 FIN ====

…tures

monrog2

Requesting changes for several cases where the new dynamic calculation can report PASS despite insufficient bootflash. Each item is documented inline with a concrete remediation and regression-test expectation.

monrog2

@lovkeshsharma702 I am requesting these two blocking changes. Both can produce a false PASS during the documented APIC-first upgrade sequence or after an image has been downloaded but still needs extraction.

Ensure bootflash validation reports the established target-version message before checking switch-version availability, with focused regression coverage. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Distinguish pre-split and split-image upgrade scenarios, credit only the staged 32-bit image during boundary crossings, and document the API-based calculations. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Summarize what the dynamic bootflash check validates and update the stale sample result message. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Describe the cleanup threshold, dynamic image-space calculation, downloaded-image handling, and validation outcomes without exposing implementation detail. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

monrog2 · 2026-09-15T14:53:37Z

@lovkeshsharma702

Integration test summary

Integration testing was run across the actual fabric versions and six configured upgrade paths. Python 2.7 and Python 3.8 unit tests passed.

For the Switch Node /bootflash usage check:

The actual-version run produced two valid upgrade failures where available space was below the calculated requirement.
Other results correctly required manual review when the target version or firmware images were unavailable.
The configured-version runs could not fully exercise the new boundary calculations because the required target images were not present in the firmware repositories.
No exception or calculation regression attributable to this PR was observed.
The overall integration jobs reported failures because of unrelated lab authentication and connectivity issues.

Example complete check hit, with fabric identifiers scrubbed and the updated recommendation:

[Check  18/102] Switch Node /bootflash usage...                                                                   FAIL - UPGRADE FAILURE!!
  Pod-ID  Node-ID    Avail (MB)  Required (MB)
  ------  ---------  ----------  -------------
  <pod>   <node>     5807.31     6044.36
  Recommended Action: Remove old, unused switch images to free bootflash space, then re-run this validation. Contact Cisco TAC if sufficient space cannot be recovered.
  Reference Document: https://datacenter.github.io/ACI-Pre-Upgrade-Validation-Script/validations/#switch-node-bootflash-usage

lovkeshsharma702 added 10 commits September 7, 2026 09:46

dynamic bootflash utilization

d584918

dynamic bootflash utilization

de146e5

dynamic bootflash utilization

330af6e

dynamic bootflash utilization

58e4fb3

dynamic bootflash utilization

8ff1465

Untrack .DS_Store and remove orphaned apic_oob_connectivity_check fix…

818c940

…tures

Sync maintUpgJob query fixture with rsp-subtree=full removal

52288b2

dynamic bootflash utilization

f9c76d2

434-dynamically-calculate-the-required-free-space-in-switch-bootflash

6d04faa

dynamic bootflash utilization

deafe40

lovkeshsharma702 requested review from monrog2 and takishida September 9, 2026 12:22

monrog2 requested changes Sep 9, 2026

View reviewed changes

Comment thread aci-preupgrade-validation-script.py Outdated

Comment thread aci-preupgrade-validation-script.py Outdated

Comment thread aci-preupgrade-validation-script.py Outdated

Comment thread aci-preupgrade-validation-script.py

Comment thread aci-preupgrade-validation-script.py

lovkeshsharma702 added 5 commits September 9, 2026 20:21

dynamic bootflash utilization

90e6cda

dynamic bootflash utilization

5f91239

dynamic bootflash utilization

aa82f35

dynamic bootflash utilization

b3c28ce

dynamic bootflash utilization

38b870c

lovkeshsharma702 requested a review from monrog2 September 10, 2026 15:31

monrog2 requested changes Sep 11, 2026

View reviewed changes

Comment thread aci-preupgrade-validation-script.py Outdated

Comment thread aci-preupgrade-validation-script.py Outdated

lovkeshsharma702 and others added 6 commits September 14, 2026 16:40

dynamic bootflash utilization

effe99e

fix: prioritize missing target version

b0007a9

Ensure bootflash validation reports the established target-version message before checking switch-version availability, with focused regression coverage. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

fix: calculate remaining crossing space

82f5d24

Distinguish pre-split and split-image upgrade scenarios, credit only the staged 32-bit image during boundary crossings, and document the API-based calculations. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

docs: clarify bootflash validation

acaecc1

Summarize what the dynamic bootflash check validates and update the stale sample result message. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

docs: explain bootflash space check

8db9a75

Describe the cleanup threshold, dynamic image-space calculation, downloaded-image handling, and validation outcomes without exposing implementation detail. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

docs: clarify bootflash remediation

64876e9

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Navigation Menu

Uh oh!

434-dynamically-calculate-the-required-free-space-in-switch-bootflash - #441

434-dynamically-calculate-the-required-free-space-in-switch-bootflash#441
lovkeshsharma702 wants to merge 21 commits into
v4.2.1-devfrom
lovsharm_bootflash_modification

lovkeshsharma702 commented Sep 9, 2026

Uh oh!

monrog2 left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

monrog2 left a comment

Uh oh!

Uh oh!

Uh oh!

monrog2 commented Sep 15, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

lovkeshsharma702 commented Sep 9, 2026

Uh oh!

monrog2 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

monrog2 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

monrog2 commented Sep 15, 2026

Integration test summary

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants