[System Tests] Please, add sync_ceph_time method to all tests with check_ceph_health method

Bug #1349379 reported by Andrey Sledzinskiy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Artem Panchenko
5.0.x
Fix Released
Medium
Artem Panchenko

Bug Description

We have 'ceph_ha_restart' test in test_restart.py which failed with timeout
Traceback (most recent call last):
  File "/usr/lib/python2.7/unittest/case.py", line 331, in run
    testMethod()
  File "/usr/lib/python2.7/unittest/case.py", line 1043, in runTest
    self._testFunc()
  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.centos.thread_5/fuelweb_test/helpers/decorators.py", line 52, in wrapper
    return func(*args, **kwagrs)
  File "/home/jenkins/workspace/master_fuelmain.system_test.centos.thread_5/fuelweb_test/tests/tests_strength/test_restart.py", line 158, in ceph_ha_restart
    check_ceph_health(self.env.get_ssh_to_remote_by_name('slave-01'))
  File "/home/jenkins/workspace/master_fuelmain.system_test.centos.thread_5/fuelweb_test/__init__.py", line 48, in wrapped
    result = func(*args, **kwargs)
  File "/home/jenkins/workspace/master_fuelmain.system_test.centos.thread_5/fuelweb_test/helpers/checkers.py", line 33, in check_ceph_health
    timeout=360)
  File "/home/jenkins/venv-nailgun-tests/local/lib/python2.7/site-packages/devops/helpers/helpers.py", line 95, in wait
    raise TimeoutError("Waiting timed out")
TimeoutError: Waiting timed out

Please, add 'sync_ceph_time' method to this test and check where we need to add it too.
Probably to all tests with restart and check ceph health check

Tags: system-tests
Revision history for this message
Andrey Sledzinskiy (asledzinskiy) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-main (master)

Fix proposed to branch: master
Review: https://review.openstack.org/110329

Changed in fuel:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-main (master)

Reviewed: https://review.openstack.org/110329
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=09b718b8cdea78b17e08c61ef3590f3d9193b3b9
Submitter: Jenkins
Branch: master

commit 09b718b8cdea78b17e08c61ef3590f3d9193b3b9
Author: Artem Panchenko <email address hidden>
Date: Tue Jul 29 18:27:42 2014 +0300

    Add sync_ceph_time method to all Ceph tests

    Sometimes after reverting snapshots Ceph health check
    alerts about clock skew on some nodes, so an additional
    time sync and ceph service restart were added to the
    test. Also, refactored methods which are used for ceph
    checks.

    Change-Id: I6c875f19cd3802e8280de48d121b7900b25a0f3a
    Closes-bug: #1349379

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-main (stable/5.0)

Fix proposed to branch: stable/5.0
Review: https://review.openstack.org/115226

Changed in fuel:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-main (stable/5.0)

Reviewed: https://review.openstack.org/115226
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=80be30c5fc20e98e37d10c217b961e9601a65081
Submitter: Jenkins
Branch: stable/5.0

commit 80be30c5fc20e98e37d10c217b961e9601a65081
Author: Artem Panchenko <email address hidden>
Date: Tue Jul 29 18:27:42 2014 +0300

    Add sync_ceph_time method to all Ceph tests

    Sometimes after reverting snapshots Ceph health check
    alerts about clock skew on some nodes, so an additional
    time sync and ceph service restart were added to the
    test. Also, refactored methods which are used for ceph
    checks.

    Change-Id: I6c875f19cd3802e8280de48d121b7900b25a0f3a
    Closes-bug: #1349379
    (cherry picked from commit 3c7a8aa6e35d9e4ae36c8b2ae3ea39baec76a211)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.