Gate mesh gateway resources on the consul class

hieradata/country/au/region/drw1/infra/storage/consul.yaml

@@ -2,6 +2,6 @@
 profiles::consul::server::bootstrap_count: 3
 profiles::consul::server::raft_multiplier: 10
 profiles::consul::server::primary_datacenter: 'au-syd1'
-profiles::consul::server::join_remote_regions: true
+profiles::consul::server::join_remote_regions: false
 profiles::consul::server::remote_regions:
   - syd1

hieradata/country/au/region/syd1/infra/storage/consul.yaml

@@ -2,6 +2,6 @@
 profiles::consul::server::bootstrap_count: 3
 profiles::consul::server::raft_multiplier: 10
 profiles::consul::server::primary_datacenter: 'au-syd1'
-profiles::consul::server::join_remote_regions: true
+profiles::consul::server::join_remote_regions: false
 profiles::consul::server::remote_regions:
   - drw1

hieradata/nodes/aubne1nxvm2132.main.unkin.net.yaml

@@ -1,4 +0,0 @@
----
-profiles::postfix::gateway::myhostname: 'mta2.edge.unkin.net'
-profiles::pki::vault::alt_names:
-  - mta2.edge.unkin.net

hieradata/nodes/aubne1nxvm2133.main.unkin.net.yaml

@@ -1,25 +0,0 @@
----
-networking_loopback0_ip: 198.18.2.162
-networking_wg_link: 198.18.2.178/31
-networking_vpc_iface: eth1
-networking_vpc_cidr: 172.18.32.0/24
-
-# wg0 p2p to prodnxsr0020; lab routes stay pinned to wg0 sourced from the /31 until ospf learns them
-wireguard::interfaces:
-  wg0:
-    addresses:
-      - 198.18.2.179/31
-    listen_port: 51820
-    table: 'off'
-    post_up:
-      - ip route replace 198.18.0.0/16 dev wg0 src 198.18.2.179
-      - ip route replace 10.10.12.200/32 dev wg0 src 198.18.2.179
-    private_key: ''
-    peers:
-      - public_key: ''
-        endpoint: 103.216.191.185:51822
-        allowed_ips:
-          - 224.0.0.5/32
-          - 198.18.0.0/16
-          - 10.10.12.200/32
-        persistent_keepalive: 25

hieradata/nodes/ausyd2nxvm2130.main.unkin.net.yaml

@@ -1,4 +0,0 @@
----
-profiles::postfix::gateway::myhostname: 'mta1.edge.unkin.net'
-profiles::pki::vault::alt_names:
-  - mta1.edge.unkin.net

hieradata/nodes/ausyd2nxvm2131.main.unkin.net.yaml

@@ -1,25 +0,0 @@
----
-networking_loopback0_ip: 198.18.2.161
-networking_wg_link: 198.18.2.176/31
-networking_vpc_iface: eth1
-networking_vpc_cidr: 172.18.30.0/24
-
-# wg0 p2p to prodnxsr0020; lab routes stay pinned to wg0 sourced from the /31 until ospf learns them
-wireguard::interfaces:
-  wg0:
-    addresses:
-      - 198.18.2.177/31
-    listen_port: 51820
-    table: 'off'
-    post_up:
-      - ip route replace 198.18.0.0/16 dev wg0 src 198.18.2.177
-      - ip route replace 10.10.12.200/32 dev wg0 src 198.18.2.177
-    private_key: ''
-    peers:
-      - public_key: ''
-        endpoint: 103.216.191.185:51821
-        allowed_ips:
-          - 224.0.0.5/32
-          - 198.18.0.0/16
-          - 10.10.12.200/32
-        persistent_keepalive: 25

hieradata/os/AlmaLinux/all_releases.yaml

@@ -147,18 +147,6 @@ profiles::yum::global::repos:
     baseurl: https://artifactapi.k8s.syd1.au.unkin.net/api/v1/remote/postgresql/common/redhat/rhel-%{facts.os.release.major}-%{facts.os.architecture}
     gpgkey: https://artifactapi.k8s.syd1.au.unkin.net/api/v1/remote/postgresql/keys/PGDG-RPM-GPG-KEY-RHEL
 
