zuul
/
zuul-jobs
Code Issues Proposed changes

Update patch set 6 

Patch Set 6: Code-Review-1

(1 comment)

Patch-set: 6
Reviewer: Gerrit User 27582 <27582@4a232e18-c5a9-48ee-94c0-e04e7cca6543>
Label: Code-Review=-1
Attention: {"person_ident":"Gerrit User 31083 \u003c31083@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"\u003cGERRIT_ACCOUNT_27582\u003e replied on the change"}
This commit is contained in:
Gerrit User 27582 2022-09-26 05:28:46 +00:00 committed by Gerrit Code Review
parent afe0bbaa5a
commit 3ed608c585
1 changed files with 21 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
6b5ad43257a9f62dce4d0188ec2d6a72b0f64bea Normal file
View File

@ -0,0 +1,21 @@
{
"comments": [
{
"unresolved": true,
"key": {
"uuid": "c7cff776_51e177b4",
"filename": "roles/upload-logs-base/library/zuul_s3_upload.py",
"patchSetId": 6
},
"lineNbr": 168,
"author": {
"id": 27582
},
"writtenOn": "2022-09-26T05:28:46Z",
"side": 1,
"message": "We had to set `delete_after_days` to 0 to disable update of the lifecycle config on every upload as we encountered the following error when there were a lot of uploads in parallel:\n\n\u003e An error occurred (OperationAborted) when calling the PutBucketLifecycleConfiguration operation: A conflicting conditional operation is currently in progress against this resource. Please try again.\n\nMaybe we can check if the lifecylce config is already correct before attempting an update. See discussion in https://review.opendev.org/c/zuul/zuul-jobs/+/858726",
"revId": "6b5ad43257a9f62dce4d0188ec2d6a72b0f64bea",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}