Update patch set 5

Patch Set 5:

(2 comments)

Patch-set: 5
This commit is contained in:
Gerrit User 8766 2016-11-09 07:54:08 +00:00 committed by Gerrit Code Review
parent 91e46372aa
commit 2bd7c65000
1 changed files with 50 additions and 0 deletions

View File

@ -0,0 +1,50 @@
{
"comments": [
{
"key": {
"uuid": "9a629dbe_5aa268af",
"filename": "nailgun/nailgun/fixtures/openstack.yaml",
"patchSetId": 5
},
"lineNbr": 105,
"author": {
"id": 8766
},
"writtenOn": "2016-11-09T07:54:08Z",
"side": 1,
"message": "Should it be images_ceph? I guess so according to the new L1511 restriction. Also, a message for this restriction says about Ceph for Images.",
"range": {
"startLine": 105,
"startChar": 41,
"endLine": 105,
"endChar": 53
},
"revId": "86c6acf49c4d48fa75b670260f264e8d9332956e",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9a629dbe_da7d981d",
"filename": "nailgun/nailgun/fixtures/openstack.yaml",
"patchSetId": 5
},
"lineNbr": 1511,
"author": {
"id": 8766
},
"writtenOn": "2016-11-09T07:54:08Z",
"side": 1,
"message": "imho \"enabled\" would be more appropriate here and will correspond other restriction messages of cluster settings.",
"range": {
"startLine": 1511,
"startChar": 94,
"endLine": 1511,
"endChar": 102
},
"revId": "86c6acf49c4d48fa75b670260f264e8d9332956e",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
}
]
}