devstack-plugin-ceph/ff1d8967fe574a7ac117f4206cd...

73 lines
2.2 KiB
Plaintext

{
"comments": [
{
"unresolved": false,
"key": {
"uuid": "cc7ab3ac_f9b0ef6e",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 7
},
"lineNbr": 0,
"author": {
"id": 20813
},
"writtenOn": "2022-01-06T20:03:14Z",
"side": 1,
"message": "Looks like devstack-plugin-ceph-cephfs-* are broken apart from this work. Waiting for more reviews but I think It\u0027s ready for +2",
"revId": "ff1d8967fe574a7ac117f4206cd17e873ab63d9b",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "df9b72e3_9ef7b8e1",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 7
},
"lineNbr": 0,
"author": {
"id": 20813
},
"writtenOn": "2022-01-24T15:55:55Z",
"side": 1,
"message": "is this patch going to be affected by the migration to cephadm? ",
"revId": "ff1d8967fe574a7ac117f4206cd17e873ab63d9b",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "338c9967_b21d2f8c",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 7
},
"lineNbr": 0,
"author": {
"id": 10459
},
"writtenOn": "2022-03-25T14:54:00Z",
"side": 1,
"message": "I don\u0027t know, but it\u0027s the new code that should be adapted. \n\nGiven the fact this change has proven to be problematic, maybe it should be under a condition, so we don\u0027t break anyone else.",
"parentUuid": "df9b72e3_9ef7b8e1",
"revId": "ff1d8967fe574a7ac117f4206cd17e873ab63d9b",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "9575fd42_b5894cfd",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 7
},
"lineNbr": 0,
"author": {
"id": 20813
},
"writtenOn": "2022-03-25T14:57:11Z",
"side": 1,
"message": "Luigi is going to create a variable for this +1",
"revId": "ff1d8967fe574a7ac117f4206cd17e873ab63d9b",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}