-
Notifications
You must be signed in to change notification settings - Fork 24
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
Ckan 2.10 upgrade #88
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
luccasmmg
approved these changes
Aug 12, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
luccasmmg
approved these changes
Aug 15, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Testing the upgrade
On the
master
branch, build/start CKAN (this is the old CKAN 2.7 with Python 2—you need to seed this portal first before testing the migration). All commands should be run inckan-cloud-docker
(unless stated otherwise).make secret
(just go with the default values)make start O=vital-strategies
make shell O=vital-strategies S=ckan C="/usr/lib/ckan/venv/bin/paster --plugin=ckan sysadmin add ckan_admin password=test1234 [email protected] -c /etc/ckan/production.ini"
make shell O=vital-strategies S=ckan C="/usr/lib/ckan/venv/bin/paster --plugin=ckanext-querytool seed_portal -c /etc/ckan/production.ini"
git switch ckan-2.10-upgrade
(you might need togit stash
first, but make sure the changes from step 2 are re-applied before rebuilding/starting again)make backup-db O=vital-strategies
(confirm that you haveckan.dump
,datastore.dump
andckan_data.tar.gz
in the current directory after running this command)make stop O=vital-strategies
datapusher-plus
:export DATAPUSHER_TYPE=datapusher-plus
(always do this in the current terminal session before starting up CKAN 2.10/Python 3 withckan-cloud-docker
)make clean-rebuild O=vital-strategies
make upgrade-db O=vital-strategies
make stop O=vital-strategies
(copy the API token that's output at the end, for the next step)make secret
again and paste the token when prompted (step 13—"Enter Datapusher API token")make start O=vital-strategies