Task #866
closedKonfigurovatelnost hawata
0%
Description
Mek:
prave instaluju hawata na ostry mentat a narazil jsem na problem, ktery bych od tebe potreboval vyresit. Ve svem kodu pouzivas natvrdo nastavene cesty ke konfiguracnim a log souborum a dalsim nastavenim. Chtelo by to, aby ses uz ted podival, jake moznosti ti catalyst nabizi z hlediska nacitani konfiguraci a abys to co nejdrive zapracoval a pak vsude pouzival. Konkretne ted mluvim treba o tride Hawat::Model::Alert, kde mas natvrdo nastavenou cestu k log souboru a k filtracni gramatice. Hned na prvni zagoogleni jsem nasel treba toto:
https://metacpan.org/module/Catalyst::Manual::Cookbook#Configure-your-application
https://metacpan.org/module/Catalyst::Plugin::ConfigLoaderNebo muzes alternativne pouzivat moji tridu Config::Parser, ale skoro bych se radsi priklanel k necemu z Catalystu, at je to jednotne (myslim z hlediska hawata).
Zatim tedy hawata na ostrem serveru nerozjedu a pockam, az budes mit cas a toto vyresis. Ja to v celem mentatovi delam tak, ze vsechno co jde nacitam z konfiguracnich souboru a obsah techto souboru generuji ve fazi buildeni balicku. Pokud buildim deb balicek, tak se mi tam doplnuji jine cesty a dalsi nastaveni, nez v pripade buildeni treba tar balicku.
Updated by Radomír Orkáč over 11 years ago
- Status changed from New to In Progress
commit e75fc6370f83b5bae2654e7ba2b8e4a87d8d3805
Konfiguracni soubor je v: www/hawat/hawat.conf
<Hawat> log_file = /var/log/mentat/catalyst.log devel = 0 <Model> <Alert> devel = 1 filter = /usr/local/mentat/tests/unit/conf/filter-grammar.conf </Alert> </Model> </Hawat>
Updated by Pavel Kácha over 11 years ago
- Status changed from In Progress to Feedback
- používat hlavní gramatiku, ne z unit testů
- cesty ke knihovnám nenastavovat v kódu ani v konfiguraci, ale v confu apache, viz hawat-apache.conf
- filter -> grammar, nejlépe do hlavní úrovně vedle log_file, ostatně i u budoucích položek dobře zvážit, jestli je potřeba je strkat do specifických větví (třeba 'devel' význam má)
- přidat konfiguraci monga (db, collection, možná další parametry)
Updated by Pavel Kácha over 11 years ago
Cestu ke gramatice Mek upravil, cesty upravil na autodetekci.
- filter -> grammarfile, nejlépe do hlavní úrovně vedle log_file, ostatně i u budoucích položek dobře zvážit, jestli je potřeba je strkat do specifických větví (třeba 'devel' význam má)
- přidat konfiguraci monga (db, collection, možná další parametry)
- přidat konfiguraci limitu položek na stránku
Updated by Jan Mach over 11 years ago
Trochu jsem vlezl do zelí Radkovi, ale jelikož už jsem se v tom hrabal při řešení kolize, která ve zdrojovém kódu nastala a následných drobých potíží vzniklých při generování a nasazení balíčku (moje chyba) jsem zároveň provedl následující drobné práce:
- Sjednocení štábní kultury
- Opět byly někde mezery místo tabů a layout se rozsypal takže nešlo poznat, kam patří která závorka.
- Sjednocení štábní kultury u definice hlaviček a argumentů funkcí a metod
- Sjednocení štábní kultury názvů - pokud už se používají globální proměnné (v tomto případě spíše statické proměnné třídy), tak by název měl být velkými písmeny
- Konfigurovatelnost
- filter přejmenováno na filter_grammar
- přidána konfigurace display_limit pro modely QUERY i ALERT
- přidána globální konfigurace log_threshold
- Implementováno nastavení smysluplných výchozích hodnot v případě, že se konfigurace nenachází v konfiguračním souboru (bylo potřeba, to byl ten problém při nasazování)
- Konfigurace specifické pro modul přepíší globální konfigurace a ty přepíší konstanty
Updated by Pavel Kácha over 11 years ago
Zbývá tady tedy ještě něco, nebo můžeme zavřít?
Updated by Jan Mach over 11 years ago
- Status changed from Feedback to Closed
Myslím, že všechno je hotovo, můžeme zavřít