-
Notifications
You must be signed in to change notification settings - Fork 13.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create StatNameHandlerNotSupportedRule to ease upgrade to Airflow 2.0 #11045
Comments
Hi, I would like to take this task can you assign me? @potiuk @mik-laj @turbaszek |
I will like take up this task. |
There's already PR fort this #11223, I rebased it and will merge once tests pass. |
The PR is stale. |
I am currently neck-deep in some k8s issues but I will be happy to jump in on this once I have more wiggle room. |
Closing as there will be no more releases of upgrade check |
This issue is part of #8765
Rule
Create
StatNameHandlerNotSupportedRule
which corresponds toentry in UPDATING.md. This rule should allow users to check if their current configuration needs any adjusting
before migration to Airflow 2.0.
How to guide
To implement a new rule, create a class that inherits from
airflow.upgrade.rules.base_rule.BaseRule
.It will be auto-registered and used by
airflow upgrade-check
command. The custom rule class has to havetitle
,description
properties and should implementcheck
method which returns a list of error messages in case ofincompatibility.
For example:
airflow/airflow/upgrade/rules/conn_type_is_not_nullable.py
Lines 25 to 42 in ea36166
Remember to open the PR against
v1-10-test
branch.The text was updated successfully, but these errors were encountered: