Update patch set 3

Patch Set 3: Code-Review-1

(2 comments)

Patch-set: 3
Reviewer: Gerrit User 8367 <8367@4a232e18-c5a9-48ee-94c0-e04e7cca6543>
Label: Code-Review=-1
Attention: {"person_ident":"Gerrit User 8367 \u003c8367@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"REMOVE","reason":"Arx Cruz replied on the change"}
This commit is contained in:
Gerrit User 8367 2021-11-30 12:04:20 +00:00 committed by Gerrit Code Review
parent 7653a35dc4
commit 6b90bac97e
1 changed files with 40 additions and 0 deletions

View File

@ -1,5 +1,28 @@
{
"comments": [
{
"unresolved": true,
"key": {
"uuid": "d21302e3_db065bc0",
"filename": "/COMMIT_MSG",
"patchSetId": 3
},
"lineNbr": 15,
"author": {
"id": 8367
},
"writtenOn": "2021-11-30T12:04:20Z",
"side": 1,
"message": "Attempt you mean, you\u0027re not sure if this will fix the problem for Centos 9? Do we have a testproject testing it on Centos 7, 8 and 9? We can\u0027t fix it for Centos 9 and break for the other previous versions.",
"range": {
"startLine": 14,
"startChar": 15,
"endLine": 15,
"endChar": 11
},
"revId": "1152704e2f97f81af06fa4983ef4393f5bcc339d",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
@ -16,6 +39,23 @@
"message": "Please change the first line as I explained before.\n\nMoreover, the need for something in c9 is not a reason for this change. If the parameter is missing in some specific deployment, then it should be added to that deployment, unless it can be detected.\n\nIf you think this is the new default and should be changed for this reason, please explain why.",
"revId": "1152704e2f97f81af06fa4983ef4393f5bcc339d",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "849ad97f_2788e277",
"filename": "config_tempest/services/image.py",
"patchSetId": 3
},
"lineNbr": 62,
"author": {
"id": 8367
},
"writtenOn": "2021-11-30T12:04:20Z",
"side": 1,
"message": "Please move this up and add proper comments. The comment above is for the line below, make it confuse for people in the future",
"revId": "1152704e2f97f81af06fa4983ef4393f5bcc339d",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}