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

[DPE-4859] Make cloud-init optional #398

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

phvalguima
Copy link
Contributor

Originally, cloud-init was mandatory to be set in order for OpenSearch to work. This PR updates the README and integration tests to remove this dependency, following recent charm changes.

Closes #359

Originally, cloud-init was mandatory to be set in order for OpenSearch to work. This PR updates the README and integration tests to remove this dependency, following recent charm changes.

Closes #359
@phvalguima phvalguima marked this pull request as ready for review August 16, 2024 09:39
reneradoi
reneradoi previously approved these changes Aug 16, 2024
delgod
delgod previously approved these changes Aug 16, 2024
Mehdi-Bendriss
Mehdi-Bendriss previously approved these changes Sep 4, 2024
@phvalguima phvalguima requested a review from izmalk September 4, 2024 12:30
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Co-authored-by: Vladimir Izmalkov <[email protected]>
@phvalguima phvalguima dismissed stale reviews from Mehdi-Bendriss, delgod, and reneradoi via 25f963f September 4, 2024 14:14
phvalguima and others added 9 commits September 4, 2024 16:14
Co-authored-by: Vladimir Izmalkov <[email protected]>
Co-authored-by: Vladimir Izmalkov <[email protected]>
Co-authored-by: Vladimir Izmalkov <[email protected]>
Co-authored-by: Vladimir Izmalkov <[email protected]>
Co-authored-by: Vladimir Izmalkov <[email protected]>
Co-authored-by: Vladimir Izmalkov <[email protected]>
Co-authored-by: Vladimir Izmalkov <[email protected]>
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.

Expected sysctl parameters should be enforced by the charm
6 participants