Skip to content

Support compact equipment disk usage changeSet - #414

Open
monrog2 wants to merge 1 commit into
v4.2.0-devfrom
issue-369-equipment-disk-usage
Open

Support compact equipment disk usage changeSet#414
monrog2 wants to merge 1 commit into
v4.2.0-devfrom
issue-369-equipment-disk-usage

Conversation

@monrog2

@monrog2 monrog2 commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • parse both legacy and compact F1820/F1821/F1822 changeSet formats
  • preserve fault-driven upgrade failures and report malformed values as NA
  • add exact result assertions and issue-derived compact-format coverage
  • document support for both APIC serializations

Validation

  • pytest tests/checks/equipment_disk_limits_exceeded/test_equipment_disk_limits_exceeded.py (3 passed)
  • pytest (917 passed)
  • python -m py_compile aci-preupgrade-validation-script.py
  • git diff --check

Closes #369

Parse both observed F1820/F1821/F1822 changeSet serializations while preserving fault-driven failure behavior and malformed values as NA.

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

monrog2 commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator Author

Integration result for Equipment Disk Limits:

  • The check completed on all 15 reachable runs represented by collected TGZ bundles.
  • A live compact F1820 payload using avail:N, ..., used:N was parsed successfully and reported 82% utilization.
  • Two live legacy payloads using avail (New: N), ..., used (New: N) were also parsed successfully and reported 81% and 100% utilization.
  • All three active faults remained upgrade-blocking, preserving the existing fault-driven behavior.
  • No parsing exceptions, NA values, or malformed Equipment Disk Limits results occurred.
  • The remaining 12 collected runs had no matching F1820/F1821/F1822 faults and the check passed.

This provides positive live validation of the new compact parser and confirms that legacy parsing remains intact.

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.

1 participant