Project

General

Profile

Actions

Backport #16441

closed

jewel: [initscripts] systemd-run is not needed in initscripts

Added by Vladislav Odintsov almost 8 years ago. Updated over 7 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
Release:
jewel
Pull request ID:
Crash signature (v1):
Crash signature (v2):


Related issues 1 (0 open1 closed)

Copied from devops - Fix #16440: [initscripts]: systemd-run is not needed in initscriptsResolvedVladislav Odintsov06/23/2016

Actions
Actions #1

Updated by Vladislav Odintsov almost 8 years ago

not a bug. should be changed to fix.
PR: https://github.com/ceph/ceph/pull/9872

base ticket:
http://tracker.ceph.com/issues/16440

Actions #2

Updated by Nathan Cutler almost 8 years ago

  • Tracker changed from Bug to Backport
  • Subject changed from [backport][jewel][initscripts]: systemd-run is not needed in initscripts to jewel: systemd-run is not needed in initscripts
  • Description updated (diff)
  • Status changed from New to In Progress

original description

systemd-run logic in initscripts was introduced because of ticket http://tracker.ceph.com/issues/7627. If we have systemd-based distro, we should use systemd unit files from systemd directory to start/stop ceph daemons. Otherwise, daemon() from /etc/init.d/functions on systemd distro starts service in system.slice and everything works well for case, for example, when we use hammer on RH7. With this code it will start daemon with daemon() function from init.d/functions. systemd-run can not be run on non-systemd distros, so it's not needed on SysV systems. I've performed successful testing with hammer on RH6, RH7 systems with ceph-osd, ceph-mon, ceph-radosgw services.

Actions #3

Updated by Nathan Cutler almost 8 years ago

  • Copied from Fix #16440: [initscripts]: systemd-run is not needed in initscripts added
Actions #4

Updated by Nathan Cutler almost 8 years ago

  • Subject changed from jewel: systemd-run is not needed in initscripts to jewel: [initscripts] systemd-run is not needed in initscripts
Actions #5

Updated by Nathan Cutler almost 8 years ago

  • Status changed from In Progress to Need More Info

Backporting note: do not process further until https://github.com/ceph/ceph/pull/9871 has been merged and cherry-pick SHA1 has been double-checked.

Actions #6

Updated by Vladislav Odintsov almost 8 years ago

Thanks, Nathan, I'll cherry-pick commits from master to backport-branch after merge to master.

Actions #7

Updated by Loïc Dachary over 7 years ago

  • Status changed from Need More Info to New
Actions #8

Updated by Loïc Dachary over 7 years ago

  • Copied from deleted (Fix #16440: [initscripts]: systemd-run is not needed in initscripts)
Actions #9

Updated by Loïc Dachary over 7 years ago

  • Status changed from New to Rejected
Actions #10

Updated by Nathan Cutler over 7 years ago

  • Status changed from Rejected to Resolved
  • Assignee set to Nathan Cutler
Actions #11

Updated by Nathan Cutler over 7 years ago

  • Target version set to v10.2.4
Actions #12

Updated by Nathan Cutler over 7 years ago

  • Project changed from Ceph to devops
Actions #13

Updated by Nathan Cutler over 7 years ago

  • Copied from Fix #16440: [initscripts]: systemd-run is not needed in initscripts added
Actions

Also available in: Atom PDF