[Sahara] Heat not always can configure neutron network

Bug #1402844 reported by Andrew Lazarev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Triaged
Medium
Pavlo Shchelokovskyy

Bug Description

In case of neutron Heat requires neutron_management_network and floating_ip_pool to be set for cluster. There are two things wrong here:

1. floating_ip_pool is not mandatory in general case. Sahara can be configured to use netns or proxy node to access VMs
2. neutron_management_network can be already configured in cluster template

Revision history for this message
Andrew Lazarev (alazarev) wrote :

Even more: neutron_management_network is now required for both template and cluster. There is no need to require it in both places (actually it is not required at all).

Revision history for this message
Andrew Lazarev (alazarev) wrote :

Looked to the code, neutron_management_network is required, but it can be set in one place - template or cluster.

summary: - [Sahara] Heat requires neutron_management_network for cluster
+ [Sahara] Heat not always can configure neutron network
description: updated
Changed in heat:
assignee: nobody → Pavlo Shchelokovskyy (pshchelo)
Revision history for this message
Pavlo Shchelokovskyy (pshchelo) wrote :

Andrew, I understand the point of not requiring neutron_management_network in both places (although here unfortunately we are stuck with deprecation period as these properties are part of Juno release already), but could you explain more why it is not required at all? what would happen if I create a cluster when

- it is nova-network and I pass this propety?
- it is neutron with multiple networks and I do not pass this property?

Revision history for this message
Andrew Lazarev (alazarev) wrote :

>it is nova-network and I pass this propety?
Sahara will return validation error

>it is neutron with multiple networks and I do not pass this property?
Sahara will return validation error

So, neutron_management_network must be provided in case (and only in case) of neutron. It should be provided either in cluster template or in cluster (or in both, value from cluster will be used in this case).

Angus Salkeld (asalkeld)
Changed in heat:
status: New → Triaged
importance: Undecided → Medium
Rico Lin (rico-lin)
Changed in heat:
milestone: none → no-priority-tag-bugs
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.