Skip to content

Commit

Permalink
fix(analytics-tab) - fix analytics tab config variable for gms (#5529)
Browse files Browse the repository at this point in the history
  • Loading branch information
salihcaan authored Aug 2, 2022
1 parent f6736bf commit 0ded025
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion docker/datahub-gms/env/docker.env
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ UI_INGESTION_DEFAULT_CLI_VERSION=0.8.41
# ELASTICSEARCH_PASSWORD=

# To disable Analytics on the UI
# ANALYTICS_ENABLED=false
# DATAHUB_ANALYTICS_ENABLED=false

# Encryption of DataHub Secrets
# SECRET_SERVICE_ENCRYPTION_KEY=<your-AES-encryption-key>
Expand Down
1 change: 1 addition & 0 deletions docs/how/updating-datahub.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ This file documents any backwards-incompatible changes in DataHub and assists pe
- #5451 `GMS_HOST` and `GMS_PORT` environment variables deprecated in `v0.8.39` have been removed. Use `DATAHUB_GMS_HOST` and `DATAHUB_GMS_PORT` instead.
- #5478 DataHub CLI `delete` command when used with `--hard` option will delete soft-deleted entities which match the other filters given.
- #5471 Looker now populates `userEmail` in dashboard user usage stats. This version of looker connnector will not work with older version of **datahub-gms** if you have `extract_usage_history` looker config enabled.
- #5529 - `ANALYTICS_ENABLED` environment variable in **datahub-gms** is now deprecated. Use `DATAHUB_ANALYTICS_ENABLED` instead.

### Potential Downtime

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ configEntityRegistry:
path: ${ENTITY_REGISTRY_CONFIG_PATH:../../metadata-models/src/main/resources/entity-registry.yml}

platformAnalytics:
enabled: ${ANALYTICS_ENABLED:true}
enabled: ${DATAHUB_ANALYTICS_ENABLED:true}

visualConfig:
assets:
Expand Down

0 comments on commit 0ded025

Please sign in to comment.