minio / mc

Unix like utilities for object store
https://min.io/download
GNU Affero General Public License v3.0
2.86k stars 548 forks source link

Update ILM add validation to accept with noncurrent-transition-newer set #4732

Closed jinapurapu closed 1 year ago

jinapurapu commented 1 year ago

Community Contribution License

All community contributions in this pull request are licensed to the project maintainers under the terms of the [Apache 2 license] (https://www.apache.org/licenses/LICENSE-2.0). By creating this pull request I represent that I have the right to license the contributions to the project maintainers under the Apache 2 license.

Description

Updates validation to accept lifecycle options with noncurrent-transition-newer set

Motivation and Context

Trying to set ILM rule using noncurrent-transition-newer flag triggered validation which was only checking transitionDays

jillii ~/mc [(HEAD detached at upstream/master)] $ ./mc ilm rule add --noncurrent-transition-newer 3 --noncurrent-transition-tier "TESTA" minio/jilltest 
mc: <ERROR> Unable to generate new lifecycle rules for the input: noncurrentversion-transition-days must be set.

How to test this PR?

Try adding a new ILM rule using the --noncurrent-transition-newer flag

Types of changes

Checklist:

krisis commented 1 year ago

@jinapurapu MinIO doesn't yet support NewerNoncurrentVersions with tiering. It is only available for NoncurrentVersionExpiration.