[system tests] Add compute node to cluster in simple mode test fails with wrong Smiles count assertion

[Andrey Sledzinskiy]

'Add compute node to cluster in simple mode' test fails on smiles count - not reproduced locally:
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/proboscis/case.py", line 296, in testng_method_mistake_capture_func
    compatability.capture_type_error(s_func)
  File "/usr/lib/python2.7/dist-packages/proboscis/compatability/exceptions_2_6.py", line 27, in capture_type_error
    func()
  File "/usr/lib/python2.7/dist-packages/proboscis/case.py", line 350, in func
    func(test_case.state.get_state())
  File "/home/jenkins/workspace/master_fuelmain.system_test.ubuntu.thread_2/fuelweb_test/helpers/decorators.py", line 49, in wrapper
    return func(*args, **kwagrs)
  File "/home/jenkins/workspace/master_fuelmain.system_test.ubuntu.thread_2/fuelweb_test/tests/test_simple.py", line 246, in simple_flat_add_compute
    self.env.verify_node_service_list("slave-02", 8)
  File "/home/jenkins/workspace/master_fuelmain.system_test.ubuntu.thread_2/fuelweb_test/models/environment.py", line 374, in verify_node_service_list
    checkers.verify_service_list(remote, smiles_count)
  File "/home/jenkins/workspace/master_fuelmain.system_test.ubuntu.thread_2/fuelweb_test/__init__.py", line 48, in wrapped
    result = func(*args, **kwargs)
  File "/home/jenkins/workspace/master_fuelmain.system_test.ubuntu.thread_2/fuelweb_test/helpers/checkers.py", line 148, in verify_service_list
    _verify()
  File "/home/jenkins/workspace/master_fuelmain.system_test.ubuntu.thread_2/fuelweb_test/helpers/checkers.py", line 138, in _verify
    smiles_count, ''.join(ret['stdout']).count(":-)"), "Smiles count")
AssertionError: Smiles count

[Mike Scherbakov (mihgen)]

Can't we simply change smiles count to make test pass?

[Mike Scherbakov (mihgen)]

Interesting enough that issue is for CentOS only, and test passes (according to system tests table results) for Ubuntu.

[Tatyanka (tatyana-leontovich)]

Not we can not, because it shoul be exactly the same count, but I believe that merged path on sync ntpdate fixes this test

[Artem Panchenko (apanchenko-8)]

was fixed here https://bugs.launchpad.net/fuel/+bug/1320815