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

Better handling of the service in stopped state #98

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

Conversation

lisuml
Copy link

@lisuml lisuml commented Nov 29, 2023

  • Prevented waiting for Elasticsearch when elasticsearch_service_state is set to stopped.
  • Avoided triggering 'restart elasticsearch' handler under the same condition.

Reasoning:
Encountered a need to configure security features mandatory for Elasticsearch 8.x. Initial attempts to configure pre-installation were unsuccessful, as these settings require Elasticsearch to be installed first. Thus, opted for post-installation configuration. This necessitated the ability to install and configure Elasticsearch without starting the service.

Note:
Security settings required by default in Elasticsearch 8.x ideally should be incorporated into the role itself. Plan to explore adding this support to the existing role when possible.

* Prevented waiting for Elasticsearch when `elasticsearch_service_state` is set
  to `stopped`.
* Avoided triggering 'restart elasticsearch' handler under the same condition.

Reasoning:
Encountered a need to configure security features mandatory for Elasticsearch
8.x. Initial attempts to configure pre-installation were unsuccessful, as these
settings require Elasticsearch to be installed first. Thus, opted for
post-installation configuration. This necessitated the ability to install and
configure Elasticsearch without starting the service.

Note:
Security settings required by default in Elasticsearch 8.x ideally should be
incorporated into the role itself. Plan to explore adding this support to the
existing role when possible.
@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
@lisuml
Copy link
Author

lisuml commented Nov 4, 2024

Ping? 😢

@github-actions github-actions bot removed the stale label Nov 10, 2024
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.

2 participants