"validate_network_vlan_ranges" check error in ML2 plugin

Bug #1264202 reported by Liping Mao
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
puppet-neutron
Fix Released
High
Unassigned

Bug Description

In puppet-neutron / manifests / plugins / ml2.pp , we have the following define :
class neutron::plugins::ml2 (
  $type_drivers = ['local', 'flat', 'vlan', 'gre', 'vxlan'],
  $tenant_network_types = ['local', 'flat', 'vlan', 'gre', 'vxlan'],
  $mechanism_drivers = ['openvswitch', 'linuxbridge'],
  $flat_networks = ['*'],
  $network_vlan_ranges = ['10:50'],
  $tunnel_id_ranges = ['20:100'],
  $vxlan_group = '224.0.0.1',
  $vni_ranges = ['10:100']
)

network_vlan_ranges should be something like: ['physnet1:10:50'].

and in puppet-neutron / manifests / plugins / ml2 / driver.pp we check the "network_vlan_ranges":
    neutron::plugins::ml2::validate_network_vlan_ranges { $network_vlan_ranges:; }

If we set $network_vlan_ranges=['physnet1:10:50'], we will get error in "neutron::plugins::ml2::validate_network_vlan_ranges".

Revision history for this message
Mathieu Gagné (mgagne) wrote :
Changed in puppet-neutron:
status: New → Confirmed
importance: Undecided → High
Revision history for this message
Emilien Macchi (emilienm) wrote :

This is fixed in https://review.openstack.org/#/c/55578/

To be more precise, in https://review.openstack.org/#/c/55578/9/lib/puppet/parser/functions/validate_network_vlan_ranges.rb
and you can see the result in unit tests: https://review.openstack.org/#/c/55578/9/spec/classes/neutron_plugins_ovs_spec.rb

When the patch will be merged, I'll also update ml2 unit tests.

Changed in puppet-neutron:
status: Confirmed → Fix Committed
Mathieu Gagné (mgagne)
Changed in puppet-neutron:
milestone: none → 6.0.0
Mathieu Gagné (mgagne)
Changed in puppet-neutron:
status: Fix Committed → Fix Released
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.