Project

General

Profile

Actions

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

Added by Matthieu CERDA over 10 years ago. Updated about 9 years ago.

Status:
Released
Priority:
1
Category:
System integration
Target version:
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 open1 closed)

Related to Rudder - Bug #4985: Remove logrotate management leftovers from rudder-server-rootReleasedJonathan CLARKE2014-06-09Actions
Actions #1

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 0 to 100
  • Pull Request set to https://github.com/Normation/rudder-packages/pull/264

PR is ready !

Actions #2

Updated by Jonathan CLARKE over 10 years ago

  • 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.

Actions #3

Updated by Matthieu CERDA over 10 years ago

  • 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)
Actions #4

Updated by Matthieu CERDA over 10 years ago

  • Parent task set to #4625
Actions #5

Updated by Matthieu CERDA over 10 years ago

Done

Actions #6

Updated by Jonathan CLARKE over 10 years ago

  • Target version changed from 135 to 2.11.0~beta1
Actions #7

Updated by Matthieu CERDA over 10 years ago

  • Assignee changed from Matthieu CERDA to Jonathan CLARKE
Actions #8

Updated by Jonathan CLARKE about 10 years ago

  • Status changed from Discussion to Pending technical review

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

Actions #9

Updated by Jonathan CLARKE about 10 years ago

  • Status changed from Pending technical review to 12
Actions #10

Updated by Jonathan CLARKE about 10 years ago

  • 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
Actions #11

Updated by Matthieu CERDA about 10 years ago

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

PR retargetted

Actions #12

Updated by Matthieu CERDA about 10 years ago

  • Status changed from 12 to Pending release

Applied in changeset packages:commit:f3ecff67418ed2fb0fbc57114c0f89e5bc6ffa0c.

Actions #13

Updated by Jonathan CLARKE about 10 years ago

Applied in changeset packages:commit:946b355b569ef198e97b68f63d73e85589515fa7.

Actions #14

Updated by Jonathan CLARKE about 10 years ago

  • Parent task changed from #4621 to #4628
Actions #15

Updated by Vincent MEMBRÉ about 10 years ago

  • Status changed from Pending release to Released

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

Actions #16

Updated by Benoît PECCATTE about 9 years ago

  • Tracker changed from 6 to User story
Actions

Also available in: Atom PDF