Fix for Python Client e2e test to ensure the cleanup command exits cleanly (kubeflow#983)

* fix(python-client): ensure the cleanup command exits

Signed-off-by: Eric Dobroveanu <[email protected]>

* chore: update gitignore with venv

Signed-off-by: Eric Dobroveanu <[email protected]>

---------

Signed-off-by: Eric Dobroveanu <[email protected]>

Author: Crazyglue

clients/python/.gitignore

@@ -5,3 +5,4 @@
 /.nox/
 /.python-version
 __pycache__/
+venv/

scripts/cleanup.sh

@@ -28,8 +28,11 @@ if [[ -n "$LOCAL" ]]; then
 
     sqlite3 test/config/ml-metadata/metadata.sqlite.db <<<"BEGIN TRANSACTION; $PARTIAL_SQL_CMD"
 else
-    echo 'Cleaning up kubernetes MySQL DB'
+    echo -n 'Cleaning up kubernetes MySQL DB...'
 
-    kubectl exec -n "$MR_NAMESPACE" -it "$(kubectl get pods -l component=db -o jsonpath="{.items[0].metadata.name}" -n "$MR_NAMESPACE")" \
-        -- mysql -u root -ptest -D "$TEST_DB_NAME" -e "START TRANSACTION; $PARTIAL_SQL_CMD"
+    kubectl exec -n "$MR_NAMESPACE" \
+        "$(kubectl get pods -l component=db -o jsonpath="{.items[0].metadata.name}" -n "$MR_NAMESPACE")" \
+        -- mysql -u root -ptest -D "$TEST_DB_NAME" -e "START TRANSACTION; $PARTIAL_SQL_CMD; COMMIT;"
+
+    echo -n 'Done cleaning up kubernetes MySQL DB'
 fi