diff --git a/9821652bc8174cc8a037b160421af23992a7dc7e b/9821652bc8174cc8a037b160421af23992a7dc7e index 72697365..4ea8fece 100644 --- a/9821652bc8174cc8a037b160421af23992a7dc7e +++ b/9821652bc8174cc8a037b160421af23992a7dc7e @@ -1,5 +1,22 @@ { "comments": [ + { + "unresolved": false, + "key": { + "uuid": "e09da03c_17ed6632", + "filename": "/PATCHSET_LEVEL", + "patchSetId": 3 + }, + "lineNbr": 0, + "author": { + "id": 28464 + }, + "writtenOn": "2024-04-20T13:39:43Z", + "side": 1, + "message": "I know that Fabiano delivered the update https://review.opendev.org/c/starlingx/metal/+/914696/4/kickstart/files/kickstart.cfg that is now being discarded for an IP address only based solution. His original solution considered very seriously this precise change but upgrades was a concern.\n\nSeems from the commit message that dual stack handling has exposed another challenge in this area causing us to flop back to the ip address only based solution.\n\nYou need to verify that this solution will work for the upgrade case, at least to the point of the upgrade install working, even if end-to-end upgrade may not be.\n\nI will +2 knowing that the dual-stack feature will eventually need to be upgrades tested.", + "revId": "9821652bc8174cc8a037b160421af23992a7dc7e", + "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543" + }, { "unresolved": false, "key": {