[system-tests] ntp.py should show a more verbose exception message in case of failed time synchronization.

Bug #1582812 reported by Dennis Dmitriev on 2016-05-17
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Dennis Dmitriev

Bug Description

Detailed bug description:
  When time synchronization is failed, the following exception message can be shown:

  TimeoutError: Time on nodes was not set with 'ntpdate':
  [(u'slave-04', ['Sun May 15 02:16:49 UTC 2016\n'])]

  There is not enough information to find the root cause of failure because there is no the result of 'ntpdate' execution.

Steps to reproduce:
  On any node of the active environment set remote NTP server in the /etc/ntpd.conf to the node without ntpd, and try to execute 'dos.py time-sync <env_name>'

Expected results:
  Verbosed exception message with the executed command and stderr.

Actual result:
  Exception message without necessary details.

  Floating, depends on external issues.

  Manual reproduce on the failed environment (not always the valid way because the cause of the fail may be fixed)

  System tests

Description of the environment:
  Any environment that use fuel-devops framework

Additional information:

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

Changed in fuel:
assignee: Fuel QA Team (fuel-qa) → Dennis Dmitriev (ddmitriev)
status: New → In Progress

Reviewed: https://review.openstack.org/317639
Committed: https://git.openstack.org/cgit/openstack/fuel-devops/commit/?id=ef5f5ba103266d62309ff1d336aa0938c13e3d67
Submitter: Jenkins
Branch: release/2.9

commit ef5f5ba103266d62309ff1d336aa0938c13e3d67
Author: Dennis Dmitriev <email address hidden>
Date: Tue May 17 20:34:04 2016 +0300

    Increase verbosity of the timesync exception messages

    Add more details to the exception messages that could help to
    find the root cause of failed time sync.

    Change-Id: I3aa063271f8b9e4c04d5963bfd981388f811af11

Change abandoned by Fuel DevOps Robot (<email address hidden>) on branch: master
Review: https://review.openstack.org/317625
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

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

Other bug subscribers