Skip to content
This repository has been archived by the owner on Sep 10, 2021. It is now read-only.

Installation & Migration issues #87

Conversation

charles-marion
Copy link
Contributor

I tried to migrate an instance and I wasn't able to login (because of the secure cookie flag).
I also tried to install Midas and it didn't work. A folder was not writable and I didn't get a message.

I fixed these issues.

Charles Marion added 2 commits February 11, 2015 17:05
…n code (now it is set to 0 in code). After a migration, you don't have the config value so it was enabling the secure cookie => you cannot log it
@charles-marion charles-marion changed the title Charles install migration Installation & Migration issues Feb 11, 2015
@zachmullen
Copy link
Member

This smells of needing a reusable function.

@cpatrick
Copy link
Contributor

@zachmullen You're right about that 😄

@charles-marion @jamiesnape In terms of the default value of the secure cookie field, I'm torn. If the server is running over HTTPS using secure is almost certainly what they want, but it does add an extra step to the upgrade of old systems.

@jamiesnape
Copy link
Contributor

The secure cookie setting is not a bug. I will update the documentation before the release.

@jamiesnape
Copy link
Contributor

The permissions issue needs to be fixed in an alternate way. This fix breaks App Engine support.
Actually this is not a bug either.

@jamiesnape
Copy link
Contributor

I think the issue for both this issues is that it just needs documentation, and I will work on that when I have time. The simple fact is that the installation and upgrade process is not identical to how it was in the previous version, and that is deliberate for security and cloud compatibility reasons.

@jamiesnape
Copy link
Contributor

Issue #88 created for documentation.

@charles-marion
Copy link
Contributor Author

Right now, if you install MIDAS on Linux. It tells you "Your system is Ok". You click on Next and nothing happen.

If you migrate (still on linux) your instance, you cannot log in and you don't know why.
Also if you were using the developpement mode, your instance became very slow and you don't know why.

In my opinion these are major issues and should be fixed...

@jamiesnape
Copy link
Contributor

@charles-marion Hence why I need to update the documentation.

@jcfr
Copy link
Contributor

jcfr commented Feb 11, 2015

@charles-marion Thanks for your contribution 👍

@jamiesnape Could the user be notified with a meaningful message ?

While discussing the secure cookie setting, could the commit 3ffe103 be integrated ? Seems this one is not subject to controversy.

@jamiesnape
Copy link
Contributor

@jcfr No, it will break App Engine support.

@jamiesnape
Copy link
Contributor

@jcfr Not really. You are checking whether a folder is writeable, but there is no concept of folders on Cloud Storage. The only solution is to document the pre-installation steps.

@jcfr
Copy link
Contributor

jcfr commented Feb 11, 2015

@jamiesnape I see. Do this mean such check should be done on a per asset store basis ? Midas should check if the given asset store is writable etc ...

@jamiesnape
Copy link
Contributor

@jcfr It should, but that would not help, unfortunately, as Midas handles configuration files differently to other files. The problem is that configuration files are copied to another location altogether during installation (not an asset store).

The correct solution, I think, is to move the configuration (other than of the database connection) to the database. I have a prototype of that (I demonstrated it in a tech tip), but it is some way from being ready for testing. That also has the side effect of significant performance improvements.

@jcfr
Copy link
Contributor

jcfr commented Feb 11, 2015

@jamiesnape Thanks for the detailed explanation. Look like there are some exciting changes on the way. I like the idea of having the database being the only entity representing the state of a server.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants