georchestra / docker

Quick start geOrchestra with docker
18 stars 22 forks source link

Directory name needs to be named "docker" #214

Open edevosc2c opened 1 year ago

edevosc2c commented 1 year ago

The directory name needs to be named docker and not something else. Or we will have issues for the security proxy routing.

Probably related to @docker: https://github.com/georchestra/docker/blob/master/docker-compose.override.yml#L43

Possible solutions:

fvanderbiest commented 1 year ago

Yes, indeed, excellent finding ! A fix here would be more than welcome.

jeanpommier commented 3 months ago

I believe that explcitly setting the project's name to docker (docker compose -p docker up) would work. Which means we can also change it to whatever we want, provided we declare the corresponding project's name

arnaud-morvan commented 1 month ago

There are multiple way to set the project name: https://docs.docker.com/compose/project-name/

I'm used to add a .env file:

COMPOSE_PROJECT_NAME=georchestra

No need to set it on command line on each call.

And it could be possible to use ${COMPOSE_PROJECT_NAME} where required, for example in https://github.com/georchestra/docker/blob/7c4e381809aacc78dc2b73369284fcbdc87033d5/docker-compose.override.yml#L43

edevosc2c commented 1 month ago

@arnaud-morvan you mean that by setting COMPOSE_PROJECT_NAME=docker in the .env file, the problem would be solved?

arnaud-morvan commented 1 month ago

If I understood correctly, pretty sure yes, it will resolve this issue.

BTW geochestra or georchestra-dev would be more meaningful than docker in this case.

edevosc2c commented 1 month ago

We can't alter the docker compose name, otherwise an existing install will lose all the volumes, as the volumes names are based on the docker compose project name.

arnaud-morvan commented 1 month ago

Yes, so be careful that when upgrading some customer projects, if might change compose project name.

arnaud-morvan commented 1 month ago

I may miss something :

We might need to provide some communication about such change.

Is it possible to not apply this file in customers upgrades ?

Available for discussion until this friday evening before my holidays.

edevosc2c commented 1 month ago

Would rather keep the docker compose project name as "docker" in order to have less friction to the existing users. It's not like it's going to matter a lot. If one is not happy with the docker compose project name, he can change it.

For the moment, I'm not going to alter this behavior this August. We may reconsider this improvement later in the year.

fvanderbiest commented 1 month ago
  • it seems not a big issue to loose some volumes in dev on such change

Agreed.

As such, this repository is not production-ready, and does not have a proper release cycle, hence we are allowed to break things.

arnaud-morvan commented 1 month ago

Yes but be careful that some production repositories are forks of this ones, subjects to merge on upgrades.

As I said, in most cases those volumes might be overridden using host folders for production, for those I know about.

superbully commented 1 month ago

hello. I have been testing for the problem with the "docker" folder name and I have not been able to get it to work by changing the project name to something different than the default name "docker". Any idea how to make it work by changing the name of the folder "docker" to another name or by setting the project name different from "docker" ? thank you all very much.