ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: "Dmitry V. Levin" <ldv@altlinux.org>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] обновления пакетов с поддержкой systemd
Date: Wed, 25 Jan 2012 01:19:22 +0400
Message-ID: <20120124211922.GB1902@altlinux.org> (raw)
In-Reply-To: <20110517135955.GC9872@altlinux.org>

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

On Tue, May 17, 2011 at 05:59:55PM +0400, Dmitry V. Levin wrote:
> On Tue, May 17, 2011 at 04:35:13PM +0400, Alexey Shabalin wrote:
> > 17 мая 2011 г. 7:42 пользователь Alexey Shabalin <a.shabalin@gmail.com> написал:
> > > 17 мая 2011 г. 7:21 пользователь Dmitry V. Levin  написал:
> > >> On Tue, May 17, 2011 at 07:12:07AM +0400, Alexey Shabalin wrote:
> > >> [...]
> > >>> Надо запустить что-то типа
> > >>> if chkconfig foobar ; then
> > >>>   chkconfig foobar on
> > >>> fi
> > >>> (если сервис включен, включить его ещё раз - в этот раз уже с
> > >>> перенаправлением на systemd)
> > >> [...]
> > >>> Я пока придумал только модифицировать post_service:
> > >>>
> > >>> --- post_service.def  2011-03-17 01:41:42.000000000 +0300
> > >>> +++ post_service      2011-05-17 07:02:27.640147135 +0400
> > >> [...]
> > >>> @@ -14,5 +30,9 @@
> > >>>       /sbin/chkconfig --add "$1"
> > >>>  else
> > >>>       /sbin/chkconfig "$1" resetpriorities ||:
> > >>> +     [ -f "$SYSTEMD_SERVICE_DIR/$1.service" ] &&
> > >>> +             systemd_is_active &&
> > >>> +             /sbin/chkconfig --no-redirect "$1" &&
> > >>> +             /sbin/chkconfig "$1" on
> > >>>       /sbin/service "$1" condrestart ||:
> > >>>  fi
> > >>
> > >> Я бы предпочел, чтобы либо chkconfig resetpriorities выполнял эту
> > >> синхронизацию, либо появился еще один режим в chkconfig для синхронизации.
> > >> Вариант с resetpriorities предпочтительнее, поскольку тогда не придется
> > >> трогать пользователей chkconfig'а.
> > >
> > > Боюсь, что я этого не осилю. Максимум, что могу - сделать всегда
> > > включение сервиса при resetpriorities :)
> > 
> > В общем у меня получилось следущее:
> > 
> > 	    return setService(name, type, where, 0);
> >          } else if (!strcmp(state, "reset"))
> >  	    return setService(name, type, where, -1);
> > -	else if (!strcmp(state, "resetpriorities"))
> > +	else if (!strcmp(state, "resetpriorities")) {
> > +	    if (!noRedirectItem && isOn(name, level)) {
> > +		forwardSystemd(name, type, "enable");
> > +	    }
> >  	    return setService(name, type, where, -2);
> > +	}
> >  	else
> >  	    usage(EXIT_FAILURE);
> >      }
> > 
> > Если используется systemd и если раньше сервис был включён, то при
> > запуске с resetpriorities ещё запускается systemctl enable для
> > сервиса.
> > Так устроит?
> 
> Судя по описанию, да.
> 
> P.S. код я не смотрел.

Посмотрел код.  Нет, так совсем не годится.  Надо сперва менять состояние
(т.е. вызывать setService, который, между прочим, вызывает
"systemctl daemon-reload"), а потом уже форвардить результат в systemd.
То же самое касается и "chkconfig reset".


-- 
ldv

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

      parent reply	other threads:[~2012-01-24 21:19 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-17  3:12 Alexey Shabalin
2011-05-17  3:21 ` Dmitry V. Levin
2011-05-17  3:42   ` Alexey Shabalin
2011-05-17 12:35     ` Alexey Shabalin
2011-05-17 13:59       ` Dmitry V. Levin
2011-05-21 12:42         ` Alexey Shabalin
2011-05-27 17:11           ` Dmitry V. Levin
2011-05-30 13:10             ` Alexey Shabalin
2011-05-30 14:22             ` Alexey Shabalin
2011-06-10 18:54               ` Alexey Shabalin
2011-06-10 18:49             ` Alexey Shabalin
2011-08-11 11:36               ` Alexey Shabalin
2012-01-24 21:19         ` Dmitry V. Levin [this message]

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=20120124211922.GB1902@altlinux.org \
    --to=ldv@altlinux.org \
    --cc=devel@lists.altlinux.org \
    /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

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git