diff mbox series

[ovs-dev] ovn: Remove SBREC_TABLE_HA_CHASSIS ctl_table

Message ID 20221024100847.763567-1-dceara@redhat.com
State Accepted
Headers show
Series [ovs-dev] ovn: Remove SBREC_TABLE_HA_CHASSIS ctl_table | expand

Checks

Context Check Description
ovsrobot/apply-robot warning apply and check: warning
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/github-robot-_ovn-kubernetes success github build: passed

Commit Message

Dumitru Ceara Oct. 24, 2022, 10:08 a.m. UTC 
From: "jun.gu" <jun.gu@easystack.cn>

Fix bug that ovn-sbctl daemon process exit when list non-existed
ha_chassis.

Submitted-at: https://github.com/ovn-org/ovn/pull/151
Signed-off-by: Jun Gu <jun.gu@easystack.cn>
---
 utilities/ovn-sbctl.c | 3 ---
 1 file changed, 3 deletions(-)

Comments

0-day Robot Oct. 24, 2022, 10:19 a.m. UTC | #1 
Bleep bloop.  Greetings Dumitru Ceara, I am a robot and I have tried out your patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


checkpatch:
ERROR: Author jun.gu <jun.gu@easystack.cn> needs to sign off.
WARNING: Unexpected sign-offs from developers who are not authors or co-authors or committers: Jun Gu <jun.gu@easystack.cn>
Lines checked: 32, Warnings: 1, Errors: 1


Please check this out.  If you feel there has been an error, please email aconole@redhat.com

Thanks,
0-day Robot
Dumitru Ceara Oct. 24, 2022, 10:31 a.m. UTC | #2 
On 10/24/22 12:08, Dumitru Ceara wrote:
> From: "jun.gu" <jun.gu@easystack.cn>
> 
> Fix bug that ovn-sbctl daemon process exit when list non-existed
> ha_chassis.
> 
> Submitted-at: https://github.com/ovn-org/ovn/pull/151
> Signed-off-by: Jun Gu <jun.gu@easystack.cn>
> ---

I pushed this to the main branch, thanks for the contribution!

Regards,
Dumitru
diff mbox series

Patch

diff --git a/utilities/ovn-sbctl.c b/utilities/ovn-sbctl.c
index e35556d34a..f60dde1b67 100644
--- a/utilities/ovn-sbctl.c
+++ b/utilities/ovn-sbctl.c
@@ -1431,9 +1431,6 @@  static const struct ctl_table_class tables[SBREC_N_TABLES] = {
     [SBREC_TABLE_HA_CHASSIS_GROUP].row_ids[0]
     = {&sbrec_ha_chassis_group_col_name, NULL, NULL},
 
-    [SBREC_TABLE_HA_CHASSIS].row_ids[0]
-    = {&sbrec_ha_chassis_col_chassis, NULL, NULL},
-
     [SBREC_TABLE_METER].row_ids[0]
     = {&sbrec_meter_col_name, NULL, NULL},