Kwiecień 2005 - lms - lists.lms.org.pl

Re: jeszcze lmsd
by Grzegorz Stanislawski 26 Kwi '05

26 Kwi '05

A.L.E.C napisał(a): > Grzegorz Stanislawski wrote: > >>>+ } else if (fval==0) { //child >>>+// //restore old handler so we can wait() for childs executed by modules >>>+// sigaction (SIGCHLD,&orig, NULL) ; > to w końcu potrzebne to jest czy nie? no wlasnie nie jestem pewien. dlatego jest wyhashowane. przy braku tego moga byc problemy z procesami odpalanymi przez moduly, a forki sa i w ping i w system i pewnie beda w nowych modulach. Dobrze by bylo przygotowac autorom modulu wmiare standardowe srodowisko Z drugiej strony jest to poki co niepotrzebne wywolanie systemowe. proponuje na razie to zostawic zachaszowane. Jak ktos beedzie mial taki problem przy pisaniu jakiegos modulu to bedzie mial gotowa podpowiedz. >>>+ }else { //parent >>>+ sleep(100); > z tym sleep(100) to coś nie bardzo, próbowałeś go uruchamiać z opcją -q, > albo bez -qf > sleep ma pozwolic systemowi zajecie sie przez chwile potomkiem i przyniesc efekt w postaci mniejszego obciazenia systemu (choc zauwazalnosc tego jest nikla) -qf jest rownowazne -q. -f nie wplywa na nic innego niz pierwszy fork co zreszta widac wyraznie w zrodlach. > zrobiłem tak: > // forking reload - we can do a job for longer than one minute > [..] > else if( fval==0 ) //child > { > setsid(); // załatwia sprawę wyświetlania czegokolwiek na > // stdout, ale nie wiem czy to jest właściwe nie wydaje mi sie ze setsid tu ma jakikolwiek sens, jesli juz mialoby byc to tam gdzie caly lmsd sie demonizuje. poza tym setsid nie sluzy temu. Wyswietlanie na stdout nie jest niczym zlym a nawet jest porzytecznie. Mozna to puscic do pliku i zrobic w ten sposob loga. Tez nalezy sie zastanowic czy przy -q warto wogole forkowac worker threada. lmsd -q jest przeznaczone (imho) do puszczenia go z palca w shellu, albo w skrypcie i kazdy kto to puszcza chcialby widziec wynik (jesnie nie to zawsze mozna dac >/dev/null) i miec pewnosc ze reload sie skonczyl. zmiana jest banalna: if(quit) fval=fork() else fval=0; w 264 lini > // exit child (reload) thread > if( quit ) > termination_handler(0); // write info to syslog > else > exit(0); > } > else if( quit ) exit(0); Zupelnie nie rozumiem po co ci ten fragment (dokladnie czemu ma tak wygladac). mysle ze debug o "reload child exit" jest bardziej czytelny. exit(0) za klamerka to zakonczenie calego demona a wiec tam powinien byc termination_handler(0). exit(0) przed klamerka tu jak i w moim patchu mozna nawet pominiac. Inna sprawa ze zamiast tylu exitow, bardziej elegancko bylo by poslugiwac sie wyskakiwaniem z petli przez break. Na razie chcialem aby moj patch byl jak najmniej inwazyjny w strukture programu. Mysle ze jest tu jeszcze sporo do zoptymalizowania. Grzegorz Stanislawski

1 0

Re: jeszcze lmsd
by A.L.E.C 26 Kwi '05

26 Kwi '05

Piotr Górski wrote: > A z moich testów wynika, że jak się nic nie wpisze w polu Crontab w > LMSUI to dany moduł odpala się co minute - lmsd ma gdzieś to czy w bazie > jest zaznaczone przeładuj czy nie... to dzisiaj poprawię (jest w patchu G.Stanisławskiego) > No i ggnotify mi kompletnie nie działa... o czym już zresztą pisałem... pisałem jak go uruchomić/kompilować, SOA#1 -- Aleksander 'A.L.E.C' Machniak http://alec.pl gg-2275252 Lan Management System Developer http://lms.alec.pl

1 0

Re: jeszcze lmsd
by Piotr Górski 26 Kwi '05

26 Kwi '05

A z moich testów wynika, że jak się nic nie wpisze w polu Crontab w LMSUI to dany moduł odpala się co minute - lmsd ma gdzieś to czy w bazie jest zaznaczone przeładuj czy nie... No i ggnotify mi kompletnie nie działa... o czym już zresztą pisałem... -- Piotrek Górski http://www.audioscrobbler.com/user/sledzik1984 Mój klucz PGP: http://tinyurl.com/3nhvd

1 0

Re: jeszcze lmsd
by A.L.E.C 26 Kwi '05

26 Kwi '05

Grzegorz Stanislawski wrote: [ciach] >> - doklada spowrotem opcje -f (run in foreground" albo jak kto woli >> "don't fork"), nie pomyślałem że to będzie potrzebne I dobrze, że ktoś się wreszcie zabrał za testy nowego demona. A oto co wymaga jeszcze dopracowania, czy też ustalenia (usunąłem fragmenty które nie budzą wątpliwości i zostaną uwzględnione) >> - poprawia blad z forkowaniem "worker thread'a" czyli sprawia ze caly >> program zachowuje sie jak na daemona przystalo (m.in nie skacze po >> pidach i nie zostawia zombich) hmm... coś mi nie chce to działać, mógłbyś jeszcze potestować i odnieść się do poniższych uwag/pytań? >> + } else if (fval==0) { //child >> +// //restore old handler so we can wait() for childs executed by modules >> +// sigaction (SIGCHLD,&orig, NULL) ; to w końcu potrzebne to jest czy nie? >> - if( quit ) >> - termination_handler(0); // write info to syslog >> - else >> - exit(0); >> +#ifdef DEBUG1 >> + syslog(LOG_INFO, "Reload child exiting..."); >> +#endif >> + >> + exit(0); >> + }else { //parent >> + sleep(100); z tym sleep(100) to coś nie bardzo, próbowałeś go uruchamiać z opcją -q, albo bez -qf zrobiłem tak: // forking reload - we can do a job for longer than one minute int fval = fork(); if( fval < 0 ) { syslog(LOG_CRIT, "Fork error. Can't reload."); if ( quit ) termination_handler(1); } else if( fval==0 ) //child { setsid(); // załatwia sprawę wyświetlania czegokolwiek na // stdout, ale nie wiem czy to jest właściwe rozwiązanie (?) [...] // exit child (reload) thread if( quit ) termination_handler(0); // write info to syslog else exit(0); } else if( quit ) exit(0); [...] co ty na to? -- Aleksander 'A.L.E.C' Machniak http://alec.pl gg-2275252 Lan Management System Developer http://lms.alec.pl

1 0

Re: dodatkowy ip dla komputera - NAT
by KubaTyszko 26 Kwi '05

26 Kwi '05

On Sat, Apr 23, 2005 at 04:36:37PM +0200, Krzysztof Lewandowski wrote: > KubaTyszko napisał(a): > >On Sat, Apr 23, 2005 at 02:46:25PM +0200, KubaTyszko wrote: > > > >>czesc. > >>chcecie taka funkcjonalnosc ? > >>wlasnie to robie "na zamowienie" i jak ktos chce to moge zacommitowac. > >>(moze byc to "regulowane" opcja np. "support nat extra ip" czy jakos > >>tak... > >> > >> > >> > > > >jakby ktos nie wiedzial o co chodzi : ;-) > >dodaje dodatkowe pole (drugi adres ip (publiczny)) dla komputerow ktorym > >robi sie NAT statyczny 1=1. > >oczywiscie co sie z tym wiaze dodaje to do wyszukiwarki, itp.itd. > > > > > > > > > Tak przyda się taka opcja na stałe w lms, ja też własnie wprowadzam > natowanie 1:1 publicznych IP dla klientów. > > KL > > -- > Krzysztof Lewandowski - krzysiek(a)lca.pl > Oficjalny Portal Miasta Legnicy - www.lca.pl > 59-220 Legnica, ul. Pomorska 56 > tel. (76) 866 49 11 biuro(a)lca.pl > Nie odpisuje na listy pisane w HTML > > w najnowszej CVS juz wszystko jest nieprzerobione sa jeszcze skrypty firewallowe itp.

1 0

Re: [lms] uwzględnienie VATu w taryfie
by A.L.E.C 26 Kwi '05

26 Kwi '05

Andrzej Kosmala wrote: > Zamierzam zmienić definicję taryfy i uwzględnić w niej VAT. Dotychczas > mam zw. > Czy w pozycji wartość ma być netto? czy pozostawić brutto a dodać tylko > wielkość PTU? brutto -- Aleksander 'A.L.E.C' Machniak http://alec.pl gg-2275252 Lan Management System Developer http://lms.alec.pl

1 0

uwzględnienie VATu w taryfie
by Andrzej Kosmala 26 Kwi '05

26 Kwi '05

Zamierzam zmienić definicję taryfy i uwzględnić w niej VAT. Dotychczas mam zw. Czy w pozycji wartość ma być netto? czy pozostawić brutto a dodać tylko wielkość PTU? Pozdrawiam Andrzej Kosmala

1 0

jeszcze lmsd
by Grzegorz Stanislawski 25 Kwi '05

25 Kwi '05

Witam. Natknalem sie na ta niespodzianke juz pare dni temu ale jakos nie mialem kiedy o tym napisac. Otoz z lmsd zniknela opcja blokujaca demonizowanie sie lmsd co w pratkyce uniemozliwia puszczanie go z innitaba co jest jednym z najbardziej sensownych (imho) sposobow odpalania najwazniejszych procesow. Postanowilem to sobie dorobic i jakaz byla moja niespodzianka gdy po chwili mialem dzialajacych kilka demonow. Przygladajac sie zrodlom natknolem sie na kolejny fork uruchamianacy nazwijmy to 'worker thread' co samo w sobie jest bardzo pozyteczne. Problem polegal na tym ze to potomek przejmowal role petli czekajcej na kolejne zdarzenie a proces macierzysty zaczynal odwalac czarna robote poczym sie konczyl. A init widzac ze proces zakonczyl dzialanie odpalal go ponownie. Musze przyznac ze najpierw myslalem ze to zwykla pomylka (jakie Alec ostatnio ci sie czesto zdarzaja ;-) ale po chwili zastanowienia doszlem do wniosku ze to wykonana specjalnie atrapa majaca ukryc problem z zombie'sami bedacymi nieuchronnym skutkiem odpalania niesynchronizowanych potomkow. W zalaczeniu jest patch ktory: - doklada spowrotem opcje -f (run in foreground" albo jak kto woli "don't fork"), - poprawia blad z forkowaniem "worker thread'a" czyli sprawia ze caly program zachowuje sie jak na daemona przystalo (m.in nie skacze po pidach i nie zostawia zombich) - dodaje warunek zapobiegajacy bezsensownemu odpalaniu system() gdy zmienna command jest pusta (brak opcji -c "[..]") - zmienia logike funkcji crontab_match, dzieki czemu 'instancjie' z pustym polem crontab nie sa odpalane przy kazdej mozliwej okazji tylko podczas zadania reloadu. - no i na koniec potwierdzenie wykonania reloadu jest zapisywane do bazy tylko w tedy jesli reload byl zadany (updatowanie za kazdym razem jest mylace jesli demon odpala takie zadania jak ping i niepotrzebnie obciaza baze) Grzegorz Stanislawski Index: lmsd.c =================================================================== RCS file: /home/cvsroot/lms/daemon/lmsd.c,v retrieving revision 1.6 diff -u -r1.6 lmsd.c --- lmsd.c 25 Apr 2005 13:48:15 -0000 1.6 +++ lmsd.c 25 Apr 2005 19:53:52 -0000 @@ -30,10 +30,12 @@ #include <stdio.h> #include <string.h> #include <dlfcn.h> - +#include <sys/types.h> +#include <sys/wait.h> + #include "lmsd.h" -int quit = 0, port = 0; +int quit = 0, port = 0, dontfork=0; char *db, *user, *passwd, *host, *dhost; unsigned char *command = NULL; unsigned char *iopt = NULL; @@ -41,6 +43,13 @@ static void parse_command_line(int argc, char **argv); static void free_module(MODULE *module); static int crontab_match(time_t tt, char *crontab); +struct sigaction act,orig; +//signal handler for SIGCHLD reaps zombie children +void sig_child(int signo) +{ + while(waitpid(-1, NULL, WNOHANG) > 0) + continue; +} int main(int argc, char *argv[]) { @@ -95,8 +104,17 @@ g->config_getbool = &config_getbool; g->config_getdouble = &config_getdouble; + // catch SIGCHLD to catch zombies ;-) + act.sa_handler = sig_child; + sigemptyset(&act.sa_mask); + act.sa_flags = 0; + sigaction (SIGCHLD, &act, &orig) ; + // termination signals handling + signal(SIGINT, termination_handler); + signal(SIGTERM, termination_handler); + // daemonize - if ( !quit ) { + if ( !quit && !dontfork) { int fval = fork(); switch (fval) { @@ -114,10 +132,6 @@ } } - // termination signals handling - signal(SIGINT, termination_handler); - signal(SIGTERM, termination_handler); - // main loop **************************************************** for (;;) { @@ -134,9 +148,13 @@ tt = cron_sync_sleep(); } - // run shell command, i.e. secure connections tuneling - system(command); - + // run shell command, i.e. secure connections tuneling + if(command!=NULL) { +#ifdef DEBUG1 + syslog(LOG_INFO, "executing command %s.", command); +#endif + system(command); + } // try to connect to database if( !(g->conn = db_connect(db,user,passwd,host,port)) ) { @@ -243,16 +261,13 @@ // forking reload - we can do a job for longer than one minute if( i_no ) { - switch (fork()) - { - case -1: + int fval = fork(); + if (fval <0) { syslog(LOG_CRIT, "Fork error. Can't reload."); if ( quit ) termination_handler(1); - break; - case 0: // continue main loop - if( quit ) exit(0); // quiet parent exit - break; - default: + } else if (fval==0) { //child +// //restore old handler so we can wait() for childs executed by modules +// sigaction (SIGCHLD,&orig, NULL) ; #ifdef DEBUG1 syslog(LOG_INFO, "DEBUG: [lmsd] Reloading..."); #endif @@ -272,6 +287,7 @@ mod->file = strdup(instances[i].module); mod->instance = strdup(instances[i].name); mod->dlh = dlopen(mod->file, RTLD_NOW); + syslog(LOG_INFO, "module '%s': loaded", mod->file); if( !mod->dlh ) { @@ -304,15 +320,19 @@ free_module(mod); } - // write reload timestamp - db_pexec(g->conn, "UPDATE daemonhosts SET lastreload=%NOW%, reload=0 WHERE name='?'", dhost); - db_disconnect(g->conn); - + // write reload timestamp only when reload was requested + if(reload) { + db_pexec(g->conn, "UPDATE daemonhosts SET lastreload=%NOW%, reload=0 WHERE name='?'", dhost); + db_disconnect(g->conn); + } // exit child (reload) thread - if( quit ) - termination_handler(0); // write info to syslog - else - exit(0); +#ifdef DEBUG1 + syslog(LOG_INFO, "Reload child exiting..."); +#endif + + exit(0); + }else { //parent + sleep(100); } for(i=0; i<i_no; i++) @@ -340,7 +360,7 @@ sscanf(REVISION, "$Id: lmsd.c,v %s", revision); - while ( (opt = getopt(argc, argv, "qvi:h:p:d:u:H:c:")) != -1 ) + while ( (opt = getopt(argc, argv, "qfvi:h:p:d:u:H:c:")) != -1 ) { switch (opt) { @@ -350,6 +370,9 @@ case 'q': quit = 1; break; + case 'f': + dontfork = 1; + break; case 'i': iopt = optarg; break; @@ -380,6 +403,7 @@ printf(" -H daemon_host\t\thost name where runs daemon (default: `hostname`)\n"); printf(" -c command\t\tshell command to run before database connecting (default: empty)\n"); printf(" -q \t\t\tdo a reload and quit\n"); + printf(" -f \t\t\tdo not fork to background\n"); printf(" -v \t\t\tprint version and copyright info\n"); printf(" -i \"instance[ ...]\" list of instances to reload\n"); exit(1); @@ -404,8 +428,8 @@ { if( cron_parse_time(&ct, crontab) != PARSE_OK ) return 0; - if( !cron_match_time(&ct, &tt) ) - return 0; + if( cron_match_time(&ct, &tt) ) + return 1; } - return 1; + return 0; }

