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

Add a variable to define PostgreSQL version #168

Open
jpmjpmjpm opened this issue Nov 1, 2020 · 2 comments · May be fixed by #169
Open

Add a variable to define PostgreSQL version #168

jpmjpmjpm opened this issue Nov 1, 2020 · 2 comments · May be fixed by #169

Comments

@jpmjpmjpm
Copy link
Contributor

This is more a question than an issue

Currently, the installed PostgreSQL version is not determined explicitly with a variable. Shouldn't a variable added? I ask the question as I have an app version installed early 2020 with PostgreSQL v12. However, I provisioned another server in October and I saw that PostgreSQL installed version is v13 while I was willing to have the same installation.

I can obviously modify my forked version but I was willing to have your views guys on the topic.

Thanks, Jean-Pierre.

@YPCrumble
Copy link
Collaborator

@jpmjpmjpm this is a really great suggestion.

I actually have a server that has two versions of Postgres installed simultaneously due to this issue...and I'm hesitant to try uninstalling the updated version for fear of breaking things. A PR would be awesome!

@jpmjpmjpm
Copy link
Contributor Author

jpmjpmjpm commented Nov 1, 2020

I created PR #169 to solve the issue. Thanks for reviewing it!

Note: by the way PR #156 is still waiting for a merge.

jpmjpmjpm pushed a commit to jpmjpmjpm/ansible-django-stack that referenced this issue Nov 2, 2020
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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants