Skip to content

[202012] vxlan+bfd test automation#6807

Merged
prsunny merged 3 commits intosonic-net:202012from
rraghav-cisco:202012
Nov 15, 2022
Merged

[202012] vxlan+bfd test automation#6807
prsunny merged 3 commits intosonic-net:202012from
rraghav-cisco:202012

Conversation

@rraghav-cisco
Copy link
Contributor

Pulling the VxLAN+BFD code from master to 202012 branch.

@lgtm-com
Copy link

lgtm-com bot commented Nov 11, 2022

This pull request introduces 3 alerts and fixes 12 when merging 385d091 into c68b24d - view on LGTM.com

new alerts:

  • 2 for Except block handles 'BaseException'
  • 1 for Unused local variable

fixed alerts:

  • 5 for Unused local variable
  • 4 for Unused import
  • 1 for Constant in conditional expression or statement
  • 1 for Testing equality to None
  • 1 for Comparison of constants

@rraghav-cisco
Copy link
Contributor Author

@siqbal1986 , @ihorchekh pls review/comment/approve.

Thanks,
rraghav

@prsunny prsunny changed the title Copying vxlan+bfd test code from master to 202012 branch. [202012] vxlan+bfd test automation Nov 11, 2022
@lgtm-com
Copy link

lgtm-com bot commented Nov 11, 2022

This pull request introduces 3 alerts and fixes 12 when merging 37d2219 into f24a9fa - view on LGTM.com

new alerts:

  • 2 for Except block handles 'BaseException'
  • 1 for Unused local variable

fixed alerts:

  • 5 for Unused local variable
  • 4 for Unused import
  • 1 for Constant in conditional expression or statement
  • 1 for Testing equality to None
  • 1 for Comparison of constants

@prsunny
Copy link
Contributor

prsunny commented Nov 11, 2022

@rraghav-cisco, I see there are few other files missing from original PR #6212.
@siqbal1986 , could you please review and merge?

@rraghav-cisco
Copy link
Contributor Author

@rraghav-cisco, I see there are few other files missing from original PR #6212. @siqbal1986 , could you please review and merge?

@prsunny , yes, they are not needed for vxlan+bfd test script.

@lgtm-com
Copy link

lgtm-com bot commented Nov 15, 2022

This pull request introduces 3 alerts and fixes 12 when merging 5d21d8b into 3d8329a - view on LGTM.com

new alerts:

  • 2 for Except block handles 'BaseException'
  • 1 for Unused local variable

fixed alerts:

  • 5 for Unused local variable
  • 4 for Unused import
  • 1 for Constant in conditional expression or statement
  • 1 for Testing equality to None
  • 1 for Comparison of constants

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants