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 2022/12/09 20:09:09 UTC

[GitHub] [airflow] mhenc opened a new issue, #28271: AIP-44 Migrate Variable to Internal API

mhenc opened a new issue, #28271:
URL: https://github.com/apache/airflow/issues/28271

   Link: https://github.com/apache/airflow/blob/main/airflow/models/variable.py
   
   Methods to migrate:
   
   - val
   - set 
   - delete
   - update
   
   Note that get_variable_from_secrets shouls still be executed locally.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] bugraoz93 commented on issue #28271: AIP-44 Migrate Variable to Internal API

Posted by GitBox <gi...@apache.org>.
bugraoz93 commented on issue #28271:
URL: https://github.com/apache/airflow/issues/28271#issuecomment-1377463324

   This story is in progress. I cannot change the Issue Status. FYI


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] bugraoz93 commented on issue #28271: AIP-44 Migrate Variable to Internal API

Posted by GitBox <gi...@apache.org>.
bugraoz93 commented on issue #28271:
URL: https://github.com/apache/airflow/issues/28271#issuecomment-1369547305

   Great information, thanks! 
   Sure, I am writing a comment now.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] mhenc commented on issue #28271: AIP-44 Migrate Variable to Internal API

Posted by GitBox <gi...@apache.org>.
mhenc commented on issue #28271:
URL: https://github.com/apache/airflow/issues/28271#issuecomment-1369543388

   Thanks!
   Just a note - you can work on a PR even now. 
   Just remember to start InternalAPI (as part of Web Server) with `[core]database_access_isolation=False` (while running e.g. Worker/Dag processor with` [core]database_access_isolation=True`.
   
   Can you also comment on https://github.com/apache/airflow/issues/28612 so I can assign you there?


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] bugraoz93 commented on issue #28271: AIP-44 Migrate Variable to Internal API

Posted by "bugraoz93 (via GitHub)" <gi...@apache.org>.
bugraoz93 commented on issue #28271:
URL: https://github.com/apache/airflow/issues/28271#issuecomment-1399243420

   Thanks a lot!


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] potiuk commented on issue #28271: AIP-44 Migrate Variable to Internal API

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on issue #28271:
URL: https://github.com/apache/airflow/issues/28271#issuecomment-1398983633

   Changed.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] bugraoz93 commented on issue #28271: AIP-44 Migrate Variable to Internal API

Posted by GitBox <gi...@apache.org>.
bugraoz93 commented on issue #28271:
URL: https://github.com/apache/airflow/issues/28271#issuecomment-1369245755

   Hey! 
   I would like to work on this issue. Of course, after closing #28267 as you suggested. 
   Meanwhile, I would also like to work on #28612 if there is no dependency.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] potiuk closed issue #28271: AIP-44 Migrate Variable to Internal API

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk closed issue #28271: AIP-44 Migrate Variable to Internal API
URL: https://github.com/apache/airflow/issues/28271


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org