From 6846341a60593f62d14795caa7bdb1c1cde7ce7b Mon Sep 17 00:00:00 2001
From: Saravanan KR <skramaja@redhat.com>
Date: Wed, 14 Feb 2018 11:26:32 +0530
Subject: [PATCH] Add role_specific tag to missing role-specific parameters

A tag 'role_specific' should be added to the parameters of
a service which are accepted as role-specific. Adding the tag
for some of the missing parameters.

Change-Id: Ic205b4c6cf219f37025e6ddb61455e1d48ec480d
---
 puppet/services/neutron-ovs-agent.yaml | 4 ++++
 puppet/services/opendaylight-ovs.yaml  | 2 ++
 puppet/services/ovn-controller.yaml    | 7 +++++++
 puppet/services/time/ptp.yaml          | 6 ++++++
 4 files changed, 19 insertions(+)

diff --git a/puppet/services/neutron-ovs-agent.yaml b/puppet/services/neutron-ovs-agent.yaml
index ec1ec16ae2..6f004fc987 100644
--- a/puppet/services/neutron-ovs-agent.yaml
+++ b/puppet/services/neutron-ovs-agent.yaml
@@ -44,6 +44,8 @@ parameters:
       scripts or be sure to keep 'datacentre' as a mapping network name.
     type: comma_delimited_list
     default: "datacentre:br-ex"
+    tags:
+      - role_specific
   NeutronTunnelTypes:
     default: 'vxlan'
     description: The tunnel types for the Neutron tenant network.
@@ -83,6 +85,8 @@ parameters:
     description: |
       Enable OVS Hardware Offload. This feature supported from OVS 2.8.0
     type: boolean
+    tags:
+      - role_specific
 
 conditions:
   no_firewall_driver: {equals : [{get_param: NeutronOVSFirewallDriver}, '']}
diff --git a/puppet/services/opendaylight-ovs.yaml b/puppet/services/opendaylight-ovs.yaml
index 3390645eab..c5da73c707 100644
--- a/puppet/services/opendaylight-ovs.yaml
+++ b/puppet/services/opendaylight-ovs.yaml
@@ -96,6 +96,8 @@ parameters:
     description: |
       Enable OVS Hardware Offload. This feature supported from OVS 2.8.0
     type: boolean
+    tags:
+      - role_specific
   EnableInternalTLS:
     type: boolean
     default: false
diff --git a/puppet/services/ovn-controller.yaml b/puppet/services/ovn-controller.yaml
index 25471de084..9587f61c39 100644
--- a/puppet/services/ovn-controller.yaml
+++ b/puppet/services/ovn-controller.yaml
@@ -48,6 +48,8 @@ parameters:
       scripts or be sure to keep 'datacentre' as a mapping network name.
     type: comma_delimited_list
     default: "datacentre:br-ex"
+    tags:
+      - role_specific
   OVNIntegrationBridge:
     description: >
       Name of the OVS bridge to use as integration bridge by OVN Controller.
@@ -61,11 +63,16 @@ parameters:
     description: The CMS options to configure in ovs db
     type: string
     default: ""
+    tags:
+      - role_specific
   OvsHwOffload:
     default: false
     description: |
       Enable OVS Hardware Offload. This feature supported from OVS 2.8.0
     type: boolean
+    tags:
+      - role_specific
+
 
 resources:
   # Merging role-specific parameters (RoleParameters) with the default parameters.
diff --git a/puppet/services/time/ptp.yaml b/puppet/services/time/ptp.yaml
index e01e2208da..da32746e45 100644
--- a/puppet/services/time/ptp.yaml
+++ b/puppet/services/time/ptp.yaml
@@ -39,14 +39,20 @@ parameters:
     default: nic1
     description: PTP interface name.
     type: string
+    tags:
+      - role_specific
   PtpSlaveMode:
     default: 1
     description: Configure PTP clock in slave mode.
     type: number
+    tags:
+      - role_specific
   PtpMessageTransport:
     default: UDPv4
     description: Configure PTP message transport protocol.
     type: string
+    tags:
+      - role_specific
 
 resources:
   RoleParametersValue: