Skip to content
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

Pin docplex version in constraints.txt for tutorials CI job #5366

Merged
merged 2 commits into from
Nov 9, 2020

Conversation

mtreinish
Copy link
Member

Summary

The recent new release of docplex has broken the released version of
aqua. There is a fix in aqua in-progress qiskit-community/qiskit-aqua#1420, but
until that is included in a release we won't be able to use the latest
version of docplex in the tutorials CI job. This commit adds a pin on
the version of docplex used in the constraints file to always use a
known working version to unblock CI.

Details and comments

The recent new release of docplex has broken the released version of
aqua. There is a fix in aqua in-progress qiskit-community/qiskit-aqua#1420, but
until that is included in a release we won't be able to use the latest
version of docplex in the tutorials CI job. This commit adds a pin on
the version of docplex used in the constraints file to always use a
known working version to unblock CI.
ajavadia
ajavadia previously approved these changes Nov 9, 2020
The windows CI images seem to have been updated and now the pip cache
step can't find the local pip cache to upload to azure. To unblock CI
this commit just removes the windows caching from CI. It will add some
execute time, but that's still better than not working.
@mtreinish mtreinish merged commit 2d3df51 into Qiskit:master Nov 9, 2020
@mtreinish mtreinish deleted the pin-docplex-ci branch November 9, 2020 20:30
@mtreinish mtreinish added the stable backport potential The bug might be minimal and/or import enough to be port to stable label Nov 10, 2020
mergify bot pushed a commit that referenced this pull request Nov 10, 2020
* Pin docplex version in constraints.txt for tutorials CI job

The recent new release of docplex has broken the released version of
aqua. There is a fix in aqua in-progress qiskit-community/qiskit-aqua#1420, but
until that is included in a release we won't be able to use the latest
version of docplex in the tutorials CI job. This commit adds a pin on
the version of docplex used in the constraints file to always use a
known working version to unblock CI.

* Remove windows pip cache

The windows CI images seem to have been updated and now the pip cache
step can't find the local pip cache to upload to azure. To unblock CI
this commit just removes the windows caching from CI. It will add some
execute time, but that's still better than not working.

(cherry picked from commit 2d3df51)
mergify bot added a commit that referenced this pull request Nov 10, 2020
…5369)

* Pin docplex version in constraints.txt for tutorials CI job

The recent new release of docplex has broken the released version of
aqua. There is a fix in aqua in-progress qiskit-community/qiskit-aqua#1420, but
until that is included in a release we won't be able to use the latest
version of docplex in the tutorials CI job. This commit adds a pin on
the version of docplex used in the constraints file to always use a
known working version to unblock CI.

* Remove windows pip cache

The windows CI images seem to have been updated and now the pip cache
step can't find the local pip cache to upload to azure. To unblock CI
this commit just removes the windows caching from CI. It will add some
execute time, but that's still better than not working.

(cherry picked from commit 2d3df51)

Co-authored-by: Matthew Treinish <mtreinish@kortar.org>
@kdk kdk added the Changelog: None Do not include in changelog label Mar 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: None Do not include in changelog stable backport potential The bug might be minimal and/or import enough to be port to stable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants