SVN: toys/stbr/src-builder-status.py

[Bartosz Świątek]

W dniu 29 listopada 2008 16:47 użytkownik Daniel Mróz <beorn w alpha.pl> napisał:
> On Sat, Nov 29, 2008 at 04:12:08PM +0100, Bartosz Świątek wrote:
>> >> Added:
>> >>    toys/stbr/src-builder-status.py   (contents, props changed)
>> >> Log:
>> >> - get spec status from src-builder
>> > [CIAP...diff]
>> > Ałaaa.... Nie lepiej tak:
> [CIAP...kod]
>
>> pewnie i lepiej, ale jak na 10min, które spędziłem nad tym po
>> zgłoszeniu zapotrzebowania na taką funkcję i tak jest nieźle i co
>> najważniejsze, program robi co ma robić.
> Nie krytykuję, tylko proponuję wprowadzenie małych usprawnień :)

Ale ja się ciesze z takiej konstruktywnej krytyki. Nikomu nie
zaszkodzi, a z pownością pomoże projektowi.

>
>> 1. Wyślij dwa zlecenia testowe na buildery z Twojego maila:
>> a) pierwsze zlecenie niech się zbuduje
>> b) drugie zlecenie niech się nie zbuduje wcale, niech nawet nie trafi
>> do queue - celem tego jest dostać mail z src-buildera z błędem, że nie
>> mógł zbudować src.rpm.
> Hmmm... Czy ja wogólę mam taką moc sprawczą? Buildery mają raczej w
> nosie mój adres email i to, co z niego przychodzi.

W takim razie musisz pogadać z naszymi RMami na ten temat.

>
>> 2. Uruchom skrypt i sprawdź czy w pierwszym wypadku dostaniesz OK lub
>> nic (tak naprawdę status poprawny nas w ogóle nie interesuje) i czy w
>> drugim przypadku wyskoczy FAILED (to i tylko to nas interesuje).
>> Oczywiście musisz też sprawdzić czy w razie podania bzdurnego speca
>> program nie wywali jakiegoś błędu - najlepiej żeby nic nie wywalił.
> Czy dostępne są takie wiadomości gdzieś publicznie? Z nagłówkami?
> Ewentualnie, czy mógłbyś mi podesłać takowe? Z nagłowkami?

Publicznie nie. Niestety bot już usunął o 1 w nocy zaległe maile, ale
mogę Ci dać obecne. Tam wprawdzie jest tylko mail z informacją OK, ale
chyba ławo będziesz umiał sobie spreparować takiego z informacja
FAILED.
http://stbr(dot)pld-linux(dot)org/beorn.txt  (zeby boty nie przechwyciły jakieś)
Jak nie będzie ci już to potrzebne to daj znać i to usunę.

>
>> 3. Jak zadziała to nic nie stoi na przeszkodzie byś mnie poprawił w
>> naszym SVNie
> Again, mam dostęp do CVS. Nic mi nie wiadomo o moich uprawnieniach do
> grzebania w repo SVN.

Masz. O ile mi wiadomo to wszyscy regularni developerzy maja dostęp
zarówno do CVS jak i SVN (ten sam login, to samo hasło).

>
>> - ja pythona nie umiem programować, tak tylko czasami z
>> doskoku coś próbuję, także na pewno popełniam błędy, które Tobie mogą
>> wydawać się zabawne.
> Nie wydają mi się zabawne, ani głupie, ani nic z tych rzeczy. Kod jest
> dobry i działa. Zaproponowałem tylko zmianę na bardziej "pythonic way",
> gdyż rozpoznaję w nim silne wpływy języka C, które mogą nieco obniżyć
> wydajność. Nic strasznego, żadnych błędów, tylko usprawnienie.

Cieszę się z każdej poprawy. A i tak, kiedyś sporo w C pisałem.

>
>> 4. Bot ściągnie nową wersje z repozytorium o pełnej godzinie (kiedyś
>> było to tylko o 1 w nocy, ale po tym jak się zaczęli
>> wpisywać/wypisywać developerzy z projektu to lepiej częściej
>> odświeżać).
> Hmmm... To w takim razie przydałaby się jeszcze informacja o
> przeznaczeniu tego skryptu, żebym nie walnął jakiejś głupoty, która
> sprawdza się w jednym środowisku, a w innym niekoniecznie.

Przeznaczenie jest dla http://stbr.pld-linux.org/ (stąd też tam
wylądowało w SVNie).
Konkretnie, żeby dostać informację:
http://stbr.pld-linux.org/?show=pawelz&date=2008-11-28%2013:30:51
zamiast błędnego "not filled".