Support powerwall.extend.yml & backup.sh.sample #186
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Supports a customised powerwall through the upgrade by preserving it before the upgrade by copying it to a .restore copy, and restoring it after the git pull before restarting the containers.
Supports a customised powerwall.yml Docker Compose file through the upgrade by preserving it before the upgrade by copying it to a .preserve copy, and restoring it after the git pull before restarting the containers.
This reverts commit 5f74ab4.
This reverts commit f5bf483.
Support Extending powerwall.yml by modifying compose-dash.sh to look for powerwall.extend.yml and including it in docker compose commands. Change backup.sh to backup.sh.sample. Modify instructions to indicate that backup.sh.sample should be copied to backup.sh
|
Released as v2.8.4 - Custom Docker Compose Support Thanks again @BJReplay !! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Apologies for six different commits. The first four are two commits and two more undoing them - the changes to update.sh.
The next two are the meat on the bones. The changes to compose-dash.sh look for the existence of powerwall.extend.yml, and if it exists, initialise a variable with -f powerwall.extend.yml to be included in the docker compose or docker-compose commands (whatever they are) so that the docker compose command works on both powerwall.yml and powerwall.extend.yml.
I used this to shut down my customised powerwall.yml, bring up the standard one (with Weather411 added back in), shut it down, bring up one with Ecowitt added back in (by creating powerwall.extend.yml).
The other change here is to the snippet in the instructions for the Ecowitt weather service to add in the version and services tags that allow this yaml to be merged.
Associated change to the readme noting this.
Final change here is a rename of backup.sh to backup.sh.sample, and associated change to the readme: I got bitten by the upgrade today when my customised backup.sh got overwritten by the upgrade.sh and a second commit with improved instructions.