Correct Runtime Artifact custom path logic/testing.

Signed-off-by: agoins <[email protected]>

Author: alyssacgoins

backend/src/v2/component/launcher_v2.go

@@ -945,6 +945,10 @@ func mergeRuntimeArtifacts(src, dst *pipelinespec.RuntimeArtifact) {
 			}
 		}
 	}
+
+	if src.CustomPath != nil && *src.CustomPath != "" {
+		dst.CustomPath = src.CustomPath
+	}
 }
 
 func getExecutorOutputFile(path string) (*pipelinespec.ExecutorOutput, error) {

sdk/python/kfp/dsl/types/artifact_types.py

@@ -91,7 +91,7 @@ def path(self) -> str:
 
     @path.setter
     def path(self, path: str) -> None:
-        self._set_custom_path(path)
+        self._set_path(path)
 
     def _get_path(self) -> Optional[str]:
         if self.custom_path:
@@ -114,7 +114,7 @@ def _get_path(self) -> Optional[str]:
 
     @property
     def custom_path(self) -> str:
-        return self._custom_path
+        return self._get_custom_path()
 
     def _get_custom_path(self) -> str:
         return self._custom_path
@@ -129,6 +129,10 @@ def _set_custom_path(self, value: str) -> None:
     def custom_path(self, value: str):
         self._custom_path = value
 
+    def set_path(self, path: str) -> None:
+        # If user specified a custom path, use it instead of the default path.
+        self._custom_path = path
+
 
 def convert_local_path_to_remote_path(path: str) -> str:
     if path.startswith(_GCS_LOCAL_MOUNT_PREFIX):