Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[C2] Make configuration .yaml files optional #5991

Open
ywarnier opened this issue Dec 19, 2024 · 0 comments
Open

[C2] Make configuration .yaml files optional #5991

ywarnier opened this issue Dec 19, 2024 · 0 comments

Comments

@ywarnier
Copy link
Member

Currently, services.yaml adds a hard requirement on the presence of config/authentication.yaml, config/hosting_limits.yaml and config/plugins.yaml.

This is not desirable, as this implies having those files in Git, which means they would be overwritten by subsequent pulls or manual Chamilo updates.

Make the requirement optional (ignore_errors in services.yaml) and convert those current 3 files to *.dist.yaml. Only these *.dist.yaml will be present in Git. Upon install or migration, and if these files are not present already, the *.dist.yaml are copied to *.yaml.

  • [] modify services.yaml to make loading optional
  • [] rename *.yaml to *.dist.yaml
  • [] add migration to create those *.yaml files from a copy of the *.dist.yaml
  • [] add installation step to create those *.yaml files from a copy of the *.dist.yaml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants