Update patch set 6

Patch Set 6:

(2 comments)

On a general note: I didn't get the external segment changes into this iteration - sorry for the confusion!

Patch-set: 6
Label: Verified=0
This commit is contained in:
Gerrit User 18031 2017-05-04 12:58:13 +00:00 committed by Gerrit Code Review
parent bcf6334e00
commit b3e5461865
1 changed files with 48 additions and 0 deletions

View File

@ -23,6 +23,30 @@
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "3f044301_d888c072",
"filename": "specs/newton/address-scope-subnet-pool-mapping.rst",
"patchSetId": 6
},
"lineNbr": 55,
"author": {
"id": 18031
},
"writtenOn": "2017-05-04T12:58:13Z",
"side": 1,
"message": "I\u0027ll change this to \"with the ml2plus core plugin.\"\n\nWe can opt to use the default subnetpool instead, and I can see the argument for it. It would essentially be the same semantics as proposed for the implicit subnetpool behavior -- GBP would use it if it\u0027s configured (i.e. the opt-in behavior is always present when using GBP workflow).",
"parentUuid": "5ff73747_cf822001",
"range": {
"startLine": 55,
"startChar": 49,
"endLine": 55,
"endChar": 69
},
"revId": "68fd5aa8be6c12d2b59d22aa6594bd32da2d7fd1",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "5ff73747_4ffcd03d",
@ -46,6 +70,30 @@
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "3f044301_f8d13c59",
"filename": "specs/newton/address-scope-subnet-pool-mapping.rst",
"patchSetId": 6
},
"lineNbr": 68,
"author": {
"id": 18031
},
"writtenOn": "2017-05-04T12:58:13Z",
"side": 1,
"message": "I agree -- and will modify this text if the consensus is to use the default subnetpools.",
"parentUuid": "5ff73747_4ffcd03d",
"range": {
"startLine": 68,
"startChar": 39,
"endLine": 68,
"endChar": 50
},
"revId": "68fd5aa8be6c12d2b59d22aa6594bd32da2d7fd1",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "5ff73747_af491361",