Update patch set 5

Patch Set 5: Code-Review-1

(2 comments)

Thanks Hemanth for working on this spec. Please see the comments in patch set 4 and 5.

Patch-set: 5
Label: Code-Review=-1
This commit is contained in:
Gerrit User 490
2014-10-17 01:42:28 +00:00
committed by Gerrit Code Review
parent 407cf9af97
commit bf32e4795a

View File

@@ -0,0 +1,38 @@
{
"comments": [
{
"key": {
"uuid": "baa201ad_98d892f0",
"filename": "specs/juno/group-based-policy-service-chaining.rst",
"patchSetId": 5
},
"lineNbr": 109,
"author": {
"id": 490
},
"writtenOn": "2014-10-17T01:42:28Z",
"side": 1,
"message": "How are these different from the ones that are present in the Heat template? It might be good to provide a specific example with the Heat template.",
"revId": "6fa1e1c997a696abb7ab5d889a86e5e97ebfa70d",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "baa201ad_7804ce49",
"filename": "specs/juno/group-based-policy-service-chaining.rst",
"patchSetId": 5
},
"lineNbr": 149,
"author": {
"id": 490
},
"writtenOn": "2014-10-17T01:42:28Z",
"side": 1,
"message": "I dont think its accurate to use RO for arguments that can be provided at the time of create, I think what you are trying to indicate is W at the time of create is allowed but not during update.\n\nSame comment applies below as well.",
"revId": "6fa1e1c997a696abb7ab5d889a86e5e97ebfa70d",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
}
]
}