Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1515268 - monitoring - doesn't reload configuration
monitoring - doesn't reload configuration
Status: CLOSED WONTFIX
Product: Red Hat Gluster Storage
Classification: Red Hat
Component: web-admin-tendrl-monitoring-integration (Show other bugs)
3.3
Unspecified Unspecified
unspecified Severity unspecified
: ---
: ---
Assigned To: Rohan Kanade
Lubos Trilety
: ZStream
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2017-11-20 08:16 EST by Martin Kudlej
Modified: 2018-08-14 07:15 EDT (History)
8 users (show)

See Also:
Fixed In Version:
Doc Type: Known Issue
Doc Text:
When the user runs 'systemctl reload' the monitoring integration configuration does not reload. To work around this issue, run 'systemctl restart'. Monitoring integration restarts as part of this operation and the configuration is done.
Story Points: ---
Clone Of:
Environment:
Last Closed: 2018-04-26 02:07:00 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)


External Trackers
Tracker ID Priority Status Summary Last Updated
Github Tendrl/gluster-integration/issues/491 None None None 2017-11-21 07:59 EST

  None (edit)
Description Martin Kudlej 2017-11-20 08:16:01 EST
Description of problem:
"systemctl reload" command doesn't reload monitoring configuration. It returns error.

Version-Release number of selected component (if applicable):
tendrl-monitoring-integration-1.5.4-4.el7rhgs.noarch

How reproducible:
100%

Steps to Reproduce:
1. get current date: $ date
2. get conf files for package: $ rpm -qc tendrl-monitoring-integration
3. reload configuration: $ systemctl reload tendrl-monitoring-integration
step 3 returns error: Job for tendrl-monitoring-integration.service invalid.
it it pass then we can continue to step 4.
4. check if conf files from 2. are touched: $ find /etc/tendrl/ -type f | xargs stat | egrep '^Access: 2|File:'

Actual results:
"systemctl reload" doesn't reload service configuration and returns error.

Expected results:
"systemctl reload" reloads service configuration.

Additional info:
Cite from systemd documentation:
reload PATTERN...
           Asks all units listed on the command line to reload their
           configuration. Note that this will reload the service-specific
           configuration, not the unit configuration file of systemd. If you
           want systemd to reload the configuration file of a unit, use the
           daemon-reload command. In other words: for the example case of
           Apache, this will reload Apache's httpd.conf in the web server,
           not the apache.service systemd unit file.

           This command should not be confused with the daemon-reload
           command.
Comment 1 Rohan Kanade 2017-11-21 07:59:51 EST
Fixed: https://github.com/Tendrl/gluster-integration/issues/491
Comment 4 Lubos Trilety 2017-11-28 06:48:02 EST
Tested with:
tendrl-monitoring-integration-1.5.4-8.el7rhgs.noarch

There's no error when reload is made.
However changes in configuration files are not reloaded. Access time on those configuration files is not changed and just to be sure I changed some settings (e.g. etcd port) without any change in behaviour of the service.
Comment 6 Lubos Trilety 2017-12-05 10:03:43 EST
Tested with:
tendrl-monitoring-integration-1.5.4-11.el7rhgs.noarch

When I tried change configuration first time there was an error in log, however the service is still running, which should not. It should behave similarly like when it is run with the bad configuration (goes to failed state and stop).
Moreover when I tried that again there was no error in log just line with this sentence 'Reloaded Monitoring Integration'.
Comment 12 Nishanth Thomas 2017-12-06 23:21:21 EST
Moving the bug out of 3.3.z

Note You need to log in before you can comment on or make changes to this bug.