-  # Rspamd repositories
-  rspamd:
-    ensure: absent
-    name: rspamd
-    descr: rspamd repository
-    target: /etc/yum.repos.d/rspamd.repo
-    baseurl: https://artifactapi.k8s.syd1.au.unkin.net/api/v1/remote/rspamd/rpm-stable/centos-%{facts.os.release.major}/x86_64
-    gpgkey: https://artifactapi.k8s.syd1.au.unkin.net/api/v1/remote/rspamd/rpm/rspamd.asc
-    gpgcheck: 1
-    repo_gpgcheck: 1
-    mirrorlist: absent
-
   # Ceph repositories
   ceph:
     ensure: absent

hieradata/roles/infra/mail/gateway.yaml

@@ -1,90 +1,25 @@
 ---
-hiera_include:
-  - redis
 
-# no consul datacentre at the edge, so nothing to register or scrape via consul
-profiles::consul::client::enable: false
-exporters::node_exporter::enable: false
-
-# ssh only from the edge vpcs (syd, mel, bne) and the lab over the vpn route
-ssh::server::options:
-  AllowUsers: '*@172.18.30.0/24 *@172.18.31.0/24 *@172.18.32.0/24 *@198.18.0.0/16 *@10.10.12.200/32'
-
-# local caching resolver for postfix and rspamd, forwarding to the lab resolver over the vpn route
-dnsmasq::no_resolv: true
-dnsmasq::cache_size: 10000
-dnsmasq::servers:
-  - 198.18.200.7
-profiles::dns::base::nameservers:
-  - 127.0.0.1
-
-profiles::yum::global::repos:
-  rspamd:
-    ensure: present
-
-# local redis for rspamd bayes and history; bayes tokens carry a ttl so volatile-ttl can evict
-redis::maxmemory: '128mb'
-redis::maxmemory_policy: 'volatile-ttl'
-
-# local rspamd, deterministic checks only; module defaults cover spf/dkim/dmarc/arc/rbl/phishing/mime_types/fuzzy_check
-profiles::rspamd::edge::rules:
-  edge.cf: 'https://git.unkin.net/unkin/rspamd-rules/raw/branch/main/edge.cf'
-profiles::rspamd::edge::local_d:
-  worker-proxy.inc: |
-    bind_socket = "127.0.0.1:11332";
-    milter = yes;
-    timeout = 120s;
-    count = 1;
-    upstream "local" {
-      default = yes;
-      self_scan = yes;
-    }
-  worker-normal.inc: |
-    enabled = false;
-  redis.conf: |
-    servers = "127.0.0.1:6379";
-  actions.conf: |
-    reject = 13;
-    add_header = 6;
-    rewrite_subject = null;
-    greylist = null;
-  greylist.conf: |
-    enabled = false;
-  gpt.conf: |
-    enabled = false;
-  classifier-bayes.conf: |
-    backend = "redis";
-    expire = 8640000;
-    autolearn {
-      spam_threshold = 6.0;
-      ham_threshold = -0.5;
-      check_balance = true;
-    }
-  spamassassin.conf: |
-    ruleset = "/etc/rspamd/rules/*.cf";
-    alpha = 0.1;
-  settings.conf: |
-    smarthost_outbound {
-      priority = high;
-      ip = "103.216.191.185";
-      want_spam = yes;
-    }
+# additional altnames
+profiles::pki::vault::alt_names:
+  - in-mta.main.unkin.net
 
 # base postfix configuration (passed to postfix class)
 postfix::relayhost: 'direct'
-postfix::myorigin: 'unkin.net'
+postfix::myorigin: 'main.unkin.net'
 postfix::mydestination: 'blank'
-postfix::mynetworks: '127.0.0.0/8 [::1]/128 103.216.191.185/32'
+postfix::mynetworks: '127.0.0.0/8 [::1]/128'
 postfix::mta: true
 postfix::manage_aliases: true
 
+# profile parameters for customization
+profiles::postfix::gateway::myhostname: 'in-mta.main.unkin.net'
+
 # postfix map content (templates)
 profiles::postfix::gateway::relay_recipients_maps:
-  '@unkin.net': 'OK'
   '@main.unkin.net': 'OK'
 
 profiles::postfix::gateway::relay_domains_maps:
-  'unkin.net': 'OK'
   'main.unkin.net': 'OK'
 
 profiles::postfix::gateway::postscreen_access_maps:
@@ -97,28 +32,21 @@ profiles::postfix::gateway::helo_access_maps:
   'unknown': 'REJECT'
   'localhost': 'REJECT You are not localhost'
 
