object: Send status response for server-side streams #620

Merged
fyrchik merged 1 commit from fyrchik/frostfs-node:fix-server-side into master 2023-08-17 12:52:42 +00:00
Owner

Previously status responses were wrapped in the gRPC error and thus
couldn't be correctly handled on client.

Introduced in c2617baf63, thanks @ale64bit for having found.

Close #617

Signed-off-by: Evgenii Stratonikov e.stratonikov@yadro.com

Previously status responses were wrapped in the gRPC error and thus couldn't be correctly handled on client. Introduced in c2617baf63, thanks @ale64bit for having found. Close #617 Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
fyrchik requested review from storage-core-committers 2023-08-17 11:28:28 +00:00
fyrchik requested review from storage-core-developers 2023-08-17 11:28:33 +00:00
fyrchik force-pushed fix-server-side from 33583f80be to c075cd191c 2023-08-17 11:34:36 +00:00 Compare
Member

Test from #617 (extended with Get checks) passes now:

=== RUN   TestPool
max obj size 67108864, obj size 67109198
container  5q7YEnjyAabhymid5qH2sAFKezWqtCYDTS15ZjpGQiwx
object  52t8sht9vxuvDBYQGCwxTEnQ51ed7QVu4Gs7w71ciPza
GetRange: s01.frostfs.devenv:8080: <nil>
GetRange: s02.frostfs.devenv:8080: <nil>
GetRange: s03.frostfs.devenv:8080: <nil>
GetRange: s04.frostfs.devenv:8080: <nil>
Get: s01.frostfs.devenv:8080: <nil>
Get: s02.frostfs.devenv:8080: <nil>
Get: s03.frostfs.devenv:8080: <nil>
Get: s04.frostfs.devenv:8080: <nil>
--- PASS: TestPool (11.22s)
PASS
ok  	git.frostfs.info/TrueCloudLab/frostfs-sdk-go/pool	11.233s
Test from #617 (extended with `Get` checks) passes now: ``` === RUN TestPool max obj size 67108864, obj size 67109198 container 5q7YEnjyAabhymid5qH2sAFKezWqtCYDTS15ZjpGQiwx object 52t8sht9vxuvDBYQGCwxTEnQ51ed7QVu4Gs7w71ciPza GetRange: s01.frostfs.devenv:8080: <nil> GetRange: s02.frostfs.devenv:8080: <nil> GetRange: s03.frostfs.devenv:8080: <nil> GetRange: s04.frostfs.devenv:8080: <nil> Get: s01.frostfs.devenv:8080: <nil> Get: s02.frostfs.devenv:8080: <nil> Get: s03.frostfs.devenv:8080: <nil> Get: s04.frostfs.devenv:8080: <nil> --- PASS: TestPool (11.22s) PASS ok git.frostfs.info/TrueCloudLab/frostfs-sdk-go/pool 11.233s ```
acid-ant approved these changes 2023-08-17 11:49:13 +00:00
fyrchik force-pushed fix-server-side from ac2fbdcdfc to fb47d2584e 2023-08-17 12:05:32 +00:00 Compare
ale64bit approved these changes 2023-08-17 12:11:50 +00:00
fyrchik force-pushed fix-server-side from fb47d2584e to e518151664 2023-08-17 12:17:24 +00:00 Compare
fyrchik force-pushed fix-server-side from e518151664 to c075cd191c 2023-08-17 12:22:02 +00:00 Compare
fyrchik merged commit 5a51b78946 into master 2023-08-17 12:52:42 +00:00
fyrchik deleted branch fix-server-side 2023-08-17 12:52:49 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-node#620
No description provided.