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

pricer failing: fetching price for 'foo': strconv.ParseFloat: parsing "": invalid syntax #139

Open
snerdish opened this issue Sep 1, 2017 · 1 comment
Labels

Comments

@snerdish
Copy link

snerdish commented Sep 1, 2017

The Pricing inspector seems to be having trouble. Not sure if this is a windows go thing.

`PS C:\Users\foo\go\bin> .\awless.exe inspect -i pricer -e
[verbose] awless v0.1.3 - loading AWS session with profile '' and region 'ap-southeast-2'
[info] Running full sync before inspection (disable it with --local flag)

[extra] sync: fetched cloudformation service took 350.6837ms
[extra] sync: fetched lambda service took 467.6854ms
[extra] sync: fetched messaging service took 625.6852ms
[extra] sync: fetched cdn service took 1.3626842s
[extra] sync: fetched dns service took 2.4587317s
[extra] sync: fetched storage service took 5.0679941s
[extra] sync: fetched access service took 5.4482759s
[extra] sync: fetched monitoring service took 20.5564768s
[extra] sync: fetched infra service took 42.5428424s
Fetching prices at http://ec2-price.com for region global

fetching price for 't2.large': strconv.ParseFloat: parsing "": invalid syntax
fetching price for 't2.micro': strconv.ParseFloat: parsing "": invalid syntax
fetching price for 't2.medium': strconv.ParseFloat: parsing "": invalid syntax
fetching price for 'i3.large': strconv.ParseFloat: parsing "": invalid syntax
fetching price for 'r4.large': strconv.ParseFloat: parsing "": invalid syntax
fetching price for 't2.nano': strconv.ParseFloat: parsing "": invalid syntax
fetching price for 'r4.xlarge': strconv.ParseFloat: parsing "": invalid syntax
fetching price for 'm4.large': strconv.ParseFloat: parsing "": invalid syntax
fetching price for 'm3.large': strconv.ParseFloat: parsing "": invalid syntax
fetching price for 't2.small': strconv.ParseFloat: parsing "": invalid syntax
fetching price for 'm4.2xlarge': strconv.ParseFloat: parsing "": invalid syntax
Instance Count Estimated total/day (no EBS)


m3.large 2
r4.xlarge 1
i3.large 1
m4.2xlarge 5
m4.large 10
t2.medium 38
t2.large 10
r4.large 2
t2.small 21
t2.micro 15
t2.nano 1
$0.00
PS C:\Users\foo\go\bin> .\awless.exe --version
awless version=v0.1.3`
go v1.9

@simcap
Copy link
Contributor

simcap commented Sep 1, 2017

@snerdish Thanks for reporting.

This could be 2 things:

  • An non response from the API we use for pricing which is not ours (nor AWS) but an ad hoc one http://ec2-price.com). It is unfortunately not very reliable.
  • or the fact that even if you run the query in region ap-southeast-2, it logs (Fetching prices at http://ec2-price.com for region global) that it fetches the price for region global (i.e. service with no region like IAM)

We will investigate that.

This pricer is indeed an inspector we implemented as a quick proof of concept for pricing awless templates and AWS infrastructure. We will be soon in the process of making it much better and reliable: using the AWS json prices files; caching them; etc...

(Note inspectors are good piece of code that can be done adn contributed by the community as well. There is actually a PR under review that has some good points as well)

@fxaguessy fxaguessy added the bug label Sep 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants