-
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
Add route module #60189
Add route module #60189
Conversation
cc @QijunPan @TommyLike @edisonxiang @freesky-edward @hwDCN @niuzhenguo @xuxiaowei0512 @zengchen1024 |
The test
|
The test
|
@QijunPan @TommyLike @edisonxiang @freesky-edward @huaweicloud @hwDCN @niuzhenguo @xuxiaowei0512 @zengchen1024 As a maintainer of a module in the same namespace this new module has been submitted to, your vote counts for shipits. Please review this module and add |
shipit |
The test
|
bot_status |
Componentslib/ansible/modules/cloud/huawei/hwc_vpc_route.py test/integration/targets/hwc_vpc_route/aliases test/integration/targets/hwc_vpc_route/tasks/main.yml Metadatawaiting_on: maintainer |
-label affects_2.9 |
80a51fb
to
1a106e3
Compare
2651abd
to
2eb031e
Compare
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.
Looks good to me. I only found a few language issues.
(Review based only on code, I didn't test anything.)
553efd5
to
baccab0
Compare
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.
LGTM
bot_status |
Componentslib/ansible/modules/cloud/huawei/hwc_vpc_route.py test/integration/targets/hwc_vpc_route/aliases test/integration/targets/hwc_vpc_route/tasks/main.yml Metadatawaiting_on: maintainer |
SUMMARY
Add a new modules hwc_vpc_route that can fetch huaweicloud Simple Message Notification.
COMPONENT NAME
lib/ansible/modules/cloud/huawei/hwc_vpc_route.py
ISSUE TYPE
New Module Pull Request
ANSIBLE VERSION
ansible 2.10
ADDITIONAL INFORMATION
I have tested the changes on the real environment.