feat: move network translation to provider and remove redundant validation

neoave / mrack

Multicloud use-case based multihost async provisioner for CIs and testing during development

Apache License 2.0

11 stars 14 forks source link

feat: move network translation to provider and remove redundant validation #117

Closed Tiboris closed 3 years ago

Tiboris commented 3 years ago

Moving network translation to provider will help us to find new network for openstack VM when there is a problem with network pools (insufficient ips) that will help to choose another pool within openstack.

Transformer now only informs about network type provider picks the network at every provision retry.

Signed-off-by: Tibor Dudlák tdudlak@redhat.com

pvoborni commented 3 years ago

When I see this change, I'm almost thinking that we can merge all providers and transformers. As most logic is being moved to providers and transformers don't do much - something for another PRs, not necessarily needed to prioritize.

Tiboris commented 3 years ago

@pvoborni probably it is moving this direction... Let us seeee