f249d2be51
Patch Set 8:
(1 comment)
Patch-set: 8
Attention: {"person_ident":"Gerrit User 28459 \u003c28459@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"\u003cGERRIT_ACCOUNT_33443\u003e replied on the change"}
56 lines
1.8 KiB
Plaintext
56 lines
1.8 KiB
Plaintext
{
|
|
"comments": [
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "8b4549c6_9340137b",
|
|
"filename": "/COMMIT_MSG",
|
|
"patchSetId": 7
|
|
},
|
|
"lineNbr": 12,
|
|
"author": {
|
|
"id": 28459
|
|
},
|
|
"writtenOn": "2022-11-04T20:05:20Z",
|
|
"side": 1,
|
|
"message": "Please include an example here to help reviewers. The less work we have to do, the quicker it merges.\n\nExisting CentOS code running on Debian\n[root@controller-0 ~(keystone_admin)]# findmnt -n -T / -o SOURCE\n/dev/sda5[/ostree/deploy/debian/deploy/bcd4ae00dd670c967b62c7e6e471f9899174f81ab7e379105901e5e88f2aba44.1]\n\nAfter modification:\n[root@controller-0 ~(keystone_admin)]# findmnt -n -T / -o SOURCE | cut -d\u0027[\u0027 -f1 \n/dev/sda5",
|
|
"revId": "f54ee5f7a7f732790f193b1e1f21b66e32fea7bc",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "1952c87f_62886e71",
|
|
"filename": "/COMMIT_MSG",
|
|
"patchSetId": 7
|
|
},
|
|
"lineNbr": 12,
|
|
"author": {
|
|
"id": 33443
|
|
},
|
|
"writtenOn": "2022-11-04T20:22:39Z",
|
|
"side": 1,
|
|
"message": "Done",
|
|
"parentUuid": "8b4549c6_9340137b",
|
|
"revId": "f54ee5f7a7f732790f193b1e1f21b66e32fea7bc",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "291f4114_cd7ed080",
|
|
"filename": "/PATCHSET_LEVEL",
|
|
"patchSetId": 7
|
|
},
|
|
"lineNbr": 0,
|
|
"author": {
|
|
"id": 28459
|
|
},
|
|
"writtenOn": "2022-11-04T20:05:20Z",
|
|
"side": 1,
|
|
"message": "Change is good. Need a commit update. Will bump vote.",
|
|
"revId": "f54ee5f7a7f732790f193b1e1f21b66e32fea7bc",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
}
|
|
]
|
|
} |