Update patch set 2
Patch Set 2: (9 comments) Patch-set: 2 Label: Verified=0
This commit is contained in:
Gerrit User 8716
Gerrit Code Review
parent
0d137bb4ff
commit
f3f94480f5
|
|
@ -23,6 +23,30 @@
|
|||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_0c00ab00",
|
||||
"filename": "specs/newton/approved/newton-instance-ha-vm-recovery-spec.rst",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 13,
|
||||
"author": {
|
||||
"id": 8716
|
||||
},
|
||||
"writtenOn": "2016-12-21T08:25:11Z",
|
||||
"side": 1,
|
||||
"message": "Done",
|
||||
"parentUuid": "1a6eadb0_3295d5dd",
|
||||
"range": {
|
||||
"startLine": 13,
|
||||
"startChar": 18,
|
||||
"endLine": 13,
|
||||
"endChar": 24
|
||||
},
|
||||
"revId": "35195b4de0890ebf7fdeb9ef10749556eceddf3f",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_f2b19d90",
|
||||
|
|
@ -46,6 +70,30 @@
|
|||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_2c056710",
|
||||
"filename": "specs/newton/approved/newton-instance-ha-vm-recovery-spec.rst",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 22,
|
||||
"author": {
|
||||
"id": 8716
|
||||
},
|
||||
"writtenOn": "2016-12-21T08:25:11Z",
|
||||
"side": 1,
|
||||
"message": "Done",
|
||||
"parentUuid": "1a6eadb0_f2b19d90",
|
||||
"range": {
|
||||
"startLine": 22,
|
||||
"startChar": 21,
|
||||
"endLine": 22,
|
||||
"endChar": 30
|
||||
},
|
||||
"revId": "35195b4de0890ebf7fdeb9ef10749556eceddf3f",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_32315506",
|
||||
|
|
@ -69,6 +117,30 @@
|
|||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_4c06231d",
|
||||
"filename": "specs/newton/approved/newton-instance-ha-vm-recovery-spec.rst",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 23,
|
||||
"author": {
|
||||
"id": 8716
|
||||
},
|
||||
"writtenOn": "2016-12-21T08:25:11Z",
|
||||
"side": 1,
|
||||
"message": "Done",
|
||||
"parentUuid": "1a6eadb0_32315506",
|
||||
"range": {
|
||||
"startLine": 22,
|
||||
"startChar": 32,
|
||||
"endLine": 23,
|
||||
"endChar": 24
|
||||
},
|
||||
"revId": "35195b4de0890ebf7fdeb9ef10749556eceddf3f",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_6d2770b6",
|
||||
|
|
@ -92,6 +164,30 @@
|
|||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_6c0bdf24",
|
||||
"filename": "specs/newton/approved/newton-instance-ha-vm-recovery-spec.rst",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 25,
|
||||
"author": {
|
||||
"id": 8716
|
||||
},
|
||||
"writtenOn": "2016-12-21T08:25:11Z",
|
||||
"side": 1,
|
||||
"message": "Done",
|
||||
"parentUuid": "1a6eadb0_6d2770b6",
|
||||
"range": {
|
||||
"startLine": 25,
|
||||
"startChar": 8,
|
||||
"endLine": 25,
|
||||
"endChar": 11
|
||||
},
|
||||
"revId": "35195b4de0890ebf7fdeb9ef10749556eceddf3f",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_2de9f814",
|
||||
|
|
@ -115,6 +211,30 @@
|
|||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_2f2d5446",
|
||||
"filename": "specs/newton/approved/newton-instance-ha-vm-recovery-spec.rst",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 49,
|
||||
"author": {
|
||||
"id": 8716
|
||||
},
|
||||
"writtenOn": "2016-12-21T08:25:11Z",
|
||||
"side": 1,
|
||||
"message": "IMHO, if VM hang or crash because of problem with libvert, then there is a high possibility that you gonna fail to live migrate the VMs. On the other hand, VM stop and cold migration would be much safer.",
|
||||
"parentUuid": "1a6eadb0_2de9f814",
|
||||
"range": {
|
||||
"startLine": 49,
|
||||
"startChar": 0,
|
||||
"endLine": 49,
|
||||
"endChar": 24
|
||||
},
|
||||
"revId": "35195b4de0890ebf7fdeb9ef10749556eceddf3f",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_cd5c04f0",
|
||||
|
|
@ -138,6 +258,30 @@
|
|||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_0f189874",
|
||||
"filename": "specs/newton/approved/newton-instance-ha-vm-recovery-spec.rst",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 60,
|
||||
"author": {
|
||||
"id": 8716
|
||||
},
|
||||
"writtenOn": "2016-12-21T08:25:11Z",
|
||||
"side": 1,
|
||||
"message": "IMHO, its better not assign new VMs to this host till further investigation about what went wrong. I prefer first disable nova-compute before take any recovery actions.",
|
||||
"parentUuid": "1a6eadb0_cd5c04f0",
|
||||
"range": {
|
||||
"startLine": 59,
|
||||
"startChar": 0,
|
||||
"endLine": 60,
|
||||
"endChar": 41
|
||||
},
|
||||
"revId": "35195b4de0890ebf7fdeb9ef10749556eceddf3f",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_4d1714ed",
|
||||
|
|
@ -161,6 +305,30 @@
|
|||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_0f2fb888",
|
||||
"filename": "specs/newton/approved/newton-instance-ha-vm-recovery-spec.rst",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 84,
|
||||
"author": {
|
||||
"id": 8716
|
||||
},
|
||||
"writtenOn": "2016-12-21T08:25:11Z",
|
||||
"side": 1,
|
||||
"message": "Done",
|
||||
"parentUuid": "1a6eadb0_4d1714ed",
|
||||
"range": {
|
||||
"startLine": 84,
|
||||
"startChar": 3,
|
||||
"endLine": 84,
|
||||
"endChar": 7
|
||||
},
|
||||
"revId": "35195b4de0890ebf7fdeb9ef10749556eceddf3f",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_0d927c57",
|
||||
|
|
@ -184,6 +352,30 @@
|
|||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_6f54acfd",
|
||||
"filename": "specs/newton/approved/newton-instance-ha-vm-recovery-spec.rst",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 87,
|
||||
"author": {
|
||||
"id": 8716
|
||||
},
|
||||
"writtenOn": "2016-12-21T08:25:11Z",
|
||||
"side": 1,
|
||||
"message": "I prefer option #3, because operator can control overall recovery workflows via masakari (not only the VM recovery, but also the host and process recovery and any other recovery might appear in future..). This way we can make mistral work flows and monitors much simple.",
|
||||
"parentUuid": "1a6eadb0_0d927c57",
|
||||
"range": {
|
||||
"startLine": 86,
|
||||
"startChar": 0,
|
||||
"endLine": 87,
|
||||
"endChar": 20
|
||||
},
|
||||
"revId": "35195b4de0890ebf7fdeb9ef10749556eceddf3f",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_6dd4d01f",
|
||||
|
|
@ -206,6 +398,30 @@
|
|||
"revId": "35195b4de0890ebf7fdeb9ef10749556eceddf3f",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "1a6eadb0_af42443c",
|
||||
"filename": "specs/newton/approved/newton-instance-ha-vm-recovery-spec.rst",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 89,
|
||||
"author": {
|
||||
"id": 8716
|
||||
},
|
||||
"writtenOn": "2016-12-21T08:25:11Z",
|
||||
"side": 1,
|
||||
"message": "Done",
|
||||
"parentUuid": "1a6eadb0_6dd4d01f",
|
||||
"range": {
|
||||
"startLine": 89,
|
||||
"startChar": 3,
|
||||
"endLine": 89,
|
||||
"endChar": 7
|
||||
},
|
||||
"revId": "35195b4de0890ebf7fdeb9ef10749556eceddf3f",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
}
|
||||
]
|
||||
}
|
||||
Loading…
Reference in New Issue