Skip to content
This repository has been archived by the owner on Jul 26, 2024. It is now read-only.

Support for pihole running in a podman container #138

Merged
merged 3 commits into from
Feb 4, 2021

Conversation

mfschumann
Copy link
Contributor

I've added support for pihole instances running in podman (instead of docker) containers.

Changes visible to the user

  • In the configuration assistant, the answer if pihole is running in a container must now be answered with the container type, i.e. 'docker' or 'podman' instead of just saying 'Yes'.

Let me know what you think.

@vmstan vmstan changed the base branch from master to 3.3.0 February 2, 2021 15:34
@vmstan vmstan merged commit c84cc27 into vmstan:3.3.0 Feb 4, 2021
vmstan added a commit that referenced this pull request Feb 9, 2021
* Catch up 3.3 (#142)

* 3.2.5 (#140)

Co-authored-by: Michael Stanclift <[email protected]>

* Adds backup cleanup back to process

* Add backup cleanup to compare script when no changes are detected.

* Trek reference

* Redundant

* 3.2.6

* 3.2.6

Co-authored-by: Michael Stanclift <[email protected]>
Co-authored-by: Michael Stanclift <[email protected]>

* Beginning 3.3.0 work

* Podman support

* Support for pihole running in a podman container (#138)

* add support for pihole running in a podman container

* revert renaming of DOCKER_CON to CON_NAME to avoid breaking existing installations

Co-authored-by: Michael Stanclift <[email protected]>

* Add sudo to call of Pihole container version

* Use dev branch pipe

* Without origin

* ascii

* rotate logo

* Move config files to settings folder

* MIssed an important one

* Relocate logs files

* Filling empty directory

* Regression of sudo fix

* Logs

* More words

* Change sync frequency options

* Remove backup automation

* Bold questions

* name

* elif

* BACKUP_RETAIN=3

* Remove automation flag

* == not !=

* Rearrange

* IF

* Default to 0

* 15 default

* More words!

* ELSE

* tighter columns

* Instance type

* Y/N

Co-authored-by: Michael Stanclift <[email protected]>
Co-authored-by: Michael Stanclift <[email protected]>
Co-authored-by: Martin F. Schumann <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants