Nike-Inc / brickflow

Pythonic Programming Framework to orchestrate jobs in Databricks Workflow
https://engineering.nike.com/brickflow/
Apache License 2.0
183 stars 36 forks source link

[BUG] Bug in the get_arch() function. It does not catch aarch64 as arm64 when creating the download link for the databricks CLI. #94

Open SirMiAss opened 6 months ago

SirMiAss commented 6 months ago

Describe the bug When installing brickflow on a arm64 linux machine the get_arch() used in the bundle_download_path() returns the wrong architecture string to build our download link. It returns https://github.com/databricks/cli/releases/download/v0.210.2/databricks_cli_0.210.2_linux_aarch64.zip while it should be https://github.com/databricks/cli/releases/download/v0.210.2/databricks_cli_0.210.2_linux_arm64.zip

To Reproduce Steps to reproduce the behavior:

  1. Use a linux-arm64 environment.
  2. Install brickflow

Expected behavior get_arch() should catch aarch64 and return arm64.

Desktop (please complete the following information):