mirror of
https://github.com/alexbelgium/hassio-addons.git
synced 2026-08-18 19:07:20 +02:00
fix(pr-2937-review): address reviewer findings on app_config migration
- Remove the orphaned .github/actions/addon-linter composite action: it references .github/scripts/prepare_addon_lint_config.py, which was never added (the normalization approach was superseded by pinning frenck/action-addon-linter directly in #2936). Confirmed unused - no workflow invokes it. - cleanuparr: HA_DATA_DIR pointed at /app_configs/cleanuparr, a path not mounted inside the container. cleanuparr's map is app_config:rw, which Supervisor mounts at /config, so data was living on the ephemeral container filesystem. Point it at /config instead (pre-existing bug, not introduced by this PR - just carried the wrong path forward). - joplin/README.md: joplin's config.yaml still declares config:rw, not app_config (it was never part of the 86-manifest migration). Revert the doc text that incorrectly started calling it the app_config mapping. - qbittorrent/00-folders.sh: fix a pre-existing copy/paste bug in the migration marker/log for the main-folder migration block - it referenced "openvpn" and a "qBitorrent" typo instead of qBittorrent. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
32
.github/actions/addon-linter/action.yml
vendored
32
.github/actions/addon-linter/action.yml
vendored
@@ -1,32 +0,0 @@
|
|||||||
---
|
|
||||||
name: Home Assistant add-on linter compatibility
|
|
||||||
description: Normalize current app_config map names for the upstream add-on linter
|
|
||||||
|
|
||||||
inputs:
|
|
||||||
path:
|
|
||||||
description: Path to the add-on directory
|
|
||||||
required: true
|
|
||||||
community:
|
|
||||||
description: Enable upstream community checks
|
|
||||||
required: false
|
|
||||||
default: "false"
|
|
||||||
|
|
||||||
runs:
|
|
||||||
using: composite
|
|
||||||
steps:
|
|
||||||
- name: Prepare manifest for upstream linter
|
|
||||||
shell: bash
|
|
||||||
env:
|
|
||||||
SOURCE_PATH: ${{ inputs.path }}
|
|
||||||
DESTINATION_PATH: ${{ github.workspace }}/.addon-lint
|
|
||||||
run: |
|
|
||||||
set -euo pipefail
|
|
||||||
python3 "$GITHUB_ACTION_PATH/../../scripts/prepare_addon_lint_config.py" \
|
|
||||||
"$SOURCE_PATH" \
|
|
||||||
"$DESTINATION_PATH"
|
|
||||||
|
|
||||||
- name: Run upstream Home Assistant add-on linter
|
|
||||||
uses: frenck/action-addon-linter@v2
|
|
||||||
with:
|
|
||||||
path: ./.addon-lint
|
|
||||||
community: ${{ inputs.community }}
|
|
||||||
@@ -1,4 +1,5 @@
|
|||||||
- Migrate legacy add-on configuration map names to current app configuration terminology.
|
- Migrate legacy add-on configuration map names to current app configuration terminology.
|
||||||
|
- Fix persistent data directory: the entrypoint now writes to the `/config` mount (the container-side path of the `app_config` map) instead of an unmounted `/app_configs/cleanuparr` path, so Cleanuparr's data survives container recreation.
|
||||||
|
|
||||||
## 2.10.2 (2026-08-01)
|
## 2.10.2 (2026-08-01)
|
||||||
- Update to latest version from Cleanuparr/Cleanuparr (changelog : https://github.com/Cleanuparr/Cleanuparr/releases)
|
- Update to latest version from Cleanuparr/Cleanuparr (changelog : https://github.com/Cleanuparr/Cleanuparr/releases)
|
||||||
|
|||||||
@@ -5,7 +5,7 @@ set -e
|
|||||||
###############################################################################
|
###############################################################################
|
||||||
# Home Assistant Addon entrypoint for Cleanuparr
|
# Home Assistant Addon entrypoint for Cleanuparr
|
||||||
# The .NET app uses /app/config as its data directory.
|
# The .NET app uses /app/config as its data directory.
|
||||||
# We symlink /app/config → /app_configs/cleanuparr (HA persistent storage)
|
# We symlink /app/config → /config (the HA app_config mount, HA persistent storage)
|
||||||
# and start ./Cleanuparr directly, bypassing the original /entrypoint.sh
|
# and start ./Cleanuparr directly, bypassing the original /entrypoint.sh
|
||||||
# which would trigger the /config Docker VOLUME mount.
|
# which would trigger the /config Docker VOLUME mount.
|
||||||
###############################################################################
|
###############################################################################
|
||||||
@@ -26,7 +26,7 @@ if [ -d /etc/cont-init.d ]; then
|
|||||||
fi
|
fi
|
||||||
|
|
||||||
# ─── Setup persistent data directory ─────────────────────────────────────────
|
# ─── Setup persistent data directory ─────────────────────────────────────────
|
||||||
HA_DATA_DIR="/app_configs/cleanuparr"
|
HA_DATA_DIR="/config"
|
||||||
echo "[Cleanuparr] Setting up data directory: $HA_DATA_DIR"
|
echo "[Cleanuparr] Setting up data directory: $HA_DATA_DIR"
|
||||||
mkdir -p "$HA_DATA_DIR"
|
mkdir -p "$HA_DATA_DIR"
|
||||||
|
|
||||||
|
|||||||
@@ -106,7 +106,7 @@ To enable email functionality for user registration and notifications:
|
|||||||
|
|
||||||
### Custom Scripts and Environment Variables
|
### Custom Scripts and Environment Variables
|
||||||
|
|
||||||
This addon supports custom scripts and environment variables through the `app_config` mapping:
|
This addon supports custom scripts and environment variables through the existing `config:rw` mapping:
|
||||||
|
|
||||||
- **Custom scripts**: See [Running Custom Scripts in Addons](https://github.com/alexbelgium/hassio-addons/wiki/Running-custom-scripts-in-Addons)
|
- **Custom scripts**: See [Running Custom Scripts in Addons](https://github.com/alexbelgium/hassio-addons/wiki/Running-custom-scripts-in-Addons)
|
||||||
- **env_vars option**: Use the add-on `env_vars` option to pass extra environment variables (uppercase or lowercase names). See https://github.com/alexbelgium/hassio-addons/wiki/Add-Environment-variables-to-your-Addon-2 for details.
|
- **env_vars option**: Use the add-on `env_vars` option to pass extra environment variables (uppercase or lowercase names). See https://github.com/alexbelgium/hassio-addons/wiki/Add-Environment-variables-to-your-Addon-2 for details.
|
||||||
|
|||||||
@@ -19,8 +19,8 @@ if [ -f /homeassistant/addons_config/qBittorrent/qBittorrent.conf ] && [ ! -f /h
|
|||||||
cp -rnp /homeassistant/addons_config/qBittorrent/* /config/qBittorrent/ &> /dev/null || true
|
cp -rnp /homeassistant/addons_config/qBittorrent/* /config/qBittorrent/ &> /dev/null || true
|
||||||
if [ -d /config/qBittorrent/addons_config ]; then rm -r /config/qBittorrent/addons_config; fi
|
if [ -d /config/qBittorrent/addons_config ]; then rm -r /config/qBittorrent/addons_config; fi
|
||||||
if [ -d /config/qBittorrent/qBittorrent ]; then rm -r /config/qBittorrent/qBittorrent; fi
|
if [ -d /config/qBittorrent/qBittorrent ]; then rm -r /config/qBittorrent/qBittorrent; fi
|
||||||
echo "Files moved to /app_configs/$HOSTNAME/openvpn" > /homeassistant/addons_config/qBittorrent/migrated
|
echo "Files moved to /app_configs/$HOSTNAME/qBittorrent" > /homeassistant/addons_config/qBittorrent/migrated
|
||||||
bashio::log.yellow "... moved files from /config/addons_config/qBittorrent to /app_configs/$HOSTNAME/qBitorrent (must be accessed with my Filebrowser addon)"
|
bashio::log.yellow "... moved files from /config/addons_config/qBittorrent to /app_configs/$HOSTNAME/qBittorrent (must be accessed with my Filebrowser addon)"
|
||||||
MIGRATED=true
|
MIGRATED=true
|
||||||
fi
|
fi
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user