From 12f24a76b6fbba4ebd9d7cf09c42f2c6c6e3c8f7 Mon Sep 17 00:00:00 2001 From: Matthew Treinish Date: Fri, 16 Feb 2024 19:08:27 -0500 Subject: [PATCH] Fix pip uninstall command in qiskit-neko (#11826) The pip uninstall command used in the qiskit neko job was missing the -y flag used to force pip to run the uninstall. Instead it was failing on the prompt to confirm the uninstall and then not uninstalling an existing install because of the failure. This has been causing us to not actually test the propsed PR because it was mixing the state of qiskit < 1.0 from pypi and >=1.0. Now that 1.0 has been released we're hitting failures in CI because all the breaking API changes we introduced in 1.0 are finally being run and the test suite is and downstream packages we were integration testing with are now broken. This commit fixes the uninstall command (although arguably it probably could be removed). The neko job will still fail with this applied as we'll need to fix it upstream but this is currently blocked on a qiskit-nature release. --- .github/workflows/neko.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/neko.yml b/.github/workflows/neko.yml index 30921c8b051..4ccb3643e37 100644 --- a/.github/workflows/neko.yml +++ b/.github/workflows/neko.yml @@ -18,4 +18,4 @@ jobs: test_selection: terra # We have to forcibly uninstall any old version of qiskit or qiskit-terra during the # changeover, because it's not possible to safely upgrade an existing installation to 1.0. - repo_install_command: "pip uninstall qiskit qiskit-terra && pip install -c constraints.txt ." + repo_install_command: "pip uninstall -y qiskit qiskit-terra && pip install -c constraints.txt ."