-
Notifications
You must be signed in to change notification settings - Fork 390
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
Control PostgreSQL installed version with 'db_version' variable #169
base: master
Are you sure you want to change the base?
Conversation
Version 12 is used in the playbook Closes jcalazan#168 Note: PostgreSQL version is managed with 'db_version' variable. However, there is no DB upgrade process with DB previous code deletion, DB stopped and restarted. This is not an issue compared to playbook version without this commit though.
…postgres-version-control
@jpmjpmjpm Thank you for the PR!!! A couple quick questions: So I understand, is your comment saying that increasing the db version will not upgrade PSQL in the way it should be upgraded? If so would you make a comment saying that above the Also - if we put in |
@YPCrumble I made a test and the PR indeed installs PostgreSQL 12.4 (latest 12 minor version) on a 'brand new server' . Which is good! What I don't know is what happens if you already have an existing previous minor version installed, let say 12.3. I suspect that 12.4 will be installed but I don't know if it will supersede 12.3. Such a test seems not easy to perform as I haven't find (based on quick Google search) a way to install an older minor version than the current one. However, the PR is better than current version of the playbook, as current version installs version 13 and starts it without it being used by the application. Note: the playbook (with or without the PR) can't be used to upgrade DB data between major versions (i.e. 12 to 13). For that this post is of interest. I imagine this was clear, but better to precise it! |
@jpmjpmjpm to avoid any potential issues, what do you think about specifying |
@YPCrumble I was looking to do so, but I don't think this is an easy process. Replacing The PR I propose may not be the best as I don't know how it will proceed in term of minor version upgrades. However, it is still an improvement compared to current version which installs a new major version on top of a previous one and starts it in parallel. I also noticed in my case that the running version for the application is still 12.4, that 13 is installed and that the psql version used is 13 connected to 12.4! For a subsequent PR, I was looking at PostgreSQL minor version upgrade post. On my side, I uninstalled 13 and everything went fine. To be used with caution though I imagine! |
Version 12 is used in the playbook
Closes #168
Note: PostgreSQL version is managed with 'db_version' variable. However, there is no DB upgrade process with DB previous code deletion, DB stopped and restarted. This is not an issue compared to playbook version without this commit though.