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

[backport][release_2.1] Fix bug with sandboxing with values from env/settings (#978) #981

Merged
merged 1 commit into from
Jan 31, 2022

Conversation

Shrews
Copy link
Contributor

@Shrews Shrews commented Jan 31, 2022

Backport of PR #978

This also standardizes .gitignore from a change that was not backported to this branch.

(cherry picked from commit 996a00d)

Fix bug with sandboxing with values from env/settings

Fixes ansible#977
Sandboxing with directory isolation did not work when values were defined via env/settings.
Unit test for this required moving the project_fixtures pytest fixture from the integration directory to the top-level test directory where it could be used by unit tests, as well.

Reviewed-by: Alexander Sowitzki <dev@eqrx.net>
Reviewed-by: None <None>
(cherry picked from commit 996a00d)
@Shrews Shrews requested a review from a team as a code owner January 31, 2022 14:20
@eqrx eqrx added the gate label Jan 31, 2022
Copy link
Contributor

@ansible-zuul ansible-zuul bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@ansible-zuul ansible-zuul bot merged commit 021919d into ansible:release_2.1 Jan 31, 2022
@Shrews Shrews deleted the backport/2.1/978 branch January 31, 2022 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants