x/collectd-openstack-plugins
Code Issues Proposed changes

Change the extension name of backup .conf files

Browse Source 
- Renamed the back-up's to .conf.b
  - Provided a reno for this change

Change-Id: I78d364a40515a63ecdb99fe5bd4dbb0010c28512
Closes-bug: #1684579
This commit is contained in:
Helena McGough 2017-04-21 13:27:54 +00:00 committed by Helena
parent 88a83e59ef
commit b02acc8a4f
2 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
devstack/libs/collectd
View File

@ -180,9 +180,9 @@ function adapt_collectd_conf {
sudo sed -i 's|File.*$|File "'$COLLECTD_LOG_FILE'"|g' $COLLECTD_CONF_DIR/01-logfile.conf
# Create a backup for all of the .conf files
sudo mkdir -p "$COLLECTD_CONF_DIR/backup"
sudo mkdir -p $COLLECTD_CONF_DIR/backup
sudo cp $COLLECTD_CONF_DIR/*.conf $COLLECTD_CONF_DIR/backup/
sudo rename 's|\.conf$|.conf.bak|' $COLLECTD_CONF_DIR/backup/*.conf
}
function config_custom_units {

7
releasenotes/notes/bug-1684579-eea80d719dd3cdc4.yaml Normal file
View File

@ -0,0 +1,7 @@
---
fixes:
- |
Fixes 'bug 1684579 https://bugs.launchpad.net/collectd-ceilometer-plugin/+bug/1684579'.
Changed the extensions for the back-up .conf files. This prevents back-up
.conf file being read if the original .conf is deleted. Thus preventing
those plugins from running when they are not configured properly.