CLDSRV-890: Log lifecycle expiration events in server access logs#6150
Conversation
Hello dvasilas,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Request integration branchesWaiting for integration branch creation to be requested by the user. To request integration branches, please comment on this pull request with the following command: Alternatively, the |
|
LGTM |
Lifecycle expiration deletions were suppressed from access logs. Add an expiration flag to distinguish them from other backbeat operations. For lifecycle expiration events: - Operation: S3.EXPIRE.OBJECT - Requester: ScalityS3LifecycleService - ClientIP/httpCode/timing/auth: undefined (log-courier will write - or "-")
121d19d to
b90440f
Compare
|
LGTM |
Codecov Report❌ Patch coverage is
Additional details and impacted files
... and 2 files with indirect coverage changes @@ Coverage Diff @@
## development/9.2 #6150 +/- ##
===================================================
+ Coverage 84.29% 84.37% +0.08%
===================================================
Files 204 204
Lines 13151 13175 +24
===================================================
+ Hits 11086 11117 +31
+ Misses 2065 2058 -7
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
/approve |
ConflictA conflict has been raised during the creation of I have not created the integration branch. Here are the steps to resolve this conflict: git fetch
git checkout -B w/9.3/improvement/CLDSRV-890-log-lifecycle-expiration-events origin/development/9.3
git merge origin/improvement/CLDSRV-890-log-lifecycle-expiration-events
# <intense conflict resolution>
git commit
git push -u origin w/9.3/improvement/CLDSRV-890-log-lifecycle-expiration-eventsThe following options are set: approve |
|
ping |
Integration data createdI have created the integration data for the additional destination branches.
The following branches will NOT be impacted:
You can set option The following options are set: approve |
Build failedThe build for commit did not succeed in branch w/9.3/improvement/CLDSRV-890-log-lifecycle-expiration-events The following options are set: approve |
Build failedThe build for commit did not succeed in branch w/9.4/improvement/CLDSRV-890-log-lifecycle-expiration-events The following options are set: approve |
|
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue CLDSRV-890. Goodbye dvasilas. The following options are set: approve |
Add lifecycle expiration deletions to access logs.
For lifecycle expiration events:
S3.EXPIRE.OBJECTScalityS3LifecycleServiceundefined(log-courier will write-or"-")Test plan