openstack
/
fuel-specs
Code Issues Proposed changes

Update patch set 8 

Patch Set 8:

(1 comment)

Patch-set: 8
Label: Verified=0
This commit is contained in:
Gerrit User 6677 2016-10-25 04:56:38 +00:00 committed by Gerrit Code Review
parent 2af979f8c6
commit 76a61bd176
1 changed files with 23 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
821597a87d546ced86e2ff7d55a80f60775936fc
View File

@ -94,6 +94,29 @@
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "da6895a0_222af92c",
"filename": "specs/11.0/fuel-security-groups-switch.rst",
"patchSetId": 8
},
"lineNbr": 61,
"author": {
"id": 6677
},
"writtenOn": "2016-10-25T04:56:38Z",
"side": 1,
"message": "Why you define it as a boolean switch? It is more natural and correct from the deployment data standpoint to select from a list of supported drivers, even it has just two (iptables and ovs native) at the moment.",
"range": {
"startLine": 52,
"startChar": 0,
"endLine": 61,
"endChar": 28
},
"revId": "821597a87d546ced86e2ff7d55a80f60775936fc",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "da6895a0_ca596a5c",