Merge pull request #361 from svtkachenko/update_sv

Removed overriden read method for Smart Variable

nailgun/entities.py

@@ -4735,18 +4735,6 @@ def __init__(self, server_config=None, **kwargs):
         }
         super(SmartVariable, self).__init__(server_config, **kwargs)
 
-    def read(self, entity=None, attrs=None, ignore=None):
-        """Fetch as many attributes as possible for this entity.
-
-        Do not read the ``variable_type`` attribute. For more information, see
-        `Bugzilla #1375881
-        <https://bugzilla.redhat.com/show_bug.cgi?id=1375881>`_.
-        """
-        if ignore is None:
-            ignore = set()
-        ignore.add('variable_type')
-        return super(SmartVariable, self).read(entity, attrs, ignore)
-
     def create_payload(self):
         """Wrap submitted data within an extra dict."""
         return {u'smart_variable': super(SmartVariable, self).create_payload()}

tests/test_entities.py

@@ -994,7 +994,6 @@ def test_ignore_arg_v1(self):
         for entity, ignored_attrs in (
                 (entities.Errata,
                  {'content_view_version', 'environment', 'repository'}),
-                (entities.SmartVariable, {'variable_type'}),
                 (entities.Subnet, {'discovery'}),
                 (entities.User, {'password'}),
         ):