Микротик. Динамический шейпер

[ronin_zp 2]

Доброй ночи.

Пытаюсь настроит динамический шейпер, но он не отрабатывает. С помощью tcpdump выловил такое
/queue/simple/set id=*1E2.=  max-limit=11000k/11000k.=burst-limit=k/11000k.=  burst-threshold=8800k/8800k.=burst-time=0/0
trap(=message=failure: no download-burst-time  done.

Чего ему не хватает?
 

Edited 2018-09-18 19:09:42 by ronin_zp

[l1ght 377]

2 часа назад, ronin_zp сказал:

Доброй ночи.

Пытаюсь настроит динамический шейпер, но он не отрабатывает. С помощью tcpdump выловил такое
/queue/simple/set id=*1E2.=  max-limit=11000k/11000k.=burst-limit=k/11000k.=  burst-threshold=8800k/8800k.=burst-time=0/0
trap(=message=failure: no download-burst-time  done.

Чего ему не хватает?
 

Цитата

message=failure: no download-burst-time 

Видимо не хватает поля download-burst-time

[ronin_zp 2]

То, что этого поля не хватает - понятно. 

Непонятно куда его прописывать. Используются simple queues. Без динамического шейпера всё работает нормально. Динамический шейпер настраивается по официальной инструкции в которой есть только то, как добавить в крон. На серверах с rscriptd динамический шейпер работает без проблем. Где и что нужно добавить в биллинге чтобы динамический шейпер заработал и на микротике? 

[l1ght 377]

по всей вероятности надо просить бак фиксы у тех кто пилил это к микротам

вроде это бобер делал

 

я конечно ткну пальцем в небо - покажите скрин скорости тарифов

 

вестимо собака где-то тут порылась

github

Edited 2018-09-19 00:02:16 by l1ght

[ronin_zp 2]

мдя, код довольно таки грязный

'burst-time' => $eachlogin['burstimetupload'] . '/' . $eachlogin['bursttimedownload']

'max-limit' => $eachlogin['speedup'] . 'k/' . $eachlogin['speed'] . 'k',

'burst-limit' => $eachlogin['burstupload'] . 'k/' . $eachlogin['speed'] . 'k',

'burst-threshold' => ($eachlogin['speedup'] * 0.8) . 'k/' . ($eachlogin['speed'] * 0.8) . 'k',

'burst-time' => $eachlogin['burstimetupload'] . '/' . $eachlogin['bursttimedownload']

 

[bobr-kun 7]

>> мдя, код довольно таки грязный

Во-первых - никто не запрещает сделать чище.

Во-вторых - код частично основан на коде другого человека.

В третьих - возможно завтра будет время на проверить...

Edited 2018-09-19 17:21:54 by bobr-kun

[ronin_zp 2]

38 минут назад, bobr-kun сказал:

Во-первых - никто не запрещает сделать чище.

ну я уже начал помогать обратив на это внимание

 

39 минут назад, bobr-kun сказал:

Во-вторых - код частично основан на коде другого человека.

 

ну это не повод мёрджить в мастер ветку без проверки
 

40 минут назад, bobr-kun сказал:

 В третьих - возможно завтра будет время на проверить...

Если нужна помощь - пишите. В пхп не силён - но вычитать код, протестировать и подебажить - всегда пожалуйста.

[bobr-kun 7]

>> ну это не повод мёрджить в мастер ветку без проверки

Это с чего это оно вдруг стало без тестирования мерджится? С чего вы взяли? Аж неделю работало на RouterOS 6.41. Потом было отключено за ненадобностью, но вот в ближайшем будущем планируется "полномасштабное использование". Так что завтра постараюсь выкроить время перепроверить на 6.43.

[ronin_zp 2]

9 минут назад, bobr-kun сказал:

Аж неделю работало на RouterOS 6.41. Потом было отключено за ненадобностью, но вот в ближайшем будущем планируется "полномасштабное использование". Так что завтра постараюсь выкроить время перепроверить на 6.43.

не заработало на 6.40.4 и 6.42.6
 

 

9 минут назад, bobr-kun сказал:

Это с чего это оно вдруг стало без тестирования мерджится? С чего вы взяли?

  как минимум из за этого, поправьте если я ошибаюсь

'max-limit' => $eachlogin['speedup'] . 'k/' . $eachlogin['speed'] . 'k',

'burst-limit' => $eachlogin['burstupload'] . 'k/' . $eachlogin['speed'] . 'k',

'burst-threshold' => ($eachlogin['speedup'] * 0.8) . 'k/' . ($eachlogin['speed'] * 0.8) . 'k',

'burst-time' => $eachlogin['burstimetupload'] . '/' . $eachlogin['bursttimedownload']

первые три строки для download одна и та же переменная, хотя в правилах динамического шейпера скорость для даунлоад тоже меняется
и в последней строке явная опечатка - burstimetupload против bursttimedownload

 

 

P.S.: кстати, как оно себя поведёт при выключенном режиме BURST в alter.ini ?

 

Edited 2018-09-19 19:50:52 by ronin_zp

[l1ght 377]

8 минут назад, ronin_zp сказал:

не заработало на 6.40.4 и 6.42.6
 

 

  как минимум из за этого, поправьте если я ошибаюсь

'max-limit' => $eachlogin['speedup'] . 'k/' . $eachlogin['speed'] . 'k',

'burst-limit' => $eachlogin['burstupload'] . 'k/' . $eachlogin['speed'] . 'k',

'burst-threshold' => ($eachlogin['speedup'] * 0.8) . 'k/' . ($eachlogin['speed'] * 0.8) . 'k',

'burst-time' => $eachlogin['burstimetupload'] . '/' . $eachlogin['bursttimedownload']

первые три строки для download одна и та же переменная, хотя в правилах динамического шейпера скорость для даунлоад тоже меняется
и в последней строке явная опечатка - burstimetupload против bursttimedownload

 

 

P.S.: кстати, как оно себя поведёт при выключенном режиме BURST в alter.ini ?

 

я скриншота так и не увидел

[ronin_zp 2]

Скриншот тарифа из какого раздела?

 

[ronin_zp 2]

 

[l1ght 377]

 

Я просил скрин скорости тарифов

Должны быть заполнены соответствующие поля

[ronin_zp 2]

Ну так первый скриншот. Первые две колонки. Уж простите - но скриншотить всю тарифную сетку я не стал, по вполне понятным причинам.

[l1ght 377]

Я не просто так скриншот просил. Просто ваш так криво сделан, что я не сразу понял что это что я просил.

И раз вы уже код начали читать то почитайте внимательно где вы провтыкали.

Edited 2018-09-20 02:28:01 by l1ght

[ronin_zp 2]

вставлял я туда другие значения, ошибка остаётся по причине burstimetupload
локально у себя я поправить могу, но со следующим обновлением опять грабли вылезут
ладно, будет время перепишу код с исправлением и с учётом отключенной опции burst и закоммичу в репозиторий 

[nightfly 1,252]

3 часа назад, ronin_zp сказал:

ладно, будет время перепишу код с исправлением и с учётом отключенной опции burst и закоммичу в репозиторий 

с этим к Бобру - он теперь занимается микротиками и прочим порно

[bobr-kun 7]

18 часов назад, ronin_zp сказал:

и в последней строке явная опечатка - burstimetupload против bursttimedownload

нет, это не опечатка - это так исторически сложилось имя поля в БД

 

18 часов назад, ronin_zp сказал:

P.S.: кстати, как оно себя поведёт при выключенном режиме BURST в alter.ini ?

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

[bobr-kun 7]

Итак: похоже, в те несколько раз, когда мне приходилось складировать еще не готовый код "на полку", чтобы быстро сделать мелкий фикс, я где-то прое...ошибся и вытащил из полки не совсем свежий код микротикового динамик шейпера, ибо вот этот кусок

log_register('MT_DN_SHAPER ' . $Action . ' done to `' . $UsersCnt . '` users');
die('OK:MT_DN_SHAPER');

я точно "выносил из froeach" - хорошо помню этот момент, когда фича работала только для одного абона, но исправно рапортовала, что для всех =). А в остальном все таки работает.

Вот с burst'ами - да, если их в тарифе не было предусмотрено - все разваливалось. Но эмпирическим путем я выяснил, что критичным для микрота является именно параметр burst-time - если он не установлен(пустой) - гембель. А вот к скоростям burst'ов таких жестких требований нет. Так что теперь мы учитываем это вот все и, если хотя бы один из четырех параметров burst'ов пуст - делаем изменения только для скорости download.

 

Теперь пойду еще потестирую немного и буду опять корячится, чтобы собрать пуллреквест...

[sanyadnepr 305]

18 часов назад, l1ght сказал:

 

Я просил скрин скорости тарифов

Должны быть заполнены соответствующие поля

В демке почему "burst полей" как на скрине нет и создать не получается ?

[l1ght 377]

5 часов назад, sanyadnepr сказал:

В демке почему "burst полей" как на скрине нет и создать не получается ?

бурсты по дефолту отключены

[sanyadnepr 305]

.

Edited 2018-09-21 06:18:19 by sanyadnepr