sntp segfaults on 6.1-95 master, time is in future

Bug #1417574 reported by Mykhaylo Slobodyan
32
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Critical
Stanislaw Bogatkin
6.0.x
Invalid
Undecided
Stanislaw Bogatkin

Bug Description

While deploying Fuel master node from 6.1-95 .iso (in VM with autotests), on console I see this:

sntp: Synchronizing with time server: sntp[983]: segfault at 0 ip 000000000041f9ea sp 00007fb3ca9e7e00 error 6 in sntp[400000+37000]
/etc/rc3.d/S57sntp: line 25: 979 Segmentation fault /usr/bin/sntp $OPTIONS $tickers &>/dev/null

Actual: it segfaults.
Expected: it shouldn't segfault.

As a side note, some manifests from that master, which were copied by scp, seem to be through time machine:
tar: deployment_1/primary-controller_1.yaml: time stamp 2037-08-20 06:17:06 is 711383003.795305921 s in the future

http://jenkins-product.srt.mirantis.net:8080/job/6.1.ubuntu.smoke_neutron/28/console

Revision history for this message
Mykhaylo Slobodyan (mykhaylo-slobodyan) wrote :
Changed in fuel:
milestone: none → 6.1
Changed in fuel:
importance: Undecided → Medium
assignee: nobody → Stanislaw Bogatkin (sbogatkin)
status: New → Confirmed
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-library (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/152963

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-main (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/152969

Changed in fuel:
importance: Medium → Critical
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-main (master)

Reviewed: https://review.openstack.org/152969
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=579dfc1479d5fc1eaa00b8459a2b75009ed64c01
Submitter: Jenkins
Branch: master

commit 579dfc1479d5fc1eaa00b8459a2b75009ed64c01
Author: Vladimir Kuklin <email address hidden>
Date: Thu Feb 5 02:22:53 2015 +0300

    Remove ntp-dev from req's and stop using sntp

    Change-Id: Id0e404009f7adccdf26fbfe7979721c2fc9a94a9
    Related-Bug: #1417574

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/152963
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=7a063444297ac2842e5f2817250bf3660d2f611d
Submitter: Jenkins
Branch: master

commit 7a063444297ac2842e5f2817250bf3660d2f611d
Author: Stanislaw Bogatkin <email address hidden>
Date: Wed Feb 4 19:02:53 2015 +0300

    Stop using sntp and start using ntpdate

    Change-Id: I598b3c7edacf0f61a6fc04efac06b25fad2be1f4
    Related-Bug: #1417574

Revision history for this message
Bogdan Dobrelya (bogdando) wrote :
Changed in fuel:
status: Confirmed → Fix Committed
Revision history for this message
Dmitry Borodaenko (angdraug) wrote :

This reverts the fix for bug #1336748 -- should it be reopened?

Revision history for this message
Stanislaw Bogatkin (sbogatkin) wrote :

I'm against that. We start using old ntpdate with timeouts that it has. Not best, but wors. So, we will use old ntpdate for some time, then if it will not fit into ours needs, new bug could be filled or this old can br reopened.

Revision history for this message
Andrey Sledzinskiy (asledzinskiy) wrote :

verified on {

    "build_id": "2015-02-15_22-54-44",
    "ostf_sha": "f9c37d0876141e1550eb4e703a8e500cd463282f",
    "build_number": "126",
    "release_versions":

{

    "2014.2-6.1":

{

    "VERSION":

{

    "build_id": "2015-02-15_22-54-44",
    "ostf_sha": "f9c37d0876141e1550eb4e703a8e500cd463282f",
    "build_number": "126",
    "api": "1.0",
    "nailgun_sha": "1e3a40dd8a17abe1d38f42da1e0dc1a6d4572666",
    "production": "docker",
    "python-fuelclient_sha": "61431ed16fc00039a269424bdbaa410277eff609",
    "astute_sha": "1f87a9b9a47de7498b4061d15a8c7fb9435709d5",
    "feature_groups":

            [
                "mirantis"
            ],
            "release": "6.1",
            "fuelmain_sha": "2054229e275d08898b5d079a6625ffcc79ae23b8",
            "fuellib_sha": "7f8d4382abfcd4338964182ebfea1d539f963e66"
        }
    }

},
"auth_required": true,
"api": "1.0",
"nailgun_sha": "1e3a40dd8a17abe1d38f42da1e0dc1a6d4572666",
"production": "docker",
"python-fuelclient_sha": "61431ed16fc00039a269424bdbaa410277eff609",
"astute_sha": "1f87a9b9a47de7498b4061d15a8c7fb9435709d5",
"feature_groups":

    [
        "mirantis"
    ],
    "release": "6.1",
    "fuelmain_sha": "2054229e275d08898b5d079a6625ffcc79ae23b8",
    "fuellib_sha": "7f8d4382abfcd4338964182ebfea1d539f963e66"

}

Changed in fuel:
status: Fix Committed → Fix Released
Revision history for this message
Stanislaw Bogatkin (sbogatkin) wrote :

We don't have new sntp in 6.0. Closed.

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.