20211215 nextcloud_db - exposed port - master branch - PR 1 of 3

Pull Requests [PR387](SensorsIot#387), [PR388](SensorsIot#388), [PR389](SensorsIot#389) added a port mapping of `9322:3306` to facilitate backup and restores - so the routines could determine when the MariaDB service was ready for business. The need for this was superseded by PRs [416](SensorsIot#416), [417](SensorsIot#417) and [418](SensorsIot#418). It is sufficient for backup and restore routines to run the health check and rely on the return status. This PR removes the `9322:3306` as no longer needed. It will also avoid the need to protect the port with an iptables rule in situations where the host is exposed to the network. Documentation also updated. Signed-off-by: Phill Kelley <[email protected]>
Paraphraser · Dec 14, 2021 · 6b4e4d7 · 6b4e4d7
1 parent f876d33
commit 6b4e4d7
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 4 deletions.
diff --git a/.templates/nextcloud/service.yml b/.templates/nextcloud/service.yml
@@ -29,8 +29,6 @@ nextcloud_db:
     - MYSQL_PASSWORD=%randomMySqlPassword%
     - MYSQL_DATABASE=nextcloud
     - MYSQL_USER=nextcloud
-  ports:
-    - "9322:3306"
   volumes:
     - ./volumes/nextcloud/db:/config
     - ./volumes/nextcloud/db_backup:/backup

diff --git a/docs/Containers/NextCloud.md b/docs/Containers/NextCloud.md
@@ -33,8 +33,6 @@ nextcloud_db:
     - MYSQL_PASSWORD=«user_password»
     - MYSQL_DATABASE=nextcloud
     - MYSQL_USER=nextcloud
-  ports:
-    - "9322:3306"
   volumes:
     - ./volumes/nextcloud/db:/config
     - ./volumes/nextcloud/db_backup:/backup