[INTTEST-49] Use ElasticSearch official Python client to make the code more readable Created: 21/May/18 Updated: 14/Jun/18 |
|
| Status: | Open |
| Project: | integration-test |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Medium |
| Reporter: | Dibya Prakash Das | Assignee: | Dibya Prakash Das |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
https://elasticsearch-py.readthedocs.io/en/master/ |
| Comments |
| Comment by Himanshu Chhabra [ 14/Jun/18 ] |
|
Hi. I am new to opendaylight. Can you elaborate the issue and also tell where the code resides in repository. |
| Comment by Dibya Prakash Das [ 14/Jun/18 ] |
|
The code for this resides in csit/scripts/push_to_elk.py. Earlier, to update the ES indexes with the data, plain HTTP requests were sent to the ES sever. This was alright but it was a bit too verbose and errors, if thrown, are not too informative. Elasticsearch offer a Python API client that can interact with the server to this. This not only made the code more readable and organized, the errors handling was better too. I have already sent a https://git.opendaylight.org/gerrit/#/c/72448/ for this and this issue will soon be closed once that is merged. |