Page MenuHomePhabricator

Fix python3 bug with cron schedule script
ClosedPublic

Authored by sashank on Fri, Aug 23, 12:21 AM.

Details

Reviewers
natekupp
Group Reviewers
Restricted Project
Commits
R1:1315ff32e2b7: Fix python3 bug with cron schedule script
Summary

Previously, the dagster-graphql path was resolved by simply replacing /bin/python with /bin/dagster-graphql. However, when the executable is named /bin/python3, the dagster-graphql path was incorrectly resolved as /bin/dagster-graphql3. This diff uses os.path.dirname to support any executable name for python

Test Plan

unit

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

sashank updated this revision to Diff 3967.Fri, Aug 23, 12:21 AM
sashank created this revision.

typo

Harbormaster completed remote builds in B3168: Diff 3967.
natekupp accepted this revision.Fri, Aug 23, 12:39 AM
This revision is now accepted and ready to land.Fri, Aug 23, 12:39 AM