From 4f268ed193f2b552b7518b64b531ff0b1a7654e3 Mon Sep 17 00:00:00 2001 From: DL6ER Date: Sat, 21 Dec 2024 07:10:46 +0100 Subject: [PATCH 01/19] Explicitly migrate from v5 to v6 Signed-off-by: DL6ER --- automated install/basic-install.sh | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/automated install/basic-install.sh b/automated install/basic-install.sh index 755d8dd9..863ac432 100755 --- a/automated install/basic-install.sh +++ b/automated install/basic-install.sh @@ -2326,6 +2326,12 @@ migrate_dnsmasq_configs() { mv /etc/dnsmasq.d/0{1,2,4,5}-pihole*.conf "${V6_CONF_MIGRATION_DIR}/" 2>/dev/null || true mv /etc/dnsmasq.d/06-rfc6761.conf "${V6_CONF_MIGRATION_DIR}/" 2>/dev/null || true + + # Finally, after everything is in place, we can create the new config file + # /etc/pihole/pihole.toml + # This file will be created with the default settings unless the user has + # changed settings via setupVars.conf or the other dnsmasq files moved above + pihole-FTL migrate v6 } main() { From 8faa28d8a01cc5e8b67058502af25ba53a3bfa0f Mon Sep 17 00:00:00 2001 From: DL6ER Date: Mon, 23 Dec 2024 08:48:05 +0100 Subject: [PATCH 02/19] Explicitly log migration process Signed-off-by: DL6ER --- automated install/basic-install.sh | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/automated install/basic-install.sh b/automated install/basic-install.sh index 863ac432..c6ac0fbf 100755 --- a/automated install/basic-install.sh +++ b/automated install/basic-install.sh @@ -2331,7 +2331,20 @@ migrate_dnsmasq_configs() { # /etc/pihole/pihole.toml # This file will be created with the default settings unless the user has # changed settings via setupVars.conf or the other dnsmasq files moved above - pihole-FTL migrate v6 + str="Migrating Pi-hole configuration to version 6" + printf " %b %s...\\n" "${INFO}" + local FTLoutput FTLstatus + FTLoutput=$(pihole-FTL migrate v6) + FTLstatus=$? + if [[ "${FTLstatus}" -eq 0 ]]; then + printf " %b %s\\n" "${TICK}" "${str}" + else + printf " %b %s\\n" "${CROSS}" "${str}" + fi + + # Print the output of the FTL migration prefacing every line with four + # spaces for alignment + printf "%b" "${FTLoutput}" | sed 's/^/ /' } main() { From d14273f958fa4b73b8cf321a936edd7e5364c035 Mon Sep 17 00:00:00 2001 From: DL6ER Date: Tue, 24 Dec 2024 02:17:50 +0100 Subject: [PATCH 03/19] Add comment where setupVars.conf goes when v6 migration is done Signed-off-by: DL6ER --- automated install/basic-install.sh | 1 + 1 file changed, 1 insertion(+) diff --git a/automated install/basic-install.sh b/automated install/basic-install.sh index c6ac0fbf..6d97d7e0 100755 --- a/automated install/basic-install.sh +++ b/automated install/basic-install.sh @@ -2331,6 +2331,7 @@ migrate_dnsmasq_configs() { # /etc/pihole/pihole.toml # This file will be created with the default settings unless the user has # changed settings via setupVars.conf or the other dnsmasq files moved above + # During migration, setupVars.conf is moved to /etc/pihole/migration_backup_v6 str="Migrating Pi-hole configuration to version 6" printf " %b %s...\\n" "${INFO}" local FTLoutput FTLstatus From 4fd77f57faead42a8d8accc3a9d59e80f92e957d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Christian=20K=C3=B6nig?= Date: Tue, 31 Dec 2024 16:42:08 +0100 Subject: [PATCH 04/19] Remove now unused function test_dpkg_lock() MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Christian König --- automated install/basic-install.sh | 22 ---------------------- 1 file changed, 22 deletions(-) diff --git a/automated install/basic-install.sh b/automated install/basic-install.sh index b1608c70..c7d479d0 100755 --- a/automated install/basic-install.sh +++ b/automated install/basic-install.sh @@ -388,28 +388,6 @@ os_check() { fi } -# This function waits for dpkg to unlock, which signals that the previous apt-get command has finished. -test_dpkg_lock() { - i=0 - printf " %b Waiting for package manager to finish (up to 30 seconds)\\n" "${INFO}" - # fuser is a program to show which processes use the named files, sockets, or filesystems - # So while the lock is held, - while fuser /var/lib/dpkg/lock >/dev/null 2>&1; do - # we wait half a second, - sleep 0.5 - # increase the iterator, - ((i = i + 1)) - # exit if waiting for more then 30 seconds - if [[ $i -gt 60 ]]; then - printf " %b %bError: Could not verify package manager finished and released lock. %b\\n" "${CROSS}" "${COL_LIGHT_RED}" "${COL_NC}" - printf " Attempt to install packages manually and retry.\\n" - exit 1 - fi - done - # and then report success once dpkg is unlocked. - return 0 -} - # Compatibility package_manager_detect() { From 79087fb252c1f88fae1d1b9eaeabea7830006672 Mon Sep 17 00:00:00 2001 From: DL6ER Date: Wed, 1 Jan 2025 18:55:49 +0100 Subject: [PATCH 05/19] SHow lighttpd disable dialog only once during v6 migration and when lighttpd is actually installed Signed-off-by: DL6ER --- automated install/basic-install.sh | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/automated install/basic-install.sh b/automated install/basic-install.sh index b1608c70..88369165 100755 --- a/automated install/basic-install.sh +++ b/automated install/basic-install.sh @@ -2301,6 +2301,11 @@ copy_to_install_log() { } disableLighttpd() { + # Return early when lighttpd is not installed + if [[ ! -f /etc/lighttpd/lighttpd.conf ]]; then + return + fi + local response # Detect if the terminal is interactive if [[ -t 0 ]]; then @@ -2345,6 +2350,9 @@ migrate_dnsmasq_configs() { return 0 fi + # Disable lighttpd server during v6 migration + disableLighttpd + # Create target directory /etc/pihole/migration_backup_v6 # and make it owned by pihole:pihole mkdir -p "${V6_CONF_MIGRATION_DIR}" @@ -2536,9 +2544,6 @@ main() { # but before starting or resttarting the ftl service disable_resolved_stublistener - # Disable lighttpd server - disableLighttpd - # Check if gravity database needs to be upgraded. If so, do it without rebuilding # gravity altogether. This may be a very long running task needlessly blocking # the update process. From 364281354c75eceb1d1eec88687cf0bd7debca64 Mon Sep 17 00:00:00 2001 From: DL6ER Date: Thu, 2 Jan 2025 13:43:32 +0100 Subject: [PATCH 06/19] Check if service is currently enabled when checking if lighttpd may need to be disabled Signed-off-by: DL6ER --- automated install/basic-install.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/automated install/basic-install.sh b/automated install/basic-install.sh index 88369165..ea8e210e 100755 --- a/automated install/basic-install.sh +++ b/automated install/basic-install.sh @@ -2301,8 +2301,8 @@ copy_to_install_log() { } disableLighttpd() { - # Return early when lighttpd is not installed - if [[ ! -f /etc/lighttpd/lighttpd.conf ]]; then + # Return early when lighttpd is not active + if ! check_service_active lighttpd; then return fi From 4d23187f864c143e526af069ae899ffdf6f891ac Mon Sep 17 00:00:00 2001 From: DL6ER Date: Mon, 6 Jan 2025 18:33:23 +0100 Subject: [PATCH 07/19] Add missing newline Signed-off-by: DL6ER --- automated install/basic-install.sh | 3 +++ 1 file changed, 3 insertions(+) diff --git a/automated install/basic-install.sh b/automated install/basic-install.sh index cb7324f5..7afdc313 100755 --- a/automated install/basic-install.sh +++ b/automated install/basic-install.sh @@ -2365,6 +2365,9 @@ migrate_dnsmasq_configs() { # Print the output of the FTL migration prefacing every line with four # spaces for alignment printf "%b" "${FTLoutput}" | sed 's/^/ /' + + # Print a blank line for separation + printf "\\n" } # Check for availability of either the "service" or "systemctl" commands From 211a88188827d290c01b3441891047f1a911be9a Mon Sep 17 00:00:00 2001 From: Dominik Date: Wed, 8 Jan 2025 19:28:15 +0100 Subject: [PATCH 08/19] Apply suggestions from code review Co-authored-by: yubiuser Signed-off-by: Dominik --- automated install/basic-install.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/automated install/basic-install.sh b/automated install/basic-install.sh index 7afdc313..a274f22d 100755 --- a/automated install/basic-install.sh +++ b/automated install/basic-install.sh @@ -2352,14 +2352,14 @@ migrate_dnsmasq_configs() { # changed settings via setupVars.conf or the other dnsmasq files moved above # During migration, setupVars.conf is moved to /etc/pihole/migration_backup_v6 str="Migrating Pi-hole configuration to version 6" - printf " %b %s...\\n" "${INFO}" + printf " %b %s...\\n" "${INFO}" "${str}" local FTLoutput FTLstatus FTLoutput=$(pihole-FTL migrate v6) FTLstatus=$? if [[ "${FTLstatus}" -eq 0 ]]; then - printf " %b %s\\n" "${TICK}" "${str}" + printf "%b %b %s\\n" "${OVER}" "${TICK}" "${str}" else - printf " %b %s\\n" "${CROSS}" "${str}" + printf "%b %b %s\\n" "${OVER}" "${CROSS}" "${str}" fi # Print the output of the FTL migration prefacing every line with four From 4be130971e0f9e5d09c7747204ff17b70bcf554e Mon Sep 17 00:00:00 2001 From: Dominik Date: Thu, 9 Jan 2025 18:32:04 +0100 Subject: [PATCH 09/19] Update automated install/basic-install.sh Co-authored-by: yubiuser Signed-off-by: Dominik --- automated install/basic-install.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/automated install/basic-install.sh b/automated install/basic-install.sh index a274f22d..719a57fc 100755 --- a/automated install/basic-install.sh +++ b/automated install/basic-install.sh @@ -2352,7 +2352,7 @@ migrate_dnsmasq_configs() { # changed settings via setupVars.conf or the other dnsmasq files moved above # During migration, setupVars.conf is moved to /etc/pihole/migration_backup_v6 str="Migrating Pi-hole configuration to version 6" - printf " %b %s...\\n" "${INFO}" "${str}" + printf " %b %s..." "${INFO}" "${str}" local FTLoutput FTLstatus FTLoutput=$(pihole-FTL migrate v6) FTLstatus=$? From bd9cc7cbb77d965d4b52015e1487872cfc7754fb Mon Sep 17 00:00:00 2001 From: RD WebDesign Date: Thu, 9 Jan 2025 16:06:38 -0300 Subject: [PATCH 10/19] Clean up unused WEBCALL code Debug Log is no longer available via web interface Signed-off-by: RD WebDesign --- advanced/Scripts/piholeDebug.sh | 32 ++++++++++++-------------------- pihole | 4 +--- 2 files changed, 13 insertions(+), 23 deletions(-) diff --git a/advanced/Scripts/piholeDebug.sh b/advanced/Scripts/piholeDebug.sh index 0a15972c..63d8c50b 100755 --- a/advanced/Scripts/piholeDebug.sh +++ b/advanced/Scripts/piholeDebug.sh @@ -1317,19 +1317,16 @@ upload_to_tricorder() { curl_to_tricorder # If we're not running in automated mode, else - # if not being called from the web interface - if [[ ! "${WEBCALL}" ]]; then - echo "" - # give the user a choice of uploading it or not - # Users can review the log file locally (or the output of the script since they are the same) and try to self-diagnose their problem - read -r -p "[?] Would you like to upload the log? [y/N] " response - case ${response} in - # If they say yes, run our function for uploading the log - [yY][eE][sS]|[yY]) curl_to_tricorder;; - # If they choose no, just exit out of the script - *) log_write " * Log will ${COL_GREEN}NOT${COL_NC} be uploaded to tricorder.\\n * A local copy of the debug log can be found at: ${COL_CYAN}${PIHOLE_DEBUG_LOG}${COL_NC}\\n";exit; - esac - fi + echo "" + # give the user a choice of uploading it or not + # Users can review the log file locally (or the output of the script since they are the same) and try to self-diagnose their problem + read -r -p "[?] Would you like to upload the log? [y/N] " response + case ${response} in + # If they say yes, run our function for uploading the log + [yY][eE][sS]|[yY]) curl_to_tricorder;; + # If they choose no, just exit out of the script + *) log_write " * Log will ${COL_GREEN}NOT${COL_NC} be uploaded to tricorder.\\n * A local copy of the debug log can be found at: ${COL_CYAN}${PIHOLE_DEBUG_LOG}${COL_NC}\\n";exit; + esac fi # Check if tricorder.pi-hole.net is reachable and provide token # along with some additional useful information @@ -1349,13 +1346,8 @@ upload_to_tricorder() { # If no token was generated else # Show an error and some help instructions - # Skip this if being called from web interface and automatic mode was not chosen (users opt-out to upload) - if [[ "${WEBCALL}" ]] && [[ ! "${AUTOMATED}" ]]; then - : - else - log_write "${CROSS} ${COL_RED}There was an error uploading your debug log.${COL_NC}" - log_write " * Please try again or contact the Pi-hole team for assistance." - fi + log_write "${CROSS} ${COL_RED}There was an error uploading your debug log.${COL_NC}" + log_write " * Please try again or contact the Pi-hole team for assistance." fi # Finally, show where the log file is no matter the outcome of the function so users can look at it log_write " * A local copy of the debug log can be found at: ${COL_CYAN}${PIHOLE_DEBUG_LOG}${COL_NC}\\n" diff --git a/pihole b/pihole index f3e14ac6..6424c793 100755 --- a/pihole +++ b/pihole @@ -73,19 +73,17 @@ listFunc() { debugFunc() { local automated - local web local check_database_integrity # Pull off the `debug` leaving passed call augmentation flags in $1 shift for value in "$@"; do [[ "$value" == *"-a"* ]] && automated="true" - [[ "$value" == *"-w"* ]] && web="true" [[ "$value" == *"-c"* ]] && check_database_integrity="true" [[ "$value" == *"--check_database"* ]] && check_database_integrity="true" done - AUTOMATED=${automated:-} WEBCALL=${web:-} CHECK_DATABASE=${check_database_integrity:-} "${PI_HOLE_SCRIPT_DIR}"/piholeDebug.sh + AUTOMATED=${automated:-} CHECK_DATABASE=${check_database_integrity:-} "${PI_HOLE_SCRIPT_DIR}"/piholeDebug.sh exit 0 } From 733dce1c4c4626cfa99ca0ac62a4114396969b83 Mon Sep 17 00:00:00 2001 From: RD WebDesign Date: Thu, 9 Jan 2025 17:34:05 -0300 Subject: [PATCH 11/19] If it's a docker container, consider the OS is supported Signed-off-by: RD WebDesign --- advanced/Scripts/piholeDebug.sh | 3 +++ 1 file changed, 3 insertions(+) diff --git a/advanced/Scripts/piholeDebug.sh b/advanced/Scripts/piholeDebug.sh index 63d8c50b..1307382c 100755 --- a/advanced/Scripts/piholeDebug.sh +++ b/advanced/Scripts/piholeDebug.sh @@ -346,6 +346,9 @@ os_check() { fi done + # If it is a docker container, we can assume the OS is supported + [ -n "${DOCKER_VERSION}" ] && valid_os=true && valid_version=true + local finalmsg if [ "$valid_os" = true ]; then log_write "${TICK} Distro: ${COL_GREEN}${detected_os^}${COL_NC}" From 0590bbba2d17b9153640c8bf0fb7e63b266191b2 Mon Sep 17 00:00:00 2001 From: RD WebDesign Date: Thu, 9 Jan 2025 18:06:06 -0300 Subject: [PATCH 12/19] Skip some checks in docker container These commands always fail in containers with "command not found" (as expected). Signed-off-by: RD WebDesign --- advanced/Scripts/piholeDebug.sh | 20 ++++++++++++++++---- 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/advanced/Scripts/piholeDebug.sh b/advanced/Scripts/piholeDebug.sh index 1307382c..b2ae94ee 100755 --- a/advanced/Scripts/piholeDebug.sh +++ b/advanced/Scripts/piholeDebug.sh @@ -492,13 +492,25 @@ run_and_print_command() { } hardware_check() { + # Note: the checks are skipped if Pi-hole is running in a docker container + + local skip_msg="${INFO} Not enough permissions inside Docker container ${COL_YELLOW}(skipped)${COL_NC}" + echo_current_diagnostic "System hardware configuration" - # Store the output of the command in a variable - run_and_print_command "lshw -short" + if [ -n "${DOCKER_VERSION}" ]; then + log_write "${skip_msg}" + else + # Store the output of the command in a variable + run_and_print_command "lshw -short" + fi echo_current_diagnostic "Processor details" - # Store the output of the command in a variable - run_and_print_command "lscpu" + if [ -n "${DOCKER_VERSION}" ]; then + log_write "${skip_msg}" + else + # Store the output of the command in a variable + run_and_print_command "lscpu" + fi } disk_usage() { From c78be8a89df47bcfbe3b2cd0e816b54ee0be1af1 Mon Sep 17 00:00:00 2001 From: RD WebDesign Date: Thu, 9 Jan 2025 18:49:08 -0300 Subject: [PATCH 13/19] No systemctl or service inside containers Signed-off-by: RD WebDesign --- advanced/Scripts/piholeDebug.sh | 22 +++++++++++++--------- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/advanced/Scripts/piholeDebug.sh b/advanced/Scripts/piholeDebug.sh index b2ae94ee..08b9bc96 100755 --- a/advanced/Scripts/piholeDebug.sh +++ b/advanced/Scripts/piholeDebug.sh @@ -823,26 +823,24 @@ dig_at() { process_status(){ # Check to make sure Pi-hole's services are running and active echo_current_diagnostic "Pi-hole processes" + # Local iterator local i + # For each process, for i in "${PIHOLE_PROCESSES[@]}"; do + local status_of_process + # If systemd if command -v systemctl &> /dev/null; then # get its status via systemctl - local status_of_process status_of_process=$(systemctl is-active "${i}") else # Otherwise, use the service command and mock the output of `systemctl is-active` - local status_of_process - # If DOCKER_VERSION is set, the output is slightly different (s6 init system on Docker) + # If it is a docker container, there is no systemctl or service. Do nothing. if [ -n "${DOCKER_VERSION}" ]; then - if service "${i}" status | grep -E '^up' &> /dev/null; then - status_of_process="active" - else - status_of_process="inactive" - fi + : else # non-Docker system if service "${i}" status | grep -E 'is\srunning' &> /dev/null; then @@ -852,8 +850,12 @@ process_status(){ fi fi fi + # and print it out to the user - if [[ "${status_of_process}" == "active" ]]; then + if [ -n "${DOCKER_VERSION}" ]; then + # If it's a Docker container, the test was skipped + log_write "${INFO} systemctl/service not installed inside docker container ${COL_YELLOW}(skipped)${COL_NC}" + elif [[ "${status_of_process}" == "active" ]]; then # If it's active, show it in green log_write "${TICK} ${COL_GREEN}${i}${COL_NC} daemon is ${COL_GREEN}${status_of_process}${COL_NC}" else @@ -870,6 +872,8 @@ ftl_full_status(){ if command -v systemctl &> /dev/null; then FTL_status=$(systemctl status --full --no-pager pihole-FTL.service) log_write " ${FTL_status}" + elif [ -n "${DOCKER_VERSION}" ]; then + log_write "${INFO} systemctl/service not installed inside docker container ${COL_YELLOW}(skipped)${COL_NC}" else log_write "${INFO} systemctl: command not found" fi From 27fb2bbe3a4bd0c13ae841f6f901b529434a37dd Mon Sep 17 00:00:00 2001 From: RD WebDesign Date: Thu, 9 Jan 2025 17:32:01 -0300 Subject: [PATCH 14/19] Replace some hardcoded paths with pihole.toml values Signed-off-by: RD WebDesign --- advanced/Scripts/piholeDebug.sh | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/advanced/Scripts/piholeDebug.sh b/advanced/Scripts/piholeDebug.sh index 08b9bc96..d0d61281 100755 --- a/advanced/Scripts/piholeDebug.sh +++ b/advanced/Scripts/piholeDebug.sh @@ -44,6 +44,14 @@ fi # shellcheck disable=SC1091 . /etc/pihole/versions +# Read the value of an FTL config key. The value is printed to stdout. +get_ftl_conf_value() { + local key=$1 + + # Obtain setting from FTL directly + pihole-FTL --config "${key}" +} + # FAQ URLs for use in showing the debug log FAQ_HARDWARE_REQUIREMENTS="${COL_CYAN}https://docs.pi-hole.net/main/prerequisites/${COL_NC}" FAQ_HARDWARE_REQUIREMENTS_PORTS="${COL_CYAN}https://docs.pi-hole.net/main/prerequisites/#ports${COL_NC}" @@ -61,10 +69,10 @@ DNSMASQ_D_DIRECTORY="/etc/dnsmasq.d" PIHOLE_DIRECTORY="/etc/pihole" PIHOLE_SCRIPTS_DIRECTORY="/opt/pihole" BIN_DIRECTORY="/usr/local/bin" -RUN_DIRECTORY="/run" LOG_DIRECTORY="/var/log/pihole" -HTML_DIRECTORY="/var/www/html" -WEB_GIT_DIRECTORY="${HTML_DIRECTORY}/admin" +HTML_DIRECTORY="$(get_ftl_conf_value "webserver.paths.webroot")" +WEBHOME_PATH="$(get_ftl_conf_value "webserver.paths.webhome")" +WEB_GIT_DIRECTORY="${HTML_DIRECTORY}${WEBHOME_PATH}" SHM_DIRECTORY="/dev/shm" ETC="/etc" @@ -79,14 +87,6 @@ PIHOLE_FTL_CONF_FILE="${PIHOLE_DIRECTORY}/pihole.toml" PIHOLE_DNSMASQ_CONF_FILE="${PIHOLE_DIRECTORY}/dnsmasq.conf" PIHOLE_VERSIONS_FILE="${PIHOLE_DIRECTORY}/versions" -# Read the value of an FTL config key. The value is printed to stdout. -get_ftl_conf_value() { - local key=$1 - - # Obtain setting from FTL directly - pihole-FTL --config "${key}" -} - PIHOLE_GRAVITY_DB_FILE="$(get_ftl_conf_value "files.gravity")" PIHOLE_FTL_DB_FILE="$(get_ftl_conf_value "files.database")" @@ -94,7 +94,7 @@ PIHOLE_FTL_DB_FILE="$(get_ftl_conf_value "files.database")" PIHOLE_COMMAND="${BIN_DIRECTORY}/pihole" PIHOLE_COLTABLE_FILE="${BIN_DIRECTORY}/COL_TABLE" -FTL_PID="${RUN_DIRECTORY}/pihole-FTL.pid" +FTL_PID="$(get_ftl_conf_value "files.pid")" PIHOLE_LOG="${LOG_DIRECTORY}/pihole.log" PIHOLE_LOG_GZIPS="${LOG_DIRECTORY}/pihole.log.[0-9].*" From 83d464e85f16202a1189ff4ee449f0933b05990a Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Sat, 11 Jan 2025 10:54:32 +0000 Subject: [PATCH 15/19] Bump eps1lon/actions-label-merge-conflict from 3.0.2 to 3.0.3 Bumps [eps1lon/actions-label-merge-conflict](https://github.com/eps1lon/actions-label-merge-conflict) from 3.0.2 to 3.0.3. - [Release notes](https://github.com/eps1lon/actions-label-merge-conflict/releases) - [Changelog](https://github.com/eps1lon/actions-label-merge-conflict/blob/main/CHANGELOG.md) - [Commits](https://github.com/eps1lon/actions-label-merge-conflict/compare/v3.0.2...v3.0.3) --- updated-dependencies: - dependency-name: eps1lon/actions-label-merge-conflict dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] --- .github/workflows/merge-conflict.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/merge-conflict.yml b/.github/workflows/merge-conflict.yml index 4e56ae3e..f169ab6f 100644 --- a/.github/workflows/merge-conflict.yml +++ b/.github/workflows/merge-conflict.yml @@ -13,7 +13,7 @@ jobs: runs-on: ubuntu-latest steps: - name: Check if PRs are have merge conflicts - uses: eps1lon/actions-label-merge-conflict@v3.0.2 + uses: eps1lon/actions-label-merge-conflict@v3.0.3 with: dirtyLabel: "PR: Merge Conflict" repoToken: "${{ secrets.GITHUB_TOKEN }}" From 0ae50eab879b3f6c5cfb50c28c47cd8881929c69 Mon Sep 17 00:00:00 2001 From: RD WebDesign Date: Sun, 12 Jan 2025 20:25:32 -0300 Subject: [PATCH 16/19] Suppress error message on `git describe` failure Signed-off-by: RD WebDesign --- advanced/Scripts/piholeDebug.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/advanced/Scripts/piholeDebug.sh b/advanced/Scripts/piholeDebug.sh index d0d61281..80fe9a35 100755 --- a/advanced/Scripts/piholeDebug.sh +++ b/advanced/Scripts/piholeDebug.sh @@ -202,7 +202,7 @@ compare_local_version_to_git_version() { if git status &> /dev/null; then # The current version the user is on local local_version - local_version=$(git describe --tags --abbrev=0); + local_version=$(git describe --tags --abbrev=0 2>&1); # What branch they are on local local_branch local_branch=$(git rev-parse --abbrev-ref HEAD); From 254b67bca3cfc9ca97e05c674fed1f1bf8482cc9 Mon Sep 17 00:00:00 2001 From: RD WebDesign Date: Mon, 13 Jan 2025 20:11:41 -0300 Subject: [PATCH 17/19] Using more user-friendly messages on `git describe` failure Signed-off-by: RD WebDesign --- advanced/Scripts/piholeDebug.sh | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/advanced/Scripts/piholeDebug.sh b/advanced/Scripts/piholeDebug.sh index 80fe9a35..e6d6197b 100755 --- a/advanced/Scripts/piholeDebug.sh +++ b/advanced/Scripts/piholeDebug.sh @@ -202,7 +202,7 @@ compare_local_version_to_git_version() { if git status &> /dev/null; then # The current version the user is on local local_version - local_version=$(git describe --tags --abbrev=0 2>&1); + local_version=$(git describe --tags --abbrev=0 2> /dev/null); # What branch they are on local local_branch local_branch=$(git rev-parse --abbrev-ref HEAD); @@ -213,7 +213,13 @@ compare_local_version_to_git_version() { local local_status local_status=$(git status -s) # echo this information out to the user in a nice format - log_write "${TICK} Version: ${local_version}" + if [ ${local_version} ]; then + log_write "${TICK} Version: ${local_version}" + elif [ -n "${DOCKER_VERSION}" ]; then + log_write "${TICK} Version: Pi-hole Docker Container ${COL_BOLD}${DOCKER_VERSION}${COL_NC}" + else + log_write "${CROSS} Version: not detected" + fi # Print the repo upstreams remotes=$(git remote -v) From 08e3257f37b0c4ae02371da49036f68fc33d6f96 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Sat, 25 Jan 2025 10:08:17 +0000 Subject: [PATCH 18/19] Bump actions/stale from 9.0.0 to 9.1.0 Bumps [actions/stale](https://github.com/actions/stale) from 9.0.0 to 9.1.0. - [Release notes](https://github.com/actions/stale/releases) - [Changelog](https://github.com/actions/stale/blob/main/CHANGELOG.md) - [Commits](https://github.com/actions/stale/compare/v9.0.0...v9.1.0) --- updated-dependencies: - dependency-name: actions/stale dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] --- .github/workflows/stale.yml | 2 +- .github/workflows/stale_pr.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/stale.yml b/.github/workflows/stale.yml index 724c29d4..a0dd9a31 100644 --- a/.github/workflows/stale.yml +++ b/.github/workflows/stale.yml @@ -17,7 +17,7 @@ jobs: issues: write steps: - - uses: actions/stale@v9.0.0 + - uses: actions/stale@v9.1.0 with: repo-token: ${{ secrets.GITHUB_TOKEN }} days-before-stale: 30 diff --git a/.github/workflows/stale_pr.yml b/.github/workflows/stale_pr.yml index 96650818..6952dcab 100644 --- a/.github/workflows/stale_pr.yml +++ b/.github/workflows/stale_pr.yml @@ -17,7 +17,7 @@ jobs: pull-requests: write steps: - - uses: actions/stale@v9.0.0 + - uses: actions/stale@v9.1.0 with: repo-token: ${{ secrets.GITHUB_TOKEN }} # Do not automatically mark PR/issue as stale From 66757de3fb4ddaa8f9ea1f379e824b6f64f51bc2 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Sat, 25 Jan 2025 10:34:03 +0000 Subject: [PATCH 19/19] Bump tox from 4.23.2 to 4.24.1 in /test Bumps [tox](https://github.com/tox-dev/tox) from 4.23.2 to 4.24.1. - [Release notes](https://github.com/tox-dev/tox/releases) - [Changelog](https://github.com/tox-dev/tox/blob/main/docs/changelog.rst) - [Commits](https://github.com/tox-dev/tox/compare/4.23.2...4.24.1) --- updated-dependencies: - dependency-name: tox dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] --- test/requirements.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/requirements.txt b/test/requirements.txt index d72475c4..7b7e679f 100644 --- a/test/requirements.txt +++ b/test/requirements.txt @@ -2,5 +2,5 @@ pyyaml == 6.0.2 pytest == 8.3.4 pytest-xdist == 3.6.1 pytest-testinfra == 10.1.1 -tox == 4.23.2 +tox == 4.24.1 pytest-clarity == 1.0.1