Sign Up
Log In
Log In
or
Sign Up
Places
All Projects
Status Monitor
Collapse sidebar
systemsmanagement:Uyuni:Master:CentOS6-Uyuni-Client-Tools
salt
removes-beacon-configuration-deprecation-warnin...
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File removes-beacon-configuration-deprecation-warning-48.patch of Package salt
From 155276e3a913c5d277f63b51db297ab8df256f44 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jo=C3=A3o=20Cavalheiro?= <cavalheiro@users.noreply.github.com> Date: Wed, 25 Oct 2017 11:05:03 +0100 Subject: [PATCH] Removes Beacon configuration Deprecation Warning (#48) The message "DeprecationWarning: Beacon configuration should be a list instead of a dictionary." was shown on the console if any beacon was defined not using lists. This was incompatible with beacon cfg loaders, namely inotify, that was not yet upgraded to support this change, causing an error whenever a list definition was used: File "/usr/lib/python2.7/site-packages/salt/beacons/inotify.py", line 247 TypeError: list indices must be integers, not dict., On same file we can read the comment: "Configuration for inotify beacon should be a dict of dicts" This commit removes the deprecation message as per @isbm suggestion. --- salt/beacons/__init__.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/salt/beacons/__init__.py b/salt/beacons/__init__.py index 21b4391227..101332a3d7 100644 --- a/salt/beacons/__init__.py +++ b/salt/beacons/__init__.py @@ -51,10 +51,6 @@ class Beacon(object): current_beacon_config = {} list(map(current_beacon_config.update, config[mod])) elif isinstance(config[mod], dict): - salt.utils.warn_until( - 'Nitrogen', - 'Beacon configuration should be a list instead of a dictionary.' - ) current_beacon_config = config[mod] if 'enabled' in current_beacon_config: -- 2.13.6
Locations
Projects
Search
Status Monitor
Help
OpenBuildService.org
Documentation
API Documentation
Code of Conduct
Contact
Support
@OBShq
Terms
openSUSE Build Service is sponsored by
The Open Build Service is an
openSUSE project
.
Sign Up
Log In
Places
Places
All Projects
Status Monitor