Update patch set 3
Patch Set 3: (6 comments) Patch-set: 3
This commit is contained in:
Gerrit User 31825
Gerrit Code Review
parent
fe4387a729
commit
f2f73e5899
|
|
@ -23,6 +23,30 @@
|
|||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "16881cd1_713c4a2c",
|
||||
"filename": "specs/xena/implemented/vm-evacuations-for-host-recovery.rst",
|
||||
"patchSetId": 3
|
||||
},
|
||||
"lineNbr": 70,
|
||||
"author": {
|
||||
"id": 31825
|
||||
},
|
||||
"writtenOn": "2021-07-30T09:52:48Z",
|
||||
"side": 1,
|
||||
"message": "I still think source_host is necessary to see the source node and destination node of the Instance more intuitively. Let\u0027s discuss it next week.",
|
||||
"parentUuid": "a83d448e_a03a3481",
|
||||
"range": {
|
||||
"startLine": 69,
|
||||
"startChar": 0,
|
||||
"endLine": 70,
|
||||
"endChar": 20
|
||||
},
|
||||
"revId": "56bc6c139d786f9b2eb7ba50eee6a2525d343cfe",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "810c725c_51d05f16",
|
||||
|
|
@ -46,6 +70,30 @@
|
|||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "95388af3_e57520ee",
|
||||
"filename": "specs/xena/implemented/vm-evacuations-for-host-recovery.rst",
|
||||
"patchSetId": 3
|
||||
},
|
||||
"lineNbr": 74,
|
||||
"author": {
|
||||
"id": 31825
|
||||
},
|
||||
"writtenOn": "2021-07-30T09:52:48Z",
|
||||
"side": 1,
|
||||
"message": "As mentioned before, I will remove the ‘evacuate_’ prefix.",
|
||||
"parentUuid": "810c725c_51d05f16",
|
||||
"range": {
|
||||
"startLine": 74,
|
||||
"startChar": 84,
|
||||
"endLine": 74,
|
||||
"endChar": 85
|
||||
},
|
||||
"revId": "56bc6c139d786f9b2eb7ba50eee6a2525d343cfe",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "42e24506_f588f42d",
|
||||
|
|
@ -68,6 +116,30 @@
|
|||
"revId": "56bc6c139d786f9b2eb7ba50eee6a2525d343cfe",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "74797336_e938a0c0",
|
||||
"filename": "specs/xena/implemented/vm-evacuations-for-host-recovery.rst",
|
||||
"patchSetId": 3
|
||||
},
|
||||
"lineNbr": 75,
|
||||
"author": {
|
||||
"id": 31825
|
||||
},
|
||||
"writtenOn": "2021-07-30T09:52:48Z",
|
||||
"side": 1,
|
||||
"message": "Done",
|
||||
"parentUuid": "42e24506_f588f42d",
|
||||
"range": {
|
||||
"startLine": 75,
|
||||
"startChar": 70,
|
||||
"endLine": 75,
|
||||
"endChar": 78
|
||||
},
|
||||
"revId": "56bc6c139d786f9b2eb7ba50eee6a2525d343cfe",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
}
|
||||
]
|
||||
}
|
||||
|
|
@ -1060,6 +1060,24 @@
|
|||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "f030333a_e396abf8",
|
||||
"filename": "specs/xena/implemented/vm-evacuations-for-host-recovery.rst",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 90,
|
||||
"author": {
|
||||
"id": 31825
|
||||
},
|
||||
"writtenOn": "2021-07-30T09:52:48Z",
|
||||
"side": 1,
|
||||
"message": "I don\u0027t think it is necessary to include migration_uuid. It is necessary to check the details of the Instance. After the state is ok, the dest_host field can be obtained naturally. However, if you need to obtain the migration_uuid, you need to query the migration record of the Instance and obtain the latest migration_uuid. This increases the query steps and affects the efficiency of the operation.",
|
||||
"parentUuid": "5cf38657_ecd197c7",
|
||||
"revId": "7799bda702d83f28406249e2c8babc3f4cae0bfe",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "359a85fc_d632460a",
|
||||
|
|
@ -1131,6 +1149,30 @@
|
|||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "5af27acc_fc731239",
|
||||
"filename": "specs/xena/implemented/vm-evacuations-for-host-recovery.rst",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 92,
|
||||
"author": {
|
||||
"id": 31825
|
||||
},
|
||||
"writtenOn": "2021-07-30T09:52:48Z",
|
||||
"side": 1,
|
||||
"message": "Okay, thanks a lot, I will change this.",
|
||||
"parentUuid": "dc1e1445_7654726b",
|
||||
"range": {
|
||||
"startLine": 92,
|
||||
"startChar": 55,
|
||||
"endLine": 92,
|
||||
"endChar": 64
|
||||
},
|
||||
"revId": "7799bda702d83f28406249e2c8babc3f4cae0bfe",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "d0363dac_3a9e3253",
|
||||
|
|
@ -1306,6 +1348,24 @@
|
|||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "cbd08262_1ba3ebc8",
|
||||
"filename": "specs/xena/implemented/vm-evacuations-for-host-recovery.rst",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 211,
|
||||
"author": {
|
||||
"id": 31825
|
||||
},
|
||||
"writtenOn": "2021-07-30T09:52:48Z",
|
||||
"side": 1,
|
||||
"message": "Oh,thanks.",
|
||||
"parentUuid": "c7f64bbd_bda595a7",
|
||||
"revId": "7799bda702d83f28406249e2c8babc3f4cae0bfe",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "78e3a21d_418f06e9",
|
||||
|
|
|
|||
Loading…
Reference in New Issue