Borislav: Fixed Makefile and renamed charts.
Also cosmetic fixes.
Includes corrections for the service name of MR used by dbc to
be "message-router", which matches the message-router chart
Patchset 2: standardize abbreviation for bus-controller to be "dbc"
Patchset 3: correct references in README to match what is deployed
Patchset 4: blew previous commit.  Also added dmaap/Makefile lint step
Change-Id: I36c07c7c1200f8e23ae12bd7357a5626346dcccd
Signed-off-by: Dominic Lunanuova <dgl@research.att.com>
Issue-ID: DMAAP-386
Signed-off-by: Dominic Lunanuova <dgl@research.att.com>
 # limitations under the License.
 
 apiVersion: v1
-description: a Helm chart to deploy ONAP DMaaP componetns in Kubernetes
+description: ONAP DMaaP components
 name: dmaap
 version: 2.0.0
 
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-make-dmaap:
-       cd charts && helm dep up dmaap-bus-controller
+make-dmaap: make-bus-controller make-message-router
+
+make-bus-controller:
+       cd charts && helm dep up dmaap-bus-controller && helm lint dmaap-bus-controller
+
+make-message-router:
+       cd charts && helm dep up message-router && helm lint message-router
+
+clean:
+       @find . -type f -name '*.tgz' -delete
 
+# Copyright © 2018  AT&T Intellectual Property.  All rights reserved.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#       http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
 # Helm Chart for ONAP DMaaP Applications
 
 ONAP includes the following Kubernetes services:
 
 1) message-router - a message bus for applications
-2) dmaap-prov - an API to provision DMaaP resources
+2) dbc-api - an API to provision DMaaP resources
 
 # Service Dependencies
 
 message-router depends on AAF
-dmaap-prov depends on AAF and Postgresql.  NOTE: until Postgresql is available as a common service in ONAP, this chart deploys a private instance based on kubernetes/stable chart
+dbc-api depends on AAF and Postgresql.
\ No newline at end of file
 
+# Copyright © 2018  AT&T Intellectual Property.  All rights reserved.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#       http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
 1. Get the application URL by running these commands:
 {{- if .Values.ingress.enabled }}
 {{- range .Values.ingress.hosts }}
 
 
 
 # application configuration
-dmaapMessageRouterService: dmaap
-aafURL: https://onap-aaf:8905/proxy/
+dmaapMessageRouterService: message-router
+aafURL: https://aaf-authz/
 topicMgrUser: m23456@dmaapbc.onap.org
 topicMgrPwd: onapdemo
 adminUser: m12345@dmaapbc.onap.org
 
 
 service:
-  type: ClusterIP
+  type: NodePort
   name: dmaap-bc
+  portName: dmaap-bc
+  portName2: dmaap-bc
   externalPort: 8080
   internalPort: 8080
   externalPort2: 8443
   internalPort2: 8443
+  nodePort: 41
+  nodePort2: 42
 
 
 # application configuration override for postgres
 postgres:
-  nameOverride: dbcl-db
+  nameOverride: dbc-pg
   service:
-    name: dbcl-postgres
-    name2: dbcl-pg-primary
-    name3: dbcl-pg-replica
+    name: dbc-postgres
+    name2: dbc-pg-primary
+    name3: dbc-pg-replica
   container:
     name:
-      primary: dbcl-pg-primary
-      replica: dbcl-pg-replica
+      primary: dbc-pg-primary
+      replica: dbc-pg-replica
   config:
     pgUserName: dmaap_admin
     pgDatabase: dmaap
     pgPrimaryPassword: onapdemodb
     pgUserPassword: onapdemodb
     pgRootPassword: onapdemodb
-
+  persistence:
+    mountSubPath: dbc/data
+    mountInitPath: dbc
 
 ingress:
   enabled: false
 
 # limitations under the License.
 
 apiVersion: v1
-appVersion: "1.0"
 description: ONAP Message Router
 name: message-router
 version: 2.0.0
 
 # limitations under the License.
 
 apiVersion: v1
