Anton Nikiforov acid-ant
  • Joined on 2023-03-06
acid-ant deleted branch bugfix/ec-writer from acid-ant/frostfs-node 2024-10-14 07:23:52 +00:00
acid-ant pushed to master at TrueCloudLab/frostfs-node 2024-10-14 07:23:51 +00:00
acd6eb1815 [#1427] object: Fix Put for EC object when node unavailable
acid-ant merged pull request TrueCloudLab/frostfs-node#1427 2024-10-14 07:23:50 +00:00
object: Fix `Put` for `EC` object when node unavailable
acid-ant commented on pull request TrueCloudLab/frostfs-node#1427 2024-10-14 05:57:43 +00:00
object: Fix Put for EC object when node unavailable

Yes, it is intentional. The idea is to iterate over all nodes and return only the last error. All other errors will be logged inside e.writePart(...).

acid-ant commented on pull request TrueCloudLab/frostfs-node#1427 2024-10-11 14:02:55 +00:00
object: Fix Put for EC object when node unavailable

Commit message updated.

acid-ant pushed to bugfix/ec-writer at acid-ant/frostfs-node 2024-10-11 14:02:46 +00:00
acd6eb1815 [#1427] object: Fix Put for EC object when node unavailable
acid-ant commented on pull request TrueCloudLab/frostfs-node#1427 2024-10-11 13:43:37 +00:00
object: Fix Put for EC object when node unavailable

Fixed.

acid-ant pushed to bugfix/ec-writer at acid-ant/frostfs-node 2024-10-11 13:42:55 +00:00
518ed4be15 [#1427] object: Fix Put for EC object when node unavailable
acid-ant commented on pull request TrueCloudLab/frostfs-node#1427 2024-10-11 13:35:10 +00:00
object: Fix Put for EC object when node unavailable

Yeah, copied. But in my defense, I will say that it has already been duplicated :-)

acid-ant commented on pull request TrueCloudLab/frostfs-node#1427 2024-10-11 13:28:58 +00:00
object: Fix Put for EC object when node unavailable

Agree, updated. Looks much better.

acid-ant pushed to bugfix/ec-writer at acid-ant/frostfs-node 2024-10-11 13:27:50 +00:00
549e54e436 [#1427] object: Fix Put for EC object when node unavailable
acid-ant commented on pull request TrueCloudLab/frostfs-node#1427 2024-10-11 13:03:39 +00:00
object: Fix Put for EC object when node unavailable

Could you describe what the bug was in the commit message? I understand the diff, but fail to understand why the old code was incorrect.

With the old code, there might be situation when…

acid-ant approved TrueCloudLab/frostfs-sdk-go#284 2024-10-11 13:01:06 +00:00
go.mod: Update api-go
acid-ant pushed to bugfix/ec-writer at acid-ant/frostfs-node 2024-10-11 12:51:10 +00:00
cd2b523a64 [#1427] object: Fix Put for EC object when node unavailable
acid-ant created pull request TrueCloudLab/frostfs-node#1427 2024-10-11 12:46:19 +00:00
[#xx] object: Fix Put for EC object when node unavailable
acid-ant pushed to master at acid-ant/frostfs-node 2024-10-11 12:41:14 +00:00
42bf03e5cc [#1411] adm/nns: Add 'delRecord'
5992ee901a [#1411] go.mod: Bump frostfs-contract to v0.20.0
dfb00083d0 [#1426] go.mod: Update sdk-go
1134760271 [#1425] services/tree: Remove eACL mentions from bearer token parsing errors
02bb7159a5 [#1425] services/tree: Remove eACL processing
Compare 26 commits »
acid-ant pushed to bugfix/ec-writer at acid-ant/frostfs-node 2024-10-11 12:40:57 +00:00
b7ad738537 [#xx] object: Fix Put for EC object when node unavailable
42bf03e5cc [#1411] adm/nns: Add 'delRecord'
5992ee901a [#1411] go.mod: Bump frostfs-contract to v0.20.0
Compare 3 commits »
acid-ant created branch bugfix/ec-writer in acid-ant/frostfs-node 2024-10-11 12:40:24 +00:00
acid-ant pushed to bugfix/ec-writer at acid-ant/frostfs-node 2024-10-11 12:40:24 +00:00
72061f0944 [#xx] object: Fix Put for EC object when node unavailable
dfb00083d0 [#1426] go.mod: Update sdk-go
1134760271 [#1425] services/tree: Remove eACL mentions from bearer token parsing errors
02bb7159a5 [#1425] services/tree: Remove eACL processing
94302235d0 [#1425] adm: Remove eACL fetching from dump-containers
Compare 7 commits »