Skip to content

Init

Martin Petráček requested to merge init into master

Znovu a lépe...

Pár věcí k zamyšlení (nejlíp hned, než to deployneme uživatelům - bude se to zpětně těžko měnit):

  • zvažujeme (někdy) nějaké další listy než ten 1 co má blokovat pokusy o připojení z wanu do lanu? Zatím je to navržené jen pro ten jeden list. Dalo by se to předělat, aby program mohl udržovat stav více listů - třeba s tím že uživatel bude chtít jen nějakou podmnožinu. V tom případě by bylo potřeba víc typů zpráv (třeba dynfw/list1/list a dynfw/list1/list místo současného dynfw/list dynfw/delta). To bysme možná měli rozhodnout než to vydáme.
  • otázka je jestli bychom delty nechtěly posílat po víc ip najednou - třeba {"deltas": ["+1.2.3.4", "-4.3.2.1"]} nebo něco tak namísto současného {"delta": "positive", "ip": "1.2.3.4"} a {"delta": "negative", "ip": "4.3.2.1"}. Možná je to jen můj pocit Cčkaře, zvyklého šetřit bajty, ale ten současný formát je imo dost neúsporný. Také změny do ipsetu by bylo lepší commitovat najednou (spouštění externího procesu) - to jde zajistit na straně tohoto programu, ale posílání víc změn najednou ze serveru by mělo větší výhody. Praxe stejně ukazuje, že změny chodí v dávkách za sebou. Třeba by to mohlo být tak, že publisher si bude bufferovat změny po X sekund a pak je pošle najednou. Ten formát s polem by měl víc flexibility: dá se tam posílat 1 i víc; v tom současném pouze 1 - tj. ze začátku by se klidně mohlo posílat pole s jednou položkou, aby to nevyžadovalo velké překopání publisheru. Ten formát bychom měli rozhodnout než to vydáme, zpětně se to bude špatně měnit.
Edited by Martin Petráček

Merge request reports