-profiles::postfix::gateway::smtp_tls_policy_maps:
-  '[mail-in.main.unkin.net]:25': 'secure match=hostname'
-
 # postfix transports
 postfix::transports:
-  'unkin.net':
-    ensure: present
-    destination: 'relay'
-    nexthop: '[mail-in.main.unkin.net]:25'
   'main.unkin.net':
     ensure: present
     destination: 'relay'
-    nexthop: '[mail-in.main.unkin.net]:25'
+    nexthop: 'mail-in.main.unkin.net:25'
 
 # postfix virtuals
 postfix::virtuals:
   'root':
     ensure: present
-    destination: 'ben@unkin.net'
+    destination: 'ben@main.unkin.net'
   'postmaster':
     ensure: present
-    destination: 'ben@unkin.net'
+    destination: 'ben@main.unkin.net'
   'abuse':
     ensure: present
-    destination: 'ben@unkin.net'
+    destination: 'ben@main.unkin.net'

hieradata/roles/infra/network/wgvpn.yaml

@@ -1,41 +0,0 @@
----
-hiera_include:
-  - wireguard
-  - frrouting
-
-# no consul datacentre at the edge, so nothing to register or scrape via consul
-profiles::consul::client::enable: false
-exporters::node_exporter::enable: false
-
-# ssh only from the lab over the vpn
-ssh::server::options:
-  AllowUsers: '*@198.18.0.0/16 *@10.10.12.200/32'
-
-# routing
-sysctl::base::values:
-  net.ipv4.ip_forward:
-    value: '1'
-
-# frrouting: p2p ospf over wg0; loopback /32, wg /31 and vpc /24 only, no default route in or out
-frrouting::daemons:
-  ospfd: true
-frrouting::ospfd_router_id: "%{hiera('networking_loopback0_ip')}"
-frrouting::ospfd_interfaces:
-  wg0:
-    area: 0.0.0.0
-    network: point-to-point
-  dum0:
-    area: 0.0.0.0
-    passive: true
-  "%{hiera('networking_vpc_iface')}":
-    area: 0.0.0.0
-    passive: true
-frrouting::ospfd_redistribute:
-  - connected
-frrouting::ospfd_redistribute_connected_permit:
-  - "%{hiera('networking_loopback0_ip')}/32"
-  - "%{hiera('networking_wg_link')}"
-  - "%{hiera('networking_vpc_cidr')}"
-frrouting::ospfd_deny_default_in: true
-frrouting::ospf_preferred_source_enable: true
-frrouting::ospf_preferred_source: "%{hiera('networking_loopback0_ip')}"

hieradata/roles/infra/storage/consul.yaml

@@ -13,9 +13,19 @@ profiles::consul::server::addresses:
   grpc_tls: "%{::networking.ip}"
 profiles::consul::server::ports:
   grpc: 8502
+  grpc_tls: 8503
   dns: 8600
   http: 8500
   https: -1
+profiles::consul::server::tls:
+  defaults:
+    ca_file: /etc/pki/ca-trust/source/anchors/vaultcaroot.pem
+    cert_file: /etc/pki/tls/vault/full_chain.crt
+    key_file: /etc/pki/tls/vault/private.key
+  internal_rpc:
+    verify_incoming: false
+    verify_outgoing: false
+    verify_server_hostname: false
 profiles::consul::server::acl:
   enabled: true
   default_policy: 'deny'
@@ -48,6 +58,18 @@ profiles::consul::client::node_rules:
   - resource: service
     segment: frr_exporter
     disposition: write
+  - resource: service
+    segment: mesh-gateway
+    disposition: write
+  - resource: service_prefix
+    segment: ''
+    disposition: read
+  - resource: node_prefix
+    segment: ''
+    disposition: read
+  - resource: agent_prefix
+    segment: ''
+    disposition: read
 profiles::consul::prepared_query::rules:
   vault:
     ensure: 'present'

modules/frrouting/manifests/init.pp

