Skip to content

Conversation

Paraphraser
Copy link

…3 of 3

See #648 for background to this PR.

Consequential changes:

  • harmonises template service definition by removing reference to port 443 and re-aligning entries.
  • harmonises default environment variables to be the same as master branch (including comments).
  • removes obsolete networks: clause.
  • removes handling for TZ, WEBPASSWORD, DNS1 and DNS2 from the modifiable environment. The DNS pair are a consequence of overall harmonisation.

Signed-off-by: Phill Kelley 34226495+Paraphraser@users.noreply.github.com

…3 of 3

See SensorsIot#648 for background to this PR.

Consequential changes:

* harmonises template service definition by removing reference to port
443 and re-aligning entries.
* harmonises default environment variables to be the same as master
branch (including comments).
* removes obsolete `networks:` clause.
* removes handling for `TZ`, `WEBPASSWORD`, `DNS1` and `DNS2` from the
modifiable environment. The DNS pair are a consequence of overall
harmonisation.

Signed-off-by: Phill Kelley <34226495+Paraphraser@users.noreply.github.com>
@Paraphraser Paraphraser changed the title 2023-01-22 Pi-hole admin password changes - experimental branch - PR … 2023-01-22 Pi-hole admin password changes - experimental branch - PR 3 of 3 Jan 22, 2023
@Slyke Slyke merged commit 91dfdf5 into SensorsIot:experimental Jan 26, 2023
@Paraphraser Paraphraser deleted the 20230122-pihole-experimental branch January 26, 2023 22:52
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.

2 participants