All of lore.kernel.org
 help / color / mirror / Atom feed
From: Loic Dachary <loic@dachary.org>
To: Owen Synge <osynge@suse.com>, ceph-devel@vger.kernel.org
Subject: Re: autodetecting init system.
Date: Tue, 12 May 2015 11:45:32 +0200	[thread overview]
Message-ID: <5551CBBC.2040303@dachary.org> (raw)
In-Reply-To: <5551B221.9070704@suse.com>

[-- Attachment #1: Type: text/plain, Size: 5305 bytes --]



On 12/05/2015 09:56, Owen Synge wrote:
> On 05/11/2015 11:45 PM, Loic Dachary wrote:
>> Hi Owen,
>>
>> It would help to provide one or two use cases where (C) solves a problem 
>> that (B) (that is the current ceph-detect-init approach) does not
>> solve.
> 
>> I sense there is something better in (C) but I can't think of a use case 
>> just now
>> (maybe because I've been thinking about erasure code all day :-).
> 
> Hi Loic,
> 
> Please note that I believe we are correct to explicitly allow the user
> to specify the init system and override any auto detection.
> 
> Hence I believe it is correct for autodetection to be able to fail.
> 
> Use case (1)
> ~~~~~~~~~~~~
> 
> When a new release of an operating system comes out with a different
> init system (B) as a static database will not immediately account for
> this solution.
> 
> For example
> 
> SUSE SLE 11 -> sysV
> SUSE SLE 12 -> systemd
> RHEL 5 -> sysV
> RHEL 6 -> upstart
> RHEL 7 -> systemd
> 
> When for example SLE12 came out ceph upstream code assumed ceph ran on
> the sysV init system until appropriate patches where taken.

So Ceph failed to run on SLE12 because it was relying on sysV ?

> 
> Since we can only test on "free as in beer" operating systems at this
> moment covering these OS's with the database and appropriate tests is
> problematic.
> 
> Use case (2)
> ~~~~~~~~~~~~
> 
> The use cases are on the latest debian, ubuntu platforms.
> 
> On both these platforms you can install alternative init systems.
> 
> on debian stable I can apt-get install the following init systems:
> 
> systemd
> upstart
> sysvinit
> 
> Hence assuming all debian stable systems are systemd (the default) is a
> false assumption as (B) does not support users changing the init system
> before installing ceph on debian and ubuntu as having a database of init
> systems cannot support all platforms.
> 
> Use case (3)
> ~~~~~~~~~~~~
> 
> By supporting (B) and (C) and emitting a warning on operating systems
> not in the database, populating the database will be quicker, and
> correcting values in the database will be easier to verify.
> 
> In some ways, the code tests its self, at run time.
> 
> Summary:
> ~~~~~~~~
> I think its a value decision, is the extra complexity of doing (B) and
> (C) worth the corner case of supporting the people who chose to use non
> default init system worth the code complexity. If we are supporting more
> than one init detection mechanism, it may well be worth supporting all
> of (A) (B) and (c).
> 
> Best regards
> 
> Owen
> 
> 
> 
>> Cheers
>>
>> On 11/05/2015 18:29, Owen Synge wrote:
>>> Dear all,
>>>
>>> Many init systems are used in linux now. Some ceph code needs to know
>>> the init system. (I must admit I have not looked into Solaris, MacOS and
>>> BSD and probably should have)
>>>
>>> It would be nice to have one function that detects the init system
>>>
>>> Since the init system can be specified in ceph and ceph-deploy
>>> explicitly it seems to be its reasonable to fail clearly to detect init
>>> system.
>>>
>>> I see 4 ways I can see to detect init system.
>>>
>>> (A) Check pid 1.
>>> (B) Use a database of OS to init mapping / compile time.
>>> (C) look for init manipulation tools and infure the init system from tools.
>>>
>>> Comments:
>>> ~~~~~~~~
>>>
>>> (A1) systmd can be detected easily with.
>>>
>>>  grep -qs systemd /proc/1/comm
>>>
>>> (A2) With init scripts such as its hard to know what the init system.
>>>
>>> (B1) For operating systems like RHEL, SLE, CENTOS, Fedora and scientific
>>> linux this works well.
>>>
>>> (B2) FOr operating systems like newer debian and ubuntu releases more
>>> than one init system can be installed and used on the OS, so making a
>>> database / doing it at compile time are not practical on all OS's
>>>
>>> (C1) This is fairly reliable.
>>>
>>> (C2) sysV tools have compatibility scripts / programs on other platforms
>>> so if you use a points system for each init system helper script you can
>>> infure systemd over sysV if sytemctrl exists for example.
>>>
>>> So to summarise this:
>>> ~~~~~~~~~~~~~~~~~~~~
>>>
>>> (1) No one system is perfect in all cases.
>>> (2) Combined these systesm can provide reliable init system detection.
>>>
>>> My proposed approach.
>>> ~~~~~~~~~~~~~~~~~~~~
>>>
>>> (I) Use all three approaches where each approach can provide and answer,
>>> or fail to provide an answer.
>>>
>>> (II) Should any approaches disagree -> fail to detect init system.
>>>
>>> (III) Should all approaches agree -> then return init system.
>>>
>>> (III) Should no approaches provide an init system -> fail to return init
>>> system.
>>>
>>> Comments
>>> ~~~~~~~~
>>>
>>> This multi layered and comparing way of doing init systems may seem
>>> complete overkill, or maybe its useful.
>>>
>>> What do you guys think?
>>>
>>> Best regards
>>>
>>> Owen
>>>
>>>
>>>
>>>
>>>
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
>>> the body of a message to majordomo@vger.kernel.org
>>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>>
>>
> 

-- 
Loïc Dachary, Artisan Logiciel Libre


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

  reply	other threads:[~2015-05-12  9:45 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-11 16:29 autodetecting init system Owen Synge
2015-05-11 16:37 ` John Spray
2015-05-11 17:26   ` Owen Synge
2015-05-11 17:34   ` Sage Weil
2015-05-12  7:25     ` Owen Synge
2015-05-11 17:28 ` Owen Synge
2015-05-11 21:45 ` Loic Dachary
2015-05-12  7:56   ` Owen Synge
2015-05-12  9:45     ` Loic Dachary [this message]
2015-05-12 10:27       ` Owen Synge
2015-05-12 12:34         ` Loic Dachary

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5551CBBC.2040303@dachary.org \
    --to=loic@dachary.org \
    --cc=ceph-devel@vger.kernel.org \
    --cc=osynge@suse.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.