@@ -10,8 +10,6 @@ class frrouting (
   Array[String] $ospfd_redistribute = [],
   Array[String] $ospfd_networks     = [],
   Boolean $ospfd_default_originate_always = false,
-  Array[Stdlib::IP::Address::V4::CIDR] $ospfd_redistribute_connected_permit = [], # only these exact connected prefixes are redistributed
-  Boolean $ospfd_deny_default_in    = false, # keep a learned default route out of the routing table
   Boolean $mpls_te_enabled                   = false,
   Optional[String] $mpls_ldp_router_id       = undef,
   Optional[String] $mpls_ldp_transport_addr  = undef,

modules/frrouting/templates/frr.conf.erb

@@ -8,9 +8,6 @@ interface <%= iface %>
 <% if params['area'] -%>
  ip ospf area <%= params['area'] %>
 <% end -%>
-<% if params['network'] -%>
- ip ospf network <%= params['network'] %>
-<% end -%>
 <% if params['passive'] == true -%>
  ip ospf passive
 <% end -%>
@@ -31,27 +28,12 @@ route-map CONNECTED-NON-K8S permit 100
 exit
 <% end -%>
 
-<% if @ospfd_redistribute_connected_permit.any? -%>
-<% @ospfd_redistribute_connected_permit.each_with_index do |prefix, i| -%>
-ip prefix-list CONNECTED-PERMIT seq <%= (i + 1) * 5 %> permit <%= prefix %>
-<% end -%>
-route-map CONNECTED-PERMIT permit 10
- match ip address prefix-list CONNECTED-PERMIT
-exit
-<% end -%>
-<% if @ospfd_deny_default_in -%>
-access-list NO-DEFAULT seq 5 deny 0.0.0.0/0 exact-match
-access-list NO-DEFAULT seq 10 permit any
-<% end -%>
-
 router ospf
  ospf router-id <%= @ospfd_router_id %>
  log-adjacency-changes detail
 <% @ospfd_redistribute.each do |type| -%>
 <% if @ospf_exclude_k8s_enable && type == 'connected' && (@k8s_cluster_cidr || @k8s_service_cidr) -%>
  redistribute connected route-map CONNECTED-NON-K8S
-<% elsif type == 'connected' && @ospfd_redistribute_connected_permit.any? -%>
- redistribute connected route-map CONNECTED-PERMIT
 <% else -%>
  redistribute <%= type %>
 <% end -%>
@@ -59,9 +41,6 @@ router ospf
 <% @ospfd_networks.each do |network| -%>
  network <%= network %>
 <% end -%>
-<% if @ospfd_deny_default_in -%>
- distribute-list NO-DEFAULT in
-<% end -%>
 <% if @ospfd_default_originate_always -%>
  default-information originate always
 <% end -%>

modules/libs/lib/facter/subnet_facts.rb

@@ -22,46 +22,27 @@ class SubnetAttributes
     '198.18.26.0/24' => { environment: 'prod', region: 'syd1', country: 'au', zone: 'common' }, # common node0010
     '198.18.27.0/24' => { environment: 'prod', region: 'syd1', country: 'au', zone: 'common' }, # common node0011
     '198.18.28.0/24' => { environment: 'prod', region: 'syd1', country: 'au', zone: 'common' }, # common node0012
-    '198.18.29.0/24' => { environment: 'prod', region: 'syd1', country: 'au', zone: 'common' }, # common node0013
-    '172.18.30.0/24' => { environment: 'prod', region: 'syd2', country: 'au', zone: 'dmz' },    # binarylane syd dmz vpc
-    '172.18.31.0/24' => { environment: 'prod', region: 'mel1', country: 'au', zone: 'dmz' },    # binarylane mel dmz vpc
-    '172.18.32.0/24' => { environment: 'prod', region: 'bne1', country: 'au', zone: 'dmz' }     # binarylane bne dmz vpc
+    '198.18.29.0/24' => { environment: 'prod', region: 'syd1', country: 'au', zone: 'common' }  # common node0013
   }.freeze
 
   # Default attributes if no subnet matches, also defined as a constant
   DEFAULT_ATTRIBUTES = { environment: 'unknown', region: 'unknown', country: 'unknown' }.freeze
 
-  # loopbacks and wg /31s sit in syd1 ranges, so they must not decide where a host lives
-  SKIPPED_INTERFACES = /\A(lo\z|dum|wg)/.freeze
-
-  # primary ip first, then the remaining interfaces in name order
-  def self.from_networking(networking)
-    match(networking['ip']) || interface_match(networking['interfaces'] || {}) || DEFAULT_ATTRIBUTES
-  end
-
-  def self.interface_match(interfaces)
-    interfaces.sort
-              .reject { |name, _interface| name.match?(SKIPPED_INTERFACES) }
-              .flat_map { |_name, interface| addresses(interface) }
-              .lazy.filter_map { |address| match(address) }.first
-  end
-
-  def self.addresses(interface)
-    (interface['bindings'] || []).map { |binding| binding['address'] }
-  end
-
-  def self.match(ip)
-    return unless ip
-
-    addr = IPAddr.new(ip)
+  # provide ip to return attributes
+  def self.attributes(ip)
     SUBNET_TO_ATTRIBUTES.each do |subnet, attrs|
