Update patch set 7
Patch Set 7: Workflow-1 (3 comments) Patch-set: 7 Reviewer: Gerrit User 16688 <16688@4a232e18-c5a9-48ee-94c0-e04e7cca6543> Label: Workflow=-1 Attention: {"person_ident":"Gerrit User 11604 \u003c11604@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"\u003cGERRIT_ACCOUNT_16688\u003e replied on the change"} Attention: {"person_ident":"Gerrit User 16688 \u003c16688@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"REMOVE","reason":"\u003cGERRIT_ACCOUNT_16688\u003e replied on the change"}
This commit is contained in:
parent
f1f67b427a
commit
9832cbc4f6
|
@ -325,6 +325,24 @@
|
|||
"revId": "09ee72205f522b16261a4ae919e03511386c0e9d",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
},
|
||||
{
|
||||
"unresolved": false,
|
||||
"key": {
|
||||
"uuid": "15a04950_bbc0d111",
|
||||
"filename": "specs/2023.1/strict-minimum-bandwidth-tunnelled-networks.rst",
|
||||
"patchSetId": 6
|
||||
},
|
||||
"lineNbr": 165,
|
||||
"author": {
|
||||
"id": 16688
|
||||
},
|
||||
"writtenOn": "2022-11-07T17:41:26Z",
|
||||
"side": 1,
|
||||
"message": "Ok, I\u0027m not going to fix what is not broken. What I\u0027ll propose is a tool that will inform about the ports that have QoS min-bw rules in overlay networks. If those ports are on hosts with an overlay network with a new RP, then we\u0027ll inform about this. Unless we have a tool to sync/heal an existing deployment, any port created before this new feature won\u0027t have an allocation.",
|
||||
"parentUuid": "151d6e60_26040d44",
|
||||
"revId": "09ee72205f522b16261a4ae919e03511386c0e9d",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
},
|
||||
{
|
||||
"unresolved": true,
|
||||
"key": {
|
||||
|
@ -377,6 +395,24 @@
|
|||
"parentUuid": "11840e74_ce293de3",
|
||||
"revId": "09ee72205f522b16261a4ae919e03511386c0e9d",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
},
|
||||
{
|
||||
"unresolved": false,
|
||||
"key": {
|
||||
"uuid": "63509a3e_889e8842",
|
||||
"filename": "specs/2023.1/strict-minimum-bandwidth-tunnelled-networks.rst",
|
||||
"patchSetId": 6
|
||||
},
|
||||
"lineNbr": 180,
|
||||
"author": {
|
||||
"id": 16688
|
||||
},
|
||||
"writtenOn": "2022-11-07T17:41:26Z",
|
||||
"side": 1,
|
||||
"message": "\"we have a long standing know issue for how neutorn is currently mangaing the inventoies.\" What is this issue? From Neutron we currently can update the inventory of a RP when the port QoS change. If that is no allowed, please open a bug for Neutron and we\u0027ll discuss about how to change this.",
|
||||
"parentUuid": "81fbd140_206b81eb",
|
||||
"revId": "09ee72205f522b16261a4ae919e03511386c0e9d",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
}
|
||||
]
|
||||
}
|
|
@ -0,0 +1,21 @@
|
|||
{
|
||||
"comments": [
|
||||
{
|
||||
"unresolved": false,
|
||||
"key": {
|
||||
"uuid": "e77b8e68_e0d08c6a",
|
||||
"filename": "/PATCHSET_LEVEL",
|
||||
"patchSetId": 7
|
||||
},
|
||||
"lineNbr": 0,
|
||||
"author": {
|
||||
"id": 16688
|
||||
},
|
||||
"writtenOn": "2022-11-07T17:41:26Z",
|
||||
"side": 1,
|
||||
"message": "I\u0027ll push a new PS tomorrow. I\u0027ll remove the section describing the script to heal the inventories. We\u0027ll reduce the scope to a simple script that will inform about the ports with min-bw rules in overlay networks.",
|
||||
"revId": "7fdc74b912aa683acb27532c45a09c93dcf38a4a",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
}
|
||||
]
|
||||
}
|
Loading…
Reference in New Issue