Skip to content

remove FQDN reference into metadata_templates#475

Merged
jeanmi151 merged 1 commit intodocker-masterfrom
datafeeder_config
Feb 10, 2026
Merged

remove FQDN reference into metadata_templates#475
jeanmi151 merged 1 commit intodocker-masterfrom
datafeeder_config

Conversation

@jeanmi151
Copy link
Contributor

No description provided.

@jeanmi151
Copy link
Contributor Author

Won't we lose the benefit of setting the domain only in one place and have it configured everywhere? Like for example in https://github.com/georchestra/docker/blob/master/.envs-common or https://github.com/georchestra/helm-charts/blob/139446f0f9b8085b6df48a7365b2ffa3f03297f3/georchestra/values.yaml#L422?

I think being able to modify the domain name only in one place is pretty neat. Some previous migrations have shown that this domain may change for the same client.

(Moving my comment from normal branch to docker branch)

It is part of 3 steps : https://github.com/georchestra/datadir/tree/docker-master?tab=readme-ov-file#3-steps-editing
mandatory before to run geOrchestra (other than with default config)
Before using this datadir, you should at least change the default FQDN (georchestra-127-0-0-1.nip.io) for yours. This can be done very easily with eg:

@pmauduit
Copy link
Member

it's unfortunate having the variable already defined elsewhere and not being able to reference it here, but considering the remaining lifetime of the datafeeder + the fact that the datadir won't generally be used by people without modifications (modification which is documented as pointed by @jeanmi151), I guess it is acceptable. I don't have a strong objection here at least.

@georchestra georchestra deleted a comment from edevosc2c Feb 10, 2026
@georchestra georchestra deleted a comment from edevosc2c Feb 10, 2026
@jeanmi151 jeanmi151 merged commit f5e43b5 into docker-master Feb 10, 2026
1 check passed
@jeanmi151 jeanmi151 deleted the datafeeder_config branch February 10, 2026 14:40
github-actions bot pushed a commit that referenced this pull request Feb 10, 2026
@github-actions
Copy link

💚 All backports created successfully

Status Branch Result
docker-25.0

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

github-actions bot added a commit that referenced this pull request Feb 10, 2026
[docker-25.0] remove FQDN reference into metadata_templates (#475)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants