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

docker_plugin: do not crash when plugin doesn't exist #553

Merged
merged 2 commits into from
Jan 13, 2023

Conversation

felixfontein
Copy link
Collaborator

SUMMARY

Fixes #552.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

docker_plugin

plugins/modules/docker_plugin.py Outdated Show resolved Hide resolved
Co-authored-by: Brian Scholer <1260690+briantist@users.noreply.github.com>
@felixfontein
Copy link
Collaborator Author

Re-triggering tests after 0142910 was added to main.

@felixfontein felixfontein reopened this Jan 13, 2023
@felixfontein felixfontein merged commit c7cbec0 into ansible-collections:main Jan 13, 2023
@felixfontein felixfontein deleted the docker-plugin-bug branch January 13, 2023 19:49
@felixfontein
Copy link
Collaborator Author

@briantist thanks for reviewing this!

felixfontein added a commit to felixfontein/community.docker that referenced this pull request Jan 13, 2023
…tions#553)

* Do not crash when plugin doesn't exist.

* Improve style.

Co-authored-by: Brian Scholer <1260690+briantist@users.noreply.github.com>

Co-authored-by: Brian Scholer <1260690+briantist@users.noreply.github.com>
(cherry picked from commit c7cbec0)
felixfontein added a commit that referenced this pull request Jan 13, 2023
* Do not crash when plugin doesn't exist.

* Improve style.

Co-authored-by: Brian Scholer <1260690+briantist@users.noreply.github.com>

Co-authored-by: Brian Scholer <1260690+briantist@users.noreply.github.com>
(cherry picked from commit c7cbec0)
patchback bot pushed a commit that referenced this pull request Jan 13, 2023
* Do not crash when plugin doesn't exist.

* Improve style.

Co-authored-by: Brian Scholer <1260690+briantist@users.noreply.github.com>

Co-authored-by: Brian Scholer <1260690+briantist@users.noreply.github.com>
(cherry picked from commit c7cbec0)
(cherry picked from commit 0ffa3cc)
felixfontein added a commit that referenced this pull request Jan 14, 2023
…557)

* Do not crash when plugin doesn't exist.

* Improve style.

Co-authored-by: Brian Scholer <1260690+briantist@users.noreply.github.com>

Co-authored-by: Brian Scholer <1260690+briantist@users.noreply.github.com>
(cherry picked from commit c7cbec0)
(cherry picked from commit 0ffa3cc)

Co-authored-by: Felix Fontein <felix@fontein.de>
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.

docker_plugin does not work in Ansible's check mode
2 participants