-
Notifications
You must be signed in to change notification settings - Fork 95
feat: Debian Trixie Support #280
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
base: main
Are you sure you want to change the base?
Conversation
Here we specifically target it as I'm unsure if Ubuntu has support for deb882 (I'm sure it does, but I havn't had time to test it)
This fixes the big warnings people were reporting due to vars files not being found.
This fixes the big warnings people were reporting due to vars files not being found.
|
Hi @ticoombs. I tried this PR on my system but ran into quite a few issues actually, around 4-5 different ones... |
And they were? My testing might have been slightly different as i skip all SSL configuration as i already had certificates. edit: oh my... how did these errors even work! Thanks for testing them for me @sanatsathaye |
|
Validated on debian13 and ubuntu24.04 with VMs. The only thing i installed before running the ansible scripts was ssh (via the installer) so i think its safe to say its okay now. |
|
Oh crap, sorry I meant to post another reply but got carried away with something else. That solves a few issues for me. But considering the changes there are quite a few points to take into consideration:
Now, after all the above has been done, also imagine a scenario where it's not a fresh Debian install, this playbook has already been run before on a system.
|
|
Thanks! All very valid points, i hadn't thought of the 12->13 upgrade path and rerunning them. For deb882 I wanted to only touch on Trixie in this PR as I wanted to limit the exposure but I think you are right. We need to either make the change for all apt related distros or none, as the clash will adversely affect users. Interesting that the server didn't have that installed. When I created a VM from the debian iso, I only chose ssh and "server-tools", (or whatever it was called). Probably won't be till next week for me to do more testing now. If you want I can add you to my repo and you can update the branch with the changes :p. Otherwise just having someone validate is amazing. 🙇 Thanks again |
|
No problem. Tbh the earliest I would be able to make the changes myself is Friday but yeah sure you can add me, I can test out on the various distros too. |
| {% endif %} | ||
| volumes: | ||
| # starting postgres 18+ this will need to be `/volumes/postgres:/var/lib/postgresql:Z` | ||
| # ref: https://github.com/pgautoupgrade/docker-pgautoupgrade?tab=readme-ov-file#error-message-when-mounting-data-to-varlibpostgresqldata-on-postgres-v18 |
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.
I also don't have any objections to you doing a postgres v18 upgrade here also. Then there's no need for the comment.
Alternatively, just remove this comment and do the postgres v18 upgrade in another PR.
warningcomment for us around postgres 18+Closes: #267