dnsmasq_ip_subnets - IPv6 netmask support

Bug #1828837 reported by Harald Jensås
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-ironic
Fix Released
Undecided
Harald Jensås

Bug Description

dnsmasq.conf require a prefix lenght as netmask for IPv6, while the puppet module takes a netmask value, for example: ffff:ffff:ffff:ffff:0000:0000:0000:0000 or ffff:ffff:ffff:ffff::

A full IPv6 subnet example:
[
 {'tag' => 'subnet1',
  'ip_range' => '2001:4888:a03:313a:c0:fe0:0:c200,2001:4888:a03:313a:c0:fe0:0:c2ff',
  'netmask' => 'ffff:ffff:ffff:ffff::',
  'gateway' => '2001:4888:a03:313a:c0:fe0:0:c000'}
]

This resulting dhcp-range in dnsmasq.conf would be:
set:subnet1,2001:4888:a03:313a:c0:fe0:0:c200,2001:4888:a03:313a:c0:fe0:0:c2ff,ffff:ffff:ffff:ffff::,10m

This is invalid, and should use the prefix lenght "64" instead of "ffff:ffff:ffff:ffff:::":
set:subnet1,2001:4888:a03:313a:c0:fe0:0:c200,2001:4888:a03:313a:c0:fe0:0:c2ff,64,10m

summary: - dnsmasq_ip_subnets - IPv6 support
+ dnsmasq_ip_subnets - IPv6 netmask support
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-ironic (master)

Fix proposed to branch: master
Review: https://review.opendev.org/658883

Changed in puppet-ironic:
assignee: nobody → Harald Jensås (harald-jensas)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-ironic (master)

Reviewed: https://review.opendev.org/658883
Committed: https://git.openstack.org/cgit/openstack/puppet-ironic/commit/?id=7b6b097d8ac8ebaf3058484ef0c3d5b464fbdc1a
Submitter: Zuul
Branch: master

commit 7b6b097d8ac8ebaf3058484ef0c3d5b464fbdc1a
Author: Harald Jensås <email address hidden>
Date: Mon May 13 19:20:05 2019 +0200

    Convert ipv6 netmask to prefix in dnsmasq.conf

    dnsmasq.conf require a prefix lenght as netmask for IPv6.
    Convert a IPv6 address netmask to prefix.

    Closes-Bug: #1828837
    Change-Id: Idf84ba30eb4eb6d202faa470209f10c9da40e80b

Changed in puppet-ironic:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-ironic 15.0.0

This issue was fixed in the openstack/puppet-ironic 15.0.0 release.

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.