Add instrumentation to subsystem reloads
authorKinsey Moore <kmoore@digium.com>
Fri, 17 Aug 2012 16:01:32 +0000 (16:01 +0000)
committerKinsey Moore <kmoore@digium.com>
Fri, 17 Aug 2012 16:01:32 +0000 (16:01 +0000)
When Asterisk is built with TEST_FRAMEWORK defined, Asterisk will now
generate TestEvent AMI events on subsystem reloads such as cdr, dnsmgr,
extconfig, etc.

(issue PQ-1126)
........

Merged revisions 371436 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 371437 from http://svn.asterisk.org/svn/asterisk/branches/10
........

Merged revisions 371438 from http://svn.asterisk.org/svn/asterisk/branches/11

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@371439 65c4cc65-6c06-0410-ace0-fbb531ad65f3

main/loader.c

index 1da6ef2..36ac3da 100644 (file)
@@ -718,7 +718,9 @@ int ast_module_reload(const char *name)
        /* Call "predefined" reload here first */
        for (i = 0; reload_classes[i].name; i++) {
                if (!name || !strcasecmp(name, reload_classes[i].name)) {
-                       reload_classes[i].reload_fn();  /* XXX should check error ? */
+                       if (!reload_classes[i].reload_fn()) {
+                               ast_test_suite_event_notify("MODULE_RELOAD", "Message: %s", name);
+                       }
                        res = 2;        /* found and reloaded */
                }
        }