Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
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
Prev Previous commit
Next Next commit
Merge remote-tracking branch 'origin/master' into update-remote-dao
  • Loading branch information
Dexter Lee committed Jun 15, 2021
commit 7308613cb2ac30341bdbb8a9c2cd5ec32bb180f9
15 changes: 11 additions & 4 deletions .github/workflows/build-and-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,14 +65,21 @@ jobs:
- name: Gradle build
run: ./gradlew build -x check -x docs-website:build
- name: Smoke test
run: |
./docker/dev.sh -d
sleep 30
./smoke-test/smoke.sh
run: ./smoke-test/smoke.sh
- name: Slack failure notification
if: failure() && github.event_name == 'push'
uses: kpritam/slack-job-status-action@v1
with:
job-status: ${{ job.status }}
slack-bot-token: ${{ secrets.SLACK_BOT_TOKEN }}
channel: github-activities

quickstart-compose-validation:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/setup-python@v2
with:
python-version: "3.6"
- name: Quickstart Compose Validation
run: ./docker/quickstart/generate_and_compare.sh
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
11 changes: 11 additions & 0 deletions docker/docker-compose.override.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,17 @@ services:
- ./mysql/init.sql:/docker-entrypoint-initdb.d/init.sql
- mysqldata:/var/lib/mysql

mysql-setup:
build:
context: ../
dockerfile: docker/mysql-setup/Dockerfile
image: acryldata/datahub-mysql-setup:head
env_file: mysql-setup/env/docker.env
hostname: mysql-setup
container_name: mysql-setup
depends_on:
- mysql

datahub-gms:
env_file: datahub-gms/env/docker.env
depends_on:
Expand Down
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
2 changes: 1 addition & 1 deletion docker/elasticsearch/env/docker.env
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
discovery.type=single-node
xpack.security.enabled=false
ES_JAVA_OPTS=-Xms1g -Xmx1g
ES_JAVA_OPTS=-Xms512m -Xmx512m
5 changes: 5 additions & 0 deletions docker/mysql-setup/env/docker.env
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
MYSQL_HOST=mysql
MYSQL_PORT=3306
MYSQL_USERNAME=datahub
MYSQL_PASSWORD=datahub
DATAHUB_DB_NAME=datahub
Loading
You are viewing a condensed version of this merge commit. You can view the full changes here.