Skip to content
This repository has been archived by the owner on Nov 15, 2024. It is now read-only.

add option to disable numa, still on by default #50

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

schofield
Copy link
Contributor

No description provided.

@schofield schofield requested a review from a team August 5, 2022 23:02
@schofield schofield changed the title add option to disable number, off by default add option to disable numa, off by default Aug 5, 2022
@schofield schofield changed the title add option to disable numa, off by default add option to disable numa, still on by default Aug 10, 2022
tasks/main.yml Outdated
- name: Create Humio SystemD template configuration
tags: humio-update
template:
src: "humio.service.j2"
Copy link
Member

@SaaldjorMike SaaldjorMike Aug 11, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wouldn't it be better just to make a small adjustment to the existing template which would leave out the /usr/bin/numactl --cpunodebind=${NUMA_NODES} --membind=${NUMA_NODES} part of ExecStart command?

Then we don't have two very similar templates to maintain.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SaaldjorMike I made the change you suggested. Have a look again please.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants