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.
This pull request includes several changes to improve the Docker setup, update Helm chart versions, and add necessary documentation. The most important changes include adding sudo and docker support, updating the Helm chart version, and modifying the
start.sh
script to handle permissions more robustly.Improvements to Docker setup:
Dockerfile
: Added sudo installation and modified chown commands to use sudo, ensuring proper permissions for theazdouser
user.Helm chart version updates:
chart/Chart.yaml
: Updated the Helm chart version to2.1.0
and the app version to3.248.0-stable-v2.1.0
.chart/values.yaml
: Updated the Docker tag to3.248.0-stable-v2.1.0
.Documentation updates:
README.md
: Added a new entry for version2.1.0
in the Helm version table and included release notes highlighting the addition of sudo and docker support.chart/README.md
: Updated the Helm version table and added release notes for version2.1.0
.Script improvements:
src/start.sh
: Modified the script to check for sudo availability and handle permissions forazdouser
and docker socket more robustly.