-      return attrs if IPAddr.new(subnet).include?(addr)
+      return attrs if IPAddr.new(subnet).include?(IPAddr.new(ip))
     end
-    nil
+
+    DEFAULT_ATTRIBUTES
   end
 end
 
-subnet_attributes = SubnetAttributes.from_networking(Facter.value(:networking))
+# Use the primary IP address from facts
+ip = Facter.value(:networking)['ip']
+
+# Call the class method directly without creating an instance
+subnet_attributes = SubnetAttributes.attributes(ip)
 
 # Add separate facts for environment, region, and country
 Facter.add('environment') { setcode { subnet_attributes[:environment] } }

modules/libs/test/subnet_facts_check.rb

@@ -1,36 +0,0 @@
-# frozen_string_literal: true
-
-# ruby modules/libs/test/subnet_facts_check.rb
-module Facter
-  def self.value(_name)
-    {}
-  end
-
-  def self.add(*); end
-end
-load File.expand_path('../lib/facter/subnet_facts.rb', __dir__)
-
-iface = ->(*addresses) { { 'bindings' => addresses.map { |address| { 'address' => address } } } }
-
-checks = {
-  'primary ip wins over other interfaces' => [
-    { 'ip' => '198.18.17.5', 'interfaces' => { 'eth1' => iface.call('198.18.13.5') } }, 'drw1'
-  ],
-  'unmatched primary ip falls back to an interface' => [
-    { 'ip' => '203.0.113.5', 'interfaces' => { 'eth0' => iface.call('203.0.113.5'), 'eth1' => iface.call('198.18.24.11') } }, 'syd1'
-  ],
-  'lo, dum and wg interfaces are skipped' => [
-    { 'ip' => '203.0.113.5',
-      'interfaces' => { 'dum0' => iface.call('198.18.2.161'), 'lo' => iface.call('198.18.19.1'),
-                        'wg0' => iface.call('198.18.2.177') } }, 'unknown'
-  ],
-  'interfaces are scanned in name order' => [
-    { 'ip' => '203.0.113.5', 'interfaces' => { 'eth2' => iface.call('198.18.24.1'), 'eth1' => iface.call('198.18.17.1') } }, 'drw1'
-  ],
-  'missing ip and interfaces resolve to unknown' => [{}, 'unknown']
-}
-
-failures = checks.reject { |_name, (networking, expected)| SubnetAttributes.from_networking(networking)[:region] == expected }
-failures.each_key { |name| warn "FAIL: #{name}" }
-abort if failures.any?
-puts "ok (#{checks.size} checks)"

modules/wireguard/manifests/interface.pp

