CLI, lifecycle: delete exceeding object versions

[AymericDu]

See #1163

[codecov-io]

Codecov Report

Merging #1363 into 4.x will increase coverage by 0.28%.
The diff coverage is 94.54%.

@@            Coverage Diff             @@
##              4.x    #1363      +/-   ##
==========================================
+ Coverage   78.15%   78.42%   +0.28%     
==========================================
  Files         300      300              
  Lines       55859    56098     +239     
  Branches     6486     6488       +2     
==========================================
+ Hits        43650    43992     +342     
+ Misses      11975    11966       -9     
+ Partials      234      140      -94

Impacted Files	Coverage Δ
meta2v2/meta2_gridd_dispatcher.c	68.19% <ø> (ø)	⬆️
tests/unit/container/test_lifecycle.py	98.71% <100%> (-0.04%)	⬇️
meta2v2/meta2_filters_action_container.c	54.61% <100%> (-2.13%)	⬇️
proxy/metacd_http.c	89.12% <100%> (-2.03%)	⬇️
meta2v2/meta2_filters_extract.c	87.88% <100%> (-1.59%)	⬇️
tests/functional/api/test_objectstorage.py	96.14% <100%> (+0.47%)	⬆️
oio/container/client.py	83.28% <100%> (+0.48%)	⬆️
tests/functional/container/test_container.py	99.28% <100%> (+0.04%)	⬆️
proxy/meta2v2_remote.c	80.43% <100%> (-0.79%)	⬇️
...ests/functional/content/test_content_versioning.py	100% <100%> (ø)	⬆️
... and 60 more

---

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 6280e64...86dcbcf. Read the comment docs.

[fvennetier]

The signature is different from LifecycleAction.match.

I agree there is a design flaw in this API: the from_element method could take an instance of ContainerLifecycle instead of (or along with) an instance of an ObjectStorageApi, to get an access to the account and container names.

[fvennetier]

I propose we extend this API with a trust_level parameter instead of full.

• trust_level=0: do not check chunks;
• trust_level=1: check if there are enough chunks to read the object;
• trust_level=2: check if all chunks are present.

[fvennetier]

In the scope of this pull-request, you can stop at trust_level=0 and trust_level=2, letting trust_level=1 to another pull-request.

[fvennetier]

To be consistent, we may need to put that inside <NonCurrentVersionExpiration>, and make it configurable (independently from the versioning configuration of the container).

[fvennetier]

Specifying max-versions < 1 is not relevant.