calm-dsl icon indicating copy to clipboard operation
calm-dsl copied to clipboard

[Bug] ZoneInfo module has a dependency on tzdata module in Windows

Open nxchoople opened this issue 1 year ago • 1 comments

Describe the bug ZoneInfo uses the time zone data natively in Linux when selecting a time zone. When you use Windows, that capability isn't there and requires the tzdata module to function. If you don't install the module on Windows, then you will receive an error that ZoneInfo can't find the time zone key.

Branch/Container master

To Reproduce Create a job with the time zone defined on a Windows OS using Calm DSL.

Expected behavior The job should be created without any errors when specifying the time zone.

Findings/Troubleshooting Done ZoneInfo needs the tzdata module on Windows.

Screenshots image

** System Configuration:**

  • OS: Windows Server 2019
  • Python: 3.11

nxchoople avatar Jul 12 '23 20:07 nxchoople

We should include the tzdata module in the requirements.txt file so that it is available when running Calm DSL natively on Windows.

nxchoople avatar Jul 12 '23 20:07 nxchoople