-description: ONAP Dmaap Zookeeper Service
-name: zookeeper
+description: ONAP Message Router Kafka Service
+name: message-router-kafka
 version: 2.0.0
 
 
+# Copyright © 2018  AT&T Intellectual Property.  All rights reserved.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#       http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
 1. Get the application URL by running these commands:
 {{- if .Values.ingress.enabled }}
 {{- range .Values.ingress.hosts }}
 
         - /root/ready.py
         args:
         - --container-name
-        - {{ .Values.zookeeper.nameOverride }}
+        - {{ .Values.zookeeper.name }}
         env:
         - name: NAMESPACE
           valueFrom:
           periodSeconds: {{ .Values.readiness.periodSeconds }}
         env:
         - name: KAFKA_ZOOKEEPER_CONNECT
-          value: "{{.Values.zookeeper.nameOverride}}:{{.Values.config.zookeeperPort}}"
+          value: "{{.Values.zookeeper.name}}:{{.Values.zookeeper.port}}"
         - name: KAFKA_ADVERTISED_HOST_NAME
           value: "{{ include "common.servicename" .  }}"
         - name: KAFKA_BROKER_ID
 
   ports:
     - port: {{ .Values.service.externalPort }}
       targetPort: {{ .Values.service.internalPort }}
-      name: {{ .Values.service.name }}
+      name: {{ .Values.service.portName }}
   selector:
     app: {{ include "common.name" . }}
     release: {{ .Release.Name }}
 
 #################################################################
 # application image
 repository: docker.io
-image: wurstmeister/kafka:latest
+image: wurstmeister/kafka:1.1.0
 pullPolicy: Always
 
 zookeeper:
-  nameOverride: zookeeper
+  name: message-router-zookeeper
+  port: 2181
 
 # flag to enable debugging - application support required
 debugEnabled: false
 
 # application configuration
-config:
-  zookeeperPort: 2181
+config: {}
 
 # default number of instances
 replicaCount: 1
 
 service:
   type: NodePort
-  name: global-kafka
+  name: message-router-kafka
+  portName: message-router-kafka
   internalPort: 9092
   externalPort: 9092
 
 
 # limitations under the License.
 
 apiVersion: v1
-description: ONAP Global Kafka Service
-name: kafka
+description: ONAP Dmaap Message Router Zookeeper Service
+name: message-router-zookeeper
 version: 2.0.0
 
 
+# Copyright © 2018  AT&T Intellectual Property.  All rights reserved.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#       http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
 1. Get the application URL by running these commands:
 {{- if .Values.ingress.enabled }}
 {{- range .Values.ingress.hosts }}
 
 
 service:
   type: NodePort
-  name: zookeeper
+  name: message-router-zookeeper
+  portName: message-router-zookeeper
   internalPort: 2181
 
 ingress:
 
 ##
 ##     Both Cambria and Kafka make use of Zookeeper.
 ##
-config.zk.servers={{.Values.zookeeper.nameOverride}}:{{.Values.config.zookeeperPort}}
-#config.zk.servers=172.17.0.1:2181
-#dmaap.onap-message-router:2181
-#10.208.128.229:2181
-#config.zk.root=/fe3c/cambria/config
+config.zk.servers={{.Values.zookeeper.name}}:{{.Values.zookeeper.port}}
 
 
 ###############################################################################
 ##     Items below are passed through to Kafka's producer and consumer
 ##     configurations (after removing "kafka.")
 ##     if you want to change request.required.acks it can take this one value
-#kafka.metadata.broker.list=localhost:9092,localhost:9093
-kafka.metadata.broker.list={{.Values.kafka.nameOverride}}:{{.Values.config.kafkaPort}}
-#kafka.metadata.broker.list=172.17.0.1:9092
-#dmaap.onap-message-router:9092
-#10.208.128.229:9092
+kafka.metadata.broker.list={{.Values.kafka.name}}:{{.Values.kafka.port}}
 ##kafka.request.required.acks=-1
 #kafka.client.zookeeper=${config.zk.servers}
 consumer.timeout.ms=100
 
