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

Use official Nginx package #1208

Merged
merged 1 commit into from
Aug 6, 2020
Merged

Use official Nginx package #1208

merged 1 commit into from
Aug 6, 2020

Conversation

swalkinshaw
Copy link
Member

@swalkinshaw swalkinshaw commented Jul 23, 2020

Extracted from #1197

Note: this will not re-install Nginx if you re-provision an already provisioned server since it's the same apt package name.

@tangrufus
Copy link
Member

Question: What is the procedure to reinstall Nginx with the official ppa?

@swalkinshaw
Copy link
Member Author

You'd have to manually uninstall via apt-get remove nginx after running these changes; then provision again to install from the new package. I don't know of a better/different way.

@swalkinshaw swalkinshaw merged commit 5b1903b into master Aug 6, 2020
@swalkinshaw swalkinshaw deleted the switch-nginx-ppa branch August 6, 2020 02:20
@swalkinshaw swalkinshaw mentioned this pull request Sep 10, 2020
3 tasks
@strarsis
Copy link
Contributor

strarsis commented Sep 21, 2020

@swalkinshaw
Copy link
Member Author

swalkinshaw commented Oct 2, 2020

@strarsis probably, but I'm wary of coding that by default into Trellis. Especially since it involves an uninstall + reinstall which means downtime.

edit: there's downtime necessary anyway, but I'd rather people know that and do it explicitly

@mockey
Copy link
Contributor

mockey commented Oct 3, 2020

Setting nginx_package_state: latest worked for me in re-provisioning.

@strarsis
Copy link
Contributor

strarsis commented Oct 4, 2020

@mockey: Even better, when a specific nginx release is used that is from the official PPA, so the nginx version is locked in, without installing just the latest available one.

@swalkinshaw
Copy link
Member Author

latest works in this specific case... but I think it could lead to unintended upgrades or breaking changes?

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 this pull request may close these issues.

4 participants