search

300Geeks / aws-elk-billing

Tools for Loading and Visualising AWS Detailed Billing with ELK(Elasticsearch, Logstash, Kibana)
MIT License
157 stars 26 forks source link

fix some assumptions about report paths #8

Closed victortrac closed 8 years ago

victortrac commented 8 years ago

tools.py makes some assumptions about path lengths and infers a report name based on the path. This PR adds in a report name config to fix this, removes a dependency on / (or lack thereof), and cleans up some printing.

droidlabour commented 8 years ago

Thanks for the PR. Although I've changed it a bit. Instead of strip it should be split on /. And added some error handling if there's no file found under S3 Report Path