[ 
https://issues.apache.org/jira/browse/GUACAMOLE-374?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17938987#comment-17938987
 ] 

Per von Zweigbergk edited comment on GUACAMOLE-374 at 3/27/25 3:31 PM:
-----------------------------------------------------------------------

Hello! I was building this from the main branch and had to scratch my head on 
this one, it looks like the POSTGRESQL_USER variable (which as per the official 
documentation of 1.5.5 is the suggested variable to use) is not properly 
respected.

Looking at guacamole-docker/entrypoint.d/000-migrate-legacy-variables.sh, I 
believe the issue is on line 97:

{{deprecate_variable "POSTGRES_USER"  "POSTGRESQL_USERNAME"}}

This maps the old variable POSTGRES_USER to POSTGRESQL_USERNAME (I'm not sure 
about this one, perhaps historically the variable was called POSTGRES_USER, as 
opposed to POSTGRESQL_USER in the 1.5.5 documentation?)

(It would probably be safe to add another mapping for POSTGRESQL_USER to 
POSTGRESQL_USERNAME.)

If this is not fixed, upgrading to 1.6.0 will break existing container 
configurations that are set up as per the manual as per version 1.5.5. And the 
error message that is logged is not at all obvious unfortunately.


was (Author: JIRAUSER308815):
Hello! I was building this from the main branch and had to scratch my head on 
this one, it looks like the POSTGRESQL_USER variable (which as per the official 
documentation of 1.5.5 is the suggested variable to use) is not properly 
respected.

Looking at guacamole-docker/entrypoint.d/000-migrate-legacy-variables.sh, I 
believe the issue is on line 97:

{{deprecate_variable "POSTGRES_USER"  "POSTGRESQL_USERNAME"}}

This maps the old variable POSTGRES_USER to POSTGRESQL_USERNAME (I'm not sure 
about this one, perhaps historically the variable was called POSTGRES_USER, as 
opposed to POSTGRESQL_USER in the 1.5.5 documentation?)

(It would probably be safe to add another mapping for POSTGRESQL_USER to 
POSTGRESQL_USERNAME.)

> Installing Guacamole with Docker improvements
> ---------------------------------------------
>
>                 Key: GUACAMOLE-374
>                 URL: https://issues.apache.org/jira/browse/GUACAMOLE-374
>             Project: Guacamole
>          Issue Type: Improvement
>          Components: Documentation, guacamole-docker
>         Environment: sven@fangorn:~$ docker version
> Client:
>  Version:      17.06.1-ce
>  API version:  1.30
>  Go version:   go1.8.3
>  Git commit:   874a737
>  Built:        Thu Aug 17 22:51:12 2017
>  OS/Arch:      linux/amd64
> Server:
>  Version:      17.06.1-ce
>  API version:  1.30 (minimum version 1.12)
>  Go version:   go1.8.3
>  Git commit:   874a737
>  Built:        Thu Aug 17 22:50:04 2017
>  OS/Arch:      linux/amd64
>  Experimental: false
>            Reporter: Sven Gottwald
>            Assignee: Mike Jumper
>            Priority: Minor
>             Fix For: 1.6.0
>
>
> I installed guacamole as docker container for the first time. Great software! 
> I followed the guide on 
> [https://guacamole.incubator.apache.org/doc/gug/guacamole-docker.html]. There 
> is a lot of unnecessary work to get the container up and running, here are 
> some suggestions to improve the documentation:
> # The link flag is a deprecated legacy feature of Docker. It may eventually 
> be removed. [Legacy container links 
> |https://docs.docker.com/engine/userguide/networking/default_network/dockerlinks/].
>  It is no problem to get guacamole running with a user defined network.
> # The database should be generated automatically on first run. The user and 
> database could be automatically created by 
> [mariadb|https://hub.docker.com/_/mariadb/].
> # ...You should add mariadb as possible backend.
> # Due to its complexity (three services) you should provide a 
> {{docker-compose.yml}} file for use with 
> [docker-compose|https://docs.docker.com/compose/]. This also creates an user 
> defined network.
> Here is an example {{docker-compose.yml}} for use with mariadb:
> {noformat}
> version: '3'
> services:
>   guacd:
>     image: guacamole/guacd
>     depends_on:
>       - db
>   guac:
>     image: guacamole/guacamole
>     ports:
>       - 8080:8080
>     depends_on:
>       - db
>       - guacd
>     environment:
>       - GUACD_HOSTNAME=guacd
>       - MYSQL_PASSWORD=x
>       - MYSQL_DATABASE=guacamole_db
>       - MYSQL_USER=guacamole_user
>       - MYSQL_HOSTNAME=db
>   db:
>     image: mariadb
>     volumes:
>       - /srv/docker/guacamole-mysql:/var/lib/mysql
>     environment:
>       - MYSQL_ROOT_PASSWORD=x
>       - MYSQL_PASSWORD=x
>       - MYSQL_DATABASE=guacamole_db
>       - MYSQL_USER=guacamole_user
> {noformat}
> I don't know why, but {{GUACD_HOSTNAME}} and {{MYSQL_HOSTNAME}} are needed 
> (maybe a bug?)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to