Update patch set 8
Patch Set 8:
(4 comments)
Patch-set: 8
Attention: {"person_ident":"Gerrit User 8313 \u003c8313@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"\u003cGERRIT_ACCOUNT_16688\u003e replied on the change"}
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:
Gerrit User 16688
Gerrit Code Review
parent
19d278c01a
commit
9c85d625de
|
|
@ -57,6 +57,30 @@
|
|||
"revId": "1b2506fea06d524218886ae5079121155a9e51b2",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
},
|
||||
{
|
||||
"unresolved": false,
|
||||
"key": {
|
||||
"uuid": "c980ab3a_a26770db",
|
||||
"filename": "specs/2023.1/strict-minimum-bandwidth-tunnelled-networks.rst",
|
||||
"patchSetId": 8
|
||||
},
|
||||
"lineNbr": 124,
|
||||
"author": {
|
||||
"id": 16688
|
||||
},
|
||||
"writtenOn": "2022-11-15T08:36:45Z",
|
||||
"side": 1,
|
||||
"message": "Right.",
|
||||
"parentUuid": "39b5e4ea_d448821b",
|
||||
"range": {
|
||||
"startLine": 124,
|
||||
"startChar": 4,
|
||||
"endLine": 124,
|
||||
"endChar": 27
|
||||
},
|
||||
"revId": "1b2506fea06d524218886ae5079121155a9e51b2",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
},
|
||||
{
|
||||
"unresolved": true,
|
||||
"key": {
|
||||
|
|
@ -74,6 +98,24 @@
|
|||
"revId": "1b2506fea06d524218886ae5079121155a9e51b2",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
},
|
||||
{
|
||||
"unresolved": false,
|
||||
"key": {
|
||||
"uuid": "9c12a5e0_3b1f9478",
|
||||
"filename": "specs/2023.1/strict-minimum-bandwidth-tunnelled-networks.rst",
|
||||
"patchSetId": 8
|
||||
},
|
||||
"lineNbr": 187,
|
||||
"author": {
|
||||
"id": 16688
|
||||
},
|
||||
"writtenOn": "2022-11-15T08:36:45Z",
|
||||
"side": 1,
|
||||
"message": "This is something to be addressed, for sure, in future RFEs.",
|
||||
"parentUuid": "83565dec_2f37dbb8",
|
||||
"revId": "1b2506fea06d524218886ae5079121155a9e51b2",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
},
|
||||
{
|
||||
"unresolved": true,
|
||||
"key": {
|
||||
|
|
@ -90,6 +132,24 @@
|
|||
"message": "one thing we discussed was wether nova\u0027s nova-manage heal-allocations command might be able to fix things in place. if not that is perhaps an improment we coudl make in the future.\n\nnova would fix the allcoation on a cold/live migrate so that is an option albit a somewhat expensive one to adress this. this supprot was added in ussuri by \nhttps://specs.openstack.org/openstack/nova-specs/specs/ussuri/implemented/support-move-ops-with-qos-ports-ussuri.html\n\na less invaisve change would be to detach and attach the ports with qos policies\nwhich was added in wallaby.\n\nhttps://specs.openstack.org/openstack/nova-specs/specs/wallaby/implemented/support-interface-attach-with-qos-ports.html\n\nthe other option is to just remove teh min bandwith qos policy form the neutron ports before upgrading.\n\ni think documenting these options is likely the minium we need to do to enabel operators to determin the best parth forward for them in the A cycle.",
|
||||
"revId": "1b2506fea06d524218886ae5079121155a9e51b2",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
},
|
||||
{
|
||||
"unresolved": false,
|
||||
"key": {
|
||||
"uuid": "e19b21e9_f16c1c03",
|
||||
"filename": "specs/2023.1/strict-minimum-bandwidth-tunnelled-networks.rst",
|
||||
"patchSetId": 8
|
||||
},
|
||||
"lineNbr": 197,
|
||||
"author": {
|
||||
"id": 16688
|
||||
},
|
||||
"writtenOn": "2022-11-15T08:36:45Z",
|
||||
"side": 1,
|
||||
"message": "I\u0027ll refer to it in this spec and then I\u0027ll include these comments in the documentation.",
|
||||
"parentUuid": "c05933b6_ad6977ec",
|
||||
"revId": "1b2506fea06d524218886ae5079121155a9e51b2",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
}
|
||||
]
|
||||
}
|
||||
|
|
@ -366,6 +366,30 @@
|
|||
"revId": "7fdc74b912aa683acb27532c45a09c93dcf38a4a",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
},
|
||||
{
|
||||
"unresolved": false,
|
||||
"key": {
|
||||
"uuid": "a2fbae7a_27fe879d",
|
||||
"filename": "specs/2023.1/strict-minimum-bandwidth-tunnelled-networks.rst",
|
||||
"patchSetId": 7
|
||||
},
|
||||
"lineNbr": 147,
|
||||
"author": {
|
||||
"id": 16688
|
||||
},
|
||||
"writtenOn": "2022-11-15T08:36:45Z",
|
||||
"side": 1,
|
||||
"message": "We agreed on keeping the 1:1 match between the network interface (physical, tunnelled) and the RP. Of course that will limit the possibility of sharing the same resource (or at least the way to model it in the Neutron placement implementation), but \"most\" of the operators usually use one network type (vlan, vxlan, geneve, etc).",
|
||||
"parentUuid": "3d08c5c4_18e84cc7",
|
||||
"range": {
|
||||
"startLine": 146,
|
||||
"startChar": 31,
|
||||
"endLine": 147,
|
||||
"endChar": 44
|
||||
},
|
||||
"revId": "7fdc74b912aa683acb27532c45a09c93dcf38a4a",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
},
|
||||
{
|
||||
"unresolved": true,
|
||||
"key": {
|
||||
|
|
|
|||
Loading…
Reference in New Issue