dathere/datapusher-plus

Migrations

Opened this issue · 3 comments

Hi Team. First of all thanks for the job with this tool.

I'm using the branch dev-v1.0 and I'm having DB troubles.
I realize that you have made changes to migrations manually.
I assumed this is a ~stable branch and maybe I'm wrong.
It looks like you are assumming no one is using it.
I just want to flag I'm trying to use datapusher+ as a extension.

Do you have any suggestion (branch) to use this as extension in a productive environment?

Are you getting the following error? Because I am...and I'm using dev-v1.0. I'm getting this error when I attempt to run ckan -c /etc/ckan/default/ckan.ini db upgrade -p datapusher_plus and I haven't quite figured out why this is happening.

`
Traceback (most recent call last):

File "/usr/lib/ckan/default/bin/ckan", line 5, in
from ckan.cli.cli import ckan

File "/usr/lib/ckan/default/lib/python3.10/site-packages/ckan/cli/cli.py", line 14, in
from ckan.config.middleware import make_app

File "/usr/lib/ckan/default/lib/python3.10/site-packages/ckan/config/middleware/init.py", line 11, in
from ckan.config.environment import load_environment

File "/usr/lib/ckan/default/lib/python3.10/site-packages/ckan/config/environment.py", line 15, in
import ckan.model as model

File "/usr/lib/ckan/default/lib/python3.10/site-packages/ckan/model/init.py", line 21, in
import ckan.model.meta as meta

File "/usr/lib/ckan/default/lib/python3.10/site-packages/ckan/model/meta.py", line 84, in
registry = orm.registry(metadata=metadata)

AttributeError: module 'sqlalchemy.orm' has no attribute 'registry'
`

@avdata99 the dev-v1.0 branch should be moved to the main branch soon @jqnatividad, this one is up to you

Regarding the migrations changes, maybe it was wrong to do all those changes directly in the branch but i was rushing to make it work also in prod environment

If you have any issues please let me know and i would try to help to resolve those

@dtmarc this error looks like issue with the sqlalchemy version
we are using 1.4.41 that comes with CKAN core requirements