Merge lp:~junaidali/charms/trusty/plumgrid-gateway/opsvm into lp:~bbaqar/charms/trusty/plumgrid-gateway/opsvm

[Junaid Ali]

[lp:~junaidali/charms/trusty/plumgrid-gateway/opsvm]

27. By Junaid Ali on 2016-03-28

fixing unit_test

[Bilal Baqar (bbaqar)]

review: Needs Fixing

[lp:~junaidali/charms/trusty/plumgrid-gateway/opsvm]

28. By Junaid Ali on 2016-03-28

Updated function description

29. By Junaid Ali on 2016-03-28

Updated function description

Unmerged revisions

29. By Junaid Ali on 2016-03-28

Updated function description

28. By Junaid Ali on 2016-03-28

Updated function description

27. By Junaid Ali on 2016-03-28

fixing unit_test

26. By Junaid Ali on 2016-03-28

Improved plumgrid service restarts

=== modified file 'hooks/pg_gw_hooks.py'
--- hooks/pg_gw_hooks.py	2016-03-27 15:24:34 +0000
+++ hooks/pg_gw_hooks.py	2016-03-28 18:36:13 +0000
@@ -14,8 +14,6 @@
     config,
 )
 
-from charmhelpers.core.host import service_running
-
 from charmhelpers.fetch import (
     apt_install,
     configure_sources,
@@ -24,7 +22,6 @@
 from pg_gw_utils import (
     register_configs,
     ensure_files,
-    restart_pg,
     restart_map,
     stop_pg,
     determine_packages,
@@ -35,6 +32,7 @@
     fabric_interface_changed,
     load_iptables,
     restart_on_change,
+    restart_on_stop,
     director_cluster_ready
 )
 
@@ -71,6 +69,8 @@
 
 
 @hooks.hook('config-changed')
+@restart_on_stop()
+@restart_on_change(restart_map())
 def config_changed():
     '''
     This hook is run when a config parameter is changed.
@@ -83,10 +83,6 @@
     if charm_config.changed('fabric-interfaces'):
         if not fabric_interface_changed():
             log("Fabric interface already set")
-        else:
-            stop_pg()
-    if charm_config.changed('external-interfaces'):
-        stop_pg()
     if (charm_config.changed('install_sources') or
         charm_config.changed('plumgrid-build') or
         charm_config.changed('install_keys') or
@@ -100,8 +96,6 @@
             load_iovisor()
     ensure_mtu()
     CONFIGS.write_all()
-    if not service_running('plumgrid'):
-        restart_pg()
 
 
 @hooks.hook('upgrade-charm')

=== modified file 'hooks/pg_gw_utils.py'
--- hooks/pg_gw_utils.py	2016-03-27 15:24:34 +0000
+++ hooks/pg_gw_utils.py	2016-03-28 18:36:13 +0000
@@ -393,6 +393,19 @@
     return True if dirs_count == 1 or dirs_count == 3 else False
 
 
+def restart_on_stop():
+    """
+    Starts plumgrid service if it is stopped
+    """
+    def wrap(f):
+        def wrapped_f(*args, **kwargs):
+            f(*args, **kwargs)
+            if not service_running('plumgrid'):
+                restart_pg()
+        return wrapped_f
+    return wrap
+
+
 def restart_on_change(restart_map):
     """
     Restart services based on configuration files changing
@@ -403,6 +416,8 @@
             f(*args, **kwargs)
             for path in restart_map:
                 if path_hash(path) != checksums[path]:
+                    if path == PG_IFCS_CONF:
+                        ensure_files()
                     restart_pg()
                     break
         return wrapped_f

=== modified file 'unit_tests/test_pg_gw_hooks.py'
--- unit_tests/test_pg_gw_hooks.py	2016-03-27 15:24:34 +0000
+++ unit_tests/test_pg_gw_hooks.py	2016-03-28 18:36:13 +0000
@@ -19,13 +19,11 @@
 TO_PATCH = [
     'remove_iovisor',
     'apt_install',
-    'apt_purge',
     'CONFIGS',
     'log',
     'configure_sources',
     'ensure_files',
     'stop_pg',
-    'restart_pg',
     'load_iovisor',
     'ensure_mtu',
     'add_lcm_key',