jens-scheffler-bosch commented on PR #32858: URL: https://github.com/apache/airflow/pull/32858#issuecomment-1657268320
> > I just made a dry run code-reading (yet) would take 1-2 days until I'd be able to do a full test. > > Besides one comment where I expect that a default matching to todays hard coded values is expected, I miss any kind of documentation. We don't want the users having to read the code to discover all secret options. Can you add some? > > Hi, @jens-scheffler-bosch , > > the document has been added in previous pull request https://github.com/apache/airflow/pull/26766/files at > > https://github.com/apache/airflow/blob/89404530aa02d49ffe724a3ab653d5c9d687dd00/docs/apache-airflow-providers-cncf-kubernetes/connections/kubernetes.rst?plain=1#L70 Ah, I see. Was not aware of this "legacy". Still for a user hard to find so not knowing about this feature maybe you rather stumble over it by crawling the source. Nevertheless adding the resource documentation and an example (JSON might not be obvious for all) would still be good. -- 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]
