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

regression: gunicorn.preload_databases in v1.9 #30

Open
gracinet opened this issue Jan 17, 2015 · 2 comments
Open

regression: gunicorn.preload_databases in v1.9 #30

gracinet opened this issue Jan 17, 2015 · 2 comments
Labels
bug

Comments

@gracinet
Copy link
Contributor

anybox.recipe.openerp introduced a uniform parser for multiline options.
It causes a regression in gunicorn.preload_database in the most common case where there is just one database. The error appears upon execution:

OperationalError: FATAL: la base de données « o » n'existe pas

(here o is the first letter of the database name)

If there are truly several databases, then the recipe fails
gracinet pushed a commit that referenced this issue Jan 17, 2015
GitHub #30: failing unit tests that reproduce the issue
9faa4ae 
Also, a passing doctest for the multine parsing to check that the result is
always an iterable, even if there's just one line.
gracinet pushed a commit that referenced this issue Jan 17, 2015
Merge pull request #28 from petrus-v/1.9
82f1486 
#30 fix regression on gunicorn.preload_databases
@gracinet
Copy link
Contributor Author

The new unit test also shows that it didn't work at all for multiple databases (edited issue description as well)

gracinet pushed a commit that referenced this issue Jan 17, 2015
GitHub #30: finalizing the unit test and changelog
92b3884
@gracinet
Copy link
Contributor Author

This needs forward-porting to master

@gracinet gracinet added the bug label Jan 17, 2015
gracinet pushed a commit that referenced this issue Jan 17, 2015
General merge from a.r.openerp-1.9
6e3139d 
This brings fixes for #23 and #30 to a.r.odoo

Conflicts:
	CHANGES.rst
	anybox/recipe/odoo/tests/test_server.py
	doc/conf.py
	setup.cfg
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@gracinet