x
/
fuel-plugin-calico
Code Issues Proposed changes

Update patch set 3 

Patch Set 3:

(3 comments)

Patch-set: 3
This commit is contained in:
Gerrit User 16019 2015-05-20 13:51:22 +00:00 committed by Gerrit Code Review
parent a92786ce4e
commit 856b03c875
1 changed files with 54 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

54
631f3396a6f94cb88d40f07bc9769c94924eb89a
View File

@ -17,6 +17,24 @@
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "7a016987_20b89ac7",
"filename": "README.md",
"patchSetId": 3
},
"lineNbr": 91,
"author": {
"id": 16019
},
"writtenOn": "2015-05-20T13:51:22Z",
"side": 1,
"message": "No, it doesn\u0027t mean that - these are just example values. I\u0027ll expand on the \u0027For example\u0027 to hopefully make this clearer.",
"parentUuid": "7a016987_f91b6d71",
"revId": "631f3396a6f94cb88d40f07bc9769c94924eb89a",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "7a016987_9996b93d",
@ -34,6 +52,24 @@
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "7a016987_202e5a43",
"filename": "specs/calico-fuel-plugin.rst",
"patchSetId": 3
},
"lineNbr": 13,
"author": {
"id": 16019
},
"writtenOn": "2015-05-20T13:51:22Z",
"side": 1,
"message": "Done",
"parentUuid": "7a016987_9996b93d",
"revId": "631f3396a6f94cb88d40f07bc9769c94924eb89a",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "7a016987_19cac94e",
@ -50,6 +86,24 @@
"revId": "631f3396a6f94cb88d40f07bc9769c94924eb89a",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "7a016987_1da3cbb3",
"filename": "specs/calico-fuel-plugin.rst",
"patchSetId": 3
},
"lineNbr": 152,
"author": {
"id": 16019
},
"writtenOn": "2015-05-20T13:51:22Z",
"side": 1,
"message": "This is consistent with the level of detail provided for other Fuel plugins, so seems to be what is required by Mirantis here for plugin certification. The test plan/report only go to them, so further details don\u0027t seem of benefit to the general reader. As such, I\u0027d propose that this is fine as it stands. Agree?",
"parentUuid": "7a016987_19cac94e",
"revId": "631f3396a6f94cb88d40f07bc9769c94924eb89a",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
}
]
}