i have a couple of concerns here  the most minor one is that this now forces us to use this decorator only on instance or class methods  is there anywhere where this is not the case  the second concern is that  if i recall correctly  the get context from function and args function was created because the decorator was used on some methods where context had to be searched for  which argues that  for those methods  it is not the first non self argument to the method  this means that  for those methods  this could result in a notify still containing the context  but missing the first argument to the method  which may be rather useful  also  it is worth pointing out that there are no other consumers of get context from function and args it is used in nova openstack common notifier api py  but the version used is of course from nova openstack common context py given that  this patch should probably also remove the function  assuming my two concerns above are not a factor 