Optimize ntpdate in tasks.yaml

Bug #1499420 reported by Sergii Golovatiuk
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Low
Sergii Golovatiuk

Bug Description

Fuel 7.0 has

ntpdate -u $(egrep '^server' /etc/ntp.conf | egrep -v '127\.127\.[0-9]+\.[0-9]+' | sed '/^#/d' | awk '{print $2}' | head -1)

in tasks.yaml. It may be replaced with less pipes as

ntpdate -u $(egrep '^server' /etc/ntp.conf | awk '!/127\.127\.[0-9]+\.[0-9]/ {print $2; exit }')

Tags: area-library
Changed in fuel:
status: New → Confirmed
importance: Undecided → Low
assignee: nobody → Sergii Golovatiuk (sgolovatiuk)
milestone: none → 8.0
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

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

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

commit dcbf7260142e3208aeb9d63de3558cc5bd0ccdf7
Author: Sergii Golovatiuk <email address hidden>
Date: Thu Sep 24 17:40:26 2015 +0200

    Refactor ntpdate in tasks.yaml

    Improve ntpdate
    - it gets servers with awk without any pipes
    - it gets list of all servers instead of one.

    Change-Id: I648e59d4a27ed9dc5886b5733053d35d58e2c7cf
    Closes-Bug: 1499420
    Signed-off-by: Sergii Golovatiuk <email address hidden>

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Anastasia Palkina (apalkina) wrote :

Verified on ISO #151

"fuel-createmirror_sha": "fe81731b76071dc0a4ffdf039080732243f5a871", "api": "1.0", "fuelmain_sha": "f5b83e3f0ccfce90b9ca797bc039c9ab50b5ee55", "astute_sha": "a090546d43c770ac27ca81c6f8c78ff0ba4a93e0", "fuel-ostf_sha": "e308a7ad68838f6e012d6d80243eb32b4f7e6c18", "openstack_version": "2015.1.0-8.0", "build_id": "151", "build_number": "151", "auth_required": true, "fuel-library_sha": "d4b67bca618c48332486339f8342413ea9b4bc3b", "fuel-nailgun-agent_sha": "08e0a11cf1f29b705e4b910d9b9db5e9b708b6e3", "fuel-nailgun_sha": "56a1ca6c1889ae5a8378198db65c0168c8c06484", "fuel-agent_sha": "5d2d5e2e9ad4f8bb01e450240d85447608d84b92", "production": "docker", "python-fuelclient_sha": "655db5e96282bbafc4dae54462a01823b6b464cb", "feature_groups": ["mirantis"], "release": "8.0", "release_versions": {"2015.1.0-8.0": {"VERSION": {"fuel-createmirror_sha": "fe81731b76071dc0a4ffdf039080732243f5a871", "api": "1.0", "fuelmain_sha": "f5b83e3f0ccfce90b9ca797bc039c9ab50b5ee55", "astute_sha": "a090546d43c770ac27ca81c6f8c78ff0ba4a93e0", "fuel-ostf_sha": "e308a7ad68838f6e012d6d80243eb32b4f7e6c18", "openstack_version": "2015.1.0-8.0", "build_id": "151", "build_number": "151", "fuel-library_sha": "d4b67bca618c48332486339f8342413ea9b4bc3b", "fuel-nailgun-agent_sha": "08e0a11cf1f29b705e4b910d9b9db5e9b708b6e3", "fuel-nailgun_sha": "56a1ca6c1889ae5a8378198db65c0168c8c06484", "fuel-agent_sha": "5d2d5e2e9ad4f8bb01e450240d85447608d84b92", "production": "docker", "python-fuelclient_sha": "655db5e96282bbafc4dae54462a01823b6b464cb", "feature_groups": ["mirantis"], "release": "8.0"}}}

Changed in fuel:
status: Fix Committed → Fix Released
Dmitry Pyzhov (dpyzhov)
tags: added: area-library
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.