ALT Linux Sisyphus discussions
 help / color / mirror / Atom feed
From: sergey ivanov <seriv@parkheights.dyndns.org>
To: ALT Linux Sisyphus discussion list <sisyphus@altlinux.ru>
Subject: Re: [sisyphus] ez-ipupdate: критика
Date: Tue, 01 Jun 2004 09:43:21 -0400
Message-ID: <40BC87F9.10304@parkheights.dyndns.org> (raw)
In-Reply-To: <20040529182705.GA10922@home-pool-173-97.com2com.ru>

Yura Zotov пишет:

>Обнаружил, что инитскрипт для ez-ipupdate срабатывает с ошибками.
>Разборки выявили как банальные опечатки, так и более
>принципиальные недочёты. На основе этого скрипта и
>/etc/init.d/template я написал новый скрипт, который работает как
>положено. Он прицеплен. Предлагаю включить в пакет ez-ipupdate
>новый скрипт вместо старого.
>
>P.S. Не совсем понял, какой смысл в выделении файла
>ez-ipupdate.pid в отдельный каталог /var/run/ez-ipupdate/.
>По-моему его было бы правильнее держать вместе с остальными
>pid-файлами в /var/run/. Ведь с lock-файлом вы (обращение к
>мантейнеру пакета) так и поступали, оставляя его в
>/var/lock/subsys/.
>  
>
Большое спасибо за критику и за предложенный скрипт.
    Я попытался разобраться с pid-файлом. Нашел способ, как его можно 
держать в /var/run а не в специальной поддиректории. Для этого нужно:
1) передать создание этого файла start-stop-daemon'у, который работает с 
правами root;
2) для этого передать с помощью хитрости, например, приклеив к параметру 
$EZ_PID ключик --make-pidfile,  который функциями 
/etc/rc.d/init.d/functions напрямую не поддерживается;
3) и потом ещё надо отнять у ez-ipupdate функциональность демона, 
отсоединяющегося от консоли в которой был запущен, иначе 
start-stop-daemon получает pid не того процесса, который будет работать, 
а того который создается для запуска
этого демона и тут же завершается.
    Поэтому я решил что лучше оставить с pid-файлами как есть, то есть 
создать для них поддиректорию в которую имеет право писать 
непривилегированный юзер.
    Однако мне показалось, что примерно ту же функциональность, что и 
pid-файлы, обеспечивают lock-файлы. И тогда непонятно, чем может быть 
оправдано одновременное применение /var/lock/subsystem/* файлов и 
pid-файлов в одном скрипте. Я пробовал, и у меня функции condrestart и 
condreload хорошо работали и с ипользованием просто пути к исполнимой 
программе /usr/sbin/ez-ipupdate, и с использованием pid-файлов.
    Однако я не решился удалить из стартового скрипта все упоминания об 
lock-файлах, так как не нашел документации или описаний, из которых 
можно было бы понять к чему это может привести. Если кто-нибудь 
подскажет, буду очень благодарен.
    Исправленная версия ez-ipupdate направлена в incoming.
   
    С уважением,
          Сергей Иванов.


      parent reply	other threads:[~2004-06-01 13:43 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-05-29 18:27 Yura Zotov
2004-05-31 18:43 ` [sisyphus] " Michael Shigorin
2004-05-31 19:57   ` Yura Zotov
2004-06-01 13:43 ` sergey ivanov [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=40BC87F9.10304@parkheights.dyndns.org \
    --to=seriv@parkheights.dyndns.org \
    --cc=sisyphus@altlinux.ru \
    /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 Sisyphus discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/sisyphus/0 sisyphus/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 sisyphus sisyphus/ http://lore.altlinux.org/sisyphus \
		sisyphus@altlinux.ru sisyphus@altlinux.org sisyphus@lists.altlinux.org sisyphus@lists.altlinux.ru sisyphus@lists.altlinux.com sisyphus@linuxteam.iplabs.ru sisyphus@list.linux-os.ru
	public-inbox-index sisyphus

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


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