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 50418, 53996) chroot: add new chroot module #54959

Merged
merged 6 commits into from Dec 3, 2019

Conversation

aplanas
Copy link
Collaborator

@aplanas aplanas commented Oct 11, 2019

What does this PR do?

Add a new chroot module in saltstack.

Based on dockermod.call, this module implement an way to execute random salt modules inside a chroot environment.

Tests written?

Yes

(backport #50418, already merged in develop)
(backport #53996, under review)

@aplanas aplanas changed the title (Backport 50418) chroot: add new chroot module (Backport 50418, 53996) chroot: add new chroot module Oct 14, 2019
@waynew waynew added this to PR needs merge to master in PRs to port to master Oct 17, 2019
@aplanas aplanas requested a review from as a code owner Nov 12, 2019
@ghost ghost requested a review from xeacott Nov 12, 2019
Also reformulate one test to avoid random ordering of kwargs parameters.
@dwoz dwoz merged commit ae1d548 into saltstack:master Dec 3, 2019
40 of 49 checks passed
PRs to port to master automation moved this from PR needs merge to master to PR merged Dec 3, 2019
@aplanas
Copy link
Collaborator Author

@aplanas aplanas commented Dec 3, 2019

Uau! Thanks a lot! I will rebase other PR that depends on this one 👍

@aplanas aplanas deleted the backport_50418 branch Dec 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants