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

Invalid argument(s) 'pool_maxconn' sent to create_engine() #494

Closed
Natim opened this issue Mar 5, 2016 · 9 comments
Closed

Invalid argument(s) 'pool_maxconn' sent to create_engine() #494

Natim opened this issue Mar 5, 2016 · 9 comments
Labels
Milestone

Comments

@Natim
Copy link
Member

Natim commented Mar 5, 2016

TypeError: Invalid argument(s) 'pool_maxconn' sent to create_engine(), using configuration PGDialect_psycopg2/QueuePoolWithMaxBacklog/Engine.  Please check that the keyword arguments are appropriate for this combination of components.

This error occured while deploying Kinto to stage. @leplatrem thoughts?

@Natim Natim added the bug label Mar 5, 2016
@almet
Copy link
Member

almet commented Mar 5, 2016

What's the configuration that was used? How can we reproduce?

@Natim
Copy link
Member Author

Natim commented Mar 6, 2016

@almet
Copy link
Member

almet commented Mar 6, 2016

Hmm, I don't have any access there. Can you copy and paste the stacktrace in this bug (I think that's something we might want to do for these issues anyway, because not all contributors have access to the sentry stack).

@Natim
Copy link
Member Author

Natim commented Mar 7, 2016

Usually I shared them publicly so it should work.

@Natim
Copy link
Member Author

Natim commented Mar 7, 2016



Stacktrace (appel le plus récent en dernier) : 

  File "app.wsgi", line 23, in <module>
    application = main(config.items('DEFAULT'), **dict(config.items('app:main')))
  File "kinto/__init__.py", line 43, in main
    default_settings=DEFAULT_SETTINGS)
  File "cliquet/initialization.py", line 530, in initialize
    config.include("cliquet", route_prefix=api_version)
  File "pyramid/config/__init__.py", line 798, in include
    c(configurator)
  File "cliquet/__init__.py", line 155, in includeme
    step_func(config)
  File "cliquet/initialization.py", line 190, in setup_storage
    backend = storage_mod.load_from_config(config)
  File "cliquet/storage/postgresql/__init__.py", line 743, in load_from_config
    client = create_from_config(config, prefix='storage_')
  File "cliquet/storage/postgresql/client.py", line 94, in create_from_config
    engine = sqlalchemy.engine_from_config(settings, prefix=prefix, url=url)
  File "sqlalchemy/engine/__init__.py", line 427, in engine_from_config
    return create_engine(url, **options)
  File "sqlalchemy/engine/__init__.py", line 386, in create_engine
    return strategy.create(*args, **kwargs)
  File "sqlalchemy/engine/strategies.py", line 144, in create
    engineclass.__name__))

@Natim
Copy link
Member Author

Natim commented Mar 7, 2016

I think it is because the config name is wrong, @phrawzty can you verify that it looks like this? http://kinto.readthedocs.org/en/latest/configuration/production.html#recommended-settings

Or paste the config here?

@leplatrem
Copy link
Contributor

Please use this template for the configuration:
https://github.com/Kinto/kinto-dist/blob/master/config/kinto.ini

@phrawzty
Copy link
Contributor

phrawzty commented Mar 7, 2016

The ini deployed to stage is legacy - I assume that at one point those maxconn variables were valid. A simple s/maxconn/size/ caused those errors to disappear, as expected. 😄

@leplatrem
Copy link
Contributor

Great! Closing this.

@leplatrem leplatrem modified the milestone: 1.12 Mar 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants