S3: Sending notification to EventBridge LIFECYCLE_TRANSITION_EVENT

[tsugumi-sys]

• Sending an event to EventBridge when the object storage class is changed.
• (Object storage class is also changed by bucket lifecycle, but I think we cannot detect it in the current moto implementation.)

from docs;

You can also change the storage class of an object that is already stored in Amazon S3 to any other storage class by making a copy of the object by using the PUT Object - Copy API operation.

(I thought we can change object's storage class with restore_object request, but we can't based on the document.)

[codecov]

Codecov Report

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

Project coverage is 95.80%. Comparing base (9aef694) to head (892ff9c).
Report is 98 commits behind head on master.

Files	Patch %	Lines
moto/s3/models.py	40.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #7448      +/-   ##
==========================================
- Coverage   95.88%   95.80%   -0.08%     
==========================================
  Files         843      854      +11     
  Lines       82578    83957    +1379     
==========================================
+ Hits        79178    80437    +1259     
- Misses       3400     3520     +120     

Flag	Coverage Δ
servertests	32.22% <40.00%> (-0.30%)	⬇️
unittests	95.77% <40.00%> (-0.08%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[bblommers]

This sounds like a good usecase to introduce a getter/setter method for the storage_class attribute.

If we send the event inside the setter method, we can be sure that this always happens - regardless of who/where/why the storage class was changed.
We'll just have to be careful to only send an event when there is a change, and not on initialization.

What do you think?

[tsugumi-sys]

This sounds like a good usecase to introduce a getter/setter method for the storage_class attribute.

I totally agree with this :)
Thanks @bblommers ! I'll implement a setter and getter for storage class attribute in this PR.

[tsugumi-sys]

we need the FakeKey.status property implementation in this PR #7455

[bblommers]

The status-attribute should already be there @tsugumi-sys - it's defined in the parent class.
My PR just overrides it and adds additional functionality.