Merge pull request #146 from mprahl/cherry-pick-ci-fix

Cherry-pick of (kubeflow#11714): Ignore empty PULL_NUMBER environment variables in SDK tests

Author: openshift-merge-bot[bot]

test/kfp-kubernetes-execution-tests/sdk_execution_tests.py

@@ -111,7 +111,7 @@ def run(test_case: TestCase) -> Tuple[str, client.client.RunPipelineResult]:
 
 def get_kfp_package_path() -> str:
     repo_name = os.environ.get('REPO_NAME', 'kubeflow/pipelines')
-    if os.environ.get('PULL_NUMBER') is not None:
+    if os.environ.get('PULL_NUMBER'):
         path = f'git+https://github.com/{repo_name}.git@refs/pull/{os.environ["PULL_NUMBER"]}/merge#subdirectory=sdk/python'
     else:
         path = f'git+https://github.com/{repo_name}.git@master#subdirectory=sdk/python'

test/sdk-execution-tests/sdk_execution_tests.py

@@ -100,7 +100,7 @@ def run(test_case: TestCase) -> Tuple[str, client.client.RunPipelineResult]:
 
 def get_kfp_package_path() -> str:
     repo_name = os.environ.get('REPO_NAME', 'kubeflow/pipelines')
-    if os.environ.get('PULL_NUMBER') is not None:
+    if os.environ.get('PULL_NUMBER'):
         path = f'git+https://github.com/{repo_name}.git@refs/pull/{os.environ["PULL_NUMBER"]}/merge#subdirectory=sdk/python'
     else:
         path = f'git+https://github.com/{repo_name}.git@master#subdirectory=sdk/python'