[PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-20 Thread via GitHub
hussein-awala opened a new pull request, #38358: URL: https://github.com/apache/airflow/pull/38358 related: #38233 To avoid breaking the execution of created triggers after the upgrade to 2.9.0, we need to encrypt the `kwargs` during the migration process and decrypt them on downgrad

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-20 Thread via GitHub
hussein-awala commented on PR #38358: URL: https://github.com/apache/airflow/pull/38358#issuecomment-2011014885 Need to test it before marking it as ready for review. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-25 Thread via GitHub
ephraimbuddy commented on code in PR #38358: URL: https://github.com/apache/airflow/pull/38358#discussion_r1537428432 ## airflow/migrations/versions/0138_2_9_0_encrypt_trigger_kwargs.py: ## @@ -0,0 +1,69 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or m

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-28 Thread via GitHub
ephraimbuddy commented on code in PR #38358: URL: https://github.com/apache/airflow/pull/38358#discussion_r1542948639 ## airflow/migrations/versions/0138_2_9_0_encrypt_trigger_kwargs.py: ## @@ -0,0 +1,69 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or m

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-28 Thread via GitHub
ashb commented on code in PR #38358: URL: https://github.com/apache/airflow/pull/38358#discussion_r1543048146 ## airflow/migrations/versions/0138_2_9_0_encrypt_trigger_kwargs.py: ## @@ -0,0 +1,69 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more cont

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-28 Thread via GitHub
ashb commented on code in PR #38358: URL: https://github.com/apache/airflow/pull/38358#discussion_r1543048146 ## airflow/migrations/versions/0138_2_9_0_encrypt_trigger_kwargs.py: ## @@ -0,0 +1,69 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more cont

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-28 Thread via GitHub
utkarsharma2 commented on code in PR #38358: URL: https://github.com/apache/airflow/pull/38358#discussion_r1543573781 ## airflow/migrations/versions/0138_2_9_0_encrypt_trigger_kwargs.py: ## @@ -0,0 +1,69 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or m

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-28 Thread via GitHub
utkarsharma2 commented on code in PR #38358: URL: https://github.com/apache/airflow/pull/38358#discussion_r1543572691 ## airflow/migrations/versions/0138_2_9_0_encrypt_trigger_kwargs.py: ## @@ -0,0 +1,69 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or m

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-28 Thread via GitHub
utkarsharma2 commented on code in PR #38358: URL: https://github.com/apache/airflow/pull/38358#discussion_r1543575210 ## airflow/migrations/versions/0138_2_9_0_encrypt_trigger_kwargs.py: ## @@ -0,0 +1,69 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or m

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-28 Thread via GitHub
hussein-awala commented on code in PR #38358: URL: https://github.com/apache/airflow/pull/38358#discussion_r1543773967 ## airflow/migrations/versions/0138_2_9_0_encrypt_trigger_kwargs.py: ## @@ -0,0 +1,69 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-31 Thread via GitHub
ephraimbuddy commented on code in PR #38358: URL: https://github.com/apache/airflow/pull/38358#discussion_r1545723958 ## airflow/migrations/versions/0140_2_9_0_encrypt_trigger_kwargs.py: ## @@ -0,0 +1,70 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or m

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-31 Thread via GitHub
hussein-awala commented on code in PR #38358: URL: https://github.com/apache/airflow/pull/38358#discussion_r1545727063 ## airflow/migrations/versions/0140_2_9_0_encrypt_trigger_kwargs.py: ## @@ -0,0 +1,70 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-31 Thread via GitHub
hussein-awala commented on code in PR #38358: URL: https://github.com/apache/airflow/pull/38358#discussion_r1545828095 ## airflow/migrations/versions/0140_2_9_0_encrypt_trigger_kwargs.py: ## @@ -0,0 +1,70 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-31 Thread via GitHub
ephraimbuddy commented on code in PR #38358: URL: https://github.com/apache/airflow/pull/38358#discussion_r154585 ## airflow/migrations/versions/0140_2_9_0_update_trigger_kwargs_type.py: ## @@ -45,26 +45,10 @@ def get_session() -> sa.orm.Session: def upgrade(): -"""

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-31 Thread via GitHub
ephraimbuddy commented on code in PR #38358: URL: https://github.com/apache/airflow/pull/38358#discussion_r1545939853 ## airflow/migrations/versions/0140_2_9_0_update_trigger_kwargs_type.py: ## @@ -0,0 +1,56 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +#

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-04-01 Thread via GitHub
hussein-awala commented on code in PR #38358: URL: https://github.com/apache/airflow/pull/38358#discussion_r1546237393 ## airflow/migrations/versions/0140_2_9_0_update_trigger_kwargs_type.py: ## @@ -0,0 +1,56 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +#

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-04-01 Thread via GitHub
hussein-awala merged PR #38358: URL: https://github.com/apache/airflow/pull/38358 -- 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-unsubscr...@ai