125 lines
4.1 KiB
Plaintext
125 lines
4.1 KiB
Plaintext
{
|
|
"comments": [
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "be210043_445ba0e1",
|
|
"filename": ".zuul.yaml",
|
|
"patchSetId": 1
|
|
},
|
|
"lineNbr": 137,
|
|
"author": {
|
|
"id": 9535
|
|
},
|
|
"writtenOn": "2023-05-17T16:28:10Z",
|
|
"side": 1,
|
|
"message": "Why are we disabling live migration? Is it because of the bug? If that\u0027s the case then shouldn\u0027t we mention it here in a comment?",
|
|
"revId": "bf4598d923bebe9ba31e6651a07596017208b37e",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "0280bf0e_cd46c949",
|
|
"filename": ".zuul.yaml",
|
|
"patchSetId": 1
|
|
},
|
|
"lineNbr": 137,
|
|
"author": {
|
|
"id": 4393
|
|
},
|
|
"writtenOn": "2023-05-17T16:30:59Z",
|
|
"side": 1,
|
|
"message": "We\u0027re not disabling live migration, we\u0027re disabling *block* migration during live migration. \"Block migration\" means that qemu copies the disks in addition to the memory when it moves the instance. This is only needed/useful for live migration between hosts that do *not* have shared storage. However, this flag _actually_ just enables a flag in tempest to validate a legacy nova api where you tell it what to do instead of it figuring it out itself. So having it enabled causes us to tell nova on shared storage to do something it shouldn\u0027t do in that environment, and is why some tests were failing with \"you asked me to migrate a shared device and I know that\u0027s wrong\".",
|
|
"parentUuid": "be210043_445ba0e1",
|
|
"revId": "bf4598d923bebe9ba31e6651a07596017208b37e",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "eb5af9d4_fbb5dd40",
|
|
"filename": ".zuul.yaml",
|
|
"patchSetId": 1
|
|
},
|
|
"lineNbr": 137,
|
|
"author": {
|
|
"id": 9535
|
|
},
|
|
"writtenOn": "2023-05-17T16:32:22Z",
|
|
"side": 1,
|
|
"message": "Ooooooh, I see. Thank you for the detailed explanation.",
|
|
"parentUuid": "0280bf0e_cd46c949",
|
|
"revId": "bf4598d923bebe9ba31e6651a07596017208b37e",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "5e267288_9237b247",
|
|
"filename": "/PATCHSET_LEVEL",
|
|
"patchSetId": 1
|
|
},
|
|
"lineNbr": 0,
|
|
"author": {
|
|
"id": 8556
|
|
},
|
|
"writtenOn": "2023-05-17T01:02:58Z",
|
|
"side": 1,
|
|
"message": "lgtm, I saw there are some other test failing but at least this change correct the configuation for block-migration",
|
|
"revId": "bf4598d923bebe9ba31e6651a07596017208b37e",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "c8111d1f_6004070d",
|
|
"filename": "/PATCHSET_LEVEL",
|
|
"patchSetId": 1
|
|
},
|
|
"lineNbr": 0,
|
|
"author": {
|
|
"id": 4393
|
|
},
|
|
"writtenOn": "2023-05-23T13:50:18Z",
|
|
"side": 1,
|
|
"message": "recheck bug 2020122",
|
|
"revId": "bf4598d923bebe9ba31e6651a07596017208b37e",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "8267641a_e3264978",
|
|
"filename": "/PATCHSET_LEVEL",
|
|
"patchSetId": 1
|
|
},
|
|
"lineNbr": 0,
|
|
"author": {
|
|
"id": 4393
|
|
},
|
|
"writtenOn": "2023-05-23T15:53:08Z",
|
|
"side": 1,
|
|
"message": "recheck devstack setup failed in a different job",
|
|
"revId": "bf4598d923bebe9ba31e6651a07596017208b37e",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "bbf78e45_d5b9b388",
|
|
"filename": "/PATCHSET_LEVEL",
|
|
"patchSetId": 1
|
|
},
|
|
"lineNbr": 0,
|
|
"author": {
|
|
"id": 4393
|
|
},
|
|
"writtenOn": "2023-05-23T19:49:10Z",
|
|
"side": 1,
|
|
"message": "recheck job timed out but was clearly making progress. likely slow node.",
|
|
"revId": "bf4598d923bebe9ba31e6651a07596017208b37e",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
}
|
|
]
|
|
} |