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

   ### Apache Airflow version
   
   2.3.0 (latest released)
   
   ### What happened
   
   Templated strings in templates_dict, op_args, op_kwargs of 
PythonVirtualenvOperator are no longer rendered. 
   
   ### What you think should happen instead
   
   All templated strings in templates_dict, op_args and op_kwargs must be 
rendered, i.e. these 3 arguments must be template_fields of 
PythonVirtualenvOperator, as it was in Airflow 2.2.3
   
   ### How to reproduce
   
   _No response_
   
   ### Operating System
   
   Ubuntu 20.04
   
   ### Versions of Apache Airflow Providers
   
   _No response_
   
   ### Deployment
   
   Virtualenv installation
   
   ### Deployment details
   
   _No response_
   
   ### Anything else
   
   This is due to template_fields class variable being set in 
PythonVirtualenvOperator
   `template_fields: Sequence[str] = ('requirements',)`
   that overrode class variable of PythonOperator 
   `template_fields = ('templates_dict', 'op_args', 'op_kwargs')`.
   I read in some discussion that wanted to make requirements a template field 
for PythonVirtualenvOperator, but we must specify all template fields of parent 
class as well.
   `template_fields: Sequence[str] = ('templates_dict', 'op_args', 'op_kwargs', 
'requirements',)`
   
   ### Are you willing to submit PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of 
Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to