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

TypeError: get_config() takes at least 2 arguments (1 given) #358

Closed
jezdez opened this issue Apr 3, 2018 · 3 comments
Closed

TypeError: get_config() takes at least 2 arguments (1 given) #358

jezdez opened this issue Apr 3, 2018 · 3 comments
Assignees
Milestone

Comments

@jezdez
Copy link

jezdez commented Apr 3, 2018

https://sentry.prod.mozaws.net/operations/data-stmo-prod/issues/3477358/

TypeError: get_config() takes at least 2 arguments (1 given)
(5 additional frame(s) were not displayed)
...
  File "flask/app.py", line 1625, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "dockerflow/flask/app.py", line 304, in _heartbeat_view
    detail = self._heartbeat_check_detail(check)
  File "dockerflow/flask/app.py", line 282, in _heartbeat_check_detail
    errors = list(filter(lambda e: e.id not in self.silenced_checks, check()))
  File "dockerflow/flask/app.py", line 388, in decorated_function
    return func(*args, **kwargs)
  File "dockerflow/flask/checks/__init__.py", line 76, in check_migrations_applied
    config = migrate.get_config()

TypeError: get_config() takes at least 2 arguments (1 given)
@jezdez jezdez self-assigned this Apr 3, 2018
@jezdez
Copy link
Author

jezdez commented Apr 3, 2018

This fixed in python-dockerflow: mozilla-services/python-dockerflow@036927b

@jezdez
Copy link
Author

jezdez commented Apr 3, 2018

Since our requirements file is installing the latest version of redash-stmo, which in turn installs python-dockerflow we'll need to do a minor bump for it to deploy on the next release.

jezdez added a commit that referenced this issue Apr 3, 2018
@jezdez
Copy link
Author

jezdez commented Apr 3, 2018

https://pypi.org/project/redash-stmo/2018.4.0/ released.

@robotblake does #360 suffice? It'd be picked up on the next deploy.

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