openstack
/
neutron-specs
Code Issues Proposed changes
neutron-specs/adcbbf9477c27990c1d9b62bdd7...

318 lines
11 KiB
Plaintext
Raw Blame History

{
"comments": [
{
"key": {
"uuid": "AAAAXX//x5Q\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 14,
"author": {
"id": 261
},
"writtenOn": "2014-04-27T23:43:22Z",
"side": 1,
"message": "I think the API already allows for specifying multiple VIPs and pools.\nI think there is an historical issue that we\u0027re forced into a 1:1 relationship between a VIP and a Pool however. Is that what you mean here?",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//xEQ\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 14,
"author": {
"id": 6072
},
"writtenOn": "2014-04-28T09:06:56Z",
"side": 1,
"message": "current API also bounds VIPs and pools as 1:1.\nYes, I\u0027ll be more explicit here. we need multiple listeners and multiple pools (base line for L7 switching)",
"parentUuid": "AAAAXX//x5Q\u003d",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//x3Q\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 27,
"author": {
"id": 261
},
"writtenOn": "2014-04-27T23:43:22Z",
"side": 1,
"message": "what is the definition of \"loadbalancer\" here?\nA \"logical\" entity as seen by the user, or the backend appliance (hw or sw) implementing the service?\nIn the latter case, I would argue that we should strive not to change the API for problems which concern the backend.\nIn the former case, it would be nice to see a discussion of:\n- problem arising from the fact a \"load balancer\" concept is missing\n- potential advantages of the solution",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//xD0\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 27,
"author": {
"id": 6072
},
"writtenOn": "2014-04-28T09:06:56Z",
"side": 1,
"message": "logical loadbalancer is referred here, of course.\n\n\u003e problem arising from the fact a \"load balancer\" concept is missing\nThat\u0027s a more-then-release-cycle discussion. So far the entity that would represent \u0027logical loadbalancer\u0027 is considered by some people an \u0027Implementation detail\u0027 for some reason. Actually we\u0027re in the middle of that discussion still.",
"parentUuid": "AAAAXX//x3Q\u003d",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//x6E\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 33,
"author": {
"id": 261
},
"writtenOn": "2014-04-27T23:43:22Z",
"side": 1,
"message": "?",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//x6o\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 56,
"author": {
"id": 261
},
"writtenOn": "2014-04-27T23:43:22Z",
"side": 1,
"message": "Is there a life-cycle dependency between the VIF and the listener? Can a listener live without a VIP?",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//xDQ\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 56,
"author": {
"id": 6072
},
"writtenOn": "2014-04-28T09:06:56Z",
"side": 1,
"message": "Will add more description. VIP is a parent object which represents L2/L3 endpoint, where listener represents L4+ endpoint on top of VIP. So listener can\u0027t live without VIP. \nI Will amend the description.",
"parentUuid": "AAAAXX//x6o\u003d",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//x7M\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 65,
"author": {
"id": 261
},
"writtenOn": "2014-04-27T23:43:22Z",
"side": 1,
"message": "This needs to be motivated.\nWhy is the workflow better in this way - in particular, why would it better than a \"flattened\" model where the listener attributes are folded into the VIP resource.\n\nI understand the point about data normalization, but my counterargument is about simplifying the user workflow.",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//xDE\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 65,
"author": {
"id": 6072
},
"writtenOn": "2014-04-28T09:06:56Z",
"side": 1,
"message": "In fact, to be backward compatible we have to have Listeners attributes on the VIP - they\u0027ll go to first listener of the VIP.\nThat\u0027s a good question whether we want to move them away completely then.",
"parentUuid": "AAAAXX//x7M\u003d",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//x7I\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 75,
"author": {
"id": 261
},
"writtenOn": "2014-04-27T23:43:22Z",
"side": 1,
"message": "I find this amusing... there is always an alternative when we talk about APIs!\n\nAlso, the same wiki you linked has alternatives in it!\nI think it would be fair to list them.",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//xC8\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 75,
"author": {
"id": 6072
},
"writtenOn": "2014-04-28T09:06:56Z",
"side": 1,
"message": "Not added yet",
"parentUuid": "AAAAXX//x7I\u003d",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//x7E\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 88,
"author": {
"id": 261
},
"writtenOn": "2014-04-27T23:43:22Z",
"side": 1,
"message": "This has been mentioned several times in this proposal.\nHowever, it\u0027s work unrelated to this blueprint and which will happen as part of another blueprint. It might be worth mentioning it.",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//xC4\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 88,
"author": {
"id": 6072
},
"writtenOn": "2014-04-28T09:06:56Z",
"side": 1,
"message": "Done",
"parentUuid": "AAAAXX//x7E\u003d",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//x6Y\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 101,
"author": {
"id": 261
},
"writtenOn": "2014-04-27T23:43:22Z",
"side": 1,
"message": "please see comment at line 65 regarding whether listener should be a top level resource.",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//x6U\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 122,
"author": {
"id": 261
},
"writtenOn": "2014-04-27T23:43:22Z",
"side": 1,
"message": "for navigation purposes the vip should also have an attribute (read only) returning the list of listeners ids.",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//xC0\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 122,
"author": {
"id": 6072
},
"writtenOn": "2014-04-28T09:06:56Z",
"side": 1,
"message": "agree",
"parentUuid": "AAAAXX//x6U\u003d",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//x6I\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 145,
"author": {
"id": 261
},
"writtenOn": "2014-04-27T23:43:22Z",
"side": 1,
"message": "This is a \"minor\" breakage to backward compatibility API, but still a breakage. It needs to be pointed out in the specification so that it could be decided how to deal with it.",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "AAAAXX//xCs\u003d",
"filename": "specs/juno/lbaas-api-and-objmodel-improvement.rst",
"patchSetId": 2
},
"lineNbr": 145,
"author": {
"id": 6072
},
"writtenOn": "2014-04-28T09:06:56Z",
"side": 1,
"message": "Good catch. We need to deprecate this attribute, not remove.",
"parentUuid": "AAAAXX//x6I\u003d",
"revId": "adcbbf9477c27990c1d9b62bdd7dc45b30a30a39",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
}
]
}
View Git Blame Copy Permalink