Add new OS available from EC2 + make translate_platform_name a little more robust

[meowtochondria]

Addresses the following exception when running python3 ./scrape.py:

Traceback (most recent call last):
  File "./scrape.py", line 1054, in <module>
    scrape("www/instances.json")
  File "./scrape.py", line 1018, in scrape
    add_pricing_info(all_instances)
  File "./scrape.py", line 234, in add_pricing_info
    ec2.add_pricing(by_type)
  File "/home/dghai/ec2instances.info/ec2.py", line 198, in add_pricing
    add_spot_pricing(imap)
  File "/home/dghai/ec2instances.info/ec2.py", line 269, in add_spot_pricing
    price["ProductDescription"], "NA"
  File "/home/dghai/ec2instances.info/ec2.py", line 38, in translate_platform_name
    return os[operating_system] + software[preinstalled_software]
KeyError: 'Red Hat Enterprise Linux (Amazon VPC)'

[EverettBerry]

I think I can actually merge this without testing - did you happen to build the docker container all the way through?

[meowtochondria]

Nope... just reran scrape.py as it seemed to be pretty straightforward. If you would like me to build the container, please let me know!

[EverettBerry]

I think we need to add Red Hat Enterprise Linux (Amazon VPC) here for this change to make it's way to the frontend

ec2instances.info/in/index.html.mako

Line 421 in 31e2fb4

% for platform in ['linux', 'rhel', 'sles', 'mswin', 'mswinSQLWeb', 'mswinSQL', 'mswinSQLEnterprise', 'linuxSQLWeb', 'linuxSQL', 'linuxSQLEnterprise']:

. Also in the JS file here

ec2instances.info/www/default.js

Line 87 in 31e2fb4

'cost-ondemand-rhel',

I can take care of that if you're unfamiliar, or you can use render.py to test that change. With that, I believe the new OS should show up in the columns dropdown.

[EverettBerry]

@meowtochondria was the frontend something you were interested in taking a crack at? Happy to make those changes myself too and merge this.

[meowtochondria]

I did try, but couldn't really make any real progress. Go for it if you understand what needs to be done!

…

-------- Original Message --------

On Sep 1, 2022, 9:21 AM, retttx wrote: ***@***.***(https://github.com/meowtochondria) was the frontend something you were interested in taking a crack at? Happy to make those changes myself too and merge this. — Reply to this email directly, [view it on GitHub](#651 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AAHIXQHZG355NSQELK7KEWLV4DJY7ANCNFSM5463Q6ZA). You are receiving this because you were mentioned.Message ID: ***@***.***>