SDK: fix setting pipeline-wide artifact_location for ResourceOp and V…

…olumeOp classes and add description field for create_experiment() function (#2025)

* fix setting pipeline-wide artifact_location for ResourceOp and VolumeOp classes

* add description field for create_experiment() function

sdk/python/kfp/_client.py

@@ -172,10 +172,11 @@ def _get_url_prefix(self):
     # In-cluster pod. We could use relative URL.
     return '/pipeline'
 
-  def create_experiment(self, name):
+  def create_experiment(self, name, description=None):
     """Create a new experiment.
     Args:
       name: the name of the experiment.
+      description: description of the experiment
     Returns:
       An Experiment object. Most important field is id.
     """
@@ -189,7 +190,7 @@ def create_experiment(self, name):
 
     if not experiment:
       logging.info('Creating experiment {}.'.format(name))
-      experiment = kfp_server_api.models.ApiExperiment(name=name)
+      experiment = kfp_server_api.models.ApiExperiment(name=name, description=description)
       experiment = self._experiment_api.create_experiment(body=experiment)
 
     if self._is_ipython():

sdk/python/kfp/compiler/compiler.py

@@ -723,8 +723,9 @@ def _compile(self, pipeline_func):
     for op in p.ops.values():
       # inject pipeline level artifact location into if the op does not have
       # an artifact location config already.
-      if artifact_location and not op.artifact_location:
-        op.artifact_location = artifact_location
+      if hasattr(op, "artifact_location"):
+        if artifact_location and not op.artifact_location:
+          op.artifact_location = artifact_location
 
       sanitized_name = K8sHelper.sanitize_k8s_name(op.name)
       op.name = sanitized_name