fix: use eth0 instead of eth1 in test recipe for Dokken containers

eth1 does not exist in Dokken containers - only eth0 is available.
Fixes dnsmasq service startup failure on all platforms.

Signed-off-by: Dan Webb <dan.webb@damacus.io>

Author: damacus

.yamllint

@@ -15,3 +15,5 @@ rules:
     min-spaces-from-content: 1
   brackets:
     max-spaces-inside: 1
+ignore: |
+  .kitchen/

migration.md

@@ -0,0 +1,60 @@
+# Migration Guide
+
+This cookbook migrated from root-level recipes and attributes to custom resources.
+
+## What changed
+
+* Root `attributes/` and `recipes/` usage was replaced by resource properties.
+* The public interface is now:
+  * `dnsmasq`
+  * `dnsmasq_dns`
+  * `dnsmasq_dhcp`
+  * `dnsmasq_managed_hosts`
+* DNS, DHCP, and managed-host settings now belong on resources instead of node attributes.
+
+## How to migrate
+
+Legacy pattern:
+
+```ruby
+include_recipe 'dnsmasq::default'
+include_recipe 'dnsmasq::dns'
+include_recipe 'dnsmasq::dhcp'
+
+node.default['dnsmasq']['dns']['server'] = '8.8.8.8'
+node.default['dnsmasq']['dhcp']['dhcp-range'] = '10.0.0.5,10.0.0.15,12h'
+```
+
+Resource-first pattern:
+
+```ruby
+dnsmasq 'default' do
+  enable_dns true
+  enable_dhcp true
+  managed_hosts_data_bag false
+  dns_config(
+    'server' => '8.8.8.8'
+  )
+  dhcp_config(
+    'dhcp-range' => '10.0.0.5,10.0.0.15,12h',
+    'except-interface' => 'lo'
+  )
+  managed_hosts(
+    '10.0.0.20' => ['router.test.lab', 'router']
+  )
+end
+```
+
+## Attribute migration
+
+Move legacy configuration hashes into resource properties:
+
+* DNS settings -> `dns_config` and `dns_options`
+* DHCP settings -> `dhcp_config` and `dhcp_options`
+* managed hosts -> `managed_hosts`, `managed_hosts_data_bag`, `managed_hosts_data_bag_item`
+
+If you prefer more granular declarations, use `dnsmasq_dns`, `dnsmasq_dhcp`, and `dnsmasq_managed_hosts` directly.
+
+## Important note
+
+The migrated cookbook still uses the `hostsfile` cookbook for managed entries under the hood, but consumers should treat the dnsmasq resources as the public API.

test/cookbooks/test/recipes/default.rb

@@ -1,6 +1,12 @@
 # frozen_string_literal: true
 
-apt_update if platform_family?('debian')
+apt_update 'update package cache' do
+  retries 30
+  retry_delay 2
+  frequency 86_400
+  action :periodic
+  only_if { platform_family?('debian') }
+end
 
 package value_for_platform_family(
   default: 'bind-utils',
@@ -21,10 +27,10 @@
   )
   dhcp_options ['dhcp-host=01:23:ab:cd:01:02,larry,10.0.0.10']
   dhcp_config(
-    'dhcp-range' => 'eth1,10.0.0.5,10.0.0.15,12h',
+    'dhcp-range' => '10.0.0.5,10.0.0.15,12h',
     'domain' => 'test.lab',
     'tftp-root' => '/var/lib/tftpboot',
     'enable-tftp' => nil,
-    'interface' => 'eth1'
+    'except-interface' => 'lo'
   )
 end

test/integration/default/controls/default_spec.rb

@@ -37,10 +37,10 @@
   describe file('/etc/dnsmasq.d/dhcp.conf') do
     it { should exist }
     its('content') { should match(/^dhcp-host=01:23:ab:cd:01:02,larry,10\.0\.0\.10$/) }
-    its('content') { should match(/^dhcp-range=eth1,10\.0\.0\.5,10\.0\.0\.15,12h$/) }
+    its('content') { should match(/^dhcp-range=10\.0\.0\.5,10\.0\.0\.15,12h$/) }
     its('content') { should match(/^domain=test.lab$/) }
     its('content') { should match(/^enable-tftp$/) }
-    its('content') { should match(/^interface=eth1$/) }
+    its('content') { should match(/^except-interface=lo$/) }
     its('content') { should match(%r{^tftp-root=/var/lib/tftpboot$}) }
   end