Project

General

Profile

Actions
Copy link

Architecture #3931

closed

Architecture #3930: Rudder-agent cron file now use check-rudder-agent script (/opt/rudder/bin/check-rudder-agent)

Modify the cron file to use /opt/rudder/bin/check-rudder-agent

Added by Nicolas PERRON over 10 years ago. Updated about 9 years ago.

Status:
Released
Priority:
1
Assignee:
Jonathan CLARKE
Category:
System techniques
Target version:
2.8.0~rc1
Pull Request:
https://github.com/Normation/rudder-t...
Effort required:
Name check:
Fix check:
Regression:

Description

Modify the cron file to use /opt/rudder/bin/check-rudder-agent instead of checking directly the CFEngine processes

Actions
Copy link
 #1

Updated by Nicolas PERRON over 10 years ago

  • Target version changed from 2.4.9 to 2.7.3
Actions
Copy link
 #2

Updated by Nicolas PERRON over 10 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/185

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

Jon, could you review it please ?

Actions
Copy link
 #3

Updated by Jonathan CLARKE over 10 years ago

  • Project changed from 24 to Rudder
  • Category set to System techniques
  • Status changed from Pending technical review to Discussion
  • Assignee changed from Jonathan CLARKE to Nicolas PERRON
  • Target version changed from 2.7.3 to 2.8.0~beta1
  • Parent task deleted (#3930)

This looks fine, but it should be on master. We do not introduce changes in released branches. Please re-target the PR.

Also, please do not create tickets about system Techniques in the Techniques project. They do not belong here, they are a core part of Rudder. Moving.

Actions
Copy link
 #4

Updated by Jonathan CLARKE over 10 years ago

  • Parent task set to #3930
Actions
Copy link
 #5

Updated by Jonathan CLARKE over 10 years ago

This PR should also include the logic to remove /etc/cron.d/rudder-agent-uuid, originally proposed in #3939.

Actions
Copy link
 #6

Updated by Nicolas PERRON over 10 years ago

  • Status changed from Discussion to Pending technical review
  • Assignee changed from Nicolas PERRON to Jonathan CLARKE

Jonathan CLARKE wrote:

This PR should also include the logic to remove /etc/cron.d/rudder-agent-uuid, originally proposed in #3939.

Done. And branch changed from 2.7 to master (2.8)

Actions
Copy link
 #7

Updated by Jonathan CLARKE over 10 years ago

  • Status changed from Pending technical review to Discussion
  • Assignee changed from Jonathan CLARKE to Nicolas PERRON

Nicolas PERRON wrote:

Jonathan CLARKE wrote:

This PR should also include the logic to remove /etc/cron.d/rudder-agent-uuid, originally proposed in #3939.

Done. And branch changed from 2.7 to master (2.8)

The PR is closed.

Actions
Copy link
 #8

Updated by Nicolas PERRON over 10 years ago

  • Pull Request changed from https://github.com/Normation/rudder-techniques/pull/185 to https://github.com/Normation/rudder-techniques/pull/196

Jonathan CLARKE wrote:

Nicolas PERRON wrote:

Jonathan CLARKE wrote:

This PR should also include the logic to remove /etc/cron.d/rudder-agent-uuid, originally proposed in #3939.

Done. And branch changed from 2.7 to master (2.8)

The PR is closed.

Oops, here is the new PR: https://github.com/Normation/rudder-techniques/pull/196

Actions
Copy link
 #9

Updated by Nicolas PERRON over 10 years ago

  • Status changed from Discussion to Pending technical review
  • Assignee changed from Nicolas PERRON to Jonathan CLARKE
Actions
Copy link
 #10

Updated by Jonathan CLARKE over 10 years ago

  • Status changed from Pending technical review to Discussion
  • Assignee changed from Jonathan CLARKE to Nicolas PERRON
Actions
Copy link
 #11

Updated by Nicolas PERRON over 10 years ago

  • Status changed from Discussion to Pending technical review
  • Assignee changed from Nicolas PERRON to Jonathan CLARKE
Actions
Copy link
 #12

Updated by Jonathan CLARKE over 10 years ago

  • Status changed from Pending technical review to Discussion
  • Assignee changed from Jonathan CLARKE to Nicolas PERRON
Actions
Copy link
 #13

Updated by Nicolas PERRON over 10 years ago

  • Target version changed from 2.8.0~beta1 to 2.8.0~rc1

I think that this should be added in Rudder 2.8

Actions
Copy link
 #14

Updated by Matthieu CERDA over 10 years ago

  • Status changed from Discussion to In progress
  • Assignee changed from Nicolas PERRON to Matthieu CERDA
  • % Done changed from 100 to 80
  • Pull Request deleted (https://github.com/Normation/rudder-techniques/pull/196)

Taking this.

Actions
Copy link
 #15

Updated by Matthieu CERDA over 10 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Matthieu CERDA to Jonathan CLARKE
  • % Done changed from 80 to 100
  • Pull Request set to https://github.com/Normation/rudder-techniques/pull/231

PR available

Actions
Copy link
 #16

Updated by Matthieu CERDA over 10 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset policy-templates:commit:4dc72dd4dea7e5d90407507c460c03d92006f3f2.

Actions
Copy link
 #17

Updated by Vincent MEMBRÉ over 10 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.8.0~rc1, which was released on Friday 25/10/2013.
Check out:

Actions
Copy link
 #18

Updated by Benoît PECCATTE about 9 years ago

  • Tracker changed from 6 to Architecture
Actions
Copy link

Also available in: Atom PDF