Comment 1 for bug 1924956

Hi,

Looking at openstack infra_driver quickly, there is some common style to have an arg `plugin` and caller gives `self` to it even if it's not used actually. So, why don't you follow the convention at once instead of using default value `None` exceptionally?