-
Notifications
You must be signed in to change notification settings - Fork 201
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
generate fake-data fails #195
Comments
@wardi do you have any immediate suggestion to add anything in the downstream Dockerfile that would make it work? |
@rantoniuk you could use the dev image instead, i.e.
@kowh-ai WDYT, should this be in the readme as well? I was considering adding a collection of little bash scripts for related tasks in dev like:
that last one would make creating test data like this a little easier. |
I actually went the other way round and just added:
to my Dockerfile along other plugins. Just as stated in the linked ckan/ckan issue, I feel like this is not a "dev" dependency. For example commands, I'd propose to add a README.md section:
docker compose exec ckan ckan -c /srv/app/ckan.ini <command>
e.g.
docker compose exec ckan ckan -c /srv/app/ckan.ini --help with most used usecases as examples. |
@rantoniuk The CKAN cli documentation does specify when it's a "Development" task eg: but yes it's a bit unclear and should be clarified a little better. There are many ways make changes to a CKAN Docker environment - running a cli command from the host, running it from within the running CKAN container, adding configuration to the locally built CKAN image like you have done @wardi I agree an update to the README would be useful and yes please your bash script additions would be much appreciated. I can update the README this week with some examples of how to run commands from the host in a running CKAN Dev container |
On a clean setup of 2.11 from pre-built docker images, the fake data generator doesn't work:
I wouldn't expect to have to build my own images for this, no?
Ref: ckan/ckan#8177
The text was updated successfully, but these errors were encountered: