[ovs-dev] [ovs-dev, ovn, 1/2] ovn-nbctl.c: Add "name" column of QoS table.

taoyunxiang at cmss.chinamobile.com taoyunxiang at cmss.chinamobile.com
Mon Dec 23 01:38:08 UTC 2019


Hi,Numan,
                I have submit v2 patchs. Please check it. 


Thanks,
Yun





taoyunxiang at cmss.chinamobile.com
From: Numan Siddique
Date: 2019-12-20 21:53
To: taoyunxiang at cmss.chinamobile.com
CC: ovs-dev
Subject: Re: [ovs-dev] [ovs-dev, ovn, 1/2] ovn-nbctl.c: Add "name" column of QoS table.
On Fri, Dec 20, 2019 at 1:39 PM taoyunxiang at cmss.chinamobile.com
<taoyunxiang at cmss.chinamobile.com> wrote:
>
> Hi, Could you review this patch。It is an important QoS feature for plugin of OVN.
 
Hi Taoyunxiang,
 
I am sorry for the delay. Looks like I missed these patches.
 
The patch doesn't apply to the present master. Can you please rebase
the series and submit v2 ?
 
Thanks
Numan
 
>
> Thanks,
> Yun
>
>
>
>
> From: taoyunxiang at cmss.chinamobile.com
> Date: 2019-11-26 15:57
> To: ovs-dev
> Subject: [ovs-dev,ovn,1/2] ovn-nbctl.c: Add "name" column of QoS table.
> Add "name" column of QoS table
>
> Add "name" column of QoS table and make qos could be list
> by command "ovn-nbctl list qos "name"".
>
> Signed-off-by: Yunxiang Tao <taoyunxiang at cmss.chinamobile.com>
>
> diff --git a/ovn-nb.ovsschema b/ovn-nb.ovsschema
> index 12999a466..02ae48e73 100644
> --- a/ovn-nb.ovsschema
> +++ b/ovn-nb.ovsschema
> @@ -1,7 +1,7 @@
>  {
>      "name": "OVN_Northbound",
>      "version": "5.18.0",
> -    "cksum": "2806349485 24196",
> +    "cksum": "80177565 24240",
>      "tables": {
>          "NB_Global": {
>              "columns": {
> @@ -204,6 +204,7 @@
>              "isRoot": false},
>          "QoS": {
>              "columns": {
> +                "name": {"type": "string"},
>                  "priority": {"type": {"key": {"type": "integer",
>                                                "minInteger": 0,
>                                                "maxInteger": 32767}}},
> diff --git a/ovn-nb.xml b/ovn-nb.xml
> index 4a93d2f4a..61e77397e 100644
> --- a/ovn-nb.xml
> +++ b/ovn-nb.xml
> @@ -1662,6 +1662,12 @@
>        </p>
>      </column>
>
> +    <column name="name">
> +      <p>
> +        A name for the logical router.
> +      </p>
> +    </column>
> +
>      <column name="direction">
>        <p>
>          The value of this field is similar to <ref colun="direction"
> diff --git a/utilities/ovn-nbctl.c b/utilities/ovn-nbctl.c
> index 8188948fa..93e37d169 100644
> --- a/utilities/ovn-nbctl.c
> +++ b/utilities/ovn-nbctl.c
> @@ -5346,6 +5346,8 @@ static const struct ctl_table_class tables[NBREC_N_TABLES] = {
>
>      [NBREC_TABLE_HA_CHASSIS_GROUP].row_ids[0]
>      = {&nbrec_ha_chassis_group_col_name, NULL, NULL},
> +
> +    [NBREC_TABLE_QOS].row_ids[0] = {&nbrec_qos_col_name, NULL, NULL},
>  };
>
>  static char *
>
>
>
> taoyunxiang at cmss.chinamobile.com
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev


More information about the dev mailing list