Skip to content

Conversation

@joergdw
Copy link
Contributor

@joergdw joergdw commented Sep 27, 2024

The bosh-cli wipes parts of the environment provided by the host-system which are in some cases essential for compilation of bosh-releases. This PR follows a conservative fix-approach by adding an option to opt-out of this default. This can (and maybe should) be extended by a fix of the “isolation” from the host-environment. For more details, see #660

aramprice added a commit to cloudfoundry/bosh-utils that referenced this pull request Dec 12, 2024
This struct attribute is only ever set to `true` by the bosh-cli, and
the value if this use case is in question, see:
- cloudfoundry/bosh-cli#660
- cloudfoundry/bosh-cli#663
- cloudfoundry/bosh-cli#673

In addition the `UseIsolatedEnv` capability can be wholely handled
within the `bosh-cli` codebase itself without the need for this featuer
in `bosh-utils`.
aramprice added a commit to cloudfoundry/bosh-utils that referenced this pull request Dec 12, 2024
This struct attribute is only ever set to `true` by the bosh-cli, and
the value if this use case is in question, see:
- cloudfoundry/bosh-cli#660
- cloudfoundry/bosh-cli#663
- cloudfoundry/bosh-cli#673

In addition the `UseIsolatedEnv` capability can be wholely handled
within the `bosh-cli` codebase itself without the need for this featuer
in `bosh-utils`.
@jpalermo jpalermo merged commit 72c44e6 into cloudfoundry:main Jan 31, 2025
aramprice added a commit to cloudfoundry/bosh-utils that referenced this pull request Jan 31, 2025
This struct attribute is only ever set to `true` by the bosh-cli, and
the value if this use case is in question, see:
- cloudfoundry/bosh-cli#660
- cloudfoundry/bosh-cli#663
- cloudfoundry/bosh-cli#673

In addition the `UseIsolatedEnv` capability can be wholely handled
within the `bosh-cli` codebase itself without the need for this featuer
in `bosh-utils`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants