Another attempt at fixing the switch dependencies - ports and bridges cannot be classes, and Vs_* deps in place for vswitch rather than the wrappers

manifests/bridge.pp

@@ -1,16 +1,15 @@
-class vswitch::bridge (
+define vswitch::bridge (
-  $name,
   $external_ids = "",
   $ensure = "present"
 ) {
   if $external_ids == "" {
     vs_bridge { $name:
       ensure       => $ensure
-    } <- Class['vswitch']
+    }
   } else {
     vs_bridge { $name:
       external_ids => $external_ids,
       ensure       => $ensure
-    } <- Class['vswitch']
+    }
   }
 }

manifests/ovs.pp

@@ -20,4 +20,7 @@ class vswitch::ovs(
     hasstatus   => true,
     status      => "/etc/init.d/openvswitch-switch status",
   }
+
+  Service['openvswitch-switch'] -> Vs_port<||>
+  Service['openvswitch-switch'] -> Vs_bridge<||>
 }

manifests/port.pp

@@ -1,10 +1,9 @@
-class vswitch::port (
+define vswitch::port (
-  $interface,
   $bridge,
   $ensure = present
 ) {
-  vs_port { $interface:
+  vs_port { $name:
     bridge   => $bridge,
     ensure   => $ensure
-  } <- Class['vswitch']
+  }
 }