jmsperu commented on PR #12847: URL: https://github.com/apache/cloudstack/pull/12847#issuecomment-4523113960
Notes on the current red checks — all three appear unrelated to this change (which is a single `executeInSequence()` boolean flip on `TakeBackupCommand` and touches no other code path): - **`codecov/project`** — reports `4.15% (-12.10%)` project-coverage delta. That number is noise: the patch surface is one line so the codecov math is degenerate. `codecov/patch` (the one that actually reflects this PR) passes. - **`build (component/test_persistent_networks ...)`** — failure is `test_vpc_force_delete` throwing `builtins.Exception` plus `test_admin_project_creation` going EXCEPTION on one retry while the other retry passed. Known flake pattern; VPC teardown / project-creation tests are not on any code path reachable from a backup `executeInSequence` flip. - **`build (component/test_lb_secondary_ip ...)`** — failure is `test_03_delete_instance` in `test_memory_limits`. Memory-limits resource-accounting tests are timing-sensitive on the simulator and have no overlap with backup orchestration. Happy to rebase to retrigger if a reviewer prefers a fresh CI run, but the change itself is unaffected by these matrices. Other 14 checks green, 8 still running. -- 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]
