feat(submodules): support for submodule passthru

modules/docker.nix

@@ -87,17 +87,8 @@ in {
       };
     }];
 
-    docker.export = mkMerge [
+    # list of exported docker images
-      (mapAttrsToList (_: i: i.image)
+    docker.export = mapAttrsToList (_: i: i.image)
-        (filterAttrs (_: i: i.registry != null) config.docker.images))
+      (filterAttrs (_: i: i.registry != null) config.docker.images);
-
-      # passthru of docker exported images if passthru is enabled on submodule
-      # and submodule has docker module loaded
-      (flatten (mapAttrsToList (_: submodule:
-        optionals
-          (submodule.passthru.enable && (elem "docker" submodule.config._module.features))
-          submodule.config.docker.export
-      ) config.submodules.instances))
-    ];
   };
 }

modules/k8s.nix

@@ -363,21 +363,10 @@ in {
       resources.${group}.${version}.${kind}.${name} = object;
     }) cfg.imports));
 
-    kubernetes.objects = mkMerge [
+    kubernetes.objects = flatten (map (type:
-      # versioned resources
+      mapAttrsToList (name: resource: moduleToAttrs resource)
-      (flatten (map (type:
+        cfg.api.resources.${type.group}.${type.version}.${type.kind}
-        mapAttrsToList (name: resource: moduleToAttrs resource)
+    ) cfg.api.types);
-          cfg.api.resources.${type.group}.${type.version}.${type.kind}
-      ) cfg.api.types))
-
-      # passthru of child kubernetes objects if passthru is enabled on submodule
-      # and submodule has k8s module loaded
-      (flatten (mapAttrsToList (_: submodule:
-        optionals
-          (submodule.passthru.enable && (elem "k8s" submodule.config._module.features))
-          submodule.config.kubernetes.objects
-      ) config.submodules.instances))
-    ];
 
     kubernetes.generated = k8s.mkHashedList {
       items = config.kubernetes.objects;

modules/submodule.nix

@@ -30,7 +30,7 @@ with lib;
     };
 
     passthru = mkOption {
-      description = "Attribute set to passthru to parent";
+      description = "Attribute set to passthru";
       default = {};
       type = types.attrs;
     };

modules/submodules.nix

@@ -44,6 +44,20 @@ let
       then throw "No module found ${name}/${if version == null then "latest" else version}"
       else head versionSortedSubmodules;
   in matchingModule;
+
+  passthruConfig = mapAttrsToList (name: opt: {
+    ${name} = mkMerge (mapAttrsToList (_: inst:
+      if inst.passthru.enable
+      then inst.config.submodule.passthru.${name} or {}
+      else {}
+    ) config.submodules.instances);
+
+    _module.args = mkMerge (mapAttrsToList (_: inst:
+      if inst.passthru.enable
+      then inst.config.submodule.passthru._module.args or {}
+      else {}
+    ) config.submodules.instances);
+  }) (removeAttrs options ["_definedNames" "_module" "submodules"]);
 in {
   imports = [ ./base.nix ];
 
@@ -248,5 +262,5 @@ in {
         };
       }];
     })
-  ];
+  ] ++ passthruConfig);
 }

tests/k8s/submodule.nix

@@ -26,7 +26,13 @@ in {
       imports = with kubenix.modules; [ submodule k8s docker ];
 
       config = {
-        submodule.name = "test-submodule";
+        submodule = {
+          name = "test-submodule";
+          passthru = {
+            kubernetes.objects = config.kubernetes.objects;
+            docker.images = config.docker.images;
+          };
+        };
 
         kubernetes.resources.pods.nginx = {
           metadata.name = name;