Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Co to integrovat se Sentry #92

Closed
westfood opened this issue Mar 6, 2017 · 22 comments
Closed

Co to integrovat se Sentry #92

westfood opened this issue Mar 6, 2017 · 22 comments
Assignees

Comments

@westfood
Copy link
Member

westfood commented Mar 6, 2017

Ahoj, pro Seeder používáme Sentry. Možná by stálo za to, aby nám sentry kontroloval agregoval chybovky.. co říkáš? // http://sentry.webarchiv.cz

@Bystroushaak Bystroushaak self-assigned this Mar 6, 2017
@Bystroushaak
Copy link
Member

Ok. Přidával jsem tam modul pro logování, který to teď loguje ke mě na server. Neměl by být problém to upravit, aby to logovalo sem. Můžeš mi k tomu poslat nějaké přihlašovací údaje, nebo tak něco?

@Bystroushaak
Copy link
Member

Jinak já se asi budu do středy ještě flákat, ve čtvrtek se pak nejspíš pustím do implementace zbytku todo. Ohledně toho bych asi potřeboval zajet a ještě to probrat.

@Bystroushaak Bystroushaak mentioned this issue Mar 6, 2017
@westfood
Copy link
Member Author

westfood commented Mar 6, 2017

Jára tu bude až od 20tého.

Poslal jsem ti pozvánku do sentry.

@Bystroushaak
Copy link
Member

Uhm. Na kterej email? Co tak koukám, tak nikde nic nevidím, ani ve spamu.

@westfood
Copy link
Member Author

westfood commented Mar 6, 2017

@Bystroushaak
Copy link
Member

Ok, tak možná je jen zaseklé zoho, snad to přijde.

@Bystroushaak
Copy link
Member

Crosslink WebarchivCZ/Seeder#14

@westfood
Copy link
Member Author

westfood commented Mar 6, 2017

Tak trable možná na mé straně, host nezná smtp.nkp.cz, dávám to na IP.. chvilku potrvá než se to zpropaguje

@westfood
Copy link
Member Author

westfood commented Mar 6, 2017

tak teď snad už cajk

@Bystroushaak
Copy link
Member

Už to došlo, díky. Mrknu na to zítra.

@kvasnicaj
Copy link
Contributor

@Bystroushaak napiš mi prosím na email, kdy by se ti to hodilo ten týden po 20. 3. a můžeme se sejít :)

Bystroushaak added a commit that referenced this issue Mar 10, 2017
@Bystroushaak
Copy link
Member

Ok, implementováno, teď to tam bleje všechny možné logovací zprávy. Aby to chodilo, přidal jsem dvě proměnné do settings.py (nastavují se přes settings.json):

{
    "LOG_TO_SENTRY": true,
    "SENTRY_DSN": "http://24f...80@sentry.webarchiv.cz/5"
}

Pozor na to SENTRY_DSN, protože to vaše web gui tam bleje blbou adresu ve formátu:

http://24f...80@10.3.0.20:9000/5

Což je nejspíš blbě jak ta IP adresa, tak ten port. S tím sentry.webarchiv.cz/5 jsem to otestoval a fungovalo to.

@Bystroushaak
Copy link
Member

Jo, málem bych zapomněl - vydal jsem novou verzi, kde je sentry integrováno.

@westfood
Copy link
Member Author

Nahodil jsem novou verzi přes Docker, ta integreace se Seederem bude fachat?

@westfood
Copy link
Member Author

Seeder=Sentry

@Bystroushaak
Copy link
Member

Mno. Funguje, ale musíš tam vyplnit to DSN do konfiguráku, který máš v dockeru. Můžu ti ho poslat emailem.

@Bystroushaak
Copy link
Member

To je podobné jako třeba s tim seeder API tokenem.

@Bystroushaak
Copy link
Member

Poslal jsem na mail.

@westfood
Copy link
Member Author

Jasně, máš to v dokumentaci?

@Bystroushaak
Copy link
Member

Mno, v dokumentaci to je, teď je otázka, jestli se to propsalo na readthedocs. Mrknu, případně fixnu.

@Bystroushaak
Copy link
Member

@westfood
Copy link
Member Author

westfood commented Mar 16, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants