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

Auto-upgrade doesn't fully upgrade database. (and breaks deployment) #808

Closed
Nhorning opened this issue Aug 22, 2012 · 4 comments
Closed

Comments

@Nhorning
Copy link

When I auto upgraded my deployment (nepalmonitor.org) all the messages said that the upgrade had taken place successfully, but when I refreshed the site I was met with a database error.

I tried the relevant scripts from the SQL directory manually in cpanel sql admin, and encountered errors when running "upgrade91-92"

Investigation showed the problem was missing " ` " around all instances of the word "settings."

For instance:

SELECT 'site_name' AS key, site_name AS value FROM settings WHERE id = 1

should read

SELECT 'site_name' AS key, site_name AS value FROM settings WHERE id = 1

This issue is already fixed in github files: https://github.com/ushahidi/Ushahidi_Web/blob/develop/sql/upgrade91-92.sql

However, it's NOT fixed in the live auto-upgrade as of 20th August. This will cause everyone using the auto-upgrade to break their deployment and fish around in SQL like I did unless the change is reflected in the live version.

@ghost ghost assigned eyedol Aug 28, 2012
@rjmackay
Copy link
Contributor

rjmackay commented Apr 4, 2013

This should be fixed now - however we should possibly reroll the relevant upgrade scripts
anything to 2.4 or 2.4.1

@rjmackay
Copy link
Contributor

rjmackay commented Apr 4, 2013

Got this wrong - its the upgrade to 2.5 that needed a fix. I've updated the downloads here https://wiki.ushahidi.com/display/WIKI/Ushahidi+Platform+Downloads
but need to change the auto upgrader links

@rjmackay
Copy link
Contributor

rjmackay commented Apr 4, 2013

@eyedol can you update the auto upgrade to use the new rackspace cloud files URLs

@eyedol
Copy link
Collaborator

eyedol commented Apr 4, 2013

Fixed!

@rjmackay rjmackay closed this as completed Apr 5, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants