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

Ansible install - move sysctl tasks to packaging logic #2725

Open
msheiny opened this issue Dec 12, 2017 · 2 comments
Open

Ansible install - move sysctl tasks to packaging logic #2725

msheiny opened this issue Dec 12, 2017 · 2 comments
Labels
goals: Improve Ansible logic / smoother install help wanted Issues we would definitely appreciate volunteer help with ops/deployment

Comments

@msheiny
Copy link
Contributor

msheiny commented Dec 12, 2017

Feature request

Description

The sysctl module we are using is particularly slow to install. It is looping over a list of sysctl_flags and running a reload (sysctl -p) after each one is laid out. Even if no changes are made this seems to be particularly annoying.

We should consider templating the sysctl file and adding a sysctl refresh handler whenever that changes. Take a look at what some other community members are doing - https://github.com/debops/ansible-sysctl/blob/master/tasks/main.yml

User Stories

As a securedrop admin, the waiting time for sysctl tasks to finish is TOO DANG HIGH

@conorsch conorsch added ops/deployment help wanted Issues we would definitely appreciate volunteer help with labels Aug 16, 2018
@conorsch
Copy link
Contributor

We could move these config items to the securedrop-keyring package, which would provide the additional benefit of allowing config changes via automatic updates. As it stands, the sysctl options are set at install time, and rarely asserted thereafter. To be clear, moving the sysctl options to a deb package would certainly reduce provisioning time, as requested in the OP.

@eloquence eloquence changed the title Ansible install - speed up sysctl tasks Ansible install - move sysctl tasks to pacakaging logic Jan 8, 2021
@conorsch
Copy link
Contributor

conorsch commented Jan 8, 2021

In the context of #2641, it's a good idea to move these sysctl settings to a package. In fact, we already did that for vm stack heap settings during #1861. Moving all the settings into a package would give us much better control, and minimize our need to pester admins.

@legoktm legoktm changed the title Ansible install - move sysctl tasks to pacakaging logic Ansible install - move sysctl tasks to packaging logic Sep 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
goals: Improve Ansible logic / smoother install help wanted Issues we would definitely appreciate volunteer help with ops/deployment
Projects
None yet
Development

No branches or pull requests

2 participants