abh1sar commented on PR #13877:
URL: https://github.com/apache/cloudstack/pull/13877#issuecomment-5439765925
> Went through the backport against #13074 and the 4.22 tree. Two notes
in-thread: line 980 (`sb.and("backupStatus", ...)`) is dead code on 4.22
because the status filter from #13254 isn't there; harmless thanks to
SearchCriteria skipping unset conditions, but worth dropping. The zoneid point
in the smoke test is inherited from main rather than introduced here; I'll fix
it on main separately. Otherwise this matches the merged change. Happy to
re-check once the conflicts with 4.22 are resolved.
Thanks @jmsperu
I removed the backupStatus code and resolved conflicts. Please check
--
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]
