Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Region selection currently offers EU, Ireland, which is incompatible with developer instances #6

Open
fbricon opened this issue May 10, 2022 · 5 comments
Labels
bug Something isn't working

Comments

@fbricon
Copy link
Collaborator

fbricon commented May 10, 2022

Region selection currently offers EU, Ireland, which is incompatible with developer instances. We could hard code a filter like supported_instance_types.includes('developer'), but I assume the allowed instance type must be bound to the user account somehow.

Screenshot 2022-05-10 at 16 08 46

@wtrocki any idea on how to proceed?

@fbricon fbricon added the bug Something isn't working label May 10, 2022
@wtrocki
Copy link

wtrocki commented May 10, 2022

That is exactly what we have been doing.

supported_instance_types.includes('developer')

Although users should be able to create standard instances.

@fbricon
Copy link
Collaborator Author

fbricon commented May 10, 2022

@wtrocki standard instance is not a choice offered by the RHOSAK wizard in
Screenshot 2022-05-10 at 16 40 44

Anyways. since CloudRegion.supported_instance_types is deprecated, I'm going with:

cloudRegions.filter(cr => cr.enabled && cr.capacity.find(rc => 'developer' === rc.instance_type));

@wtrocki
Copy link

wtrocki commented May 10, 2022

As user you need to pay for that option

@fbricon
Copy link
Collaborator Author

fbricon commented May 10, 2022

so how can I know if a user can use standard instances?

@wtrocki
Copy link

wtrocki commented May 11, 2022

There is this account management sdk that we use for this purpose but it is very low level API that requires mapping of some types. I would not recommend to use it. We need improvements on AMS API side. Once that is done integrations should be simpler.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants