chore(sdk): Remove unnecessary code (kubeflow#1062)

Author: Tomcli

sdk/python/kfp_tekton/compiler/compiler.py

@@ -1956,34 +1956,3 @@ def _find_items(obj, search_key, current_path="", results_dict=dict()) -> dict:
 
   if non_k8s_annotations:
     raise RuntimeError(error_msg_tmplt % ("annotations", json.dumps(non_k8s_annotations, sort_keys=False, indent=2)))
-
-  # TODO: Tekton pipeline parameter validation
-  #   workflow = workflow.copy()
-  #   # Working around Argo lint issue
-  #   for argument in workflow['spec'].get('arguments', {}).get('parameters', []):
-  #     if 'value' not in argument:
-  #       argument['value'] = ''
-  #   yaml_text = dump_yaml(workflow)
-  #   if '{{pipelineparam' in yaml_text:
-  #     raise RuntimeError(
-  #         '''Internal compiler error: Found unresolved PipelineParam.
-  # Please create a new issue at https://github.com/kubeflow/kfp-tekton/issues
-  # attaching the pipeline code and the pipeline package.'''
-  #     )
-
-  # TODO: Tekton lint, if a tool exists for it
-  #   # Running Argo lint if available
-  #   import shutil
-  #   import subprocess
-  #   argo_path = shutil.which('argo')
-  #   if argo_path:
-  #     result = subprocess.run([argo_path, 'lint', '/dev/stdin'], input=yaml_text.encode('utf-8'),
-  #                             stdout=subprocess.PIPE, stderr=subprocess.PIPE)
-  #     if result.returncode:
-  #       raise RuntimeError(
-  #         '''Internal compiler error: Compiler has produced Argo-incompatible workflow.
-  # Please create a new issue at https://github.com/kubeflow/kfp-tekton/issues
-  # attaching the pipeline code and the pipeline package.
-  # Error: {}'''.format(result.stderr.decode('utf-8'))
-  #       )
-  pass

sdk/python/kfp_tekton/tekton.py

@@ -179,7 +179,6 @@ def CEL_ConditionOp(condition_statement):
            DEFAULT_CONDITION_OUTPUT_KEYWORD)
     ConditionOp_template = components.load_component_from_text(ConditionOp_yaml)
     ConditionOp = ConditionOp_template(condition_statement)
-    ConditionOp.add_pod_annotation("valid_container", "false")
     return ConditionOp