Don't return pip install with no packages

If an empty list is provided no command should be returned to avoid a broken command eg. `pip install `
kubeflow · Sep 14, 2021 · 519751d · 519751d
1 parent 6252b09
commit 519751d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/sdk/RELEASE.md b/sdk/RELEASE.md
@@ -11,6 +11,7 @@
 ## Deprecations
 
 ## Bug Fixes and Other Changes
+* Fix component decorator could result in invalid component if `install_kfp_package=False`. [\#6527](https://github.com/kubeflow/pipelines/pull/6527))
 
 ## Documentation Updates
 

diff --git a/sdk/python/kfp/v2/components/component_factory.py b/sdk/python/kfp/v2/components/component_factory.py
@@ -35,7 +35,7 @@ def _python_function_name_to_component_name(name):
 def _get_packages_to_install_command(
         package_list: Optional[List[str]] = None) -> List[str]:
     result = []
-    if package_list is not None:
+    if package_list:
         install_pip_command = 'python3 -m ensurepip'
         install_packages_command = (
             'PIP_DISABLE_PIP_VERSION_CHECK=1 python3 -m pip install --quiet \