Skip to content

Commit

Permalink
chore: avoid setting environment variables except for local cases
Browse files Browse the repository at this point in the history
  • Loading branch information
cpcloud committed Dec 31, 2024
1 parent 66b3ed2 commit 2c55209
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 5 deletions.
2 changes: 2 additions & 0 deletions .env
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,5 @@ MYSQL_PWD="ibis"
MSSQL_SA_PASSWORD="1bis_Testing!"
DRUID_URL="druid://localhost:8082/druid/v2/sql"
SPARK_CONFIG=./docker/spark-connect/conf.properties
AWS_PROFILE="ibis-testing"
AWS_REGION="us-east-2"
1 change: 0 additions & 1 deletion flake.nix
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,6 @@
inherit shellHook;

PYSPARK_PYTHON = "${env}/bin/python";
AWS_PROFILE = "ibis-testing";

# needed for mssql+pyodbc
ODBCSYSINI = pkgs.writeTextDir "odbcinst.ini" ''
Expand Down
8 changes: 4 additions & 4 deletions ibis/backends/athena/tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@


IBIS_ATHENA_S3_STAGING_DIR = env.get("IBIS_ATHENA_S3_STAGING_DIR", "s3://ibis-testing/")
IBIS_ATHENA_REGION_NAME = env.get("IBIS_ATHENA_REGION_NAME", "us-east-2")
IBIS_ATHENA_PROFILE_NAME = env.get("IBIS_ATHENA_PROFILE_NAME", "ibis-testing")
AWS_REGION = env.get("AWS_REGION", "us-east-2")
AWS_PROFILE = env.get("AWS_PROFILE")
CONNECT_ARGS = dict(
s3_staging_dir=IBIS_ATHENA_S3_STAGING_DIR,
region_name=IBIS_ATHENA_REGION_NAME,
profile_name=IBIS_ATHENA_PROFILE_NAME,
region_name=AWS_REGION,
profile_name=AWS_PROFILE,
)


Expand Down

0 comments on commit 2c55209

Please sign in to comment.