-
Notifications
You must be signed in to change notification settings - Fork 5.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
make sure lifecycle rule ID is present #10084
make sure lifecycle rule ID is present #10084
Conversation
hmm not sure whether it is really required - per DOC
But rule cannot be created without ID via website .... On other hand this DOC mentions:
ID enforcing just decrease confusion i guess ... Furthermore i'm not sure what impact it will have on already set-up rules with duplicated IDs .... Any opinion on that ? |
@findmyname666 AWS does not require ID field.. but it generates a random one when it is not set. This could break some deployments. Maybe you can also do the same.. setting a random ID when parsing the lifecycle configuration when an S3 client sends it. |
@findmyname666 can you do this? please use |
The function
There are two approaches in my mind:
maybe we can create package |
just write the function like in pkg/event no new package is needed @findmyname666 |
@harshavardhana please take a look ;) THX |
Mint Automation
10084-7cc3288/mint-xl.sh.log:
Deleting image on docker hub |
Description
PR #10053 (#10027) introduced check that lifecycle rule ID must be unique. Based on that we should also check that ID is provided for new rules.
Motivation and Context
AWS S3 enforces presence of rule ID as well.
How to test this PR?
Types of changes
Checklist:
commit-id
orPR #
here)