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

Using docker provider for molecule testing results in indefinite hang on pihole installation. #23

Open
r-pufky opened this issue Sep 18, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@r-pufky
Copy link
Owner

r-pufky commented Sep 18, 2023

Pulling this issue out into a separate bug. See links for additional context.

Using docker for testing in molecule results in the pihole installer indefinitely hanging, during DNS service restart using systemd.

This tracks the resolution of this issue and is a blocker for the next release.

added the molecule github workflow with above images for testing. still failing at pihole install and not seeing clear error message https://github.com/juju4/ansible_pihole/actions/runs/6207856797/job/16853461529#step:7:353

See: #21 (comment)

Disabling systemd-resolved DNSStubListener and restarting systemd-resolved

It is hanging on restarting systemd-resolved and returning an exit code of 1. This is the docker/systemd interaction I mentioned above. Succeeds in a full VM -- I've been poking around on it on my side working backwards from vagrant/libvirt.

Let's bug this and assign yourself to it so we can get the base framework in to work with; we can discuss more about fixing in that bug.

on dependencies, imho role should include whatever is needed to be played. base docker image allow testing with minimal image vs lxd or github images which are more full systems and somewhat silent dependencies like git or some ansible modules-related python packages.

The role was always intended to be run on baremetal or a VM, not in a docker container. I'm not against adding those silent dependencies, but please either add a comment above that block or another list (and include in the install tasks) that clearly delineates what/why those packages are for (e.g. molecule testing on docker). This also helps when checking deps for major version upgrades.

Originally posted by @r-pufky in #21 (comment)

@r-pufky
Copy link
Owner Author

r-pufky commented Mar 18, 2024

Initial testing framework has been committed to mainline; @juju4.

@r-pufky r-pufky added this to the 4.0 milestone Apr 4, 2024
@r-pufky r-pufky self-assigned this Apr 4, 2024
@r-pufky r-pufky added the enhancement New feature or request label Apr 4, 2024
@juju4
Copy link
Contributor

juju4 commented Apr 7, 2024

Thanks a lot @r-pufky !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants