-
-
Notifications
You must be signed in to change notification settings - Fork 112
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
[Upgrade Ubuntu] changes for uk_staging2 #937
Conversation
I guess the two versions were required during a transition period. All servers have been running 3.1.4 for a while now so that's all we should need.
It is a thousand times more readable than the default JSON format.
And thus use Postgres 12 which is packaged with Ubuntu 20.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
✅ Successful deployment now, after restarting and re-provisioning with the multi_redis role.
Now to provision ofn-security. It went so well.. until right at the end:
It was left in "exited" state which doesn't sound good. So I manually restarted (which is the next step anyway)
🤷 I'm not sure exactly what the "reload suricata rules" task is trying to do, but if suricata is functioning then I don't want to investigate! |
The host_id variable is only used for the new relic data. I'd like to :
|
This reverts commit f1fd8a8.
And documentation, wow what a difference it could make.
This makes it much clearer when you SSH into the machine. I wanted to also changed the host_id to use an underscore, like uk_staging, to match our inventory names. But apparently underscores are not allowed for hostnames. And dashes are not allowed in ansible inventory names.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
The service_facts task was failing on uk_staging2. Dunno why, but I don't think we need any of this anymore. It was a safety check for when deploying to an old version of OFN, but I dont' think we need to handle that gracefully anymore.
For reference, here's the error occurring for ansible built-in command Due to: ansible/ansible#68536 |
I'm sure it will catch someone out one day.
If this works, probably also need to apply it in roles/webserver/handlers/main.yml
Although I'm not quite finished with uk_staging server, I think it's high time this PR is reviewed. |
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.
Good work.
- name: run preflight checks | ||
include_tasks: preflight_checks.yml |
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.
Good call.
ruby_versions: | ||
- version: 2.7.3 | ||
env: | ||
RUBY_CONFIGURE_OPTS: "{{ ruby_compile_options }}" | ||
- version: 3.0.3 | ||
- version: 3.1.4 | ||
env: | ||
RUBY_CONFIGURE_OPTS: "{{ ruby_compile_options }}" |
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.
Ah, good one. New Ruby versions are now automatically installed during deploys. So we don't need to run Ansible for that. This is just for the initial version.
I wish that we could skip the version and take the version number from the ofn code but this role requires a version. So if we want to set up rbenv without installing ruby straight away then we can't use this role. Just one of the annoying Ansible things that we could replace with shell scripts...
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.
Nice one !
Documentation
Updated: