Project

General

Profile

Actions
Copy link

User story #4620

closed

User story #4628: Only check a component status in the distributePolicy when it is present

Let rudder-agent handle the logrotate files instead of packages

User story #4620: Let rudder-agent handle the logrotate files instead of packages

Added by Matthieu CERDA over 12 years ago. Updated over 11 years ago.

Status:
Released
Priority:
1 (highest)
Assignee:
Jonathan CLARKE
Category:
System integration
Target version:
2.11.0~beta1
Pull Request:
https://github.com/Normation/rudder-p...
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:

Description

We should let Rudder itself handle the logrotate files, and just make the packaging clean them up in case of a deinstallation.

Related issues 1 (0 open — 1 closed)

Related to Rudder - Bug #4985: Remove logrotate management leftovers from rudder-server-rootReleasedJonathan CLARKEActions

Updated by Matthieu CERDA over 12 years ago Actions
Copy link
 #1

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

PR is ready !

Updated by Jonathan CLARKE over 12 years ago Actions
Copy link
 #2

  • Status changed from Pending technical review to Discussion
  • Assignee changed from Jonathan CLARKE to Matthieu CERDA
  • Target version changed from 2.10.0~beta1 to 135

The PR looks OK to me, but this is in no way a bug. It works just fine currently. So, please make this into a User story (meta), with sub-tickets (you can just make this ticket a sub-ticket, since there is already a PR, and add the meta ticket).

However, 2.10 is now in feature freeze, so this will not be merged until the first release has been made and the 2.10 branch has been created (next week). Please keep it open, so that is is managed then.

Updated by Matthieu CERDA over 12 years ago Actions
Copy link
 #3

  • Tracker changed from Bug to 6
  • Subject changed from We should let Rudder itself handle the logrotate files to We should let Rudder itself handle the logrotate files (integration)

Updated by Matthieu CERDA over 12 years ago Actions
Copy link
 #4

  • Parent task set to #4625

Updated by Matthieu CERDA over 12 years ago Actions
Copy link
 #5

Done

Updated by Jonathan CLARKE over 12 years ago Actions
Copy link
 #6

  • Target version changed from 135 to 2.11.0~beta1

Updated by Matthieu CERDA over 12 years ago Actions
Copy link
 #7

  • Assignee changed from Matthieu CERDA to Jonathan CLARKE

Updated by Jonathan CLARKE over 12 years ago Actions
Copy link
 #8

  • Status changed from Discussion to Pending technical review

This looks good to me. Let's merge it when we're ready.

Updated by Jonathan CLARKE over 12 years ago Actions
Copy link
 #9

  • Status changed from Pending technical review to 12

Updated by Jonathan CLARKE over 12 years ago Actions
Copy link
 #10

  • Project changed from 34 to Rudder
  • Subject changed from We should let Rudder itself handle the logrotate files (integration) to Let rudder-agent handle the logrotate files instead of packages
  • Category set to System integration
  • Parent task changed from #4625 to #4621

Updated by Matthieu CERDA about 12 years ago Actions
Copy link
 #11

  • Pull Request changed from https://github.com/Normation/rudder-packages/pull/264 to https://github.com/Normation/rudder-packages/pull/352

PR retargetted

Updated by Matthieu CERDA about 12 years ago Actions
Copy link
 #12

  • Status changed from 12 to Pending release

Applied in changeset packages:commit:f3ecff67418ed2fb0fbc57114c0f89e5bc6ffa0c.

Updated by Jonathan CLARKE about 12 years ago Actions
Copy link
 #13

Applied in changeset packages:commit:946b355b569ef198e97b68f63d73e85589515fa7.

Updated by Jonathan CLARKE about 12 years ago Actions
Copy link
 #14

  • Parent task changed from #4621 to #4628

Updated by Vincent MEMBRÉ about 12 years ago Actions
Copy link
 #15

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.11.0~beta1 that was released yesterday.

Updated by Benoît PECCATTE over 11 years ago Actions
Copy link
 #16

  • Tracker changed from 6 to User story
Actions
Copy link

Also available in: PDF Atom