ferruzzi commented on a change in pull request #20819:
URL: https://github.com/apache/airflow/pull/20819#discussion_r782696197
##########
File path: airflow/providers/amazon/aws/operators/eks.py
##########
@@ -89,36 +91,45 @@ class EksCreateClusterOperator(BaseOperator):
If this is None or empty then the default boto3 behaviour is used.
:type region: str
- If compute is assigned the value of 'nodegroup', the following are
required:
+ If compute is assigned the value of 'nodegroup':
Review comment:
We've now added some that are available but non-required. The ones that
were there are required if you define compute='nodegroup' but
`create_nodegroup_kwargs` is optional with that set. Can you suggest a concise
way to rephrase it that still makes sense? The ideas I came up with were long
and/or awkward, and this sounded better.
--
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]
