[system_test]test deploy_node_multiple_interfaces failed with AssertionError: Condition was False
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Fuel for OpenStack |
Fix Released
|
High
|
Andrey Sledzinskiy |
Bug Description
Scenario:
Deploy cluster with networks allocated on different interfaces
Scenario:
1. Create cluster in Ha mode with 1 controller
2. Add 1 node with controller role
3. Add 1 node with compute role
4. Add 1 node with cinder role
5. Split networks on existing physical interfaces
6. Deploy the cluster
7. Verify network configuration on each deployed node
8. Run network verification
https:/
FAIL: Deploy cluster with networks allocated on different interfaces
-------
Traceback (most recent call last):
File "/home/
compatabili
File "/home/
func()
File "/home/
func(
File "/home/
result = func(*args, **kwargs)
File "/home/
self.
File "/home/
checkers.
File "/home/
interface_
AssertionError: Condition was False.
summary: |
- [system_test]deploy_node_multiple_interfaces failed with + [system_test]test deploy_node_multiple_interfaces failed with + AssertionError: Condition was False |
tags: | added: area-qa |
Changed in fuel: | |
assignee: | Fuel QA Team (fuel-qa) → Andrey Sledzinskiy (asledzinskiy) |
tags: | removed: non-release |
Previously this test passed because check 'verify_ network_ configuration' was skipped (all networks where untagged) here:
https:/ /github. com/openstack/ fuel-qa/ blob/df45a35b8b d0969d850f566c0 62dcec4c62d8978 /fuelweb_ test/helpers/ checkers. py#L97- L98
Currently, by default we assign networks to different interfaces too, but VLAN tags remain for mgmt and storage networks. Looks like such behavior was introduced by https:/ /github. com/openstack/ fuel-qa/ commit/ 7ce54c7d16d8bf1 297ac08accf00d0 8823b8130f or recent changes to Nailgun. So 'verify_ network_ configuration' is not skipped now and it tries to check that IPs are assigned to interfaces correctly, but it does that in a wrong way.
Anyway, 'verify_ network_ configuration' method is deprecated a long time ago and should be refactored or removed (I prefer the second option, because we already have an advanced check for networks in tests for networking templates: https:/ /github. com/openstack/ fuel-qa/ blob/df45a35b8b d0969d850f566c0 62dcec4c62d8978 /fuelweb_ test/tests/ test_net_ templates_ base.py# L165 ).