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.

Sign up for GitHub

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

Jump to bottom

Odoo Critical error "failed to initialize database" once server_environemnt is installed #44

Closed
dreispt opened this issue Mar 31, 2020 · 8 comments
Closed

Odoo Critical error "failed to initialize database" once server_environemnt is installed #44

dreispt opened this issue Mar 31, 2020 · 8 comments

Comments

@dreispt
Copy link
Member

dreispt commented Mar 31, 2020 

server-env/server_environment/serv_config.py", line 50, in <module>
    "The parameter 'running_env' has not be set neither in base config "
Exception: The parameter 'running_env' has not be set neither in base config file option -c or in openerprc.
We strongly recommend against using the rc file but instead use an explicit config file with this content:
[options]
running_env = dev

I get the point that the additional configuration is very important.
But does it justify preventing the service from starting?
I think this would be a log message, rather than a error with a traceback.
@simahawk
Copy link
Contributor

IMO we should default to running_env=dev + show a warning that is running in dev mode.
@guewen ?

@guewen
Copy link
Member

guewen commented Mar 31, 2020

I agree!

@amh-mw
Copy link

amh-mw commented Mar 31, 2020

Per https://github.com/OCA/server-env/tree/13.0/server_environment#configuration

Values associated to keys containing 'passw' are only displayed in the 'dev' environment.

I'm not sure that we should default to a setting that exposes passwords...

@dreispt
Copy link
Member Author

dreispt commented Mar 31, 2020

I'm not sure what "displayed" means (printed out to the server log?), but following @amh-mw comment, I suggest using test as the default value, rather than dev..

@guewen
Copy link
Member

guewen commented Apr 1, 2020

IIRC, the module automatically replaces password fields with * chars in form views unless the running env is dev. This is a good remark @amh-mw, considering this, it would be better to use test by default indeed.

