User story #3939: Remove temporary rudder-agent-uuid cron file - Rudder - Issue Tracker

Actions

Copy link

User story #3939

closed

User story #3938: Since #3925 the temporary rudder-agent-uuid cron file must be removed by the System Techniques

Remove temporary rudder-agent-uuid cron file

Added by Nicolas PERRON over 11 years ago. Updated over 9 years ago.

Status:

Rejected

Priority:

1 (highest)

Assignee:

Jonathan CLARKE

Category:

Techniques

Target version:

2.4.9

Pull Request:

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

UX impact:

Suggestion strength:

User visibility:

Effort required:

Name check:

Fix check:

Regression:

Description

Remove with System Techniques temporary /etc/cron.d/rudder-agent-uuid.

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Nicolas PERRON over 11 years ago

Tracker changed from Bug to 6

Actions

Copy link

Updated by Nicolas PERRON over 11 years ago

Status changed from New to Pending technical review
Assignee changed from Nicolas PERRON to Jonathan CLARKE
% Done changed from 0 to 100
Pull Request set to https://github.com/Normation/rudder-techniques/pull/189

Pull Request URL added: https://github.com/Normation/rudder-techniques/pull/189

Jon, could you review it please ?

It will need to be merged after #3925

Actions

Copy link

Updated by Jonathan CLARKE over 11 years ago

Status changed from Pending technical review to Rejected

This does not make sense. The cron file "rudder-agent-uuid" will only be useless once the main cron file "rudder-agent" contains a call to the script that also checks the UUID. That is introduced in #3930 and will only be in 2.8. Therefore, we must not remove this in 2.4.*.

Please add this logic to the PR in #3930. I'll reject this.

Actions

Copy link