File removes-beacon-configuration-deprecation-warnin... of Package salt.7212

Overview Repositories Revisions Requests Users Attributes Meta

File removes-beacon-configuration-deprecation-warning-48.patch of Package salt.7212

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

Places

File removes-beacon-configuration-deprecation-warning-48.patch of Package salt.7212

Places