RouterGateway and ExtraRoute do ReST calls to calculate dependencies
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Heat |
Fix Released
|
Medium
|
Rabi Mishra |
Bug Description
RouterGateways depend on any RouterInterface attached to the same Router. Originally this was checked simply by looking up the `router_id` property of the RouterInterface and comparing it to the `router_id` property of the RouterGateway.
This changed in https:/
IMHO doing a lookup on the API is pointless anyway. The RouterGateway resource takes only a `router_id` (UUID), so the template author will either:
* Create the router in the template and use get_resource (in this case both the RouterGateway and the RouterInterface get passed a UUID for the router, so we can just directly compare them - but see below); OR
* Pass the router UUID in as a parameter and use get_param (in this case both the RouterGateway and the RouterInterface get passed a UUID for the router, so we can just directly compare them).
What they will approximately never do is pass in both the router UUID and router name as parameters and use the name for the `router` property in RouterInterface and the UUID for the `router_id` property in RouterGateway, so that we can't directly compare them without first resolving the name using the Neutron API.
IMHO unconditionally adding in an expensive API call to solve this total non-issue is crazy. The fix should be just to go back to essentially what we were doing before https:/
The same thing applies to OS::Neutron:
See further discussion in https:/
description: | updated |
Changed in heat: | |
assignee: | nobody → Oleksii Chuprykov (ochuprykov) |
Changed in heat: | |
milestone: | ocata-1 → ocata-2 |
Changed in heat: | |
milestone: | ocata-2 → ocata-3 |
Changed in heat: | |
milestone: | ocata-3 → ocata-rc1 |
Fix proposed to branch: master /review. openstack. org/424032
Review: https:/