openstack
/
neutron-specs
Code Issues Proposed changes

Update patch set 4 

Patch Set 4:

(25 comments)

Patch-set: 4
Attention: {"person_ident":"Gerrit User 35432 \u003c35432@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"REMOVE","reason":"\u003cGERRIT_ACCOUNT_35432\u003e replied on the change"}
Attention: {"person_ident":"Gerrit User 4694 \u003c4694@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"\u003cGERRIT_ACCOUNT_35432\u003e replied on the change"}
This commit is contained in:
Gerrit User 35432 2023-10-20 13:10:35 +00:00 committed by Gerrit Code Review
parent fa5ff5f610
commit b0c161868c
2 changed files with 591 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

570
0e8d530ba16d94428a6a9581107003516ae76648
View File

@ -40,6 +40,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "2adc00a3_123085b4",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 109,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "oh, sorry for the mistake, and thank you so much for your help.",
"parentUuid": "c5e75da3_a87c7283",
"range": {
"startLine": 109,
"startChar": 11,
"endLine": 109,
"endChar": 19
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -63,6 +87,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "2aad0723_a03d6aa1",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 175,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "85db8f59_4c088674",
"range": {
"startLine": 175,
"startChar": 3,
"endLine": 175,
"endChar": 11
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -86,6 +134,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "d0e594c8_0f44fa1f",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 195,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "That\u0027s right, the LSP of the left OVN deployment (in the same TS) is automatically created by ovn-ic daemon.",
"parentUuid": "23d23500_c5a9ccfc",
"range": {
"startLine": 195,
"startChar": 2,
"endLine": 195,
"endChar": 13
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -109,6 +181,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "af026eee_fa1a7f09",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 233,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Yes, the operator needs to set manually or via some kind of automation. The kube-ovn has manually used this for testing in the past.\nhttps://github.com/kubeovn/kube-ovn/blob/v1.11.11/docs/cluster-interconnection.md",
"parentUuid": "91e3347d_018dfbce",
"range": {
"startLine": 233,
"startChar": 30,
"endLine": 233,
"endChar": 69
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -132,6 +228,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "744299fe_c9bc2f69",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 259,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "e10a1511_5b9afc7b",
"range": {
"startLine": 259,
"startChar": 27,
"endLine": 259,
"endChar": 38
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -155,6 +275,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "48416082_00c466c7",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 266,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "thanks",
"parentUuid": "135a3c9d_fc968073",
"range": {
"startLine": 266,
"startChar": 30,
"endLine": 266,
"endChar": 36
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -178,6 +322,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "49dad793_7f991ffe",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 268,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "abde2cca_f0f28e29",
"range": {
"startLine": 268,
"startChar": 54,
"endLine": 268,
"endChar": 63
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -201,6 +369,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "cf4fe694_b42debce",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 274,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "yeah, it makes sense and souds better. Thanks",
"parentUuid": "829d7fe6_69e34511",
"range": {
"startLine": 274,
"startChar": 20,
"endLine": 274,
"endChar": 38
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -224,6 +416,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "06177f15_36764d95",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 300,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "21b3ec17_7d9b110d",
"range": {
"startLine": 300,
"startChar": 20,
"endLine": 300,
"endChar": 38
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -247,6 +463,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "05293491_cedb4993",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 320,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "1e91e679_a39743ae",
"range": {
"startLine": 320,
"startChar": 18,
"endLine": 320,
"endChar": 25
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -270,6 +510,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "7c02032b_a4a974a0",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 324,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "yeah, much better this way, thanks.",
"parentUuid": "7d261a3f_cdaf3f92",
"range": {
"startLine": 321,
"startChar": 42,
"endLine": 324,
"endChar": 5
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -293,6 +557,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "18038a96_d6817aac",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 329,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "1d4c0362_d1958de7",
"range": {
"startLine": 329,
"startChar": 20,
"endLine": 329,
"endChar": 38
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -316,6 +604,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "404500ec_c44bc362",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 331,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "6abb351b_04930f43",
"range": {
"startLine": 331,
"startChar": 35,
"endLine": 331,
"endChar": 47
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -339,6 +651,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "7ae7f27a_0f62b1ec",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 395,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "32e0ddab_e3f588ee",
"range": {
"startLine": 395,
"startChar": 5,
"endLine": 395,
"endChar": 12
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -362,6 +698,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "12935d65_119a7c60",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 398,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "b2ddac39_b10b3cc3",
"range": {
"startLine": 397,
"startChar": 22,
"endLine": 398,
"endChar": 69
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -385,6 +745,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "30c26de9_e80eac30",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 406,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "5bd6d5fa_11063991",
"range": {
"startLine": 406,
"startChar": 20,
"endLine": 406,
"endChar": 39
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -431,6 +815,54 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "16e40f1f_96582e68",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 410,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "d8d8a765_9c2f8adf",
"range": {
"startLine": 410,
"startChar": 53,
"endLine": 410,
"endChar": 56
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "b007156d_b26a9b4f",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 410,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "81b84a1b_b74ca0d6",
"range": {
"startLine": 410,
"startChar": 35,
"endLine": 410,
"endChar": 43
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -477,6 +909,54 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "72a0b126_3e4765ce",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 446,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "417ebd8e_3f9759e8",
"range": {
"startLine": 446,
"startChar": 53,
"endLine": 446,
"endChar": 56
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "aa53f024_797f6c9d",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 446,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "371a7e8e_cf9589c3",
"range": {
"startLine": 446,
"startChar": 35,
"endLine": 446,
"endChar": 43
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -500,6 +980,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "553c8eb0_ed4a030b",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 474,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "3dd4d24f_f2f33e76",
"range": {
"startLine": 474,
"startChar": 63,
"endLine": 474,
"endChar": 71
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -523,6 +1027,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "67266ac8_f9538cdd",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 476,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "c3936668_b3e36b0e",
"range": {
"startLine": 476,
"startChar": 4,
"endLine": 476,
"endChar": 7
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -546,6 +1074,30 @@
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "854277d3_52b06f72",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 478,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Done",
"parentUuid": "cdf5c0b4_5fb173c9",
"range": {
"startLine": 477,
"startChar": 0,
"endLine": 478,
"endChar": 29
},
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -562,6 +1114,24 @@
"message": "How about some documentation. I think a chapter explaining how this works would be very helpful here: https://github.com/openstack/neutron/tree/master/doc/source/admin",
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "deb3a05f_bf4873e3",
"filename": "specs/2024.1/ml2ovn-coexistence-support-ovn-interconnect.rst",
"patchSetId": 3
},
"lineNbr": 520,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "I also believe it would be super helpful. Are you talking about some kind of how-to document to exemplify the deployment configuration? I can check references on how to do this there.",
"parentUuid": "cec3048b_45b33629",
"revId": "0e8d530ba16d94428a6a9581107003516ae76648",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}

21
72e6a1d535864eaed953c58f5d68f9d0d6c815f9 Normal file
View File

@ -0,0 +1,21 @@
{
"comments": [
{
"unresolved": false,
"key": {
"uuid": "0823b51b_bc92d032",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 4
},
"lineNbr": 0,
"author": {
"id": 35432
},
"writtenOn": "2023-10-20T13:10:35Z",
"side": 1,
"message": "Thanks for your feedback Miguel 😊",
"revId": "72e6a1d535864eaed953c58f5d68f9d0d6c815f9",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}