Storage list for lens

[roman-khimov]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 20 lines in your changes are missing coverage. Please review.

Project coverage is 23.65%. Comparing base (5533aa8) to head (04a482e).
Report is 7 commits behind head on master.

Files	Patch %	Lines
cmd/neofs-lens/internal/storage/list.go	0.00%	19 Missing ⚠️
cmd/neofs-lens/internal/storage/root.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2852      +/-   ##
==========================================
- Coverage   23.66%   23.65%   -0.02%     
==========================================
  Files         769      770       +1     
  Lines       44533    44514      -19     
==========================================
- Hits        10540    10531       -9     
+ Misses      33142    33129      -13     
- Partials      851      854       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[cthulhu-rider]

overall good

desc says that cmd lists objects in blobstor. At the same time, it does not access the blobstor, only metabase is read. May be this worth to be clarified

[roman-khimov]

May be this worth to be clarified

Done.

[carpawell]

@roman-khimov, as @cthulhu-rider said that is a meta operation and the lens' meta subcommand already has it, so why do we need it? It can be confusing.

[carpawell]

magic? can lose 1025-th object and do not understand it?

[roman-khimov]

It will list everything. Yeah, magic.

[carpawell]

oh, almost endless for, did not see it. check context b/w iteration then to make it controllable?

[roman-khimov]

IIUC meta requires going over all metabases, also I think this one can be made to work without metabase.

[carpawell]

The work is done and it does what it claims so OK, but I would not add this command as the blobstor is a too complex thing to inspect IMO, and it is not about storage only but about managing and background processes too. Also, it is changed quite often.