Skip to content
This repository was archived by the owner on Dec 27, 2024. It is now read-only.

Named containers #351

Merged
merged 3 commits into from
Oct 14, 2021
Merged

Conversation

cthulhu-rider
Copy link
Contributor

Based on nspcc-dev/neofs-api#177.
Re-compilation TBD

@cthulhu-rider cthulhu-rider added the enhancement Improving existing functionality label Oct 11, 2021
@cthulhu-rider cthulhu-rider self-assigned this Oct 11, 2021
cthulhu-rider pushed a commit to cthulhu-rider/neofs-api-go that referenced this pull request Oct 11, 2021
…tegration

Define const `SysAttributeName` and `SysAttributeZone` keys to NNS
attributes.

Signed-off-by: Leonard Lyubich <leonard@nspcc.ru>
@cthulhu-rider cthulhu-rider force-pushed the feature/container-names branch from 44adfee to fba67c6 Compare October 11, 2021 08:08
@codecov
Copy link

codecov bot commented Oct 11, 2021

Codecov Report

Merging #351 (dedb9dc) into master (a8fa0f9) will increase coverage by 0.07%.
The diff coverage is 100.00%.

❗ Current head dedb9dc differs from pull request most recent head e34edef. Consider uploading reports for the commit e34edef to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #351      +/-   ##
==========================================
+ Coverage   62.98%   63.05%   +0.07%     
==========================================
  Files         120      120              
  Lines       14110    14138      +28     
==========================================
+ Hits         8887     8915      +28     
  Misses       4216     4216              
  Partials     1007     1007              
Impacted Files Coverage Δ
pkg/container/attribute.go 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a8fa0f9...e34edef. Read the comment docs.

cthulhu-rider pushed a commit to cthulhu-rider/neofs-api-go that referenced this pull request Oct 11, 2021
…e names

Implement `GetNativeNameWithZone` / `SetNativeNameWithZone` function which
gets / sets `__NEOFS_NAME` and `__NEOFS_ZONE` container attributes.

Signed-off-by: Leonard Lyubich <leonard@nspcc.ru>
@cthulhu-rider cthulhu-rider marked this pull request as ready for review October 11, 2021 09:58
alexvanin
alexvanin previously approved these changes Oct 11, 2021
cthulhu-rider pushed a commit to cthulhu-rider/neofs-api-go that referenced this pull request Oct 11, 2021
…e names

Implement `GetNativeNameWithZone` / `SetNativeNameWithZone` function which
gets / sets `__NEOFS_NAME` and `__NEOFS_ZONE` container attributes.

Signed-off-by: Leonard Lyubich <leonard@nspcc.ru>
@cthulhu-rider cthulhu-rider force-pushed the feature/container-names branch from 7e195a0 to 6171d57 Compare October 11, 2021 14:13
carpawell
carpawell previously approved these changes Oct 12, 2021
Copy link
Contributor

@alexvanin alexvanin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Protofiles can be recompiled.

cthulhu-rider pushed a commit to cthulhu-rider/neofs-api-go that referenced this pull request Oct 14, 2021
Define `SysAttributeZoneDefault` constant for default zone in V2 code. Add
`SetNativeName` function which sets name with default zone.

Signed-off-by: Leonard Lyubich <leonard@nspcc.ru>
Leonard Lyubich added 3 commits October 14, 2021 11:53
…tegration

Define const `SysAttributeName` and `SysAttributeZone` keys to NNS
attributes.

Signed-off-by: Leonard Lyubich <leonard@nspcc.ru>
…e names

Implement `GetNativeNameWithZone` / `SetNativeNameWithZone` function which
gets / sets `__NEOFS_NAME` and `__NEOFS_ZONE` container attributes.

Signed-off-by: Leonard Lyubich <leonard@nspcc.ru>
Define `SysAttributeZoneDefault` constant for default zone in V2 code. Add
`SetNativeName` function which sets name with default zone.

Signed-off-by: Leonard Lyubich <leonard@nspcc.ru>
@alexvanin alexvanin merged commit db1ed76 into nspcc-dev:master Oct 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Improving existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants