Skip to content

Fix: Docker qvm/quilc in rigetti/forest image.#1437

Merged
dbanty merged 1 commit intorcfrom
1434-fix-forest-docker-image
Apr 15, 2022
Merged

Fix: Docker qvm/quilc in rigetti/forest image.#1437
dbanty merged 1 commit intorcfrom
1434-fix-forest-docker-image

Conversation

@dbanty
Copy link
Contributor

@dbanty dbanty commented Apr 15, 2022

Description

Currently, the latest rigetti/forest image fails to start quilc or qvm with the same error seen on Ubuntu in #1434. The fix is simply specifying the version of Debian that comes with the right shared libraries.

Ultimately we will also want to update the qvm and quilc binaries at some point so they work on up-to-date Debian-like distros.

Checklist

  • The PR targets the rc branch (not master).
  • The above description motivates these changes.
  • There is a unit test that covers these changes.
  • All new and existing tests pass locally and on the PR's checks.
  • Parameters and return values have type hints with PEP 484 syntax.
  • Functions and classes have useful Sphinx-style docstrings.
  • All code follows Black style and obeys flake8 conventions.
  • (New Feature) The docs have been updated accordingly.
  • (Bugfix) The associated issue is referenced above using auto-close keywords.
  • The changelog is updated, including author and PR number (@username, Test ignore #1234).

@dbanty dbanty requested a review from a team as a code owner April 15, 2022 17:05
@dbanty dbanty merged commit d8ffa42 into rc Apr 15, 2022
@dbanty dbanty deleted the 1434-fix-forest-docker-image branch April 15, 2022 18:32
@rigetti-githubbot
Copy link

🎉 This PR is included in version 3.2.0-rc.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@rigetti-githubbot
Copy link

🎉 This PR is included in version 3.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants