From 2cb069498c93ec4097ec85c0f2f14fad26c0311e Mon Sep 17 00:00:00 2001
From: Gerrit User 11819 <11819@4a232e18-c5a9-48ee-94c0-e04e7cca6543>
Date: Thu, 16 Mar 2017 07:44:27 +0000
Subject: [PATCH] Update patch set 1

Patch Set 1: Code-Review-1

(4 comments)

Patch-set: 1
Label: Code-Review=-1
---
 03d3b5cff88417d303e75b7d86fc24653102ba26 | 92 ++++++++++++++++++++++++
 1 file changed, 92 insertions(+)

diff --git a/03d3b5cff88417d303e75b7d86fc24653102ba26 b/03d3b5cff88417d303e75b7d86fc24653102ba26
index 32c99ce2..b40baec1 100644
--- a/03d3b5cff88417d303e75b7d86fc24653102ba26
+++ b/03d3b5cff88417d303e75b7d86fc24653102ba26
@@ -1,5 +1,97 @@
 {
   "comments": [
+    {
+      "key": {
+        "uuid": "3a1ff146_e76d6a05",
+        "filename": "tricircle/network/qos.py",
+        "patchSetId": 1
+      },
+      "lineNbr": 24,
+      "author": {
+        "id": 11819
+      },
+      "writtenOn": "2017-03-16T07:44:27Z",
+      "side": 1,
+      "message": "from tricircle.db import api as db_api",
+      "range": {
+        "startLine": 24,
+        "startChar": 0,
+        "endLine": 24,
+        "endChar": 33
+      },
+      "revId": "03d3b5cff88417d303e75b7d86fc24653102ba26",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
+      "unresolved": false
+    },
+    {
+      "key": {
+        "uuid": "3a1ff146_12dc82e3",
+        "filename": "tricircle/network/qos.py",
+        "patchSetId": 1
+      },
+      "lineNbr": 27,
+      "author": {
+        "id": 11819
+      },
+      "writtenOn": "2017-03-16T07:44:27Z",
+      "side": 1,
+      "message": "inherited from QoSPlugin will call driver and rpc, it\u0027s overhead, could be removed. According to the devref from Neutron: https://github.com/openstack/neutron/blob/02f893a56d5df6319cb880db497e3c0e8c3dbe65/doc/source/devref/quality_of_service.rst it\u0027s recommend to inherit from QoSPluginBase, and borrow most of code from QoSPlugin, or inherit from QoSPlugin, but need to provide simplification for RPC and driver_manager calling",
+      "range": {
+        "startLine": 27,
+        "startChar": 36,
+        "endLine": 27,
+        "endChar": 45
+      },
+      "revId": "03d3b5cff88417d303e75b7d86fc24653102ba26",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
+      "unresolved": false
+    },
+    {
+      "key": {
+        "uuid": "3a1ff146_c71b0e7f",
+        "filename": "tricircle/network/qos.py",
+        "patchSetId": 1
+      },
+      "lineNbr": 40,
+      "author": {
+        "id": 11819
+      },
+      "writtenOn": "2017-03-16T07:44:27Z",
+      "side": 1,
+      "message": "move it below update policy in central plugin",
+      "range": {
+        "startLine": 39,
+        "startChar": 7,
+        "endLine": 40,
+        "endChar": 51
+      },
+      "revId": "03d3b5cff88417d303e75b7d86fc24653102ba26",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
+      "unresolved": false
+    },
+    {
+      "key": {
+        "uuid": "3a1ff146_d244aa75",
+        "filename": "tricircle/network/qos.py",
+        "patchSetId": 1
+      },
+      "lineNbr": 63,
+      "author": {
+        "id": 11819
+      },
+      "writtenOn": "2017-03-16T07:44:27Z",
+      "side": 1,
+      "message": "no need, please refer to QoSPlugin delete_policy",
+      "range": {
+        "startLine": 56,
+        "startChar": 5,
+        "endLine": 63,
+        "endChar": 68
+      },
+      "revId": "03d3b5cff88417d303e75b7d86fc24653102ba26",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
+      "unresolved": false
+    },
     {
       "key": {
         "uuid": "5a22e58a_eb3f45d4",