Skip to content

Commit

Permalink
fix: convert tz
Browse files Browse the repository at this point in the history
  • Loading branch information
cbini committed Jan 11, 2024
1 parent 3b97bd4 commit 4f6053f
Show file tree
Hide file tree
Showing 6 changed files with 20 additions and 9 deletions.
6 changes: 4 additions & 2 deletions src/teamster/core/powerschool/schedules.py
Original file line number Diff line number Diff line change
Expand Up @@ -74,8 +74,10 @@ def _schedule(
"partition_column"
]

latest_materialization_fmt = latest_materialization_datetime.format(
"YYYY-MM-DDTHH:mm:ss.SSSSSS"
latest_materialization_fmt = (
latest_materialization_datetime.in_timezone(
tz=execution_timezone
).format("YYYY-MM-DDTHH:mm:ss.SSSSSS")
)

[(count,)] = db_powerschool.engine.execute_query(
Expand Down
11 changes: 8 additions & 3 deletions src/teamster/core/powerschool/sensors.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,10 @@


def build_powerschool_sensor(
name, asset_defs: list[AssetsDefinition], minimum_interval_seconds=None
name,
asset_defs: list[AssetsDefinition],
execution_timezone,
minimum_interval_seconds=None,
):
@sensor(
name=name,
Expand Down Expand Up @@ -61,8 +64,10 @@ def _sensor(
) # type: ignore
)

latest_materialization_fmt = latest_materialization_datetime.format(
"YYYY-MM-DDTHH:mm:ss.SSSSSS"
latest_materialization_fmt = (
latest_materialization_datetime.in_timezone(
tz=execution_timezone
).format("YYYY-MM-DDTHH:mm:ss.SSSSSS")
)

[(count,)] = db_powerschool.engine.execute_query(
Expand Down
3 changes: 2 additions & 1 deletion src/teamster/kippcamden/powerschool/sensors.py
Original file line number Diff line number Diff line change
@@ -1,11 +1,12 @@
from teamster.core.powerschool.sensors import build_powerschool_sensor

from .. import CODE_LOCATION
from .. import CODE_LOCATION, LOCAL_TIMEZONE
from .assets import partition_assets

partition_sensor = build_powerschool_sensor(
name=f"{CODE_LOCATION}_powerschool_partition_sensor",
asset_defs=partition_assets,
execution_timezone=LOCAL_TIMEZONE,
minimum_interval_seconds=(60 * 10),
)

Expand Down
3 changes: 2 additions & 1 deletion src/teamster/kippmiami/powerschool/sensors.py
Original file line number Diff line number Diff line change
@@ -1,11 +1,12 @@
from teamster.core.powerschool.sensors import build_powerschool_sensor

from .. import CODE_LOCATION
from .. import CODE_LOCATION, LOCAL_TIMEZONE
from .assets import partition_assets

partition_sensor = build_powerschool_sensor(
name=f"{CODE_LOCATION}_powerschool_partition_sensor",
asset_defs=partition_assets,
execution_timezone=LOCAL_TIMEZONE,
minimum_interval_seconds=(60 * 10),
)

Expand Down
3 changes: 2 additions & 1 deletion src/teamster/kippnewark/powerschool/sensors.py
Original file line number Diff line number Diff line change
@@ -1,11 +1,12 @@
from teamster.core.powerschool.sensors import build_powerschool_sensor

from .. import CODE_LOCATION
from .. import CODE_LOCATION, LOCAL_TIMEZONE
from .assets import partition_assets

partition_sensor = build_powerschool_sensor(
name=f"{CODE_LOCATION}_powerschool_partition_sensor",
asset_defs=partition_assets,
execution_timezone=LOCAL_TIMEZONE,
minimum_interval_seconds=(60 * 10),
)

Expand Down
3 changes: 2 additions & 1 deletion tests/sensors/test_powerschool_sensor.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
)
from teamster.core.utils.functions import get_dagster_cloud_instance
from teamster.kippnewark.powerschool.assets import partition_assets
from teamster.staging import LOCAL_TIMEZONE


def test_powerschool_sensor():
Expand All @@ -15,7 +16,7 @@ def test_powerschool_sensor():
)

dynamic_partition_sensor = build_powerschool_sensor(
name="test", asset_defs=partition_assets
name="test", asset_defs=partition_assets, execution_timezone=LOCAL_TIMEZONE
)

sensor_results = dynamic_partition_sensor(
Expand Down

0 comments on commit 4f6053f

Please sign in to comment.