ferruzzi edited a comment on issue #19641:
URL: https://github.com/apache/airflow/issues/19641#issuecomment-978658741


   How strongly do you feel about that?  Leaving them explicitly called out 
like that makes it obvious they are mandatory and enforces that, which (maybe?) 
makes it easier for new users to get started.  I kind of like that.  Would it 
be a deal-breaker this way?
   
   I started implementing it and realized that there is an issue with the 
current version of the Operator in the main branch; going to have to sort that 
out first.  Which version of airflow/operators are you using?  That should help 
me sort out what/where it went wrong.   With the current main branch, I can 
create a cluster, but creating a nodegroup fails to attach to the cluster.  I 
presume you aren't seeing that on whichever version you are on?


-- 
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...@airflow.apache.org

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


Reply via email to