Skip to content
This repository has been archived by the owner on Jan 5, 2023. It is now read-only.

Fix unnecessary warnings when creating/destroying docker networks #48

Merged
merged 2 commits into from
Apr 14, 2021
Merged

Fix unnecessary warnings when creating/destroying docker networks #48

merged 2 commits into from
Apr 14, 2021

Conversation

mtpettyp
Copy link

@mtpettyp mtpettyp commented Apr 7, 2021

Fixes #47

@ssbarnea ssbarnea self-requested a review April 14, 2021 13:59
@ssbarnea ssbarnea added the bug This issue/PR relates to a bug. label Apr 14, 2021
@ssbarnea ssbarnea merged commit 84372ec into ansible-community:master Apr 14, 2021
ssbarnea pushed a commit that referenced this pull request May 26, 2021
* Fix unnecessary warnings when creating/destroying docker networks

Fixes #47

* Fix flake8-black reformat
ssbarnea pushed a commit that referenced this pull request May 26, 2021
* Fix unnecessary warnings when creating/destroying docker networks

Fixes #47

* Fix flake8-black reformat
@hassenius
Copy link
Collaborator

@mtpettyp @ssbarnea while looking around in the code I noticed this line: https://github.com/ansible-community/molecule-docker/blob/main/molecule_docker/playbooks/destroy.yml#L44

loop: "{{ molecule_yml.platforms | molecule_get_docker_networks('absent') }}"

which should probably have been removed in this PR too since the state variable is no longer there

@hassenius hassenius mentioned this pull request Jul 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug This issue/PR relates to a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unnecessary warnings when creating/deleting docker networks
3 participants