-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix pip uninstall command in qiskit-neko #11826
Conversation
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.
One or more of the the following people are requested to review this:
|
Pull Request Test Coverage Report for Build 7936918676Details
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huh, surprised that pip doesn't automatically recognise when stdin's not a tty (or alternatively that whatever runner setup makes it seem like stdin is a tty).
@Mergifyio backport stable/0.46 |
✅ Backports have been created
|
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. (cherry picked from commit 12f24a7) # Conflicts: # .github/workflows/neko.yml
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. (cherry picked from commit 12f24a7)
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. (cherry picked from commit 12f24a7) Co-authored-by: Matthew Treinish <mtreinish@kortar.org>
Summary
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.
Details and comments