@@ -4,8 +4,6 @@ define wireguard::interface (
   Optional[Stdlib::Port]               $listen_port = undef,
   Optional[Integer[1280, 9000]]        $mtu         = undef,
   Optional[Sensitive[String[1]]]       $private_key = undef,
-  Optional[String[1]]                  $table       = undef,
-  Array[String[1]]                     $post_up     = [],
   Array[Struct[{
     public_key           => String[1],
     allowed_ips          => Variant[String[1], Array[String[1], 1]],

modules/wireguard/templates/wg.conf.erb

@@ -9,17 +9,11 @@ ListenPort = <%= @listen_port %>
 <% if @mtu -%>
 MTU = <%= @mtu %>
 <% end -%>
-<% if @table -%>
-Table = <%= @table %>
-<% end -%>
 <% if @key -%>
 PrivateKey = <%= @key %>
 <% else -%>
 PostUp = wg set %i private-key /etc/wireguard/%i.key
 <% end -%>
-<% @post_up.each do |cmd| -%>
-PostUp = <%= cmd %>
-<% end -%>
 <% @peers.each do |peer| -%>
 
 [Peer]

site/profiles/manifests/consul/client.pp

@@ -12,13 +12,8 @@ class profiles::consul::client (
   Array[Hash]          $node_rules         = [],
   Hash                 $ports              = {},
   Stdlib::IP::Address  $host_addr          = $facts['networking']['ip'],
-  Boolean              $enable             = true,
 ) {
 
-  unless $enable {
-    return()
-  }
-
   if $facts['enc_role'] != $members_role {
 
     # set a datacentre/cluster name

site/profiles/manifests/consul/mesh_gateway.pp

@@ -0,0 +1,52 @@
+# profiles::consul::mesh_gateway
+class profiles::consul::mesh_gateway (
+  Stdlib::IP::Address $address    = $facts['networking']['ip'],
+  Stdlib::Port        $port       = 8443,
+  Stdlib::Port        $http_port  = 8500,
+  Stdlib::Port        $grpc_port  = 8502,
+  Stdlib::Absolutepath $token_file = '/etc/consul.d/mesh-gateway.token',
+) {
+  include profiles::consul::client
+
+  # include after profiles::consul::server, which declares Class['consul'] only once /data is mounted
+  if defined(Class['consul']) {
+    package { 'envoy':
+      ensure => installed,
+    }
+
+    file { $token_file:
+      ensure  => file,
+      owner   => 'consul',
+      group   => 'consul',
+      mode    => '0600',
+      require => Class['consul'],
+      content => Sensitive(fqdn_uuid("${facts['networking']['fqdn']}-${profiles::consul::client::secret_id_salt}")),
+    }
+
+    systemd::unit_file { 'consul-mesh-gateway.service':
+      enable    => true,
+      active    => true,
+      content   => @("UNIT"),
+        [Unit]
+        Description=Consul mesh gateway (envoy)
+        Requires=consul.service
+        After=consul.service
+
+        [Service]
+        User=consul
+        Group=consul
+        ExecStart=/usr/bin/consul connect envoy -gateway=mesh -register -service=mesh-gateway -expose-servers \
+          -address=${address}:${port} -wan-address=${address}:${port} \
+          -http-addr=http://${address}:${http_port} -grpc-addr=http://${address}:${grpc_port} \
+          -token-file=${token_file}
+        Restart=on-failure
+        RestartSec=5
+
+        [Install]
+        WantedBy=multi-user.target
+        | UNIT
+      require   => [Package['envoy'], Class['consul']],
+      subscribe => File[$token_file],
+    }
+  }
+}

site/profiles/manifests/consul/server.pp

@@ -11,6 +11,7 @@ class profiles::consul::server (
   Hash      $acl = {},
   Hash      $ports = {},
   Hash      $addresses = {},
+  Hash      $tls = {},
   Boolean   $members_lookup     = false,
   String    $members_role       = undef,
   Array     $consul_servers     = [],
@@ -112,6 +113,8 @@ class profiles::consul::server (
           'acl'                  => $acl,
           'ports'                => $ports,
           'addresses'            => $addresses,
+          'tls'                  => $tls,
+          'auto_reload_config'   => true,
           'disable_remote_exec'  => $disable_remote_exec,
           'disable_update_check' => $disable_update_check,
           'domain'               => $domain,
@@ -129,7 +132,7 @@ class profiles::consul::server (
           'advertise_addr'       => $advertise_addr,
           'retry_join'           => $servers_array,
           'retry_join_wan'       => $remote_servers_array,
-          'connect'              => { 'enabled' => true },
+          'connect'              => { 'enabled' => true, 'enable_mesh_gateway_wan_federation' => true },
           'recursors'            => ['198.18.19.16'],
         },
       }

site/profiles/manifests/postfix/gateway.pp

@@ -3,7 +3,8 @@ class profiles::postfix::gateway (
   Stdlib::Absolutepath $tls_key_file  = '/etc/pki/tls/vault/certificate.pem',
   Stdlib::Absolutepath $tls_ca_file   = '/etc/pki/ca-trust/extracted/pem/tls-ca-bundle.pem',
   String $myhostname                  = $trusted['certname'],
-  String $message_size_limit          = '52428800',
+  String $message_size_limit          = '133169152',
+  String $mailbox_size_limit          = '133169152',
   String $local_transport             = 'error:No local mail delivery',
   Boolean $enable_postscreen          = true,
   Array[String] $alias_maps           = [
@@ -11,12 +12,12 @@ class profiles::postfix::gateway (
     'hash:/etc/postfix/aliases',
   ],
   Array[String] $postscreen_dnsbl_sites = [
-    'zen.spamhaus.org=127.0.0.[2..11]*3',
-    'b.barracudacentral.org=127.0.0.2*2',
-    'bl.mailspike.net=127.0.0.2*1',
-    'psbl.surriel.com=127.0.0.2*1',
-    'bl.spameatingmonkey.net=127.0.0.2*2',
-    'bl.spamcop.net=127.0.0.2',
+    'zen.spamhaus.org*3',
+    'b.barracudacentral.org=127.0.0.[2..11]*2',
+    'bl.spameatingmonkey.net*2',
+    'bl.spamcop.net',
+    'dnsbl.sorbs.net',
+    'swl.spamhaus.org*-4',
     'list.dnswl.org=127.[0..255].[0..255].0*-2',
     'list.dnswl.org=127.[0..255].[0..255].1*-4',
     'list.dnswl.org=127.[0..255].[0..255].[2..3]*-6',
@@ -24,6 +25,7 @@ class profiles::postfix::gateway (
   Array[String] $smtpd_client_restrictions = [
     'permit_sasl_authenticated',
     'permit_mynetworks',
+    'reject_rbl_client zen.spamhaus.org',
   ],
   Array[String] $smtpd_sender_restrictions = [
     'permit_sasl_authenticated',
@@ -45,7 +47,6 @@ class profiles::postfix::gateway (
     'permit_mynetworks',
     'reject_unauth_destination',
   ],
-  String $smtpd_milters = 'inet:127.0.0.1:11332',
   Hash[String, String] $smtp_tls_policy_maps = {},
   Hash[String, String] $sender_canonical_maps = {},
   Hash[String, String] $sender_access_maps = {},
@@ -72,7 +73,7 @@ class profiles::postfix::gateway (
         'value' => 'permit_mynetworks, cidr:/etc/postfix/postscreen_access'
       },
       'postscreen_blacklist_action' => {
-        'value' => 'drop'
+        'value' => 'enforce'
       },
       'postscreen_cache_map' => {
         'value' => 'btree:$data_directory/postscreen_cache'
@@ -86,10 +87,6 @@ class profiles::postfix::gateway (
       'postscreen_dnsbl_threshold' => {
         'value' => '2'
       },
-      # dnswl-listed clients (net score <= -1) skip the slow greet tests; el9 postfix 3.5 predates the allowlist rename
-      'postscreen_dnsbl_whitelist_threshold' => {
-        'value' => '-1'
-      },
       'postscreen_greet_action' => {
         'value' => 'enforce'
       },
@@ -111,6 +108,9 @@ class profiles::postfix::gateway (
     'alias_database' => {
       'value' => $alias_maps_string
     },
+    'default_destination_recipient_limit' => {
+      'value' => '1'
+    },
     'disable_vrfy_command' => {
       'value' => 'yes'
     },
@@ -129,6 +129,9 @@ class profiles::postfix::gateway (
     'local_transport' => {
       'value' => $local_transport
     },
+    'mailbox_size_limit' => {
+      'value' => $mailbox_size_limit
+    },
     'message_size_limit' => {
       'value' => $message_size_limit
     },
@@ -138,20 +141,8 @@ class profiles::postfix::gateway (
     'non_smtpd_milters' => {
       'ensure' => 'blank'
     },
-    'milter_connect_timeout' => {
-      'value' => '10s'
-    },
-    'milter_content_timeout' => {
-      'value' => '60s'
-    },
-    'milter_default_action' => {
-      'value' => 'tempfail'
-    },
-    'milter_mail_macros' => {
-      'value' => 'i {mail_addr} {client_addr} {client_name} {auth_authen}'
-    },
-    'milter_protocol' => {
-      'value' => '6'
+    'qmqpd_authorized_clients' => {
+      'value' => '127.0.0.1 [::1]'
     },
     'recipient_canonical_maps' => {
       'value' => 'hash:/etc/postfix/recipient_canonical'
@@ -186,7 +177,7 @@ class profiles::postfix::gateway (
     'smtp_tls_session_cache_database' => {
       'value' => 'btree:/var/lib/postfix/smtp_tls_session_cache'
     },
-    'smtp_send_xforward_command' => {
+    'smtp_use_tls' => {
       'value' => 'yes'
     },
     'smtpd_banner' => {
@@ -220,7 +211,7 @@ class profiles::postfix::gateway (
       'value' => 'check_helo_access hash:/etc/postfix/helo_access, reject_invalid_hostname'
     },
     'smtpd_milters' => {
-      'value' => $smtpd_milters
+      'value' => 'inet:127.0.0.1:33333'
     },
     'smtpd_recipient_restrictions' => {
       'value' => join($smtpd_recipient_restrictions, ', ')
@@ -264,10 +255,13 @@ class profiles::postfix::gateway (
     'smtpd_tls_session_cache_timeout' => {
       'value' => '3600s'
     },
+    'smtpd_use_tls' => {
+      'value' => 'yes'
+    },
     'tls_medium_cipherlist' => {
       'value' => join([
         'ECDSA+AESGCM:ECDH+AESGCM:DH+AESGCM:ECDSA+AES:ECDH+AES:DH+AES',
-        'RSA+AESGCM:RSA+AES:!aNULL:!MD5:!DSS:!3DES'
+        'ECDH+3DES:DH+3DES:RSA+AESGCM:RSA+AES:RSA+3DES:!aNULL:!MD5:!DSS'
       ], ':')
     },
     'tls_preempt_cipherlist' => {

site/profiles/manifests/rspamd/edge.pp

@@ -1,68 +0,0 @@
-# local rspamd milter for the mail edge gateways
-# - local_d: /etc/rspamd/local.d file name => content
-# - rules: /etc/rspamd/rules file name => url, re-fetched every puppet run; fetch failures keep the last good copy
-class profiles::rspamd::edge (
-  Hash[String, String]           $local_d = {},
-  Hash[String, Stdlib::HTTPSUrl] $rules   = {},
-) {
-
-  package { 'rspamd':
-    ensure => installed,
-  }
-
-  $local_d.each |$name, $content| {
-    file { "/etc/rspamd/local.d/${name}":
-      ensure  => file,
-      owner   => 'root',
-      group   => 'root',
-      mode    => '0644',
-      content => $content,
-      require => Package['rspamd'],
-      notify  => Service['rspamd'],
-    }
-  }
-
-  file { '/etc/rspamd/rules':
-    ensure  => directory,
-    owner   => 'root',
-    group   => 'root',
-    mode    => '0755',
-    require => Package['rspamd'],
-  }
-
-  $rules.each |$name, $url| {
-    $path = "/etc/rspamd/rules/${name}"
-
-    # empty placeholder so rspamd starts before the first successful fetch
-    file { $path:
-      ensure  => file,
-      owner   => 'root',
-      group   => 'root',
-      mode    => '0644',
-      content => '',
-      replace => false,
-    }
-
-    # a failed or unchanged fetch skips the swap without failing the run
-    exec { "rspamd_rules_fetch_${name}":
-      command => "/usr/bin/mv -f ${path}.new ${path}",
-      onlyif  => "/usr/bin/curl -fsS --max-time 30 -o ${path}.new ${url} && ! /usr/bin/cmp -s ${path}.new ${path}",
-      require => File[$path],
-      notify  => Exec['rspamd_reload'],
-    }
-  }
-
-  # reload keeps the milter answering while config changes apply
-  service { 'rspamd':
-    ensure  => running,
-    enable  => true,
-    restart => '/usr/bin/systemctl reload rspamd',
-    require => Package['rspamd'],
-  }
-
-  exec { 'rspamd_reload':
-    command     => '/usr/bin/systemctl reload rspamd',
-    refreshonly => true,
-    require     => Service['rspamd'],
-  }
-}

site/roles/manifests/infra/mail/gateway.pp

@@ -6,11 +6,6 @@ class roles::infra::mail::gateway {
   }else{
     include profiles::defaults
     include profiles::base
-    include dnsmasq
-    include profiles::rspamd::edge
     include profiles::postfix::gateway
-
-    # resolv.conf points at dnsmasq, so it must be running first
-    Class['dnsmasq'] -> Class['profiles::dns::resolvconf']
   }
 }

site/roles/manifests/infra/network/wgvpn.pp

@@ -1,12 +0,0 @@
-# roles::infra::network::wgvpn
-# a wireguard vpn edge host; routes its vpc to the lab over wg0 with ospf
-#
-class roles::infra::network::wgvpn {
-  if $facts['firstrun'] {
-    include profiles::defaults
-    include profiles::firstrun::init
-  }else{
-    include profiles::defaults
-    include profiles::base
-  }
-}

site/roles/manifests/infra/storage/consul.pp

@@ -8,5 +8,6 @@ class roles::infra::storage::consul {
     include profiles::base
     include profiles::base::datavol
     include profiles::consul::server
+    include profiles::consul::mesh_gateway
   }
 }