You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/06/01 04:46:44 UTC
[GitHub] [airflow] jhtimmins opened a new pull request #16190: call resource based fab methods.
jhtimmins opened a new pull request #16190:
URL: https://github.com/apache/airflow/pull/16190
The previous PR (#16077) added wrappers that use the updated resource/action names around the default FAB methods. This PR updates `security.py` to use those new methods.
Successor to #16077 and the next step in #15398.
This PR can be merged whenever it passes.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] jhtimmins commented on pull request #16190: call resource based fab methods.
Posted by GitBox <gi...@apache.org>.
jhtimmins commented on pull request #16190:
URL: https://github.com/apache/airflow/pull/16190#issuecomment-851800644
@ashb @jedcunningham @kaxil If one of y'all can take a look at this. This can be merged whenever it passes.
Thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #16190: call resource based fab methods.
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #16190:
URL: https://github.com/apache/airflow/pull/16190#issuecomment-852186935
The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest main or amend the last commit of the PR, and push it with --force-with-lease.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] jhtimmins merged pull request #16190: call resource based fab methods.
Posted by GitBox <gi...@apache.org>.
jhtimmins merged pull request #16190:
URL: https://github.com/apache/airflow/pull/16190
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org