39 lines
1.3 KiB
Plaintext
39 lines
1.3 KiB
Plaintext
{
|
|
"comments": [
|
|
{
|
|
"key": {
|
|
"uuid": "6cc1e7ff_74356ba6",
|
|
"filename": "kuryr_kubernetes/controller/drivers/lbaasv2.py",
|
|
"patchSetId": 1
|
|
},
|
|
"lineNbr": 84,
|
|
"author": {
|
|
"id": 23567
|
|
},
|
|
"writtenOn": "2021-02-25T08:12:05Z",
|
|
"side": 0,
|
|
"message": "this change perhaps does not belong to this patch, but to a different (bug/typo) one, as we did not realize the if was in the wrong place when adding SCTP support. Anyway, as there has not been any release in between, perhaps it can be quickly done and merge in a different patch without a need for a launchpad bug (no strong opinion though)",
|
|
"revId": "10ea8698586bd172a3d4b0215f6774102348ac69",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
|
"unresolved": true
|
|
},
|
|
{
|
|
"key": {
|
|
"uuid": "2b03cdd1_67fa1c31",
|
|
"filename": "kuryr_kubernetes/controller/drivers/lbaasv2.py",
|
|
"patchSetId": 1
|
|
},
|
|
"lineNbr": 84,
|
|
"author": {
|
|
"id": 32533
|
|
},
|
|
"writtenOn": "2021-02-25T08:42:53Z",
|
|
"side": 0,
|
|
"message": "Alright, that works. I will, thanks.",
|
|
"parentUuid": "6cc1e7ff_74356ba6",
|
|
"revId": "10ea8698586bd172a3d4b0215f6774102348ac69",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
|
"unresolved": true
|
|
}
|
|
]
|
|
} |