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

Add custom playbook to deploy IPMI exporter #90

Open
wants to merge 7 commits into
base: rocky-preprod
Choose a base branch
from

Conversation

jovial
Copy link
Collaborator

@jovial jovial commented Oct 16, 2019

The idea is you run: kayobe playbook run $PWD/ipmi-exporter.yml --vault-password-file ~/vaultpassword . This will generate prometheus configuration in the kayobe config directory under:
$KAYOBE_CONFIG_PATH/kolla/config/prometheus. When the config changes, a handler is invoked to notify you to reconfigure prometheus. You should do so with: kayobe overcloud service reconfigure -kt prometheus --kolla-skip-tags common.

This requires RSE-Cambridge/kolla-ansible#29 to merge the prometheus config.

These are generated with:

kayobe playbook run $PWD/ipmi-exporter.yml --vault-password-file ~/vaultpassword
It did not parse:

e=\"/etc/prometheus/prometheus.yml\"): parsing YAML file /etc/prometheus/prometheus.yml: yaml: unmarshal errors:\n  line 22: cannot unmarshal !!str `default` into []string"
+ sudo -E kolla_set_configs
This now seems to be in a functional state.
@@ -0,0 +1,25 @@
modules:
default:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Micro nit: maybe 4 spaces rather than 8?

@@ -0,0 +1,7 @@
---
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is a question really... not sure about why this is here, I guess it was generated by the others scripts?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yep, these files were generated. We could probably move the templating here instead. Do you think that would be cleaner?

@@ -0,0 +1,31 @@
---
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same as above, I guess its cool it only needs to drop into this directory, but I guess it was copied here using the above playbook.

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