genegr commented on PR #13050: URL: https://github.com/apache/cloudstack/pull/13050#issuecomment-4353442945
The three failures from Trillian (`tid-15962`) look unrelated to this change: - `test_01_redundant_vpc_site2site_vpn` (`test_vpc_vpn.py`, 413 s) — VPC site-to-site VPN. **The same test also failed on #13059's run** (`tid-15963`), which only touches `AdaptiveDataStoreLifeCycleImpl.java` and has zero overlap with this PR's `FlashArrayAdapter` edit. Same test failing on two completely unrelated PRs reads like a flake or environment-side issue, not a regression. - `test_02_list_cpvm_vm` and `test_04_cpvm_internals` (both `test_ssvm.py`, 0.04 s / 0.05 s) — Console-Proxy/SSVM tests. The sub-tenth-of-a-second runtime suggests the test setup phase failed (CPVM not in the expected state) rather than the test itself catching a behavioural bug; this PR doesn't touch system-VM code. Happy to dig into any of them if you suspect otherwise — but as a sanity check, this PR's diff is one file, `FlashArrayAdapter.java`, exclusively in `getManagedStorageStats()` and a new `getArrayTotalCapacity()` helper, neither of which a VPN or CPVM smoke test would exercise. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
