Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(gms): Merge MAE, MCE consumers into GMS #2690

Merged
merged 19 commits into from
Jun 15, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions datahub-kubernetes/datahub/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,11 @@ dependencies:
- name: datahub-mae-consumer
version: 0.2.1
repository: file://./charts/datahub-mae-consumer
condition: datahub-mae-consumer.enabled
condition: global.datahub_standalone_consumers_enabled
- name: datahub-mce-consumer
version: 0.2.1
repository: file://./charts/datahub-mce-consumer
condition: datahub-mce-consumer.enabled
condition: global.datahub_standalone_consumers_enabled
- name: datahub-ingestion-cron
version: 0.2.1
repository: file://./charts/datahub-ingestion-cron
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,14 @@ spec:
periodSeconds: {{ .Values.readinessProbe.periodSeconds }}
failureThreshold: {{ .Values.readinessProbe.failureThreshold }}
env:
{{- if not .Values.global.datahub_standalone_consumers_enabled }}
- name: MCE_CONSUMER_ENABLED
value: "true"
- name: MAE_CONSUMER_ENABLED
value: "true"
{{- end }}
- name: DATAHUB_ANALYTICS_ENABLED
value: "{{ .Values.global.datahub_analytics_enabled }}"
- name: EBEAN_DATASOURCE_USERNAME
value: "{{ .Values.global.sql.datasource.username }}"
- name: EBEAN_DATASOURCE_PASSWORD
Expand Down
2 changes: 2 additions & 0 deletions datahub-kubernetes/datahub/charts/datahub-gms/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -146,6 +146,8 @@ readinessProbe:
#This section is useful if we are installing this chart separately for testing
# helm install datahub-gms datahub-gms/
global:
datahub_analytics_enabled: true

elasticsearch:
host: "elasticsearch"
port: "9200"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,8 @@ spec:
periodSeconds: {{ .Values.readinessProbe.periodSeconds }}
failureThreshold: {{ .Values.readinessProbe.failureThreshold }}
env:
- name: MAE_CONSUMER_ENABLED
value: "true"
- name: GMS_HOST
value: {{ printf "%s-%s" .Release.Name "datahub-gms" }}
- name: GMS_PORT
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,8 @@ spec:
periodSeconds: {{ .Values.readinessProbe.periodSeconds }}
failureThreshold: {{ .Values.readinessProbe.failureThreshold }}
env:
- name: MCE_CONSUMER_ENABLED
value: "true"
- name: KAFKA_BOOTSTRAP_SERVER
value: "{{ .Values.global.kafka.bootstrap.server }}"
- name: KAFKA_SCHEMAREGISTRY_URL
Expand Down
12 changes: 0 additions & 12 deletions datahub-kubernetes/datahub/quickstart-values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,18 +15,6 @@ datahub-frontend:
ingress:
enabled: false

datahub-mae-consumer:
enabled: true
image:
repository: linkedin/datahub-mae-consumer
tag: "v0.8.1"

datahub-mce-consumer:
enabled: true
image:
repository: linkedin/datahub-mce-consumer
tag: "v0.8.1"

elasticsearchSetupJob:
enabled: true
image:
Expand Down
2 changes: 2 additions & 0 deletions datahub-kubernetes/datahub/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,9 @@ datahubUpgrade:
tag: "v0.8.1"

global:

datahub_analytics_enabled: true
datahub_standalone_consumers_enabled: false

