Update patch set 1
Patch Set 1: Code-Review-1 (2 comments) Patch-set: 1 Reviewer: Gerrit User 33455 <33455@4a232e18-c5a9-48ee-94c0-e04e7cca6543> Label: Code-Review=-1 Attention: {"person_ident":"Gerrit User 33455 \u003c33455@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"REMOVE","reason":"Hiromu Asahina replied on the change"} Attention: {"person_ident":"Gerrit User 33492 \u003c33492@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"Hiromu Asahina replied on the change"}
This commit is contained in:
parent
7a8a004bc5
commit
100c87c924
|
@ -0,0 +1,38 @@
|
|||
{
|
||||
"comments": [
|
||||
{
|
||||
"unresolved": true,
|
||||
"key": {
|
||||
"uuid": "7ea3d86d_d09b6534",
|
||||
"filename": "/COMMIT_MSG",
|
||||
"patchSetId": 1
|
||||
},
|
||||
"lineNbr": 10,
|
||||
"author": {
|
||||
"id": 33455
|
||||
},
|
||||
"writtenOn": "2022-04-14T09:33:46Z",
|
||||
"side": 1,
|
||||
"message": "nits, but `how to specify the target namespace in InstantiateVnfRequst` might be misleading as the present document also mentions the priority of namespaces used by tacker, which is a main part of this enhancement. If you agree, please mention that.\n\nAlso, although other docs patches don\u0027t refer a BP, I\u0027d like to suggest to refer the related BP: https://blueprints.launchpad.net/tacker/+spec/k8s-namespace.",
|
||||
"revId": "11aee2396190764e69d0dcd2f72637314640ca7c",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
},
|
||||
{
|
||||
"unresolved": false,
|
||||
"key": {
|
||||
"uuid": "2e5513b8_b9da8828",
|
||||
"filename": "/PATCHSET_LEVEL",
|
||||
"patchSetId": 1
|
||||
},
|
||||
"lineNbr": 0,
|
||||
"author": {
|
||||
"id": 33455
|
||||
},
|
||||
"writtenOn": "2022-04-14T09:33:46Z",
|
||||
"side": 1,
|
||||
"message": "Thank you for the patch.\n\nPlease find my comment.",
|
||||
"revId": "11aee2396190764e69d0dcd2f72637314640ca7c",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
}
|
||||
]
|
||||
}
|
Loading…
Reference in New Issue