+# Copyright © 2018  AT&T Intellectual Property.  All rights reserved.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#       http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
 1. Get the application URL by running these commands:
 {{- if .Values.ingress.enabled }}
 {{- range .Values.ingress.hosts }}
 
--- /dev/null
+# Copyright © 2017 Amdocs, Bell Canada
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#       http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+apiVersion: v1
+kind: ConfigMap
+metadata:
+  name: {{ include "common.fullname" . }}-msgrtrapi-prop-configmap
+  namespace: {{ include "common.namespace" . }}
+data:
+{{ tpl (.Files.Glob "resources/config/dmaap/MsgRtrApi.properties").AsConfig . | indent 2 }}
+---
+apiVersion: v1
+kind: ConfigMap
+metadata:
+  name: {{ include "common.fullname" . }}-cadi-prop-configmap
+  namespace: {{ include "common.namespace" . }}
+data:
+{{ tpl (.Files.Glob "resources/config/dmaap/cadi.properties").AsConfig . | indent 2 }}
 
         - /root/ready.py
         args:
         - --container-name
-        - {{ .Values.kafka.nameOverride }}
+        - {{ .Values.kafka.name }}
         - --container-name
-        - {{ .Values.zookeeper.nameOverride }}
+        - {{ .Values.zookeeper.name }}
         env:
         - name: NAMESPACE
           valueFrom:
 
 image: onap/dmaap/dmaap-mr:1.1.4
 pullPolicy: Always
 
-nameOverride: dmaap
-
 kafka:
-  nameOverride: global-kafka
+  name: message-router-kafka
+  port: 9092
 zookeeper:
-  nameOverride: zookeeper
+  name: message-router-zookeeper
+  port: 2181
 
 # flag to enable debugging - application support required
 debugEnabled: false
 
 # application configuration
-config:
-  zookeeperPort: 2181
-  kafkaPort: 9092
+config: {}
 
 # default number of instances
 replicaCount: 1
 
   readinessImage: readiness-check:2.0.0
   loggingRepository: docker.elastic.co
   loggingImage: beats/filebeat:5.5.0
-  postgresDatabase: dmaap
-  postgresUser: dmaap_admin
-  postgresPassword: onapdemodb
\ No newline at end of file
 
+++ /dev/null
-apiVersion: v1
-kind: ConfigMap
-metadata:
-  name: {{ include "common.fullname" . }}-msgrtrapi-prop-configmap
-  namespace: {{ include "common.namespace" . }}
-data:
-{{ tpl (.Files.Glob "resources/config/dmaap/MsgRtrApi.properties").AsConfig . | indent 2 }}
----
-apiVersion: v1
-kind: ConfigMap
-metadata:
-  name: {{ include "common.fullname" . }}-cadi-prop-configmap
-  namespace: {{ include "common.namespace" . }}
-data:
-{{ tpl (.Files.Glob "resources/config/dmaap/cadi.properties").AsConfig . | indent 2 }}
 
     version: ~2.0.0
     repository: '@local'
     condition: log.enabled
-  - name: message-router
-    version: ~2.0.0
-    repository: '@local'
-    condition: message-router.enabled
   - name: mock
     version: ~0.1.0
     repository: '@local'
 
   enabled: false
 dcaegen2:
   enabled: false
+dmaap:
+  enabled: false
 esr:
   enabled: false
 log:
   enabled: false
-message-router:
-  enabled: false
 mock:
   enabled: false
 oof:
 
   enabled: false
 dcaegen2:
   enabled: false
+dmaap:
+  enabled: false
 esr:
   enabled: false
 log:
   enabled: false
-message-router:
-  enabled: false
 mock:
   enabled: false
 msb:
 
   enabled: true
 dcaegen2:
   enabled: true
+dmaap:
+  enabled: true
 esr:
   enabled: true
 log: