Bug 20142

Summary: Не управляет запущенностью сервера
Product: Sisyphus Reporter: Anton V. Boyarshinov <boyarsh>
Component: alterator-squidAssignee: Anton V. Boyarshinov <boyarsh>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: major    
Priority: P3 CC: aen, boyarsh, cas, manowar, vitty
Version: unstable   
Hardware: all   
OS: Linux   
Bug Depends on:    
Bug Blocks: 19564    

Description Anton V. Boyarshinov 2009-05-22 19:19:01 MSD
Не управляет запущенностью сервера и даже не имеет ссылки на соотествтующий интерфейс
Comment 1 AEN 2009-05-25 03:27:28 MSD
Долго думал над фразой "Не управляет запущенностью сервера". Имеется в виду, что нет управления запуском/установкой сервера squid?
Comment 2 Anton V. Boyarshinov 2009-05-25 13:35:00 MSD
(В ответ на комментарий №1)
> Долго думал над фразой "Не управляет запущенностью сервера". Имеется в виду,
> что нет управления запуском/установкой сервера squid?

Да. Но, разумеется, запуском, а не установкой
Comment 3 manowar@altlinux.org 2009-05-25 16:07:27 MSD
Я говорил со Стасом на эту тему, ещё когда начинал писать модуль. Он мне ответил тогда, что не видит смысла в разделении автоматического и ручного запуска, т.е. chkconfig и serivce, потому что "работающий, но вынесенный из конфигурации сервер никому не нужен". Поэтому запуском и остановкой управляет тот же переключатель, что и автозапуском. Он должен быть на страничке "Обновить конфигурацию".
Comment 4 Vitaly Kuznetsov 2009-05-25 16:27:24 MSD
ну, т.е. текущая проблема в том, что в пятничном сервере не работает прокси (пока руками не запустишь сервис)
Comment 5 Anton V. Boyarshinov 2009-05-25 16:40:00 MSD
(В ответ на комментарий №3)
>  Он должен быть на страничке "Обновить
> конфигурацию".
Это интерфейсный ужас и неконсистентность.. Для пользователя совершенно неочевидно, что после кнопки "применить" надо нажимать ещё что-то..
Comment 6 manowar@altlinux.org 2009-05-27 14:15:08 MSD
> ну, т.е. текущая проблема в том, что в пятничном сервере не работает прокси
> (пока руками не запустишь сервис)

По какому признаку/событию он должен стартовать?
Comment 7 manowar@altlinux.org 2009-05-27 14:18:37 MSD
> Это интерфейсный ужас и неконсистентность.. Для пользователя совершенно
> неочевидно, что после кнопки "применить" надо нажимать ещё что-то..

Предлагаешь обновлять всю конфигурацию при изменении отдельных параметров в любом из разделов? И сервер каждый раз перезапускать?
Comment 8 Andrey Cherepanov 2009-05-27 15:38:54 MSD
(В ответ на комментарий №6)
> > ну, т.е. текущая проблема в том, что в пятничном сервере не работает прокси
> > (пока руками не запустишь сервис)
> 
> По какому признаку/событию он должен стартовать?
Как настройка OpenVPN-сервера: вверху указывается чекбокс 

  [x] Включить службу OpenVPN

Лучше сделать по аналогии. Включается по кнопку "Применить".
Comment 9 Andrey Cherepanov 2009-05-27 15:39:47 MSD
(В ответ на комментарий №7)
> > Это интерфейсный ужас и неконсистентность.. Для пользователя совершенно
> > неочевидно, что после кнопки "применить" надо нажимать ещё что-то..
> 
> Предлагаешь обновлять всю конфигурацию при изменении отдельных параметров в
> любом из разделов? И сервер каждый раз перезапускать?
Нет, только при нажатии на кнопку "Применить" на главном экране.
Comment 10 Andrey Cherepanov 2009-05-27 15:58:23 MSD
Кстати, при нажатии на "Обновить конфигурацию" - [x] Включить сервис прокси-сервера

squid не запускается.
Comment 11 manowar@altlinux.org 2009-06-02 16:14:27 MSD
Как я понял, проблема была в неправильном конфиге, т.е. после обновления конфигурационного файла squid не запускался и вручную тоже. Т.е. эта ошибка не существует сама по себе, а является следствием других ошибок.
Comment 12 Anton V. Boyarshinov 2009-06-02 18:33:12 MSD
(В ответ на комментарий №11)
> Как я понял, проблема была в неправильном конфиге, т.е. после обновления
> конфигурационного файла squid не запускался и вручную тоже. Т.е. эта ошибка не
> существует сама по себе, а является следствием других ошибок.
Это относится только к коментарию #10, но не к предыдущему.
Comment 13 manowar@altlinux.org 2009-06-03 13:19:07 MSD
Могу предложить следующее: нажатие на кнопку "Обновить конфигурацию" будет приводить к неявному нажатию на "Применить", т.е. перед обновлением файлов все текущие изменения параметров формы будут включены в текущую конфигурацию. Согласен с тем, что интуитивно пользователь хочет настроить сервер в соответствие с теми параметрами, которые он в данный момент видит.

