tombstone: Mark expiration_epoch field as deprecated

There is no need to transfer data that's already set in other place and require equality. Closes #215. Signed-off-by: Leonard Lyubich <[email protected]>
nspcc-dev · Feb 26, 2024 · 4199b40 · 4199b40
1 parent dcead71
commit 4199b40
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -14,6 +14,7 @@
 ### Removed
 
 ### Deprecated
+- `Tombstone.expiration_epoch` field in favor of corresponding object attribute (#215)
 
 ## [2.15.0] - 2024-01-30
 

diff --git a/proto-docs/tombstone.md b/proto-docs/tombstone.md
@@ -31,7 +31,7 @@ purged from the NeoFS network.
 
 | Field | Type | Label | Description |
 | ----- | ---- | ----- | ----------- |
-| expiration_epoch | [uint64](#uint64) |  | Last NeoFS epoch number of the tombstone lifetime. It's set by the tombstone creator depending on the current NeoFS network settings. A tombstone object must have the same expiration epoch value in `__NEOFS__EXPIRATION_EPOCH` attribute. Otherwise, the tombstone will be rejected by a storage node. |
+| expiration_epoch | [uint64](#uint64) |  | Last NeoFS epoch number of the tombstone lifetime. It's set by the tombstone creator depending on the current NeoFS network settings. A tombstone object must have the same expiration epoch value in `__NEOFS__EXPIRATION_EPOCH` attribute. Otherwise, the tombstone will be rejected by a storage node. DEPRECATED. Set corresponding object attribute only. |
 | split_id | [bytes](#bytes) |  | 16 byte UUID used to identify the split object hierarchy parts. Must be unique inside a container. All objects participating in the split must have the same `split_id` value. |
 | members | [neo.fs.v2.refs.ObjectID](#neo.fs.v2.refs.ObjectID) | repeated | List of objects to be deleted. |
 

diff --git a/tombstone/types.proto b/tombstone/types.proto
@@ -14,7 +14,8 @@ message Tombstone {
   // creator depending on the current NeoFS network settings. A tombstone object
   // must have the same expiration epoch value in `__NEOFS__EXPIRATION_EPOCH`
   // attribute. Otherwise, the tombstone will be rejected by a storage node.
-  uint64 expiration_epoch = 1 [json_name = "expirationEpoch"];
+  // DEPRECATED. Set corresponding object attribute only.
+  uint64 expiration_epoch = 1 [json_name = "expirationEpoch", deprecated = true];
 
   // 16 byte UUID used to identify the split object hierarchy parts. Must be
   // unique inside a container. All objects participating in the split must