Fix ConfigManagement hierarchyController definition

[unlimitedbits]

Hi,

currently the spec.hierarchyController definition is not working for the ConfigManagement template in the acm and config-sync modules.

The PR fixes the following things:

• hierarchy_controller is renamed to hierarchyController in the resource definition
• quotation marks added by the yamlencode function are removed
• correct indentation with more than one entry in the hiearchy_controller variable

Example

With the hierarchy_controller variable defined as:

hierarchy_controller = {
    enabled                         = true
    enablePodTreeLabels             = true
    enableHierarchicalResourceQuota = true
  }

the output with v14.0.1 is as follows:

apiVersion: configmanagement.gke.io/v1
kind: ConfigManagement
metadata:
  name: config-management
spec:
  # clusterName is required and must be unique among all managed clusters
  clusterName: test
  policyController:
    # policyController definitions
  git:
    # git definitions

  hierarchy_controller:
    "enableHierarchicalResourceQuota": true
"enablePodTreeLabels": true
"enabled": true

output with this fix:

apiVersion: configmanagement.gke.io/v1
kind: ConfigManagement
metadata:
  name: config-management
spec:
  # clusterName is required and must be unique among all managed clusters
  clusterName: test
  policyController:
    # policyController definitions
  git:
    # git definitions

  hierarchyController:
    enableHierarchicalResourceQuota: true
    enablePodTreeLabels: true
    enabled: true