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

Adds pid_mode feature to docker driver #1535

Merged
merged 2 commits into from
Feb 26, 2019
Merged

Adds pid_mode feature to docker driver #1535

merged 2 commits into from
Feb 26, 2019

Conversation

angelbarrera92
Copy link
Contributor

Solves #1534
Adds pid_mode configuration parameter listen in docker ansible module

Thanks!

@wilmardo
Copy link
Contributor

wilmardo commented Nov 1, 2018

Looks fine to me!

Copy link
Contributor

@decentral1se decentral1se left a comment

Choose a reason for hiding this comment

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

@decentral1se
Copy link
Contributor

Can you rebase @angelbarrera92?

Added to agenda in ansible/community#427. Please join if you have time!

@angelbarrera92
Copy link
Contributor Author

@lwm i will rebase asap ;)

Signed-off-by: Angel Barrera <angel.barrera@intelygenz.com>
@themr0c themr0c added this to the v.2.21 milestone Jan 31, 2019
@angelbarrera92
Copy link
Contributor Author

Could you re-launch the tests of this change? I think it failed by timeout:

The job exceeded the maximum time limit for jobs, and has been terminated.

Thanks!

@decentral1se
Copy link
Contributor

Restarted @angelbarrera92! Thanks for your patience 👍

@webknjaz
Copy link
Member

Hey @angelbarrera92, it looks like there are some conflicts with the current master. Would you mind resolving them, please?

@angelbarrera92
Copy link
Contributor Author

Resolved!

@decentral1se decentral1se modified the milestones: v.2.21, v2.20 Feb 26, 2019
@decentral1se decentral1se merged commit 9dfa125 into ansible:master Feb 26, 2019
@decentral1se
Copy link
Contributor

🚀

ssbarnea pushed a commit to ssbarnea/molecule that referenced this pull request Feb 27, 2019
Signed-off-by: Angel Barrera <angel.barrera@intelygenz.com>
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.

5 participants