<div dir="ltr">Several properties are added to entity in `DriverBase.make_pickleable`[1] but the existence check is only applied in `_add_entity_type`.<div><br></div><div>I think it may not be necessary after we prepended vitrage namespace[2]. I could not think of a scenario to let user shadow the built-in `entity_type`.</div><div><br></div><div>Is there any particular reason for keeping it? Or should we remove it?</div><div><div><pre style="background-color:rgb(43,43,43);color:rgb(169,183,198);font-family:menlo;font-size:9pt"><span style="color:rgb(136,136,198)">@</span><span style="background-color:rgb(52,65,52)">staticmethod</span><br><span style="color:rgb(204,120,50);font-weight:bold">def </span><span style="font-weight:bold">_add_entity_type</span>(entity<span style="color:rgb(204,120,50)">, </span>entity_type):<br>    <span style="color:rgb(204,120,50);font-weight:bold">if </span>DSProps.ENTITY_TYPE <span style="color:rgb(204,120,50);font-weight:bold">not in </span>entity:<br>        entity[DSProps.ENTITY_TYPE] = entity_type</pre><div><br></div><div>[1]: <a href="https://github.com/openstack/vitrage/blob/master/vitrage/datasources/driver_base.py#L54">https://github.com/openstack/vitrage/blob/master/vitrage/datasources/driver_base.py#L54</a> <br><div>[2]: <a href="https://review.openstack.org/#/c/399417/">https://review.openstack.org/#/c/399417/</a> </div><div><br></div></div></div></div></div>