simahawk added a commit to simahawk/server-env that referenced this issue Apr 1, 2020
@simahawk
server_environemnt: running_env default to dev (fix OCA#44)
2f0359a
simahawk added a commit to simahawk/server-env that referenced this issue Apr 1, 2020
@simahawk
server_environemnt: running_env default to dev (fix OCA#44)
d1d6259
@simahawk
Copy link
Contributor

simahawk commented Apr 1, 2020

ah, reading now these remarks... 😅
I'll update my PR then :)

simahawk added a commit to simahawk/server-env that referenced this issue Apr 1, 2020
@simahawk
server_environemnt: running_env default to test (fix OCA#44)
168d7ca
@simahawk
Copy link
Contributor

simahawk commented Apr 1, 2020

#45

simahawk added a commit to simahawk/server-env that referenced this issue Apr 1, 2020
@simahawk
server_environment: running_env default to test (fix OCA#44)
3ef4aea
simahawk added a commit to simahawk/server-env that referenced this issue Apr 1, 2020
@simahawk
fixup! server_environment: running_env default to test (fix OCA#44)
e6e3196
simahawk added a commit to simahawk/server-env that referenced this issue Apr 2, 2020
@simahawk
server_environment: running_env default to test (fix OCA#44)
eb94b9e
simahawk added a commit to simahawk/server-env that referenced this issue Apr 2, 2020
@simahawk
server_environment: running_env default to test (fix OCA#44)
4f3b17c
@pedrobaeza
Copy link
Member

I suppose this is the same error I have now in OCA/l10n-spain:

https://travis-ci.org/github/OCA/l10n-spain/jobs/670689281#L2437

Is it?

@pedrobaeza pedrobaeza mentioned this issue Apr 3, 2020
Merged
1 task
simahawk added a commit to simahawk/server-env that referenced this issue Apr 10, 2020
@simahawk
fixup! server_environment: running_env default to test (fix OCA#44)
5df8a5a
simahawk added a commit to simahawk/server-env that referenced this issue Apr 10, 2020
@simahawk
fixup! server_environment: running_env default to test (fix OCA#44)
3024745
pedrobaeza added a commit that referenced this issue Apr 10, 2020
@pedrobaeza
Merge pull request #45 from simahawk/13-default-running-env
44d7573 
server_environment: running_env default to `test` (fix #44)
alexey-pelykh pushed a commit to brainbeanapps/server-env that referenced this issue Apr 15, 2020
@simahawk @alexey-pelykh
server_environment: running_env default to test (fix OCA#44)
60e368e
alexey-pelykh pushed a commit to brainbeanapps/server-env that referenced this issue Apr 15, 2020
@simahawk @alexey-pelykh
fixup! server_environment: running_env default to test (fix OCA#44)
2d2cb1f
alexey-pelykh pushed a commit to brainbeanapps/server-env that referenced this issue Apr 15, 2020
@simahawk @alexey-pelykh
server_environment: running_env default to test (fix OCA#44)
c3f2b6f
alexey-pelykh pushed a commit to brainbeanapps/server-env that referenced this issue Apr 15, 2020
@simahawk @alexey-pelykh
fixup! server_environment: running_env default to test (fix OCA#44)
c167f54
alexey-pelykh pushed a commit to brainbeanapps/server-env that referenced this issue Apr 16, 2020
@simahawk @alexey-pelykh
server_environment: running_env default to test (fix OCA#44)
7d6ebde
alexey-pelykh pushed a commit to brainbeanapps/server-env that referenced this issue Apr 16, 2020
@simahawk @alexey-pelykh
fixup! server_environment: running_env default to test (fix OCA#44)
ecce24d
guewen added a commit to guewen/odoo-cloud-platform that referenced this issue May 11, 2020
@guewen
Fix build after changes in server_environment
3b178e0 
Now the default env is "test", and we don't need the directory
at all.
See OCA/server-env#44
@guewen guewen mentioned this issue May 11, 2020
Merged
guewen added a commit to guewen/odoo-cloud-platform that referenced this issue May 11, 2020
@guewen
Fix build after changes in server_environment
8524eef 
Now the default env is "test", and we don't need the directory
at all.
See OCA/server-env#44
@guewen guewen mentioned this issue May 11, 2020
Merged
guewen added a commit to guewen/odoo-cloud-platform that referenced this issue May 11, 2020
@guewen
Fix build after changes in server_environment
9bdd4b6 
Now the default env is "test", and we don't need the directory
at all.
See OCA/server-env#44
@guewen guewen mentioned this issue May 11, 2020
Merged
mourad-ehm pushed a commit to akretion/server-env that referenced this issue Oct 12, 2020
@simahawk @mourad-ehm
server_environment: running_env default to test (fix OCA#44)
c54bf8c
mourad-ehm pushed a commit to akretion/server-env that referenced this issue Oct 12, 2020
@simahawk @mourad-ehm
fixup! server_environment: running_env default to test (fix OCA#44)
c5ea873
sbejaoui pushed a commit to acsone/server-env that referenced this issue Oct 14, 2020
@simahawk @sbejaoui
server_environment: running_env default to test (fix OCA#44)
34e312e
sbejaoui pushed a commit to acsone/server-env that referenced this issue Oct 14, 2020
@simahawk @sbejaoui
fixup! server_environment: running_env default to test (fix OCA#44)
307f415
JuMiSanAr pushed a commit to camptocamp/server-env that referenced this issue Oct 18, 2021
@simahawk @JuMiSanAr
server_environment: running_env default to test (fix OCA#44)
7dd60f3
JuMiSanAr pushed a commit to camptocamp/server-env that referenced this issue Oct 18, 2021
@simahawk @JuMiSanAr
fixup! server_environment: running_env default to test (fix OCA#44)
847b9e6
JuMiSanAr pushed a commit to camptocamp/server-env that referenced this issue Oct 22, 2021
@simahawk @JuMiSanAr
server_environment: running_env default to test (fix OCA#44)
c40bd96
JuMiSanAr pushed a commit to camptocamp/server-env that referenced this issue Oct 22, 2021
@simahawk @JuMiSanAr
fixup! server_environment: running_env default to test (fix OCA#44)
f9d5d2a
simahawk added a commit to camptocamp/server-env that referenced this issue Oct 22, 2021
@simahawk
server_environment: running_env default to test (fix OCA#44)
41c6f4a
FrancoMaxime pushed a commit to acsone/server-env that referenced this issue Sep 1, 2022
@simahawk @FrancoMaxime
server_environment: running_env default to test (fix OCA#44)
9006aaa
FrancoMaxime pushed a commit to acsone/server-env that referenced this issue Oct 4, 2022
@simahawk @FrancoMaxime
server_environment: running_env default to test (fix OCA#44)
aab6427
moitabenfdz pushed a commit to DynAppsNV/server-env that referenced this issue Nov 13, 2023
@simahawk @moitabenfdz
server_environment: running_env default to test (fix OCA#44)
569b00c
thienvh332 pushed a commit to thienvh332/server-env that referenced this issue Sep 20, 2024
@simahawk @thienvh332
server_environment: running_env default to test (fix OCA#44)
d8dff23
thienvh332 pushed a commit to thienvh332/server-env that referenced this issue Sep 24, 2024
@simahawk @thienvh332
server_environment: running_env default to test (fix OCA#44)
bbfca6c
sbidoul pushed a commit that referenced this issue Sep 30, 2024
@simahawk @sbidoul
server_environment: running_env default to test (fix #44)
790631b
chaule97 pushed a commit to chaule97/server-env that referenced this issue Oct 1, 2024
@simahawk @chaule97
server_environment: running_env default to test (fix OCA#44)
56df164
chaule97 pushed a commit to chaule97/server-env that referenced this issue Oct 1, 2024
@simahawk @chaule97
server_environment: running_env default to test (fix OCA#44)
3df25fa
chaule97 pushed a commit to chaule97/server-env that referenced this issue Oct 1, 2024
@simahawk @chaule97
server_environment: running_env default to test (fix OCA#44)
09d549f
sbidoul pushed a commit that referenced this issue Oct 2, 2024
@simahawk @sbidoul
server_environment: running_env default to test (fix #44)
db39734
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@simahawk @guewen @amh-mw @dreispt @pedrobaeza