-
Notifications
You must be signed in to change notification settings - Fork 128
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
Set cmd_env for trusty and xenial only #573
Conversation
Only set the command environment for the supported releases which require it.
I was installing the cassandra charm on bionic with the following set in /etc/apt/apt.conf.d/95-juju-proxy-settings:
This was failing with the following Traceback:
My change resolved the Traceback I encountered and the install was able to complete. |
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.
This looks good to me. Given the git history this returns the behavior to pre commit 0f198e4 and still handles < bionic.
This reverts commit 29c9c2c. This change also removes the comment regarding software-properties as it was adding confusion. add-apt-repository is provided by the software-properties package. While http_proxy and https_proxy env vars appear to work with add-apt-repository, apt.conf settings Acquire::http::Proxy and Acquire::https::Proxy do not appear to work with add-apt-repository. Fixes juju#602
Revert "Set cmd_env for trusty and xenial only (#573)"
This reverts commit 29c9c2c. This change also removes the comment regarding software-properties as it was adding confusion. add-apt-repository is provided by the software-properties package. While http_proxy and https_proxy env vars appear to work with add-apt-repository, apt.conf settings Acquire::http::Proxy and Acquire::https::Proxy do not appear to work with add-apt-repository. Fixes juju#602 (cherry picked from commit ba4dae4)
This reverts commit 29c9c2c. This change also removes the comment regarding software-properties as it was adding confusion. add-apt-repository is provided by the software-properties package. While http_proxy and https_proxy env vars appear to work with add-apt-repository, apt.conf settings Acquire::http::Proxy and Acquire::https::Proxy do not appear to work with add-apt-repository. Fixes juju#602 (cherry picked from commit ba4dae4)
Revert "Set cmd_env for trusty and xenial only (#573)"
Only set the command environment for the supported releases which require it.