-
Notifications
You must be signed in to change notification settings - Fork 82
Include gluster-prometheus-exporter rpm in glusterd2 container #1308
Include gluster-prometheus-exporter rpm in glusterd2 container #1308
Conversation
retest this please |
Nightly rpm is now setup http://artifacts.ci.centos.org/gluster/gluster-prometheus-nightly/ (centosci PR: gluster/centosci#36) Removed |
- name: Configure gluster-exporter to use glusterd2 | ||
replace: | ||
path: /etc/gluster-exporter/gluster-exporter.toml | ||
regexp: 'gluster-mgmt = "glusterd"' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May want to make these matches a bit more robust ala: '^gluster-mgmt =.*'
(or something alone those lines)... Just to ensure the replacement happens even if the default options changes subtly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed as suggested.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than the use of the command module to enable the service, the remaining changes look good.
value: GD2_ENDPOINTS | ||
|
||
- name: Enable gluster-exporter.service | ||
command: systemctl enable gluster-exporter.service |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should be using the service module instead of the command module. Any reason not to?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
referred other instance in the same file(Line 109)
- Depends on rpm spec file PR gluster/gluster-prometheus#26 - Nightly rpm building is setup gluster/centosci#36 - Depends of environment variable related PR gluster/gluster-prometheus#66 Signed-off-by: Aravinda VK <avishwan@redhat.com>
Updated to use "service" module instead of "command". @kshlm please review |
Added rpm spec to build the rpm gluster-prometheus#26
Support for
GD2_ENDPOINTS
environment variable gluster-prometheus#66Signed-off-by: Aravinda VK avishwan@redhat.com