Skip to content

Commit

Permalink
fix(cli): Make datahub quickstart work with latest docker compose in …
Browse files Browse the repository at this point in the history
…M1 (datahub-project#6891)

Co-authored-by: Harshal Sheth <[email protected]>
  • Loading branch information
2 people authored and cccs-Dustin committed Feb 1, 2023
1 parent 41b65bc commit 591a79e
Showing 1 changed file with 2 additions and 9 deletions.
11 changes: 2 additions & 9 deletions metadata-ingestion/src/datahub/cli/docker_cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -71,15 +71,8 @@ class Architectures(Enum):

@functools.lru_cache()
def _docker_subprocess_env() -> Dict[str, str]:
try:
DOCKER_COMPOSE_PLATFORM: Optional[str] = (
subprocess.run(["uname", "-m"], stdout=subprocess.PIPE)
.stdout.decode("utf-8")
.rstrip()
)
except FileNotFoundError:
# On Windows, uname is not available.
DOCKER_COMPOSE_PLATFORM = None
# platform.machine() is equivalent to `uname -m`, as per https://stackoverflow.com/a/45124927/5004662
DOCKER_COMPOSE_PLATFORM: str = "linux/" + platform.machine()

env = {
**os.environ,
Expand Down

0 comments on commit 591a79e

Please sign in to comment.