Update patch set 2
Patch Set 2: (3 comments) Patch-set: 2 Label: Verified=0
This commit is contained in:
parent
0f312dafb0
commit
7d83cde3b9
|
@ -0,0 +1,21 @@
|
|||
{
|
||||
"comments": [
|
||||
{
|
||||
"key": {
|
||||
"uuid": "3fa7e38b_ba90e74f",
|
||||
"filename": "attributes/default.rb",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 30,
|
||||
"author": {
|
||||
"id": 13252
|
||||
},
|
||||
"writtenOn": "2019-11-18T15:12:13Z",
|
||||
"side": 1,
|
||||
"message": "Wouldn\u0027t you drop \"telemetry\" from this loop completely, if that is defining the API endpoint?",
|
||||
"revId": "0c072b88b959519504a59cfa672902d35fa4af7f",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
}
|
||||
]
|
||||
}
|
|
@ -0,0 +1,38 @@
|
|||
{
|
||||
"comments": [
|
||||
{
|
||||
"key": {
|
||||
"uuid": "3fa7e38b_da93634d",
|
||||
"filename": "attributes/ceilometer_conf.rb",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 5,
|
||||
"author": {
|
||||
"id": 13252
|
||||
},
|
||||
"writtenOn": "2019-11-18T15:12:13Z",
|
||||
"side": 0,
|
||||
"message": "How is this related to dropping telemetry-api?",
|
||||
"revId": "fd3d9ceb621e5f82a8278cce23e623393a98c160",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "3fa7e38b_7a6fcf5f",
|
||||
"filename": "spec/gnocchi_install_spec.rb",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 14,
|
||||
"author": {
|
||||
"id": 13252
|
||||
},
|
||||
"writtenOn": "2019-11-18T15:12:13Z",
|
||||
"side": 0,
|
||||
"message": "Hmm, gnocchi-api seems different from telemetry-api, why is this being dropped?",
|
||||
"revId": "fd3d9ceb621e5f82a8278cce23e623393a98c160",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
}
|
||||
]
|
||||
}
|
Loading…
Reference in New Issue