Opened 4 months ago

Closed 4 months ago

Last modified 4 months ago

#656 closed улучшение (не будем делать)

Добавить в веб-интерфейс настройку стратума для ntp сервера 127.127.1.0

Reported by: san Owned by: alx
Priority: средний Milestone: 1 очередь
Component: sw Keywords:
Cc:

Description

Мотивация была описана #654

Change History (12)

in reply to:  description comment:1 by alx, 4 months ago

Replying to san:

Мотивация была описана #654

В тикете #654 ничего не говорится о веб-интерфейсе.

comment:2 by san, 4 months ago

Да, поэтому и создал этот тикет.
В тикете #654 описано зачем пользователю нужна эта настройка. А раз она нужна пользователю, то логично её вывести в веб-интерфейс.

in reply to:  2 comment:3 by alx, 4 months ago

Resolution: не будем делать
Status: newclosed

Replying to san:

Да, поэтому и создал этот тикет.

В описании тикета написано, что мотивация предложения этого тикета описана в тикеате #654. Но это не так - в тикете #654 этой мотивации нет.

В тикете #654 описано зачем пользователю нужна эта настройка.

И эта настройка есть, и даже уже прописана в конфиг-файле по умолчанию. Пользователю достаточно изменить одну цифру - и он получит желаемое.

А раз она нужна пользователю, то логично её вывести в веб-интерфейс.

Не вижу в этом логики. У ntpd нет веб-интерфейса, он настраивается через текстовый конфиг.

Возможно ты забыл, но мы уже подобный вопрос обсуждали на совещании с участием директора. По результату совещания мы пришли к решению, что подобной ерундой заниматься не будем. Не вижу смысла опять возвращаться к этому вопросу.

comment:4 by san, 4 months ago

Нет я не забыл, но у директора мы обсуждали то, что не будем добавлять в нашу конфигурацию "чужие" настройки.
А этот тикет о добавлении возможности настройки этого параметра пользователем через веб-интерфейс платы SW-01(аналогично тому как пользователь может удалять и добавлять ntp-сервера).

Last edited 4 months ago by san (previous) (diff)

in reply to:  4 comment:5 by alx, 4 months ago

Replying to san:

Нет я не забыл, но у директора мы обсуждали то, что не будем добавлять в нашу конфигурацию "чужие" настройки.

Так это же ровно то, что ты и предложил - добавить в веб-интерфейс swd настройку от ntpd. Для нас (swd) это чужая настройка - swd синхронизацией времени не занимается, это работа ntpd.

А этот тикет о добавлении возможности настройки этого параметра пользователем через веб-интерфейс платы SW-01(аналогично тому как пользователь может удалять и добавлять ntp-сервера).

Не вижу здесь подобия - в тикете же речь идет не о сервере, а о встроенных аппаратных часах...

comment:6 by san, 4 months ago

Не вижу здесь подобия - в тикете же речь идет не о сервере, а о встроенных аппаратных часах...

А настройка серверов это не чужая настройка?

in reply to:  6 ; comment:7 by alx, 4 months ago

Replying to san:

Не вижу здесь подобия - в тикете же речь идет не о сервере, а о встроенных аппаратных часах...

А настройка серверов это не чужая настройка?

Как посмотреть. :) Для добавления/удаления серверов в swd реализован ntpd клиент. Формально это его настройки: например при старте swd этот клиент читает из конфига список серверов и передает запросы добавления серверов ntpd.

comment:8 by san, 4 months ago

Если список серверов не считать "чужой" настройкой, то почему тогда мы не храним его в конфиге, как другие свои настройки?

in reply to:  7 comment:9 by alx, 4 months ago

Replying to alx:

например при старте swd этот клиент читает из конфига список серверов и передает запросы добавления серверов ntpd.

Прошу прощения, я ошибся. Не происходит этого при старте swd (я посмотрел код).

in reply to:  8 comment:10 by alx, 4 months ago

Replying to san:

Если список серверов не считать "чужой" настройкой, то почему тогда мы не храним его в конфиге, как другие свои настройки?

По моему недосмотру. :( К сожалению, когда я реализовывал этот функционал, было принято неудачное решение. Так как это было очень давно, я сейчас, конечно, не помню ход моих мыслей. На сегодняшний день решение было бы другим. Я поэтому и ошибся в прошлом комментарии - не сомневался в том, что сделано именно так, пока не посмотрел код...

comment:11 by san, 4 months ago

"чужие" настройки.

Хм.. а нет ли какой-то возможности настройку стратума встроенных часов сделать "своей" настройкой?
(как, например, получилось с настройкой переменных sysContact, sysName, sysLocation)

in reply to:  11 comment:12 by alx, 4 months ago

Replying to san:

а нет ли какой-то возможности настройку стратума встроенных часов сделать "своей" настройкой?
(как, например, получилось с настройкой переменных sysContact, sysName, sysLocation)

Мне о такой возможности неизвестно. Но если ты или кто-то другое знает, как это сделать - присылайте патчи, я рассмотрю...

Note: See TracTickets for help on using tickets.