elasticsearch:
host: "elasticsearch-master"
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,6 @@ private List<UpgradeStep> buildUpgradeSteps(
final List<UpgradeStep> steps = new ArrayList<>();
steps.add(new RemoveAspectV2TableStep(server));
steps.add(new GMSQualificationStep());
steps.add(new MAEQualificationStep());
steps.add(new UpgradeQualificationStep(server));
steps.add(new CreateAspectTableStep(server));
steps.add(new IngestDataPlatformsStep(entityService));
Expand Down
6 changes: 6 additions & 0 deletions docker/datahub-gms/env/docker.env
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,12 @@ NEO4J_URI=bolt://neo4j
NEO4J_USERNAME=neo4j
NEO4J_PASSWORD=datahub

MAE_CONSUMER_ENABLED=true
MCE_CONSUMER_ENABLED=true

# Uncomment to disable persistence of client-side analytics events
# DATAHUB_ANALYTICS_ENABLED=false

# Uncomment to configure kafka topic names
# Make sure these names are consistent across the whole deployment
# METADATA_AUDIT_EVENT_NAME=MetadataAuditEvent_v4
Expand Down
2 changes: 2 additions & 0 deletions docker/datahub-gms/env/docker.mariadb.env
Original file line number Diff line number Diff line change
Expand Up @@ -11,3 +11,5 @@ NEO4J_HOST=http://neo4j:7474
NEO4J_URI=bolt://neo4j
NEO4J_USERNAME=neo4j
NEO4J_PASSWORD=datahub
MAE_CONSUMER_ENABLED=true
MCE_CONSUMER_ENABLED=true
3 changes: 3 additions & 0 deletions docker/datahub-gms/env/docker.postgres.env
Original file line number Diff line number Diff line change
Expand Up @@ -11,3 +11,6 @@ NEO4J_HOST=http://neo4j:7474
NEO4J_URI=bolt://neo4j
NEO4J_USERNAME=neo4j
NEO4J_PASSWORD=datahub
MAE_CONSUMER_ENABLED=true
MCE_CONSUMER_ENABLED=true

1 change: 1 addition & 0 deletions docker/datahub-mae-consumer/env/docker.env
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
MAE_CONSUMER_ENABLED=true
KAFKA_BOOTSTRAP_SERVER=broker:29092
KAFKA_SCHEMAREGISTRY_URL=http://schema-registry:8081
ELASTICSEARCH_HOST=elasticsearch
Expand Down
1 change: 1 addition & 0 deletions docker/datahub-mce-consumer/env/docker.env
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
MCE_CONSUMER_ENABLED=true
KAFKA_BOOTSTRAP_SERVER=broker:29092
KAFKA_SCHEMAREGISTRY_URL=http://schema-registry:8081
GMS_HOST=datahub-gms
Expand Down
23 changes: 23 additions & 0 deletions docker/docker-compose.consumers.dev.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
version: '3.8'
services:
datahub-mae-consumer:
image: linkedin/datahub-mae-consumer:debug
build:
context: datahub-mae-consumer
dockerfile: Dockerfile
args:
APP_ENV: dev
volumes:
- ./datahub-mae-consumer/start.sh:/datahub/datahub-mae-consumer/scripts/start.sh
- ../metadata-jobs/mae-consumer-job/build/libs/:/datahub/datahub-mae-consumer/bin/

datahub-mce-consumer:
image: linkedin/datahub-mce-consumer:debug
build:
context: datahub-mce-consumer
dockerfile: Dockerfile
args:
APP_ENV: dev
volumes:
- ./datahub-mce-consumer/start.sh:/datahub/datahub-mce-consumer/scripts/start.sh
- ../metadata-jobs/mce-consumer-job/build/libs/:/datahub/datahub-mce-consumer/bin
31 changes: 31 additions & 0 deletions docker/docker-compose.consumers.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
# Service definitions for standalone Kafka consumer containers.
version: '3.8'
services:
datahub-mae-consumer:
build:
context: ../
dockerfile: docker/datahub-mae-consumer/Dockerfile
image: linkedin/datahub-mae-consumer:${DATAHUB_VERSION:-latest}
env_file: datahub-mae-consumer/env/docker.env
hostname: datahub-mae-consumer
container_name: datahub-mae-consumer
ports:
- "9091:9091"
depends_on:
- kafka-setup
- elasticsearch-setup
- neo4j

datahub-mce-consumer:
build:
context: ../
dockerfile: docker/datahub-mce-consumer/Dockerfile
image: linkedin/datahub-mce-consumer:${DATAHUB_VERSION:-latest}
env_file: datahub-mce-consumer/env/docker.env
hostname: datahub-mce-consumer
container_name: datahub-mce-consumer
ports:
- "9090:9090"
depends_on:
- kafka-setup
- datahub-gms
22 changes: 0 additions & 22 deletions docker/docker-compose.dev.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,25 +50,3 @@ services:
APP_ENV: dev
volumes:
- ../datahub-frontend/build/stage/datahub-frontend:/datahub-frontend

datahub-mae-consumer:
image: linkedin/datahub-mae-consumer:debug
build:
context: datahub-mae-consumer
dockerfile: Dockerfile
args:
APP_ENV: dev
volumes:
- ./datahub-mae-consumer/start.sh:/datahub/datahub-mae-consumer/scripts/start.sh
- ../metadata-jobs/mae-consumer-job/build/libs/:/datahub/datahub-mae-consumer/bin/

datahub-mce-consumer:
image: linkedin/datahub-mce-consumer:debug
build:
context: datahub-mce-consumer
dockerfile: Dockerfile
args:
APP_ENV: dev
volumes:
- ./datahub-mce-consumer/start.sh:/datahub/datahub-mce-consumer/scripts/start.sh
- ../metadata-jobs/mce-consumer-job/build/libs/:/datahub/datahub-mce-consumer/bin
29 changes: 0 additions & 29 deletions docker/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -160,35 +160,6 @@ services:
depends_on:
- datahub-gms

datahub-mae-consumer:
build:
context: ../
dockerfile: docker/datahub-mae-consumer/Dockerfile
image: linkedin/datahub-mae-consumer:${DATAHUB_VERSION:-latest}
env_file: datahub-mae-consumer/env/docker.env
hostname: datahub-mae-consumer
container_name: datahub-mae-consumer
ports:
- "9091:9091"
depends_on:
- kafka-setup
- elasticsearch-setup
- neo4j

datahub-mce-consumer:
build:
context: ../
dockerfile: docker/datahub-mce-consumer/Dockerfile
image: linkedin/datahub-mce-consumer:${DATAHUB_VERSION:-latest}
env_file: datahub-mce-consumer/env/docker.env
hostname: datahub-mce-consumer
container_name: datahub-mce-consumer
ports:
- "9090:9090"
depends_on:
- kafka-setup
- datahub-gms

networks:
default:
name: datahub_network
Expand Down
39 changes: 2 additions & 37 deletions docker/quickstart/docker-compose.quickstart.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,48 +55,13 @@ services:
- NEO4J_URI=bolt://neo4j
- NEO4J_USERNAME=neo4j
- NEO4J_PASSWORD=datahub
- MAE_CONSUMER_ENABLED=true
- MCE_CONSUMER_ENABLED=true
hostname: datahub-gms
image: linkedin/datahub-gms:${DATAHUB_VERSION:-latest}
mem_limit: 850m
ports:
- 8080:8080
datahub-mae-consumer:
container_name: datahub-mae-consumer
depends_on:
- kafka-setup
- elasticsearch-setup
- neo4j
environment:
- KAFKA_BOOTSTRAP_SERVER=broker:29092
- KAFKA_SCHEMAREGISTRY_URL=http://schema-registry:8081
- ELASTICSEARCH_HOST=elasticsearch
- ELASTICSEARCH_PORT=9200
- NEO4J_HOST=http://neo4j:7474
- NEO4J_URI=bolt://neo4j
- NEO4J_USERNAME=neo4j
- NEO4J_PASSWORD=datahub
- GMS_HOST=datahub-gms
- GMS_PORT=8080
hostname: datahub-mae-consumer
image: linkedin/datahub-mae-consumer:${DATAHUB_VERSION:-latest}
mem_limit: 256m
ports:
- 9091:9091
datahub-mce-consumer:
container_name: datahub-mce-consumer
depends_on:
- kafka-setup
- datahub-gms
environment:
- KAFKA_BOOTSTRAP_SERVER=broker:29092
- KAFKA_SCHEMAREGISTRY_URL=http://schema-registry:8081
- GMS_HOST=datahub-gms
- GMS_PORT=8080
hostname: datahub-mce-consumer
image: linkedin/datahub-mce-consumer:${DATAHUB_VERSION:-latest}
mem_limit: 384m
ports:
- 9090:9090
elasticsearch:
container_name: elasticsearch
environment:
Expand Down
2 changes: 2 additions & 0 deletions gms/war/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@ ext.apiProject = project(':gms:api')

dependencies {
runtime project(':gms:factories')
runtime project(':metadata-jobs:mce-consumer')
runtime project(':metadata-jobs:mae-consumer')

runtime externalDependency.h2
runtime externalDependency.logbackClassic
Expand Down
Loading