fix(network_manager_networks): reduced pillar.get calls

Author: alxwr

openvpn/network_manager_networks/files/connection.jinja

@@ -1,5 +1,6 @@
 {%- from "openvpn/macros.jinja" import multipart_param with context -%}
-{%- set config = salt['pillar.get']('openvpn:network_manager:networks:'+network_name, {}) -%}
+{%- from "openvpn/map.jinja" import map with context %}
+
 {%- macro pairs(data) -%}
 {%-   for key, value in data|dictsort -%}
 {{ key|replace('_', '-') }}={{ value }}
@@ -24,8 +25,9 @@ uuid={{ salt['cmd.run']("python2 -c \"import uuid; print uuid.uuid5(uuid.NAMESPA
 {%-   if segment == 'vpn' %}
 {%-     if config[segment].get('service_type', 'org.freedesktop.NetworkManager.openvpn') == 'org.freedesktop.NetworkManager.openvpn' %}
 {%-       set instance_name = config.get('_vpn_instance', network_name) %}
-{%-       set instance_data = salt['pillar.get']('openvpn:client:'+instance_name, {}) %}
-{%-       set vpn_data = salt['pillar.get']('openvpn:network_manager:networks:{}:{}'.format(network_name, segment), instance_data, True) %}
+{%-       set instance_data = clients.get(instance_name, {}) %}
+{%-       set vpn_data = instance_data %}
+{%-       do vpn_data.update(config.get(segment, {})) %}
 connection-type=tls
 service-type={{ vpn_data.pop('service_type') }}
 

openvpn/network_manager_networks/init.sls

@@ -8,9 +8,12 @@ include:
   pkg.installed: []
 {%- endfor %}
 
-{%- set networks = salt['pillar.get']('openvpn:network_manager:networks', {}) %}
+{%- set openvpn_pillar = salt['pillar.get']('openvpn', {}) %}
+{%- set clients = openvpn_pillar.get('client', {}) %}
+{%- set networks = openvpn_pillar.get('network_manager', {}).get('networks', {}) %}
 
 {%- for name, data in networks.items() %}
+{%-   set config = networks.get(name, {}) %}
 "/etc/NetworkManager/system-connections/{{ name }}":
 {%-   if data.get('remove', False) %}
   file.absent: []
@@ -20,6 +23,8 @@ include:
     - source: salt://openvpn/network_manager_networks/files/connection.jinja
     - defaults:
       network_name: "{{ name }}"
+      config: {{ config | json }}
+      clients: {{ clients | json }}
     - mode: 600
     - onchanges_in:
       - cmd: network_manager_connection_reload