Size: a a a

2021 March 30

ST

Serhii Tomak in Accel-PPP
а что было?
источник

ST

Serhii Tomak in Accel-PPP
ща регулярку скину
источник

D

Dimka88 in Accel-PPP
Если регулярки составлены не верно, т.е. без exclude то поведение ожидаемое
источник

ST

Serhii Tomak in Accel-PPP
пповерю
источник

VG

Vladislav Grishenko in Accel-PPP
xxxx yyyy
и вот в таком виде становится очень плохо, ведь если лочить каждый отдельный get_opt, мы не можем гарантировать актуальность конфига
я об этом и говорю, достаточно сериализовать эвент релоада.
с гарантией модулю что в процессе выполнения калбэка в конфиге ничего не изменится, это не сложно. и не требует изменений каждого модуля
источник

VG

Vladislav Grishenko in Accel-PPP
xxxx yyyy
который:
1. работает
2. позволяет обновлять все модулям конфиг одновременно, а не по очереди
это брак
источник

VG

Vladislav Grishenko in Accel-PPP
так никто не делает, потому что совершенно не поддерживаемое получается, и будет выстреливать в ноги в будущем
источник

VG

Vladislav Grishenko in Accel-PPP
давай лучше эти же усилия вложим в более управляемую архитектуру? ну круто же будет, не иметь 100500 затычек, а знать что не протечет, скажем так. by design
источник

VG

Vladislav Grishenko in Accel-PPP
относительно задержек на релоад конфига - это действительно не важно irl, пока не импактит датаплейн.
в случае accel-ppp из плохого может случиться - так только что какая-то сессия создастся со все еще старыми настройками, все остальное работает в ядре.
или клиент не получит ответ на retryable connection attempt
источник

Е

Евгений Вайд... in Accel-PPP
Если будете релоад чинить, то реально ли сделать изменения всего конфига доступным через релоад?
источник

DK

Dmitry Koval in Accel-PPP
Евгений Вайд
Если будете релоад чинить, то реально ли сделать изменения всего конфига доступным через релоад?
+++
источник

VG

Vladislav Grishenko in Accel-PPP
не в первую очередь. сначала нужно починить существующее, фичи - после
источник

DK

Dmitry Koval in Accel-PPP
многие вещи удобно было бы менять через cli
источник

VG

Vladislav Grishenko in Accel-PPP
и лучше case-by-case, не любой релоад применим в рантайме для существующих сессий
источник

DK

Dmitry Koval in Accel-PPP
тот же loglevel, или вланы слушаемые для ipoe
источник

VG

Vladislav Grishenko in Accel-PPP
loglevel кажется и так меняется, нет?
источник

DK

Dmitry Koval in Accel-PPP
нет, только для модулей verbose 0/1
источник

VG

Vladislav Grishenko in Accel-PPP
ок. в ближайших планах такого пока нет
источник

xy

xxxx yyyy in Accel-PPP
Vladislav Grishenko
так никто не делает, потому что совершенно не поддерживаемое получается, и будет выстреливать в ноги в будущем
хорошо, давай тогда попробуем так:
я сделаю глобальный лок, который будет указывать на то, что конфиг консистентен, и каждый модуль будет сам решать, зависеть от него, или нет (вот только по идее надо будет зависеть всем, иначе стреляем в ногу).
а также предусмотрю костыли, чтобы уже внутри функций конфига доступ всегда был сериализован, и небыло неразберихи.
т.е. то, что в модуле где-то останутся старые настройки - проблема не шерифа, а индейцев, значит надо было читать опции из конфига правильно.
источник

VG

Vladislav Grishenko in Accel-PPP
xxxx yyyy
хорошо, давай тогда попробуем так:
я сделаю глобальный лок, который будет указывать на то, что конфиг консистентен, и каждый модуль будет сам решать, зависеть от него, или нет (вот только по идее надо будет зависеть всем, иначе стреляем в ногу).
а также предусмотрю костыли, чтобы уже внутри функций конфига доступ всегда был сериализован, и небыло неразберихи.
т.е. то, что в модуле где-то останутся старые настройки - проблема не шерифа, а индейцев, значит надо было читать опции из конфига правильно.
зачем? модуль вообще не должно волновать. ему дали - он взял
источник