-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Update ce_vrrp to fix bugs #59677
Update ce_vrrp to fix bugs #59677
Conversation
cc @QijunPan @TommyLike @edisonxiang @freesky-edward @hwDCN @niuzhenguo @numone213 @xuxiaowei0512 @zengchen1024 @zhongjun2 |
@yanzhangi, just so you are aware we have a dedicated Working Group for network. |
cc @QijunPan @TommyLike @edisonxiang @freesky-edward @hwDCN @niuzhenguo @numone213 @xuxiaowei0512 @zengchen1024 @zhongjun2 |
@@ -587,9 +587,9 @@ def check_params(self): | |||
if not self.advertise_interval.isdigit(): | |||
self.module.fail_json( | |||
msg='Error: The value of advertise_interval is an integer.') | |||
if int(self.advertise_interval) < 1 or int(self.advertise_interval) > 255000: | |||
if int(self.advertise_interval) < 1000 or int(self.advertise_interval) > 255000: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个地方的判断仅仅是一个范围可能还不够,它还要求是1000的倍数。
交换机上呈现的是秒,这里是毫秒。
+1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
bot_status |
Componentslib/ansible/modules/network/cloudengine/ce_vrrp.py Metadatawaiting_on: maintainer |
SUMMARY
Manages VRRP interface attributes on HUAWEI CloudEngine devices
When a parameter is of the bool type, for example: If the value is True or yes, an error is reported in the module
ISSUE TYPE
COMPONENT NAME
lib/ansible/modules/network/cloudengine/ce_vrrp.py
ADDITIONAL INFORMATION