The classes are always available (even when the rbac is not enabled) so we 
should be able to create migrations for them somehow.

Edit: thinking about it a bit more I'm not sure it makes sense to create 
Alembic migrations for the Flask RBAC models: if we import the model classes in 
the migration then the migrations can change over time (we got stung with this 
already once on our _own_ migrations), or we have to duplicate the model 
classes in the migration file, which means we have to manually keep them in 
sync/notice changes.

[ Full content available at: 
https://github.com/apache/incubator-airflow/pull/3796 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to