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 (#6891)

Co-authored-by: Harshal Sheth <hsheth2@gmail.com>
  • Loading branch information
pedro93 and hsheth2 authored Dec 30, 2022
1 parent e81a3ad commit 594fc1b
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 594fc1b

Please sign in to comment.