Bug #2440

rudder-server-root init script doesn't have right permissions on SLES or RHEL

Added by Nicolas PERRON over 3 years ago. Updated 6 months ago.

Status:ReleasedStart date:2012-04-13
Priority:N/ADue date:
Assignee:Nicolas PERRON% Done:

100%

Category:Packaging
Target version:2.3.7
Pull Request: How to reproduce:

Reproduced: Found in version(s):

Description

# /etc/init.d/rudder-server-root
sudo: /etc/init.d/rudder-server-root: command not found
# ls -lh /etc/init.d/rudder-server-root 
-rw-r--r-- 1 root root 1,7K avril 11 03:42 /etc/init.d/rudder-server-root

It must be 0755 not 0600

Associated revisions

Revision 83f4e305
Added by Nicolas PERRON over 3 years ago

Fixes #2440 Set mode 0755 for /etc/init.d/rudder-server-root for SLES

Revision 1328f2c5
Added by Jonathan CLARKE over 3 years ago

Merge branch 'branches/rudder/2.3' into branches/rudder/2.4

  • branches/rudder/2.3:
    Fixes #2440 Set mode 0755 for /etc/init.d/rudder-server-root for SLES
    Fixes #2288 Force jetty init script to start after slapd for Debian and SLES

History

#1 Updated by Jonathan CLARKE over 3 years ago

Is this really the case on all platforms? I'm sure it's not, because I use that script often (on Debian Squeeze)

#2 Updated by Nicolas PERRON over 3 years ago

  • Subject changed from rudder-server-root init script doesn't have right permissions to rudder-server-root init script doesn't have right permissions on SLES or RHEL

Jonathan CLARKE wrote:

Is this really the case on all platforms? I'm sure it's not, because I use that script often (on Debian Squeeze)

No, it's only on RPM because of a mistake on rudder-server-root SPEC file.

#3 Updated by Nicolas PERRON over 3 years ago

  • Status changed from New to Pending technical review
  • % Done changed from 0 to 100

Applied in changeset commit:83f4e305abf073a1b59014df58e43edcb955cda1.

#4 Updated by Nicolas PERRON over 3 years ago

This instruction in SPEC file doesn't set files mode but directories:

%defattr(-, root, root, 0755)

#5 Updated by Jonathan CLARKE over 3 years ago

  • Status changed from Pending technical review to Released

Looks good to me

#6 Updated by Nicolas PERRON over 2 years ago

  • Project changed from Rudder to packages
  • Category deleted (11)

#7 Updated by Benoît PECCATTE 6 months ago

  • Project changed from packages to Rudder
  • Category set to Packaging

Also available in: Atom PDF