Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

a bug in the method ChassisExist #3049

Closed
DWC666 opened this issue Jul 18, 2023 · 2 comments · Fixed by #3072
Closed

a bug in the method ChassisExist #3049

DWC666 opened this issue Jul 18, 2023 · 2 comments · Fixed by #3072
Labels
bug Something isn't working

Comments

@DWC666
Copy link

DWC666 commented Jul 18, 2023

Expected Behavior

when the chassis not exist, the method ChassisExist return false

Actual Behavior

the method ChassisExist return true

Steps to Reproduce the Problem

if the output is a empty str, the method will return true, because len(strings.Split(output, "\n")) is 1, not 0.
image

Additional Info

  • Kubernetes version:

    Output of kubectl version:

    (paste your output here)
    
  • kube-ovn version:

v1.11.3


- operation-system/kernel version:

**Output of `awk -F '=' '/PRETTY_NAME/ { print $2 }' /etc/os-release`:**
**Output of `uname -r`:**

(paste your output here)


<!-- Any other additional information -->
@zhangzujian
Copy link
Member

Could you please commit a PR to fix it?

@zhangzujian zhangzujian added the bug Something isn't working label Jul 18, 2023
@DWC666
Copy link
Author

DWC666 commented Jul 18, 2023

@zhangzujian
okay

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants