Update patch set 11
Patch Set 11: (2 comments) Patch-set: 11 Reviewer: Gerrit User 6469 <6469@4a232e18-c5a9-48ee-94c0-e04e7cca6543>
This commit is contained in:
parent
234388b811
commit
153fdf0abf
|
@ -0,0 +1,50 @@
|
|||
{
|
||||
"comments": [
|
||||
{
|
||||
"key": {
|
||||
"uuid": "2e00d691_a1402e4f",
|
||||
"filename": "octavia_tempest_plugin/tests/scenario/v2/test_traffic_ops.py",
|
||||
"patchSetId": 11
|
||||
},
|
||||
"lineNbr": 1406,
|
||||
"author": {
|
||||
"id": 6469
|
||||
},
|
||||
"writtenOn": "2021-06-01T08:28:26Z",
|
||||
"side": 1,
|
||||
"message": "The _listener_pool_create call above will already ensure the load balancer is in provisioning status ACTIVE. In principle running this check again wouldn\u0027t hurt but it is unnecessary and takes up a bit more resources and time which we\u0027ve been trying to cut back due to limited CI resources.",
|
||||
"range": {
|
||||
"startLine": 1402,
|
||||
"startChar": 8,
|
||||
"endLine": 1406,
|
||||
"endChar": 65
|
||||
},
|
||||
"revId": "93fd31dbb8792557a4e022a7c41e8cbadf88eb25",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "c7b68d9c_a789c77a",
|
||||
"filename": "octavia_tempest_plugin/tests/scenario/v2/test_traffic_ops.py",
|
||||
"patchSetId": 11
|
||||
},
|
||||
"lineNbr": 1409,
|
||||
"author": {
|
||||
"id": 6469
|
||||
},
|
||||
"writtenOn": "2021-06-01T08:28:26Z",
|
||||
"side": 1,
|
||||
"message": "Any particular reason to disable session persistence?",
|
||||
"range": {
|
||||
"startLine": 1409,
|
||||
"startChar": 33,
|
||||
"endLine": 1409,
|
||||
"endChar": 49
|
||||
},
|
||||
"revId": "93fd31dbb8792557a4e022a7c41e8cbadf88eb25",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
}
|
||||
]
|
||||
}
|
Loading…
Reference in New Issue