Skip to content

Conversation

@david-leifker
Copy link
Collaborator

@david-leifker david-leifker commented Oct 29, 2025

Summary

Updates for IAM Support

Changes Made

Fixes

  • ✅ MCE Consumer should share cloud detection logic with GMS/SystemUpdate

SQL Setup Configuration

  • ✅ Removed incorrect IAM_ROLE environment variable - IAM authentication is now auto-detected
  • ✅ Added DATAHUB_SQL_SETUP_ENABLED - enables SQL setup functionality without CLI arguments
  • ✅ Updated IAM authentication documentation with correct auto-detection logic

Elasticsearch User Create

  • ✅ Optionally create an Elasticsearch user or role mapping and policy

Cloud Provider Configuration

  • ✅ Added AWS_SESSION_TOKEN - for temporary AWS credentials
  • ✅ Added GCP_PROJECT - for GCP project auto-detection

Impact

Documents all environment variables used by datahub-upgrade sql-setup and elasticsearch upgrade steps, correcting IAM authentication behavior.

Important Notes

Primarily tested RDS MySQL + AWS IAM

@github-actions github-actions bot added docs Issues and Improvements to docs devops PR or Issue related to DataHub backend & deployment labels Oct 29, 2025
@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label Oct 29, 2025
@codecov
Copy link

codecov bot commented Oct 29, 2025

Bundle Report

Bundle size has no change ✅

@datahub-cyborg datahub-cyborg bot added pending-submitter-merge and removed needs-review Label for PRs that need review from a maintainer. labels Oct 29, 2025
@david-leifker david-leifker merged commit b390dd4 into master Oct 29, 2025
40 checks passed
@david-leifker david-leifker deleted the feat-iam-setup branch October 29, 2025 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops PR or Issue related to DataHub backend & deployment docs Issues and Improvements to docs pending-submitter-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants