It looks like a mistake here

[SunHui2980]

the code between line 107 and line 115 means to delete the ch which is passed into this function from the mgmtConnManager's watchers slice, right?

[onf-bot]

Can one of the admins verify this patch?

[onf-cla-manager]

Hi @SunHui2980, this is the ONF bot 🤖 I'm glad you want to contribute to our projects! However, before accepting your contribution, we need to ask you to sign a Contributor License Agreement (CLA). You can do it online, it will take only a few minutes:

✒️ 👉 https://cla.opennetworking.org

After signing, make sure to add your Github user ID SunHui2980 to the agreement.

For more information or help:"
https://wiki.opennetworking.org/x/BgCUI

[onf-cla-manager]

Hi @SunHui2980, this is the ONF bot 🤖 I'm glad you want to contribute to our projects! However, before accepting your contribution, we need to ask you to sign a Contributor License Agreement (CLA). You can do it online, it will take only a few minutes:

✒️ 👉 https://cla.opennetworking.org

After signing, make sure to add your Github user ID SunHui2980 to the agreement.

For more information or help:"
https://wiki.opennetworking.org/x/BgCUI

[gab-arrobo]

@SunHui2980, are you planning to sign the ONF CLA?