61 lines
1.7 KiB
Plaintext
61 lines
1.7 KiB
Plaintext
{
|
|
"comments": [
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "99fe7582_aea7b2ce",
|
|
"filename": "/PATCHSET_LEVEL",
|
|
"patchSetId": 4
|
|
},
|
|
"lineNbr": 0,
|
|
"author": {
|
|
"id": 9926
|
|
},
|
|
"writtenOn": "2022-08-31T12:03:32Z",
|
|
"side": 1,
|
|
"message": "pending responses to Bob\u0027s comments",
|
|
"revId": "ce80e6c9c999a88d2bacdb93a4d766b4bbcdfa77",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "f7129646_2a90be1b",
|
|
"filename": "sw-patch/cgcs-patch/cgcs_patch/patch_agent.py",
|
|
"patchSetId": 4
|
|
},
|
|
"lineNbr": 34,
|
|
"author": {
|
|
"id": 28459
|
|
},
|
|
"writtenOn": "2022-08-31T06:24:13Z",
|
|
"side": 1,
|
|
"message": "Consider changing to \"patch_deployments_cleaned\". Might be clearer what it currently signifies",
|
|
"range": {
|
|
"startLine": 34,
|
|
"startChar": 0,
|
|
"endLine": 34,
|
|
"endChar": 31
|
|
},
|
|
"revId": "ce80e6c9c999a88d2bacdb93a4d766b4bbcdfa77",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "71b4cfa0_f516b020",
|
|
"filename": "sw-patch/cgcs-patch/cgcs_patch/patch_agent.py",
|
|
"patchSetId": 4
|
|
},
|
|
"lineNbr": 697,
|
|
"author": {
|
|
"id": 28459
|
|
},
|
|
"writtenOn": "2022-08-31T06:24:13Z",
|
|
"side": 1,
|
|
"message": "Should we set the flag at the end of delete_older_deployments() or at line 420 above incase there is an exception when running the ostree commands so that we re-try?",
|
|
"revId": "ce80e6c9c999a88d2bacdb93a4d766b4bbcdfa77",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
}
|
|
]
|
|
} |