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

Fix docker and podman examples to work with multiple platforms #4068

Merged
merged 2 commits into from
Oct 17, 2023

Conversation

clickthisnick
Copy link
Contributor

@clickthisnick clickthisnick commented Oct 16, 2023

Previous behavior is your molecule group would only contain the last platform since the elements were not merging:
https://docs.ansible.com/ansible/latest/collections/ansible/builtin/combine_filter.html

TASK [Display uname info] ******************************************************
ok: [foo] => {
    "msg": "Linux...\n"
}

New behavior is all of your platforms will be in the host inventory

TASK [Display uname info] ******************************************************
ok: [foo] => {
    "msg": "Linux...\n"
}
ok: [bar] => {
    "msg": "Linux...\n"
}

Fixes: #4058

@clickthisnick clickthisnick temporarily deployed to ack October 16, 2023 20:34 — with GitHub Actions Inactive
@ssbarnea ssbarnea added the bug label Oct 17, 2023
@ssbarnea ssbarnea temporarily deployed to ack October 17, 2023 10:44 — with GitHub Actions Inactive
@ssbarnea ssbarnea merged commit 1b23606 into ansible:main Oct 17, 2023
15 checks passed
@ssbarnea ssbarnea changed the title Issue #4058 - Docs Bug for docker/podman Fix docker and podman examples to work with multiple platforms Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

When including more than 1 entry in platforms molecule_inventory.yml doesn't contain all entries
2 participants