c3y1huang

Test result [#1283](https://ci.longhorn.io/job/private/job/longhorn-tests-regression/1283/)

Test results: - [#1345](https://ci.longhorn.io/job/private/job/longhorn-tests-regression/1345/) (coretest/upgrade) - [#1344](https://ci.longhorn.io/job/private/job/longhorn-tests-regression/1344/console) (full) :warning: - [#1346](https://ci.longhorn.io/job/private/job/longhorn-tests-regression/1346/console) (full)

> * [#1344](https://ci.longhorn.io/job/private/job/longhorn-tests-regression/1344/console) (full) :warning: Investigating...

Client-pool: - Full - https://ci.longhorn.io/job/private/job/longhorn-tests-regression/1364/ - Full - https://ci.longhorn.io/job/private/job/longhorn-tests-regression/1368/ - Core - https://ci.longhorn.io/job/private/job/longhorn-tests-regression/1363/ - Core - https://ci.longhorn.io/job/private/job/longhorn-tests-regression/1367/ [On-demand](https://github.com/longhorn/longhorn-manager/pull/1348): - Full - https://ci.longhorn.io/job/private/job/longhorn-tests-regression/1362/ - Full - https://ci.longhorn.io/job/private/job/longhorn-tests-regression/1366/ - Core - https://ci.longhorn.io/job/private/job/longhorn-tests-regression/1361/...

> * In the previous design, we are distributing the grpc calls to multiple instance manager pods > Not sure if you are referring to the gRPC process communication? This...

Note: @FrankYang0529 is currently working on backing image backup and restore. AFAIK the implementation will have: 1. New backup cmd to the backing image manager to upload to the backup...

Backing image backup/restore, ref https://github.com/longhorn/longhorn/pull/4385.

> From the code, look like the job will still shows as completed even if the backup/snapshot fails: https://github.com/longhorn/longhorn-manager/blob/9e639a08c8c6c0b520bd5b1e6a3792406fa75dd4/app/recurring_job.go#L187-L191 > > Is this intentional, cc @c3y1huang ? No. We should...

Adding a note here, [this PR](https://github.com/longhorn/longhorn-manager/pull/1234) have some side-effects caused unstable test results. Ref [test_restore_basic#774](https://ci.longhorn.io/job/private/job/longhorn-tests-regression/774/testReport/tests/test_basic/), [test_restore_basic#775](https://ci.longhorn.io/job/private/job/longhorn-tests-regression/775/testReport/tests/test_basic/) ``` "Error querying backup status: Missing required parameter backupID" ```

Close this first as now we have no requirement for using a proxy client pool.