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

CASMPET-6860 deploy node-exporter via cephadm shell in ansible #217

Merged
merged 2 commits into from
Nov 22, 2023

Conversation

leliasen-hpe
Copy link
Contributor

@leliasen-hpe leliasen-hpe commented Nov 20, 2023

Summary and Scope

Deploy node-exporter via cephadm shell in ansible that deploys smartmon on storage nodes.
We believe this change is necessary because when applying the CSM 1.4.2 patch on Alvarez, the smartmon enablement on storage nodes script failed. This was likely caused by the ceph-common version on the system. To be able to apply the node-exporter.yml file which enables smartmon, node-exported was deployed via cephadm shell. Applying node exporter via the cephadm shell should work every time and doesn't depend on the ceph-common version installed on the node.

This has been tested on Beau.

Issues and Related PRs

List and characterize relationship to Jira/Github issues and other pull requests. Be sure to list dependencies.

Testing

List the environments in which these changes were tested.

Tested on:

  • Tested on Beau. I applied this CFS configuration to nodes with and without the admin.keyring to see that both cases worked.
    Testing was not done on a metal system. This will be tested on every storage node upgrade.

Test description:

Tested configuring storage nodes with these CFS changes. There were no errors.

Pull Request Checklist

  • Version number(s) incremented, if applicable
  • Copyrights updated
  • License file intact
  • Target branch correct
  • CHANGELOG.md updated
  • Testing is appropriate and complete, if applicable
  • HPC Product Announcement prepared, if applicable

@leliasen-hpe leliasen-hpe requested a review from a team as a code owner November 20, 2023 15:29
@leliasen-hpe leliasen-hpe marked this pull request as draft November 20, 2023 18:25
@leliasen-hpe leliasen-hpe marked this pull request as ready for review November 20, 2023 18:41
@leliasen-hpe leliasen-hpe merged commit 6a103c9 into master Nov 22, 2023
9 checks passed
@leliasen-hpe leliasen-hpe deleted the CASMPET-6860 branch November 22, 2023 16:31
mharding-hpe added a commit that referenced this pull request Nov 30, 2023
…700670684

[chore] master -> develop from PR #217 (CASMPET-6860)
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.

3 participants