diff --git a/ironic/objects/fields.py b/ironic/objects/fields.py index 528e998b26..1b77789450 100644 --- a/ironic/objects/fields.py +++ b/ironic/objects/fields.py @@ -14,8 +14,6 @@ # under the License. import ast -import hashlib -import inspect from oslo_versionedobjects import fields as object_fields @@ -57,10 +55,7 @@ class StringFieldThatAcceptsCallable(object_fields.StringField): default = self._default if (self._default != object_fields.UnspecifiedDefault and callable(self._default)): - default = "%s-%s" % ( - self._default.__name__, - hashlib.md5(inspect.getsource( - self._default).encode()).hexdigest()) + default = '' % default.__name__ return '%s(default=%s,nullable=%s)' % (self._type.__class__.__name__, default, self._nullable) diff --git a/ironic/tests/unit/objects/test_fields.py b/ironic/tests/unit/objects/test_fields.py index 02ce1222d9..35cc050e60 100644 --- a/ironic/tests/unit/objects/test_fields.py +++ b/ironic/tests/unit/objects/test_fields.py @@ -13,9 +13,6 @@ # License for the specific language governing permissions and limitations # under the License. -import hashlib -import inspect - from ironic.common import exception from ironic.objects import fields from ironic.tests import base as test_base @@ -73,8 +70,6 @@ class TestStringFieldThatAcceptsCallable(test_base.TestCase): def test_default_function(): return "default value" - self.test_default_function_hash = hashlib.md5( - inspect.getsource(test_default_function).encode()).hexdigest() self.field = fields.StringFieldThatAcceptsCallable( default=test_default_function) @@ -102,8 +97,8 @@ class TestStringFieldThatAcceptsCallable(test_base.TestCase): self.field.coerce('obj', 'attr', None)) def test__repr__includes_default_function_name_and_source_hash(self): - expected = ('StringAcceptsCallable(default=test_default_function-%s,' - 'nullable=False)' % self.test_default_function_hash) + expected = ('StringAcceptsCallable(default=,nullable=False)') self.assertEqual(expected, repr(self.field)) diff --git a/tools/bandit.yml b/tools/bandit.yml index d99694d395..028d1a214d 100644 --- a/tools/bandit.yml +++ b/tools/bandit.yml @@ -89,7 +89,6 @@ tests: # (optional) list skipped test IDs here, eg '[B101, B406]': skips: - B104 - - B303 - B604 ### (optional) plugin settings - some test plugins require configuration data