Visitar URL original
Fix example timetable for AIP-39 howto by uranusjr · Pull Request #18475 · apache/airflow · GitHub
Skip to content

Fix example timetable for AIP-39 howto - #18475

Merged
kaxil merged 3 commits into
apache:mainfrom
uranusjr:aip-39-example-timetable-schedule-fix
Sep 23, 2021
Merged

kaxil merged 3 commits into
apache:mainfrom
uranusjr:aip-39-example-timetable-schedule-fix

Conversation

@uranusjr

@uranusjr uranusjr commented Sep 23, 2021 •

Copy link
Copy Markdown
Member

The wonderful QA team caught me doing stupid things. This fixes that. Plus:

  • Adding dag_id and start_date to the example DAG. This was also suggested by the QA team.
  • A bunch of tests making sure the example timetable is providing a sane schedule so I have less chances getting more mistakes merged 😛

@uranusjr uranusjr added the AIP-39 Timetables label Sep 23, 2021
@uranusjr uranusjr added this to the Airflow 2.2.0 milestone Sep 23, 2021
@uranusjr
uranusjr requested a review from kaxil as a code owner September 23, 2021 15:26
Comment thread airflow/example_dags/plugins/workday.py Outdated

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤦

@uranusjr
uranusjr force-pushed the aip-39-example-timetable-schedule-fix branch from fb19320 to 5eedd50 Compare September 23, 2021 15:30
@uranusjr uranusjr changed the title Add arbitrary start_date so DAG in timetable howto Fix example timetable for AIP-39 howto Sep 23, 2021
Comment thread docs/apache-airflow/howto/timetable.rst Resolved
Comment thread tests/timetatbles/__init__.py Resolved
@uranusjr
uranusjr force-pushed the aip-39-example-timetable-schedule-fix branch from 5eedd50 to 00ffe5e Compare September 23, 2021 18:14
@uranusjr
uranusjr requested a review from ashb September 23, 2021 18:15
@github-actions github-actions Bot added the full tests needed We need to run full set of tests for this PR to merge label Sep 23, 2021
@github-actions

Copy link
Copy Markdown
Contributor

The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest main at your convenience, or amend the last commit of the PR, and push it with --force-with-lease.

@kaxil
kaxil merged commit 7006068 into apache:main Sep 23, 2021
@uranusjr
uranusjr deleted the aip-39-example-timetable-schedule-fix branch September 23, 2021 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AIP-39 Timetables full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants