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

Pre-impelment database parameters inside the container #106

Open
compgeniuses opened this issue Mar 8, 2024 · 3 comments
Open

Pre-impelment database parameters inside the container #106

compgeniuses opened this issue Mar 8, 2024 · 3 comments

Comments

@compgeniuses
Copy link

At the moment, the container seems to be asking the user from the user interface to add their database details, which is countr intuitive,

WOuld it not be great if we could have the container automatically use the configured variables to acheive this.

We can do this by

Mounting the following file inside the glpi files

config_db.php

then inside it, we add the follwing

youll need to adopt the variables to yours,

Wouldnt this solve the problem?

A previous attempt here solved the problem https://github.com/stephdl/nethserver-glpi-10-latest/blob/ns7/root/etc/e-smith/templates/etc/glpi/config_db.php/10base

@DiouxX
Copy link
Owner

DiouxX commented Mar 19, 2024

I understand the request and it's true that it could be more intuitive.

I wonder if it wouldn't be better to leave the possibility of entering database information via the installation.

In my opinion, both options should be available.

@compgeniuses
Copy link
Author

its better in a docker setup because most of the time, the person deploying is parsing the env variables somehow, so if these database are entered somehwere, then it makes sense that they are used and implemented.

Using your image, we have implemented it here https://github.com/geniusdynamics/docker-glpi
we are also attempting to have the same implemented with the glpi pre-built into the docker image. that way even when ofline, someone can start the image its on the genius-glpi branch of the same link

@DiouxX
Copy link
Owner

DiouxX commented Mar 19, 2024

All right, I'll add this feature to the roadmap.

Before that, I need to update the image to use Debian 12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants