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
Don't validate ip address for mgmt interface #56136
Conversation
This now adds support to run eos tests against ansible/ansible. Depends-On: ansible/ansible#56126 Depends-On: ansible/ansible#56131 Depends-On: ansible/ansible#56136 Signed-off-by: Paul Belanger <pabelanger@redhat.com>
@@ -23,7 +23,7 @@ | |||
# Items from those subsets are present | |||
- "result.ansible_facts.ansible_net_filesystems is defined" #hw | |||
- "result.ansible_facts.ansible_net_memtotal_mb > 10" #hw | |||
- "result.ansible_facts.ansible_net_interfaces.Management1.ipv4.masklen > 1" # interfaces | |||
- "result.ansible_facts.ansible_net_interfaces.Management1" # interfaces |
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.
Is there anything we can assert about this?
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.
We can, but we should update the testing here to create the ip address first.
This now adds support to run eos tests against ansible/ansible. Depends-On: ansible/ansible#56126 Depends-On: ansible/ansible#56131 Depends-On: ansible/ansible#56136 Depends-On: ansible/ansible#56140 Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Pull-request updated, HEAD is now 84ee713 |
This now adds support to run eos tests against ansible/ansible. Depends-On: ansible/ansible#56126 Depends-On: ansible/ansible#56131 Depends-On: ansible/ansible#56136 Depends-On: ansible/ansible#56140 Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This now adds support to run eos tests against ansible/ansible. Depends-On: ansible/ansible#56126 Depends-On: ansible/ansible#56131 Depends-On: ansible/ansible#56136 Depends-On: ansible/ansible#56140 Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This now adds support to run eos tests against ansible/ansible. Depends-On: ansible/ansible#56126 Depends-On: ansible/ansible#56131 Depends-On: ansible/ansible#56136 Depends-On: ansible/ansible#56140 Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This now adds support to run eos tests against ansible/ansible. Depends-On: ansible/ansible#56126 Depends-On: ansible/ansible#56131 Depends-On: ansible/ansible#56136 Depends-On: ansible/ansible#56140 Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This now adds support to run eos tests against ansible/ansible. Depends-On: ansible/ansible#56126 Depends-On: ansible/ansible#56131 Depends-On: ansible/ansible#56136 Depends-On: ansible/ansible#56140 Signed-off-by: Paul Belanger <pabelanger@redhat.com>
It is possible the EOS appliance doesn't have an IP address on the management1 interface, instead just check we have found that interface. Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Pull-request updated, HEAD is now f0a0d2a |
Depends-On: ansible/ansible#56180 Depends-On: ansible/ansible#56235 Depends-On: ansible/ansible#56136 Signed-off-by: Paul Belanger <pabelanger@redhat.com>
) It is possible the EOS appliance doesn't have an IP address on the management1 interface, instead just check we have found that interface. Signed-off-by: Paul Belanger <pabelanger@redhat.com> (cherry picked from commit f9589bd) Co-authored-by: Paul Belanger <pabelanger@redhat.com>
It is possible the EOS appliance doesn't have an IP address on the management1 interface, instead just check we have found that interface. Signed-off-by: Paul Belanger <pabelanger@redhat.com> (cherry picked from commit f9589bd) Co-authored-by: Paul Belanger <pabelanger@redhat.com>
) It is possible the EOS appliance doesn't have an IP address on the management1 interface, instead just check we have found that interface. Signed-off-by: Paul Belanger <pabelanger@redhat.com> (cherry picked from commit f9589bd) Co-authored-by: Paul Belanger <pabelanger@redhat.com>
) It is possible the EOS appliance doesn't have an IP address on the management1 interface, instead just check we have found that interface. Signed-off-by: Paul Belanger <pabelanger@redhat.com> (cherry picked from commit f9589bd) Co-authored-by: Paul Belanger <pabelanger@redhat.com>
SUMMARY
Dont' assume mgmt interface has an ip address
ISSUE TYPE
COMPONENT NAME
eos_facts