Update patch set 5

Patch Set 5: Code-Review+2

(1 comment)

Patch-set: 5
Reviewer: Gerrit User 16137 <16137@4a232e18-c5a9-48ee-94c0-e04e7cca6543>
Label: Code-Review=+2
Attention: {"person_ident":"Gerrit User 13252 \u003c13252@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"Tobias Urdin replied on the change"}
This commit is contained in:
Gerrit User 16137 2021-11-03 20:33:30 +00:00 committed by Gerrit Code Review
parent 2a1a4e2242
commit 54799814cc
1 changed files with 18 additions and 0 deletions

View File

@ -16,6 +16,24 @@
"message": "After having built the option to test via n-t-p option I have not switched the default for now, but I still think that that would be the proper path forward. So I would like to ask reviewers on their opinion about this: Do we need to wait a cycle before we switch the default or should we do it at once? Or maybe not at all?\n\nIf we intend to switch later, should we at least announce that plan in the release note?",
"revId": "1a40f3706d3b0b01e27904b1e1423bd209cd7771",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "526fd4c3_5835b7f6",
"filename": "neutron_dynamic_routing/db/bgp_dragentscheduler_db.py",
"patchSetId": 5
},
"lineNbr": 42,
"author": {
"id": 16137
},
"writtenOn": "2021-11-03T20:33:30Z",
"side": 1,
"message": "I think we shoud notify with a deprecation/upgrade in release note, then change next release, that would prepare anybody for the default change one release prior. Then also a new release for the default change.",
"parentUuid": "c470304b_ea30847c",
"revId": "1a40f3706d3b0b01e27904b1e1423bd209cd7771",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}