Не согласен с тем, что эта проблема обсуждается под заголовком "Не управляет запущенностью сервера". По мне так 1 проблема -- 1 багрепорт. Было бы здорово завести на эту тему отдельный, а этот закрыть.
Comment 14 Anton V. Boyarshinov 2009-06-03 15:11:42 MSD
(В ответ на комментарий №13)
> Могу предложить следующее: нажатие на кнопку "Обновить конфигурацию" будет
> приводить к неявному нажатию на "Применить", т.е. перед обновлением файлов все
> текущие изменения параметров формы будут включены в текущую конфигурацию.
> Согласен с тем, что интуитивно пользователь хочет настроить сервер в
> соответствие с теми параметрами, которые он в данный момент видит.

и называться эта кнопка "Обновить конфигурацию" должна "применить". Двух кнопок быть не должно: это внутренная жизнь соответсвтующего модуля alterator.
Comment 15 manowar@altlinux.org 2009-06-05 18:24:26 MSD
Тогда у меня возникает такой вопрос: при переходе на подчинённые страницы ("Порты", "Сети" и "Группы") изменения на главной странице должны запоминаться автоматом или по кнопке "Применить"? Если по "Применить", то что заставит пользователя нажать на эту кнопку второй раз, после того, как он выполнит настройку "Портов" или "Сетей"? Да он может и вообще не вернуться на главную страницу, посчитав, что достаточно нажать применить на той, где он что-то менял. И он будет прав. Понимаешь какое дело: запоминание новых значений параметров -- это одна задача, а воплощение _всей_ полученной конфигурации "в жизнь" -- это другая задача. Поэтому нужны две кнопки.
Comment 16 Vitaly Kuznetsov 2009-06-05 18:35:50 MSD
Кнопка должна быть одна - "Применить", применяющая _все_ изменения, на любых страницах.
Comment 17 manowar@altlinux.org 2009-06-06 01:14:04 MSD
На которую администратор будет нажимать в самую последнюю очередь, так? Т.е. переходя с главной страницы на второстепенную он на неё не нажмёт, потому что не все запланированные им изменения конфигурации ещё внесены. А внеся изменения во второстепенных разделах он уже не вернётся на главную, чтобы там торжественно нажать на применить, так, и конфигурационные файлы не будут обновлены.

Понимаешь, единственный надёжный способ скрыть от пользователя тот факт, что у нас имеется две конфигурации (одна в альтераторе, другая в конфиге), которые необходимо синхронизировать -- это производить синхронизацию после внесения каждого изменения в конфигурацию. Учитывая что каждая синхронизация должна сопровождаться перезагрузкой сервера, это будет крайне неэффективно. Например тебе нужно открыть 10 новых портов. Ты что, будешь перезагружать сервер после добавления каждого порта? Конечно ты захочешь спокойно отредактировать список портов, а затем уже "применить" новую конфигураци. И так со всей конфигурацией в целом. Так что новая кнопка нужна -- она является отражением кардинального отличия данного модуля от всех прочих. Думаю, что попытка скрыть это отличие принесёт нам много дополнительных хлопот.
Comment 18 Vitaly Kuznetsov 2009-06-07 00:57:05 MSD
Мы делаем продукт для тех, кто вообще не понимает что такое "конфигурация" и что значит её "обновить". Поэтому я считаю гораздо меньшим злом reload squid-а по первому чиху чем какие-то не очень понятные кнопки.
Вообще, идеальная схема работы должна выглядеть так: захожу в конфигуратор, правлю какие хочу значения (в этот момент ничего не происходит), нажимаю "Применить" - и тут, вуаля, все новые настройки вступают в строй. Вне зависимости от разных/одинаковых страниц и прочей внутри-альтераторовской кухни.
Comment 19 manowar@altlinux.org 2009-06-15 12:15:03 MSD
Т.е. ты предлагаешь оставить кнопку "Применить" только на первой странице? Или что?
Comment 20 Vitaly Kuznetsov 2009-06-15 13:51:44 MSD
(В ответ на комментарий №19)
> Т.е. ты предлагаешь оставить кнопку "Применить" только на первой странице? Или
> что?

Я предлагаю убрать кнопку "Обновить конфигурацию". В кнопках "Применить" на каждой странице я ничего плохого не вижу.
Comment 21 Anton V. Boyarshinov 2009-06-19 12:12:27 MSD
(В ответ на комментарий №20)

> Я предлагаю убрать кнопку "Обновить конфигурацию". В кнопках "Применить" на
> каждой странице я ничего плохого не вижу.
Предлагаю заменить в этом тексте слово "предлагаю" на слово "требую" ;))
Comment 22 manowar@altlinux.org 2009-07-02 15:13:40 MSD
Теперь конфигурация обновляется налету.