Project

General

Profile

Actions

Bug #911

closed

Zvážit využití dřívějších example klientů

Added by Jan Soukal over 11 years ago. Updated over 11 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Jan Soukal
Category:
-
Target version:
Start date:
02/26/2013
Due date:
% Done:

0%

Estimated time:

Description

Např. využít sendera, který náhodně generuje "detekovanou" událost. Lze využít při testování funkčnosti klienta na "relativně" rozumných datech např. proti warden-sandbox.

Actions #1

Updated by Jan Soukal over 11 years ago

Jan Soukal wrote:

Např. využít sendera, který náhodně generuje "detekovanou" událost. Lze využít při testování funkčnosti klienta na "relativně" rozumných datech např. proti warden-sandbox.

Skript s pseudo-náhodným senderem přiložen do klientské knihovny mezi ostatní ukázkové klienty - revize 06617017.

Pojmenoval jsem ho zatím v souladu s ostatními jmény example-sender-2.pl.txt, ale možná, má-li být klient striktně testovací, by snesl i jiné pojmenování (test-sender.pl.txt?). Některé drobnosti jsem v kódu oproti původní verzi upravil, aby byl klient kompatibilní s aktuální podobou připravovaného serveru 2.2. Testoval jsem klienta proti 2.2 serveru na warden-c.cesnet.cz.

TODO:
  1. Přejmenovat?
  2. Zvážit, zda do komentářů v kódu testovacího klienta (ale např. i do README) vložit explicitní popis tohoto klienta s doporučeným způsobem použití. Tedy že je to ryze testovací klient (ale to jsou i ostatní), co slouží ke kontaktu s warden-sandbox a že v žádném případě by neměl nic reportovat na provozní server.
Actions #2

Updated by Jan Soukal over 11 years ago

  • Status changed from New to Feedback
Actions #3

Updated by Pavel Kácha over 11 years ago

  • Pojmenoval bych ho třeba example-sender-random.
  • Má význam, aby examply v dokumentaci měly příponu .txt, když jde prakticky o funkční skripty?
  • warden_path v examplu by asi neměla ukazovat do /home/soukal.
Actions #4

Updated by Jan Soukal over 11 years ago

Pavel Kácha wrote:

  • Pojmenoval bych ho třeba example-sender-random.

Souhlasím; hotovo v revizi a97b1cdd

  • Má význam, aby examply v dokumentaci měly příponu .txt, když jde prakticky o funkční skripty?

Toto jsme navrhli ještě s Honzou kvůli tomu, aby bylo opravdu zjevné, že jde o ukázkové klienty. Můžeme to samozřejmě změnit, pokud se domluvíme, že je to (už) zbytečné.

  • warden_path v examplu by asi neměla ukazovat do /home/soukal.

Odkazy do /home/soukal jsou relikty z mého vývoje a testování, kdy si vyvíjím mimo /opt proto, protože klienti dost možná budou nasazování na nejrůznějších lokalitách a taky proto, protože mám spoustu aktivních modifikací či verzí nasypaných u mě v home. Je to pro mě i jednodušší, že si nemusím pokaždé, co si updatnu klienta z repozitáře znovu přepisovat cesty (stejně jsem z velké části asi jediný, kdo klienty stahuje přímo z repozitáře). Při vydávání balíčku tyto cesty vždy upravuju na standardní opt/warden-client. (Tento bod částečně vysvětluje, proč jsem vytvořil ticket #962.)

Actions #5

Updated by Pavel Kácha over 11 years ago

  • Status changed from Feedback to Closed

Ok, ack.

Actions

Also available in: Atom PDF