Skip to content

Adding data and log path variable #95

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

chadek
Copy link

@chadek chadek commented May 3, 2023

The role currently not allow to change log and data path, this PR allow it

@github-actions
Copy link

github-actions bot commented Sep 3, 2023

This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark issues as stale.

@github-actions github-actions bot added the stale label Sep 3, 2023
@fmiqbal
Copy link

fmiqbal commented Sep 5, 2023

also need this

@github-actions github-actions bot removed the stale label Sep 10, 2023
@p3l1
Copy link
Contributor

p3l1 commented Nov 15, 2023

I would also make use of this PR.

@pulse-mind
Copy link

Hi, would you like to accept and merge this PR please ?

Copy link

This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark issues as stale.

@github-actions github-actions bot added the stale label Jun 23, 2024
@pulse-mind
Copy link

Hi, would you like to accept and merge this PR please ?

@github-actions github-actions bot removed the stale label Jun 30, 2024
Copy link

github-actions bot commented Nov 3, 2024

This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark issues as stale.

@github-actions github-actions bot added the stale label Nov 3, 2024
@p3l1
Copy link
Contributor

p3l1 commented Nov 4, 2024

Hey, this PR is still relevant and the feature would enhance the role further. Thanks!

@pierrejego
Copy link

@geerlingguy thanks for this playbook. Do you think you can merge this PR ? This would be a major update for me.

pierrejego added a commit to georchestra/ansible that referenced this pull request Jan 8, 2025
Path and logs are already propose as a PR in gueerlingguy repo. Waiting for validation geerlingguy/ansible-role-elasticsearch#95

If not merge in the week I will add a new task to change path and logs value after installed.
landryb pushed a commit to georchestra/ansible that referenced this pull request Jan 17, 2025
* Change elastic roles to geerlingguy.elasticsearch role

Update version in requirement to fit latest ansible version

Installation working and deployement succeded but still some work to do to change defaut folder to store data and logs
el_data_dirs and  es_config

link to #142

* add extra vars

Path and logs are already propose as a PR in gueerlingguy repo. Waiting for validation geerlingguy/ansible-role-elasticsearch#95

If not merge in the week I will add a new task to change path and logs value after installed.

* This allow to change logs and data path for elasticsearch

Change requirement to add specific role ( fork from geerlingguy because no response to PR)
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.

6 participants