e3c2ea2bac
Patch Set 8: (1 comment) Patch-set: 8
73 lines
2.6 KiB
Plaintext
73 lines
2.6 KiB
Plaintext
{
|
|
"comments": [
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "d42d306c_1f65cfe3",
|
|
"filename": "/PATCHSET_LEVEL",
|
|
"patchSetId": 8
|
|
},
|
|
"lineNbr": 0,
|
|
"author": {
|
|
"id": 32594
|
|
},
|
|
"writtenOn": "2024-01-10T20:27:13Z",
|
|
"side": 1,
|
|
"message": "recheck old logs",
|
|
"revId": "98901bd27a07478a46f95b933d443a959215ff92",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "b5b3a5cc_e8f6f885",
|
|
"filename": "/PATCHSET_LEVEL",
|
|
"patchSetId": 8
|
|
},
|
|
"lineNbr": 0,
|
|
"author": {
|
|
"id": 16643
|
|
},
|
|
"writtenOn": "2024-01-31T22:50:14Z",
|
|
"side": 1,
|
|
"message": "Thanks for working on this Ashley.. this is a mega cherry-pick to fix a bunch of issues; and it enables testing with cephadm and multinode. I\u0027m inclined to get this to merge and address the remaining issues in further patches: \n\n\nboth native cephfs and nfs jobs are failing:\n\n1) native cephfs has hit a tempest setup issue; unrelated.. lets try to address this in a different change.. \n2) cephfs/nfs : ceph-mon command not found.. this worked in the native cephfs job: https://zuul.opendev.org/t/openstack/build/6f22737940c14552bec39846a30fbc0c/log/controller/logs/devstacklog.txt#3454 \nthe difference is that this job uses centos-stream-9 and the other one uses Ubuntu Jammy. \"ceph-mon\" is a separate package in centos-stream-9. so we can add it and further triage failures in a different change.",
|
|
"revId": "98901bd27a07478a46f95b933d443a959215ff92",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "dfa78f9b_93f02fdf",
|
|
"filename": "/PATCHSET_LEVEL",
|
|
"patchSetId": 8
|
|
},
|
|
"lineNbr": 0,
|
|
"author": {
|
|
"id": 29632
|
|
},
|
|
"writtenOn": "2024-02-08T14:53:21Z",
|
|
"side": 1,
|
|
"message": "LGTM, thank you for working on this change! :)",
|
|
"revId": "98901bd27a07478a46f95b933d443a959215ff92",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "dfa2bf04_0de0056e",
|
|
"filename": "/PATCHSET_LEVEL",
|
|
"patchSetId": 8
|
|
},
|
|
"lineNbr": 0,
|
|
"author": {
|
|
"id": 29632
|
|
},
|
|
"writtenOn": "2024-02-08T14:53:36Z",
|
|
"side": 1,
|
|
"message": "TY for clarifying :)",
|
|
"parentUuid": "b5b3a5cc_e8f6f885",
|
|
"revId": "98901bd27a07478a46f95b933d443a959215ff92",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
}
|
|
]
|
|
} |