Paul Wyatt
02/11/2021, 2:55 AMOperation name: RunsRootQuery
Message: Exactly 5 or 6 columns has to be specified for iteratorexpression.
Path: ["repositoriesOrError","nodes",1,"schedules",1,"futureTicks"]
Locations: [{"line":183,"column":3}]
I'm hopeful that moving off the system cron scheduler will remediate, but any guidance is nonetheless helpfulprha
02/11/2021, 3:17 AMPaul Wyatt
02/11/2021, 6:12 PMdef basic_example_pipeline_schedule(preset: PresetDefinition) -> ScheduleDefinition:
"""Defines the schedule for the pipeline above."""
return ScheduleDefinition(
name="basic_example_pipeline_every_minute",
cron_schedule="*/5 * * * *",
pipeline_name="basic_example_pipeline",
mode=preset.mode,
run_config=preset.run_config,
environment_vars=get_environment_variables_for_scheduler()
)
preset
looks like
PresetDefinition(
"gcp_env_aware",
mode="gcp",
run_config={
"execution": {"multiprocess": {"config": {"max_concurrent": 4}}},
"resources": {
"warehouse": {"config": {"project": "virta-eng-prod"}},
"cloud_storage": {
"config": {
"bucket": os.getenv("DEFAULT_STORAGE_BUCKET"),
"project": "virta-eng-prod"
},
},
"io_manager": {"config": {"gcs_bucket": os.getenv("DEFAULT_STORAGE_BUCKET")}},
"pagerduty": {"config": {"routing_key": PAGERDUTY_KEY}},
},
},
)
prha
02/11/2021, 6:21 PMcron_schedule
argument that I see in your example… I suspect there’s a specific cron_schedule string that is generating the error. We do need to catch problematic cron strings earlier (at definition time?) and surface a better error message.Paul Wyatt
02/11/2021, 6:21 PMprha
02/11/2021, 6:25 PMcron_schedule
issue. With 0.10.0
we started using a cron-parsing library croniter
to project future schedule ticks (new 0.10.0
feature). The error that’s being thrown here is by that library. Previously, with the cron scheduler, I think we just copied the cron schedule to a system crontab, which might have a more permissive structure (e.g. accepts comments)?Paul Wyatt
02/11/2021, 6:26 PMprha
02/11/2021, 6:28 PM0.10.5
is being released today which will prevent the runs page from erroring out due to this issue (though the future schedule ticks view might still error).Paul Wyatt
02/11/2021, 6:38 PMprha
02/11/2021, 6:40 PMPaul Wyatt
02/11/2021, 6:42 PM