Bug 1705565 - [RFE] Add m5ad, m5.metal, md5.metal, r5.metal, r5ad, r5d.metal and z1d.metal AWS instance types to CFME
Summary: [RFE] Add m5ad, m5.metal, md5.metal, r5.metal, r5ad, r5d.metal and z1d.metal ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers
Version: 5.10.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.11.0
Assignee: Adam Grare
QA Contact: Matouš Mojžíš
Red Hat CloudForms Documentation
URL:
Whiteboard:
Depends On:
Blocks: 1728708
TreeView+ depends on / blocked
 
Reported: 2019-05-02 13:39 UTC by Matouš Mojžíš
Modified: 2019-12-13 14:55 UTC (History)
5 users (show)

Fixed In Version: 5.11.0.4
Doc Type: Enhancement
Doc Text:
Clone Of:
: 1728708 (view as bug list)
Environment:
Last Closed: 2019-12-13 14:55:06 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
metal instances (163.97 KB, image/png)
2019-07-25 15:38 UTC, Matouš Mojžíš
no flags Details

Description Matouš Mojžíš 2019-05-02 13:39:42 UTC
Description of problem:
https://aws.amazon.com/blogs/aws/new-amd-epyc-powered-amazon-ec2-m5ad-and-r5ad-instances/
https://aws.amazon.com/about-aws/whats-new/2019/02/introducing-five-new-amazon-ec2-bare-metal-instances/

Version-Release number of selected component (if applicable):
5.10.4.1

How reproducible:
Always

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 2 Alexander Zagaynov 2019-05-10 01:11:37 UTC
PR: https://github.com/ManageIQ/manageiq-providers-amazon/pull/532

Comment 3 Alexander Zagaynov 2019-05-10 01:54:29 UTC
PR merged

Comment 4 Matouš Mojžíš 2019-06-27 14:39:51 UTC
Verification failed.
r5d.metal and z1d.metal are still not in CFME.
I created new BZ where these two instance types are included: https://bugzilla.redhat.com/show_bug.cgi?id=1724686
Maybe this BZ can be verified/closed in favor of the other BZ.

Comment 5 Satoe Imaishi 2019-07-09 20:26:40 UTC
Matous, looks like the missing types are covered in a separate BZ, so I think it makes sense to move this to VERIFIED. If you do, please update the BZ summary to remove types not yet included :)

Comment 6 Matouš Mojžíš 2019-07-10 11:13:49 UTC
Okay then verified in 5.11.0.10. Instance types are now in CFME.

Comment 8 Alexander Zagaynov 2019-07-10 16:52:55 UTC
Matous, r5, r5d, and z1d "bare" instances have no "metal" word in their names. It is defined by the AWS price-list API. They named just "r5" and so on.
I understand there is no naming consistency with other "metal" instance types, like "m5.metal", but this is what we have on the AWS side and in thir gem/API.
If we'll start to name it on our own, we should expect issues when provisioning them.

Comment 9 Matouš Mojžíš 2019-07-25 15:38:30 UTC
Created attachment 1593435 [details]
metal instances

Alex,
I am checking the list from CFME Flavors with instance types that are available for launch from AWS Console.
r5/r5d/z1d instances have "metal" world in their name.
I would say it makes sense to have it the same way as AWS console has.

Comment 10 Alexander Zagaynov 2019-09-03 17:22:10 UTC
Fixed in https://bugzilla.redhat.com/show_bug.cgi?id=1724686


Note You need to log in before you can comment on or make changes to this bug.