Bug #6000: Error output on Time settings at first run - Rudder - Issue Tracker

Actions

Copy link

Bug #6000

closed

Error output on Time settings at first run

Added by Nicolas CHARLES almost 10 years ago. Updated over 9 years ago.

Status:

Released

Priority:

N/A

Assignee:

Nicolas CHARLES

Category:

Techniques

Target version:

2.10.10

Pull Request:

https://github.com/Normation/rudder-t...

Severity:

UX impact:

User visibility:

Effort required:

Priority:

Name check:

Fix check:

Regression:

Description

The first execution of Time Settings 3.0 technique outputs following message on Centos 6

2014-12-15T13:08:49+0000    error: /default/check_clock_configuration/files/'/etc/ntp.conf'[0]: Promised to edit '/etc/ntp.conf', but file does not exist
2014-12-15T13:08:56+0000   notice: /default/check_clock_con[FAILED]on/commands/'/etc/init.d/ntpd'[0]: Q: "...nit.d/ntpd rest": Shutting down ntpd: 
Q: "...nit.d/ntpd rest": Starting ntpd:                    [  OK  ]

and ntp is not correctly configured
Relaunching the agent solves the issue

Tested on 2.11, may fail on earlier version on Technique/Rudder

Subtasks 1 (0 open — 1 closed)

Actions

Copy link

Updated by Jonathan CLARKE almost 10 years ago

Status changed from New to In progress
Assignee set to Jonathan CLARKE

This happens because the Technique tries to edit the ntp.conf file before checking the package is installed, so it does not exist.

Adding a simple check for this file's presence and using the result class of the package promise fixes this. PR in progress.

Actions

Copy link

Updated by Jonathan CLARKE almost 10 years ago

This is not CentOS specific, and is present from version 1.0 of this Technique.

Actions

Copy link

Updated by Jonathan CLARKE almost 10 years ago

Status changed from In progress to Pending technical review
Assignee changed from Jonathan CLARKE to Nicolas CHARLES
Target version changed from 2.11.6 to 2.10.9
Pull Request set to https://github.com/Normation/rudder-techniques/pull/616