[sisyphus] I: spamassassin 3.4.0

[sisyphus] I: spamassassin 3.4.0

[Sergey]

Приветствую.

Что-то давно у нас не обновлялся Spamassassin...
Обновление в задании #121727, кому интересно, проверяйте
(apt-repo add 121727). Список изменений, пока, такой:

- NMU: 3.4.0 (ALT #30063)
- fixed stop() and reload() functions in init.d/spamd (ALT #28789)
- added lsb init header for init.d/spamd
- set user "mail" for spamd's child processes (fixed write
  the /var/spool/spamassassin/bayes_journal)
- removed /var/run/spamd directory (/var/run used for pid file)
- changed permisson for sa-update-keys directory to 700

-- 
С уважением, Сергей
a_s_y@sama.ru

Re: [sisyphus] I: spamassassin 3.4.0

[Dmitry V. Levin]

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

On Fri, Jun 20, 2014 at 12:21:17PM +0400, Sergey wrote:
> Приветствую.
> 
> Что-то давно у нас не обновлялся Spamassassin...
> Обновление в задании #121727, кому интересно, проверяйте

warning [i586]: perl-Mail-SpamAssassin=3.4.0-alt1: circular dependencies on spamassassin-rules=3.4.0-alt1
warning [x86_64]: perl-Mail-SpamAssassin=3.4.0-alt1: circular dependencies on spamassassin-rules=3.4.0-alt1

> (apt-repo add 121727). Список изменений, пока, такой:
> 
> - NMU: 3.4.0 (ALT #30063)
> - fixed stop() and reload() functions in init.d/spamd (ALT #28789)

Fixed?

> - added lsb init header for init.d/spamd
> - set user "mail" for spamd's child processes (fixed write
>   the /var/spool/spamassassin/bayes_journal)

"mail" вместо "spamd" для spamd?

> - removed /var/run/spamd directory (/var/run used for pid file)
> - changed permisson for sa-update-keys directory to 700


-- 
ldv

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

Re: [sisyphus] I: spamassassin 3.4.0

[Sergey]

On Friday 20 June 2014, Dmitry V. Levin wrote:

> > Что-то давно у нас не обновлялся Spamassassin...
> > Обновление в задании #121727, кому интересно, проверяйте
> 
> warning [i586]: perl-Mail-SpamAssassin=3.4.0-alt1: circular dependencies on spamassassin-rules=3.4.0-alt1
> warning [x86_64]: perl-Mail-SpamAssassin=3.4.0-alt1: circular dependencies on spamassassin-rules=3.4.0-alt1

Это место я не трогал пока, так было, я не знаю смысл
этой зависимости. Можно этот вопрос решить в рамках 
http://bugzilla.altlinux.org/26682 позже.

> > (apt-repo add 121727). Список изменений, пока, такой:
> > 
> > - NMU: 3.4.0 (ALT #30063)
> > - fixed stop() and reload() functions in init.d/spamd (ALT #28789)
> 
> Fixed?

Ну так работает же. Или где-то не работает ? Можно "rewrote",
или "temporary rewrote", но я не знаю срок этого temporary...

> > - set user "mail" for spamd's child processes (fixed write
> >   the /var/spool/spamassassin/bayes_journal)
> 
> "mail" вместо "spamd" для spamd?

Почему-то /var/spool/spamassassin пакуется с %attr(0775,root,mail),
это место я не менял, пользователя просто подогнал под него. На
самом деле, я сейчас склоняюсь к мысли вернуть root, так как этот
bayes_journal растёт в бесконечность, как выяснилось, если не чистить
руками. Надо на крон задание вешать. Вот думаю, что лучше root вернуть,
но в скрипте останется переменная CHILDUSER, которую можно будет 
переопределить в /etc/sysconfig/spamd

-- 
С уважением, Сергей
a_s_y@sama.ru

Re: [sisyphus] I: spamassassin 3.4.0

[Sergey]

On Friday 20 June 2014, Sergey wrote:

> Почему-то /var/spool/spamassassin пакуется с %attr(0775,root,mail),
> это место я не менял, пользователя просто подогнал под него. На
> самом деле, я сейчас склоняюсь к мысли вернуть root, так как этот
> bayes_journal растёт в бесконечность, как выяснилось, если не чистить
> руками.

Вернул старое поведение в плане bayes_journal, теперь так:

- NMU: 3.4.0 (ALT #30063)
- rewrote stop() and reload() functions in init.d/spamd (ALT #28789)
- added lsb init header for init.d/spamd
- removed /var/run/spamd directory (/var/run used for pid file)
- changed permisson for sa-update-keys directory to 700
- added and described CHILDUSER variable in /etc/sysconfig/spamd

В принципе, "-u mail" можно было и раньше добавить в /etc/sysconfig/spamd
в SPAMDOPTIONS. Теперь ситуация с bayes_journal там просто дополнтельно
описана.

-- 
С уважением, Сергей
a_s_y@sama.ru

Re: [sisyphus] I: spamassassin 3.4.0

[Dmitry V. Levin]

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

On Fri, Jun 20, 2014 at 02:58:33PM +0400, Sergey wrote:
> On Friday 20 June 2014, Dmitry V. Levin wrote:
> 
> > > Что-то давно у нас не обновлялся Spamassassin...
> > > Обновление в задании #121727, кому интересно, проверяйте
> > 
> > warning [i586]: perl-Mail-SpamAssassin=3.4.0-alt1: circular dependencies on spamassassin-rules=3.4.0-alt1
> > warning [x86_64]: perl-Mail-SpamAssassin=3.4.0-alt1: circular dependencies on spamassassin-rules=3.4.0-alt1
> 
> Это место я не трогал пока, так было, я не знаю смысл
> этой зависимости. Можно этот вопрос решить в рамках 
> http://bugzilla.altlinux.org/26682 позже.
> 
> > > (apt-repo add 121727). Список изменений, пока, такой:
> > > 
> > > - NMU: 3.4.0 (ALT #30063)
> > > - fixed stop() and reload() functions in init.d/spamd (ALT #28789)
> > 
> > Fixed?
> 
> Ну так работает же. Или где-то не работает ? Можно "rewrote",
> или "temporary rewrote", но я не знаю срок этого temporary...

В скрипте есть и другие функции.
Если их надо фиксить, то, наверное, сразу все 4, а не половину.

> > > - set user "mail" for spamd's child processes (fixed write
> > >   the /var/spool/spamassassin/bayes_journal)
> > 
> > "mail" вместо "spamd" для spamd?
> 
> Почему-то /var/spool/spamassassin пакуется с %attr(0775,root,mail),
> это место я не менял, пользователя просто подогнал под него. На
> самом деле, я сейчас склоняюсь к мысли вернуть root, так как этот
> bayes_journal растёт в бесконечность, как выяснилось, если не чистить
> руками. Надо на крон задание вешать. Вот думаю, что лучше root вернуть,
> но в скрипте останется переменная CHILDUSER, которую можно будет 
> переопределить в /etc/sysconfig/spamd

Раньше дочерние процессы работали под пользователем spamd.
Что изменилось, почему сейчас речь идет о root и mail?


-- 
ldv

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

Re: [sisyphus] I: spamassassin 3.4.0

[Sergey]

On Friday 20 June 2014, Dmitry V. Levin wrote:

> > Ну так работает же. Или где-то не работает ? Можно "rewrote",
> > или "temporary rewrote", но я не знаю срок этого temporary...
> 
> В скрипте есть и другие функции.
> Если их надо фиксить, то, наверное, сразу все 4, а не половину.

Этих только две. В start() start_daemon() работает нормально, stop()
и reload() переписаны, в restart() вызываются stop() и start().
А вот про пятую, действительно, забыл. Надо ещё status() заменить.

>> Почему-то /var/spool/spamassassin пакуется с %attr(0775,root,mail),
>> это место я не менял,

> Раньше дочерние процессы работали под пользователем spamd.

Нет, или это было очень давно.

> Что изменилось, почему сейчас речь идет о root и mail ?

RUNAS=root

и

start()
{
        start_daemon --pidfile "$PIDFILE" --lockfile "$LOCKFILE" --expect-user "$RUNAS" -- \
                        spamd -d --pidfile=$PIDFILE $CHILDUSEROPT $SPAMDOPTIONS
        RETVAL=$?
        return $RETVAL
}

остались без изменений (кроме добавленного $CHILDUSEROPT в последней сборке).

То есть, запуск шёл от root. Это вот старый spamd, запущенный с конфигом по-умолчанию:

# ps aux|grep spam
root      6192 34.5  1.7  41080 37432 ?        Ss   23:19   0:03 /usr/bin/spamd -d --pidfile=/var/run/spamd.pid
root      6193  0.0  1.6  41080 35424 ?        S    23:19   0:00 spamd child
root      6194  0.0  1.6  41080 35424 ?        S    23:19   0:00 spamd child

При этом, в логе можно было видеть такие строчки:

spamd[578745]: spamd: still running as root: user not specified with -u, not found, or set to root, falling back to nobody
spamd[578745]: bayes: cannot write to /var/spool/spamassassin/bayes_journal, bayes db update ignored: Permission denied

-- 
С уважением, Сергей
a_s_y@sama.ru

Re: [sisyphus] I: spamassassin 3.4.0

[Sergey]

On Friday 20 June 2014, Sergey wrote:

> - rewrote stop() and reload() functions in init.d/spamd (ALT #28789)
 
Очередная итерация. init.d/spamd переписан с учётом того, что
start-stop-daemon в --exec нормально понимает симлинк на бинарник,
на который указывает /proc/<pid>/exe

-- 
С уважением, Сергей
a_s_y@sama.ru

Re: [sisyphus] I: spamassassin 3.4.0

[Sergey]

On Friday 20 June 2014, Sergey wrote:

> > warning [i586]: perl-Mail-SpamAssassin=3.4.0-alt1: circular dependencies on spamassassin-rules=3.4.0-alt1
> > warning [x86_64]: perl-Mail-SpamAssassin=3.4.0-alt1: circular dependencies on spamassassin-rules=3.4.0-alt1
> 
> Это место я не трогал пока, так было, я не знаю смысл
> этой зависимости. Можно этот вопрос решить в рамках 
> http://bugzilla.altlinux.org/26682 позже.
 
"Позже" наступает. Если вчитаться в ветку про sa-update
(https://lists.altlinux.org/pipermail/sisyphus/2010-January/344869.html),
возникает предположение, что эта зависимость нужна. Альтернатива - вернуть
сборку spamassassin-rules в основной пакет, но это лишит возможности
обновлять spamassassin-rules отдельно. С одной стороны, это не очень
хорошо, но, с другой, большой вопрос, будет ли кто-то когда-то обновлять
пакет spamassassin-rules отдельно. При обновлении до 3.4.1, видимо, опять
оставлю это место, как есть.

-- 
С уважением, Сергей.