Repository: ansible/ansible-runner

Issue:
Different data passed to status_handler plugin vs kwargs
Hi,

As you can see from my PR, I'm using `ansible.runner.run(status_handler=my_handler)`. I noticed that [Runner.status_callback](https://github.com/ansible/ansible-runner/blob/ce3c90f97fc7c7e7bdc6a53494503b9a907875c0/ansible_runner/runner.py#L76) passes different "data" to `status_handler` defined via **kwargs** vs `status_handler` defined in **plugins**.
Basically, plugins get a dict with *status* and *runner_ident*, opposed to the "kwargs-status_handler" that only gets the status.
Is there any reasoning behind? For me this looks odd, I'd like to get the runner-ident, too. 

If you agree to change this, I could create another PR if you like.

Discussion:
softwarefactory-project-zuul[bot]: Build failed.

- [tox ](https://ansible.softwarefactory-project.io/logs/61/161/7b31a8f7b603d672b9c8fbb4371fb1c1aa200446/check/tox/3807b7a/) : FAILURE in 4m 49s


Patch:
diff --git a/ansible_runner/runner.py b/ansible_runner/runner.py
index 23b711b..2e8a6cf 100644
--- a/ansible_runner/runner.py
+++ b/ansible_runner/runner.py
@@ -71,12 +71,11 @@ class Runner(object):
 
     def status_callback(self, status):
         self.status = status
+        status_data = dict(status=status, runner_ident=str(self.config.ident))
         for plugin in ansible_runner.plugins:
-            ansible_runner.plugins[plugin].status_handler(self.config,
-                                                          dict(status=status,
-                                                               runner_ident=str(self.config.ident)))
+            ansible_runner.plugins[plugin].status_handler(self.config, status_data)
         if self.status_handler is not None:
-            self.status_handler(status)
+            self.status_handler(status_data)
 
     def run(self):
         '''


Test patch:
diff --git a/test/unit/test_runner.py b/test/unit/test_runner.py
index c527ab6..1ac3c67 100644
--- a/test/unit/test_runner.py
+++ b/test/unit/test_runner.py
@@ -109,5 +109,5 @@ def test_status_callback_interface(rc):
     runner.status_handler = mock.Mock()
     runner.status_callback("running")
     assert runner.status_handler.call_count == 1
-    runner.status_handler.assert_called_with('running')
+    runner.status_handler.assert_called_with(dict(status='running', runner_ident=str(rc.ident)))
     assert runner.status == 'running'