The way the on-demand policy loading is implemented is bad. The code needs to be updated if we add a new attribute to existing rule or policy class, or add a new rule -- and there is no code that checks consistency or if the loaded attributes are valid.

I've changed to code to be more generic (in the second commit).

could you please rename this var, so there's no trailing underscore

+1 after fixing minor comment above

could you please rename this var, so there's no trailing underscore

type is function in Python. Adding underscore suffix is a standard way to avoid using conflicting names (type_ is used on couple of other places in code already).

type is function in Python. Adding underscore suffix is a standard way to avoid using conflicting names (type_ is used on couple of other places in code already).

I know that. I mean that you can rename it to something more meaningful and more descriptive like xyz_type

type is function in Python. Adding underscore suffix is a standard way to avoid using conflicting names (type_ is used on couple of other places in code already).

I know that. I mean that you can rename it to something more meaningful and more descriptive like xyz_type

Renamed to item_type.