1 0

Re: GGNotify
by A.L.E.C 25 Kwi '05

25 Kwi '05

Piotr Górski wrote: >> Apr 25 13:53:00 localhost lmsd: Unable to load module >> '/var/www/html/biuro.prokocim.org/daemon/modules/ggnotify/ggnotify.so': >> /var/www/html/biuro.prokocim.org/daemon/modules/ggnotify/ggnotify.so: >> undefined symbol: gg_login > > Już próbowałem paru różnych wersji libgadu - zarówno z ekg jak i ekg2 i > za cholere nie moge zmusić ggnotify z 1.5.6 do pracy... Czy ktoś mógłby > mi powiedzieć jak zmusić ten moduł do pracy? robię (mniej więcej) tak: cd daemon/modules/ggnotify wget dev.null.pl/ekg/ekg-1.5.tar.gz tar zxf ekg-1.5.tar.gz cd ekg-1.5 ./configure --enable-shared make make install ldconfig cd .. w Makefile zmieniam (ale równie dobrze można to zakomentować): # dla libgadu i ekg -------------- LDFLAGS += -L/usr/local/lib CFLAGS += -I/usr/local/include w końcu kompilacja ggnotify: make i działa. Pozdrawiam -- Aleksander 'A.L.E.C' Machniak http://alec.pl gg-2275252 Lan Management System Developer http://lms.alec.pl

1 0

GGNotify
by Piotr Górski 25 Kwi '05

25 Kwi '05

> Apr 25 13:53:00 localhost lmsd: Unable to load module '/var/www/html/biuro.prokocim.org/daemon/modules/ggnotify/ggnotify.so': /var/www/html/biuro.prokocim.org/daemon/modules/ggnotify/ggnotify.so: undefined symbol: gg_login Już próbowałem paru różnych wersji libgadu - zarówno z ekg jak i ekg2 i za cholere nie moge zmusić ggnotify z 1.5.6 do pracy... Czy ktoś mógłby mi powiedzieć jak zmusić ten moduł do pracy? -- Piotrek Górski http://www.audioscrobbler.com/user/sledzik1984 Mój klucz PGP: http://tinyurl.com/3nhvd

1 0